Skip to content

chore(deps): update dependency @node-datachannel/linux-arm64-gnu to v0.33.4 - #640

Closed
renovate[bot] wants to merge 1 commit into
mainfrom
renovate/node-datachannel-linux-arm64-gnu-0.x
Closed

renovate[bot] wants to merge 1 commit into
mainfrom
renovate/node-datachannel-linux-arm64-gnu-0.x

Conversation

@renovate

@renovate renovate Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@node-datachannel/linux-arm64-gnu 0.33.20.33.4 age adoption passing confidence

Release Notes

murat-dogan/node-datachannel (@​node-datachannel/linux-arm64-gnu)

v0.33.4

Compare Source

What's Changed

New Contributors

Full Changelog: murat-dogan/node-datachannel@v0.33.1...v0.33.4

v0.33.3

Compare Source


Configuration

📅 Schedule: (in timezone America/Los_Angeles)

  • Branch creation
    • Only on Monday (* * * * 1)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying mouseterm with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7242731
Status: ✅  Deploy successful!
Preview URL: https://610860af.mouseterm.pages.dev
Branch Preview URL: https://renovate-node-datachannel-li.mouseterm.pages.dev

View logs

@dormouse-bot dormouse-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feedback on work in progress — this is a draft, so no merge verdict here.

Splitting this platform package out of its family breaks an invariant the repo states normatively, and Build & Test is already red on it.

The disclosure gate fails on every machine that isn't linux-arm64-gnu. optionalSiblingsAtSameVersion in website/scripts/generate-deps.js describes an uninstallable optional dependency from a sibling declared in the same optionalDependencies block at the identical version string — with @node-datachannel/linux-arm64-gnu alone at 0.33.4 and the other five still at 0.33.2, no sibling matches and the generator throws "…is not installed and neither is any sibling declared beside it at the same version, so it cannot be described". That's the Build & Test failure at the Dependency disclosure is current step. docs/specs/security-supply-chain.md → "Disclosure" carries the rule: "described from a sibling in the same optionalDependencies block at the same exact version string — published in lockstep, so the disclosure is identical on every machine."

The bump also doesn't change which binary loads. node-datachannel@0.33.2 declares its own optional '@node-datachannel/linux-arm64-gnu': 0.33.2 (see its snapshot in pnpm-lock.yaml), and per the comment in standalone/scripts/build-sidecar-proxy.mjs the addon "resolves its platform package and detect-libc relative to its own __dirname" — so on Linux arm64 the loader still picks up the 0.33.2 build nested under the wrapper, and the sidecar-root 0.33.4 copy rides along unused. The libdatachannel v0.24.5 DTLS-handshake fix these release notes advertise won't reach the running sidecar until node-datachannel itself moves.

Both point at the same resolution: the wrapper and all six platform pins have to move together, in one PR.

The durable fix is a Renovate group. renovate.json has no rule for this family, so one upstream release fans out into a PR per platform that each break the invariant independently — #637 and #639 fail Build & Test at 24s with the same step. The config already carries five lockstep groups (react, react-router, tailwindcss, storybook, vite, xterm) written for exactly this reason, and the lockstep property here is already stated in the supply-chain spec rather than being an inference. A sixth in the same shape would make Renovate open one PR that bumps node-datachannel and all six platform pins together:

{
  "description": "node-datachannel publishes the wrapper and one prebuilt package per platform from one repo at one version, and website/scripts/generate-deps.js describes an uninstallable platform package from a sibling declared at the SAME exact version string — so a split bump makes the disclosure generator throw on every machine that cannot install the bumped platform. The wrapper also resolves its platform package relative to its own __dirname, so bumping a platform pin without the wrapper ships a binary nothing loads. Keep them in one PR",
  "matchManagers": ["npm"],
  "matchPackageNames": ["node-datachannel", "@node-datachannel/**"],
  "groupName": "node-datachannel",
  "groupSlug": "node-datachannel",
  "separateMajorMinor": false
}

I can open that against main as its own PR if you want it — say the word and I'll push it. Renovate would then regenerate #637/#639/#640 as a single grouped PR on the next Monday window.

nedtwigg added a commit that referenced this pull request Sep 14, 2026
Lands the whole 0.33.2 -> 0.33.4 bump in one commit: the core package plus
all six prebuilt platform packages the sidecar declares beside it, the
lockfile, and the regenerated dependency disclosure.

Renovate split this round into seven PRs (#637, #639, #640, #643, #644,
#645, #647) and every one of them is red on its own, because
website/scripts/generate-deps.js requires the declared siblings to sit at
the same version. The renovate.json group in the previous commit prevents
the next round from splitting; this commit clears the current one.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HHoyvmFzcmMyFbsTSp8FmL
nedtwigg added a commit that referenced this pull request Sep 14, 2026
Lands the whole 0.33.2 -> 0.33.4 bump in one commit: the core package plus
all six prebuilt platform packages the sidecar declares beside it, the
lockfile, and the regenerated dependency disclosure.

Renovate split this round into seven PRs (#637, #639, #640, #643, #644,
#645, #647) and every one of them is red on its own, because
website/scripts/generate-deps.js requires the declared siblings to sit at
the same version. The renovate.json group in the previous commit prevents
the next round from splitting; this commit clears the current one.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HHoyvmFzcmMyFbsTSp8FmL
nedtwigg added a commit that referenced this pull request Sep 14, 2026
… it to v0.33.4 (#642)

* chore(renovate): bump the node-datachannel prebuild family as one group

* chore(deps): update the node-datachannel family to v0.33.4

Lands the whole 0.33.2 -> 0.33.4 bump in one commit: the core package plus
all six prebuilt platform packages the sidecar declares beside it, the
lockfile, and the regenerated dependency disclosure.

Renovate split this round into seven PRs (#637, #639, #640, #643, #644,
#645, #647) and every one of them is red on its own, because
website/scripts/generate-deps.js requires the declared siblings to sit at
the same version. The renovate.json group in the previous commit prevents
the next round from splitting; this commit clears the current one.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HHoyvmFzcmMyFbsTSp8FmL

* chore(renovate): say the grouped node-datachannel PR still needs a disclosure commit

Renovate never runs website/scripts/generate-deps.js, so the grouped PR
arrives with a stale website/src/data/dependencies-npm.json and a red
`Dependency disclosure is current` step — one commit to clear, where a
split bump throws before it can produce a snapshot at all. Without the
note the next session reads that red as the grouping not working.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HHoyvmFzcmMyFbsTSp8FmL

* chore(renovate): name both node-datachannel failure modes, not just the throw

A prebuild-only bump throws, because no declared sibling sits at the new
version. A core-only bump does not: it installs cleanly, the prebuild then
resolves twice — once from the root pin, once nested under the new core —
and the regenerated disclosure lists every prebuild at both versions, as
#647's `Build & Test` shows ("version": "0.33.2, 0.33.4" on all six rows).
The description covered only the first half.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HHoyvmFzcmMyFbsTSp8FmL

---------

Co-authored-by: dormouse-bot <287024035+dormouse-bot@users.noreply.github.com>
Co-authored-by: Ned Twigg <ned.twigg@diffplug.com>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
@nedtwigg

nedtwigg commented Sep 14, 2026

Copy link
Copy Markdown
Member

Superseded by #642, which landed the whole node-datachannel 0.33.2 → 0.33.4 round in one commit — core plus all six prebuilds, the lockfile, and the regenerated website/src/data/dependencies-npm.json.

Split this way none of the seven could go green: the six prebuild-only PRs, this one among them, made generate-deps.js throw because no declared sibling sat at the new version, while the core-only bump (#647) failed instead on a snapshot diff listing every prebuild at "0.33.2, 0.33.4". #642 also adds a Renovate lockstep group for the family, so future releases arrive as a single grouped PR rather than seven.

@nedtwigg nedtwigg closed this Sep 14, 2026
@nedtwigg
nedtwigg deleted the renovate/node-datachannel-linux-arm64-gnu-0.x branch September 14, 2026 22:32
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.

2 participants