The SKILL.md rewrite dropped phases that used these files.
No remaining references in the skill.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Strip 90% of the skill: checklist generation, size gates, stacked-PR
seeds, replan escalation, dispatch matrices, elaborate envelopes,
headless mode ceremony.
What's left: start the dev server, open the browser, iterate with the
user. 753 -> 82 lines.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Strip Phase 2 (entry gate) entirely: fork-PR trust boundary, review
artifact SHA binding, verdict check, CI status check, and merge-base
offer. These belong in a deploy/merge gate, not a polish skill.
Polish is "can this be better?" — you shouldn't need a green CI pass
or a review artifact to look at your feature in a browser.
Removes trust-fork:1 and accept-stale-review:1 arguments.
Renumbers phases: 0 (input) -> 1 (branch) -> 2 (dev server) ->
3 (checklist) -> 4 (envelope).
866 -> 753 lines (-113 lines of gate logic).
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Combine two sed forks into one in parse_env_port (3 forks -> 2)
- Eliminate double-grep on framework config files (read file once)
- Remove WHAT-narrating comments, keep only the WHY comment
- Fix Unicode arrows in dev-server-next.md to ASCII (AGENTS.md convention)
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Address PR #568 feedback from @tmchow:
1. Framework coverage: add Nuxt, Astro, Remix, SvelteKit signatures to
detect-project-type.sh with recipe reference files for each.
2. Monorepo detection: when root detection finds zero matches, probe
subdirectories up to depth 3 for framework signatures. Output grammar
extended with type@dir and multiple:type@dir,type@dir forms.
3. Package-manager detection: new resolve-package-manager.sh script with
lockfile priority cascade (pnpm > yarn > bun > npm).
4. Port resolution: new resolve-port.sh with 8-probe cascade replacing
the inline bash in SKILL.md. Adds framework config parsing, .env
quote/comment stripping, and removes unreliable AGENTS.md grep.
Includes review-autofix: empty-string guard on framework config port
validation (P0), compact -p3000 Procfile syntax support (P1).
128 tests pass (72 new across 3 test files).
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Adds /ce:polish-beta — the polish phase that runs AFTER /ce:review but
BEFORE merge. Starts a dev server from .claude/launch.json, generates a
user-testable checklist, lets the user mark items for fixes, and
dispatches polish sub-agents in parallel. Emits stacked-PR seeds for
oversized items and a replan seed when the whole batch is too big to
polish as one unit.
Five phases: input triage, branch/PR acquisition, entry gate, dev-server
lifecycle, checklist+size-gate+dispatch, envelope. Headless mode mirrors
ce:review's output envelope shape and terminates with `Polish complete`.
Cross-cutting changes:
- ce:review now writes metadata.json alongside findings so polish can
verify the artifact matches the current branch and HEAD.
- Tests pin the intentional resolve-base.sh duplication between
ce-review and ce-polish-beta (self-contained skill rule).
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Kieran Klaassen