Skip to content

Commit fdb924c

Browse files
committed
docs(cli): drop the dead tracker number from bin/run.js's docblock and keep the sentence
The docblock above the process.stderr error listener ended a sentence with " — #14874", a tracker number that no longer resolves. No commit decided that number, so it cannot be re-anchored to one; the sentence already states the measured fact in words ("files names only dist, but npm packs a bin target regardless"), so only the dash and the number go. This is ruling C+D's form D for a line that ships in the tarball. Comment only: the non-comment token stream of bin/run.js is identical before and after (179 tokens), and the file keeps its 258 lines. A patch changeset is owed because npm packs a bin target regardless of files. Claude-Session: https://claude.ai/code/session_01VvcEokUG1tvVxkceYfR5XB Co-authored-by: Claude <noreply@anthropic.com>
1 parent 013f97d commit fdb924c

2 files changed

Lines changed: 14 additions & 1 deletion

File tree

‎.changeset/20594-cli-bin-form-d.md‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
---
2+
'@objectstack/cli': patch
3+
---
4+
5+
A docblock line in `@objectstack/cli`'s `bin/run.js` no longer cites a tracker number
6+
7+
The docblock above `bin/run.js`'s `process.stderr` `error` listener ended a
8+
sentence with a tracker number that no longer resolves on GitHub. The number is
9+
gone and the sentence stays: `files` names only `dist`, but npm packs a `bin`
10+
target regardless, which is the measured fact the number was pointing at. The
11+
file ships because of that same packing rule, which is why this is a release
12+
note at all. Comment only: no command, flag, exit code, error code, export or
13+
runtime behaviour changes.

‎packages/cli/bin/run.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,7 @@ try {
222222
* serves, exit 0 at a 20 s SIGTERM, having written 7926 bytes over 16.6 s. So
223223
* the crash costs the run at its FIRST diagnostic line and 20 of its 21 stderr
224224
* writes, on the entry point a customer's install actually runs (`files` names
225-
* only `dist`, but npm packs a `bin` target regardless — #14874).
225+
* only `dist`, but npm packs a `bin` target regardless).
226226
*
227227
* ⛔ Deliberately NOT narrowed to `error.code === 'EPIPE'`, for the reason
228228
* `bin/run-dev.js` records: the reason to tolerate is not WHICH error it is.

0 commit comments

Comments
 (0)