docs(readme): name loadout as reference consumer of the mint path (lr-de6c5c) - #28
Conversation
…-de6c5c) Gatekeeper README references loadout zero times while loadout's README names gatekeeper three times with the seam framing already in place. Add a 'Composes with' section mirroring loadout's own vocabulary: loadout is the reference consumer via its TokenProvider seam, the integration is optional in both directions, and neither project imports the other. Extend docs/ROLES.md's returned-identity section with a one-line reference-consumer pointer rather than restating the mechanism. No product code changes.
|
PEACHES — clean (1 nit)
Factual accuracy verified:
Vocabulary matches loadout/README.md:
Duplication control:
Optionality both directions:
Public surface:
Docs-only verified: README.md and docs/ROLES.md only |
PEACHES review on PR #28 flagged a doubled 'all' in the 'or no provider at all all work equally well' sentence. Fixed to 'or no provider at all each work equally well', which agrees with the distributive sense over the three-item list (a static token, a bring-your-own minting command, or no provider at all each work equally well as a TokenProvider).
|
PEACHES — clean Prior context: Reviewed at SHA 2459886, found one nit (doubled "all") on README line 48, now fixed. Re-reviewing at new SHA be6fbce per NAOMI gate binding verdicts to head SHA. This pass: Diff confirms typo fix in place: "or no provider at all each work equally well" is grammatically correct and reads better (distributive across the three-item list). Cross-references to loadout (README "Composes with"/docs/ROLES.md "Reference consumer") are valid anchors. Optionality framing preserved: Gatekeeper users don't require loadout; loadout users don't require Gatekeeper. Prose is clean. No findings. |
|
BOBBIE — clean PR #28 is genuinely docs-only: README.md (+26) and docs/ROLES.md (+7), 33 insertions, 0 deletions, no code/config/workflow files touched. Verified against 72a7126..be6fbce both via the GitHub read path and local git diff --stat (identical result). Residual exposure checks on the new prose (public repo, credential-minting service):
gitleaks run against working tree: no leaks found. No code changed, so semgrep/osv-scanner have no reachable sink to scan. Zero findings. Clean audit. scanners_run: gitleaks (no leaks found); semgrep/osv-scanner not applicable (no code files in diff) |
|
Merged via clagentic-loadout v0.2.0
|
What
Documents the loadout/gatekeeper integration from gatekeeper's side, which was previously undocumented. Before this change, clagentic-loadout's README referenced gatekeeper three times with a public link and seam framing; clagentic-gatekeeper's README referenced loadout zero times. A reader arriving at gatekeeper had no way to discover that loadout exists, that it is the reference consumer of gatekeeper's minting path, or how the two compose.
Changes
What this is NOT
Coordination note (for Scope C / loadout docs/provisioning.md, still open)
The loadout side should describe the same mechanism as: loadout's TokenProvider seam optionally consumes gatekeeper's verified App slug (returned via --json / Token.AppSlug) as a resolution tier above its own github_app.slugs config, with config remaining the standalone fallback. This PR's README/ROLES.md wording is the gatekeeper-side anchor to cross-reference against so the two ends do not drift into two descriptions of one fact.
Test status
go build ./... and go test ./... pass (docs-only change; no code paths affected). No markdown-specific lint target exists in the Makefile.
Task: lr-de6c5c