Skip to content

fix(config): skip skills install without npx#17

Merged
laipz8200 merged 2 commits into
mainfrom
laipz8200/config-init-npx-skip
May 14, 2026
Merged

fix(config): skip skills install without npx#17
laipz8200 merged 2 commits into
mainfrom
laipz8200/config-init-npx-skip

Conversation

@laipz8200
Copy link
Copy Markdown
Owner

Summary

  • Check for npx before prompting for or running the Agent Skills installer.
  • Mark the Agent Skills setup step as failed and skipped when npx is unavailable, while keeping the saved config usable.
  • Align dynamic setup colors so running steps are blue, completed steps are green, and failed steps are red.
  • Update English and Chinese docs for the skipped installer behavior.

Impact

Users without Node/npm can still complete confluence-cli config init; only the optional Agent Skills package install is skipped.

Root Cause

The setup flow assumed npx was available and treated installer startup failures as config init failures after the config had already been saved.

@laipz8200 laipz8200 marked this pull request as ready for review May 14, 2026 20:58
@laipz8200 laipz8200 enabled auto-merge (squash) May 14, 2026 20:59
@laipz8200 laipz8200 merged commit 8f10a80 into main May 14, 2026
1 check passed
@laipz8200 laipz8200 deleted the laipz8200/config-init-npx-skip branch May 14, 2026 20:59
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