Skip to content

chore: update dependencies in package.json - #59

Merged
sontrinh16 merged 4 commits into
chains/realiofrom
feat/update-package
Aug 21, 2026
Merged

chore: update dependencies in package.json#59
sontrinh16 merged 4 commits into
chains/realiofrom
feat/update-package

Conversation

@lichdu29

@lichdu29 lichdu29 commented Jun 8, 2026

Copy link
Copy Markdown
  • upgraded @cosmjs/stargate from ^0.33.0 to ^0.39.0
  • downgraded @socialgouv/matomo-next from ^1.6.1 to ^1.2.1
  • downgraded ethereumjs-util from ^7.1.5 to ^7.0.2
  • upgraded @graphql-codegen packages to major versions:
    • @graphql-codegen/cli from ^5.0.5 to ^7.1.2
    • @graphql-codegen/client-preset from ^3.0.1 to ^6.0.1
    • @graphql-codegen/fragment-matcher from ^4.0.1 to ^7.0.1
    • @graphql-codegen/typescript from ^3.0.4 to ^6.0.2
    • @graphql-codegen/typescript-operations from ^3.0.4 to ^6.0.3
    • @graphql-codegen/typescript-react-apollo from ^3.3.7 to ^4.4.2

- upgraded @cosmjs/stargate from ^0.33.0 to ^0.39.0
- downgraded @socialgouv/matomo-next from ^1.6.1 to ^1.2.1
- downgraded ethereumjs-util from ^7.1.5 to ^7.0.2
- upgraded @graphql-codegen packages to major versions:
  - @graphql-codegen/cli from ^5.0.5 to ^7.1.2
  - @graphql-codegen/client-preset from ^3.0.1 to ^6.0.1
  - @graphql-codegen/fragment-matcher from ^4.0.1 to ^7.0.1
  - @graphql-codegen/typescript from ^3.0.4 to ^6.0.2
  - @graphql-codegen/typescript-operations from ^3.0.4 to ^6.0.3
  - @graphql-codegen/typescript-react-apollo from ^3.3.7 to ^4.4.2
karlkeppner1 added a commit that referenced this pull request Aug 7, 2026
Ports the dependency upgrades from #59 onto chains/realio-testnet so they
soak on testnet before promoting to chains/realio.

#59 targets chains/realio and cannot simply be retargeted: the two chain
branches have diverged in parallel (39 commits each way, 67 files
differing), so changing its base would recompute the diff to 40 commits /
78 files and sweep in every chains/realio-only change. Cherry-picking
conflicts on package-lock.json, which differs +106/-501 between branches.
So the package.json intent is reapplied here and the lockfile regenerated
on this base.

  @cosmjs/stargate                         ^0.33.0 -> ^0.39.0
  @graphql-codegen/cli                     ^5.0.5  -> ^7.1.2
  @graphql-codegen/client-preset           ^3.0.1  -> ^6.0.1
  @graphql-codegen/fragment-matcher        ^4.0.1  -> ^7.0.1
  @graphql-codegen/typescript              ^3.0.4  -> ^6.0.2
  @graphql-codegen/typescript-operations   ^3.0.4  -> ^6.0.3
  @graphql-codegen/typescript-react-apollo ^3.3.7  -> ^4.4.2

Two changes in #59 are deliberately NOT carried over, as they move
backwards and look unintentional:

  ethereumjs-util          ^7.1.5 -> ^7.0.2
  @socialgouv/matomo-next  ^1.6.1 -> ^1.2.1

Lockfile regenerated with npm 11.19.0 so the min-release-age=3 cooldown in
.npmrc actually applies -- that option needs npm >= 11.10.0 and is ignored
by older npm. Verified npm ci --dry-run under npm 10.9.2 (what node 23 in
common-test ships) to rule out lockfile-format skew.

Re-resolving the tree also clears the three critical advisories currently
failing npm audit on this branch (next, shell-quote, tar).

Co-authored-by: Claude Opus 5 (1M context) <[email protected]>
Brings this PR to dependency parity with chains/realio-testnet, where the
same upgrades landed as #61 and soaked.

Dropped two changes from the original commit that moved backwards and
appear unintentional:

  ethereumjs-util          ^7.0.2 -> ^7.1.5  (restored)
  @socialgouv/matomo-next  ^1.2.1 -> ^1.6.1  (restored)

The seven genuine upgrades are unchanged and now match testnet exactly:
@cosmjs/stargate ^0.39.0 and the six @graphql-codegen majors.

Also carries the security fixes that landed on testnet as #60, without
which npm audit --audit-level=critical fails on this base:

  overrides.tar          ^7.5.22   (GHSA-r292-9mhp-454m and others)
  overrides.shell-quote  ^1.10.0   (command injection)
  next                   ^15.5.22  (GHSA-9qr9-h5gf-34mp, RCE)

package.json is now identical to chains/realio-testnet for all ten
entries plus overrides.

Lockfile regenerated on the chains/realio base with npm 11.19.0 so the
min-release-age=3 cooldown in .npmrc applies. Verified:

  npm ci --dry-run under npm 10.9.2 (node 23 in common-test)  exit 0
  npm audit --audit-level=critical                            exit 0
  0 critical / 15 high (was 2 critical / 16 high)
  docker build of the full image, including npm run build:next

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
@karlkeppner1

Copy link
Copy Markdown

Pushed a commit aligning this PR with chains/realio-testnet, where the same upgrades landed as #61 and have now soaked. @lichdu29 — flagging two changes I reverted, in case they were deliberate.

Reverted (looked unintentional — both moved backwards)

package this PR had restored to
ethereumjs-util ^7.0.2 ^7.1.5
@socialgouv/matomo-next ^1.2.1 ^1.6.1

Moving backwards on a range can reintroduce advisories a later release fixed. If these were intentional, say so and I'll put them back.

Unchanged

The seven genuine upgrades are exactly as you had them, and now match testnet: @cosmjs/stargate ^0.39.0 plus the six @graphql-codegen majors.

Added

chains/realio never received the security fixes that landed on testnet as #60, and without them npm audit --audit-level=critical fails on this base (2 critical). Carried over:

package.json is now identical to chains/realio-testnet for all ten entries plus overrides.

Verification

Lockfile regenerated on the chains/realio base with npm 11.19.0, so the min-release-age=3 cooldown in .npmrc actually applies (it needs npm >= 11.10.0 and is silently ignored below that).

  • npm ci --dry-run under npm 10.9.2 — the npm that node-version: 23 ships in common-testexit 0
  • npm audit --audit-level=criticalexit 0 (0 critical / 15 high, was 2 critical / 16 high)
  • Full docker build including npm run build:nextsucceeds, so the six codegen majors don't break the Next.js build

⚠️ Before merging

chains/realio's Dockerfile has the same unpinned npm i -g npm@latest that just broke the testnet build (run 31193609639) — npm@latest is now 12.0.2, which dropped node 23, and this image is node:23-alpine with engine-strict=true.

common-test doesn't run a Docker build on PRs, so this PR will go green and the mainnet build will fail after merge. #62 fixes it for testnet; chains/realio needs the same pin. I left it out here to keep this PR scoped to dependencies — happy to add it if you'd rather land both together.

Without this, merging the dependency changes in this PR would go green on
common-test and then break the mainnet build, because chains/realio still
carries the two unpinned installs that were already fixed on
chains/realio-testnet.

Dockerfile: npm i -g npm@latest -> [email protected]

  npm@latest now resolves to 12.0.2, whose engines are
  ^22.22.2 || ^24.15.0 || >=26.0.0. This image is node:23-alpine and
  .npmrc sets engine-strict=true, so the install is a hard error. This is
  exactly what broke chains/realio-testnet (run 31193609639), fixed there
  by #62. 11.19.0 is the newest npm still covering node 23, and is
  >= 11.10.0 so the min-release-age=3 cooldown keeps applying.

common-test.yaml: drop `npm i -g @nestjs/cli nx`

  Neither binary is ever invoked -- the job only runs npm ci and npm audit.
  Neither package is in package.json, there is no nx.json or nest-cli.json,
  and no install lifecycle scripts need them on PATH. Unpinned and run
  before npm ci, so it resolved both plus their full transitive trees on
  every PR. Same change merged to testnet as #60.

Verified with a full docker build: npm i -g [email protected] succeeds, npm ci
completes, npm run build:next completes, image tags.
npm audit --audit-level=critical exits 0.

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
@sontrinh16
sontrinh16 merged commit 15acede into chains/realio Aug 21, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants