tsa-pr-reviewlisted

AST-grounded PR / diff review. One workflow → per-file risk ranking, blast radius per changed symbol, the exact pytest command to gate merge, any architecture-constraint violations, and a final BLOCK / REVIEW / APPROVE verdict — in ~1–2k tokens and 4–6 MCP calls. Goes beyond a generic LLM diff-read because only TSA's `analyze_change_impact` returns a deterministic `verification_command` + `queue_ledger`, and only the persisted call graph can enumerate true callers/callees of changed symbols. (Per `docs/internal/COMPETITOR_HEAD_TO_HEAD_2026-05-23.md`: CodeGraphContext silently indexes 0 files, grep-ast crashes on Python 3.14, wrale miscounts imports. A pure-LLM diff reader has none of these guard-rails — it cannot reach the AST cache to enumerate callers, cannot consult `architectural-constraints.yml`, and invents a pytest command instead of reading the one TSA computed.) Use when: - Reviewing a local diff (`git diff`, staged, branch-vs-main) - Reviewing a GitHub PR URL - User asks "is this PR safe to merge?"
aimasteracc/tree-sitter-analyzer · ★ 36 · Code & Development · score 78

Install: claude install-skill aimasteracc/tree-sitter-analyzer

# tsa-pr-review — AST-grounded PR review with merge-gate verdict > Replaces "LLM reads the diff and guesses" with a deterministic pipeline that > grounds the review in (a) the AST diff, (b) the persisted call graph, and > (c) architectural-constraints.yml. Output is a structured verdict an agent > can act on — including the **exact** pytest command to run before merge. ## When to use - Local diff review: unstaged, staged, or branch-vs-main - GitHub PR review by URL (uses `gh` under the hood) - CI / agent pre-merge gate - Any time a generic "read the diff" review would be too shallow **Don't use** when: - The diff is a one-line typo in a comment or doc - The change is in a file with no AST coverage (e.g. binary, image, .lock) - You only need to *describe* the diff, not *judge* it → use plain `git diff` ## Why this beats a generic LLM diff-read | Concern | LLM-only diff read | tsa-pr-review | |----------------------------------|----------------------|------------------------------------------| | Classify signature vs body change| Heuristic | `codegraph_pr_review` AST diff | | Exact tests to run | Invented / hallucinated | `analyze_change_impact.verification_command` | | Caller blast radius | Grep guess | `codegraph_callers` from persisted graph | | Callee regression surface | Read N files | `codegraph_callees` from persisted graph | | Architecture