Skip to content

fix(api): preserve CAS ownership in workspace publication and preflight - #1223

Merged
DecisionNerd merged 2 commits into
mainfrom
fix/1222-workspace-cas-ownership
Sep 10, 2026
Merged

DecisionNerd merged 2 commits into
mainfrom
fix/1222-workspace-cas-ownership

Conversation

@DecisionNerd

@DecisionNerd DecisionNerd commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Workspace publication after CAS construction passed a generation-tree path for an object-backed participant, and composition preflight read an empty catalog from that nonexistent tree. Select the declared ownership, retain the CAS publication lease through CURRENT, and decode the authenticated CAS catalog through the existing streaming reader.

The public regression preserves all 18 original payload identities and inventory entries through non-promoting ontology adoption/retry/reopen, establishes semantic bindings through public composition publication, adds qualified child data, and verifies exact queries through export/full verification/clean import. No encoding default changes. Same-name promotion and removing authority from retained typed data remain explicitly tracked under #1221.

Validation: public regression; 53 ontology-related API tests; cargo clippy --workspace -- -D warnings; make pre-push-fast; make gate-registry-check; final formatting and diff checks. Prebuilt fixture: 2.03 s, 129,120 KiB RSS; separate syscall trace and measurement limitations are recorded in the evidence JSON. Required CI remains the merge gate.

Closes #1222


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 15b48651-ebe5-459d-a43d-ee77ee3bb35e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

❤️ Share

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

@github-actions github-actions Bot added core Core source code changes documentation Improvements or additions to documentation labels Sep 10, 2026
@DecisionNerd
DecisionNerd merged commit cba602d into main Sep 10, 2026
23 checks passed
@DecisionNerd
DecisionNerd deleted the fix/1222-workspace-cas-ownership branch September 10, 2026 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Core source code changes documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(api): preserve CAS graph ownership in workspace publication

1 participant