feat(worktree): add ScorpioFS backend - #447
Open
Ivanbeethoven wants to merge 8 commits into
Open
Conversation
Resolve 14 conflicts across 8 files and re-fit the ScorpioFS backend onto
main's reworked worktree registry.
Conflict resolutions, all keeping BOTH sides:
- src/internal/mod.rs, docs/commands/worktree.md, src/utils/util.rs and
tests/command/worktree_test.rs: additive, each side's new item kept.
- src/command/worktree.rs and worktree-fuse.rs: this branch's `Scorpiofs`
match arm alongside main's reshaped `List { schema_version }` and new
`Doctor` arms; import block and `--help` EXAMPLES merged.
- src/command/status_untracked.rs: the candidate-path fast path now runs
inside main's io-blocked collection and racily-clean guard —
`collect_tracked_worktree_changes` takes the filtered `tracked_files`,
and `scan_candidate_paths` gained the `io_blocked` field. It calls
`scan_file`, which already records those events, so a blocked candidate
propagates instead of vanishing.
- Cargo.lock: taken from main and re-resolved.
Fixes the merged tree needed to build:
- `scorpiofs = "=0.3.1"` was never published to crates.io, so the lockfile
could not resolve at all. Repinned to `=0.4.0`, which exposes every API
this branch uses — `cli::antares_overrides` (identical signature),
`util::config::init_config_with`, `daemon::antares::{AntaresDaemon,
AntaresServiceImpl}` and `new_external_state`.
- main reworked the registry: `WorktreeState::worktrees` is now `entries`,
`WorktreeEntry` carries `worktree_id`/`state`/`epoch`, and
`gc_worktree_scoped_rows` became `gc_worktree_scoped_rows_strict(db, id,
directory_gone) -> Result`. The ScorpioFS attach/detach/rollback paths
follow the patterns `worktree add`/`remove` already use: attach takes a
`next_epoch()` registration and records `linked_history = Existed`,
detach sweeps strictly (directory gone) and fails the command on error,
and the attach rollback sweeps best-effort with a warning.
- `command_scope`'s exhaustive match had no arm for the new hidden
`scorpiofs-worker`. Classified `Repository` like `libra service` rather
than under-claiming it read-only, and added to the long-running
exemption in `command_holds_shared_maintenance_lock` — a resident FUSE
worker holding the shared maintenance lock for the life of the mount
would starve every deletion phase.
- COMPATIBILITY.md and docs/development/commands/README.md rows plus a new
docs/development/commands/scorpiofs-worker.md, required by the
compat_matrix_alignment guards.
- Seven clippy findings under `-D warnings`: three collapsible_if, one
needless_borrow, one field_reassign_with_default, and an OpenOptions
`create` without an explicit `truncate`.
Verified: cargo check --all-targets --all-features, clippy -D warnings,
cargo +nightly fmt --all --check, and the compat_matrix_alignment suite
(8/8) all pass. The full `cargo test --all` run was still in progress at
commit time; CI covers it.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The ScorpioFS backend creates `<storage>/scorpiofs`, which the `every_storage_rooted_file_is_a_classified_gc_source` guard requires to appear in `GC_OBJECT_FILE_SOURCE_INVENTORY` or `NOT_AN_OBJECT_SOURCE`. Register it as `NonRoot` rather than excluding it: `base_oid` names a REMOTE monorepo revision the mount projects, not a local object-store id, so rooting it would demand objects this repository never wrote -- the same shape as MERGE_RR. Libra owns the durable desired state here precisely because the embedded worker persists none; the mount's own Git writes reach the object store through the ordinary commit path, which roots them. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Unrelated to ScorpioFS, but `compat-offline-core` cannot go green without them. Attribution: run 31259005383 (branch codex/fix-windows-release-build, same day, no shared files) fails on the byte-identical set of tests, so none of these are caused by this PR. They are main's own, introduced after the last green run on 2026-07-27. - worktree_scope (0ce8f77): five tests call `std::env::set_current_dir` directly. `#[serial]` only orders them against other `#[serial]` tests, not against the CWD_LOCK every `ChangeDirGuard` holds, so they yanked the process cwd out from under whatever fixture was mid-flight. Take the lock. `a_pinned_scope_survives_a_cwd_change` additionally pinned the ambient cwd, which `RequestScope::resolve` refuses outside a repository -- it only ever passed by stealing a moment when another test had parked the cwd in someone else's fixture, which is the same moment it broke that test. Give it a real repository. - ai/hooks/runtime (6da73c6): `CaptureScope` entered the ingest path and resolves `RepoIdentity`, but `ingest_fresh_conn` never wrote `libra.repoid`, so 14 tests failed deterministically. Seed it, as `libra init` does. `ingest_fails_loud_when_table_missing` now drops one table from a complete schema instead of using a schema-less database, so it still tests what it claims; the tombstone test asserts against the claim diagnostic that now carries the refusal. - utils/path: the commondir fixture's target lacked the repository database, so it is no longer TERMINAL common storage. - rename_detect (5c1b7bb, a26cc9a): the LFS classification resolves `working_dir()` infallibly on the pooled io thread, which panicked the worker outside a repository and surfaced to the caller as `IoTimeout`. Give both tests a repository. A jammed pool reports `IoTimeout` too, and the pool is process-global -- the seam tests abandon reads that keep sleeping in a worker past their own end -- so the negative test drains the pool and retries. A seam that really fired would still blow every attempt. Verified: `cargo test --lib` 4403 passed, 0 failed. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Contributor
|
CI 归因说明（为什么这个 PR 里有几处与 ScorpioFS 无关的测试改动）
其余 18 条是 main 的既有红线。对照实验：同一天（2026-08-08）的 run 31259005383，分支
修复都收在 本地验证： |
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
2 participants
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
Validation