ci(deploy-tests): update rust v10 version, and reorder the sequence

[brbrr]

No description provided.

[Copilot]

Pull request overview

Updates the “Deploy and Test” GitHub Actions workflow to adjust the ordering of downstream test jobs and bump the Starknet Rust RPC v0.10 test reference.

Changes:

• Added/moved the starknet-rust-rpcv10 job earlier in the workflow and updated its ref to 0.19.1.
• Reordered job dependencies so starknet-js-rpcv08 waits on starknet-rs-rpcv09.
• Updated starknet-rs-rpcv09 dependency chain (currently contains an invalid needs reference).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Claude finished @brbrr's task in 50s —— View job

---

Claude review — ci(deploy-tests): update rust v10 version, and reorder the sequence

• Read the PR diff (git diff origin/main...HEAD)
• Verify the job dependency graph at current HEAD
• Cross-check earlier bot review comments against current state

---

Summary

The change reorders the serial starknet-rs/starknet-rust chain and bumps the v0.10 Rust test ref 0.19.0 → 0.19.1. Looks correct as currently committed. All needs references resolve to existing jobs.

The serial chain at current HEAD (lines 114–136):

deploy → starknet-rust-rpcv10 → starknet-rs-rpcv09 → starknet-rs-rpcv08

Job	needs	Resolves?
starknet-rust-rpcv10 (114)	[deploy]	✅
starknet-rs-rpcv09 (122)	[deploy, starknet-rust-rpcv10]	✅
starknet-rs-rpcv08 (130)	[deploy, starknet-rs-rpcv09]	✅

Re: earlier bot comments (now stale)

Both prior review comments appear to have been written against an earlier commit and no longer apply to current HEAD:

• Copilot — "needs references a non-existent job starknet-rs-rpcv10" (line 124): Resolved. Line 123 reads needs: [deploy, starknet-rust-rpcv10] — note rust, not rs, which is the real job at line 114. No invalid reference remains.
• "starknet-rs-rpcv08 detaches from the serial chain": Resolved. Line 131 has needs: [deploy, starknet-rs-rpcv09], so v8 still runs last in the serial chain rather than concurrently.

Notes (nit)

• The reorder reverses the execution order from v8 → v9 → v10 to v10 → v9 → v8. This is consistent with the PR title and reads as intentional (newest version first). No action needed — just confirming the chain is still fully serialized against the single deployed node.

Verdict: No blocking or important issues. The dependency graph is valid and the chain remains serial. 👍

Note: I can't push edits to files under .github/workflows/ (GitHub App workflow-scope restriction), but none are needed here.
• ci/update-rust-tests

[github-actions]

Claude finished @brbrr's task in 33s —— View job

---

Claude review — ci(deploy-tests): update rust v10 version, and reorder the sequence

• Read the PR diff (git diff origin/main...HEAD)
• Verify the job dependency graph at current HEAD
• Cross-check earlier bot review comments against current state

---

Summary

This PR reorders the serial starknet-rs/starknet-rust test chain (newest-version-first) and bumps the v0.10 Rust test ref 0.19.0 → 0.19.1. It's clean — all needs references resolve to defined jobs and the chain remains fully serial against the single deployed node.

Serial chain at current HEAD (lines 114–136):

deploy → starknet-rust-rpcv10 → starknet-rs-rpcv09 → starknet-rs-rpcv08

Job	needs	Resolves?
starknet-rust-rpcv10 (114)	[deploy]	✅
starknet-rs-rpcv09 (122)	[deploy, starknet-rust-rpcv10]	✅
starknet-rs-rpcv08 (130)	[deploy, starknet-rs-rpcv09]	✅

Earlier bot comments — now stale

Both prior review comments were written against an earlier commit and no longer apply:

• Copilot — "needs references non-existent job starknet-rs-rpcv10": Resolved. Line 123 reads needs: [deploy, starknet-rust-rpcv10] (rust, not rs), matching the real job at line 114.
• "starknet-rs-rpcv08 detaches from the serial chain": Resolved. Line 131 has needs: [deploy, starknet-rs-rpcv09], so v8 still runs last in the serial chain.

Verdict: No blocking, important, or nit-level issues. The dependency graph is valid and the suites remain serialized. 👍
• ci/update-rust-tests

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.55%. Comparing base (ea473bf) to head (bf664b4).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3711      +/-   ##
==========================================
+ Coverage   76.52%   76.55%   +0.03%     
==========================================
  Files         405      405              
  Lines       36942    36942              
==========================================
+ Hits        28269    28281      +12     
+ Misses       6680     6673       -7     
+ Partials     1993     1988       -5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.