Repository navigation
Add context hygiene at phase boundaries and abort budget - #32
Merged
Merged
Conversation
Closes #19 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
dmccoystephenson
left a comment
Member
Author
There was a problem hiding this comment.
Self-review rubric:
- Scope: PASS — 2 files (create-dev-loop.md, RESEARCH.md); 29 lines total, each tied to a #19 AC item or the CLAUDE.md PR-27 Implementations rule.
- Tests-new: PASS — boundary-summary instructions are self-exemplifying (re-reading them at the next phase boundary is the test). The abort-budget edge case is exercised by this very cycle staying under it (well under 500 tool calls / 200k tokens for this PR).
- Tests-fix: N/A — not a bug fix.
- Sibling structure: PASS — three boundary-summary blocks share an identical shape (
**X summary (re-read at the start of Phase Y).**heading + bullet list of structured fields); edge-case entry matches the**Label:** prose + numbered stepspattern of the existing edge cases above it. - Sibling renames: N/A — no identifiers renamed.
- Docs: PASS — Phase 7 sources (
create-dev-loop.md,README.md) reviewed; README is high-level and doesn't enumerate inner-phase summary blocks. RESEARCH.md §4 gets an Implementations entry per the CLAUDE.md PR-27 rule. - Issue resolution: PASS — all five AC items from #19 met: (1) Phase 2 plan summary; (2) Phase 3 implementation summary; (3) Phase 8 cycle summary; (4) abort-budget edge case with explicit thresholds; (5) gap-issue template is concrete (title format + 4-bullet body shape).
- Validation: PASS — placeholder count unchanged at 20; no new fenced blocks introduced (the new prose uses inline backticks only); diff under scope ceiling.
- {{EXTERNAL_SIGNAL_LABEL}}: N/A — no CI; manual validation against the diff is the anchor (which I just did).
- Placeholder-consistency: N/A — no new placeholders.
- Escaping: N/A — no new fenced code blocks added.
Adversarial probe — 'Aren't 500 tool calls / 200k tokens just made up numbers?' Yes — RESEARCH.md §4 cites the half-life model but doesn't give a calibrated threshold. The 500/200k figures match #19's own placeholder values from the issue body, which the author marked as 'numbers to calibrate empirically.' Documented limitation; current values are reasonable starting points that future cycles can refine based on observed abort triggers.
dmccoystephenson
added a commit
that referenced
this pull request
Aug 3, 2026
Closes #19 Co-authored-by: BuildTools <unconfigured@null.spigotmc.org> Co-authored-by: Claude Opus 4.7 (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.
Summary
Four additions to the generated dev-loop template, grounded in RESEARCH.md §4 (long-horizon decay + context rot):
Closes #19
Test plan
{{placeholders}}introduced)create-dev-loop.mdexists; "Phase 2", "Phase 3 — Implementation", "Phase 8 — Merge", "Edge cases" headings all confirmed present before editinggh issue view 19and confirmed title/body matchcreate-dev-loop.mdandRESEARCH.md; no matchesOpen question (#19 noted it)
Whether
ScheduleWakeupshould be used to fresh-context-restart on abort. Out of scope for this PR — the abort instruction tells the agent to exit, and the user (or harness) initiates the next cycle.drafted by Claude on behalf of Daniel Stephenson