Skip to content

Launch foundation: scoped npm, global profile, better errors - #1

Merged
shilpi1958 merged 1 commit into
mainfrom
launch/foundation-scoped-global-profile
Jul 30, 2026
Merged

Launch foundation: scoped npm, global profile, better errors#1
shilpi1958 merged 1 commit into
mainfrom
launch/foundation-scoped-global-profile

Conversation

@shilpi1958

Copy link
Copy Markdown
Owner

Summary

  • Publish package as @shilpi1958/pr-explainer (CLI binary stays pr-explainer); Action installs the scoped name
  • Add pr-explainer init and default profile/explainers under ~/.pr-explainer/ so you don’t need a per-repo profile
  • Improve errors for missing PRs, unmerged PRs, Claude spending caps/auth, and empty/missing profiles
  • Update README troubleshooting + launch/design install copy

Test plan

  • npm install -g @shilpi1958/pr-explainer (or npm install -g . from this branch)
  • pr-explainer init creates ~/.pr-explainer/learning-profile.md
  • From /tmp, pr-explainer https://github.com/facebook/react/pull/32 works with no local profile
  • Bare pr-explainer 99999 in this repo shows the “use a URL” hint
  • Unmerged PR URL explains merged-only requirement

Made with Cursor

… errors.

Bare npm name was taken; scoped package plus pr-explainer init (~/.pr-explainer) so the CLI works from any directory, with actionable gh/claude failure messages for launch.

Co-authored-by: Cursor <cursoragent@cursor.com>
@shilpi1958
shilpi1958 merged commit 55f6e04 into main Jul 30, 2026
@shilpi1958
shilpi1958 deleted the launch/foundation-scoped-global-profile branch July 30, 2026 16:08
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