docs: standardize skill routing contracts - #222
Merged
Conversation
kosako
force-pushed
the
codex/issue-214-review-executor-verdict
branch
from
July 15, 2026 16:14
91ec0c2 to
d2a3999
Compare
kosako
force-pushed
the
codex/issue-215-routing-contracts
branch
from
July 15, 2026 16:37
18b4aa4 to
df5af80
Compare
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
marked this pull request as ready for review
July 15, 2026 16:47
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
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
personal-asset-minerhuman approval to buildsha256:635d2edf05f7a2a13623152276b548c23f6c3ac61fc08647c80de6745bd74c20and artifact kindskillCloses #215
Follow-up evaluation
Independent review
41b7e7b..df5af80):must 0 / should 2 / nit 3, verdict WARNING.bf27b32.41b7e7b..bf27b32): all five prior findings RESOLVED, no new regressions;must 0 / should 0 / nit 0, verdict APPROVE.Validation
scripts/tests/*.shself-testsscripts/check-manifests.sh— 21 manifests passedscripts/check-injection.sh— expected medium runtime-state findings only; current build-bound human approval verifiedscripts/build.sh— 42 artifacts builtscripts/register.sh— 42 registered, pending 0artifact_kind=skillnamespersonal-asset-minercatalog build ID exactly matches the approved hash and artifact kind