Skip to content

test(core): approving the first of three queued calls cancels none - #6441

Open
Hmbown wants to merge 1 commit into
mainfrom
test/queued-approvals-not-cancelled
Open

Hmbown wants to merge 1 commit into
mainfrom
test/queued-approvals-not-cancelled

Conversation

@Hmbown

@Hmbown Hmbown commented Sep 23, 2026

Copy link
Copy Markdown
Owner

No-Issue: regression coverage for the desktop approval queue failure (private app #97); no public issue.

Desktop dogfood (app #97) saw an approval fail the call it approved and
then cancel the queued calls behind it. 59c2a16 fixed the posture
re-check for a single call; this pins the queue: one model step emits
three gated shell calls, the client republishes its unchanged posture
with the first approval as the desktop app does, and every call must
run, in order, with none cancelled.

Mined from the unreviewed 0.10.1 WIP branch (two files, unchanged).
Regression coverage for behavior already on main, not a new fix.

Evidence: 1 passed, 0 failed (13,258 skipped) on current main.

🤖 Generated with Claude Code

Desktop dogfood (app #97) saw an approval fail the call it approved and
then cancel the queued calls behind it. 59c2a16 fixed the posture
re-check for a single call; this pins the queue: one model step emits
three gated shell calls, the client republishes its unchanged posture
with the first approval as the desktop app does, and every call must
run, in order, with none cancelled.

Mined from the unreviewed 0.10.1 WIP branch (two files, unchanged).
Regression coverage for behavior already on main, not a new fix.

Evidence: 1 passed, 0 failed (13,258 skipped) on current main.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 23, 2026 16:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

This branch has not been deployed

No deployments
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