Skip to content

fix: make Claude skill install opt-in#559

Open
ekkoitac wants to merge 1 commit into
ax-llm:mainfrom
ekkoitac:fix/issue-483-opt-in-claude-skills
Open

fix: make Claude skill install opt-in#559
ekkoitac wants to merge 1 commit into
ax-llm:mainfrom
ekkoitac:fix/issue-483-opt-in-claude-skills

Conversation

@ekkoitac

@ekkoitac ekkoitac commented Jul 3, 2026

Copy link
Copy Markdown

Fixes #483

What changed:

  • Stop adding scripts.postinstall to the published dist package.json when src/ax/scripts exists.
  • Keep the explicit ax CLI entry, copied skills, and bundled scripts/postinstall.mjs available for opt-in use through npx @ax-llm/ax setup-claude and npx @ax-llm/ax remove-claude.
  • Add a postbuild regression test for the case where package scripts are bundled.

Validation:

  • npm ci completed.
  • npx vitest run scripts/postbuild.test.js passed before the change with 3 tests.
  • npx vitest run scripts/postbuild.test.js passed after the change with 4 tests.
  • npx biome format scripts/postbuild.js scripts/postbuild.test.js reported no fixes.
  • git diff --check passed.
  • Manual postbuild inspection of src/ax/dist/package.json: scripts is absent, bin.ax still points to ./cli/index.mjs, dist/scripts/postinstall.mjs is still copied, and 15 skill markdown files are still copied under dist/skills.

Not run:

  • Full npm run test; this patch only changes postbuild package metadata and the matching regression test.

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.

Auto-installing Claude skill should be opt-in, not automatic

1 participant