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.