Review Implementation (internal scope/classification engine)
openbooklet.com/s/review-implementation-internal-scopeclassification-engineopenbooklet.com/s/[email protected]GET /api/v1/skills/review-implementation-internal-scopeclassification-engineThe classification engine the review/audit skills compose: it consumes the **synthesized findings table** (the fused output of the applicable per-axis passes), verifies every applicable axis is represented, and returns the classified decision table â then stops. Never refactors or edits code. It o
The manager's **"can this ship?"** gate. A read-first audit over the *entire* PR â its SPEC, all phases, docs, CI status, and review receipt â that returns a single verdict: **merge-ready** or a ranked list of **blockers**. **Never edits, refactors, or merges.** The human merges, or an active `s
Internal agentic-workflow maintenance: after SKILL.md edits, bump semver, lint authoring rules, and synchronize changelogs, READMEs, routing metadata, and migrations. Triggers: "bump the skill", "update the changelog", "version bump".
Internal shared owner of evidence-grounded authoring: the fixed claim/authority/evidence/freshness/unknown row, the ordered inventory-evidence-draft-readiness passes, `artifactRevisionId` rotation, and the no-progress rule. Consumed by `design-feature`, `plan-feature`, `plan-feature-scaffold` and `plan-fix`. Never reviews or approves. Not a menu entry.
Repair persisted fix-now findings. `review-change` and `audit-pr` classify; this skill fixes each root cause, never relabeling, deferring, or weakening its check.
Generate incremental, diff-driven developer guides through the project's detected docs adapter. Never regenerate the whole site, scaffold it, or edit source. Triggers: "generate-docs", "generate the docs", "document this unit".
Internal contract: the single owner of the bounded pre-write implementation discovery step. One read-only mapper that closes seven evidence questions, emits one fixed compact map, and routes READY | REPLAN | NEEDS-DESIGN | BLOCKED before any branch, planning, or source write. Consumed only by execute-phase on the pre-write route. Not a menu entry.
Adapt the workflow scaffold to a new repository or add only missing substrate blocks to an existing install. Every install, hook, and overwrite needs explicit consent. Triggers: "init-workspace", "set up agentic workflow", "upgrade workflow scaffold".
Capture the *why* and the *what-next* of a working session â the context that git history alone never records. A commit says what changed; a session log says what you were trying to do, what you decided, and where to resume.
Internal machine-result contract for headless agentic-workflow drivers. The executable source is @gtrabanco/agentic-workflow-schema: strict Envelope v2 for workflow-status, compact SkillOutcome v1 for driven work, compatibility parsing, and deterministic document snapshots. Not a menu entry.
Internal contract: the single owner of the eight phase-lint rules, the fixed PASS/BLOCKED result, and the normalized phase fingerprint. Consumed by plan-feature-scaffold, plan-fix, and execute-phase. Not a menu entry.
Internal step of plan-feature: turn a feature-request issue into a scoped, sized, roadmap-mapped SPEC **product half** (capability closure satisfied) with Closes #N traceability.
Internal owner of the pre-execution review cycle and the planning ledgers: independence, unioned findings, counter-evidence dismissal, no-progress, `CONVERGENCE-ANOMALY`, and the evidence/obligation/findings tables. Consumed by `review-spec`, `review-plan`, and the authoring skills. Not a menu entry.
Product-wide health check. It only writes `docs/audits/<n>-<YYYY-MM-DD>.md`; every proposed action remains a user decision.
Composed by `review-change` / `product-audit` within their conversation â on any agent, follow this file inline as the routed step. **Findings only; never edits, never refactors.**
Composed by `review-change` / `product-audit` within their conversation â on any agent, follow this file inline as the routed step. **Findings only; never edits, never refactors.**
Composed by `review-change` / `product-audit` within their conversation â on any agent, follow this file inline as the routed step. **Findings only; never edits, never refactors.**
Composed by `review-change` / `product-audit` within their conversation â on any agent, follow this file inline as the routed step. **Findings only; never edits, never refactors.**
Composed by `review-change` / `product-audit` within their conversation â on any agent, follow this file inline as the routed step. **Findings only; never edits, never refactors.**
Composed by `review-change` / `product-audit` within their conversation â on any agent, follow this file inline as the routed step. **Findings only; never edits, never refactors.**
Composed by `review-change` / `product-audit` within their conversation â on any agent, follow this file inline as the routed step. **Findings only; never edits, never refactors.**
Auto-indexed from gtrabanco/agentic-workflow
Are you the author? Claim this skill to take ownership and manage it.
Related Skills
graceful-error-recovery
Use this skill when a tool call, command, or API request fails. Diagnose the root cause systematically before retrying or changing approach. Do not retry the same failing call without first understanding why it failed.
audience-aware-communication
Use this skill when writing any explanation, documentation, or response that will be read by someone else. Match vocabulary, depth, and format to the audience's expertise level before writing.
Refactoring Expert
Expert in systematic code refactoring, code smell detection, and structural optimization. Use PROACTIVELY when encountering duplicated code, long methods, complex conditionals, or any code quality issues. Detects code smells and applies proven refactoring techniques without changing external behavior.
Research Expert
Specialized research expert for parallel information gathering. Use for focused research tasks with clear objectives and structured output requirements.
clarify-ambiguous-requests
Use this skill when the user's request is ambiguous, under-specified, or could be interpreted in multiple ways. If proceeding with a wrong assumption would waste significant work, always ask exactly one focused clarifying question before doing anything.
structured-step-by-step-reasoning
Use this skill for any problem that involves multiple steps, tradeoffs, or non-trivial logic. Think out loud before answering to improve accuracy and transparency. Apply whenever the answer is not immediately obvious.