fix: resolve Node compile issue, use nightly formatting, pin Rust to 1.93, and fix Claude 401s

[Vui-Chee]

• fix compile issue on node
• set nightly formatting for just format
• pin rust toolchain version
• fix 401 issue on claude step
• use rust 1.93

[Vui-Chee]

Code Review: fix: addr compile issue on main

Summary: Fixes a compile error from the Reth v1.10.2 → v1.11.0 upstream upgrade — task_executor() method was removed in favor of direct field access (.task_executor). Also switches justfile formatting to cargo +nightly fmt and removes the unnecessary id-token: write permission from the top-level Claude workflow.

---

Issues

🟡 Warning — CLAUDE.md docs are stale after the justfile change

CLAUDE.md still documents:

just check-format   # Check formatting (cargo fmt --all -- --check)

But check-format now calls cargo +nightly fmt --all -- --check. Should be updated to reflect the nightly requirement.

🟡 Warning — id-token: write still present in claude-review job

The top-level workflow permissions had id-token: write removed (good, least-privilege), but the claude-review job at line 78 of .github/workflows/claude.yml still declares it as a job-level override. If OIDC auth isn't needed there, remove it for consistency.

🔵 Suggestion — Nightly toolchain not pinned

cargo +nightly fmt uses whatever nightly is installed locally/on CI. Consider pinning via rust-toolchain.toml to prevent format churn across different nightly versions.

---

Positives

• Surgical fix: The Rust change is a single targeted field access correction — no logic touched.
• Correct diagnosis: Directly addresses the upstream breaking change.
• Least-privilege improvement: Removing id-token: write from the top-level workflow is the right call.

---

Verdict: LGTM — the core compile fix is correct. The warnings above are non-blocking.