pr-review

Reviews pull requests against Lumen design-system conventions with platform-routed, severity-scored findings.

23|5|Updated Apr 3, 2025
One-click install
npx skills add https://github.com/LedgerHQ/lumen --skill pr-review-ledgerhq
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: pr-review
Source: https://github.com/LedgerHQ/lumen/tree/main/.claude/skills/pr-review
Command: npx skills add https://github.com/LedgerHQ/lumen --skill pr-review-ledgerhq

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Reviewing changes in a cross-platform React and React Native monorepo is error-prone because the same finding category means different things on web (Tailwind) versus native (useStyleSheet), and manual reviewers often miss version-plan requirements or apply the wrong platform's rules. ## Core Features & Use Cases - Platform routing: Maps each changed file to React web or React Native guidelines so Tailwind rules never leak onto RN files and vice versa. - Severity-scored findings: Produces a flat findings table with 1-10 severity ratings, categories, and exact file:line locations, sorted by severity. - Version plan verification: Checks Nx version plans for missing plans, wrong bump types, and multi-package grouping. - Use Case: Paste a GitHub PR URL or point at your current branch, and receive a structured review covering correctness, type safety, performance, API parity, and release readiness while skipping issues already caught by ESLint. ## Quick Start Ask the assistant to review the current branch or a specific GitHub PR against Lumen's design-system conventions.

Frequently Asked Questions about pr-review

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

FAQPage Schema
How do I review a GitHub PR for a React design system?▼

Provide the PR URL or number and the review fetches it with gh pr view and gh pr diff, reads each changed file in full, and returns a severity-scored findings table. Local branches are reviewed by diffing against origin/main.

How does the review handle React Native versus web code?▼

Changed paths are mapped to their library and platform first, then the matching rules are applied: Tailwind conventions for libs/ui-react and useStyleSheet conventions for libs/ui-rnative. When both platforms are touched, an API-parity check compares prop names, defaults, and variants.

Does the review flag lint and formatting issues?▼

No. Issues already enforced by ESLint, such as banned imports, import ordering, type-style rules, and formatting, are explicitly excluded. The review focuses on problems a linter cannot catch, like wrong abstractions or raw Tailwind colors instead of design tokens.

What happens if a PR is missing an Nx version plan?▼

If production code under libs/*/src changed without a version plan, it is reported as a Major 8/10 finding in the Release category. Non-patch bump types, multi-package plan files, and wrong package names are flagged as Moderate 6/10.

Can a review return zero findings?▼

Yes. Zero findings is an explicitly valid result, and the review will say so and recommend approval. Findings are never padded or speculative; every reported issue must cite a concrete file and line.