{
  "schema_version": 2,
  "kind": "shortening-method",
  "format": "agent-skill",
  "id": "caveman-review",
  "name": "Caveman reviews",
  "category": "Technical",
  "summary": "Reduce review feedback to its location, problem, and useful fix.",
  "use_cases": [
    "Pull request comments",
    "Code review findings",
    "Review feedback cleanup"
  ],
  "word_count": 130,
  "url": "https://sho.rten.it/methods/caveman-review/",
  "instructions_url": "https://sho.rten.it/methods/caveman-review/SKILL.md",
  "skill_url": "https://sho.rten.it/methods/caveman-review/SKILL.md",
  "json_url": "https://sho.rten.it/methods/caveman-review/llms.txt",
  "plain_text_url": "https://sho.rten.it/methods/caveman-review/prompt.txt",
  "license": "MIT",
  "sources_url": "https://sho.rten.it/sources/#caveman-review",
  "skill_name": "caveman-review",
  "skill_description": "Reduce review feedback to its location, problem, and useful fix. Use for Pull request comments, Code review findings, Review feedback cleanup.",
  "agents_md_url": "https://sho.rten.it/methods/caveman-review/AGENTS.md",
  "sources": [
    {
      "title": "JuliusBrussee/caveman · Caveman reviews",
      "url": "https://github.com/JuliusBrussee/caveman/blob/309834233183478e6fd7800d26e1fbaa6210274e/skills/caveman-review/SKILL.md",
      "repository": "https://github.com/JuliusBrussee/caveman",
      "revision": "309834233183478e6fd7800d26e1fbaa6210274e",
      "path": "skills/caveman-review/SKILL.md",
      "license": "MIT",
      "license_url": "https://github.com/JuliusBrussee/caveman/blob/309834233183478e6fd7800d26e1fbaa6210274e/LICENSE",
      "license_file": "licenses/juliusbrussee--caveman.txt",
      "copyright": "Copyright (c) 2026 Julius Brussee",
      "repository_stars": 104780,
      "checked_at": "2026-09-10",
      "license_scope": "Pinned commit predates the split-license v2 import. Root LICENSE is MIT with no scope exclusions.",
      "adaptation": "Preserved the location/problem/fix format and exceptions for findings that need explanation. Removed emoji severity markers, automatic triggers, and publishing implications. Changed removal of all hedging to preservation of evidence limits. Pinned the MIT-only pre-v2 commit."
    }
  ],
  "instructions": "Write concise, actionable review comments from the supplied findings or diff. For a straightforward finding, use one line: file and line, problem, then the fix. Keep exact locations and symbols. Include the reason when the consequence is not obvious.\n\nUse bug, risk, nit, or question labels when they help distinguish severity. A style preference is not a bug. If evidence is incomplete, keep the uncertainty or ask a direct question. Do not invent a defect, line number, or replacement API.\n\nRemove praise repeated in every comment, throat-clearing, and descriptions of code the reader can already see. Use a paragraph when security impact, architectural disagreement, or an unfamiliar concept needs explanation. Brevity must not hide the trigger or consequence. Return comments ready to paste; do not submit them or change code.",
  "example": {
    "context": "Two supplied code-review findings with different certainty",
    "before": "In src/retry.ts at line 78, count is incremented after await send(). If send() throws, the increment is skipped and the loop condition count < 3 remains true, so the request can retry indefinitely. Please move the increment before await send(). A separate concern is in src/cache.ts at line 44: cacheKey does not contain locale. I cannot tell from this diff whether an earlier layer already separates the cache by locale. Please confirm that boundary before treating this as a cross-locale caching bug.",
    "after": "src/retry.ts:L78: bug: A thrown `send()` skips `count` increment, so `count < 3` can permit endless retries. Increment before `await send()`.\nsrc/cache.ts:L44: question: `cacheKey` omits `locale`. Does an earlier layer separate locales?",
    "must_preserve": [
      "Exact file locations and identifiers",
      "Thrown send skips increment; count < 3 can remain true",
      "Move increment before await send()",
      "Locale issue is a question, not an established bug",
      "Possible earlier separation must be checked"
    ],
    "omitted": [
      "Cuts reviewer narration and makes the proven defect distinct from the unresolved concern."
    ],
    "kind": "illustrative"
  }
}