Re-vendor the murderboard to 81a0927: the stale copy had nothing to say about cost - #513
Merged
Merged
Conversation
…ay about cost
The freshness gate refused a review, correctly. bugarach sat at 3a6a8fb while upstream had
moved to 81a0927, and the difference is not cosmetic: upstream added a COST section after a
run on 2026-09-07 spent a two-day usage limit under a model that could not finish and
produced no review at all. The shape of that failure is the point — you do not get a partial
review for a partial price, you get no review and the full bill.
What the stale copy was missing, in its own terms:
* this process is a FAN-OUT and there is no cheap murderboard, because what scales to
stakes is HOW the roles are run and never WHICH ones;
* do not start a run on a model you cannot afford to exhaust — Claude Opus 5 is named as
known good;
* ask the human before every run, because the other way this wastes money is being fired
early, at a draft that was not ready, by a session that decided on its own that
something was a deliverable;
* if blocked or declined, STOP — do not re-invoke, do not run the roles by hand to route
around a refusal, and never economise by running fewer roles, because a review missing
roles is indistinguishable in the report from a clean one.
That last clause is the one this repo would have violated silently, which is exactly what a
freshness gate is for.
Re-copied by tools/murderboard_revendor.py, which is surgical about stamps for a reason
documented in its own header. The three locally adapted tools were NOT rewritten: none of
them changed upstream between the two commits, so the body difference the tool reports is
bugarach's own `# instrument:` line on line 3, and it is still there in all three. The
eleven compiled agent files already agree with the new process file.
Landed on main before the review that needed it, not on the leaf branch that wanted it: a
vendored copy on a feature branch leaves every new worktree inheriting the old one, which
the skill warns about in terms.
NOT ADOPTED, and named so it is a decision rather than an oversight: upstream also ships
murderboard_model_gate.sh, a PreToolUse hook that reads the running model and blocks the
call-up before the fan-out starts, and TERMS.md. bugarach vendors neither. The gate is the
mechanized half of everything above, and wiring it is worth doing on purpose rather than as
a side effect of a re-vendor.
Co-Authored-By: Claude Opus 5 (1M context) <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
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.
The freshness gate refused a review, correctly. bugarach sat at
3a6a8fb; upstream is81a0927, and the difference is not cosmetic.Upstream added a cost section after a run on 2026-09-07 spent a two-day usage limit under a model that could not finish and produced no review at all. That is the shape of the failure: you do not get a partial review for a partial price, you get no review and the full bill.
What the stale copy was missing, in its own terms:
That last clause is what this repo would otherwise have violated silently, which is what a freshness gate is for.
What was and was not touched
Re-copied with
tools/murderboard_revendor.py, which is surgical about stamps for reasons documented in its own header. The three locally adapted tools were not rewritten: none changed upstream between the two commits, so the body difference the tool reports is bugarach's own# instrument:line on line 3 — verified still present in all three. The eleven compiled agent files already agree with the new process file.Landed on
main, not on the branch that wanted it. A vendored copy on a feature branch leaves every new worktree inheriting the old one; the skill warns about this in terms.Not adopted, deliberately
Upstream also ships
murderboard_model_gate.sh— aPreToolUsehook that reads the running model and blocks the call-up before the fan-out starts — andTERMS.md. bugarach vendors neither. The gate is the mechanized half of everything above, and wiring it deserves to be a decision rather than a side effect of a re-vendor.🤖 Generated with Claude Code