Model PR Diff Dossier
Documentsmodel-pr-diff-dossierModel PR Diff Dossier is an Agent Skill for documents published in the BBuf/AI-Infra-Auto-Driven-SKILLS repository on GitHub: a SKILL.md instruction file an AI agent can load to do this task.
Use when creating or revising model PR optimization history documents for SGLang, vLLM, or another serving framework that cite GitHub PRs. Requires manual, per-PR source-diff review and documentation of motivation, key implementation approach, most important code excerpts, reviewed files, and validation implications instead of generated or one-line summaries.
How to use this skill
Bring this guide into your coding agent with a prompt tailored to the tool you use.
- Open your project in Codex.
- Copy and paste the prompt below.
- Review files and risks before approving any action.
Inspect the Agent Skill at https://github.com/BBuf/AI-Infra-Auto-Driven-SKILLS/blob/HEAD/skills/model-optimization/model-pr-diff-dossier/SKILL.md and all supporting files. Report provenance, permissions, scripts, dependencies, and security risks. Propose an exact installation into .agents/skills/model-pr-diff-dossier, but wait for my approval before writing files or running scripts.
Copying this prompt does not install or run the skill. Review third-party files before use.
Model PR Diff Dossier
Use this skill before publishing any model PR optimization history document that cites framework PRs.
Non-Negotiable Standard
Do not summarize a PR with only a title-level sentence. Do not use a script or bulk generator to fill motivation, implementation notes, or code excerpts.
For every PR cited as model optimization evidence, the document must include or link to a diff-reviewed PR card with:
- PR link, title, state, merge time when available, additions/deletions, and changed-file count.
- Motivation: why the PR existed, inferred from PR body, title, issue context, docs changes, tests, and code diff.
- Key implementation idea: what runtime path changed and how the patch implements the change.
- Key code excerpts: short, relevant snippets from the actual diff, not invented pseudocode.
- Reviewed files: important files from the full diff, grouped by runtime/docs/tests where possible.
- Validation implications: tests, benchmark paths, launch flags, or regression lanes implied by the diff.
- Diff coverage note: state that the full diff was fetched/read and include diff line count.
Workflow
- Collect exact PR links from the target model history files. Use GitHub PR URLs, not bare
#123text. - Open each PR diff directly with GitHub,
gh pr diff, or the local framework repository commit. Read the changed source files, not just the PR title. - For merged PRs, cross-check the final mainline code in the relevant framework checkout when the diff is ambiguous.
- Write the PR card manually in the matching model history document. Use
references/card-schema.mdwhen you need the exact card shape. The card must name concrete files/functions/classes and include a short real code excerpt. - For docs-only or config-only PRs, quote the exact command/config line that changed and explain why it matters for serving or validation.
- After each model family, review the cards for repeated shallow words such as "follow-up", "bugfix", or "optimization"; replace them with concrete implementation detail.
- Run repository tests and formatting before publishing.
Review Gate
A model PR history is not ready if any PR card says only "follow-up", "bugfix", "docs", or "optimization" without:
- named files/functions/classes touched by the diff,
- a concrete motivation,
- a concrete implementation summary,
- and at least one code excerpt or an explicit reason why the PR is docs-only.