Skip to content

Flat waits are a blocker, cost is tokens, and fixes for reds land first - #491

Merged
Japabu merged 3 commits into
mainfrom
wt/toyos-rolerules
Sep 24, 2026
Merged

Japabu merged 3 commits into
mainfrom
wt/toyos-rolerules

Conversation

@Japabu

@Japabu Japabu commented Sep 24, 2026

Copy link
Copy Markdown
Collaborator

Two role-file changes the owner ordered.

  • implementer.md, reviewer.md: never a flat wait, in code or in a test. Wait on the event, bounded by a timeout that fails loudly. A fixed delay is allowed only where a hardware document mandates it and offers no notification, cited at the site. No defensive code. The reviewer ranks a flat wait as a BLOCKER.
  • orchestrator.md:
    • The cost is Claude tokens and the owner's time; CI minutes are free. Agent transcripts record usage, and across 907 agents cost follows how long an agent runs.
    • A fix for a red lands ahead of feature work.
    • A nightly name red three nights running gets an owner issue that day, and is deleted if nobody owns it a week later. The flake census found four fixes stranded for a week and a nightly red 42 nights running.

Documents only.

🤖 Generated with Claude Code

https://claude.ai/code/session_014iqcj4jDKpaiDX8B7CMvmK

Japabu and others added 2 commits September 24, 2026 11:30
The owner's ruling: never a flat wait time — sleeping and assuming the thing
happened is always a sign of bad code, in the tree and in its tests. The one
exception is a time a hardware document mandates where no notification
exists, cited at the site. Code waits on the event itself, bounded by a
timeout that fails loudly; no defensive code; fail fast, never degrade
silently.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014iqcj4jDKpaiDX8B7CMvmK
The owner: the cost is Claude tokens and the owner's time, not CI minutes. The
transcripts' usage over 907 agents showed cost following how long an agent
runs. The flake census showed four written fixes stranded for a week behind
unrelated reds, and a nightly red 42 nights running with names nobody owned.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014iqcj4jDKpaiDX8B7CMvmK
@Japabu
Japabu enabled auto-merge September 24, 2026 09:30
@Japabu
Japabu added this pull request to the merge queue Sep 24, 2026
Merged via the queue into main with commit cfca6a4 Sep 24, 2026
2 checks passed
@Japabu
Japabu deleted the wt/toyos-rolerules branch September 28, 2026 09:46
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.

1 participant