Repository navigation
fix(docs): document the install source that actually resolves - #16
Conversation
`pm install github.com/unbraind/pm-github` — the command this README has always
advertised — fails:
Extension entry file is missing at
"/tmp/pm-extension-source-XXXXXX/dist/index.js"
`dist/` is in .gitignore and no build output is committed, and pm does not
build when installing from a GitHub source; it copies the repository as-is.
So the entry file named by manifest.json has never been present for anyone
installing this package the documented way.
Fixed by documenting `pm install npm:pm-github`, verified end to end. npm is this
package's real distribution channel — the Daily Release workflow publishes it
and `prepack` builds `dist` into the tarball — so the npm source needs no
build output in git.
Note the `npm:` prefix is required. A bare `pm install pm-github` fails with
`Local extension source does not exist`, because a bare target resolves only
a local directory or a bundled alias, never the registry.
Considered and rejected: committing `dist/`. It puts build artifacts under
version control, invites drift (pm-web already tracks 72 dist files against
the 30 a clean build produces), and adds a conflict surface to the
multi-agent merge workflow this fleet is built around.
Tracked in pm-github-q6ql.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
|
🧙 Sourcery has finished reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Warning Review limit reached
Next review available in: 9 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Sourcery asked for this on the pm-beads and pm-csv PRs: the README showed the working command without saying why the two obvious alternatives do not work. A bare `pm install <pkg>` resolves only a local directory or a bundled alias and never reaches the registry, and a `github.com/unbraind/<pkg>` source cannot work because pm copies a GitHub source as-is without building it while this repository does not commit `dist/`. Someone hitting either failure now has the answer in the README instead of an entry-file-missing error. Applied to all 6 repos in this batch for consistency, not just the two Sourcery happened to review. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Updated: the README now explains why the This came from @sourcery-ai on the sibling pm-beads and pm-csv PRs — it asked for a note on why the prefix is needed and that GitHub sources are not built on install. Applying it across all 6 repos in this batch so the docs stay identical. Recording the two failure modes, since neither error explains itself:
|
The bug
The install command this README has always advertised does not work:
dist/is in.gitignoreand no build output is committed, and pm does not build when installing from a GitHub source — it copies the repository as-is. So the entry file named bymanifest.jsonhas never been present for anyone installing this package the documented way.Verified on pm 2026.7.25 in a clean
pm init --workspacetracker.The fix
Document
pm install npm:pm-github, verified end to end (ok: true,failed_count: 0, the command group resolves afterwards). npm is this package's real distribution channel — the Daily Release workflow publishes it andprepackbuildsdistinto the tarball — so the npm source needs no build output in git.The
npm:prefix is required. A barepm install pm-githubfails withLocal extension source does not exist, because a bare target resolves only a local directory or a bundled alias, never the registry. (That distinction is easy to miss:pm install todosandpm install beadsdo succeed, because pm-cli bundles@unbrained/pm-todosand@unbrained/pm-beadsunder those aliases — different packages from these repos.)Considered and rejected: committing
dist/It would also fix the GitHub source, and 9 sibling repos do exactly that. Rejected here because it puts build artifacts under version control, invites drift — pm-web already tracks 72
distfiles against the 30 a clean build produces — and adds a conflict surface to the multi-agent merge workflow this fleet is built around.Scope
This is one of 6 identical PRs.
pm-beads,pm-brief,pm-csv,pm-github,pm-linearandpm-todosall shipped the same broken install line.pm items
Summary by Sourcery
Clarify the documented installation method to use the npm registry source that actually resolves, and record the associated pm issue in project history.
Bug Fixes:
Documentation:
pm install npm:pm-github --globalinstead of the non-functional GitHub source command.Chores:
Summary by cubic
Fix README install instructions to use the npm source
pm install npm:pm-github --global, replacing the broken GitHub-based command that failed due to missing build output. Add a note explaining that thenpm:prefix is required (barepm install pm-githubonly resolves local/bundled aliases) and why the GitHub source cannot work (pm doesn’t build from GitHub and this repo doesn’t commitdist/).Written for commit e4d1309. Summary will update on new commits.