Skip to content

fix: use npx @red-codes/agentguard in user-facing docs and messages#888

Merged
jpleva91 merged 2 commits intomainfrom
fix/848-npx-scoped-package-name
Mar 25, 2026
Merged

fix: use npx @red-codes/agentguard in user-facing docs and messages#888
jpleva91 merged 2 commits intomainfrom
fix/848-npx-scoped-package-name

Conversation

@jpleva91
Copy link
Copy Markdown
Collaborator

Closes #848

Implementation Summary

What changed:

  • apps/cli/src/postinstall.ts: Post-install hint messages now show npx @red-codes/agentguard claude-init / copilot-init instead of the broken unscoped form
  • apps/cli/src/commands/demo.ts: Get-started output updated to use the scoped package name
  • apps/cli/tests/cli-demo.test.ts: Updated test expectation to match new output
  • CLAUDE.md: Quick Start code examples updated

Note: Generated hook commands continue to use npx --no-install agentguard (resolves local binary via node_modules/.bin without registry fallback) — this is intentional and tested separately in cli-postinstall.test.ts.

How to verify:

  1. pnpm install && pnpm build --filter=@red-codes/agentguard — builds cleanly
  2. pnpm test --filter=@red-codes/agentguard — 760 tests pass
  3. Review post-install output: customize hints now reference @red-codes/agentguard
  4. Run agentguard demo — get-started line now shows correct scoped name

Tier C scope check:

  • Files changed: 4 (limit: 5)
  • Lines changed: ~8 (limit: 300)
  • Breaking changes: None

Tier C implementation by copilot-cli — AgentGuard three-tier governance

…loses #848)

Update all user-facing references to the CLI from 'npx agentguard' (which
fails with HTTP 404 because no unscoped package is registered) to the correct
scoped form 'npx @red-codes/agentguard'.

Changes:
- apps/cli/src/postinstall.ts: post-install wizard hints
- apps/cli/src/commands/demo.ts: get-started output
- apps/cli/tests/cli-demo.test.ts: update test expectation to match
- CLAUDE.md: Quick Start code examples

Note: generated hook commands continue to use 'npx --no-install agentguard'
(which resolves the local binary via node_modules/.bin without registry
fallback) — this is intentional and tested separately.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
jpleva91 pushed a commit that referenced this pull request Mar 25, 2026
PRs #869 and #870 both merged this cycle. New PR #888 (fix/npx-scoped-package-name) open with lint CI failure — assigned to senior-coder. No P0/P1 issues. Health: yellow.

https://claude.ai/code/session_018LQYdGroNHJMB7pmDbUJBW
Prettier formatting applied to apps/cli/src/postinstall.ts to fix CI
format check failure on run 23547828259.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jpleva91
Copy link
Copy Markdown
Collaborator Author

AgentGuard CI Triage Bot (claude-code:opus:developer) — automated fix applied

Diagnosis

  • Failed run: 23547715374
  • Category: format
  • Root cause: apps/cli/src/postinstall.ts had Prettier formatting issues (inconsistent code style)
  • Governance context: no governance denials detected

Fix Applied

Ran pnpm format:fix to apply Prettier formatting to apps/cli/src/postinstall.ts. Only the single file flagged by CI was modified.

Verification

Full suite passed locally: build, typecheck, lint, format, test (760 tests across 43 test files)


Automated fix by triage-failing-ci skill on 2026-03-25T15:08:00Z

jpleva91 added a commit that referenced this pull request Mar 25, 2026
PRs #869 and #870 both merged this cycle. New PR #888 (fix/npx-scoped-package-name) open with lint CI failure — assigned to senior-coder. No P0/P1 issues. Health: yellow.

https://claude.ai/code/session_018LQYdGroNHJMB7pmDbUJBW

Co-authored-by: Claude <noreply@anthropic.com>
jpleva91 pushed a commit that referenced this pull request Mar 25, 2026
Health: GREEN. Previous blockers resolved. Go kernel landed,
v2.6.0 released. 10 PRs merged since last cycle. 1 open PR (#888)
CI green, awaiting review. Sprint goal updated to post-release
stabilization.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jpleva91 jpleva91 merged commit eb9253a into main Mar 25, 2026
5 checks passed
jpleva91 pushed a commit that referenced this pull request Mar 25, 2026
All 4065 tests passing across 18 packages. No regressions.
PR #888 merged — blocker resolved, health upgraded to green.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jpleva91 pushed a commit that referenced this pull request Mar 25, 2026
Health: GREEN. Previous blockers resolved. Go kernel landed,
v2.6.0 released. 10 PRs merged since last cycle. 1 open PR (#888)
CI green, awaiting review. Sprint goal updated to post-release
stabilization.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jpleva91 added a commit that referenced this pull request Mar 25, 2026
Health: GREEN. Previous blockers resolved. Go kernel landed,
v2.6.0 released. 10 PRs merged since last cycle. 1 open PR (#888)
CI green, awaiting review. Sprint goal updated to post-release
stabilization.

Co-authored-by: Jared <jared@agentguard.dev>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jpleva91 pushed a commit that referenced this pull request Mar 25, 2026
- Health: GREEN — all clear
- Merged PR #905 (previous EM report, CI green)
- PR #888 (npx scoped fix) confirmed merged
- 0 open PRs, 0 P0/P1 issues, 0 blockers
- Sprint: maintenance mode, v3.0 ROADMAP issues open for planning

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

Bug: npx agentguard fails with 404 — unscoped package name not registered on npm

1 participant