Add silent dry-run mode for AL issue triage - #8316
Merged
Merged
Conversation
Use the ALTool MCP server for symbol download and compilation, expose the credential variable names ALTool expects, normalize agent output, and archive reports without posting by default. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3828f007-4f8d-494d-80d3-b83a70071398
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3828f007-4f8d-494d-80d3-b83a70071398
Give the triage agent the same deterministic symbol-download wrapper used internally, backed by the freshly installed public prerelease ALTool, instead of exposing MCP configuration to the agent. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3828f007-4f8d-494d-80d3-b83a70071398
Thaddeus Loke (thloke)
marked this pull request as ready for review
August 18, 2026 07:59
Steffen Balslev (SBalslev)
approved these changes
Aug 18, 2026
Give the public triage agent deterministic skills for project creation, symbol download, compilation, analyzers, publishing, tests, and E2E verification while keeping direct compiler and endpoint access prohibited. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3828f007-4f8d-494d-80d3-b83a70071398
Pass the BC tenant through symbol, publish, and test operations and fail before agent execution unless a prerelease ALTool symbol download succeeds. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 3828f007-4f8d-494d-80d3-b83a70071398
Thomas Pedersen (thpeder)
approved these changes
Aug 19, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 81bc8143-7b41-4a6c-948e-4a1e29437587
Thomas Pedersen (thpeder)
approved these changes
Aug 19, 2026
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
3 participants
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.
Summary
download-al-symbolsskill using the freshly installed public prerelease ALTool, matching the internal workflow;BC_SERVER_USERNAME/BC_SERVER_PASSWORD, which ALTool actually reads;alc.exe, and manual/dev/packagescalls;Issue #8313 diagnosis
The workflow exported
BC_USERNAME/BC_PASSWORD, but ALTool resolves non-interactive UserPassword credentials only fromBC_SERVER_USERNAME/BC_SERVER_PASSWORD. The agent also bypassed ALTool by invokingalc.exedirectly, then treated a manual/dev/packages401 as evidence that symbols were unavailable.The replacement workflow installs the latest prerelease
Microsoft.Dynamics.BusinessCentral.Development.Tools, records that exactal.exeasALTOOL_PATH, and requires the deterministic symbol skill before compilation. The skill owns ALTool's internal symbol-download protocol so the agent only invokes the command-line workflow.Verification
18.0.40.43394-betainspected in an isolated tool path;