Skip to content

docs: standardize skill routing contracts - #222

Merged
kosako merged 3 commits into
mainfrom
codex/issue-215-routing-contracts
Jul 15, 2026
Merged

docs: standardize skill routing contracts#222
kosako merged 3 commits into
mainfrom
codex/issue-215-routing-contracts

Conversation

@kosako

@kosako kosako commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • standardize the 12 runtime skill descriptions around outcome, trigger mode, positive/negative routing boundaries, side effects, and composition
  • add a catalog-derived runtime inventory and representative positive/negative routing queries
  • preserve the merged diagnose/continuation/review/handoff authorization gates while clarifying neighboring routes
  • tighten withheld-GitHub-content, mixed/unknown-review-author, ambiguous-defect, and third-party-policy boundaries
  • rebind personal-asset-miner human approval to build sha256:635d2edf05f7a2a13623152276b548c23f6c3ac61fc08647c80de6745bd74c20 and artifact kind skill

Closes #215

Follow-up evaluation

Independent review

  • Routing preflight: all three commits are authored by Codex, so the required reviewer is Claude.
  • Restacked Fable review (41b7e7b..df5af80): must 0 / should 2 / nit 3, verdict WARNING.
  • Fix commit: bf27b32.
  • Final cumulative Fable re-review (41b7e7b..bf27b32): all five prior findings RESOLVED, no new regressions; must 0 / should 0 / nit 0, verdict APPROVE.

Validation

  • all 17 scripts/tests/*.sh self-tests
  • scripts/check-manifests.sh — 21 manifests passed
  • scripts/check-injection.sh — expected medium runtime-state findings only; current build-bound human approval verified
  • scripts/build.sh — 42 artifacts built
  • scripts/register.sh — 42 registered, pending 0
  • all 12 runtime frontmatter descriptions parse and remain within 1024 bytes (571–876 bytes)
  • catalog query returns 12 unique artifact_kind=skill names
  • personal-asset-miner catalog build ID exactly matches the approved hash and artifact kind
  • staged public-safety gate passed for the review fix

@kosako
kosako force-pushed the codex/issue-214-review-executor-verdict branch from 91ec0c2 to d2a3999 Compare July 15, 2026 16:14
kosako and others added 2 commits July 16, 2026 01:34
Refs #215

Co-Authored-By: Codex <noreply@openai.com>
Refs #215

Co-Authored-By: Codex <noreply@openai.com>
@kosako
kosako force-pushed the codex/issue-215-routing-contracts branch from 18b4aa4 to df5af80 Compare July 15, 2026 16:37
@kosako
kosako changed the base branch from codex/issue-214-review-executor-verdict to main July 15, 2026 16:37
Refs #215

Co-Authored-By: Codex <noreply@openai.com>
@kosako
kosako marked this pull request as ready for review July 15, 2026 16:47
@kosako
kosako merged commit 155329d into main Jul 15, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

standardize skill descriptions and cross-skill routing contracts

1 participant