Skip to content

fix: use npx @insforge/cli in all user-facing messages#46

Merged
tonychang04 merged 1 commit intomainfrom
fix/use-npx-in-cli-messages
Apr 1, 2026
Merged

fix: use npx @insforge/cli in all user-facing messages#46
tonychang04 merged 1 commit intomainfrom
fix/use-npx-in-cli-messages

Conversation

@tonychang04
Copy link
Copy Markdown
Contributor

@tonychang04 tonychang04 commented Apr 1, 2026

Summary

  • Replaced all bare insforge command references with npx @insforge/cli in user-facing messages
  • Updated error messages, help text, deployment status output, and migration instructions across 6 files
  • Users may not have the CLI installed globally, so npx @insforge/cli is the correct recommendation

Test plan

  • Run npx @insforge/cli docs --help and verify examples show npx @insforge/cli
  • Trigger an auth error and verify message says npx @insforge/cli login
  • Trigger a project-not-linked error and verify message says npx @insforge/cli link

🤖 Generated with Claude Code

Note

Replace insforge with npx @insforge/cli in all user-facing CLI messages

Updates all user-facing strings across commands and error classes to reference npx @insforge/cli instead of insforge. This affects timeout guidance in deployment commands, auth error messages in AuthError and ProjectNotLinkedError, login prompts in credentials.ts, and help examples in the docs command.

Macroscope summarized 15a85d3.

Summary by CodeRabbit

Release Notes

  • Documentation
    • Updated CLI command guidance and error messages throughout the application to reference npx @insforge/cli instead of insforge for all commands, including deployments status, docs, link, and login.

Users may not have the CLI installed globally, so recommend
`npx @insforge/cli` instead of bare `insforge` in error messages,
help text, and status output.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tonychang04 tonychang04 enabled auto-merge (squash) April 1, 2026 19:56
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

Walkthrough

The PR updates CLI guidance messages across 6 files to use the npx @insforge/cli`` invocation pattern instead of the bare insforge command. Changes affect error messages, help text, and user-facing guidance in commands and library error classes. No logic or control flow alterations.

Changes

Cohort / File(s) Summary
Command Guidance Updates
src/commands/create.ts, src/commands/deployments/deploy.ts, src/commands/docs.ts, src/commands/info.ts
Updated user-facing guidance in CLI commands to use npx @insforge/cli`` invocation pattern instead of bare insforge command across deployment status checks, documentation help text, and project linking instructions.
Error Message Updates
src/lib/credentials.ts, src/lib/errors.ts
Updated default error messages in authentication and project linking error classes to reference npx @insforge/cli login and npx @insforge/cli link instead of bare invocation patterns, including constructor default parameters.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • jwfing
  • CarmenDou

Poem

🐰 A rabbit hops through command lines so bright,
Trading bare insforge for npx in the night,
Each message now shines with the proper incant,
Installing sans globals—a modernist chant!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and specifically summarizes the main change: replacing bare insforge command references with npx @insforge/cli`` throughout user-facing messages in the codebase.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/use-npx-in-cli-messages

Comment @coderabbitai help to get the list of available commands and usage tips.

@tonychang04 tonychang04 merged commit e98908a into main Apr 1, 2026
2 of 3 checks passed
@tonychang04 tonychang04 mentioned this pull request Apr 1, 2026
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.

2 participants