skills.rest
zetaalphavector
zetaalphavectorOFFICIAL
Explore all skills by this author →
WebsiteGitHub

review-pr

Review GitHub pull requests for compliance with RAGElos Factory+Registry architecture, evaluators, and security practices.

129|7|Updated Oct 10, 2023
Search Tags:#testing#architecture#security#review#PR#factory-registry
One-click install
npx skills add https://github.com/zetaalphavector/RAGElo --skill review-pr-zetaalphavector
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: review-pr
Source: https://github.com/zetaalphavector/RAGElo/tree/main/.agents/skills/review-pr
Command: npx skills add https://github.com/zetaalphavector/RAGElo --skill review-pr-zetaalphavector

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

PR review guidelines ensure code changes align with RAGElo's architecture, uphold the Chameleon Principle, and protect API stability, security, and overall quality.

Core Features & Use Cases

  • Validates PRs against the Factory+Registry pattern, evaluator conventions, and configuration standards.
  • Assesses adherence to existing code patterns, testing coverage, and security considerations.
  • Use Case: Review PRs proposing evaluator changes, new prompts, or API surface changes to ensure seamless integration.

Quick Start

Provide a concise, structured PR review summarizing findings and recommended actions.

Frequently Asked Questions about review-pr

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

FAQPage Schema
How do I automate PR review for architecture compliance and security?▼

Automate PR review by evaluating code changes against Factory+Registry architecture, evaluator conventions, and security practices. It validates pattern conformance, API stability, and test coverage, returning a structured review summarizing findings and recommended actions.

What does a Factory+Registry pattern compliance check for PRs evaluate?▼

A Factory+Registry pattern compliance check evaluates PR changes for pattern conformance, API stability, test coverage, and security implications. It verifies modifications to evaluators, LLM providers, configurations, or tooling align with repository architecture.

Can I use automated PR review for evaluator changes and configuration updates?▼

Yes, automated PR review applies to PRs modifying evaluators, LLM providers, configurations, or tooling. It validates changes against existing code patterns, configuration standards, and security considerations to ensure seamless integration.

How does PR review enforce API stability and testing coverage?▼

PR review enforces API stability and testing coverage by assessing adherence to existing code patterns and verifying test coverage. It identifies changes that violate API stability or lack sufficient tests, returning recommended actions.

What is the best way to review PRs proposing new prompts or API surface changes?▼

The best way to review PRs proposing new prompts or API surface changes is to validate them against evaluator conventions and configuration standards. This ensures pattern conformance, protects API stability, and upholds the Chameleon Principle.

When do I need architecture-focused PR review for evaluator modifications?▼

You need architecture-focused PR review when PRs modify evaluators, LLM providers, configurations, or tooling. It verifies pattern conformance, API stability, test coverage, and security implications to ensure seamless integration across the repository.

Related Skills

Explore More Software Engineering Skills→
lhotarilhotari
8

pr-review

Review GitHub pull requests locally using metadata and diff context.

Community
Advanced
drdatarulzdrdatarulz
3

integration-test-v3

Automate integration testing for merged implementations using Testcontainers.

Community
Advanced
AvihusittonAvihusitton

logic-lens

Detect logic flaws, race conditions, and security risks in code reviews.

Community
Intermediate
TopengDevTopengDev

preflight

Run local linting, testing, and building checks before pushing code.

Community
Advanced