Delete envelope and size-gate test files, remove resolve-base.sh
packaging tests from converter.test.ts — all tested features that
were removed in the SKILL.md rewrite.
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>
- Remove plugin version fields from marketplace.json -- canonical
versions live in each plugin's plugin.json. Duplicating them created
drift that release-please couldn't maintain.
- Remove version sync logic from metadata.ts (description sync kept)
- Fix release-preview test to compute expected versions dynamically
from current manifests instead of hardcoding them
* docs: capture codex skill prompt model
* fix: align codex workflow conversion
* chore: remove deprecated workflows:* skill aliases
The workflows:brainstorm, workflows:plan, workflows:work, workflows:review,
and workflows:compound aliases have been deprecated long enough. Remove them
and update skill counts (46 → 41) across plugin.json, marketplace.json,
README, and CLAUDE.md.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---------
Co-authored-by: Trevin Chow <trevin@trevinchow.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
* fix(kiro): parse .mcp.json wrapper key and support remote MCP servers
* refactor: extract unwrapMcpServers helper to deduplicate parser logic
Address review feedback by extracting the mcpServers unwrap logic
into a shared helper used by both loadMcpServers and loadMcpPaths.
Bump plugin to 2.39.0 with features from Matt Van Horn (@mvanhorn):
context budget precheck, plan sequence numbers, review serial mode,
agent-browser debugging commands, test-browser port detection, lfg
phase gating, and Context7 API key auth.
Also fixes MCP server merge order so plugin servers correctly
overwrite stale entries during sync.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
When Claude makes parallel tool calls and a PreToolUse hook command
fails, the thrown error can crash the entire batch, causing API 400
errors. Wrap generated tool.execute.before handlers in try-catch so
failures are logged but non-fatal.
Fixes#85
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The sync path's mergeJsonConfigAtKey had incoming entries overwriting
existing user entries on conflict. Reverse the spread order so user
config wins, matching the install path's existing behavior.
Also add merge feedback logging and a test for the sync merge path.
Fixes#125
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
OpenClaw rejects generated plugin manifests that omit configSchema, even for tool plugins with no user configuration. Always emit an empty object schema so converted installs boot cleanly.\n\nAdd converter and writer regression coverage for the manifest shape.\n\nFixes #224
Kieran Klaassen