review-verify

Name: review-verify
Author: gtrabanco

32installs2forks

How to install review-verify?

gtrabanco/agentic-workflow/review-verify

$npx -y skills add gtrabanco/agentic-workflow --skill review-verify

Installs into the current project.

Summary

Internal run-it verification review pass of the agentic-workflow review pack — composed in-turn by review-change and product-audit; not a menu entry. Runs the project's gate and the changed behavior for real — commands, exit codes, observed output — instead of assuming; what cann

Files · 1

View on GitHub

SKILL.md

1	# Review Verify (internal)
2
3	Composed by `review-change` / `product-audit` within their conversation — on any
4	agent, follow this file inline as the routed step. **Findings only; never edits,
5	never refactors.**
6
7	## Scope
8
9	The diff or path/glob the caller passes; default the current change vs the
10	default branch. State the scope at the top of the returned table.
11
12	## Checklist (evaluate EVERY item — none is optional; n/a must be stated)
13
14	✓ The project's verification gate ran in THIS review (type-check, tests, build
15	per the project's declared commands) — paste exit status per command
16	✓ The change's primary claimed behavior was exercised for real (run the
17	CLI/endpoint/function; paste the actual observed output)
18	✓ At least one failure-mode from the SPEC's dev scenarios was reproduced (bad
19	input, missing file, error path) and behaved as specified
20	✓ New/changed tests actually fail when the implementation is reverted or
21	broken (mutate or stash to prove they assert something — restore after)
22	✓ No test was skipped/disabled to get green (grep for skip/only/todo markers
23	in the diff)
24	✓ The build artifact/dev server starts cleanly (no new warnings that indicate
25	breakage)
26	✓ Anything that can only be confirmed by a human (visual, device, locale,
27	load) is listed explicitly under "Manual" — never silently dropped
28
29	## Return exactly
30
31	```
32	REVIEW VERIFY — scope: <scope>
33
34	\| # \| Finding \| Sev \| Evidence \| Suggested fix \|
35	\|---\|---------\|-----\|----------\|---------------\|
36	\| 1 \| <what> \| critical\|major\|minor \| <file:line> \| <smallest action> \|
37
38	Checklist: <n> evaluated, <n> pass, <n> findings, <n> n/a (<which + why>)
39	Summary: <1-2 sentences>
40	Manual (a human must check): <bullet list, or "none">
41	Decision: PASS \| FAIL
42	```
43
44	FAIL if any critical or major finding is open; PASS otherwise. Minor findings
45	never block — they route to the caller's triage step.
46
47	## Done when
48
49	- Every checklist item was evaluated with evidence (file:line or command output)
50	or explicitly marked n/a with the reason.
51	- The fixed-format block above is returned — nothing more, nothing less — and
52	no code was changed.

People who liked this also viewed

6 picks

Type

review-verify

32installs2forks

View on GitHub

How to install review-verify?

gtrabanco/agentic-workflow/review-verify

$npx -y skills add gtrabanco/agentic-workflow --skill review-verify

Installs into the current project.

Summary

Files · 1

View on GitHub

SKILL.md

1	# Review Verify (internal)
2
3	Composed by `review-change` / `product-audit` within their conversation — on any
4	agent, follow this file inline as the routed step. **Findings only; never edits,
5	never refactors.**
6
7	## Scope
8
9	The diff or path/glob the caller passes; default the current change vs the
10	default branch. State the scope at the top of the returned table.
11
12	## Checklist (evaluate EVERY item — none is optional; n/a must be stated)
13
14	✓ The project's verification gate ran in THIS review (type-check, tests, build
15	per the project's declared commands) — paste exit status per command
16	✓ The change's primary claimed behavior was exercised for real (run the
17	CLI/endpoint/function; paste the actual observed output)
18	✓ At least one failure-mode from the SPEC's dev scenarios was reproduced (bad
19	input, missing file, error path) and behaved as specified
20	✓ New/changed tests actually fail when the implementation is reverted or
21	broken (mutate or stash to prove they assert something — restore after)
22	✓ No test was skipped/disabled to get green (grep for skip/only/todo markers
23	in the diff)
24	✓ The build artifact/dev server starts cleanly (no new warnings that indicate
25	breakage)
26	✓ Anything that can only be confirmed by a human (visual, device, locale,
27	load) is listed explicitly under "Manual" — never silently dropped
28
29	## Return exactly
30
31	```
32	REVIEW VERIFY — scope: <scope>
33
34	\| # \| Finding \| Sev \| Evidence \| Suggested fix \|
35	\|---\|---------\|-----\|----------\|---------------\|
36	\| 1 \| <what> \| critical\|major\|minor \| <file:line> \| <smallest action> \|
37
38	Checklist: <n> evaluated, <n> pass, <n> findings, <n> n/a (<which + why>)
39	Summary: <1-2 sentences>
40	Manual (a human must check): <bullet list, or "none">
41	Decision: PASS \| FAIL
42	```
43
44	FAIL if any critical or major finding is open; PASS otherwise. Minor findings
45	never block — they route to the caller's triage step.
46
47	## Done when
48
49	- Every checklist item was evaluated with evidence (file:line or command output)
50	or explicitly marked n/a with the reason.
51	- The fixed-format block above is returned — nothing more, nothing less — and
52	no code was changed.

People who liked this also viewed

6 picks

Type