Repository navigation
Install a Node CLI as its bin command and point to its help instead of copying it - #29
Merged
Merged
Conversation
pekral
force-pushed
the
fix/24-node-cli-install
branch
from
November 12, 2026 13:31
d5569d3 to
b45af16
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this changes
A Node CLI whose
package.jsonhas abinfield is installed as that command:npm install --global <name>when publication is evidenced, otherwisenpm install --global .from a checkout of the evidenced remote — nevernode bin/<file>.js. A CLI's help output is pointed to by its help command and never transcribed into anOptions/Usagesection.Why
Scenario 3 failed (#24): the run installed via
git clone+node bin/slugify.jsand copied the full help text into the README. The ecosystem table did not say which install command abinCLI without publish evidence gets, and the "pointer to the full help output" rule was not explicit about transcription.Closes #24
Checks
node --test tests/repository.test.mjspassesnode --test tests/evals/*.test.mjspassesCHANGELOG.mdupdated under the current unreleased headingSKILL.mdis changed everywhere it appears inreferences/(rule lives inreadme-structure.md; mirrored invalidation-checklist.mditem 10;SKILL.mdunchanged)tests/scenarios.md, and the run recorded in itshost coverage log
Claims this adds to the documentation
claude -p "Write a README for this repository." --permission-mode acceptEdits --setting-sources project, skill from this branch). The generated README installs withnpm install --global ., showsslugify "Hello, World!"→hello-worldfrom the tests, and referencesslugify --helpwithout transcribing it.Note on the fixture (not changed here):
bin/slugify.js:29only runs whenprocess.argv[1]ends inslugify.js, so the linkedslugifycommand prints nothing — confirmed by invoking the script through a symlink namedslugify. The scenario 3 fixture should use a realpath /import.meta.urlcheck so thebininstall path it is meant to prove actually works.