Repository navigation
Report structured graph and retained material preflight outcomes - #115
Merged
Merged
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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 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
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.
Current user scope: backward compatibility is not required. Existing projects and older consumers must perform their own one-time migration/adaptation to current Tuldok formats. Intentional breaking format changes need no legacy adapter or automatic migration framework. Current-version schema/contract validation, honest version errors, authority/custody/accounting and backup/restore integrity remain required. Historical unchanged-contract evidence below records the actual source; it is not a compatibility acceptance obligation.
Graph and retained Study-material preflight refusals now expose bounded typed reports with request provenance and null unknown identities. Successful retained external checks bind exact composition/Model/inner-material/graph identities; builtin reads refuse before native probing. Admission, device readiness, custody and publication remain later owner checks.
The ordered two-commit source contains graph
1442f1849c5894500d69823e60bfcd3d644dc683followed by materialcd420c44c42ff85487965a858865170cfc0a9358, based on main360bbda950cc48ed3c3d9193539039b68e0ac8cb. Combined:31 files,+2708/-31, including retained evidence. All126 main contracts and legacy discovery remain unchanged; graphv4 and materialv1 are additive. Cold material reads and Trial receipt/replay behavior remain unchanged. README/workflows are unchanged.Local source-bound qualification: graph65tests/200.178s; material75tests/246.263s plus final strengthened9tests/20.883s, each zero skips/failures/errors. The75 and9 subjects have separate test hashes and identical production bytes. Independent reviews cleared each scoped slice; original failures and raw evidence remain retained under the two plan report directories. These are historical local results on their named subjects; fresh hosted validation on the published head is recorded below.
Expected PR workflows: Dataset lifecycle, M1 workspace interface, Inert extension catalog and Candidate package captured CPU. Static extension authoring kit has no matching changed path and runs normally on a future main push. No full C1/T1/M1, PUB-1/SCI-1, native builtin, scientific-quality or hardware acceptance is claimed. Graph-execution details in the bounded retained projection remain owner-attested; runtime/device/admission/resource/custody/publication facts are deferred where specified.
Fresh integration: exact original commits and full candidate tree verified in the authorized fresh task; all23 framed transfer hashes, manifest blobs, full recursive tree,31-file scope and unchanged README/workflows match. Published head is
cd420c44c42ff85487965a858865170cfc0a9358; GitHub merge checkoutf24ed37c1a062b8f3f25e961810f7ef8467f1500has the exact candidate treecdcfd9e2f348af530149314936c5efafbc6d8162.Fresh independent source review cleared the combined slice, independently recomputed all126 prior contracts and legacy discovery, and passed26 pure tests in3.803s with zero skips/errors/failures after isolated dependency provisioning. Earlier missing-driver import errors remain distinct from that successful run. Current12 neutral HTTP/CLI acceptance also passed97.352s with zero skips/errors/failures; classifier comparison was explicitly excluded locally because no qualified training runtime was selected. Historical75-case regression and9-case follow-up remain separate original subjects.
Hosted checks on this exact head: lifecycle passed; capturedCPU, workspaceUI and catalog passed. Raw completed-run log retrieval returned Forbidden; run/check/step metadata remain readable, so no unobserved hosted test counts or zero-skip claim is made. All four applicable hosted checks are terminal success at the exact published head. Fresh main remains360bbda and its tested merge tree equals the candidate. Independent review is clear; normal dependency-first integration is authorized.