Skip to content

build: unblock --features supervisor (stale ractor Cargo.lock pin; fix already on main)#575

Merged
AdaWorldAPI merged 1 commit into
mainfrom
claude/jirak-math-theorems-harvest-rfii13
Jun 21, 2026
Merged

build: unblock --features supervisor (stale ractor Cargo.lock pin; fix already on main)#575
AdaWorldAPI merged 1 commit into
mainfrom
claude/jirak-math-theorems-harvest-rfii13

Conversation

@AdaWorldAPI

@AdaWorldAPI AdaWorldAPI commented Jun 21, 2026

Copy link
Copy Markdown
Owner

Unblock the ractor-actor path — stale lock pin, not a fork bug

Hunting the cheapest real OUT-leg progress, found lance-graph-supervisor/Cargo.toml carried a hard BLOCKED (2026-06-14): "ractor fork ~2 commits behind, non-exhaustive MessagingErr::Saturated match, --features supervisor will NOT compile." That note is stale — verified and fixed:

  1. The fix is already on AdaWorldAPI/ractor main. /home/user/ractor (= origin/main, 0 ahead) has 2bc7819 fix: handle MessagingErr::Saturated at all three match sites; MessagingErr is not #[non_exhaustive] and derived_actor.rs handles all four variants. cargo check -p ractor --no-default-features --features tokio_runtime is green standalone.
  2. The build failed only on a stale Cargo.lock pin3f86d0a (the merge commit before the all-sites fix). cargo update -p ractor advanced it 3f86d0a → f4c474f4 (fixed main).
  3. cargo check -p lance-graph-supervisor --features supervisor now compiles clean (verified — warnings only; callcenter's lance/datafusion are optional and off under this feature, so the build is light, no disk blowup).

Net: the ractor-actor path — the mailbox-as-owner substrate the OUT-leg S4/run-NaN depend on — is OPEN, unblocked without the heavy planner build or symbiont.

Changes

  • Cargo.lock — ractor 3f86d0a → f4c474f4
  • crates/lance-graph-supervisor/Cargo.tomlBLOCKEDRESOLVED (with the verified explanation)
  • .claude/board/AGENT_LOG.md — cont.³⁰

🤖 Generated with Claude Code


Generated by Claude Code

Summary by CodeRabbit

  • Chores
    • Resolved a build compilation issue by updating dependency lock files. The previously blocked build now compiles cleanly.

…(fix was already on main)

The lance-graph-supervisor "BLOCKED (2026-06-14)" note was stale. The
MessagingErr::Saturated non-exhaustive-match fix is already merged on
AdaWorldAPI/ractor main (all-three-sites fix; enum not #[non_exhaustive];
derived_actor.rs handles all 4 variants; ractor compiles standalone). The
build failed only because Cargo.lock pinned a pre-fix git rev (3f86d0a, the
merge commit before the fix).

- cargo update -p ractor: 3f86d0a -> f4c474f4 (fixed main).
- cargo check -p lance-graph-supervisor --features supervisor now compiles
  clean (verified; warnings only — callcenter's lance/datafusion are optional
  and off under this feature, so the build is light).
- supervisor Cargo.toml: BLOCKED -> RESOLVED comment.

Opens the ractor-actor path (mailbox-as-owner substrate the OUT-leg S4/run-NaN
depend on) without the heavy planner build or symbiont. AGENT_LOG cont.30.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CcpLeEC3XK8Eye53GKBVvi
@coderabbitai

coderabbitai Bot commented Jun 21, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 91d36182-07c5-43c3-994b-ea8b9ce6c2cf

📥 Commits

Reviewing files that changed from the base of the PR and between 158987b and 9f4abc1.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • .claude/board/AGENT_LOG.md
  • crates/lance-graph-supervisor/Cargo.toml

📝 Walkthrough

Walkthrough

Two metadata files are updated to reflect resolution of a previously blocked ractor supervisor build. The Cargo.toml comment for lance-graph-supervisor changes from BLOCKED (2026-06-14) to RESOLVED (2026-06-21), and a new entry is prepended to AGENT_LOG.md documenting that the block was caused by a stale Cargo.lock pin rather than a missing upstream fix.

Changes

Ractor Supervisor Unblock

Layer / File(s) Summary
Ractor fork status: BLOCKED → RESOLVED
crates/lance-graph-supervisor/Cargo.toml, .claude/board/AGENT_LOG.md
Cargo.toml comment updated from BLOCKED (2026-06-14) to RESOLVED (2026-06-21), noting that --features supervisor now compiles after cargo update. Agent log entry added describing the stale Cargo.lock root cause and the resolution steps.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • AdaWorldAPI/lance-graph#495: Directly related — that PR recorded the original BLOCKED state for the lance-graph-supervisor ractor fork; this PR resolves the same block.

Poem

🐇 The lock file was stale, the build wouldn't start,
A ghost of old revisions haunting the cart.
One cargo update, the pins fell in place,
MessagingErr::Saturated found at last in the base!
BLOCKED turns RESOLVED — what a hop-tastic day,
The supervisor compiles and the rabbits can play! 🎉

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly describes the main change: unblocking the supervisor build by resolving a stale Cargo.lock pin issue with the ractor dependency.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@AdaWorldAPI AdaWorldAPI merged commit 46ce40b into main Jun 21, 2026
6 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