Skip to content

build: upgrade graph-cli to 0.98 and graph-ts to 0.38 - #264

Open
rickstaa wants to merge 2 commits into
mainfrom
build/graph-cli-0.98
Open

build: upgrade graph-cli to 0.98 and graph-ts to 0.38#264
rickstaa wants to merge 2 commits into
mainfrom
build/graph-cli-0.98

Conversation

@rickstaa

@rickstaa rickstaa commented Sep 8, 2026

Copy link
Copy Markdown
Member

Upgrades graph-cli from 0.69 to 0.98 and graph-ts from 0.29 to 0.38 together, since 0.98's codegen emits an Int8 import that graph-ts 0.29 does not export. graph-cli 0.98 also requires an explicit immutable argument on every @entity, which is why #227 failed CI. All 52 entities get immutable: false to keep current semantics. Marking the event entities immutable is a separate indexing optimisation for later.

Also in this PR:

  • Dockerfile moves to node 22, which graph-cli 0.98 requires.
  • The seven harmless bumps from chore(deps): bump the minor-and-patch group with 9 updates #261 (fs-extra, ts-node, typechain, @types/mocha, hardhat-ethers, hardhat-waffle) are folded in, so that grouped PR can be closed.
  • The four resolutions that only existed for graph-cli 0.69's dependencies are removed, and scoped ones are added for the vulnerable transitives 0.98 brings (glob, semver, cross-spawn, undici 7, yaml 2). See Remove the temporary yarn resolutions as their parents are upgraded #269.
  • One upstream advisory with no release available is allow-listed in dependency-review, with the reasoning in a comment on that line.

The graph-ts bump moves AssemblyScript from 0.19 to 0.27. The mappings compile without changes. yarn codegen and yarn build pass. Clears the five js-yaml 3 alerts. Needs a yarn test run before merge since the mapping runtime changes. Supersedes #226, #227 and #261.

@rickstaa
rickstaa force-pushed the chore/deps-security branch 2 times, most recently from 9833a4c to 2504b23 Compare September 9, 2026 08:41
Base automatically changed from chore/deps-security to main September 9, 2026 09:24
@rickstaa
rickstaa force-pushed the build/graph-cli-0.98 branch from 2c9c440 to 16fc721 Compare September 9, 2026 09:39
Copilot AI lite review requested due to automatic review settings September 9, 2026 09:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@rickstaa
rickstaa force-pushed the build/graph-cli-0.98 branch from 16fc721 to 7446928 Compare September 9, 2026 09:40
Copilot AI review requested due to automatic review settings September 9, 2026 09:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings September 9, 2026 09:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

🚀 Subgraph Studio preview deployed

Item Details
Version label pr-264-a04d223-34336257214
Query endpoint https://api.studio.thegraph.com/query/31909/livepeer-ci/pr-264-a04d223-34336257214
curl -H 'Content-Type: application/json' \
  -d '{"query":"{ protocol(id: \"0\") { inflation } }"}' \
  https://api.studio.thegraph.com/query/31909/livepeer-ci/pr-264-a04d223-34336257214

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