From on-loop
Defines pass/fail criteria for phase transitions in the on-loop pipeline
How this skill is triggered — by the user, by Claude, or both
Slash command
/on-loop:quality-gateThe summary Claude sees in its skill listing — used to decide when to auto-load this skill
This skill defines the quality criteria that must be met before transitioning between phases.
This skill defines the quality criteria that must be met before transitioning between phases.
All paths below use <session-dir> to refer to the active session directory (e.g., .on-loop/sessions/20260426_143052_user-management-api/).
Check: Architect agent notes exist and contain a valid specification.
| Criteria | Required |
|---|---|
<session-dir>/agent-notes/architect.md exists | Yes |
| Specification has functional requirements | Yes |
| Specification has non-functional requirements | Yes |
| Security considerations documented | Yes |
| Architecture diagram present | Recommended |
On fail: Cannot proceed. Report to user.
Check: Plan is written and actionable.
| Criteria | Required |
|---|---|
<session-dir>/plan.md has content | Yes |
| Tasks are listed with assignments | Yes |
| Constraints documented | Yes |
On fail: Orchestrator rewrites plan.
Check: Coding agent completed implementation.
| Criteria | Required |
|---|---|
<session-dir>/agent-notes/coding.md exists | Yes |
Files created/modified listed in changes.log | Yes |
| No CRITICAL issues self-reported | Yes |
| Code compiles/parses without errors | Yes |
On fail: Cannot proceed. Coding agent must resolve.
Check: All tests pass.
| Criteria | Required |
|---|---|
<session-dir>/agent-notes/testing.md exists | Yes |
| All tests pass (zero failures) | Yes |
| Happy path tests exist | Yes |
| Error path tests exist | Yes |
On fail (retries remain): Transition back to CODE with test feedback. On fail (retries exhausted): Record TODO, proceed to SECURITY.
Check: No critical security vulnerabilities.
| Criteria | Required |
|---|---|
<session-dir>/agent-notes/security.md exists | Yes |
| No CRITICAL findings | Yes |
| No unmitigated HIGH findings | Yes |
| OWASP Top 10 review completed | Yes |
On fail (retries remain): Transition back to CODE with security findings. On fail (retries exhausted): Record TODO, proceed to DOC/BUILD.
Check: Both documentation and build agents completed.
| Criteria | Required |
|---|---|
<session-dir>/agent-notes/documentation.md exists | Yes |
<session-dir>/agent-notes/build.md exists | Yes |
| README exists or was updated | Recommended |
| CI configuration exists | Recommended |
On fail: Warn but proceed to REVIEW (doc/build issues are non-blocking).
Check: Reviewer approved the code.
| Criteria | Required |
|---|---|
<session-dir>/agent-notes/reviewer.md exists | Yes |
Verdict is APPROVE | Yes |
| No CRITICAL issues | Yes |
On fail (retries remain): Transition back to CODE with review feedback. On fail (retries exhausted): Record TODO, proceed to GIT with warnings.
Check: Git operations completed successfully.
| Criteria | Required |
|---|---|
| All changed files staged and committed | Yes |
| Branch pushed to origin | Yes |
PR created via gh pr create | Yes |
pr_url set in state.json | Yes |
On fail: Report git error to user. The user can retry with /on-loop-resume --from=GIT.
| Transition | Max Retries | Trigger |
|---|---|---|
| TEST → CODE | 3 | Test failures |
| SECURITY → CODE | 2 | CRITICAL/HIGH findings |
| REVIEW → CODE | 2 | REQUEST_CHANGES verdict |
Total maximum code iterations: 1 (initial) + 3 + 2 + 2 = 8
When retries are exhausted:
state.json[timestamp] orchestrator DECISION retry_exhausted — <details>npx claudepluginhub joestein/on-loop --plugin on-loopProvides CDSS development patterns for drug interaction checking, dose validation, clinical scoring (NEWS2, qSOFA), and alert classification integrated into EMR workflows.