feat: R06 sandbox, R07 graft, R08 projection redaction - #57
Open
Siddhartha-singh01 wants to merge 1 commit into
Open
feat: R06 sandbox, R07 graft, R08 projection redaction#57Siddhartha-singh01 wants to merge 1 commit into
Siddhartha-singh01 wants to merge 1 commit into
Conversation
Implement remaining §10 conformance surfaces: sandbox mode rejects NON_IDEMPOTENT_WRITE (R06), graft_artifact_ref never copies THOUGHT (R07), and shared redact helpers for projected context (R08). Python + Go parity, conformance tests, README/CHANGELOG. Closes #54, #55, #56. Signed-off-by: Siddhartha Singh <siddharthagithub0007@gmail.com>
Collaborator
|
merge conflict, fix it please |
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
2 participants
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
Implements the three open §10 issues after R03:
RunMode.SANDBOXrejectsNON_IDEMPOTENT_WRITEbefore side effectsgraft_artifact_ref/trajir/graft— artifact ref only, never THOUGHTredact_projection_context/trajir/redact— secrets + THOUGHT scrub for projected contextCloses #54, #55, #56.
Design notes (senior constraints)
exec_toolandmake_run_step/ GoExecTool+RunStep; default remainsliveso demos and R01/R02 unchanged.ARTIFACT_REF; refuses private kinds; does not copy source node history..tirredacted export (runtime/redact); CONSTRAINT nodes stay present with scrubbed fields (compatible with R04 when feat: R04 default context projector with CONSTRAINT budget safety #53 merges). Does not require projector on main.Test plan
pytest conformance/r06_* r07_* r08_*+ related unit testsgo build ./trajir/...andgo veton new packages