Skip to content

feat(cli)!: remove sync command#2669

Merged
Patrick-Erichsen merged 5 commits into
mainfrom
pe/claw-336-remove-sync-command
Jun 16, 2026
Merged

feat(cli)!: remove sync command#2669
Patrick-Erichsen merged 5 commits into
mainfrom
pe/claw-336-remove-sync-command

Conversation

@Patrick-Erichsen

@Patrick-Erichsen Patrick-Erichsen commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

"## Summary\n\n- remove the public clawhub sync command and its broad local-root scanning implementation\n- make clawhub skill publish <path> skip unchanged content, default new skills to 1.0.0, and default changed skills to the next patch when --version is omitted\n- restore the reusable skill-publish workflow as a thin catalog enumerator that invokes ordinary skill publish once per skill folder\n- prepare the ClawHub CLI 0.22.0 release\n\n## Proof\n\nBuilt CLI against a local registry fixture:\n\ntext\n$ clawhub skill publish new-skill --dry-run --json\n{\n \"status\": \"would-publish\",\n \"slug\": \"new-skill\",\n \"version\": \"1.0.0\",\n \"latestVersion\": null\n}\n\n$ clawhub skill publish changed-skill --dry-run --json\n{\n \"status\": \"would-publish\",\n \"slug\": \"changed-skill\",\n \"version\": \"2.0.1\",\n \"latestVersion\": \"2.0.0\"\n}\n\n$ clawhub sync\nerror: unknown command 'sync'\nexit=1\n\n$ clawhub skill publish --help | grep -E -- \"--version|--dry-run|--json|--bump\"\n --version <version> Explicit version (defaults to 1.0.0 or next patch)\n --dry-run Preview without publishing\n --json Output JSON\n\n\nReusable workflow runner against a two-skill catalog fixture:\n\njson\n{\n \"summary\": {\n \"wouldPublish\": 1,\n \"published\": 0,\n \"alreadySynced\": 1,\n \"skipped\": 0,\n \"failed\": 0\n },\n \"wouldPublish\": [{ \"slug\": \"changed\", \"version\": \"2.0.1\" }],\n \"alreadySynced\": [{ \"slug\": \"unchanged\", \"version\": \"1.0.0\" }]\n}\n\n\n## Tests\n\n- bun run ci:static\n- bun run ci:unit\n- bun run ci:packages\n- bun run ci:types-build\n- bun run ci:e2e-http\n- actionlint .github/workflows/skill-publish.yml\n- node scripts/clawhub-cli-npm-release-check.mjs --tag v0.22.0\n- .agents/skills/autoreview/scripts/autoreview --mode branch (clean after fixing resolver-404 finding)

  • .agents/skills/autoreview/scripts/autoreview --mode local (compatibility finding consciously rejected per explicit decision to remove the bump input)\n"

@Patrick-Erichsen Patrick-Erichsen requested a review from a team as a code owner June 16, 2026 01:08
@vercel

vercel Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clawhub Ready Ready Preview, Comment Jun 16, 2026 1:16am

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