fxa-review-quick

Review FXA commits for security, conventions, logic, tests, and migrations.

683|238|Updated Jun 3, 2015
One-click install
npx skills add https://github.com/mozilla/fxa --skill fxa-review-quick
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: fxa-review-quick
Source: https://github.com/mozilla/fxa/tree/main/.claude/skills/fxa-review-quick
Command: npx skills add https://github.com/mozilla/fxa --skill fxa-review-quick

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill provides a fast FXA-specific commit review that analyzes the latest commit for security issues, coding conventions, logic/bug risks, test coverage, and migrations, helping teams ship safer FXA changes faster.

Core Features & Use Cases

  • FXA-specific security review across the latest commit to identify vulnerabilities.
  • Conformance and code-quality checks aligned with FXA conventions to reduce drift.
  • Logic, bug, and migration analysis to surface risks before merging.
  • Tests and coverage assessment to gauge test strength and detection gaps.
  • Deterministic, single-context execution with no subagents for predictable results.

Quick Start

Provide a commit reference (default HEAD) to review, and the tool will perform a fast FXA-specific, single-pass review of security, conventions, logic, tests, and migrations.

Frequently Asked Questions about fxa-review-quick

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

FAQPage Schema
How do I review a commit for security and code quality issues in FXA repositories?

A single-pass commit review analyzes the latest commit or a specified ref in FXA repositories to surface security vulnerabilities, convention drift, logic bugs, test gaps, and migration risks for safer merges.

What does an FXA-specific commit review check for?

An FXA-specific commit review checks for security issues, coding convention conformance, logic and bug risks, test coverage gaps, and migration risks to ensure deterministic coverage before shipping changes.

How do I run a fast commit review without using subagents?

You can run a fast commit review without subagents by using a deterministic, single-context execution tool that applies a predefined set of checks directly to the specified commit ref in your FXA repository.

Can I review an older commit ref instead of HEAD in an FXA repository?

Yes, you can review an older commit ref instead of HEAD by providing a specific commit reference, and the review tool will apply its security, test, migration, and code quality checks to that exact ref.

Does this commit review tool assess test coverage and database migrations?

Yes, the commit review tool assesses test coverage strength and detection gaps, while also analyzing database migrations to surface potential risks before the FXA changes are merged.

What is the best way to ensure deterministic code review results for FXA changes?

The best way to ensure deterministic code review results for FXA changes is to use a single-context execution tool that applies a predefined set of checks, avoiding subagents to guarantee predictable and consistent coverage.