The MCP answers the decisions waiting for the owner as a ui:// card with ship and decline on it #11852

The asks card already had Ship, Decline and Decide buttons, but pressing one only ran a dry run. The card never showed the page's confirm step, so nothing could be settled from it. This pull adds that step.

What changed

• Plan card: pressing Ship or Decline now returns a plan card titled "Confirm". It lists each ask as before → after · outcome, and its status line reads like the page's header, for example Close: 1 of 1 · 0 refused or skipped · undo afterwards.
• Confirm button: the plan card carries one button, such as "Close 1" (the page's own wording). It re-calls asks_act with dryRun:false, which returns the same receipt sentence and Undo card as before.
• Where the code is: the plan card is built in src/web/studio/cards.ts. src/http/mcp/tools/asks_act.ts now hands the card the action, items and reason on a dry run. The ui://forge/asks row in docs/MCP.md is updated.
• Token scope: the buttons call the same tools, so the token's scope still decides what can be done. A read-only token can plan but its confirm is refused; the test checks this.
• Test: test/mcp-asks-card-decide.test.ts lists asks, presses Decline, checks nothing changed, confirms, checks the receipt, and checks the page shows the ask before and not after.

What I did not do

• I added no tool, argument or schema field, so tools/list does not grow.
• I did not change the shared card script; the plan card is plain data drawn the same escaped way as the others.
• The test shows the ask gone from /asks, but does not fetch /asks?op=… to check the receipt banner.

Checks: the gate and tsc are clean. I ran only mcp-size, the cards tests and the new test (all pass), not the full suite.

To try it: call the asks tool and press Decline on an assigned ask. Check the plan card, then press "Close 1". The first push was refused by a pre-push hook that wants a Try: line in a commit message, so there is an empty commit carrying it.

🤖 Generated with Claude Code

Closes #11816

The asks card already had Ship, Decline and Decide buttons, but pressing one only ran a dry run. The card never showed the page's confirm step, so nothing could be settled from it. This pull adds that step.

What changed

• Plan card: pressing Ship or Decline now returns a plan card titled "Confirm". It lists each ask as before → after · outcome, and its status line reads like the page's header, for example Close: 1 of 1 · 0 refused or skipped · undo afterwards.
• Confirm button: the plan card carries one button, such as "Close 1" (the page's own wording). It re-calls asks_act with dryRun:false, which returns the same receipt sentence and Undo card as before.
• Where the code is: the plan card is built in src/web/studio/cards.ts. src/http/mcp/tools/asks_act.ts now hands the card the action, items and reason on a dry run. The ui://forge/asks row in docs/MCP.md is updated.
• Token scope: the buttons call the same tools, so the token's scope still decides what can be done. A read-only token can plan but its confirm is refused; the test checks this.
• Test: test/mcp-asks-card-decide.test.ts lists asks, presses Decline, checks nothing changed, confirms, checks the receipt, and checks the page shows the ask before and not after.

What I did not do

• I added no tool, argument or schema field, so tools/list does not grow.
• I did not change the shared card script; the plan card is plain data drawn the same escaped way as the others.
• The test shows the ask gone from /asks, but does not fetch /asks?op=… to check the receipt banner.

Checks: the gate and tsc are clean. I ran only mcp-size, the cards tests and the new test (all pass), not the full suite.

To try it: call the asks tool and press Decline on an assigned ask. Check the plan card, then press "Close 1". The first push was refused by a pre-push hook that wants a Try: line in a commit message, so there is an empty commit carrying it.

🤖 Generated with Claude Code

I've addressed all five review points and pushed to the same branch.

Code (src/web/studio/cards.ts)

• The Confirm button's items now drop empty strings and are sliced to 100, the limit asks_act accepts.
• A missing counts.refused now counts as 0, so the status no longer prints NaN refused or skipped.

Tests (test/mcp-asks-card-decide.test.ts)

• Reason: a close plan with a reason produces Confirm args that carry reason.
• Nothing changes: a plan with only skipped items shows the skipped rows and a non-zero refused or skipped count, and has no button.
• ok:false: a refused plan returns the error and no card.
• Cap: 150 items, some empty, come back as 100 non-empty items.
• Missing count: a plan without counts.refused prints "0 refused or skipped".
• Page's words: the card row for an ask is checked against the /asks page for the same ask. The card's row label has the ask's title and "assigned to you", and the page has both too. The two don't use the same sentence, so the test compares those two parts rather than the full text.

Checks

• bun scripts/design-lint.ts is clean: no colour, size or component added outside the system.
• bun scripts/gate.ts and bun x tsc --noEmit printed nothing, which I took as clean.
• I ran the card and mcp-asks test files: 287 pass, 2 skipped, 0 fail.
• scripts/fast.ts said the diff was too broad to name (1658 test files, about 141 minutes), so I picked those files myself.