Uh oh!
There was an error while loading. Please reload this page.
docs: record the deferred solution method-policy questions in-code (#297) - #298
Merged
antoinetoussaint-byte merged 2 commits intoAug 17, 2026
Conversation
) The four remaining tasks from #295 are cross-repo (the CLI solution command group and the host executor) or consumer-dependent, so they have no substantive code deliverable in core. Give them a durable home next to the enforcement code so closing the tracker loses no context, and sharpen the Ceiling doc that over-stated "provenance": the gate enforces caller-asserted intent, not an authority-derived grant. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…us (#297) The review found the added package-doc paragraph reported cross-repo *status* ("stay provisional until", "nothing yet binds", "still-open step", "do not exist in this repo yet") for work that happens in another repo. No change in core would ever prompt updating it, so it goes false — and actively misleading, right beside the enforcement code — the day the codefly CLI wires a real provenance-derived ceiling. Reshape to the durable design properties instead: the gate enforces caller-asserted intent (not authority), and the tier cut plus Render's REGISTRY_READ vs OFFLINE classification are the consuming host's to establish. Also collapse the re-derived artifact_reference rationale to a pointer at solution.proto, its source of truth, so the two copies can't drift; and align the TestOperationCeilingsAdmitExactlyTheirRPCs comment on "intent-provenance" so the test's vocabulary matches the type doc. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
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 freeto 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.
Closes#297.
Summary
core— the CLI solution command group (in thecodeflybinary repo) and the host executor. I confirmed nothing incoredrives the lifecycle throughsolution.Clientor any ceiling; only the definitions and the interceptor install inagents/manager/loader.goexist. So there is no substantive code deliverable incorefor these tasks, and none is fabricated here.TestSolutionRPCMethodPolicyIsMachineEnforceablepinsRender=REGISTRY_READ/LOCAL_WRITE;TestOperationCeilingsAdmitExactlyTheirRPCspins theInspect < Scaffold < Render < Publishcut). What was not recorded anywhere durable is the reasoning that these are provisional — it lived only in the transient solution: wire first host caller and settle the method-policy open questions #295 → refactor: collapse the Solution two-API split onto solution.Client (#295) #296 → solution: remaining method-policy open questions (successor to #295) #297 trackers, which is exactly what the tracker feared losing.Ceilingdoc claimed the constructors give the ceiling "a provenance," but that is only intent provenance (which operation a caller named). The gate enforces caller-asserted intent, not an authority-derived grant — a real trust-boundary caveat a reader ofEnforcingClientInterceptorshould not miss.What is not done here (remains open, cross-repo)
Per the issue's own premise, all four tasks "only become answerable once a real caller exists," and that caller is not in
core:codeflybinary repo.Closing #297 discharges its role as a core-side placeholder (the deferred design context now lives in the code); the actual work moves to the repos where it can be done. If the team would rather keep #297 open until the cross-repo caller lands, drop the
Closes #297.line on merge.Test plan
go build ./solution/...go vet ./solution/...go test ./solution/...(documentation-only change; behavior and existing guard tests unchanged)🤖 Generated with Claude Code