review-implementation

Solid

Internal findings engine composed by review-change (and reused by the audit skills): two-phase find → classify pass ending in a classified decision table (fix-now / postpone / ignore / intentional-tradeoff). Findings only — never refactors.

Code & Development 19 stars 3 forks Updated 1 weeks ago MIT

Install

View on GitHub

Quality Score: 82/100

Stars 20%
43
Recency 20%
90
Frontmatter 20%
70
Documentation 15%
100
Issue Health 10%
50
License 10%
100
Description 5%
100

Skill Content

# Review Implementation (internal engine) The findings engine the review/audit skills compose: it **produces findings and a decision table, and stops** — never refactors or edits code. It owns the **review axes** (Phase 1) and the **classification rubric** (Phase 2) that `review-change`, `audit-pr`, and `product-audit` reference instead of restating. ## When to use - Invoked by `review-change` (the user-facing review entry) as its engine; the audit skills reference its rubric. - Run directly only when you want the raw classified pass without the platform-adaptive orchestration `review-change` adds. ## Scope Default target is the **current change** (branch diff vs. the default branch); accept an explicit path/glob to widen or narrow it. State the scope at the top of the report so the reader knows what was and wasn't reviewed. ## Step 0 — Discover the project (always first) Per the agent guide's **Workflow conventions** + **documentation map**, then read what THIS skill needs: the architecture/layering rules, the testing philosophy, and any runtime/platform, security, money, i18n/SEO/a11y and bundle rules. Pull the project's specific risk axes from its guardrail skills where present (architecture-pattern, runtime/platform, domain-rules). The axis list below is the default; the project's docs refine it. ## Context budget (hard rule) The scope is the diff. Beyond it, read **at most 10 non-diff files in full** for surrounding context (callers, contracts, tests); targe...

Details

Author
gtrabanco
Repository
gtrabanco/agentic-workflow
Created
1 months ago
Last Updated
1 weeks ago
Language
TypeScript
License
MIT

Bundled in these plugins

Similar Skills

Semantically similar based on skill content — not just same category

Code & Development Solid

review-change

Platform-adaptive review orchestrator. Reviews the current change by running review-implementation (find → classify) AND the workflow's own internal review pack — only the passes that apply to this project and this change (review-code, review-security, review-verify, review-debt, review-design, review-a11y, review-brand, review-perf, review-seo) — never the inapplicable ones (no a11y/SEO/brand for a CLI, library, or infra change). Self-contained: no external review skills required; installed platform skills run as optional extras. Synthesizes one classified report plus an explicit manual-verification checklist. Findings only — never refactors. On Claude Code and want hand-tuned per-skill model/effort tiers? Install the `#claude` branch instead (`npx skills add gtrabanco/agentic-workflow#claude`) — see the README. This branch is model-agnostic: the skill inherits whatever model and effort your agent session is already using. Triggers: "review this change", "full review before merge", "review-change", "run the

19 Updated 1 weeks ago
gtrabanco
AI & Automation Solid

review-code

Internal correctness + simplification review pass of the agentic-workflow review pack — composed in-turn by review-change and product-audit; not a menu entry. Checks correctness, error handling, duplication, dead code, and simplification opportunities against the project's own conventions. Findings only; never edits code.

19 Updated 1 weeks ago
gtrabanco
Web & Frontend Solid

review-design

Internal UI/UX design review pass of the agentic-workflow review pack — composed in-turn by review-change and product-audit; not a menu entry. Checks the changed UI against the project's design doc: consistency, states, responsiveness, and reuse — applies only when the project has a UI and the change touches it. Findings only; never edits code.

19 Updated 1 weeks ago
gtrabanco