cookbook-review-pr

Reviews SGLang Cookbook pull requests against a config-driven contribution checklist.

33.0k|8.4k|Updated Jan 8, 2024
One-click install
npx skills add https://github.com/sgl-project/sglang --skill cookbook-review-pr
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: cookbook-review-pr
Source: https://github.com/sgl-project/sglang/tree/main/.claude/skills/cookbook-review-pr
Command: npx skills add https://github.com/sgl-project/sglang --skill cookbook-review-pr

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Reviewing SGLang Cookbook pull requests requires checking dozens of rules across config files, MDX pages, benchmarks, navigation, and Mintlify syntax, which is slow and error-prone when done manually.

Core Features & Use Cases

  • Automated PR Fetching: Pulls the PR diff, metadata, duplicate candidates, and prior reviewer comments via the GitHub CLI.
  • 15-Section Checklist: Validates file hygiene, config quality, the 5-dimension cell matrix, benchmarks, doc-config parity, commands, frontmatter, navigation, links, Mintlify MDX syntax, quantization rules, scope, duplicates, and grammar.
  • Structured Verdicts: Outputs per-file PASS/ISSUE/BLOCK findings plus an overall APPROVE, REQUEST CHANGES, or BLOCKED recommendation.
  • Use Case: A maintainer runs the skill on PR #1234 adding a new model page and immediately sees that the config is missing accuracyLabels, a cell bakes in --reasoning-parser, and the docs.json nav entry still has an .mdx extension.

Quick Start

Ask the assistant to run /cookbook-review-pr followed by the pull request number to review that SGLang Cookbook PR against the full checklist.

Frequently Asked Questions about cookbook-review-pr

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

FAQPage Schema
How do I review an SGLang Cookbook pull request?

Run /cookbook-review-pr followed by the PR number. The skill fetches the diff and metadata with the GitHub CLI, checks for duplicate PRs, runs every checklist item, and outputs per-file verdicts plus an overall recommendation.

What does the cookbook PR review checklist check?

It covers 15 areas: file hygiene, config quality, the 5-dimension cell matrix, benchmarks, doc-config parity, launch commands and ports, MDX frontmatter, navigation and homepage cards, links and factual accuracy, Mintlify MDX syntax, quantization rules, scope, duplicates, build validation, and grammar.

Which files should a cookbook model-add PR touch?

Only docs/src/snippets/configs/<vendor>*.mdx, docs/docs.json, the category intro.mdx vendor card, and a logo PNG for new vendors. The shared engines _deployment.jsx and _playground.jsx must not change in a model-add PR.

Why does the review flag python -m sglang.launch_server in docs?

The launch_server entry point is deprecated; cookbook pages must use sglang serve on port 30000. The skill flags any prose or cell that reintroduces the old launcher or a mismatched port such as 8000.

Can the review skill validate Mintlify MDX syntax?

Yes. It flags forbidden Docusaurus admonitions, pipe tables, inline details blocks, unlabeled code fences, and missing Accordion wrappers, and suggests running mint validate and mint broken-links in the docs directory.