Skip to content

feat(wizrd-review): replace plugin with our own review prompt#5

Merged
filippofilip95 merged 1 commit into
mainfrom
feat/wizrd-own-review-prompt
May 19, 2026
Merged

feat(wizrd-review): replace plugin with our own review prompt#5
filippofilip95 merged 1 commit into
mainfrom
feat/wizrd-own-review-prompt

Conversation

@filippofilip95
Copy link
Copy Markdown
Member

What

Replace `code-review@claude-code-plugins` plugin invocation with our own inline prompt in `.github/workflows/wizrd-review.yml`.

Why

The plugin works but trades off our harness conventions:

  • Plugin only auto-discovers `CLAUDE.md`; we want `REVIEW_RULES.md → CODE_REVIEW.md → CLAUDE.md → .claude/ci-agent-context.md`
  • Plugin's output format ≠ our `🔴 BLOCKING / 🟡 REQUIRED / 🟢 ADVISORY / ✅ Good` format
  • Plugin doesn't know about our B/R/A rule codes
  • One less external dependency to track

We keep the good ideas from the plugin: skip closed/draft/trivial/already-reviewed, parallel sub-agents via the Task tool, confidence-score gate (default ≥80).

The critical bit — explicit `gh pr comment` step — is now hard-wired into the prompt with a "do NOT skip this step" instruction. That's what was missing in the original composite (v1 agent mode doesn't auto-post).

Quality

  • Review: self
  • Tests: will be validated on slovenskaznamka-web PR #234 next push
  • Gate: dispatch — pass

Impact

  • After merge + v1 retag: slovenskaznamka-web's existing `wizrd.yml` picks it up automatically
  • Other consuming repos unaffected until they adopt the reusable workflows

Drop dependency on anthropics/claude-code code-review plugin. Inline the
instructions so we control:
- discovery order: REVIEW_RULES.md → CODE_REVIEW.md → CLAUDE.md → .claude/ci-agent-context.md
- our B/R/A rule-code conventions
- our BLOCKING/REQUIRED/ADVISORY/Good output format
- explicit gh pr comment posting step (the bit Claude was skipping in agent mode)
- confidence threshold (default 80) and skip-trivial gating from the plugin design

Keeps the parallel sub-agent pattern via the Task tool. No marketplace hop.
@filippofilip95 filippofilip95 merged commit 796df17 into main May 19, 2026
@filippofilip95 filippofilip95 deleted the feat/wizrd-own-review-prompt branch May 19, 2026 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant