Skip to content

Remove the "src/server/handlers/plot-plan-runs.test.ts": 640 entry from…#424

Merged
jayminwest merged 3 commits into
mainfrom
warren/run_hrgcqj6bd7qr
Jun 17, 2026
Merged

Remove the "src/server/handlers/plot-plan-runs.test.ts": 640 entry from…#424
jayminwest merged 3 commits into
mainfrom
warren/run_hrgcqj6bd7qr

Conversation

@jayminwest

Copy link
Copy Markdown
Owner

Summary

Remove plot-plan-runs.test.ts file-size budget entry (warren-e304)

Run

  • Warren run: run_hrgcqj6bd7qr
  • Agent: pi
  • Cost: $0.326 (30 in / 2.5k out / 307.2k cache-r)

Seeds

  • warren-e304 — Remove the "src/server/handlers/plot-plan-runs.test.ts": 640 entry from scripts/file-size-budgets.json (delete that single line). Do NOT add budget entries for the new sibling files (plot-plan-runs.validation.test.ts, plot-plan-runs.test-helpers.ts) — they must default-pass under the 500-line threshold (confirm wc -l for each shows < 500). Per docs/CONSTITUTION.md Article VI, before declaring done run a repo-wide search for the old path across ALL file types — rg -n "plot-plan-runs.test" --hidden -g '!node_modules' . (or git grep -n "plot-plan-runs.test") PLUS an explicit sweep of Dockerfile, docker-compose.yml, .github/workflows/*.yml, src/supervisor/ spawn/config strings, scripts/acceptance/, and docs/ — and fix any stale reference (file moves have broken production here before; encode the check, do not assume it). Verify: bun run check:size exits 0 AND bun run check:all is fully green (every gate stays green; the removed entry leaves all resulting files default-passing under the 500-line threshold).

Commits (3)

  • d933a6f Remove plot-plan-runs.test.ts file-size budget entry (warren-e304)
  • b013149 Close warren-e304
  • c9d5cdd chore(warren): seeds state

Files changed

.seeds/issues.jsonl                                        | 4 ++--
 .seeds/plans.jsonl                                         | 2 +-
 scripts/acceptance/scenarios/31-plot-plan-run-synthesis.ts | 2 +-
 scripts/file-size-budgets.json                             | 1 -
 4 files changed, 4 insertions(+), 5 deletions(-)

Prompt

Show prompt
work on sd warren-e304

🤖 Opened by warren run run_hrgcqj6bd7qr

jayminwest and others added 3 commits June 17, 2026 10:58
The file was decomposed below the 500-line limit in warren-59db, so the
640-line grandfather entry is no longer needed; the three resulting
siblings default-pass. Also fix a stale acceptance-scenario comment
referencing the pre-split handlers.plot-plan-runs.test.ts path.
@jayminwest jayminwest enabled auto-merge (squash) June 17, 2026 10:59
@jayminwest jayminwest merged commit 7de0899 into main Jun 17, 2026
2 checks passed
@jayminwest jayminwest deleted the warren/run_hrgcqj6bd7qr branch June 17, 2026 11:00
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