Repository navigation
Tell sessions to wait on a backgrounded command by its own task - #183
Merged
Merged
Conversation
The headless text every Session prompt ends with now says to wait on a command the Bash tool moved to the background by its task id or output file, never by process names or patterns, since other sessions on the machine run the same commands. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
4 tasks
Merged
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
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
The shared headless text every Session prompt ends with (
HEADLESSinsrc/prompt.rs) now says how to wait on a command that the Bash tool moves to the background:The text lives only in that one constant, so the fresh, continuing, Spec review, resume and every Repair prompt pick it up. The Prompts and skills page (
site/prompts/index.html) was regenerated.Evidence
cargo test --test fresh_runfailed ongives_the_agent_the_fresh_promptonce the expected prompt had the new sentence andHEADLESSdidn't yet.After:
fresh_run,continuation,mergeableandcipass with the new wording, and so does the golden-file testprompts_page_shows_what_the_binary_sends. All 24 test binaries pass.cargo clippy --all-targetsandcargo fmt --checkare clean.Merge Danger
Door: two-way
Only prompt text changes. A revert restores the old wording.
Blast Radius: prompts
Every Session gets one more sentence, and nothing else changes.
Unaddressed findings
Standards
pgrep/ps | grepexamples. Skipped. The issue asks for these examples, and the ban comes with the positive instruction. The examples name the exact pattern that cost the One Run notification per Spec run #148 run about 7 minutes.HEADLESStext is copied into 4 test files. Skipped. This was already the case before this branch. Those tests pin the whole prompt on purpose, and changing that is out of scope here.Spec
None.
Closes #180
🤖 Generated with Claude Code