review-issue

Validate PR implementations against enriched GitHub issue specs using mix consistency and mix test.

11|Updated Dec 28, 2024
One-click install
npx skills add https://github.com/wadvanced/aurora_uix --skill review-issue-wadvanced
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: review-issue
Source: https://github.com/wadvanced/aurora_uix/tree/main/.claude/skills/review-issue
Command: npx skills add https://github.com/wadvanced/aurora_uix --skill review-issue-wadvanced

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Review-issue eliminates the uncertainty of whether an implementation truly satisfies an enriched spec by running the project quality gate, verifying acceptance-criterion completeness, and recording any missing tests or coverage directly back into the issue.

Core Features & Use Cases

  • Automated quality gate: Runs mix consistency and mix test, turning failures into actionable, line-cited flags.
  • Spec-driven acceptance-criterion review: Scores each AC as PASS/FAIL/N/A based on the diff scope and (when required) test coverage or Lite-mode test-hints.
  • Deterministic gap reporting: Writes missing gaps, coverage issues, and blocking quality flags into a structured marker block in the GitHub issue body for the next iteration.

Quick Start

Run the review skill after code-issue finishes to execute the quality gate, assess AC completeness against the enriched spec, and update the issue’s review-gaps marker block with any outstanding gaps.

Frequently Asked Questions about review-issue

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I validate a PR implementation against a GitHub issue spec in Elixir?▼

To validate a PR implementation, the review-issue skill runs mix consistency and mix test, scoring each acceptance criterion against the actual changed files to ensure the Elixir/Phoenix code matches the enriched GitHub issue specification.

How does acceptance criterion scoring work for Phoenix LiveView pull requests?▼

Acceptance criterion scoring evaluates each criterion as PASS, FAIL, or N/A by computing the diff scope via origin/main...HEAD and enforcing review mode rules, checking whether test ownership requires actual test coverage or Lite-mode test-hints for Phoenix LiveView changes.

Can I use mix test quality gate enforcement for Ash CRUD-related work?▼

Yes, you can enforce CI quality gates for Ash CRUD-related work by running mix test and mix consistency, which transforms any failures into actionable, line-cited flags and verifies that the implementation satisfies the enriched spec criteria.

What is the best way to record missing test coverage gaps back into a GitHub issue?▼

The best way to record missing test coverage gaps is to persist them into a structured marker block within the GitHub issue body, creating deterministic gap reporting for the next development iteration.

Do I need an enriched-spec block to run spec validation on my Elixir pull request?▼

Yes, you need an enriched-spec block fetched from the GitHub issue because spec validation requires this enriched specification to score acceptance criteria and determine whether test ownership mandates full coverage or Lite-mode test-hints.

Why does mix consistency fail during PR review after code changes?▼

Mix consistency fails during PR review when the implementation does not match the enriched spec, and the quality gate captures these failures as actionable, line-cited flags written into the issue's review-gaps marker block.