The previous fix added a Step 3 that ran the project's tests/lint/typecheck
before fixes, so Step 5 could compare and label failures as pre-existing
vs regression. That was machinery in service of one bullet ("say so explicitly
if a check was already failing pre-simplification") that the agent couldn't
honestly satisfy without a baseline.
Simpler resolution: delete the bullet. Step 4 now just runs the project's
checks, surfaces any failure, and forbids relaxing assertions to make checks
pass. The agent can still mention an unrelated-looking failure organically
when the output makes it obvious; it just isn't instructed to make
attribution claims it can't back up. Net 15 lines lighter than the prior
approach and lands closer to where the skill was before the feedback round.
Before applying fixes, Step 3 now runs the project's existing test
suite, lint, and typecheck and records each result. Step 5 compares
post-fix results against the baseline and labels failures as
regression / pre-existing / unattributed / no-checks-configured
instead of guessing. Without a baseline, a pre-existing red check
was indistinguishable from a regression introduced by simplification,
defeating the skill's "preserves exact functionality" premise.
Three parallel reviewers (reuse, quality, efficiency) inspect the diff,
the orchestrator fixes their findings, and the project's existing checks
are rerun to verify behavior is preserved. Distinct from ce-code-review,
which targets correctness and security rather than simplification.
Agents spawned from LFG were blocking forever at the AskUserQuestion
prompt with no user present to respond. In mode:pipeline, default to
headless and skip step 2 entirely.
Bump 3.0.6 -> 3.0.7
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Port scan (find_free_port) only runs when PIPELINE_MODE=1
- Dev server auto-start only runs in pipeline mode; manual invocations
print a help message and stop
- LFG step 6 now passes mode:pipeline to ce-test-browser so parallel
agents claim non-colliding ports automatically
- Bump version 3.0.5 -> 3.0.6
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Always verify preferred port is free; scan upward until finding one
- Auto-start dev server (bin/dev / rails server / npm run dev) on the
claimed port if nothing is listening — no more "please start your server"
- Pass PORT= explicitly so parallel agents on the same machine never
collide on 3000
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add ce-commit-push-pr as step 7 so LFG ends with a pushed branch and open PR
- Remove optional ralph-loop step (step 1) -- simplifies the pipeline
- Renumber all steps and fix cross-references accordingly
- Bump version 3.0.3 -> 3.0.4
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>