docs(coding-agent): refresh workflowz notice with advisor + provenance patterns#2647
docs(coding-agent): refresh workflowz notice with advisor + provenance patterns#2647metaphorics wants to merge 1 commit into
Conversation
…e patterns Add two patterns to the hidden workflowz orchestration notice: "Advisor consult" (consult the advisor tool at a hard fork before committing, phrased "if enabled" so it is independent of the advisor-tool change) and "Provenance-rich context" (brief each subagent on which files/commits and what "done" means via a '/home/alpha/.omp/agent/sessions/-odin-oh-my-pi/2026-06-15T05-57-55-286Z_019ec9db-a3d6-7000-bad4-21ce1689b5f3/local' brief). Content-only; no code change.
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the internal coding-agent workflow notice guidance by adding two orchestration patterns and documenting the change in the package changelog.
Changes:
- Added “Advisor consult” guidance for using the
advisortool at hard decision points. - Added “Provenance-rich context” guidance for briefing subagents via
local://briefs. - Documented the notice refresh in
packages/coding-agent/CHANGELOG.md.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/coding-agent/src/prompts/system/workflow-notice.md | Adds two new orchestration patterns to the workflow notice. |
| packages/coding-agent/CHANGELOG.md | Records the workflow notice update under Unreleased. |
Comments suppressed due to low confidence (1)
packages/coding-agent/CHANGELOG.md:1
- The changelog refers to a hidden
workflowzorchestration notice, but the updated file isworkflow-notice.md. Ifworkflowzisn’t an intentional, established identifier, rename it to match the actual notice name to avoid confusion for readers and release notes consumers.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
roboomp
left a comment
roboomp
left a comment
There was a problem hiding this comment.
P2: small, coherent prompt/changelog refresh, but it depends on an advisor tool that is not present on the target branch.
Headline finding: workflowz would inject guidance to call a tool the merged code cannot expose unless the advisor-tool PR lands first or this wording changes.
Thanks for the focused update.
| - **Completeness critic** — a final agent that asks "what's missing — modality not run, claim unverified, file unread?"; its answer is the next round. | ||
| - **Budget/count loops** — `while len(bugs) < 10:` to hit a target, or `while budget.total and budget.remaining() > 50_000:` to scale depth to the turn budget; `log()` each round. | ||
| - **No silent caps** — if you bound coverage (top-N, no-retry, sampling), `log()` what you dropped; silent truncation reads as "covered everything" when it didn't. | ||
| - **Advisor consult** — at a hard fork or when stuck, if the `advisor` tool is enabled, consult it for a second opinion from a more-capable paired model before committing to a direction; cheaper than a full judge panel for a single decision. |
There was a problem hiding this comment.
should-fix: This notice now tells the agent to consult an advisor tool, but this PR/base branch does not register any advisor tool or mention it anywhere else in packages/coding-agent/packages/agent (advisor only appears in this added prompt/changelog text). If this lands before the advisor-tool PR, workflowz injects guidance for a tool the model cannot call; either stack this after that implementation or word this against an already-available mechanism.
Closes #2642 — adds Advisor-consult and provenance-rich-context patterns to the workflowz orchestration notice.
Depends on #2646.