complete-partial-pr

Evaluates partial pull requests to identify missing integration work for Pydantic AI patches.

19.2k|2.5k|Updated Jun 21, 2024
One-click install
npx skills add https://github.com/pydantic/pydantic-ai --skill complete-partial-pr
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: complete-partial-pr
Source: https://github.com/pydantic/pydantic-ai/tree/main/.agents/skills/complete-partial-pr
Command: npx skills add https://github.com/pydantic/pydantic-ai --skill complete-partial-pr

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill helps you evaluate a contribution that fixes only one visible symptom and determine whether the broader user workflow, integration contract, and adjacent surfaces are still incomplete.

Core Features & Use Cases

  • Scope validation: Checks whether a patch covers the full behavior expected by users, not just the first failing test.
  • Integration review: Maps related paths such as parsing, streaming, request/response handling, tools, metadata, docs, and tests.
  • Maintainable completion: Guides conservative implementation so the fix stays aligned with project history, specs, and maintainers' decisions.
  • Use case: A PR fixes one provider edge case, but this Skill helps confirm whether sync and async flows, roundtrip behavior, and regression tests also need updates.

Quick Start

Use this skill to review the partial PR, identify missing integration surfaces, and complete the fix with targeted tests.

Frequently Asked Questions about complete-partial-pr

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

FAQPage Schema
How do I evaluate if a pull request covers the full integration contract?

To evaluate a pull request against the full integration contract, map related codepaths like parsing, streaming, tools, and metadata to identify missing work beyond the initial patch. This ensures the PR satisfies complete user workflows rather than just fixing one failing test.

What is partial PR scope validation in code review?

Partial PR scope validation in code review is the process of checking whether a patch covers the full behavior expected by users, not just the first visible symptom. It involves mapping adjacent surfaces like sync and async flows to guarantee complete spec compliance.

How do I complete a partial Pydantic AI patch with regression tests?

To complete a partial Pydantic AI patch, apply conservative implementation aligned with project history and specs, then add focused regression tests. This validates that sync and async flows, roundtrip behavior, and adjacent workflow variants function correctly.

Does fixing one provider edge case require updates to streaming and parsing paths?

Fixing one provider edge case often requires updates to streaming and parsing paths because integration contracts demand consistent behavior across surfaces. You must check historical decisions and map related routes to ensure sync, async, and roundtrip behaviors are fully covered.

When should I not use a narrow patch fix during integration testing?

You should not use a narrow patch fix during integration testing when adjacent paths like request and response handling, tools, metadata, or docs remain incomplete. If the patch fails to satisfy the full integration contract, conservative implementation and broader regression tests are required.