Skip to content

Give Ask agents Whiteboard's writing guidance - #955

Merged
milanb17 merged 1 commit into
mainfrom
mb/whiteboard-mcp-ask-guidance-d41886
Oct 6, 2026
Merged

milanb17 merged 1 commit into
mainfrom
mb/whiteboard-mcp-ask-guidance-d41886

Conversation

@milanb17

@milanb17 milanb17 commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

What

The context an Ask agent reads before its first question (askContext in packages/review/src/review-api/http.ts) now:

  • Inline answers: replaces "Answer the question." with the guidance that carries over from authoring: write for a staff engineer, lead with the answer, keep it short, explain components and data flow before functions, and check claims against code it has read.
  • Review edits: when the reviewer asks to change the review, tells the agent to read session_get_instructions({}) first (the CLI form for agents without MCP), for the guidelines and each component's fields, and to skip the create-a-review flow since the review exists.

Why

Authoring guidance reaches an agent only through session_get_instructions. An Ask agent answering inline has no reason to call it, so it got none of that guidance; and one asked to edit the review was pointed at session_edit without the component reference.

The full authoring guide is not inlined: most of it is the review-building workflow, and its review-source: link format differs from the plain src/app.ts:42 links the Ask panel opens.

Testing

Typecheck clean; src/ask and src/review-api tests pass. Not yet exercised in a live Ask session.

An Ask agent answering inline never called session_get_instructions, so it
got none of Whiteboard's guidance. Its first prompt now carries the style
that applies to an inline answer, and points it at session_get_instructions
before it edits the review, for the guidelines and component fields.

Agent-Session: d4745dd1-0fc1-49ef-8545-286495939081
Agent-Session: b0dfef22-75f9-4974-8799-a762750d0c3f
Agent-Session: 42392086-921d-4eab-b427-21748e1316e5
@milanb17
milanb17 merged commit aa5a247 into main Oct 6, 2026
1 check passed
@milanb17
milanb17 deleted the mb/whiteboard-mcp-ask-guidance-d41886 branch October 6, 2026 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants