Repository navigation
Run subgraph-deploy from flake.lock, bumped to rainix 777d0d3 (Node.js 22.23.4) - #22
Open
Siddharth2207 wants to merge 2 commits into
Open
Siddharth2207 wants to merge 2 commits into
Siddharth2207 wants to merge 2 commits into
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…d0d3 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Siddharth2207
marked this pull request as ready for review
November 2, 2026 09:17
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bumps this repo's
flake.lockrainix entry fromf22d4dcto777d0d31dd4e5a3cd0d74db2e844c502d4cce3e9(rainixmain, 2026-10-24; only that lock node changed) and runssubgraph-deployfrom.#subgraph-shellinstead of a hardcodedgithub:rainlanguage/rainix/<sha>#subgraph-shellref, matching raindex.deploy#2925.flake.nixalready re-exports every rainix devShell (devShells = rainix.devShells.${system}), so no flake change was needed.The step holding
ORMI_DEPLOY_KEY(npm ci+graph deploy) now runs Node.js 22.23.4 from rainlanguage/rainix#419, which carries the 21 October 2026 Node.js security release fixes, and stays pinned to reviewed rainix via the lock rather than rainix main.The lock bump also moves this repo's other
nix develop/nix build .#...users (the default shell in this workflow's ABI check and codegen steps, andsubgraph-report.yaml'snix build .#metaboard-subgraph-report) to rainix777d0d3. The rainix reusable CI workflows pin their ownRAINIX_SHAand are unaffected.This is not a pure Node bump: please read before merging
Every rainix revision with the Node fix also includes rainix#412 ("subgraph-deploy: fail on a refused deploy, label versions by build"), so this pin cannot take the Node fix alone. Between
553ae01and777d0d3,subgraph-deploychanges in three ways:<address>-<commit>and is now<address>-<build hash>, where the hash israinix-static subgraph-build-hashover each network'sgraph buildoutput. The first run after this merges will not find the existing labels, so it will treat every network as not yet deployed.lib/subgraph-deploy.sh). A network whose build has not changed since its last deploy may therefore be refused on that first run. rainix#412 makes a refused deploy fail the run instead of reading as success, so expect the first dispatch to fail for those networks rather than pass silently.graph buildper network now runs before the probe, andgraph deploygets--output-dir.Unchanged: the log lines this workflow parses (
Deploying subgraph <name>/<version>...,Deploy of <name>/<version> failed,already deployed, skipping) are identical at both revisions, as are the deploy key handling and the hardcoded Ormi endpoint.Opened as a draft. Someone who runs these deploys should confirm how the first run behaves with the label change, or decide it is acceptable, before this merges.
Test plan
Local, against this branch's
flake.lockshells:.#subgraph-shell:node --version= v22.23.4,subgraph-deploypresentforge soldeer install,forge build,forge fmt --check,forge lint -D warnings,slither .,reuse lint,rainix-sol-single-contract,pre-commit run --all-filesforge test -vvvwith fork RPCs: 24/24 passedpre-commit run --all-files,rainix-rs-static,cargo testnix build .#metaboard-subgraph-reportmain: assemble, manifest ABI check,npm ci && graph codegen,graph build --networkfor all 7 networks innetworks.jsonworkflow_dispatchrun of this workflow after merge, checking each network's outcome🤖 Generated with Claude Code