Repository navigation
Add guarded artifact and rehearsal integration - #4
Merged
Merged
Conversation
Author
Post-rebase native verificationVerified the rebased PR head
|
ghost2788
marked this pull request as ready for review
August 31, 2026 20:05
SamG-Coder
force-pushed
the
codex/gate3-infinity
branch
from
September 1, 2026 19:03
57904f8 to
484ae82
Compare
Owner
|
Thank you for the contribution and for the thorough verification work. The guarded artifact exchange and rehearsal integration are a useful addition to Studio. |
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.
Summary
Adds a small, capability-gated integration seam to the existing nine-tool Studio surface:
three_studio_projectcan import and guarded-export one configured-root-relative canonical JSON artifact.three_studio_applygains the exact, hash-guardedartifact.json.patchoperation.three_studio_jobcan invoke a configured external rehearsal using only a repository-relative run-spec reference.Security and ownership boundary
tools/rehearsal-sidecar/src/cli.mjswithshell: falseand a minimal environment.Upstream compatibility
This branch is rebased onto current
main(e0fe5b0). Conflict resolution unions the new guarded seam with upstream'ssceneExport,blankproject template,material.look.patch, Pathwire bridge codec, and Plainform runtime/tests. The generated nine-tool contract is 85,793 bytes, so its compact total-schema test ceiling moves narrowly from 85,000 to 86,000 bytes; the stricter 73,000-byte apply-schema ceiling and one-MiB transport ceiling are unchanged.Verification
node --test tests/*.test.mjs: 741 passed, 0 failed on Nodev24.19.0.Review focus
Feedback is especially welcome on the generality and naming of the artifact exchange and fixed rehearsal hook. This proposal does not vendor an external project or establish a maintained fork.