8000 budget: reviewers and the final gate get the contract copy in their host worktree; live run 17 by adeelahmad · Pull Request #24 · adeelahmad/aloop · GitHub
[go: up one dir, main page]

Skip to content

budget: reviewers and the final gate get the contract copy in their host worktree; live run 17 - #24

Merged
adeelahmad merged 1 commit into
masterfrom
claude/project-thread-qerfr9
Oct 18, 2026
Merged

adeelahmad merged 1 commit into
masterfrom
claude/project-thread-qerfr9

Conversation

@adeelahmad
Copy link
Copy Markdown
Owner

Requested by Adeel · project thread

Before: on Claude Code the host creates a sub-agent's worktree at dispatch, so budget hook copies the supervisor's task contract into it on the worker's first tool call, and selfcheck there finds the task. Only the three workers (RED, SCAFFOLD, GREEN) took that path; the structural reviewer and the final gate went through the observe-only metering and never got the copy. Every reviewer's first self-check therefore blocked "STORY_DIR unset" until it guessed the contract by hand: 2, 1 and 3 blocks in live runs 15, 16 and 17.

After: the observe path also finds the agent's worktree and copies the contract when the dispatch prompt names a task, and the observed role's state records its task, attempt and story dir. Nothing about metering changes: observe-only roles are still never told or stopped. ROADMAP M0.8 and OVERVIEW record live run 17 (13/13, 12m58s, $0.95, 2 supervisor wakes, 0 plugin-source reads, 6 blocks).

How: a new case in scripts/test/budget-roles.test.py (a structural reviewer dispatched into a host worktree gets the copy and its state names the task) written first and failing, then ~10 lines in plugin/bin/budget's observe. All budget tests and make ci pass locally.


Generated by Claude Code

…ost worktree (3 of run 17's 6 blocks); run 17

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015QxyVf3NgwsgRyYpBgDDTC
@adeelahmad
adeelahmad merged commit 508c995 into master Oct 18, 2026
4 checks passed
adeelahmad pushed a commit that referenced this pull request Oct 18, 2026
…n/ (master red after #23 met #24)

#24 added a case expecting the copy at .agentic/task.env; #23 moved the
worktree markers to .aladdin/. budget already writes through
_paths.marker_dir, so only the test's expected path was stale.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014j4wLin3oJTtnvjE9Lq2VV
adeelahmad added a commit that referenced this pull request Oct 18, 2026
…met #24

ROADMAP §4.11: test the README; fix master's budget-roles case after #23 met #24
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.

2 participants

0