Uh oh!
There was an error while loading. Please reload this page.
Remove agentic-workflows guide from prompt and move reference to .github/aw - #36782
Merged
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
pelikhan
commented
Jun 4, 2026
Collaborator
@copilot dramatically reduce size of prompt and remove the flag instead |
…gating Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
ContributorAuthor
Done in 08f1009. Reverted the |
CopilotAI
changed the title
Gate agentic-workflows audit guide behind explicit Reduce size of agentic-workflows guide promptJun 4, 2026
audit: true flag…ne guide to 2 lines Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
ContributorAuthor
Done. Created 163 chars, down from 1,641 (90% reduction). |
pelikhan
reviewed
Jun 4, 2026
| #### `audit` — Inspect a specific run | ||
| - `run_id`: run ID or run/job URL (including step anchors), as string or number (alias for `run_id_or_url`) | ||
| - `run_id_or_url`: run ID or run/job URL (including step anchors), as string or number | ||
| Use MCP tools (NOT shell commands): use `logs` to find runs, use `audit` for a deep dive into a specific run. |
ContributorAuthor
There was a problem hiding this comment.
Added in the latest commit.
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
pelikhan
reviewed
Jun 4, 2026
| @@ -1,32 +1,3 @@ | |||
| <agentic-workflows-guide> | |||
Collaborator
There was a problem hiding this comment.
@copilot remove this file completely, update compiler and recompile.
ContributorAuthor
There was a problem hiding this comment.
Done. Removed agentic_workflows_guide.md, stripped the injection from the compiler (unified_prompt_step.go, prompts.go, prompt_constants.go), removed the related tests, updated the golden file, and recompiled all 239 workflows — the guide reference is now gone from every lock file.
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
CopilotAI
changed the title
Reduce size of agentic-workflows guide promptRemove agentic-workflows guide from prompt and move reference to .github/awJun 4, 2026
pelikhan
approved these changes
Jun 4, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Removes the inline agentic_workflows_guide.md prompt injection from compiled workflows to reduce prompt overhead, and introduces a .github/aw document intended to serve as the agentic-workflows MCP tool reference.
Changes:
- Removed agentic-workflows guide prompt injection logic and deleted the guide markdown asset.
- Added
.github/aw/agentic-workflows-mcp.mdas the intended replacement reference material. - Recompiled workflows/goldens to remove
agentic_workflows_guide.mdfrom generated prompt assembly.
Show a summary per file
| File | Description |
|---|---|
| pkg/workflow/unified_prompt_step.go | Removes agentic-workflows guide section injection from prompt section collection. |
| pkg/workflow/testdata/TestWasmGolden_CompileFixtures/smoke-copilot.golden | Updates golden fixture to no longer concatenate the removed guide file. |
| pkg/workflow/prompts.go | Removes hasAgenticWorkflowsTool helper (no longer needed). |
| pkg/workflow/prompts_test.go | Removes tests that asserted guide inclusion behavior. |
| pkg/workflow/prompt_constants.go | Removes agenticWorkflowsGuideFile constant. |
| actions/setup/md/agentic_workflows_guide.md | Deletes the previously injected guide markdown file. |
| .github/workflows/workflow-normalizer.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/weekly-blog-post-writer.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/static-analysis-report.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/smoke-copilot.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/smoke-copilot-arm.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/smoke-codex.lock.yml | Recompiled lock file; updates prompt assembly/heredoc IDs after regeneration. |
| .github/workflows/smoke-claude.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/smoke-call-workflow.lock.yml | Recompiled lock file; updates prompt assembly/heredoc IDs after regeneration. |
| .github/workflows/security-review.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/schema-feature-coverage.lock.yml | Recompiled lock file; updates prompt assembly/heredoc IDs after regeneration. |
| .github/workflows/safe-output-health.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/q.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/python-data-charts.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/prompt-clustering-analysis.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/necromancer.lock.yml | Recompiled lock file; updates prompt assembly/heredoc IDs after regeneration. |
| .github/workflows/metrics-collector.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/mcp-inspector.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/issue-arborist.lock.yml | Recompiled lock file; updates prompt assembly/heredoc IDs after regeneration. |
| .github/workflows/grumpy-reviewer.lock.yml | Recompiled lock file; updates prompt assembly/heredoc IDs after regeneration. |
| .github/workflows/example-workflow-analyzer.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/duplicate-code-detector.lock.yml | Recompiled lock file; updates prompt assembly/heredoc IDs after regeneration. |
| .github/workflows/dev.lock.yml | Recompiled lock file; updates prompt assembly/heredoc IDs after regeneration. |
| .github/workflows/dev-hawk.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/deep-report.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/daily-security-observability.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/daily-safe-output-optimizer.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/daily-rendering-scripts-verifier.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/daily-observability-report.lock.yml | Recompiled lock file; removes guide prompt concatenation + regen IDs. |
| .github/workflows/daily-firewall-report.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/daily-fact.lock.yml | Recompiled lock file; updates prompt assembly/heredoc IDs after regeneration. |
| .github/workflows/daily-cli-tools-tester.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/daily-cache-strategy-analyzer.lock.yml | Recompiled lock file; removes guide prompt concatenation + regen IDs. |
| .github/workflows/daily-ambient-context-optimizer.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/daily-agentrx-trace-optimizer.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/daily-agent-of-the-day-blog-writer.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/cloclo.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/aw-failure-investigator.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/audit-workflows.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/api-consumption-report.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/agentic-token-trend-audit.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/agentic-token-audit.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/agent-persona-explorer.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/workflows/agent-performance-analyzer.lock.yml | Recompiled lock file; removes guide prompt concatenation. |
| .github/skills/agentic-workflows/SKILL.md | Updates router’s referenced .github/aw files list (currently points at a missing file). |
| .github/aw/agentic-workflows-mcp.md | Adds the new MCP tool reference document intended to replace the removed injected guide. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comments suppressed due to low confidence (1)
pkg/workflow/prompts_test.go:634
- The agentic-workflows guide inclusion tests were removed, but there isn't a replacement assertion covering the new behavior (guide is never injected, even when the tool is enabled). Adding a targeted regression test would prevent reintroducing prompt bloat in future refactors.
- Files reviewed: 51/51 changed files
- Comments generated: 5
Comment on lines
29
to
31
| - `.github/aw/llms.md` | ||
| - `.github/aw/mcp-clis.md` | ||
| - `.github/aw/memory.md` |
Comment on lines
+15
to
+16
| No required parameters. | ||
Comment on lines
+29
to
+32
| | `firewall` / `no_firewall` | Filter by firewall status | | ||
| | `filtered_integrity` | Only runs with DIFC integrity-filtered events in gateway logs | | ||
| | `after_run_id` / `before_run_id` | Paginate by run database ID | | ||
Comment on lines
+37
to
+40
| | Parameter | Description | | ||
| |---|---| | ||
| | `run_id_or_url` | Run ID or run/job URL (including step anchors), as string or number | | ||
Comment on lines
+43
to
+45
| Recompile workflow `.md` files into `.lock.yml` files. | ||
| **MCP equivalent of**: `gh aw compile` |
This was referenced Jun 4, 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 freeto 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.
The
agentic_workflows_guide.md(~1,628 chars) was injected into all workflows withagentic-workflowsenabled, adding significant prompt overhead. This removes the file entirely and moves the full reference to a.github/awinstructions file.Changes
agentic_workflows_guide.md— removed completely; the compiler no longer injects any guide into the prompt foragentic-workflowsworkflowsunified_prompt_step.go/prompts.go/prompt_constants.go— removed the guide injection block,hasAgenticWorkflowsToolhelper, and theagenticWorkflowsGuideFileconstant.github/aw/agentic-workflows-mcp.md— new instructions file containing the full MCP tool reference (parameters and descriptions forstatus,logs,audit, andcompile) for use by coding agents and Copilot Chatagentic_workflows_guide.mdreference is gone from every lock file