Skip to content

docs: distinguish target acceptance from local verification #3740

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
eneekoruiz wants to merge 3 commits into
base: main
Choose a base branch
from
Open

docs: distinguish target acceptance from local verification #3740

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
73286a2
docs: distinguish target acceptance from local verification
eneekoruiz Oct 26, 2026
d4c1fec
docs: sync Pi verification guidance and clarify target evidence
eneekoruiz Oct 26, 2026
03834ce
docs: carry target acceptance guidance to Codex and Kiro
eneekoruiz Oct 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion manifests/context-packs/skill-triggers@1.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"id": "hand-seeded",
"source": "manual-curation-pending-regeneration"
},
"registryDigest": "846ac8826dd91e81cd838c2b55d7be536af427eb453efa127501f891e5778ff8",
"registryDigest": "5d93820285d52a85a9e4525f465c3e083e42edcadcfa901efbd328fbc20d55b6",
"schemaVersion": 1,
"triggers": {
"skill:agent-runtime-gateway-smoke-test": [
Expand Down
14 changes: 14 additions & 0 deletions skills/verification-loop/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,18 @@ Review each changed file for:
- Missing error handling
- Potential edge cases

## Target Acceptance

When the requested outcome requires deployment or target-environment verification,
report target acceptance separately from local checks. Record the environment and
installed version, the evidence, and the next permitted check or precise access
dependency. Passing tests, review, merge, or release does not verify that target.
Comment thread
eneekoruiz marked this conversation as resolved.
Outdated
Do not access production or make external changes without authorization. For
local-only tasks, mark target acceptance as not applicable.
Comment thread
eneekoruiz marked this conversation as resolved.

For example: local checks PASS; staging verification NOT PERFORMED (no access);
next step: an authorized maintainer runs the staging smoke test.

## Output Format

After running all phases, produce a verification report:
Expand All @@ -110,6 +122,8 @@ Security: [PASS/FAIL] (X issues)
Diff: [X files changed]

Overall: [READY/NOT READY] for PR
Target: [VERIFIED/FAILED/NOT PERFORMED/NOT APPLICABLE] (environment/version, evidence)
Next permitted step: [check or access dependency; none if complete/not applicable]

Issues to Fix:
1. ...
Expand Down
Loading