Repository navigation
Add a full-send path to brigade for items the owner names - #701
Merged
Merged
Conversation
When the owner asks for full send on an item, fire --station full-send gives the worker tstack's full-send, sol-loop, file-pr, and babysit-pr skills at the commit tstack.json pins, in place of a poteto-mode playbook. The writer is Opus at xhigh and the sol-loop and gate are Sol at high. launch and a usage limit never ask roles.py for a seat. The worker drafts the PR into the reports directory, and the item lands through the landing queue after an independent gate review. scripts/sync_tstack.py shows what changed upstream and --bump moves the pin. scripts/check.py reports a tstack link that disagrees with the pin. Briefs for every other station are unchanged.
…it from full-send briefs The landing trunk need not be origin/main, and the standing rules ask a worker to save failed test output under reports/.
…hrases from full-send The author is t3dotgg in tstack.json. The landing queue opens a PR only in some landing modes, and the peak split is not a term the skill defines.
Applies the review's edits from reports/D57-review-1.md at 5dd7bd27634099104f299ea0022aa39de4944b69.
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.
Add a full-send path to brigade for items the owner names
The owner wants a way to run one item end to end with tstack's full-send method (full-send, sol-loop, file-pr, babysit-pr) instead of a poteto-mode playbook, with an Opus writer and Sol review rounds. Until now brigade could only seat an item through the standard worker roles. A full-send item had no brief shape, no seat, and no way to park on a usage limit, because
roles.pyrejects any role it does not know.This change adds that path, gated on the owner's own "full send" message in the coordinator thread.
brigade.py fire --station full-sendstarts the item. Its brief links the four tstack skills at the committstack.jsonpins, in place of the poteto-mode playbook line, theMode:lines, and the seat rule. The brief tells the worker to run each sol-loop round as a read-onlydelegate_taskoncodex/gpt-6.1-solat high, draft the PR intoreports/<item>-pr.md, open no PR, and leave babysit-pr and the merge to the coordinator and the landing queue.brigade_lib/full_send.pyholds the defaults. The writer isclaudeAgent/claude-opus-5-5at xhigh, and the sol-loop and gate arecodex/gpt-6.1-solat high.brigade.py launchprints the writer seat withoutroles.pyand exits 1 while aclaudeAgentseat is cooling.brigade.py dish --usage-limitrecords the cooldown and parks a full-send item, with no backup seat.full-send.mdsets the trigger and says the worker's own Sol rounds never count as the gate. The item lands only through the landing queue after one independent gate review on Sol.tstack.jsonpins the tstack repository, commit1085050f3baed1ac724caa4a8166145fb2ac88af, the four skill paths, the MIT license, and the author. Nothing from tstack is copied.scripts/sync_tstack.pyprints what changed in the four skills since the pin, and--bumpmoves the pin and every link undert3/anddocs/; the build then regeneratesskills/. It refuses when a skill is missing at the new commit.scripts/check.pyreports a tstack link whose commit disagrees with the pin.Proved by: new tests for the full-send brief, launch, cooldown refusal, and usage-limit park, which fail with the fix removed, plus tests for the pin check and the sync script against a modeled GitHub; briefs for the seven playbooks in light and full mode rendered byte-identical at the base and at head; and a fresh-child run that put "full send T1" on the full-send path and routed a plain request to the bug-fix playbook.
Coordinated by claudeAgent/claude-opus-5-5. Written by claudeAgent/claude-opus-5-5. Reviewed by codex/gpt-6.1-sol.