fix: lowercase npm package name to @unabandoned/jsonstream - #4
Merged
Conversation
npm rejects capital letters in new package names, so "@unabandoned/JSONStream"
cannot be published (and cannot even be resolved as an npm: alias — it fails
name validation with "name can no longer contain capital letters"). Rename the
published package to the lowercase "@unabandoned/jsonstream".
Consumers keep the dependency key "JSONStream" (aliased to
npm:@unabandoned/jsonstream), so node_modules/JSONStream and require('JSONStream')
are unchanged. The GitHub repo name and the JSONStream.* API are unaffected.
This was referenced Aug 14, 2026
00o-sh
added a commit
that referenced
this pull request
Aug 14, 2026
🤖 I have created a release *beep* *boop* --- ## [1.4.0](jsonstream-v1.3.5...jsonstream-v1.4.0) (2026-08-14) ### Features * add option for emitting keys ([9aa2446](9aa2446)) * fork and onboard @unabandoned/JSONStream (vendor through, node:test suite) ([#1](#1)) ([f11ffef](f11ffef)) ### Bug Fixes * lowercase npm package name to @unabandoned/jsonstream ([#4](#4)) ([76dfce6](76dfce6)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
00o-sh
added a commit
to unabandoned/deps-sort
that referenced
this pull request
Aug 14, 2026
Repoints the `JSONStream` runtime dependency to `npm:@unabandoned/jsonstream@^1`, so the abandoned upstream (and its abandoned `through` dependency) leaves deps-sort' transitive tree. Part of adopting JSONStream into the program — see unabandoned/JSONStream#1 / unabandoned/JSONStream#4 and the tracking issue unabandoned/browser-pack#3. The dependency **key** stays `JSONStream` (aliased to `npm:@unabandoned/jsonstream`), so `node_modules/JSONStream` and `require('JSONStream')` are unchanged — the fork is a drop-in with the same `JSONStream.parse` / `JSONStream.stringify` API, no code changes. ## Verification Installed deps-sort's full tree with `@unabandoned/jsonstream` (packed from the fork) standing in for the alias and ran the suite: **9 tests, 9 pass, 0 fail** on Node 22.22 and Node 24.19. `require('JSONStream')` resolved to the fork. ## Ordering note⚠️ **CI here stays red until `@unabandoned/jsonstream` is published to npm** (unabandoned/JSONStream#4 → release → publish). `npm install` can't resolve the alias until then; this is the expected publish-ordering dependency, verified above via a local install of the exact package contents. Holding ready-for-review in the meantime. --- _Generated by [Claude Code](https://claude.ai/code/session_01Wa4Hra9ctCDzbpPUaQB9mG)_
00o-sh
added a commit
to unabandoned/insert-module-globals
that referenced
this pull request
Aug 14, 2026
Repoints the `JSONStream` runtime dependency to `npm:@unabandoned/jsonstream@^1`, so the abandoned upstream (and its abandoned `through` dependency) leaves insert-module-globals' transitive tree. Part of adopting JSONStream into the program — see unabandoned/JSONStream#1 / unabandoned/JSONStream#4 and the tracking issue unabandoned/browser-pack#3. The dependency **key** stays `JSONStream` (aliased to `npm:@unabandoned/jsonstream`), so `node_modules/JSONStream` and `require('JSONStream')` are unchanged — the fork is a drop-in with the same `JSONStream.parse` / `JSONStream.stringify` API, no code changes. ## Verification Installed the full tree with `@unabandoned/jsonstream` (packed from the fork) standing in for the alias and ran the suite: **17 pass, 0 fail, 1 skip** (the pre-existing `windows partitions` platform skip) on Node 22.22 and Node 24.19. `require('JSONStream')` resolved to the fork. ## Ordering note⚠️ **CI here stays red until `@unabandoned/jsonstream` is published to npm** (unabandoned/JSONStream#4 → release → publish). `npm install` can't resolve the alias until then; this is the expected publish-ordering dependency, verified above via a local install of the exact package contents. Holding ready-for-review in the meantime. --- _Generated by [Claude Code](https://claude.ai/code/session_01Wa4Hra9ctCDzbpPUaQB9mG)_
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.
Follow-up to #1. The onboarding merged with the package named
@unabandoned/JSONStream, but npm forbids uppercase letters in new package names — that name can't be published, and it can't even be resolved as annpm:alias (npm fails name validation with "name can no longer contain capital letters", which is exactly what brokenpm installin the first repoint PR).This renames the published package to the lowercase
@unabandoned/jsonstream.What does not change:
unabandoned/JSONStream.JSONStream, aliased tonpm:@unabandoned/jsonstream@^1, sonode_modules/JSONStreamandrequire('JSONStream')are unchanged — no consumer code changes.JSONStream.parse/JSONStream.stringifyAPI is untouched.Also updates
packagein.unabandoned.ymlto match (validated).node --test test/*.jsstill passes (28/28) — the rename is inert to behaviour.Generated by Claude Code