Skip to content

fix(mount): complete full-root initial synchronization - #43

Merged
miyaontherelay merged 7 commits into
mainfrom
fix/full-root-initial-sync-0830
Aug 30, 2026
Merged

fix(mount): complete full-root initial synchronization#43
miyaontherelay merged 7 commits into
mainfrom
fix/full-root-initial-sync-0830

Conversation

@miyaontherelay

Copy link
Copy Markdown
Contributor

Summary

  • require the detached initial Relayfile materialization to complete its full traversal before reporting readiness
  • use the supported bootstrap read-concurrency ceiling for the foreground readiness barrier while leaving the daemon default unchanged
  • preserve transport failures from mount-path creation and readiness polling for caller retry classification

Verification

  • full sandbox test suite: 804 passed, 15 intentionally skipped, 0 failed (819 total)
  • git diff --check: clean

This is a dev-path repair only. It does not deploy, provision a provider, or change production configuration.

Session-Id: 01a050a5-207f-7921-9f63-55dc686a3185
Session-Id: 01a050a5-207f-7921-9f63-55dc686a3185
Session-Id: 01a050a5-207f-7921-9f63-55dc686a3185
Session-Id: 01a050a5-207f-7921-9f63-55dc686a3185
@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 60eb9068-707d-4e88-a3ba-da03cb63bc32


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.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

You’re at about 98% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.

Re-trigger cubic

@miyaontherelay

Copy link
Copy Markdown
Contributor Author

Review-only head update:

  • exact head: 4d49b32
  • GitHub reports CLEAN / MERGEABLE
  • CI is green with 0 failed and 0 pending checks
  • Cubic reviewed this exact head and reported no issues; focused cross-repo review is in progress

No merge, deployment, provider run, spend, or production/configuration change was performed.

Session-Id: 01a050a5-207f-7921-9f63-55dc686a3185
@miyaontherelay

Copy link
Copy Markdown
Contributor Author

Exact-head review update for d424e1816410d665a2a318f51a03b0614c5b5cb3:

  • verifies every exact-layout root's public Relayfile state after --once
  • missing, malformed, or still-bootstrapping state fails closed with resumable TEMPFAIL 75
  • daemon start is suppressed until persisted readiness is complete
  • full local test suite PASS; focused mount/orchestrator tests 49/49 PASS; changed source typecheck PASS
  • independent diff review: PASS, no concrete findings

No deploy, provider launch, paid run, production/config/secret/cap change, or merge performed. Literal dev convergence remains unproven.

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 4 files (changes from recent commits).

You’re at about 98% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread src/mount-script.ts Outdated
Comment thread src/orchestrator.ts
Session-Id: 01a050a5-207f-7921-9f63-55dc686a3185

Session-Id: 01a050a5-207f-7921-9f63-55dc686a3185
@miyaontherelay

Copy link
Copy Markdown
Contributor Author

Exact-head correction update for 31b651dd4440da38be964cea335ee62433c31ecc:

  • strict JSON parsing now fails closed on success-shaped truncated state
  • readiness validation explicitly checks its guest Node prerequisite
  • exit 69/75 is contained in a subshell, preserving lifecycle continueOnInitialSyncFailure
  • duplicate TEMPFAIL 75 detail removed
  • added missing-Node, truncated-JSON, and real POSIX lifecycle continuation regressions
  • full local suite PASS; focused changed-source typecheck PASS; independent re-review PASS with no correctness/security findings

No merge, deploy, provider launch, paid run, production/config/secret/cap change performed. Literal dev convergence remains unproven.

@miyaontherelay

Copy link
Copy Markdown
Contributor Author

Final exact-head status for 31b651dd4440da38be964cea335ee62433c31ecc: GitHub reports CLEAN / MERGEABLE; Build & Test, CodeRabbit, and Cubic all SUCCESS. Cubic reports all prior findings addressed. No merge, deployment, provider run, spend, or production/configuration change was performed.

Session-Id: 01a050a5-207f-7921-9f63-55dc686a3185
@miyaontherelay
miyaontherelay merged commit 27b4249 into main Aug 30, 2026
3 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.

1 participant