Skip to content

Replace settings commit suppression flags with explicit origins - #161

Merged
KeyffMS merged 11 commits into
mainfrom
agent/issue-116-explicit-settings-change-origin
Sep 24, 2026
Merged

KeyffMS merged 11 commits into
mainfrom
agent/issue-116-explicit-settings-change-origin

Conversation

@KeyffMS

@KeyffMS KeyffMS commented Sep 24, 2026

Copy link
Copy Markdown
Owner

Addresses #116.

Changes:

  • publishes SettingsChangedEventArgs with an explicit transaction origin;
  • gives each settings use-case context a private origin token;
  • removes ConfigurationForm._committingGridValue;
  • removes VisualProfileManagerForm._committingLocalChange;
  • removes RuntimeCoordinator._committingSettings;
  • local UI/runtime commands perform their intended post-commit follow-up exactly once and ignore only their own generic callback;
  • external changes still refresh interested components;
  • nested/reentrant commits retain independent origins;
  • existing failure rollback and profile-manager refresh behavior remain covered;
  • documents the synchronous event timing and origin contract.

Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
…nUseCases.cs

Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
…eUseCases.cs

Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
Addresses #116.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
@KeyffMS
KeyffMS merged commit 4a8849e into main Sep 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant