build(deps): bump stream-json from 1.5.0 to 3.5.0 - #265
Conversation
Bumps [stream-json](https://github.com/uhop/stream-json) from 1.5.0 to 3.5.0. - [Commits](uhop/stream-json@1.5.0...3.5.0) --- updated-dependencies: - dependency-name: stream-json dependency-version: 3.5.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
added
dependencies
![lifeomic-review-bot[bot]](https://avatars.githubusercontent.com/u/29172293?s=60&v=4){kind=small}
| "stream-chain": "^2.1.0", | ||
| "stream-csv-as-json": "^1.0.1", | ||
| "stream-json": "^1.2.1", | ||
| "stream-json": "^3.5.0", |
There was a problem hiding this comment.
Blocking: stream-json@3.x is ESM-only, requires Node ≥22, and removed the CommonJS subpath stream-json/streamers/StreamValues (replaced by stream-json/streamers/stream-values.js). The CLI still require()s the old path from lib/cmds/fhir_cmds/ingest.js and lib/cmds/ontologies_cmds/import.js, causing MODULE_NOT_FOUND at load time (yarn test fails for fhir-test and ontologies.test). Either stay on 1.x or migrate those commands (and likely stream-chain to v4+) to the new ESM API before merging this bump.
This review was generated by review-bot.
There was a problem hiding this comment.
Scope — Dependabot bumps direct stream-json from ^1.5.0 to ^3.5.0 in package.json with a matching yarn.lock refresh. The change is dependency-only but crosses two major versions and breaks the CLI's stdin JSON ingestion paths.
[blocking] stream-json@3.x is ESM-only ("type": "module"), requires Node ≥22, and no longer exposes the CommonJS subpath stream-json/streamers/StreamValues. This repo still require()s that path from lib/cmds/fhir_cmds/ingest.js and lib/cmds/ontologies_cmds/import.js. Local yarn test fails at module load with MODULE_NOT_FOUND for both test/unit/commands/fhir-test.js and test/unit/commands/ontologies.test.js. Merge is blocked until the bump is constrained to a 1.x-compatible release or the consumers are migrated to the v3 ESM API (including likely stream-chain v4+ alignment).
CI — GitHub checks were still pending at review time; the failure reproduces locally and is independent of CI state.
Regression risk — High for fhir ingest and ontologies import until import paths, module system, and stream-chain peer alignment are addressed.
Upstream: uhop/stream-json (1.5.0→3.5.0; see compare view in PR body).
Note: Review generated using Cursor model
composer-2.5.
This review was generated by review-bot.
Bumps stream-json from 1.5.0 to 3.5.0.
Commits
4b41d26New version: 3.5.0.a869fb9AddedmaxDepthoption.bfe59b8Bump the npm-deps group across 1 directory with 4 updates (#209)63c0663Bump the npm-deps group with 2 updates (#206)74f792cBump actions/checkout from 6 to 7 in the gh-actions group (#207)d7317a8New version: 3.4.0.bc18492Fixed an unsettled promise.7ba2b98Prepare for new version.8bc469fUpdated wiki.a6393cfReorg in file utilities.Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.