fix(storage): count V2 catalog manifest CAS as logical refs - #1090
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Warning Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption. 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. Comment |
This comment has been minimized.
This comment has been minimized.
|
CI Gate on this PR is blocked by an independent demand-capture race (deferred one-hop expand drop leaking into the next optimized two-hop Sibling fix: #1092 (closes #1091). After that merges to main, rebase/re-run this PR for exact-head CI Gate. |
Compact graph-files manifests attributed physical CAS objects under catalog_and_manifests without matching logical references, which made OVHC-AGENCY S18 rung assembly fail closed after an otherwise passed lifecycle. Co-authored-by: Cursor <cursoragent@cursor.com>
bb45527 to
1e3be72
Compare
Summary
physical_objects > logical_references.Closes #1089
Test plan
CARGO_TARGET_DIR=... cargo test -p graphforge-storage --lib compact_v2_catalog_manifestCARGO_TARGET_DIR=... cargo test -p graphforge-storage --lib qualification_rejects_category_physicalMade with Cursor
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.