[Bug 649034] Track the Expense Agent activator - #11293
Draft
Prangshuman Das (t-prda) wants to merge 7 commits into
Draft
[Bug 649034] Track the Expense Agent activator#11293Prangshuman Das (t-prda) wants to merge 7 commits into
Prangshuman Das (t-prda) wants to merge 7 commits into
Conversation
Refresh the platform Agent record only after successful Expense Agent setup persistence so the sidecar identifies the latest configuring administrator. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 988fb581-008c-4e0d-99c1-285cd45d49e7
Prangshuman Das (t-prda)
temporarily deployed
to
triage
September 10, 2026 00:28 — with
GitHub Actions
Inactive
Mark the platform setup buffer as updated when Expense Agent-specific configuration changes so Agent Setup owns the attribution write. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 988fb581-008c-4e0d-99c1-285cd45d49e7
Force the platform Agent update only when an existing Expense Agent is activated, reactivated, or deactivated. Settings-only saves preserve the previous attribution. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 988fb581-008c-4e0d-99c1-285cd45d49e7
Update the logical Agent record through a permission-bearing helper after successful activation so Configured by tracks the last activator, matching Sales Order Agent behavior. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 988fb581-008c-4e0d-99c1-285cd45d49e7
Use an object ID that is free across the Expense Agent app family so localized builds and backports can consume the W1 app without a collision. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 988fb581-008c-4e0d-99c1-285cd45d49e7
Assign the unchanged display name before modifying the logical Agent record so code analysis recognizes the intentional attribution update. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 988fb581-008c-4e0d-99c1-285cd45d49e7
Rename the attribution helper to EA Setup Mgt. and move Agent identity resolution, change detection, setup persistence, and activator attribution behind the shared setup boundary. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 988fb581-008c-4e0d-99c1-285cd45d49e7
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Status: unresolved - keep draft
Playwright reproduced the failure on
tenant2-1, companyCRONUS W1: PRDAS successfully activated Expense Agent, but the EA avatar task pane still displayed Configured by TESTUSER2 after reopening. The User row modifier changed to PRDAS; the Agent Data modifier did not.The previous helper diagnostic was not proof of a fix: it ran as the user already recorded in Agent Data. The proposed no-op attribution write below is therefore not validated and does not fix the reproduced scenario.
Read-only runtime inspection explains the no-op behavior: equal assignments are optimized away;
Agent Datahas a system modify trigger, so same-valueModifyAllalso falls back to the same path. SOA's enabled save additionally updates instructions throughAgent.SetInstructions; the identity-sync explanation alone was incomplete.Do not merge or backport this draft until a supported audit-update fix passes the actual two-user UI scenario.
Proposed changes (not a verified fix)
SystemModifiedByidentifies the latest activatorEA Setup Mgt.codeunitValidation
tenant2-1tenant2-1into isolated project cachestenant2-1tenant2-1Integration notes
PersistAgentStateunchangedExpense Agent Entra App Mgt.; this PR keepsEA Setup Mgt.limited to the Business Central Agent frameworkreleases/29.xandreleases/29.0, keeping later backports surgicalFixes AB#649034