Skip to content

Bump pinned resolutions and refresh lockfile for Dependabot alerts - #71

Merged
dggrunzweig merged 3 commits into
devfrom
fix/dependabot-bump-pinned-resolutions
Jul 29, 2026
Merged

Bump pinned resolutions and refresh lockfile for Dependabot alerts#71
dggrunzweig merged 3 commits into
devfrom
fix/dependabot-bump-pinned-resolutions

Conversation

@sophiehouser

Copy link
Copy Markdown
Contributor

Clears 13 of the 15 open Dependabot alerts on @fragment-dev/node-client. Most of them existed because this repo's resolutions block pins packages to exact versions that earlier advisories had made safe and later advisories made vulnerable again — the pins were holding the tree below the patched releases.

Pins bumped in package.json:

Re-resolved within existing ranges, no package.json change needed: js-yaml → 4.3.0, @babel/core → 7.29.7, shell-quote → 1.10.0.

Left open deliberately: the two immutable alerts (GHSA hash-collision and List trie-overflow DoS). immutable resolves to 3.7.6 because @ardatan/relay-compiler@12.0.0 pins ~3.7.6, reached via @graphql-tools/relay-operation-optimizer under the graphql-codegen plugins. The advisories' first patched version is 4.3.9, so clearing them means forcing relay-compiler across the breaking 3.x → 4.x boundary. This repo has no Relay usage, so that optimizer never runs. Proposed for dismissal separately rather than fixed here.

Verified: yarn typecheck clean. yarn test:ci passes 5 tests; the 12 failures are pre-existing and environmental — tests/generated-sdk.test.ts and one other live-API suite throw Please provide CLIENT_ID, CLIENT_SECRET, ... from getClient before any request is made, so they need credentials this run did not have.

🤖 Generated with Claude Code

Regenerates generated/generated.ts and the dist build artifacts so
`yarn build && git diff --exit-code` passes in CI. The committed
files had drifted behind the live schema (new PeriodFilter scalar,
DateFilter.within/withinBalanceUTCOffset, StringMatchFilter.matchesAny,
LedgerAccount.payment, SchemaPaymentInput).
Regenerates tests/fixtures/generated-test-client.ts so the
"Verify test schema generated files are up-to-date" CI step passes.
The fixture had drifted behind the live schema (PeriodFilter scalar,
AddLedgerEntries batch types, LedgerAccount.payment, matchesAny, etc.).
tests/fixtures/test-schema-queries.graphql regenerated identically, no change.
@dggrunzweig
dggrunzweig merged commit 7e61abc into dev Jul 29, 2026
4 checks 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.

2 participants