fix: restore agent instruction guidance - #9593
Conversation
🔒 AI Validation PendingThis PR is from an external contributor. AI validation will be performed after manual review. Maintainers: Add the |
There was a problem hiding this comment.
🟡 Changes recommended
The required screenshot for a designer-ui visual change is missing.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Restores localized guidance and documentation for agent system instructions while preserving Foundry-managed hiding behavior.
Changes:
- Restores explanatory text and documentation link.
- Adds localization resources.
- Adds visible/hidden-state regression tests.
File summaries
| File | Description |
|---|---|
Localize/lang/strings.json |
Adds guidance localization strings. |
libs/designer-ui/src/lib/agentinstruction/index.tsx |
Renders guidance when system instructions are visible. |
libs/designer-ui/src/lib/agentinstruction/__test__/agentinstruction.spec.tsx |
Tests guidance visibility and link attributes. |
Review details
- Files reviewed: 3/3 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
sunxiayi please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
Commit Type
Risk Level
What & Why
Restores the missing system-instruction guidance and the “Tips for writing agent instructions” documentation link in
AgentInstructionEditor.The guidance was removed from the shared editor in #8402, which caused the content reported in #8531 to disappear in both Standard and Consumption experiences. The restored content remains inside the existing
!hideSystemInstructionsbranch so Foundry-managed scenarios that hide the system editor do not display unrelated guidance.Fixes #8531.
Impact of Change
Test Plan
AgentInstructionEditor;pnpm --filter @microsoft/designer-ui test:lib(120 files, 1088 tests);pnpm --filter @microsoft/designer-ui build:lib; Biome; andtsc --noEmitContributors
Screenshots/Videos