cookbook-doc-review · git:20260901.0bb6ca7 · 2026-09-01 · sha256 7f4c5121dd438e2b
cookbook-doc-review git:20260901.0bb6ca7A
Immutable. This exact content is served forever at /api/v1/blob/7f4c5121dd438e2b.
--- name: cookbook-doc-review description: >- Review AGS Cookbook documentation for language, wording, logic, and Markdown layout from an external AGS user's perspective. Use before handing off changes to public READMEs, tutorials, reference guides, or example documentation anywhere in this repository. Do not use for source-code review or explicit contributor/governance documents. --- # Review AGS Cookbook Documentation Review only. Do not edit files during the review pass. ## Review Independence For a final review, use a fresh reviewer context when delegation is available. Provide the documentation paths, diff boundary, and user's documentation goal, but not the implementer's intended verdict, suspected findings, proposed rewrite, or implementation evidence. If a fresh reviewer is unavailable, state that limitation. ## Core Question Can a user who has only the public documentation understand the goal, prepare the prerequisites, run the steps, recognize success or failure, recover from common problems, and clean up without asking a repository maintainer or inspecting any other repository file? Every review decision should help answer this question. ## Audience Review every public cookbook document as material for people who want to use AGS. Readers are not expected to maintain this repository, publish its official artifacts, know its review history, or understand internal implementation decisions. This applies to CLI and SDK walkthroughs, Sandbox workflows, storage and networking guides, browser and mobile examples, custom-image instructions, Deployment tutorials, API references, and user-facing files under `dockerfiles/` or `skills/`. Explicit contributor and agent-governance files such as `CONTRIBUTING.md`, `AGENTS.md`, and `docs/agents/` are outside this skill. ## Review Scope Treat the documentation as a black box. Read the changed document in full and follow only the public documentation links that the reader must follow to complete the task. Review both the English and translated versions when both exist. Do not inspect source code, tests, scripts, Dockerfiles, lockfiles, CI, issues, pull requests, commits, internal notes, implementation logs, or unrelated repository files. Do not fill a documentation gap from prior product knowledge. If the reader needs information that the reviewed documentation does not provide, report the gap instead of researching the answer elsewhere. Use four primary review axes. ### 1. Language - English reads as natural technical English, not a literal translation or an internal engineering report. - Chinese reads as natural technical Chinese, not word-for-word English with unnecessary mixed-language filler. - Grammar, punctuation, tense, person, capitalization, and singular/plural use are consistent. - Terms are introduced before use and remain consistent across headings, prose, commands, and translations. - Proper nouns, commands, flags, API fields, headers, versions, and protocol names stay exact. - English is canonical, while Chinese preserves meaning rather than sentence structure. ### 2. Wording - Sentences are direct, concise, and specific about what the reader should do or expect. - Remove filler, repetition, inflated claims, stacked caveats, and details that do not change a user decision. - Avoid AI- or review-report phrasing such as "this proves", "authoritative path", "supported assertion", "comprehensive", or ritual conclusions. - Distinguish requirements, recommendations, optional diagnostics, examples, and observed results without legalistic or contract-heavy prose. - Do not address readers as repository maintainers. Build or publishing guides target resources the reader owns rather than the repository's official namespaces and release workflow. - Preserve necessary warnings, security boundaries, and compatibility limits; simplify their expression instead of deleting them for style. ### 3. Logic - The document moves in the order a reader acts: purpose, prerequisites, setup, execution, expected result, recovery, and cleanup where applicable. - A step does not depend on an undefined term, missing value, hidden command, or later explanation. - The document itself supplies every instruction and decision needed after the stated prerequisites; it does not rely on repository knowledge the reader cannot see. - Paragraphs have one clear job. Headings and transitions make the relationship between sections obvious. - Required and optional paths do not interrupt or contradict each other. - Causes and consequences are stated accurately; observed behavior is not promoted into a platform guarantee. - Repeated explanations are consolidated, and cross-references replace unnecessary duplication. - English and Chinese describe the same sequence, conditions, and outcome. ### 4. Layout - Heading levels reflect the actual hierarchy and make the main path easy to scan. - Paragraphs are short enough to read comfortably; lists are used for genuine sets or sequences, not to fragment ordinary prose. - Tables are used for comparisons or repeated fields, not as decoration. - Commands, configuration, expected output, and explanations stay adjacent. - Code fences, list indentation, blank lines, links, and inline code follow valid Markdown and render cleanly. - Long commands remain copyable. Placeholders are visually clear and explained before execution. - English and translated documents keep matching code, configuration, resource names, image references, and machine output. ## Documentation-Only Guardrail This skill reviews documentation quality and sufficiency, not implementation correctness. Check internal consistency, command continuity, security guidance, and whether the documented result follows from the documented steps. Do not use implementation evidence to rescue unclear or incomplete prose. Preserve necessary secret-handling, permission, lifecycle, compatibility, and cleanup guidance. If a factual claim cannot be assessed from the documentation alone, record it as an unverified residual risk rather than researching other files. ## Findings Report only concrete reader-facing problems. For each finding include: - axis: `Language`, `Wording`, `Logic`, or `Layout`; - severity and confidence; - file and line; - the problematic text or structure; - how it confuses, slows, misleads, or blocks the reader; - the missing or conflicting information visible from the documentation; - the smallest useful correction direction. One finding may name a secondary axis, but choose one primary axis. Do not report vague preferences, generic style advice, or differences that are merely personal taste. ## Output Return: 1. `Verdict: PASS` or `Verdict: FAIL`. 2. A one-line assessment for each axis: Language, Wording, Logic, and Layout. 3. Findings ordered by severity. Write `No findings` when applicable. 4. Claims that remain unverifiable from the documentation alone, if any. Use `FAIL` when a material issue remains in any primary axis or when an edit would make the documentation factually unsafe. Use `PASS` only when the reviewed path is natural, concise, logically ordered, scannable, and usable from the documentation alone.