Repository navigation
Define issued credential retention contracts and their ownership - #182
freeqaz-openai wants to merge 23 commits into
Conversation
| /** Original protected source owner. Release drains original issuer settlement and | ||
| * late finalizers, including failed construction; it never destroys a shared source. | ||
| * Closing material or releasing this lease proves neither revocation nor COMMIT. */ | ||
| export interface ProtectedSourceLease { |
There was a problem hiding this comment.
we should probably move interfaces into their own file and separate "consumed externally" from an internal interface. What is the boundary our modules need to expose? This change may need to cut across this PR + base ones to align on a shared, clean code setup. If this is already established for the repo, we need to follow that convention. Modules should be "thin and deep" in the API they expose. thin = small API surface; deep = rich functionality.
|
Please remove the GitHub-specific profile requirement from the shared issued-credential retention contract.
This is separate from the GitHub-specific source loader and custody dependencies, which already belong under the GitHub provider directory. #184 inherits this contract through the stack; its new schema registry does not introduce the same restriction. Reviewed head: |
A successful mint does not establish durable custody. This PR defines the internal contracts that retain the original issuer result, protected material, State transaction and cleanup obligation until the outcome is known.
The compiler fixtures check both owners. All 15 original issuance rejection cases remain, with one added check that the OCC lease cannot expose GitHub signing material. Internal handles remain outside the public OCC entry point.
Validation
The refreshed revision passed 141 local provider, protected-material, HTTP-head and JSON conformance cases, with zero failures or skips. The full workspace TypeScript build and all three compiler fixture projects passed; all 247 negative assertions remain, including the original issuance rejection cases.
Formatting, workspace boundaries, CI suite audit, documentation length/build/link validation, generated OpenAPI/reference and whitespace checks passed. Documentation validation checked 119 pages and 1,435 links. Hosted CI must still validate this refreshed head. No live-provider, installed-runtime or PostgreSQL verification was rerun.
Dependencies and limits
Based on #177, with #175 included as the write-issuer supplier. This is a declaration and compiler-conformance change. The source loader, retained issuance implementation, Work/IAM admission and production Agent caller remain separate integration work; these checks do not establish live-provider or installed behavior.