Replace fixed timeout with output-stability detection; add ctrl+c to cancel tool calls#2
Merged
Cicolas merged 8 commits intorelease/v1.0.4from Mar 10, 2026
Merged
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- finish-worktree now defaults to release/v<CARGO_VERSION> instead of release/next - new ship-release skill: reviews open PRs on the release branch, warns on unchecked test cases, asks user to include/skip each, then creates a release PR to main Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Owner
Author
|
Release v1.0.4
…g push - release.yml now triggers on both release/* → main PR merge and v* tag push - On PR merge: reads version from Cargo.toml, creates+pushes the tag, then builds - On tag push: uses the pushed tag directly, skips tag creation step - Removes tag-release.yml (now redundant) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Replace F3 context push with read_terminal tool description - Remove F3 from keybindings table - Update release instructions to mention release branch merge workflow Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…cancel tool calls - PendingCapture now tracks last_line_count/last_change instead of a fixed deadline - Fires 300 ms after PTY output stops growing (command finished) instead of 1500 ms fixed wait - Added CancelToolCall action: ctrl+c in LLM tab cancels in-progress tool calls, clears pending capture, and unlocks the terminal Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> fix: don't fire output capture before command produces any output The stability timer was starting from command-send time, so commands taking >300 ms to produce output would trigger the capture prematurely. Now the 300 ms stability window only starts once output appears; commands that produce no output at all fall back to a 5 s timeout. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Cicolas
force-pushed
the
worktree-fix-timeout-reading
branch
from
4458460 to
31083c4
Compare
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
PendingCapturenow trackslast_line_count/last_changeinstead of a fixed 1500 ms deadlineCancelToolCallaction: pressing ctrl+c in the LLM tab cancels any in-progress tool call, clears the pending capture, appends a[tool call cancelled by user]message to history, and unlocks the terminalNotes
Part of a batch of worktree changes targeting
release/next.Tests
ls) via tool call — output should be forwarded to Claude ~300 ms after it completesfind /) via tool call — output should be forwarded ~300 ms after the last line appears[tool call cancelled by user]shown in chat🤖 Generated with Claude Code