From e6f49f2bd1b4ce1775af1ac93eb92d20f9944ab7 Mon Sep 17 00:00:00 2001 From: Tym Rabchuk Date: Thu, 9 Apr 2026 19:44:00 -0400 Subject: [PATCH 1/4] Add mega-pr skill for combined parallel PR review Convenience skill that launches both devkit:tri-review and pr-review-toolkit:review-pr simultaneously. Context-activated via natural language ("mega pr", "mega review"). --- skills/mega-pr/SKILL.md | 46 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) create mode 100644 skills/mega-pr/SKILL.md diff --git a/skills/mega-pr/SKILL.md b/skills/mega-pr/SKILL.md new file mode 100644 index 0000000..6774b32 --- /dev/null +++ b/skills/mega-pr/SKILL.md @@ -0,0 +1,46 @@ +--- +name: mega-pr +description: Run both tri-review and pr-review-toolkit review-pr in parallel for maximum coverage — use when the user asks for mega PR review, mega-pr, mega review, full PR review, or wants both tri-review and pr-review-toolkit at once. +--- + +# Mega PR Review + +Run `/devkit:tri-review` and `/pr-review-toolkit:review-pr` simultaneously for maximum review coverage across all available agents. + +## Step 1: Launch Both Reviews in Parallel + +Invoke both skills at the same time using the Skill tool: + +**[PARALLEL]** — both calls in a single message: + +1. `Skill: devkit:tri-review` — dispatches Claude + Codex + Gemini reviewers +2. `Skill: pr-review-toolkit:review-pr` — dispatches specialized aspect reviewers (silent-failure-hunter, type-design-analyzer, test-analyzer, code-reviewer, etc.) + +Pass through any user-provided arguments (custom prompt, specific files) to both skills. + +## Step 2: Present Combined Results + +After both complete, present a single unified report: + +``` +## Mega PR Review: {branch_name} + +### tri-review Results +{output from tri-review — consensus + unique findings} + +### pr-review-toolkit Results +{output from pr-review-toolkit — categorized by severity} + +### Summary +- Total agents participated: {count} +- Critical issues: {count} +- Warnings: {count} +- Suggestions: {count} +``` + +## Rules + +- Always run both in parallel — never sequential +- If one skill fails, still present the other's results +- Do not deduplicate across the two — let the user see both perspectives +- Pass user arguments to both skills unchanged From d91fdddbd70cf5d41f98d3f18d6306f6b7362e29 Mon Sep 17 00:00:00 2001 From: Tym Rabchuk Date: Thu, 9 Apr 2026 19:46:02 -0400 Subject: [PATCH 2/4] Fix linter: shorten skill description line --- skills/mega-pr/SKILL.md | 31 ++++++++++++++----------------- 1 file changed, 14 insertions(+), 17 deletions(-) diff --git a/skills/mega-pr/SKILL.md b/skills/mega-pr/SKILL.md index 6774b32..4047b94 100644 --- a/skills/mega-pr/SKILL.md +++ b/skills/mega-pr/SKILL.md @@ -1,46 +1,43 @@ --- name: mega-pr -description: Run both tri-review and pr-review-toolkit review-pr in parallel for maximum coverage — use when the user asks for mega PR review, mega-pr, mega review, full PR review, or wants both tri-review and pr-review-toolkit at once. +description: Run tri-review and pr-review-toolkit review-pr in parallel for maximum coverage — use when asked for mega PR review, mega-pr, mega review, full PR review, or both review tools at once. --- # Mega PR Review -Run `/devkit:tri-review` and `/pr-review-toolkit:review-pr` simultaneously for maximum review coverage across all available agents. +Run `/devkit:tri-review` and `/pr-review-toolkit:review-pr` simultaneously for maximum review coverage. -## Step 1: Launch Both Reviews in Parallel +## Step 1: Launch Both Reviews -Invoke both skills at the same time using the Skill tool: - -**[PARALLEL]** — both calls in a single message: +**[PARALLEL]** — invoke both in a single message using the Skill tool: 1. `Skill: devkit:tri-review` — dispatches Claude + Codex + Gemini reviewers 2. `Skill: pr-review-toolkit:review-pr` — dispatches specialized aspect reviewers (silent-failure-hunter, type-design-analyzer, test-analyzer, code-reviewer, etc.) -Pass through any user-provided arguments (custom prompt, specific files) to both skills. +Pass any user-provided arguments (custom prompt, specific files) to both skills unchanged. ## Step 2: Present Combined Results -After both complete, present a single unified report: +After both complete, present a unified report: ``` -## Mega PR Review: {branch_name} +## Mega PR Review: ### tri-review Results -{output from tri-review — consensus + unique findings} + ### pr-review-toolkit Results -{output from pr-review-toolkit — categorized by severity} + ### Summary -- Total agents participated: {count} -- Critical issues: {count} -- Warnings: {count} -- Suggestions: {count} +- Total agents participated: +- Critical issues: +- Warnings: +- Suggestions: ``` ## Rules - Always run both in parallel — never sequential - If one skill fails, still present the other's results -- Do not deduplicate across the two — let the user see both perspectives -- Pass user arguments to both skills unchanged +- Do not deduplicate — let the user see both perspectives From fb22ba56b8905f5d6ba0762e4060d90387d2a718 Mon Sep 17 00:00:00 2001 From: Tym Rabchuk Date: Thu, 9 Apr 2026 19:48:15 -0400 Subject: [PATCH 3/4] Update skill counts to 19 in README and ROADMAP --- README.md | 2 +- ROADMAP.md | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 4fb6ee9..e0342b1 100644 --- a/README.md +++ b/README.md @@ -307,7 +307,7 @@ Self-Improvement (self:* commands) ``` devkit/ ├── commands/ # 24 slash commands -├── skills/ # 18 context-activated skills +├── skills/ # 19 context-activated skills ├── agents/ # 6 agents (reviewer, researcher, improver, ...) ├── hooks/ # 10 hooks (safety, security, quality gates) ├── workflows/ # 18 YAML workflow definitions diff --git a/ROADMAP.md b/ROADMAP.md index 03c93a0..bea7156 100644 --- a/ROADMAP.md +++ b/ROADMAP.md @@ -34,6 +34,6 @@ Items below were on the roadmap but determined to be unnecessary — either alre | Stop hook redesign | Still fires every turn, but exits early with `approve` when no files are changed — near-instant on clean trees, so the performance concern is moot. Revisit only if it causes measurable latency. | | Cost event hooks | Budget enforcement already exists in the Go engine via `overBudget()` + `addCost()` callbacks with hard limits | | Execution registry | Step tracking already handled by SQLite via `lib.DB` with status, cost, and timing per step | -| Preset library | The 18 YAML workflows and 18 skills already serve this purpose | +| Preset library | The 18 YAML workflows and 19 skills already serve this purpose | | Framework-specific review checklists | `lang-review.sh` covers language-level patterns; framework-specific rules are better added per-project via hookify | | Conditional hook firing | Hooks already self-filter internally (extension checks, changed-file checks); a generic condition system adds complexity for no current need | From 147a9ab561afdfddfc91d3ba5b2e2cdf22657602 Mon Sep 17 00:00:00 2001 From: Tym Rabchuk Date: Thu, 9 Apr 2026 19:50:32 -0400 Subject: [PATCH 4/4] =?UTF-8?q?Fix=20ROADMAP=20header=20skill=20count:=201?= =?UTF-8?q?8=20=E2=86=92=2019?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- ROADMAP.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ROADMAP.md b/ROADMAP.md index bea7156..230b052 100644 --- a/ROADMAP.md +++ b/ROADMAP.md @@ -3,7 +3,7 @@ ## Implemented - **24 slash commands** — Lifecycle workflows, self-improvement loops, multi-agent dispatch, project health audit, post-PR monitoring, AST repo mapping, autoresearch-inspired self-audit, autoloop, setup-rules -- **18 context-activated skills** — 9 auto-trigger workflows (test-gen, doc-gen, changelog, onboard, research, deep-research, scrape, autoloop, adr) + 6 coding principles (executing, clean-code, DRY, YAGNI, dont-reinvent, stuck) + 2 tools (gcli, creating-workflows) + 1 iteration memory (scratchpad) +- **19 context-activated skills** — 9 auto-trigger workflows (test-gen, doc-gen, changelog, onboard, research, deep-research, scrape, autoloop, adr) + 6 coding principles (executing, clean-code, DRY, YAGNI, dont-reinvent, stuck) + 2 tools (gcli, creating-workflows) + 1 iteration memory (scratchpad) + 1 orchestration (mega-pr) - **6 agents** — Scoped tool access, worktree isolation, model assignment - **10 hooks** — Safety (destructive command blocking, edit-time security patterns, PR gate), observability (audit trail, slop detection, post-validation, subagent verification, language-aware code review), optimization (RTK token compression) - **Graceful degradation** — tri:* commands work with 1-3 agents depending on installed CLIs