Skip to content

Answer refused sends at once; rehearsal copies start open - #2010

Merged
trissim merged 2 commits into
mainfrom
fix/refusal-answers-first
Oct 19, 2026
Merged

trissim merged 2 commits into
mainfrom
fix/refusal-answers-first

Conversation

@trissim

@trissim trissim commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

Refused sends answered only after the wire lock freed (tools.py:1081 recorded first). Rehearsal copies carried the live maintenance files, which broke #1999's early check on every rehearsal.

🤖 Generated with Claude Code

Copilot AI balanced review requested due to automatic review settings October 19, 2026 22:00

Copilot AI left a comment

Copy link
Copy Markdown

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.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@trissim
trissim force-pushed the fix/refusal-answers-first branch from 8c0a706 to 4850f9a Compare October 19, 2026 22:02
A refused send recorded itself under the wire lock before raising, so any
install or rehearsal holding that lock made the refusal hang like a dead
server. The record now waits on its own thread; the caller hears the
refusal at once.

copy_bus copied the live root's maintenance files into the copy, so the
early client check refused every command on the copy. The copy skips them.

Src-Budget: 4
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@trissim
trissim force-pushed the fix/refusal-answers-first branch from 4850f9a to f52cb45 Compare October 19, 2026 22:02
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
trissim added a commit that referenced this pull request Oct 19, 2026
Integrate #2010 #1999 #2011: refused sends answer at once; rotate stops crashing
@trissim
trissim merged commit b7bc628 into main Oct 19, 2026
1 of 4 checks passed
@trissim
trissim deleted the fix/refusal-answers-first branch October 20, 2026 01:59
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