[{"data":1,"prerenderedAt":-1},["ShallowReactive",2],{"skill-apache-doris-repo-review":3,"mdc-ai6267-key":39,"related-repo-apache-doris-repo-review":3819,"related-org-apache-doris-repo-review":3912},{"slug":4,"name":4,"fn":5,"description":6,"org":7,"tags":11,"stars":22,"repoUrl":23,"updatedAt":24,"license":25,"forks":26,"topics":27,"repo":34,"sourceUrl":37,"mdContent":38},"doris-repo-review","review Apache Doris pull requests","Given a PR URL (`\u002Fdoris-repo-review https:\u002F\u002Fgithub.com\u002Fapache\u002Fdoris\u002Fpull\u002F66807`), first check whether the current directory's branch and commit match that PR, and if they do not, align the current directory to the PR head without disturbing local work (refuse to switch when tracked files are modified and hand the decision back to the user); then review it with the same pipeline apache\u002Fdoris CI runs (Code Review Runner) - main-agent risk scan, 1-3 full-review subagents plus risk-focused subagents, shared-ledger convergence (at most 3 rounds), and one English plus one Chinese review document written into review-docs\u002F. A local run has no GitHub inline comments, so every finding must carry a path:line anchor and a verbatim snippet, verified by a script. When the review passes (no Blocker and no Major), it also posts one strongly formatted, machine-readable PASS comment on the PR from the local gh account - review date, reviewed commit sha, status, model, and notes for maintainers - after showing the exact body to the user for confirmation. Use when the user says \"\u002Fdoris-repo-review \u003CPR URL>\", \"review this PR\", \"review it the way the CI pipeline does\", or \"review this PR locally like CI\". Read-only against the source - no build, no tests, no changes to repository files; the PASS comment is the only GitHub write, and a REQUEST_CHANGES review posts nothing at all.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},"apache","Apache Software Foundation","https:\u002F\u002Fpexgzepcugksgbtrxkhf.supabase.co\u002Fstorage\u002Fv1\u002Fobject\u002Fpublic\u002Forg-logos\u002Fapache.png",[12,16,19],{"name":13,"slug":14,"type":15},"GitHub","github","tag",{"name":17,"slug":18,"type":15},"Code Review","code-review",{"name":20,"slug":21,"type":15},"Engineering","engineering",19,"https:\u002F\u002Fgithub.com\u002Fapache\u002Fdoris-skills","2026-08-20T03:53:29.014373",null,4,[28,29,30,31,32,33],"agent","cli","database","doris","mcp","skills",{"repoUrl":23,"stars":22,"forks":26,"topics":35,"description":36},[28,29,30,31,32,33],"Agent skills for Apache Doris","https:\u002F\u002Fgithub.com\u002Fapache\u002Fdoris-skills\u002Ftree\u002FHEAD\u002Fskills\u002Fdoris-repo-review","---\nname: doris-repo-review\ndescription: Given a PR URL (`\u002Fdoris-repo-review https:\u002F\u002Fgithub.com\u002Fapache\u002Fdoris\u002Fpull\u002F66807`), first check whether the current directory's branch and commit match that PR, and if they do not, align the current directory to the PR head without disturbing local work (refuse to switch when tracked files are modified and hand the decision back to the user); then review it with the same pipeline apache\u002Fdoris CI runs (Code Review Runner) - main-agent risk scan, 1-3 full-review subagents plus risk-focused subagents, shared-ledger convergence (at most 3 rounds), and one English plus one Chinese review document written into review-docs\u002F. A local run has no GitHub inline comments, so every finding must carry a path:line anchor and a verbatim snippet, verified by a script. When the review passes (no Blocker and no Major), it also posts one strongly formatted, machine-readable PASS comment on the PR from the local gh account - review date, reviewed commit sha, status, model, and notes for maintainers - after showing the exact body to the user for confirmation. Use when the user says \"\u002Fdoris-repo-review \u003CPR URL>\", \"review this PR\", \"review it the way the CI pipeline does\", or \"review this PR locally like CI\". Read-only against the source - no build, no tests, no changes to repository files; the PASS comment is the only GitHub write, and a REQUEST_CHANGES review posts nothing at all.\n---\n\n# Local pipeline-style Doris code review\n\nThis skill reviews a Pull Request against a **local clone of the Apache Doris source repository**.\nIt is a contributor-side workflow, not a cluster-operations skill: it never talks to a running\ncluster and never needs one.\n\nUsage:\n\n```\n\u002Fdoris-repo-review https:\u002F\u002Fgithub.com\u002Fapache\u002Fdoris\u002Fpull\u002F66807\n\u002Fdoris-repo-review https:\u002F\u002Fgithub.com\u002Fapache\u002Fdoris\u002Fpull\u002F66807 focus on class loading and compatibility\n\u002Fdoris-repo-review 66807            # a bare number defaults to apache\u002Fdoris\n```\n\nThis ports the review flow of `apache\u002Fdoris\u002F.github\u002Fworkflows\u002Fcode-review-runner.yml` to a local\nmachine: **first align the current directory to the PR head**, take the diff from local git, and\nproduce one English and one Chinese document under the current directory's `review-docs\u002F`.\nEverything else - required reading, main-agent risk scan, subagent split, shared ledger, at most\n3 convergence rounds, an explicit conclusion per checkpoint - matches CI.\n\n**The scope is the one worktree you are standing in.** No other worktree is scanned, chosen, or\ncreated; no environment variable selects a different directory; nothing is assumed about the\nmachine's directory layout.\n\nLet `$S` be this skill's own `scripts\u002F` directory (with a default Claude Code install that is\n`~\u002F.claude\u002Fskills\u002Fdoris-repo-review\u002Fscripts`):\n\n```bash\n$S\u002Falign-to-pr.sh \u003CPR> --check                      # diagnosis only, changes nothing\n$S\u002Falign-to-pr.sh \u003CPR> --out \"$CTX\u002Falign.env\"       # step 1: align the current directory\n$S\u002Fprepare-review-context.sh --ctx \"$CTX\" --align \"$CTX\u002Falign.env\"   # step 2: gather context\npython3 $S\u002Fverify-anchors.py --ctx \"$CTX\" --doc \u003Cen> --doc \u003Czh>      # step 9: verify anchors\n$S\u002Fpost-pass-comment.sh --ctx \"$CTX\" --model \u003Cid> ... --dry-run      # step 10: PASS comment\n```\n\n| File | Purpose |\n|---|---|\n| `scripts\u002Falign-to-pr.sh` | Resolve the PR, diagnose how the current directory relates to it, align it to the PR head |\n| `scripts\u002Fprepare-review-context.sh` | Produce the authoritative diff, new-side line ranges, required AGENTS.md list, existing comments, ledger skeleton |\n| `scripts\u002Fverify-anchors.py` | Check that every `path:line` anchor really exists and that both documents expose the same finding IDs |\n| `scripts\u002Fpost-pass-comment.sh` | Render and post the machine-readable PASS comment; refuses everything that is not a pass |\n| `references\u002Fprompts.md` | Subagent prompt templates (CI wording, carried over verbatim) |\n| `references\u002Fdoc-templates.md` | Templates for both documents, anchor format, verdict rule |\n| `references\u002Fpr-comment-format.md` | The `doris-repo-review\u002Fv1` comment schema, field meanings, and how a program reads it back |\n\nRequirements: `git`, an authenticated `gh` CLI, `jq`, and `python3`. The clone must have full\nhistory (`git fetch --unshallow` on a shallow one), because the authoritative diff is a three-dot\ndiff from the merge base.\n\n---\n\n## 0. Ground rules\n\n1. **Touch only the current directory, and never disturb local work.** When the current directory\n   has **modified tracked files**, refuse to switch, report the situation, and let the user commit\n   or stash it themselves - **never stash, reset, or delete a branch on the user's behalf**, and\n   never go hunting for some other directory to work in. Every switch is\n   `git checkout --detach`, so branch refs stay exactly where they were.\n2. **Read-only review.** Do not build, do not run tests, do not modify any source file in the\n   repository. The only local writes allowed are the two documents under `review-docs\u002F` and the\n   context directory `$CTX`.\n   **Exactly one thing may ever be written to GitHub**: the PASS comment of step 10, only when the\n   verdict is APPROVE, only through `post-pass-comment.sh`, and only after the user has seen the\n   rendered body and said go. A REQUEST_CHANGES review posts nothing. No inline comments, no\n   review submission, no labels, no edits to the PR body - and never a comment on any PR other\n   than the one being reviewed.\n3. **The diff has exactly one source**: `$CTX\u002Fpr.diff` and `$CTX\u002Fpr_changed_files.txt`. Do not\n   reach for `gh pr diff`, the web UI, or a hand-rolled `git diff` to get the change list - a\n   different way of fetching it means a different base.\n4. **Confirm a path before reading it.** If a path is not already confirmed by\n   `pr_changed_files.txt`, `pr.diff`, or the output of an earlier successful command, run\n   `rg --files` to confirm it first.\n5. **Do not stop at the first blocking issue.** Work through the changed files, the related\n   control flow, the tests, and the parallel or special-case paths.\n6. **Every suspicion must reach a conclusion**: it becomes a finding, is excluded as \"already\n   covered by an existing comment\", or is ruled out with concrete code evidence and written into\n   \"Considered and Dismissed\". Silently dropping a suspicion means the review is not finished.\n7. **There are no inline comments locally**, so an anchor is the reader's only way in. Every\n   finding must carry a `path:line` anchor (new-side line numbers) plus a verbatim snippet, and\n   the documents must pass `verify-anchors.py` at the end.\n\n---\n\n## 1. Align the current directory\n\n```bash\nCTX=\"\u003Csession scratchpad>\u002Freview-ctx\"\n$S\u002Falign-to-pr.sh \u003CPR> --out \"$CTX\u002Falign.env\"\n```\n\nThe script does four things, **all of them against the current worktree only**:\n\n1. **Resolve the PR**: a URL, `owner\u002Frepo#N`, or a bare number are all accepted; `gh api` supplies\n   title, state, head ref\u002Frepo\u002Fsha, and base ref\u002Fsha.\n2. **Diagnose consistency** (the two checks the user explicitly wants):\n   - `branch check`: is the current branch the PR's head branch?\n   - `commit check`: `same` \u002F `ahead:N` \u002F `behind:N` \u002F `diverged:a\u002Fb` \u002F `unrelated`\n   **Relay both lines to the user verbatim.** In particular `ahead:N` means there are local commits\n   that were never pushed, and **they are not part of the review**.\n3. **Fetch the PR head**: `git fetch \u003Cupstream> +refs\u002Fpull\u002FN\u002Fhead:refs\u002Fdoris-review\u002Fpr-N`. It goes\n   through `refs\u002Fpull\u002F*`, so a fork PR needs no extra remote. The fetched sha must equal the head\n   sha from the API; if it does not, the PR was pushed to while the context was being prepared -\n   the script re-reads the API once and errors out asking for a re-run if it still differs (this\n   mirrors CI's \"PR changed while review context was being prepared\" guard).\n4. **Align the current directory**:\n\n   | State of the current directory | Action |\n   |---|---|\n   | HEAD already equals the PR head | `ALIGN_MODE=already`, nothing to do |\n   | Clean | `git checkout --detach \u003CPR head>`, `ALIGN_MODE=switched` |\n   | Modified tracked files \u002F a rebase, merge, or cherry-pick in progress | `ALIGN_MODE=blocked`, **refuse and exit** |\n\n   \"Clean\" means **no modified tracked files**. Untracked files do not count (`git checkout` never\n   deletes them, and refuses outright when it would overwrite one), so an untracked directory such\n   as `review-docs\u002F` does not affect the verdict and survives the switch.\n\n**What to do when BLOCKED**: relay the dirty-file list the script printed, then ask the user to\npick - (a) commit it themselves, (b) stash it themselves, or (c) start over in a different\ndirectory. **Do not perform any of those for them**; once they have chosen and acted, re-run this\nskill. To see the diagnosis without changing anything, use `--check` (it still prints the full\nbranch check \u002F commit check before reporting BLOCKED).\n\nOnce aligned, `$CTX\u002Falign.env` supplies `WORKDIR` (= the current directory), `DOCS_ROOT`\n(= `WORKDIR`), `PR_HEAD_SHA` \u002F `PR_BASE_SHA`, `PREV_REF` (the restore point),\n`BRANCH_MATCHES_PR`, and `LOCAL_VS_PR`.\n\n---\n\n## 2. Prepare the context\n\n```bash\n$S\u002Fprepare-review-context.sh \\\n    --ctx \"$CTX\" --align \"$CTX\u002Falign.env\" [--focus \"\u003Cuser focus>\"] [--fresh]\n```\n\n`--align` carries over `WORKDIR`, the PR number, the upstream repo, and the PR base sha, and\nverifies the checkout really is parked on the PR head (if it is not, the script errors out and\nasks you to redo step 1). `--fresh` wipes and rebuilds everything including the ledger; without it\nthe ledger is preserved so a multi-round run can continue. Whatever free text the user typed after\nthe PR URL on the command line is the `--focus` value.\n\nOutput (under `$CTX`):\n\n| File | Contents |\n|---|---|\n| `meta.env` | PR info, base\u002Fhead sha, diff range, `DOCS_ROOT`, `ALIGN_MODE`, date, dirty-file count |\n| `pr.diff` \u002F `pr_changed_files.txt` \u002F `pr_changed_files_status.txt` | The authoritative diff and change list |\n| `changed_line_ranges.txt` \u002F `.tsv` | **New-side (post-change) line ranges** - the source for anchors |\n| `pr_commits.txt` \u002F `pr_diffstat.txt` | Commit list and change size |\n| `pr_review_threads.md` | Existing inline comments (at most 30 threads, each body truncated at 1200 chars) |\n| `required_agents.txt` \u002F `required_agents_prompt.txt` | Required AGENTS.md files, derived from the changed files' ancestor directories |\n| `review_focus.txt` | The user's focus points |\n| `worktree_status.txt` | Uncommitted changes in the review tree (**out of review scope**; call them out in the documents) |\n| `ledger\u002F` | Shared-ledger skeleton |\n\nIf `BASE_SOURCE` is not `PR base sha (matches CI)`, the baseline differs from CI's and the\ndocuments must say so.\n\n---\n\n## 3. Required reading (in this order)\n\nBefore looking at any code, the main agent reads, in order:\n\n1. `$WORKDIR\u002F.claude\u002Fskills\u002Fcode-review\u002FSKILL.md` - the Doris review checklist; follow it strictly\n   throughout. On an older branch that does not have it, fetch it with\n   `gh api repos\u002Fapache\u002Fdoris\u002Fcontents\u002F.claude\u002Fskills\u002Fcode-review\u002FSKILL.md?ref=master`.\n2. **Every** AGENTS.md listed in `$CTX\u002Frequired_agents.txt`, each one read in full. Listing the\n   directory or grepping the path does not count.\n3. `$CTX\u002Fpr_review_threads.md` - treat every existing comment as already-known context. **Do not\n   raise** the same or a substantially similar issue again, even phrased differently. Raise a\n   similar concern only when this PR introduces a genuinely different instance somewhere else that\n   the existing comments do not cover, and say why it is distinct.\n4. `$CTX\u002Freview_focus.txt` - do the full review as usual and pay extra attention to these points;\n   the final documents must respond to each one, even if the conclusion is \"no additional issue\n   found for this point\".\n5. All of `$CTX\u002Fpr.diff` plus `$CTX\u002Fpr_commits.txt`. When the diff is large (> 5000 lines), read it\n   in chunks by directory or module until it is covered, and describe how you read it in \"Coverage\n   and Limits\" - do not pretend you read it all.\n\n---\n\n## 4. Main-agent initial risk scan (before spawning any subagent)\n\n**This must be finished before any subagent is spawned.** Read the whole change, understand every\nmechanism it touches, then answer: if this PR is wrong, where is it most likely to be wrong? Which\npoints look risky to you?\n\nWrite the result into `$CTX\u002Fledger\u002F00-main-risk-scan.md`, each entry carrying: an ID, the changed\nfiles\u002Flines involved, the related mechanism to inspect, why it is suspicious, the upstream or\ndownstream files that must be read alongside it, and the **specific question** a risk-focused\nsubagent has to answer.\n\nAn empty risk scan means you have not understood the PR yet - go back and read it; do not skip\nthis step.\n\n---\n\n## 5. Spawn the subagents\n\nSplit along the coverage the code-review skill requires, and **send them all in one message so\nthey run concurrently** (general-purpose subagents):\n\n| Type | Count | Responsibility |\n|---|---|---|\n| Full-review subagent | 1-3 (depending on PR complexity) | Each owns one slice; **their union must cover the whole PR** and every checkpoint the skill requires |\n| Risk-focused subagent | one per suspicious mechanism | Answers exactly one question from the risk scan, independently of the full-review subagents |\n\nSplit by **module or mechanism** (for example \"FE SPI interface and load path\", \"Ranger plugin\nimplementation and class loading\", \"build scripts and packaging\u002Fdeployment\"), not by dividing the\nfile count evenly - a split that lands in the middle of a mechanism leaves both halves unable to\nsee the bug. Give every subagent its own ledger file, `$CTX\u002Fledger\u002Fsub-\u003Cround>-\u003Cid>.md`.\n\nTake the prompts from `references\u002Fprompts.md`: section A (shared preamble) plus section B\n(full review) or section C (risk-focused), substituting `{CTX}` \u002F `{REPO_ROOT}` (= `$WORKDIR`) \u002F\n`{BASE_SHA}` \u002F `{HEAD_SHA}` \u002F `{ROUND}` \u002F `{AGENT_ID}` \u002F `{FOCUS}`. **Every subagent prompt must\nstate the ledger directory and the path of that subagent's own file.**\n\n> **One implementation difference from CI**: CI uses a single `subagent_review_findings.md` with\n> sections; locally, concurrent writes to one file collide on patch, so this becomes **one file per\n> owner** under `ledger\u002F`. The semantics are unchanged - a single shared source of truth, everyone\n> reads all of it, each writes only their own, the main agent merges.\n\n---\n\n## 6. Main agent merges, verifies, deduplicates\n\n**Re-read the entire `ledger\u002F` directory every time a subagent returns**, then process each\ncandidate into `$CTX\u002Fledger\u002Fmain-merged.md`:\n\n- **Verify**: read the code yourself to confirm it; do not take a subagent's conclusion at face\n  value. The common failure modes are treating a local pattern that an upstream guarantee already\n  covers as a bug, and an \"if A then B\" whose A has no concrete scenario.\n- **Deduplicate**: against the existing candidates and against the existing comments in\n  `pr_review_threads.md`.\n- **Assign a status**: `accepted` \u002F `dismissed_with_evidence` \u002F `duplicated`. A dismissal must come\n  with concrete code evidence, which goes verbatim into the document's \"Considered and Dismissed\".\n- **Fill the risk items back in**: update `Status` and `Final conclusion` for every entry in\n  `00-main-risk-scan.md`.\n\nWhen this step ends, **no candidate may be left without a status**.\n\n---\n\n## 7. Convergence loop (at most 3 rounds)\n\nOne round = step 5 + step 6. Record the outcome in the `Convergence Rounds` table of\n`main-merged.md`.\n\n- Every subagent returned `NO_NEW_VALUABLE_FINDINGS` → go to step 8.\n- New valuable candidates remain → start another round: **re-slice the coverage** based on what\n  this round taught you (do not re-dispatch the same split unchanged), and add risk-focused\n  subagents for any newly suspicious mechanism.\n- **Cap of 3 rounds.** If the cap is reached and new candidates still appear, finish normally but\n  state in the verdict and in \"Coverage and Limits\" that **this review did not converge**.\n\n---\n\n## 8. Final sweep\n\nBefore writing the documents, walk explicitly through the changed-file list and the open-candidate\nlist:\n\n- Was every changed file covered by at least one subagent? Cover the rest yourself.\n- Does every suspicion have a conclusion?\n- Does every applicable item in Part 1.3 of the code-review skill have an explicit conclusion?\n- Is there anywhere you are still unsure about, or that may not have been investigated deeply\n  enough? Investigate it now.\n\nOnly after the sweep may you write the documents.\n\n---\n\n## 9. Produce the two documents\n\nWrite them into `review-docs\u002F` in the **current directory** (`mkdir -p` it if needed):\n\n```\nreview-docs\u002Fpr-\u003CN>-review.en.md\nreview-docs\u002Fpr-\u003CN>-review.zh.md\n```\n\n`references\u002Fdoc-templates.md` holds the templates, the anchor format, and the verdict rule\n(`Blocker`\u002F`Major` → REQUEST_CHANGES; only `Minor`\u002F`Nit` → APPROVE). The essentials:\n\n- The two documents are **equivalent in content**: same finding IDs, same order, same anchors. The\n  Chinese one is a real Chinese review, not a word-for-word translation; identifiers, paths, log\n  messages, config names, code snippets, and the severity words stay in their original form.\n- The document header must state: the PR link and state, the **head sha actually reviewed**, the\n  diff range, the review directory (`WORKDIR`), and the `branch check` \u002F `commit check` results\n  from step 1.\n- Every finding must carry: a severity, a `path:line` anchor, a verbatim snippet, and the four\n  parts **what is wrong \u002F why it happens \u002F when it bites \u002F suggested fix**. Give a diff patch when\n  the fix is small and self-contained; use prose only for architectural problems.\n- The \"Critical Checkpoints\" table needs a conclusion per row; mark a non-applicable one `n\u002Fa` with\n  a half-line reason - **never delete the row**.\n- \"Response to Review Focus\" answers each of the user's focus points.\n- \"Considered and Dismissed\" lists every excluded suspicion together with its evidence.\n- \"Coverage and Limits\" states: what was read in depth versus skimmed, how the subagents were\n  split, what was not verified locally (builds, tests, anything needing a real cluster), and the\n  uncommitted changes from `worktree_status.txt` that were excluded.\n\nWhen they are written, running the verifier is **mandatory**; if it fails, fix the documents and\nre-run until it passes:\n\n```bash\npython3 $S\u002Fverify-anchors.py --ctx \"$CTX\" \\\n    --doc review-docs\u002Fpr-\u003CN>-review.en.md \\\n    --doc review-docs\u002Fpr-\u003CN>-review.zh.md\n```\n\nIt checks that anchor paths exist, that line numbers are inside the file, that every finding has at\nleast one anchor, and that the EN and ZH finding-ID sets match; it also flags anchors pointing at\nunchanged context lines - which is usually where a miscomputed line number shows up.\n\n---\n\n## 10. Post the PASS comment to the PR\n\n**Only when the verdict is APPROVE** (no `Blocker`, no `Major`) and `verify-anchors.py` has\npassed. A REQUEST_CHANGES review posts nothing at all - say so in the closing report and stop.\n\nWrite the notes, dry-run, get a go, post:\n\n```bash\n# At most 5 bullets, each anchored where it can be. Skip the file when there is nothing to say.\ncat > \"$CTX\u002Fpr-comment-notes.md\" \u003C\u003C'EOF'\n- `fe\u002Ffe-core\u002Fsrc\u002Fmain\u002Fjava\u002Forg\u002Fapache\u002Fdoris\u002FX.java:214` — \u003Cwhat the maintainer should know>\nEOF\n\n$S\u002Fpost-pass-comment.sh --ctx \"$CTX\" \\\n    --model \"\u003Cexact model id of this session>\" --effort \"${CLAUDE_EFFORT:-unknown}\" \\\n    --findings \u003Cblocker>,\u003Cmajor>,\u003Cminor>,\u003Cnit> \\\n    --rounds \u003Cr> --converged \u003Ctrue|false> \\\n    --notes-file \"$CTX\u002Fpr-comment-notes.md\" \\\n    --dry-run\n```\n\n- **`--model` is the exact model id of the session doing the review** (`claude-opus-5[1m]`,\n  `gpt-5.6-sol`, …), taken from what this session was told about itself - never a guess, never a\n  bare family name. `--effort` comes from `$CLAUDE_EFFORT`. The comment is a public, signed\n  statement about who reviewed the code; both fields are what make it auditable.\n- The dry run runs **every** precondition and prints the exact body. **Show that body to the user\n  and wait for a go**, then re-run the identical command **without `--dry-run`**. Never post\n  without that confirmation.\n- The script refuses to post when: any `Blocker`\u002F`Major` is present; the live PR head no longer\n  equals the reviewed commit; the PR is not open (`--allow-closed` overrides); the notes are\n  malformed or more than five; `converged: false` came without a note. A refusal is a real signal\n  - relay it, do not work around it.\n- **The PR head moved** means the author pushed during the review: the review is stale, so re-run\n  the whole skill instead of posting.\n- An earlier v1 comment from the same account for the **same** commit is edited in place; a new\n  commit gets a new comment, so each push leaves exactly one record.\n- Findings counts, rounds and `converged` must match the documents written in step 9. The counts\n  are of *accepted* findings, not of candidates.\n- `references\u002Fpr-comment-format.md` holds the schema, the field meanings, and the parser snippet.\n  **Never hand-write or hand-edit this comment** - the format is a contract other programs read.\n\n---\n\n## 11. Closing report\n\nTell the user:\n\n1. The paths of both documents, the verdict (REQUEST_CHANGES \u002F APPROVE), the finding count per\n   severity, and whether the rounds converged.\n2. The `branch check` \u002F `commit check` results - especially `ahead:N` (unpushed commits that were\n   not reviewed).\n3. **What happened to the PASS comment**: the URL when one was posted or updated, or the reason\n   nothing was posted (the verdict was REQUEST_CHANGES, the PR head moved, the user said no).\n4. **Where the current directory now stands**: with `ALIGN_MODE=switched` it is detached on the PR\n   head, and `git checkout \u003CPREV_REF>` restores it. **Do not switch back automatically** - the user\n   may still want to read the code.\n5. `review-docs\u002F` **is not gitignored in the doris repository**, so **do not commit it\n   automatically**; leave that to the user.\n\n---\n\n## 12. Mapping to the CI pipeline\n\n| CI (code-review-runner.yml) | Local |\n|---|---|\n| `codex exec --goal` (gpt-5.6-sol, xhigh) | The main agent of this session |\n| checkout the PR head sha | `align-to-pr.sh` detaches **the current directory** to the same sha |\n| `git diff BASE...HEAD` as the authoritative diff | Same, produced by `prepare-review-context.sh` |\n| \"PR changed while preparing\" guard | The fetched sha must equal the API head sha, otherwise a re-run is required |\n| `prepare_review_agents.py` collecting AGENTS.md | Same ancestor-directory algorithm, built into the script |\n| Fetch existing inline threads (30 threads \u002F 1200 chars) | Same jq |\n| Text after `\u002Freview` = review focus | Free text after the PR URL → `--focus` |\n| Single-file ledger with sections | A `ledger\u002F` directory, one file per owner |\n| Main risk scan → 1-3 full-review subagents + risk-focused → merge → ≤3 rounds | Identical |\n| `gh pr review` \u002F Reviews API posting inline comments | **Two `review-docs\u002F` documents (EN + ZH) with `path:line` anchors** |\n| CI's review verdict is visible on the PR itself | On a pass, one `doris-repo-review\u002Fv1` comment from the local account (commit sha, timestamp, model, findings, notes); on REQUEST_CHANGES, nothing - the documents stay local |\n| 60-minute timeout | No hard timeout, but likewise do not let one round turn into unbounded digging |\n\n---\n\n## 13. Common traps\n\n- **The local checkout is ahead of the PR head.** `commit check: ahead:N` means there are unpushed\n  commits and **they are not part of the review**. This is the easiest thing for a reader to\n  misread, so state it both in the document header and in the closing report.\n- **Anchoring to the wrong side.** `changed_line_ranges.txt` gives **post-change** line numbers,\n  which match what a file read shows in the current directory. A pure deletion has no line on the\n  new side - anchor the seam line and note `(deleted, base line N)`.\n- **Switching invalidates the current directory's incremental build.** `output\u002F`, `be\u002Fbuild_*`, and\n  `*\u002Ftarget\u002F` are all built per commit, so switching to the PR head and back usually forces a\n  rebuild. Warn the user before switching.\n- **Uncommitted changes are not in the diff.** `git diff BASE...HEAD` only sees committed content.\n  Alignment already guarantees no dirty tracked files at switch time, but the user may have edited\n  something afterwards; anything in `worktree_status.txt` must be named in the documents' \"Coverage\n  and Limits\".\n- **Subagents racing on the ledger.** One file per owner exists precisely for this; always hard-code\n  each subagent's own file name in its prompt.\n- **Re-raising an issue that already has a comment.** CI forbids it, and so does a local run - read\n  `pr_review_threads.md` first.\n- **A subagent's conclusion cannot be taken at face value.** The main agent must read the code back\n  to confirm it; any \"if A then B\" that cannot name a concrete scenario for A is downgraded or\n  dropped.\n- **Do not commit `review-docs\u002F`.** The doris repository does not ignore it, and an automatic commit\n  would slip it into the PR.\n- **The PASS comment is public and signed with the user's name.** It goes to a public Apache PR\n  from their GitHub account, so it is posted only after they have seen the exact body. Treat a\n  script refusal as final rather than something to route around, and never \"tidy up\" the rendered\n  body by hand - a program reads it.\n- **A pass is not a merge approval.** The comment states that a local pipeline review found no\n  Blocker and no Major on one specific commit. It carries no CI signal and no Apache sign-off, and\n  the `\u003Csub>` disclaimer line says exactly that - keep it.\n- **Counts drift between the documents and the comment.** `--findings` must be the accepted\n  findings of step 9, not the candidate count from the ledger; re-count from the written documents\n  before posting.\n",{"data":40,"body":41},{"name":4,"description":6},{"type":42,"children":43},"root",[44,53,67,72,85,113,123,152,434,597,641,645,652,849,852,858,931,943,1213,1238,1310,1313,1319,1403,1437,1448,1681,1702,1705,1711,1716,1804,1807,1813,1823,1843,1848,1851,1857,1869,1939,1958,2034,2070,2073,2079,2103,2195,2206,2209,2215,2235,2279,2282,2288,2293,2316,2321,2324,2330,2357,2366,2407,2514,2526,2632,2637,2640,2646,2677,2682,3027,3209,3212,3218,3223,3321,3324,3330,3595,3598,3604,3813],{"type":45,"tag":46,"props":47,"children":49},"element","h1",{"id":48},"local-pipeline-style-doris-code-review",[50],{"type":51,"value":52},"text","Local pipeline-style Doris code review",{"type":45,"tag":54,"props":55,"children":56},"p",{},[57,59,65],{"type":51,"value":58},"This skill reviews a Pull Request against a ",{"type":45,"tag":60,"props":61,"children":62},"strong",{},[63],{"type":51,"value":64},"local clone of the Apache Doris source repository",{"type":51,"value":66},".\nIt is a contributor-side workflow, not a cluster-operations skill: it never talks to a running\ncluster and never needs one.",{"type":45,"tag":54,"props":68,"children":69},{},[70],{"type":51,"value":71},"Usage:",{"type":45,"tag":73,"props":74,"children":78},"pre",{"className":75,"code":77,"language":51},[76],"language-text","\u002Fdoris-repo-review https:\u002F\u002Fgithub.com\u002Fapache\u002Fdoris\u002Fpull\u002F66807\n\u002Fdoris-repo-review https:\u002F\u002Fgithub.com\u002Fapache\u002Fdoris\u002Fpull\u002F66807 focus on class loading and compatibility\n\u002Fdoris-repo-review 66807            # a bare number defaults to apache\u002Fdoris\n",[79],{"type":45,"tag":80,"props":81,"children":83},"code",{"__ignoreMap":82},"",[84],{"type":51,"value":77},{"type":45,"tag":54,"props":86,"children":87},{},[88,90,96,98,103,105,111],{"type":51,"value":89},"This ports the review flow of ",{"type":45,"tag":80,"props":91,"children":93},{"className":92},[],[94],{"type":51,"value":95},"apache\u002Fdoris\u002F.github\u002Fworkflows\u002Fcode-review-runner.yml",{"type":51,"value":97}," to a local\nmachine: ",{"type":45,"tag":60,"props":99,"children":100},{},[101],{"type":51,"value":102},"first align the current directory to the PR head",{"type":51,"value":104},", take the diff from local git, and\nproduce one English and one Chinese document under the current directory's ",{"type":45,"tag":80,"props":106,"children":108},{"className":107},[],[109],{"type":51,"value":110},"review-docs\u002F",{"type":51,"value":112},".\nEverything else - required reading, main-agent risk scan, subagent split, shared ledger, at most\n3 convergence rounds, an explicit conclusion per checkpoint - matches CI.",{"type":45,"tag":54,"props":114,"children":115},{},[116,121],{"type":45,"tag":60,"props":117,"children":118},{},[119],{"type":51,"value":120},"The scope is the one worktree you are standing in.",{"type":51,"value":122}," No other worktree is scanned, chosen, or\ncreated; no environment variable selects a different directory; nothing is assumed about the\nmachine's directory layout.",{"type":45,"tag":54,"props":124,"children":125},{},[126,128,134,136,142,144,150],{"type":51,"value":127},"Let ",{"type":45,"tag":80,"props":129,"children":131},{"className":130},[],[132],{"type":51,"value":133},"$S",{"type":51,"value":135}," be this skill's own ",{"type":45,"tag":80,"props":137,"children":139},{"className":138},[],[140],{"type":51,"value":141},"scripts\u002F",{"type":51,"value":143}," directory (with a default Claude Code install that is\n",{"type":45,"tag":80,"props":145,"children":147},{"className":146},[],[148],{"type":51,"value":149},"~\u002F.claude\u002Fskills\u002Fdoris-repo-review\u002Fscripts",{"type":51,"value":151},"):",{"type":45,"tag":73,"props":153,"children":157},{"className":154,"code":155,"language":156,"meta":82,"style":82},"language-bash shiki shiki-themes material-theme-lighter material-theme material-theme-palenight","$S\u002Falign-to-pr.sh \u003CPR> --check                      # diagnosis only, changes nothing\n$S\u002Falign-to-pr.sh \u003CPR> --out \"$CTX\u002Falign.env\"       # step 1: align the current directory\n$S\u002Fprepare-review-context.sh --ctx \"$CTX\" --align \"$CTX\u002Falign.env\"   # step 2: gather context\npython3 $S\u002Fverify-anchors.py --ctx \"$CTX\" --doc \u003Cen> --doc \u003Czh>      # step 9: verify anchors\n$S\u002Fpost-pass-comment.sh --ctx \"$CTX\" --model \u003Cid> ... --dry-run      # step 10: PASS comment\n","bash",[158],{"type":45,"tag":80,"props":159,"children":160},{"__ignoreMap":82},[161,200,250,297,385],{"type":45,"tag":162,"props":163,"children":166},"span",{"class":164,"line":165},"line",1,[167,173,179,184,189,194],{"type":45,"tag":162,"props":168,"children":170},{"style":169},"--shiki-light:#90A4AE;--shiki-default:#EEFFFF;--shiki-dark:#BABED8",[171],{"type":51,"value":172},"$S\u002Falign-to-pr.sh ",{"type":45,"tag":162,"props":174,"children":176},{"style":175},"--shiki-light:#39ADB5;--shiki-default:#89DDFF;--shiki-dark:#89DDFF",[177],{"type":51,"value":178},"\u003C",{"type":45,"tag":162,"props":180,"children":181},{"style":169},[182],{"type":51,"value":183},"PR",{"type":45,"tag":162,"props":185,"children":186},{"style":175},[187],{"type":51,"value":188},">",{"type":45,"tag":162,"props":190,"children":191},{"style":169},[192],{"type":51,"value":193}," --check                      ",{"type":45,"tag":162,"props":195,"children":197},{"style":196},"--shiki-light:#90A4AE;--shiki-light-font-style:italic;--shiki-default:#546E7A;--shiki-default-font-style:italic;--shiki-dark:#676E95;--shiki-dark-font-style:italic",[198],{"type":51,"value":199},"# diagnosis only, changes nothing\n",{"type":45,"tag":162,"props":201,"children":203},{"class":164,"line":202},2,[204,208,212,216,220,225,230,235,241,245],{"type":45,"tag":162,"props":205,"children":206},{"style":169},[207],{"type":51,"value":172},{"type":45,"tag":162,"props":209,"children":210},{"style":175},[211],{"type":51,"value":178},{"type":45,"tag":162,"props":213,"children":214},{"style":169},[215],{"type":51,"value":183},{"type":45,"tag":162,"props":217,"children":218},{"style":175},[219],{"type":51,"value":188},{"type":45,"tag":162,"props":221,"children":222},{"style":169},[223],{"type":51,"value":224}," --out ",{"type":45,"tag":162,"props":226,"children":227},{"style":175},[228],{"type":51,"value":229},"\"",{"type":45,"tag":162,"props":231,"children":232},{"style":169},[233],{"type":51,"value":234},"$CTX",{"type":45,"tag":162,"props":236,"children":238},{"style":237},"--shiki-light:#91B859;--shiki-default:#C3E88D;--shiki-dark:#C3E88D",[239],{"type":51,"value":240},"\u002Falign.env",{"type":45,"tag":162,"props":242,"children":243},{"style":175},[244],{"type":51,"value":229},{"type":45,"tag":162,"props":246,"children":247},{"style":196},[248],{"type":51,"value":249},"       # step 1: align the current directory\n",{"type":45,"tag":162,"props":251,"children":253},{"class":164,"line":252},3,[254,259,263,267,271,276,280,284,288,292],{"type":45,"tag":162,"props":255,"children":256},{"style":169},[257],{"type":51,"value":258},"$S\u002Fprepare-review-context.sh --ctx ",{"type":45,"tag":162,"props":260,"children":261},{"style":175},[262],{"type":51,"value":229},{"type":45,"tag":162,"props":264,"children":265},{"style":169},[266],{"type":51,"value":234},{"type":45,"tag":162,"props":268,"children":269},{"style":175},[270],{"type":51,"value":229},{"type":45,"tag":162,"props":272,"children":273},{"style":169},[274],{"type":51,"value":275}," --align ",{"type":45,"tag":162,"props":277,"children":278},{"style":175},[279],{"type":51,"value":229},{"type":45,"tag":162,"props":281,"children":282},{"style":169},[283],{"type":51,"value":234},{"type":45,"tag":162,"props":285,"children":286},{"style":237},[287],{"type":51,"value":240},{"type":45,"tag":162,"props":289,"children":290},{"style":175},[291],{"type":51,"value":229},{"type":45,"tag":162,"props":293,"children":294},{"style":196},[295],{"type":51,"value":296},"   # step 2: gather context\n",{"type":45,"tag":162,"props":298,"children":299},{"class":164,"line":26},[300,306,311,316,321,326,330,334,339,344,349,354,358,362,366,371,376,380],{"type":45,"tag":162,"props":301,"children":303},{"style":302},"--shiki-light:#E2931D;--shiki-default:#FFCB6B;--shiki-dark:#FFCB6B",[304],{"type":51,"value":305},"python3",{"type":45,"tag":162,"props":307,"children":308},{"style":169},[309],{"type":51,"value":310}," $S",{"type":45,"tag":162,"props":312,"children":313},{"style":237},[314],{"type":51,"value":315},"\u002Fverify-anchors.py",{"type":45,"tag":162,"props":317,"children":318},{"style":237},[319],{"type":51,"value":320}," --ctx",{"type":45,"tag":162,"props":322,"children":323},{"style":175},[324],{"type":51,"value":325}," \"",{"type":45,"tag":162,"props":327,"children":328},{"style":169},[329],{"type":51,"value":234},{"type":45,"tag":162,"props":331,"children":332},{"style":175},[333],{"type":51,"value":229},{"type":45,"tag":162,"props":335,"children":336},{"style":237},[337],{"type":51,"value":338}," --doc",{"type":45,"tag":162,"props":340,"children":341},{"style":175},[342],{"type":51,"value":343}," \u003C",{"type":45,"tag":162,"props":345,"children":346},{"style":237},[347],{"type":51,"value":348},"e",{"type":45,"tag":162,"props":350,"children":351},{"style":169},[352],{"type":51,"value":353},"n",{"type":45,"tag":162,"props":355,"children":356},{"style":175},[357],{"type":51,"value":188},{"type":45,"tag":162,"props":359,"children":360},{"style":237},[361],{"type":51,"value":338},{"type":45,"tag":162,"props":363,"children":364},{"style":175},[365],{"type":51,"value":343},{"type":45,"tag":162,"props":367,"children":368},{"style":237},[369],{"type":51,"value":370},"z",{"type":45,"tag":162,"props":372,"children":373},{"style":169},[374],{"type":51,"value":375},"h",{"type":45,"tag":162,"props":377,"children":378},{"style":175},[379],{"type":51,"value":188},{"type":45,"tag":162,"props":381,"children":382},{"style":196},[383],{"type":51,"value":384},"      # step 9: verify anchors\n",{"type":45,"tag":162,"props":386,"children":388},{"class":164,"line":387},5,[389,394,398,402,406,411,415,420,424,429],{"type":45,"tag":162,"props":390,"children":391},{"style":169},[392],{"type":51,"value":393},"$S\u002Fpost-pass-comment.sh --ctx ",{"type":45,"tag":162,"props":395,"children":396},{"style":175},[397],{"type":51,"value":229},{"type":45,"tag":162,"props":399,"children":400},{"style":169},[401],{"type":51,"value":234},{"type":45,"tag":162,"props":403,"children":404},{"style":175},[405],{"type":51,"value":229},{"type":45,"tag":162,"props":407,"children":408},{"style":169},[409],{"type":51,"value":410}," --model ",{"type":45,"tag":162,"props":412,"children":413},{"style":175},[414],{"type":51,"value":178},{"type":45,"tag":162,"props":416,"children":417},{"style":169},[418],{"type":51,"value":419},"id",{"type":45,"tag":162,"props":421,"children":422},{"style":175},[423],{"type":51,"value":188},{"type":45,"tag":162,"props":425,"children":426},{"style":169},[427],{"type":51,"value":428}," ... --dry-run      ",{"type":45,"tag":162,"props":430,"children":431},{"style":196},[432],{"type":51,"value":433},"# step 10: PASS comment\n",{"type":45,"tag":435,"props":436,"children":437},"table",{},[438,457],{"type":45,"tag":439,"props":440,"children":441},"thead",{},[442],{"type":45,"tag":443,"props":444,"children":445},"tr",{},[446,452],{"type":45,"tag":447,"props":448,"children":449},"th",{},[450],{"type":51,"value":451},"File",{"type":45,"tag":447,"props":453,"children":454},{},[455],{"type":51,"value":456},"Purpose",{"type":45,"tag":458,"props":459,"children":460},"tbody",{},[461,479,496,521,538,555,572],{"type":45,"tag":443,"props":462,"children":463},{},[464,474],{"type":45,"tag":465,"props":466,"children":467},"td",{},[468],{"type":45,"tag":80,"props":469,"children":471},{"className":470},[],[472],{"type":51,"value":473},"scripts\u002Falign-to-pr.sh",{"type":45,"tag":465,"props":475,"children":476},{},[477],{"type":51,"value":478},"Resolve the PR, diagnose how the current directory relates to it, align it to the PR head",{"type":45,"tag":443,"props":480,"children":481},{},[482,491],{"type":45,"tag":465,"props":483,"children":484},{},[485],{"type":45,"tag":80,"props":486,"children":488},{"className":487},[],[489],{"type":51,"value":490},"scripts\u002Fprepare-review-context.sh",{"type":45,"tag":465,"props":492,"children":493},{},[494],{"type":51,"value":495},"Produce the authoritative diff, new-side line ranges, required AGENTS.md list, existing comments, ledger skeleton",{"type":45,"tag":443,"props":497,"children":498},{},[499,508],{"type":45,"tag":465,"props":500,"children":501},{},[502],{"type":45,"tag":80,"props":503,"children":505},{"className":504},[],[506],{"type":51,"value":507},"scripts\u002Fverify-anchors.py",{"type":45,"tag":465,"props":509,"children":510},{},[511,513,519],{"type":51,"value":512},"Check that every ",{"type":45,"tag":80,"props":514,"children":516},{"className":515},[],[517],{"type":51,"value":518},"path:line",{"type":51,"value":520}," anchor really exists and that both documents expose the same finding IDs",{"type":45,"tag":443,"props":522,"children":523},{},[524,533],{"type":45,"tag":465,"props":525,"children":526},{},[527],{"type":45,"tag":80,"props":528,"children":530},{"className":529},[],[531],{"type":51,"value":532},"scripts\u002Fpost-pass-comment.sh",{"type":45,"tag":465,"props":534,"children":535},{},[536],{"type":51,"value":537},"Render and post the machine-readable PASS comment; refuses everything that is not a pass",{"type":45,"tag":443,"props":539,"children":540},{},[541,550],{"type":45,"tag":465,"props":542,"children":543},{},[544],{"type":45,"tag":80,"props":545,"children":547},{"className":546},[],[548],{"type":51,"value":549},"references\u002Fprompts.md",{"type":45,"tag":465,"props":551,"children":552},{},[553],{"type":51,"value":554},"Subagent prompt templates (CI wording, carried over verbatim)",{"type":45,"tag":443,"props":556,"children":557},{},[558,567],{"type":45,"tag":465,"props":559,"children":560},{},[561],{"type":45,"tag":80,"props":562,"children":564},{"className":563},[],[565],{"type":51,"value":566},"references\u002Fdoc-templates.md",{"type":45,"tag":465,"props":568,"children":569},{},[570],{"type":51,"value":571},"Templates for both documents, anchor format, verdict rule",{"type":45,"tag":443,"props":573,"children":574},{},[575,584],{"type":45,"tag":465,"props":576,"children":577},{},[578],{"type":45,"tag":80,"props":579,"children":581},{"className":580},[],[582],{"type":51,"value":583},"references\u002Fpr-comment-format.md",{"type":45,"tag":465,"props":585,"children":586},{},[587,589,595],{"type":51,"value":588},"The ",{"type":45,"tag":80,"props":590,"children":592},{"className":591},[],[593],{"type":51,"value":594},"doris-repo-review\u002Fv1",{"type":51,"value":596}," comment schema, field meanings, and how a program reads it back",{"type":45,"tag":54,"props":598,"children":599},{},[600,602,608,610,616,618,624,626,631,633,639],{"type":51,"value":601},"Requirements: ",{"type":45,"tag":80,"props":603,"children":605},{"className":604},[],[606],{"type":51,"value":607},"git",{"type":51,"value":609},", an authenticated ",{"type":45,"tag":80,"props":611,"children":613},{"className":612},[],[614],{"type":51,"value":615},"gh",{"type":51,"value":617}," CLI, ",{"type":45,"tag":80,"props":619,"children":621},{"className":620},[],[622],{"type":51,"value":623},"jq",{"type":51,"value":625},", and ",{"type":45,"tag":80,"props":627,"children":629},{"className":628},[],[630],{"type":51,"value":305},{"type":51,"value":632},". The clone must have full\nhistory (",{"type":45,"tag":80,"props":634,"children":636},{"className":635},[],[637],{"type":51,"value":638},"git fetch --unshallow",{"type":51,"value":640}," on a shallow one), because the authoritative diff is a three-dot\ndiff from the merge base.",{"type":45,"tag":642,"props":643,"children":644},"hr",{},[],{"type":45,"tag":646,"props":647,"children":649},"h2",{"id":648},"_0-ground-rules",[650],{"type":51,"value":651},"0. Ground rules",{"type":45,"tag":653,"props":654,"children":655},"ol",{},[656,689,728,770,804,814,824],{"type":45,"tag":657,"props":658,"children":659},"li",{},[660,665,667,672,674,679,681,687],{"type":45,"tag":60,"props":661,"children":662},{},[663],{"type":51,"value":664},"Touch only the current directory, and never disturb local work.",{"type":51,"value":666}," When the current directory\nhas ",{"type":45,"tag":60,"props":668,"children":669},{},[670],{"type":51,"value":671},"modified tracked files",{"type":51,"value":673},", refuse to switch, report the situation, and let the user commit\nor stash it themselves - ",{"type":45,"tag":60,"props":675,"children":676},{},[677],{"type":51,"value":678},"never stash, reset, or delete a branch on the user's behalf",{"type":51,"value":680},", and\nnever go hunting for some other directory to work in. Every switch is\n",{"type":45,"tag":80,"props":682,"children":684},{"className":683},[],[685],{"type":51,"value":686},"git checkout --detach",{"type":51,"value":688},", so branch refs stay exactly where they were.",{"type":45,"tag":657,"props":690,"children":691},{},[692,697,699,704,706,711,713,718,720,726],{"type":45,"tag":60,"props":693,"children":694},{},[695],{"type":51,"value":696},"Read-only review.",{"type":51,"value":698}," Do not build, do not run tests, do not modify any source file in the\nrepository. The only local writes allowed are the two documents under ",{"type":45,"tag":80,"props":700,"children":702},{"className":701},[],[703],{"type":51,"value":110},{"type":51,"value":705}," and the\ncontext directory ",{"type":45,"tag":80,"props":707,"children":709},{"className":708},[],[710],{"type":51,"value":234},{"type":51,"value":712},".\n",{"type":45,"tag":60,"props":714,"children":715},{},[716],{"type":51,"value":717},"Exactly one thing may ever be written to GitHub",{"type":51,"value":719},": the PASS comment of step 10, only when the\nverdict is APPROVE, only through ",{"type":45,"tag":80,"props":721,"children":723},{"className":722},[],[724],{"type":51,"value":725},"post-pass-comment.sh",{"type":51,"value":727},", and only after the user has seen the\nrendered body and said go. A REQUEST_CHANGES review posts nothing. No inline comments, no\nreview submission, no labels, no edits to the PR body - and never a comment on any PR other\nthan the one being reviewed.",{"type":45,"tag":657,"props":729,"children":730},{},[731,736,738,744,746,752,754,760,762,768],{"type":45,"tag":60,"props":732,"children":733},{},[734],{"type":51,"value":735},"The diff has exactly one source",{"type":51,"value":737},": ",{"type":45,"tag":80,"props":739,"children":741},{"className":740},[],[742],{"type":51,"value":743},"$CTX\u002Fpr.diff",{"type":51,"value":745}," and ",{"type":45,"tag":80,"props":747,"children":749},{"className":748},[],[750],{"type":51,"value":751},"$CTX\u002Fpr_changed_files.txt",{"type":51,"value":753},". Do not\nreach for ",{"type":45,"tag":80,"props":755,"children":757},{"className":756},[],[758],{"type":51,"value":759},"gh pr diff",{"type":51,"value":761},", the web UI, or a hand-rolled ",{"type":45,"tag":80,"props":763,"children":765},{"className":764},[],[766],{"type":51,"value":767},"git diff",{"type":51,"value":769}," to get the change list - a\ndifferent way of fetching it means a different base.",{"type":45,"tag":657,"props":771,"children":772},{},[773,778,780,786,788,794,796,802],{"type":45,"tag":60,"props":774,"children":775},{},[776],{"type":51,"value":777},"Confirm a path before reading it.",{"type":51,"value":779}," If a path is not already confirmed by\n",{"type":45,"tag":80,"props":781,"children":783},{"className":782},[],[784],{"type":51,"value":785},"pr_changed_files.txt",{"type":51,"value":787},", ",{"type":45,"tag":80,"props":789,"children":791},{"className":790},[],[792],{"type":51,"value":793},"pr.diff",{"type":51,"value":795},", or the output of an earlier successful command, run\n",{"type":45,"tag":80,"props":797,"children":799},{"className":798},[],[800],{"type":51,"value":801},"rg --files",{"type":51,"value":803}," to confirm it first.",{"type":45,"tag":657,"props":805,"children":806},{},[807,812],{"type":45,"tag":60,"props":808,"children":809},{},[810],{"type":51,"value":811},"Do not stop at the first blocking issue.",{"type":51,"value":813}," Work through the changed files, the related\ncontrol flow, the tests, and the parallel or special-case paths.",{"type":45,"tag":657,"props":815,"children":816},{},[817,822],{"type":45,"tag":60,"props":818,"children":819},{},[820],{"type":51,"value":821},"Every suspicion must reach a conclusion",{"type":51,"value":823},": it becomes a finding, is excluded as \"already\ncovered by an existing comment\", or is ruled out with concrete code evidence and written into\n\"Considered and Dismissed\". Silently dropping a suspicion means the review is not finished.",{"type":45,"tag":657,"props":825,"children":826},{},[827,832,834,839,841,847],{"type":45,"tag":60,"props":828,"children":829},{},[830],{"type":51,"value":831},"There are no inline comments locally",{"type":51,"value":833},", so an anchor is the reader's only way in. Every\nfinding must carry a ",{"type":45,"tag":80,"props":835,"children":837},{"className":836},[],[838],{"type":51,"value":518},{"type":51,"value":840}," anchor (new-side line numbers) plus a verbatim snippet, and\nthe documents must pass ",{"type":45,"tag":80,"props":842,"children":844},{"className":843},[],[845],{"type":51,"value":846},"verify-anchors.py",{"type":51,"value":848}," at the end.",{"type":45,"tag":642,"props":850,"children":851},{},[],{"type":45,"tag":646,"props":853,"children":855},{"id":854},"_1-align-the-current-directory",[856],{"type":51,"value":857},"1. Align the current directory",{"type":45,"tag":73,"props":859,"children":861},{"className":154,"code":860,"language":156,"meta":82,"style":82},"CTX=\"\u003Csession scratchpad>\u002Freview-ctx\"\n$S\u002Falign-to-pr.sh \u003CPR> --out \"$CTX\u002Falign.env\"\n",[862],{"type":45,"tag":80,"props":863,"children":864},{"__ignoreMap":82},[865,892],{"type":45,"tag":162,"props":866,"children":867},{"class":164,"line":165},[868,873,878,882,887],{"type":45,"tag":162,"props":869,"children":870},{"style":169},[871],{"type":51,"value":872},"CTX",{"type":45,"tag":162,"props":874,"children":875},{"style":175},[876],{"type":51,"value":877},"=",{"type":45,"tag":162,"props":879,"children":880},{"style":175},[881],{"type":51,"value":229},{"type":45,"tag":162,"props":883,"children":884},{"style":237},[885],{"type":51,"value":886},"\u003Csession scratchpad>\u002Freview-ctx",{"type":45,"tag":162,"props":888,"children":889},{"style":175},[890],{"type":51,"value":891},"\"\n",{"type":45,"tag":162,"props":893,"children":894},{"class":164,"line":202},[895,899,903,907,911,915,919,923,927],{"type":45,"tag":162,"props":896,"children":897},{"style":169},[898],{"type":51,"value":172},{"type":45,"tag":162,"props":900,"children":901},{"style":175},[902],{"type":51,"value":178},{"type":45,"tag":162,"props":904,"children":905},{"style":169},[906],{"type":51,"value":183},{"type":45,"tag":162,"props":908,"children":909},{"style":175},[910],{"type":51,"value":188},{"type":45,"tag":162,"props":912,"children":913},{"style":169},[914],{"type":51,"value":224},{"type":45,"tag":162,"props":916,"children":917},{"style":175},[918],{"type":51,"value":229},{"type":45,"tag":162,"props":920,"children":921},{"style":169},[922],{"type":51,"value":234},{"type":45,"tag":162,"props":924,"children":925},{"style":237},[926],{"type":51,"value":240},{"type":45,"tag":162,"props":928,"children":929},{"style":175},[930],{"type":51,"value":891},{"type":45,"tag":54,"props":932,"children":933},{},[934,936,941],{"type":51,"value":935},"The script does four things, ",{"type":45,"tag":60,"props":937,"children":938},{},[939],{"type":51,"value":940},"all of them against the current worktree only",{"type":51,"value":942},":",{"type":45,"tag":653,"props":944,"children":945},{},[946,972,1063,1088],{"type":45,"tag":657,"props":947,"children":948},{},[949,954,956,962,964,970],{"type":45,"tag":60,"props":950,"children":951},{},[952],{"type":51,"value":953},"Resolve the PR",{"type":51,"value":955},": a URL, ",{"type":45,"tag":80,"props":957,"children":959},{"className":958},[],[960],{"type":51,"value":961},"owner\u002Frepo#N",{"type":51,"value":963},", or a bare number are all accepted; ",{"type":45,"tag":80,"props":965,"children":967},{"className":966},[],[968],{"type":51,"value":969},"gh api",{"type":51,"value":971}," supplies\ntitle, state, head ref\u002Frepo\u002Fsha, and base ref\u002Fsha.",{"type":45,"tag":657,"props":973,"children":974},{},[975,980,982],{"type":45,"tag":60,"props":976,"children":977},{},[978],{"type":51,"value":979},"Diagnose consistency",{"type":51,"value":981}," (the two checks the user explicitly wants):",{"type":45,"tag":983,"props":984,"children":985},"ul",{},[986,997],{"type":45,"tag":657,"props":987,"children":988},{},[989,995],{"type":45,"tag":80,"props":990,"children":992},{"className":991},[],[993],{"type":51,"value":994},"branch check",{"type":51,"value":996},": is the current branch the PR's head branch?",{"type":45,"tag":657,"props":998,"children":999},{},[1000,1006,1007,1013,1015,1021,1022,1028,1029,1035,1036,1042,1047,1049,1054,1056,1061],{"type":45,"tag":80,"props":1001,"children":1003},{"className":1002},[],[1004],{"type":51,"value":1005},"commit check",{"type":51,"value":737},{"type":45,"tag":80,"props":1008,"children":1010},{"className":1009},[],[1011],{"type":51,"value":1012},"same",{"type":51,"value":1014}," \u002F ",{"type":45,"tag":80,"props":1016,"children":1018},{"className":1017},[],[1019],{"type":51,"value":1020},"ahead:N",{"type":51,"value":1014},{"type":45,"tag":80,"props":1023,"children":1025},{"className":1024},[],[1026],{"type":51,"value":1027},"behind:N",{"type":51,"value":1014},{"type":45,"tag":80,"props":1030,"children":1032},{"className":1031},[],[1033],{"type":51,"value":1034},"diverged:a\u002Fb",{"type":51,"value":1014},{"type":45,"tag":80,"props":1037,"children":1039},{"className":1038},[],[1040],{"type":51,"value":1041},"unrelated",{"type":45,"tag":60,"props":1043,"children":1044},{},[1045],{"type":51,"value":1046},"Relay both lines to the user verbatim.",{"type":51,"value":1048}," In particular ",{"type":45,"tag":80,"props":1050,"children":1052},{"className":1051},[],[1053],{"type":51,"value":1020},{"type":51,"value":1055}," means there are local commits\nthat were never pushed, and ",{"type":45,"tag":60,"props":1057,"children":1058},{},[1059],{"type":51,"value":1060},"they are not part of the review",{"type":51,"value":1062},".",{"type":45,"tag":657,"props":1064,"children":1065},{},[1066,1071,1072,1078,1080,1086],{"type":45,"tag":60,"props":1067,"children":1068},{},[1069],{"type":51,"value":1070},"Fetch the PR head",{"type":51,"value":737},{"type":45,"tag":80,"props":1073,"children":1075},{"className":1074},[],[1076],{"type":51,"value":1077},"git fetch \u003Cupstream> +refs\u002Fpull\u002FN\u002Fhead:refs\u002Fdoris-review\u002Fpr-N",{"type":51,"value":1079},". It goes\nthrough ",{"type":45,"tag":80,"props":1081,"children":1083},{"className":1082},[],[1084],{"type":51,"value":1085},"refs\u002Fpull\u002F*",{"type":51,"value":1087},", so a fork PR needs no extra remote. The fetched sha must equal the head\nsha from the API; if it does not, the PR was pushed to while the context was being prepared -\nthe script re-reads the API once and errors out asking for a re-run if it still differs (this\nmirrors CI's \"PR changed while review context was being prepared\" guard).",{"type":45,"tag":657,"props":1089,"children":1090},{},[1091,1096,1097,1185,1189,1191,1196,1198,1204,1206,1211],{"type":45,"tag":60,"props":1092,"children":1093},{},[1094],{"type":51,"value":1095},"Align the current directory",{"type":51,"value":942},{"type":45,"tag":435,"props":1098,"children":1099},{},[1100,1116],{"type":45,"tag":439,"props":1101,"children":1102},{},[1103],{"type":45,"tag":443,"props":1104,"children":1105},{},[1106,1111],{"type":45,"tag":447,"props":1107,"children":1108},{},[1109],{"type":51,"value":1110},"State of the current directory",{"type":45,"tag":447,"props":1112,"children":1113},{},[1114],{"type":51,"value":1115},"Action",{"type":45,"tag":458,"props":1117,"children":1118},{},[1119,1138,1162],{"type":45,"tag":443,"props":1120,"children":1121},{},[1122,1127],{"type":45,"tag":465,"props":1123,"children":1124},{},[1125],{"type":51,"value":1126},"HEAD already equals the PR head",{"type":45,"tag":465,"props":1128,"children":1129},{},[1130,1136],{"type":45,"tag":80,"props":1131,"children":1133},{"className":1132},[],[1134],{"type":51,"value":1135},"ALIGN_MODE=already",{"type":51,"value":1137},", nothing to do",{"type":45,"tag":443,"props":1139,"children":1140},{},[1141,1146],{"type":45,"tag":465,"props":1142,"children":1143},{},[1144],{"type":51,"value":1145},"Clean",{"type":45,"tag":465,"props":1147,"children":1148},{},[1149,1155,1156],{"type":45,"tag":80,"props":1150,"children":1152},{"className":1151},[],[1153],{"type":51,"value":1154},"git checkout --detach \u003CPR head>",{"type":51,"value":787},{"type":45,"tag":80,"props":1157,"children":1159},{"className":1158},[],[1160],{"type":51,"value":1161},"ALIGN_MODE=switched",{"type":45,"tag":443,"props":1163,"children":1164},{},[1165,1170],{"type":45,"tag":465,"props":1166,"children":1167},{},[1168],{"type":51,"value":1169},"Modified tracked files \u002F a rebase, merge, or cherry-pick in progress",{"type":45,"tag":465,"props":1171,"children":1172},{},[1173,1179,1180],{"type":45,"tag":80,"props":1174,"children":1176},{"className":1175},[],[1177],{"type":51,"value":1178},"ALIGN_MODE=blocked",{"type":51,"value":787},{"type":45,"tag":60,"props":1181,"children":1182},{},[1183],{"type":51,"value":1184},"refuse and exit",{"type":45,"tag":1186,"props":1187,"children":1188},"br",{},[],{"type":51,"value":1190},"\"Clean\" means ",{"type":45,"tag":60,"props":1192,"children":1193},{},[1194],{"type":51,"value":1195},"no modified tracked files",{"type":51,"value":1197},". Untracked files do not count (",{"type":45,"tag":80,"props":1199,"children":1201},{"className":1200},[],[1202],{"type":51,"value":1203},"git checkout",{"type":51,"value":1205}," never\ndeletes them, and refuses outright when it would overwrite one), so an untracked directory such\nas ",{"type":45,"tag":80,"props":1207,"children":1209},{"className":1208},[],[1210],{"type":51,"value":110},{"type":51,"value":1212}," does not affect the verdict and survives the switch.",{"type":45,"tag":54,"props":1214,"children":1215},{},[1216,1221,1223,1228,1230,1236],{"type":45,"tag":60,"props":1217,"children":1218},{},[1219],{"type":51,"value":1220},"What to do when BLOCKED",{"type":51,"value":1222},": relay the dirty-file list the script printed, then ask the user to\npick - (a) commit it themselves, (b) stash it themselves, or (c) start over in a different\ndirectory. ",{"type":45,"tag":60,"props":1224,"children":1225},{},[1226],{"type":51,"value":1227},"Do not perform any of those for them",{"type":51,"value":1229},"; once they have chosen and acted, re-run this\nskill. To see the diagnosis without changing anything, use ",{"type":45,"tag":80,"props":1231,"children":1233},{"className":1232},[],[1234],{"type":51,"value":1235},"--check",{"type":51,"value":1237}," (it still prints the full\nbranch check \u002F commit check before reporting BLOCKED).",{"type":45,"tag":54,"props":1239,"children":1240},{},[1241,1243,1249,1251,1257,1259,1265,1267,1272,1274,1280,1281,1287,1288,1294,1296,1302,1303,1309],{"type":51,"value":1242},"Once aligned, ",{"type":45,"tag":80,"props":1244,"children":1246},{"className":1245},[],[1247],{"type":51,"value":1248},"$CTX\u002Falign.env",{"type":51,"value":1250}," supplies ",{"type":45,"tag":80,"props":1252,"children":1254},{"className":1253},[],[1255],{"type":51,"value":1256},"WORKDIR",{"type":51,"value":1258}," (= the current directory), ",{"type":45,"tag":80,"props":1260,"children":1262},{"className":1261},[],[1263],{"type":51,"value":1264},"DOCS_ROOT",{"type":51,"value":1266},"\n(= ",{"type":45,"tag":80,"props":1268,"children":1270},{"className":1269},[],[1271],{"type":51,"value":1256},{"type":51,"value":1273},"), ",{"type":45,"tag":80,"props":1275,"children":1277},{"className":1276},[],[1278],{"type":51,"value":1279},"PR_HEAD_SHA",{"type":51,"value":1014},{"type":45,"tag":80,"props":1282,"children":1284},{"className":1283},[],[1285],{"type":51,"value":1286},"PR_BASE_SHA",{"type":51,"value":787},{"type":45,"tag":80,"props":1289,"children":1291},{"className":1290},[],[1292],{"type":51,"value":1293},"PREV_REF",{"type":51,"value":1295}," (the restore point),\n",{"type":45,"tag":80,"props":1297,"children":1299},{"className":1298},[],[1300],{"type":51,"value":1301},"BRANCH_MATCHES_PR",{"type":51,"value":625},{"type":45,"tag":80,"props":1304,"children":1306},{"className":1305},[],[1307],{"type":51,"value":1308},"LOCAL_VS_PR",{"type":51,"value":1062},{"type":45,"tag":642,"props":1311,"children":1312},{},[],{"type":45,"tag":646,"props":1314,"children":1316},{"id":1315},"_2-prepare-the-context",[1317],{"type":51,"value":1318},"2. Prepare the context",{"type":45,"tag":73,"props":1320,"children":1322},{"className":154,"code":1321,"language":156,"meta":82,"style":82},"$S\u002Fprepare-review-context.sh \\\n    --ctx \"$CTX\" --align \"$CTX\u002Falign.env\" [--focus \"\u003Cuser focus>\"] [--fresh]\n",[1323],{"type":45,"tag":80,"props":1324,"children":1325},{"__ignoreMap":82},[1326,1334],{"type":45,"tag":162,"props":1327,"children":1328},{"class":164,"line":165},[1329],{"type":45,"tag":162,"props":1330,"children":1331},{"style":169},[1332],{"type":51,"value":1333},"$S\u002Fprepare-review-context.sh \\\n",{"type":45,"tag":162,"props":1335,"children":1336},{"class":164,"line":202},[1337,1342,1346,1350,1354,1359,1363,1367,1371,1375,1380,1384,1389,1393,1398],{"type":45,"tag":162,"props":1338,"children":1339},{"style":302},[1340],{"type":51,"value":1341},"    --ctx",{"type":45,"tag":162,"props":1343,"children":1344},{"style":175},[1345],{"type":51,"value":325},{"type":45,"tag":162,"props":1347,"children":1348},{"style":169},[1349],{"type":51,"value":234},{"type":45,"tag":162,"props":1351,"children":1352},{"style":175},[1353],{"type":51,"value":229},{"type":45,"tag":162,"props":1355,"children":1356},{"style":237},[1357],{"type":51,"value":1358}," --align",{"type":45,"tag":162,"props":1360,"children":1361},{"style":175},[1362],{"type":51,"value":325},{"type":45,"tag":162,"props":1364,"children":1365},{"style":169},[1366],{"type":51,"value":234},{"type":45,"tag":162,"props":1368,"children":1369},{"style":237},[1370],{"type":51,"value":240},{"type":45,"tag":162,"props":1372,"children":1373},{"style":175},[1374],{"type":51,"value":229},{"type":45,"tag":162,"props":1376,"children":1377},{"style":169},[1378],{"type":51,"value":1379}," [--focus ",{"type":45,"tag":162,"props":1381,"children":1382},{"style":175},[1383],{"type":51,"value":229},{"type":45,"tag":162,"props":1385,"children":1386},{"style":237},[1387],{"type":51,"value":1388},"\u003Cuser focus>",{"type":45,"tag":162,"props":1390,"children":1391},{"style":175},[1392],{"type":51,"value":229},{"type":45,"tag":162,"props":1394,"children":1395},{"style":237},[1396],{"type":51,"value":1397},"]",{"type":45,"tag":162,"props":1399,"children":1400},{"style":169},[1401],{"type":51,"value":1402}," [--fresh]\n",{"type":45,"tag":54,"props":1404,"children":1405},{},[1406,1412,1414,1419,1421,1427,1429,1435],{"type":45,"tag":80,"props":1407,"children":1409},{"className":1408},[],[1410],{"type":51,"value":1411},"--align",{"type":51,"value":1413}," carries over ",{"type":45,"tag":80,"props":1415,"children":1417},{"className":1416},[],[1418],{"type":51,"value":1256},{"type":51,"value":1420},", the PR number, the upstream repo, and the PR base sha, and\nverifies the checkout really is parked on the PR head (if it is not, the script errors out and\nasks you to redo step 1). ",{"type":45,"tag":80,"props":1422,"children":1424},{"className":1423},[],[1425],{"type":51,"value":1426},"--fresh",{"type":51,"value":1428}," wipes and rebuilds everything including the ledger; without it\nthe ledger is preserved so a multi-round run can continue. Whatever free text the user typed after\nthe PR URL on the command line is the ",{"type":45,"tag":80,"props":1430,"children":1432},{"className":1431},[],[1433],{"type":51,"value":1434},"--focus",{"type":51,"value":1436}," value.",{"type":45,"tag":54,"props":1438,"children":1439},{},[1440,1442,1447],{"type":51,"value":1441},"Output (under ",{"type":45,"tag":80,"props":1443,"children":1445},{"className":1444},[],[1446],{"type":51,"value":234},{"type":51,"value":151},{"type":45,"tag":435,"props":1449,"children":1450},{},[1451,1466],{"type":45,"tag":439,"props":1452,"children":1453},{},[1454],{"type":45,"tag":443,"props":1455,"children":1456},{},[1457,1461],{"type":45,"tag":447,"props":1458,"children":1459},{},[1460],{"type":51,"value":451},{"type":45,"tag":447,"props":1462,"children":1463},{},[1464],{"type":51,"value":1465},"Contents",{"type":45,"tag":458,"props":1467,"children":1468},{},[1469,1500,1529,1558,1582,1599,1623,1640,1664],{"type":45,"tag":443,"props":1470,"children":1471},{},[1472,1481],{"type":45,"tag":465,"props":1473,"children":1474},{},[1475],{"type":45,"tag":80,"props":1476,"children":1478},{"className":1477},[],[1479],{"type":51,"value":1480},"meta.env",{"type":45,"tag":465,"props":1482,"children":1483},{},[1484,1486,1491,1492,1498],{"type":51,"value":1485},"PR info, base\u002Fhead sha, diff range, ",{"type":45,"tag":80,"props":1487,"children":1489},{"className":1488},[],[1490],{"type":51,"value":1264},{"type":51,"value":787},{"type":45,"tag":80,"props":1493,"children":1495},{"className":1494},[],[1496],{"type":51,"value":1497},"ALIGN_MODE",{"type":51,"value":1499},", date, dirty-file count",{"type":45,"tag":443,"props":1501,"children":1502},{},[1503,1524],{"type":45,"tag":465,"props":1504,"children":1505},{},[1506,1511,1512,1517,1518],{"type":45,"tag":80,"props":1507,"children":1509},{"className":1508},[],[1510],{"type":51,"value":793},{"type":51,"value":1014},{"type":45,"tag":80,"props":1513,"children":1515},{"className":1514},[],[1516],{"type":51,"value":785},{"type":51,"value":1014},{"type":45,"tag":80,"props":1519,"children":1521},{"className":1520},[],[1522],{"type":51,"value":1523},"pr_changed_files_status.txt",{"type":45,"tag":465,"props":1525,"children":1526},{},[1527],{"type":51,"value":1528},"The authoritative diff and change list",{"type":45,"tag":443,"props":1530,"children":1531},{},[1532,1548],{"type":45,"tag":465,"props":1533,"children":1534},{},[1535,1541,1542],{"type":45,"tag":80,"props":1536,"children":1538},{"className":1537},[],[1539],{"type":51,"value":1540},"changed_line_ranges.txt",{"type":51,"value":1014},{"type":45,"tag":80,"props":1543,"children":1545},{"className":1544},[],[1546],{"type":51,"value":1547},".tsv",{"type":45,"tag":465,"props":1549,"children":1550},{},[1551,1556],{"type":45,"tag":60,"props":1552,"children":1553},{},[1554],{"type":51,"value":1555},"New-side (post-change) line ranges",{"type":51,"value":1557}," - the source for anchors",{"type":45,"tag":443,"props":1559,"children":1560},{},[1561,1577],{"type":45,"tag":465,"props":1562,"children":1563},{},[1564,1570,1571],{"type":45,"tag":80,"props":1565,"children":1567},{"className":1566},[],[1568],{"type":51,"value":1569},"pr_commits.txt",{"type":51,"value":1014},{"type":45,"tag":80,"props":1572,"children":1574},{"className":1573},[],[1575],{"type":51,"value":1576},"pr_diffstat.txt",{"type":45,"tag":465,"props":1578,"children":1579},{},[1580],{"type":51,"value":1581},"Commit list and change size",{"type":45,"tag":443,"props":1583,"children":1584},{},[1585,1594],{"type":45,"tag":465,"props":1586,"children":1587},{},[1588],{"type":45,"tag":80,"props":1589,"children":1591},{"className":1590},[],[1592],{"type":51,"value":1593},"pr_review_threads.md",{"type":45,"tag":465,"props":1595,"children":1596},{},[1597],{"type":51,"value":1598},"Existing inline comments (at most 30 threads, each body truncated at 1200 chars)",{"type":45,"tag":443,"props":1600,"children":1601},{},[1602,1618],{"type":45,"tag":465,"props":1603,"children":1604},{},[1605,1611,1612],{"type":45,"tag":80,"props":1606,"children":1608},{"className":1607},[],[1609],{"type":51,"value":1610},"required_agents.txt",{"type":51,"value":1014},{"type":45,"tag":80,"props":1613,"children":1615},{"className":1614},[],[1616],{"type":51,"value":1617},"required_agents_prompt.txt",{"type":45,"tag":465,"props":1619,"children":1620},{},[1621],{"type":51,"value":1622},"Required AGENTS.md files, derived from the changed files' ancestor directories",{"type":45,"tag":443,"props":1624,"children":1625},{},[1626,1635],{"type":45,"tag":465,"props":1627,"children":1628},{},[1629],{"type":45,"tag":80,"props":1630,"children":1632},{"className":1631},[],[1633],{"type":51,"value":1634},"review_focus.txt",{"type":45,"tag":465,"props":1636,"children":1637},{},[1638],{"type":51,"value":1639},"The user's focus points",{"type":45,"tag":443,"props":1641,"children":1642},{},[1643,1652],{"type":45,"tag":465,"props":1644,"children":1645},{},[1646],{"type":45,"tag":80,"props":1647,"children":1649},{"className":1648},[],[1650],{"type":51,"value":1651},"worktree_status.txt",{"type":45,"tag":465,"props":1653,"children":1654},{},[1655,1657,1662],{"type":51,"value":1656},"Uncommitted changes in the review tree (",{"type":45,"tag":60,"props":1658,"children":1659},{},[1660],{"type":51,"value":1661},"out of review scope",{"type":51,"value":1663},"; call them out in the documents)",{"type":45,"tag":443,"props":1665,"children":1666},{},[1667,1676],{"type":45,"tag":465,"props":1668,"children":1669},{},[1670],{"type":45,"tag":80,"props":1671,"children":1673},{"className":1672},[],[1674],{"type":51,"value":1675},"ledger\u002F",{"type":45,"tag":465,"props":1677,"children":1678},{},[1679],{"type":51,"value":1680},"Shared-ledger skeleton",{"type":45,"tag":54,"props":1682,"children":1683},{},[1684,1686,1692,1694,1700],{"type":51,"value":1685},"If ",{"type":45,"tag":80,"props":1687,"children":1689},{"className":1688},[],[1690],{"type":51,"value":1691},"BASE_SOURCE",{"type":51,"value":1693}," is not ",{"type":45,"tag":80,"props":1695,"children":1697},{"className":1696},[],[1698],{"type":51,"value":1699},"PR base sha (matches CI)",{"type":51,"value":1701},", the baseline differs from CI's and the\ndocuments must say so.",{"type":45,"tag":642,"props":1703,"children":1704},{},[],{"type":45,"tag":646,"props":1706,"children":1708},{"id":1707},"_3-required-reading-in-this-order",[1709],{"type":51,"value":1710},"3. Required reading (in this order)",{"type":45,"tag":54,"props":1712,"children":1713},{},[1714],{"type":51,"value":1715},"Before looking at any code, the main agent reads, in order:",{"type":45,"tag":653,"props":1717,"children":1718},{},[1719,1737,1755,1773,1784],{"type":45,"tag":657,"props":1720,"children":1721},{},[1722,1728,1730,1736],{"type":45,"tag":80,"props":1723,"children":1725},{"className":1724},[],[1726],{"type":51,"value":1727},"$WORKDIR\u002F.claude\u002Fskills\u002Fcode-review\u002FSKILL.md",{"type":51,"value":1729}," - the Doris review checklist; follow it strictly\nthroughout. On an older branch that does not have it, fetch it with\n",{"type":45,"tag":80,"props":1731,"children":1733},{"className":1732},[],[1734],{"type":51,"value":1735},"gh api repos\u002Fapache\u002Fdoris\u002Fcontents\u002F.claude\u002Fskills\u002Fcode-review\u002FSKILL.md?ref=master",{"type":51,"value":1062},{"type":45,"tag":657,"props":1738,"children":1739},{},[1740,1745,1747,1753],{"type":45,"tag":60,"props":1741,"children":1742},{},[1743],{"type":51,"value":1744},"Every",{"type":51,"value":1746}," AGENTS.md listed in ",{"type":45,"tag":80,"props":1748,"children":1750},{"className":1749},[],[1751],{"type":51,"value":1752},"$CTX\u002Frequired_agents.txt",{"type":51,"value":1754},", each one read in full. Listing the\ndirectory or grepping the path does not count.",{"type":45,"tag":657,"props":1756,"children":1757},{},[1758,1764,1766,1771],{"type":45,"tag":80,"props":1759,"children":1761},{"className":1760},[],[1762],{"type":51,"value":1763},"$CTX\u002Fpr_review_threads.md",{"type":51,"value":1765}," - treat every existing comment as already-known context. ",{"type":45,"tag":60,"props":1767,"children":1768},{},[1769],{"type":51,"value":1770},"Do not\nraise",{"type":51,"value":1772}," the same or a substantially similar issue again, even phrased differently. Raise a\nsimilar concern only when this PR introduces a genuinely different instance somewhere else that\nthe existing comments do not cover, and say why it is distinct.",{"type":45,"tag":657,"props":1774,"children":1775},{},[1776,1782],{"type":45,"tag":80,"props":1777,"children":1779},{"className":1778},[],[1780],{"type":51,"value":1781},"$CTX\u002Freview_focus.txt",{"type":51,"value":1783}," - do the full review as usual and pay extra attention to these points;\nthe final documents must respond to each one, even if the conclusion is \"no additional issue\nfound for this point\".",{"type":45,"tag":657,"props":1785,"children":1786},{},[1787,1789,1794,1796,1802],{"type":51,"value":1788},"All of ",{"type":45,"tag":80,"props":1790,"children":1792},{"className":1791},[],[1793],{"type":51,"value":743},{"type":51,"value":1795}," plus ",{"type":45,"tag":80,"props":1797,"children":1799},{"className":1798},[],[1800],{"type":51,"value":1801},"$CTX\u002Fpr_commits.txt",{"type":51,"value":1803},". When the diff is large (> 5000 lines), read it\nin chunks by directory or module until it is covered, and describe how you read it in \"Coverage\nand Limits\" - do not pretend you read it all.",{"type":45,"tag":642,"props":1805,"children":1806},{},[],{"type":45,"tag":646,"props":1808,"children":1810},{"id":1809},"_4-main-agent-initial-risk-scan-before-spawning-any-subagent",[1811],{"type":51,"value":1812},"4. Main-agent initial risk scan (before spawning any subagent)",{"type":45,"tag":54,"props":1814,"children":1815},{},[1816,1821],{"type":45,"tag":60,"props":1817,"children":1818},{},[1819],{"type":51,"value":1820},"This must be finished before any subagent is spawned.",{"type":51,"value":1822}," Read the whole change, understand every\nmechanism it touches, then answer: if this PR is wrong, where is it most likely to be wrong? Which\npoints look risky to you?",{"type":45,"tag":54,"props":1824,"children":1825},{},[1826,1828,1834,1836,1841],{"type":51,"value":1827},"Write the result into ",{"type":45,"tag":80,"props":1829,"children":1831},{"className":1830},[],[1832],{"type":51,"value":1833},"$CTX\u002Fledger\u002F00-main-risk-scan.md",{"type":51,"value":1835},", each entry carrying: an ID, the changed\nfiles\u002Flines involved, the related mechanism to inspect, why it is suspicious, the upstream or\ndownstream files that must be read alongside it, and the ",{"type":45,"tag":60,"props":1837,"children":1838},{},[1839],{"type":51,"value":1840},"specific question",{"type":51,"value":1842}," a risk-focused\nsubagent has to answer.",{"type":45,"tag":54,"props":1844,"children":1845},{},[1846],{"type":51,"value":1847},"An empty risk scan means you have not understood the PR yet - go back and read it; do not skip\nthis step.",{"type":45,"tag":642,"props":1849,"children":1850},{},[],{"type":45,"tag":646,"props":1852,"children":1854},{"id":1853},"_5-spawn-the-subagents",[1855],{"type":51,"value":1856},"5. Spawn the subagents",{"type":45,"tag":54,"props":1858,"children":1859},{},[1860,1862,1867],{"type":51,"value":1861},"Split along the coverage the code-review skill requires, and ",{"type":45,"tag":60,"props":1863,"children":1864},{},[1865],{"type":51,"value":1866},"send them all in one message so\nthey run concurrently",{"type":51,"value":1868}," (general-purpose subagents):",{"type":45,"tag":435,"props":1870,"children":1871},{},[1872,1893],{"type":45,"tag":439,"props":1873,"children":1874},{},[1875],{"type":45,"tag":443,"props":1876,"children":1877},{},[1878,1883,1888],{"type":45,"tag":447,"props":1879,"children":1880},{},[1881],{"type":51,"value":1882},"Type",{"type":45,"tag":447,"props":1884,"children":1885},{},[1886],{"type":51,"value":1887},"Count",{"type":45,"tag":447,"props":1889,"children":1890},{},[1891],{"type":51,"value":1892},"Responsibility",{"type":45,"tag":458,"props":1894,"children":1895},{},[1896,1921],{"type":45,"tag":443,"props":1897,"children":1898},{},[1899,1904,1909],{"type":45,"tag":465,"props":1900,"children":1901},{},[1902],{"type":51,"value":1903},"Full-review subagent",{"type":45,"tag":465,"props":1905,"children":1906},{},[1907],{"type":51,"value":1908},"1-3 (depending on PR complexity)",{"type":45,"tag":465,"props":1910,"children":1911},{},[1912,1914,1919],{"type":51,"value":1913},"Each owns one slice; ",{"type":45,"tag":60,"props":1915,"children":1916},{},[1917],{"type":51,"value":1918},"their union must cover the whole PR",{"type":51,"value":1920}," and every checkpoint the skill requires",{"type":45,"tag":443,"props":1922,"children":1923},{},[1924,1929,1934],{"type":45,"tag":465,"props":1925,"children":1926},{},[1927],{"type":51,"value":1928},"Risk-focused subagent",{"type":45,"tag":465,"props":1930,"children":1931},{},[1932],{"type":51,"value":1933},"one per suspicious mechanism",{"type":45,"tag":465,"props":1935,"children":1936},{},[1937],{"type":51,"value":1938},"Answers exactly one question from the risk scan, independently of the full-review subagents",{"type":45,"tag":54,"props":1940,"children":1941},{},[1942,1944,1949,1951,1957],{"type":51,"value":1943},"Split by ",{"type":45,"tag":60,"props":1945,"children":1946},{},[1947],{"type":51,"value":1948},"module or mechanism",{"type":51,"value":1950}," (for example \"FE SPI interface and load path\", \"Ranger plugin\nimplementation and class loading\", \"build scripts and packaging\u002Fdeployment\"), not by dividing the\nfile count evenly - a split that lands in the middle of a mechanism leaves both halves unable to\nsee the bug. Give every subagent its own ledger file, ",{"type":45,"tag":80,"props":1952,"children":1954},{"className":1953},[],[1955],{"type":51,"value":1956},"$CTX\u002Fledger\u002Fsub-\u003Cround>-\u003Cid>.md",{"type":51,"value":1062},{"type":45,"tag":54,"props":1959,"children":1960},{},[1961,1963,1968,1970,1976,1977,1983,1985,1991,1993,1999,2000,2006,2007,2013,2014,2020,2021,2027,2029],{"type":51,"value":1962},"Take the prompts from ",{"type":45,"tag":80,"props":1964,"children":1966},{"className":1965},[],[1967],{"type":51,"value":549},{"type":51,"value":1969},": section A (shared preamble) plus section B\n(full review) or section C (risk-focused), substituting ",{"type":45,"tag":80,"props":1971,"children":1973},{"className":1972},[],[1974],{"type":51,"value":1975},"{CTX}",{"type":51,"value":1014},{"type":45,"tag":80,"props":1978,"children":1980},{"className":1979},[],[1981],{"type":51,"value":1982},"{REPO_ROOT}",{"type":51,"value":1984}," (= ",{"type":45,"tag":80,"props":1986,"children":1988},{"className":1987},[],[1989],{"type":51,"value":1990},"$WORKDIR",{"type":51,"value":1992},") \u002F\n",{"type":45,"tag":80,"props":1994,"children":1996},{"className":1995},[],[1997],{"type":51,"value":1998},"{BASE_SHA}",{"type":51,"value":1014},{"type":45,"tag":80,"props":2001,"children":2003},{"className":2002},[],[2004],{"type":51,"value":2005},"{HEAD_SHA}",{"type":51,"value":1014},{"type":45,"tag":80,"props":2008,"children":2010},{"className":2009},[],[2011],{"type":51,"value":2012},"{ROUND}",{"type":51,"value":1014},{"type":45,"tag":80,"props":2015,"children":2017},{"className":2016},[],[2018],{"type":51,"value":2019},"{AGENT_ID}",{"type":51,"value":1014},{"type":45,"tag":80,"props":2022,"children":2024},{"className":2023},[],[2025],{"type":51,"value":2026},"{FOCUS}",{"type":51,"value":2028},". ",{"type":45,"tag":60,"props":2030,"children":2031},{},[2032],{"type":51,"value":2033},"Every subagent prompt must\nstate the ledger directory and the path of that subagent's own file.",{"type":45,"tag":2035,"props":2036,"children":2037},"blockquote",{},[2038],{"type":45,"tag":54,"props":2039,"children":2040},{},[2041,2046,2048,2054,2056,2061,2063,2068],{"type":45,"tag":60,"props":2042,"children":2043},{},[2044],{"type":51,"value":2045},"One implementation difference from CI",{"type":51,"value":2047},": CI uses a single ",{"type":45,"tag":80,"props":2049,"children":2051},{"className":2050},[],[2052],{"type":51,"value":2053},"subagent_review_findings.md",{"type":51,"value":2055}," with\nsections; locally, concurrent writes to one file collide on patch, so this becomes ",{"type":45,"tag":60,"props":2057,"children":2058},{},[2059],{"type":51,"value":2060},"one file per\nowner",{"type":51,"value":2062}," under ",{"type":45,"tag":80,"props":2064,"children":2066},{"className":2065},[],[2067],{"type":51,"value":1675},{"type":51,"value":2069},". The semantics are unchanged - a single shared source of truth, everyone\nreads all of it, each writes only their own, the main agent merges.",{"type":45,"tag":642,"props":2071,"children":2072},{},[],{"type":45,"tag":646,"props":2074,"children":2076},{"id":2075},"_6-main-agent-merges-verifies-deduplicates",[2077],{"type":51,"value":2078},"6. Main agent merges, verifies, deduplicates",{"type":45,"tag":54,"props":2080,"children":2081},{},[2082,2094,2096,2102],{"type":45,"tag":60,"props":2083,"children":2084},{},[2085,2087,2092],{"type":51,"value":2086},"Re-read the entire ",{"type":45,"tag":80,"props":2088,"children":2090},{"className":2089},[],[2091],{"type":51,"value":1675},{"type":51,"value":2093}," directory every time a subagent returns",{"type":51,"value":2095},", then process each\ncandidate into ",{"type":45,"tag":80,"props":2097,"children":2099},{"className":2098},[],[2100],{"type":51,"value":2101},"$CTX\u002Fledger\u002Fmain-merged.md",{"type":51,"value":942},{"type":45,"tag":983,"props":2104,"children":2105},{},[2106,2116,2132,2163],{"type":45,"tag":657,"props":2107,"children":2108},{},[2109,2114],{"type":45,"tag":60,"props":2110,"children":2111},{},[2112],{"type":51,"value":2113},"Verify",{"type":51,"value":2115},": read the code yourself to confirm it; do not take a subagent's conclusion at face\nvalue. The common failure modes are treating a local pattern that an upstream guarantee already\ncovers as a bug, and an \"if A then B\" whose A has no concrete scenario.",{"type":45,"tag":657,"props":2117,"children":2118},{},[2119,2124,2126,2131],{"type":45,"tag":60,"props":2120,"children":2121},{},[2122],{"type":51,"value":2123},"Deduplicate",{"type":51,"value":2125},": against the existing candidates and against the existing comments in\n",{"type":45,"tag":80,"props":2127,"children":2129},{"className":2128},[],[2130],{"type":51,"value":1593},{"type":51,"value":1062},{"type":45,"tag":657,"props":2133,"children":2134},{},[2135,2140,2141,2147,2148,2154,2155,2161],{"type":45,"tag":60,"props":2136,"children":2137},{},[2138],{"type":51,"value":2139},"Assign a status",{"type":51,"value":737},{"type":45,"tag":80,"props":2142,"children":2144},{"className":2143},[],[2145],{"type":51,"value":2146},"accepted",{"type":51,"value":1014},{"type":45,"tag":80,"props":2149,"children":2151},{"className":2150},[],[2152],{"type":51,"value":2153},"dismissed_with_evidence",{"type":51,"value":1014},{"type":45,"tag":80,"props":2156,"children":2158},{"className":2157},[],[2159],{"type":51,"value":2160},"duplicated",{"type":51,"value":2162},". A dismissal must come\nwith concrete code evidence, which goes verbatim into the document's \"Considered and Dismissed\".",{"type":45,"tag":657,"props":2164,"children":2165},{},[2166,2171,2173,2179,2180,2186,2188,2194],{"type":45,"tag":60,"props":2167,"children":2168},{},[2169],{"type":51,"value":2170},"Fill the risk items back in",{"type":51,"value":2172},": update ",{"type":45,"tag":80,"props":2174,"children":2176},{"className":2175},[],[2177],{"type":51,"value":2178},"Status",{"type":51,"value":745},{"type":45,"tag":80,"props":2181,"children":2183},{"className":2182},[],[2184],{"type":51,"value":2185},"Final conclusion",{"type":51,"value":2187}," for every entry in\n",{"type":45,"tag":80,"props":2189,"children":2191},{"className":2190},[],[2192],{"type":51,"value":2193},"00-main-risk-scan.md",{"type":51,"value":1062},{"type":45,"tag":54,"props":2196,"children":2197},{},[2198,2200,2205],{"type":51,"value":2199},"When this step ends, ",{"type":45,"tag":60,"props":2201,"children":2202},{},[2203],{"type":51,"value":2204},"no candidate may be left without a status",{"type":51,"value":1062},{"type":45,"tag":642,"props":2207,"children":2208},{},[],{"type":45,"tag":646,"props":2210,"children":2212},{"id":2211},"_7-convergence-loop-at-most-3-rounds",[2213],{"type":51,"value":2214},"7. Convergence loop (at most 3 rounds)",{"type":45,"tag":54,"props":2216,"children":2217},{},[2218,2220,2226,2228,2234],{"type":51,"value":2219},"One round = step 5 + step 6. Record the outcome in the ",{"type":45,"tag":80,"props":2221,"children":2223},{"className":2222},[],[2224],{"type":51,"value":2225},"Convergence Rounds",{"type":51,"value":2227}," table of\n",{"type":45,"tag":80,"props":2229,"children":2231},{"className":2230},[],[2232],{"type":51,"value":2233},"main-merged.md",{"type":51,"value":1062},{"type":45,"tag":983,"props":2236,"children":2237},{},[2238,2251,2263],{"type":45,"tag":657,"props":2239,"children":2240},{},[2241,2243,2249],{"type":51,"value":2242},"Every subagent returned ",{"type":45,"tag":80,"props":2244,"children":2246},{"className":2245},[],[2247],{"type":51,"value":2248},"NO_NEW_VALUABLE_FINDINGS",{"type":51,"value":2250}," → go to step 8.",{"type":45,"tag":657,"props":2252,"children":2253},{},[2254,2256,2261],{"type":51,"value":2255},"New valuable candidates remain → start another round: ",{"type":45,"tag":60,"props":2257,"children":2258},{},[2259],{"type":51,"value":2260},"re-slice the coverage",{"type":51,"value":2262}," based on what\nthis round taught you (do not re-dispatch the same split unchanged), and add risk-focused\nsubagents for any newly suspicious mechanism.",{"type":45,"tag":657,"props":2264,"children":2265},{},[2266,2271,2273,2278],{"type":45,"tag":60,"props":2267,"children":2268},{},[2269],{"type":51,"value":2270},"Cap of 3 rounds.",{"type":51,"value":2272}," If the cap is reached and new candidates still appear, finish normally but\nstate in the verdict and in \"Coverage and Limits\" that ",{"type":45,"tag":60,"props":2274,"children":2275},{},[2276],{"type":51,"value":2277},"this review did not converge",{"type":51,"value":1062},{"type":45,"tag":642,"props":2280,"children":2281},{},[],{"type":45,"tag":646,"props":2283,"children":2285},{"id":2284},"_8-final-sweep",[2286],{"type":51,"value":2287},"8. Final sweep",{"type":45,"tag":54,"props":2289,"children":2290},{},[2291],{"type":51,"value":2292},"Before writing the documents, walk explicitly through the changed-file list and the open-candidate\nlist:",{"type":45,"tag":983,"props":2294,"children":2295},{},[2296,2301,2306,2311],{"type":45,"tag":657,"props":2297,"children":2298},{},[2299],{"type":51,"value":2300},"Was every changed file covered by at least one subagent? Cover the rest yourself.",{"type":45,"tag":657,"props":2302,"children":2303},{},[2304],{"type":51,"value":2305},"Does every suspicion have a conclusion?",{"type":45,"tag":657,"props":2307,"children":2308},{},[2309],{"type":51,"value":2310},"Does every applicable item in Part 1.3 of the code-review skill have an explicit conclusion?",{"type":45,"tag":657,"props":2312,"children":2313},{},[2314],{"type":51,"value":2315},"Is there anywhere you are still unsure about, or that may not have been investigated deeply\nenough? Investigate it now.",{"type":45,"tag":54,"props":2317,"children":2318},{},[2319],{"type":51,"value":2320},"Only after the sweep may you write the documents.",{"type":45,"tag":642,"props":2322,"children":2323},{},[],{"type":45,"tag":646,"props":2325,"children":2327},{"id":2326},"_9-produce-the-two-documents",[2328],{"type":51,"value":2329},"9. Produce the two documents",{"type":45,"tag":54,"props":2331,"children":2332},{},[2333,2335,2340,2342,2347,2349,2355],{"type":51,"value":2334},"Write them into ",{"type":45,"tag":80,"props":2336,"children":2338},{"className":2337},[],[2339],{"type":51,"value":110},{"type":51,"value":2341}," in the ",{"type":45,"tag":60,"props":2343,"children":2344},{},[2345],{"type":51,"value":2346},"current directory",{"type":51,"value":2348}," (",{"type":45,"tag":80,"props":2350,"children":2352},{"className":2351},[],[2353],{"type":51,"value":2354},"mkdir -p",{"type":51,"value":2356}," it if needed):",{"type":45,"tag":73,"props":2358,"children":2361},{"className":2359,"code":2360,"language":51},[76],"review-docs\u002Fpr-\u003CN>-review.en.md\nreview-docs\u002Fpr-\u003CN>-review.zh.md\n",[2362],{"type":45,"tag":80,"props":2363,"children":2364},{"__ignoreMap":82},[2365],{"type":51,"value":2360},{"type":45,"tag":54,"props":2367,"children":2368},{},[2369,2374,2376,2382,2384,2390,2392,2398,2399,2405],{"type":45,"tag":80,"props":2370,"children":2372},{"className":2371},[],[2373],{"type":51,"value":566},{"type":51,"value":2375}," holds the templates, the anchor format, and the verdict rule\n(",{"type":45,"tag":80,"props":2377,"children":2379},{"className":2378},[],[2380],{"type":51,"value":2381},"Blocker",{"type":51,"value":2383},"\u002F",{"type":45,"tag":80,"props":2385,"children":2387},{"className":2386},[],[2388],{"type":51,"value":2389},"Major",{"type":51,"value":2391}," → REQUEST_CHANGES; only ",{"type":45,"tag":80,"props":2393,"children":2395},{"className":2394},[],[2396],{"type":51,"value":2397},"Minor",{"type":51,"value":2383},{"type":45,"tag":80,"props":2400,"children":2402},{"className":2401},[],[2403],{"type":51,"value":2404},"Nit",{"type":51,"value":2406}," → APPROVE). The essentials:",{"type":45,"tag":983,"props":2408,"children":2409},{},[2410,2422,2454,2473,2492,2497,2502],{"type":45,"tag":657,"props":2411,"children":2412},{},[2413,2415,2420],{"type":51,"value":2414},"The two documents are ",{"type":45,"tag":60,"props":2416,"children":2417},{},[2418],{"type":51,"value":2419},"equivalent in content",{"type":51,"value":2421},": same finding IDs, same order, same anchors. The\nChinese one is a real Chinese review, not a word-for-word translation; identifiers, paths, log\nmessages, config names, code snippets, and the severity words stay in their original form.",{"type":45,"tag":657,"props":2423,"children":2424},{},[2425,2427,2432,2434,2439,2441,2446,2447,2452],{"type":51,"value":2426},"The document header must state: the PR link and state, the ",{"type":45,"tag":60,"props":2428,"children":2429},{},[2430],{"type":51,"value":2431},"head sha actually reviewed",{"type":51,"value":2433},", the\ndiff range, the review directory (",{"type":45,"tag":80,"props":2435,"children":2437},{"className":2436},[],[2438],{"type":51,"value":1256},{"type":51,"value":2440},"), and the ",{"type":45,"tag":80,"props":2442,"children":2444},{"className":2443},[],[2445],{"type":51,"value":994},{"type":51,"value":1014},{"type":45,"tag":80,"props":2448,"children":2450},{"className":2449},[],[2451],{"type":51,"value":1005},{"type":51,"value":2453}," results\nfrom step 1.",{"type":45,"tag":657,"props":2455,"children":2456},{},[2457,2459,2464,2466,2471],{"type":51,"value":2458},"Every finding must carry: a severity, a ",{"type":45,"tag":80,"props":2460,"children":2462},{"className":2461},[],[2463],{"type":51,"value":518},{"type":51,"value":2465}," anchor, a verbatim snippet, and the four\nparts ",{"type":45,"tag":60,"props":2467,"children":2468},{},[2469],{"type":51,"value":2470},"what is wrong \u002F why it happens \u002F when it bites \u002F suggested fix",{"type":51,"value":2472},". Give a diff patch when\nthe fix is small and self-contained; use prose only for architectural problems.",{"type":45,"tag":657,"props":2474,"children":2475},{},[2476,2478,2484,2486,2491],{"type":51,"value":2477},"The \"Critical Checkpoints\" table needs a conclusion per row; mark a non-applicable one ",{"type":45,"tag":80,"props":2479,"children":2481},{"className":2480},[],[2482],{"type":51,"value":2483},"n\u002Fa",{"type":51,"value":2485}," with\na half-line reason - ",{"type":45,"tag":60,"props":2487,"children":2488},{},[2489],{"type":51,"value":2490},"never delete the row",{"type":51,"value":1062},{"type":45,"tag":657,"props":2493,"children":2494},{},[2495],{"type":51,"value":2496},"\"Response to Review Focus\" answers each of the user's focus points.",{"type":45,"tag":657,"props":2498,"children":2499},{},[2500],{"type":51,"value":2501},"\"Considered and Dismissed\" lists every excluded suspicion together with its evidence.",{"type":45,"tag":657,"props":2503,"children":2504},{},[2505,2507,2512],{"type":51,"value":2506},"\"Coverage and Limits\" states: what was read in depth versus skimmed, how the subagents were\nsplit, what was not verified locally (builds, tests, anything needing a real cluster), and the\nuncommitted changes from ",{"type":45,"tag":80,"props":2508,"children":2510},{"className":2509},[],[2511],{"type":51,"value":1651},{"type":51,"value":2513}," that were excluded.",{"type":45,"tag":54,"props":2515,"children":2516},{},[2517,2519,2524],{"type":51,"value":2518},"When they are written, running the verifier is ",{"type":45,"tag":60,"props":2520,"children":2521},{},[2522],{"type":51,"value":2523},"mandatory",{"type":51,"value":2525},"; if it fails, fix the documents and\nre-run until it passes:",{"type":45,"tag":73,"props":2527,"children":2529},{"className":154,"code":2528,"language":156,"meta":82,"style":82},"python3 $S\u002Fverify-anchors.py --ctx \"$CTX\" \\\n    --doc review-docs\u002Fpr-\u003CN>-review.en.md \\\n    --doc review-docs\u002Fpr-\u003CN>-review.zh.md\n",[2530],{"type":45,"tag":80,"props":2531,"children":2532},{"__ignoreMap":82},[2533,2569,2604],{"type":45,"tag":162,"props":2534,"children":2535},{"class":164,"line":165},[2536,2540,2544,2548,2552,2556,2560,2564],{"type":45,"tag":162,"props":2537,"children":2538},{"style":302},[2539],{"type":51,"value":305},{"type":45,"tag":162,"props":2541,"children":2542},{"style":169},[2543],{"type":51,"value":310},{"type":45,"tag":162,"props":2545,"children":2546},{"style":237},[2547],{"type":51,"value":315},{"type":45,"tag":162,"props":2549,"children":2550},{"style":237},[2551],{"type":51,"value":320},{"type":45,"tag":162,"props":2553,"children":2554},{"style":175},[2555],{"type":51,"value":325},{"type":45,"tag":162,"props":2557,"children":2558},{"style":169},[2559],{"type":51,"value":234},{"type":45,"tag":162,"props":2561,"children":2562},{"style":175},[2563],{"type":51,"value":229},{"type":45,"tag":162,"props":2565,"children":2566},{"style":169},[2567],{"type":51,"value":2568}," \\\n",{"type":45,"tag":162,"props":2570,"children":2571},{"class":164,"line":202},[2572,2577,2582,2586,2591,2595,2600],{"type":45,"tag":162,"props":2573,"children":2574},{"style":237},[2575],{"type":51,"value":2576},"    --doc",{"type":45,"tag":162,"props":2578,"children":2579},{"style":237},[2580],{"type":51,"value":2581}," review-docs\u002Fpr-",{"type":45,"tag":162,"props":2583,"children":2584},{"style":175},[2585],{"type":51,"value":178},{"type":45,"tag":162,"props":2587,"children":2588},{"style":169},[2589],{"type":51,"value":2590},"N",{"type":45,"tag":162,"props":2592,"children":2593},{"style":175},[2594],{"type":51,"value":188},{"type":45,"tag":162,"props":2596,"children":2597},{"style":237},[2598],{"type":51,"value":2599},"-review.en.md",{"type":45,"tag":162,"props":2601,"children":2602},{"style":169},[2603],{"type":51,"value":2568},{"type":45,"tag":162,"props":2605,"children":2606},{"class":164,"line":252},[2607,2611,2615,2619,2623,2627],{"type":45,"tag":162,"props":2608,"children":2609},{"style":237},[2610],{"type":51,"value":2576},{"type":45,"tag":162,"props":2612,"children":2613},{"style":237},[2614],{"type":51,"value":2581},{"type":45,"tag":162,"props":2616,"children":2617},{"style":175},[2618],{"type":51,"value":178},{"type":45,"tag":162,"props":2620,"children":2621},{"style":169},[2622],{"type":51,"value":2590},{"type":45,"tag":162,"props":2624,"children":2625},{"style":175},[2626],{"type":51,"value":188},{"type":45,"tag":162,"props":2628,"children":2629},{"style":237},[2630],{"type":51,"value":2631},"-review.zh.md\n",{"type":45,"tag":54,"props":2633,"children":2634},{},[2635],{"type":51,"value":2636},"It checks that anchor paths exist, that line numbers are inside the file, that every finding has at\nleast one anchor, and that the EN and ZH finding-ID sets match; it also flags anchors pointing at\nunchanged context lines - which is usually where a miscomputed line number shows up.",{"type":45,"tag":642,"props":2638,"children":2639},{},[],{"type":45,"tag":646,"props":2641,"children":2643},{"id":2642},"_10-post-the-pass-comment-to-the-pr",[2644],{"type":51,"value":2645},"10. Post the PASS comment to the PR",{"type":45,"tag":54,"props":2647,"children":2648},{},[2649,2654,2656,2661,2663,2668,2670,2675],{"type":45,"tag":60,"props":2650,"children":2651},{},[2652],{"type":51,"value":2653},"Only when the verdict is APPROVE",{"type":51,"value":2655}," (no ",{"type":45,"tag":80,"props":2657,"children":2659},{"className":2658},[],[2660],{"type":51,"value":2381},{"type":51,"value":2662},", no ",{"type":45,"tag":80,"props":2664,"children":2666},{"className":2665},[],[2667],{"type":51,"value":2389},{"type":51,"value":2669},") and ",{"type":45,"tag":80,"props":2671,"children":2673},{"className":2672},[],[2674],{"type":51,"value":846},{"type":51,"value":2676}," has\npassed. A REQUEST_CHANGES review posts nothing at all - say so in the closing report and stop.",{"type":45,"tag":54,"props":2678,"children":2679},{},[2680],{"type":51,"value":2681},"Write the notes, dry-run, get a go, post:",{"type":45,"tag":73,"props":2683,"children":2685},{"className":154,"code":2684,"language":156,"meta":82,"style":82},"# At most 5 bullets, each anchored where it can be. Skip the file when there is nothing to say.\ncat > \"$CTX\u002Fpr-comment-notes.md\" \u003C\u003C'EOF'\n- `fe\u002Ffe-core\u002Fsrc\u002Fmain\u002Fjava\u002Forg\u002Fapache\u002Fdoris\u002FX.java:214` — \u003Cwhat the maintainer should know>\nEOF\n\n$S\u002Fpost-pass-comment.sh --ctx \"$CTX\" \\\n    --model \"\u003Cexact model id of this session>\" --effort \"${CLAUDE_EFFORT:-unknown}\" \\\n    --findings \u003Cblocker>,\u003Cmajor>,\u003Cminor>,\u003Cnit> \\\n    --rounds \u003Cr> --converged \u003Ctrue|false> \\\n    --notes-file \"$CTX\u002Fpr-comment-notes.md\" \\\n    --dry-run\n",[2686],{"type":45,"tag":80,"props":2687,"children":2688},{"__ignoreMap":82},[2689,2697,2737,2745,2753,2762,2786,2842,2938,2989,3018],{"type":45,"tag":162,"props":2690,"children":2691},{"class":164,"line":165},[2692],{"type":45,"tag":162,"props":2693,"children":2694},{"style":196},[2695],{"type":51,"value":2696},"# At most 5 bullets, each anchored where it can be. Skip the file when there is nothing to say.\n",{"type":45,"tag":162,"props":2698,"children":2699},{"class":164,"line":202},[2700,2705,2710,2714,2718,2723,2727,2732],{"type":45,"tag":162,"props":2701,"children":2702},{"style":302},[2703],{"type":51,"value":2704},"cat",{"type":45,"tag":162,"props":2706,"children":2707},{"style":175},[2708],{"type":51,"value":2709}," >",{"type":45,"tag":162,"props":2711,"children":2712},{"style":175},[2713],{"type":51,"value":325},{"type":45,"tag":162,"props":2715,"children":2716},{"style":169},[2717],{"type":51,"value":234},{"type":45,"tag":162,"props":2719,"children":2720},{"style":237},[2721],{"type":51,"value":2722},"\u002Fpr-comment-notes.md",{"type":45,"tag":162,"props":2724,"children":2725},{"style":175},[2726],{"type":51,"value":229},{"type":45,"tag":162,"props":2728,"children":2729},{"style":175},[2730],{"type":51,"value":2731}," \u003C\u003C",{"type":45,"tag":162,"props":2733,"children":2734},{"style":175},[2735],{"type":51,"value":2736},"'EOF'\n",{"type":45,"tag":162,"props":2738,"children":2739},{"class":164,"line":252},[2740],{"type":45,"tag":162,"props":2741,"children":2742},{"style":237},[2743],{"type":51,"value":2744},"- `fe\u002Ffe-core\u002Fsrc\u002Fmain\u002Fjava\u002Forg\u002Fapache\u002Fdoris\u002FX.java:214` — \u003Cwhat the maintainer should know>\n",{"type":45,"tag":162,"props":2746,"children":2747},{"class":164,"line":26},[2748],{"type":45,"tag":162,"props":2749,"children":2750},{"style":175},[2751],{"type":51,"value":2752},"EOF\n",{"type":45,"tag":162,"props":2754,"children":2755},{"class":164,"line":387},[2756],{"type":45,"tag":162,"props":2757,"children":2759},{"emptyLinePlaceholder":2758},true,[2760],{"type":51,"value":2761},"\n",{"type":45,"tag":162,"props":2763,"children":2765},{"class":164,"line":2764},6,[2766,2770,2774,2778,2782],{"type":45,"tag":162,"props":2767,"children":2768},{"style":169},[2769],{"type":51,"value":393},{"type":45,"tag":162,"props":2771,"children":2772},{"style":175},[2773],{"type":51,"value":229},{"type":45,"tag":162,"props":2775,"children":2776},{"style":169},[2777],{"type":51,"value":234},{"type":45,"tag":162,"props":2779,"children":2780},{"style":175},[2781],{"type":51,"value":229},{"type":45,"tag":162,"props":2783,"children":2784},{"style":169},[2785],{"type":51,"value":2568},{"type":45,"tag":162,"props":2787,"children":2789},{"class":164,"line":2788},7,[2790,2795,2799,2804,2808,2813,2818,2823,2828,2833,2838],{"type":45,"tag":162,"props":2791,"children":2792},{"style":302},[2793],{"type":51,"value":2794},"    --model",{"type":45,"tag":162,"props":2796,"children":2797},{"style":175},[2798],{"type":51,"value":325},{"type":45,"tag":162,"props":2800,"children":2801},{"style":237},[2802],{"type":51,"value":2803},"\u003Cexact model id of this session>",{"type":45,"tag":162,"props":2805,"children":2806},{"style":175},[2807],{"type":51,"value":229},{"type":45,"tag":162,"props":2809,"children":2810},{"style":237},[2811],{"type":51,"value":2812}," --effort",{"type":45,"tag":162,"props":2814,"children":2815},{"style":175},[2816],{"type":51,"value":2817}," \"${",{"type":45,"tag":162,"props":2819,"children":2820},{"style":169},[2821],{"type":51,"value":2822},"CLAUDE_EFFORT",{"type":45,"tag":162,"props":2824,"children":2825},{"style":175},[2826],{"type":51,"value":2827},":-",{"type":45,"tag":162,"props":2829,"children":2830},{"style":169},[2831],{"type":51,"value":2832},"unknown",{"type":45,"tag":162,"props":2834,"children":2835},{"style":175},[2836],{"type":51,"value":2837},"}\"",{"type":45,"tag":162,"props":2839,"children":2840},{"style":169},[2841],{"type":51,"value":2568},{"type":45,"tag":162,"props":2843,"children":2845},{"class":164,"line":2844},8,[2846,2851,2855,2860,2865,2869,2874,2878,2883,2887,2891,2895,2899,2904,2908,2912,2916,2920,2925,2930,2934],{"type":45,"tag":162,"props":2847,"children":2848},{"style":237},[2849],{"type":51,"value":2850},"    --findings",{"type":45,"tag":162,"props":2852,"children":2853},{"style":175},[2854],{"type":51,"value":343},{"type":45,"tag":162,"props":2856,"children":2857},{"style":237},[2858],{"type":51,"value":2859},"blocke",{"type":45,"tag":162,"props":2861,"children":2862},{"style":169},[2863],{"type":51,"value":2864},"r",{"type":45,"tag":162,"props":2866,"children":2867},{"style":175},[2868],{"type":51,"value":188},{"type":45,"tag":162,"props":2870,"children":2871},{"style":237},[2872],{"type":51,"value":2873},",",{"type":45,"tag":162,"props":2875,"children":2876},{"style":175},[2877],{"type":51,"value":178},{"type":45,"tag":162,"props":2879,"children":2880},{"style":237},[2881],{"type":51,"value":2882},"majo",{"type":45,"tag":162,"props":2884,"children":2885},{"style":169},[2886],{"type":51,"value":2864},{"type":45,"tag":162,"props":2888,"children":2889},{"style":175},[2890],{"type":51,"value":188},{"type":45,"tag":162,"props":2892,"children":2893},{"style":237},[2894],{"type":51,"value":2873},{"type":45,"tag":162,"props":2896,"children":2897},{"style":175},[2898],{"type":51,"value":178},{"type":45,"tag":162,"props":2900,"children":2901},{"style":237},[2902],{"type":51,"value":2903},"mino",{"type":45,"tag":162,"props":2905,"children":2906},{"style":169},[2907],{"type":51,"value":2864},{"type":45,"tag":162,"props":2909,"children":2910},{"style":175},[2911],{"type":51,"value":188},{"type":45,"tag":162,"props":2913,"children":2914},{"style":237},[2915],{"type":51,"value":2873},{"type":45,"tag":162,"props":2917,"children":2918},{"style":175},[2919],{"type":51,"value":178},{"type":45,"tag":162,"props":2921,"children":2922},{"style":237},[2923],{"type":51,"value":2924},"ni",{"type":45,"tag":162,"props":2926,"children":2927},{"style":169},[2928],{"type":51,"value":2929},"t",{"type":45,"tag":162,"props":2931,"children":2932},{"style":175},[2933],{"type":51,"value":188},{"type":45,"tag":162,"props":2935,"children":2936},{"style":169},[2937],{"type":51,"value":2568},{"type":45,"tag":162,"props":2939,"children":2941},{"class":164,"line":2940},9,[2942,2947,2951,2955,2959,2964,2968,2973,2978,2984],{"type":45,"tag":162,"props":2943,"children":2944},{"style":237},[2945],{"type":51,"value":2946},"    --rounds",{"type":45,"tag":162,"props":2948,"children":2949},{"style":175},[2950],{"type":51,"value":343},{"type":45,"tag":162,"props":2952,"children":2953},{"style":169},[2954],{"type":51,"value":2864},{"type":45,"tag":162,"props":2956,"children":2957},{"style":175},[2958],{"type":51,"value":188},{"type":45,"tag":162,"props":2960,"children":2961},{"style":237},[2962],{"type":51,"value":2963}," --converged",{"type":45,"tag":162,"props":2965,"children":2966},{"style":175},[2967],{"type":51,"value":343},{"type":45,"tag":162,"props":2969,"children":2970},{"style":175},[2971],{"type":51,"value":2972},"true",{"type":45,"tag":162,"props":2974,"children":2975},{"style":175},[2976],{"type":51,"value":2977},"|",{"type":45,"tag":162,"props":2979,"children":2981},{"style":2980},"--shiki-light:#6182B8;--shiki-default:#82AAFF;--shiki-dark:#82AAFF",[2982],{"type":51,"value":2983},"false",{"type":45,"tag":162,"props":2985,"children":2986},{"style":169},[2987],{"type":51,"value":2988},"> \\\n",{"type":45,"tag":162,"props":2990,"children":2992},{"class":164,"line":2991},10,[2993,2998,3002,3006,3010,3014],{"type":45,"tag":162,"props":2994,"children":2995},{"style":237},[2996],{"type":51,"value":2997},"    --notes-file",{"type":45,"tag":162,"props":2999,"children":3000},{"style":175},[3001],{"type":51,"value":325},{"type":45,"tag":162,"props":3003,"children":3004},{"style":169},[3005],{"type":51,"value":234},{"type":45,"tag":162,"props":3007,"children":3008},{"style":237},[3009],{"type":51,"value":2722},{"type":45,"tag":162,"props":3011,"children":3012},{"style":175},[3013],{"type":51,"value":229},{"type":45,"tag":162,"props":3015,"children":3016},{"style":169},[3017],{"type":51,"value":2568},{"type":45,"tag":162,"props":3019,"children":3021},{"class":164,"line":3020},11,[3022],{"type":45,"tag":162,"props":3023,"children":3024},{"style":237},[3025],{"type":51,"value":3026},"    --dry-run\n",{"type":45,"tag":983,"props":3028,"children":3029},{},[3030,3077,3109,3151,3161,3172,3192],{"type":45,"tag":657,"props":3031,"children":3032},{},[3033,3044,3045,3051,3053,3059,3061,3067,3069,3075],{"type":45,"tag":60,"props":3034,"children":3035},{},[3036,3042],{"type":45,"tag":80,"props":3037,"children":3039},{"className":3038},[],[3040],{"type":51,"value":3041},"--model",{"type":51,"value":3043}," is the exact model id of the session doing the review",{"type":51,"value":2348},{"type":45,"tag":80,"props":3046,"children":3048},{"className":3047},[],[3049],{"type":51,"value":3050},"claude-opus-5[1m]",{"type":51,"value":3052},",\n",{"type":45,"tag":80,"props":3054,"children":3056},{"className":3055},[],[3057],{"type":51,"value":3058},"gpt-5.6-sol",{"type":51,"value":3060},", …), taken from what this session was told about itself - never a guess, never a\nbare family name. ",{"type":45,"tag":80,"props":3062,"children":3064},{"className":3063},[],[3065],{"type":51,"value":3066},"--effort",{"type":51,"value":3068}," comes from ",{"type":45,"tag":80,"props":3070,"children":3072},{"className":3071},[],[3073],{"type":51,"value":3074},"$CLAUDE_EFFORT",{"type":51,"value":3076},". The comment is a public, signed\nstatement about who reviewed the code; both fields are what make it auditable.",{"type":45,"tag":657,"props":3078,"children":3079},{},[3080,3082,3087,3089,3094,3096,3107],{"type":51,"value":3081},"The dry run runs ",{"type":45,"tag":60,"props":3083,"children":3084},{},[3085],{"type":51,"value":3086},"every",{"type":51,"value":3088}," precondition and prints the exact body. ",{"type":45,"tag":60,"props":3090,"children":3091},{},[3092],{"type":51,"value":3093},"Show that body to the user\nand wait for a go",{"type":51,"value":3095},", then re-run the identical command ",{"type":45,"tag":60,"props":3097,"children":3098},{},[3099,3101],{"type":51,"value":3100},"without ",{"type":45,"tag":80,"props":3102,"children":3104},{"className":3103},[],[3105],{"type":51,"value":3106},"--dry-run",{"type":51,"value":3108},". Never post\nwithout that confirmation.",{"type":45,"tag":657,"props":3110,"children":3111},{},[3112,3114,3119,3120,3125,3127,3133,3135,3141,3143],{"type":51,"value":3113},"The script refuses to post when: any ",{"type":45,"tag":80,"props":3115,"children":3117},{"className":3116},[],[3118],{"type":51,"value":2381},{"type":51,"value":2383},{"type":45,"tag":80,"props":3121,"children":3123},{"className":3122},[],[3124],{"type":51,"value":2389},{"type":51,"value":3126}," is present; the live PR head no longer\nequals the reviewed commit; the PR is not open (",{"type":45,"tag":80,"props":3128,"children":3130},{"className":3129},[],[3131],{"type":51,"value":3132},"--allow-closed",{"type":51,"value":3134}," overrides); the notes are\nmalformed or more than five; ",{"type":45,"tag":80,"props":3136,"children":3138},{"className":3137},[],[3139],{"type":51,"value":3140},"converged: false",{"type":51,"value":3142}," came without a note. A refusal is a real signal\n",{"type":45,"tag":983,"props":3144,"children":3145},{},[3146],{"type":45,"tag":657,"props":3147,"children":3148},{},[3149],{"type":51,"value":3150},"relay it, do not work around it.",{"type":45,"tag":657,"props":3152,"children":3153},{},[3154,3159],{"type":45,"tag":60,"props":3155,"children":3156},{},[3157],{"type":51,"value":3158},"The PR head moved",{"type":51,"value":3160}," means the author pushed during the review: the review is stale, so re-run\nthe whole skill instead of posting.",{"type":45,"tag":657,"props":3162,"children":3163},{},[3164,3166,3170],{"type":51,"value":3165},"An earlier v1 comment from the same account for the ",{"type":45,"tag":60,"props":3167,"children":3168},{},[3169],{"type":51,"value":1012},{"type":51,"value":3171}," commit is edited in place; a new\ncommit gets a new comment, so each push leaves exactly one record.",{"type":45,"tag":657,"props":3173,"children":3174},{},[3175,3177,3183,3185,3190],{"type":51,"value":3176},"Findings counts, rounds and ",{"type":45,"tag":80,"props":3178,"children":3180},{"className":3179},[],[3181],{"type":51,"value":3182},"converged",{"type":51,"value":3184}," must match the documents written in step 9. The counts\nare of ",{"type":45,"tag":3186,"props":3187,"children":3188},"em",{},[3189],{"type":51,"value":2146},{"type":51,"value":3191}," findings, not of candidates.",{"type":45,"tag":657,"props":3193,"children":3194},{},[3195,3200,3202,3207],{"type":45,"tag":80,"props":3196,"children":3198},{"className":3197},[],[3199],{"type":51,"value":583},{"type":51,"value":3201}," holds the schema, the field meanings, and the parser snippet.\n",{"type":45,"tag":60,"props":3203,"children":3204},{},[3205],{"type":51,"value":3206},"Never hand-write or hand-edit this comment",{"type":51,"value":3208}," - the format is a contract other programs read.",{"type":45,"tag":642,"props":3210,"children":3211},{},[],{"type":45,"tag":646,"props":3213,"children":3215},{"id":3214},"_11-closing-report",[3216],{"type":51,"value":3217},"11. Closing report",{"type":45,"tag":54,"props":3219,"children":3220},{},[3221],{"type":51,"value":3222},"Tell the user:",{"type":45,"tag":653,"props":3224,"children":3225},{},[3226,3231,3255,3265,3297],{"type":45,"tag":657,"props":3227,"children":3228},{},[3229],{"type":51,"value":3230},"The paths of both documents, the verdict (REQUEST_CHANGES \u002F APPROVE), the finding count per\nseverity, and whether the rounds converged.",{"type":45,"tag":657,"props":3232,"children":3233},{},[3234,3235,3240,3241,3246,3248,3253],{"type":51,"value":588},{"type":45,"tag":80,"props":3236,"children":3238},{"className":3237},[],[3239],{"type":51,"value":994},{"type":51,"value":1014},{"type":45,"tag":80,"props":3242,"children":3244},{"className":3243},[],[3245],{"type":51,"value":1005},{"type":51,"value":3247}," results - especially ",{"type":45,"tag":80,"props":3249,"children":3251},{"className":3250},[],[3252],{"type":51,"value":1020},{"type":51,"value":3254}," (unpushed commits that were\nnot reviewed).",{"type":45,"tag":657,"props":3256,"children":3257},{},[3258,3263],{"type":45,"tag":60,"props":3259,"children":3260},{},[3261],{"type":51,"value":3262},"What happened to the PASS comment",{"type":51,"value":3264},": the URL when one was posted or updated, or the reason\nnothing was posted (the verdict was REQUEST_CHANGES, the PR head moved, the user said no).",{"type":45,"tag":657,"props":3266,"children":3267},{},[3268,3273,3275,3280,3282,3288,3290,3295],{"type":45,"tag":60,"props":3269,"children":3270},{},[3271],{"type":51,"value":3272},"Where the current directory now stands",{"type":51,"value":3274},": with ",{"type":45,"tag":80,"props":3276,"children":3278},{"className":3277},[],[3279],{"type":51,"value":1161},{"type":51,"value":3281}," it is detached on the PR\nhead, and ",{"type":45,"tag":80,"props":3283,"children":3285},{"className":3284},[],[3286],{"type":51,"value":3287},"git checkout \u003CPREV_REF>",{"type":51,"value":3289}," restores it. ",{"type":45,"tag":60,"props":3291,"children":3292},{},[3293],{"type":51,"value":3294},"Do not switch back automatically",{"type":51,"value":3296}," - the user\nmay still want to read the code.",{"type":45,"tag":657,"props":3298,"children":3299},{},[3300,3305,3307,3312,3314,3319],{"type":45,"tag":80,"props":3301,"children":3303},{"className":3302},[],[3304],{"type":51,"value":110},{"type":51,"value":3306}," ",{"type":45,"tag":60,"props":3308,"children":3309},{},[3310],{"type":51,"value":3311},"is not gitignored in the doris repository",{"type":51,"value":3313},", so ",{"type":45,"tag":60,"props":3315,"children":3316},{},[3317],{"type":51,"value":3318},"do not commit it\nautomatically",{"type":51,"value":3320},"; leave that to the user.",{"type":45,"tag":642,"props":3322,"children":3323},{},[],{"type":45,"tag":646,"props":3325,"children":3327},{"id":3326},"_12-mapping-to-the-ci-pipeline",[3328],{"type":51,"value":3329},"12. Mapping to the CI pipeline",{"type":45,"tag":435,"props":3331,"children":3332},{},[3333,3349],{"type":45,"tag":439,"props":3334,"children":3335},{},[3336],{"type":45,"tag":443,"props":3337,"children":3338},{},[3339,3344],{"type":45,"tag":447,"props":3340,"children":3341},{},[3342],{"type":51,"value":3343},"CI (code-review-runner.yml)",{"type":45,"tag":447,"props":3345,"children":3346},{},[3347],{"type":51,"value":3348},"Local",{"type":45,"tag":458,"props":3350,"children":3351},{},[3352,3371,3397,3422,3435,3454,3467,3493,3513,3526,3562,3582],{"type":45,"tag":443,"props":3353,"children":3354},{},[3355,3366],{"type":45,"tag":465,"props":3356,"children":3357},{},[3358,3364],{"type":45,"tag":80,"props":3359,"children":3361},{"className":3360},[],[3362],{"type":51,"value":3363},"codex exec --goal",{"type":51,"value":3365}," (gpt-5.6-sol, xhigh)",{"type":45,"tag":465,"props":3367,"children":3368},{},[3369],{"type":51,"value":3370},"The main agent of this session",{"type":45,"tag":443,"props":3372,"children":3373},{},[3374,3379],{"type":45,"tag":465,"props":3375,"children":3376},{},[3377],{"type":51,"value":3378},"checkout the PR head sha",{"type":45,"tag":465,"props":3380,"children":3381},{},[3382,3388,3390,3395],{"type":45,"tag":80,"props":3383,"children":3385},{"className":3384},[],[3386],{"type":51,"value":3387},"align-to-pr.sh",{"type":51,"value":3389}," detaches ",{"type":45,"tag":60,"props":3391,"children":3392},{},[3393],{"type":51,"value":3394},"the current directory",{"type":51,"value":3396}," to the same sha",{"type":45,"tag":443,"props":3398,"children":3399},{},[3400,3411],{"type":45,"tag":465,"props":3401,"children":3402},{},[3403,3409],{"type":45,"tag":80,"props":3404,"children":3406},{"className":3405},[],[3407],{"type":51,"value":3408},"git diff BASE...HEAD",{"type":51,"value":3410}," as the authoritative diff",{"type":45,"tag":465,"props":3412,"children":3413},{},[3414,3416],{"type":51,"value":3415},"Same, produced by ",{"type":45,"tag":80,"props":3417,"children":3419},{"className":3418},[],[3420],{"type":51,"value":3421},"prepare-review-context.sh",{"type":45,"tag":443,"props":3423,"children":3424},{},[3425,3430],{"type":45,"tag":465,"props":3426,"children":3427},{},[3428],{"type":51,"value":3429},"\"PR changed while preparing\" guard",{"type":45,"tag":465,"props":3431,"children":3432},{},[3433],{"type":51,"value":3434},"The fetched sha must equal the API head sha, otherwise a re-run is required",{"type":45,"tag":443,"props":3436,"children":3437},{},[3438,3449],{"type":45,"tag":465,"props":3439,"children":3440},{},[3441,3447],{"type":45,"tag":80,"props":3442,"children":3444},{"className":3443},[],[3445],{"type":51,"value":3446},"prepare_review_agents.py",{"type":51,"value":3448}," collecting AGENTS.md",{"type":45,"tag":465,"props":3450,"children":3451},{},[3452],{"type":51,"value":3453},"Same ancestor-directory algorithm, built into the script",{"type":45,"tag":443,"props":3455,"children":3456},{},[3457,3462],{"type":45,"tag":465,"props":3458,"children":3459},{},[3460],{"type":51,"value":3461},"Fetch existing inline threads (30 threads \u002F 1200 chars)",{"type":45,"tag":465,"props":3463,"children":3464},{},[3465],{"type":51,"value":3466},"Same jq",{"type":45,"tag":443,"props":3468,"children":3469},{},[3470,3483],{"type":45,"tag":465,"props":3471,"children":3472},{},[3473,3475,3481],{"type":51,"value":3474},"Text after ",{"type":45,"tag":80,"props":3476,"children":3478},{"className":3477},[],[3479],{"type":51,"value":3480},"\u002Freview",{"type":51,"value":3482}," = review focus",{"type":45,"tag":465,"props":3484,"children":3485},{},[3486,3488],{"type":51,"value":3487},"Free text after the PR URL → ",{"type":45,"tag":80,"props":3489,"children":3491},{"className":3490},[],[3492],{"type":51,"value":1434},{"type":45,"tag":443,"props":3494,"children":3495},{},[3496,3501],{"type":45,"tag":465,"props":3497,"children":3498},{},[3499],{"type":51,"value":3500},"Single-file ledger with sections",{"type":45,"tag":465,"props":3502,"children":3503},{},[3504,3506,3511],{"type":51,"value":3505},"A ",{"type":45,"tag":80,"props":3507,"children":3509},{"className":3508},[],[3510],{"type":51,"value":1675},{"type":51,"value":3512}," directory, one file per owner",{"type":45,"tag":443,"props":3514,"children":3515},{},[3516,3521],{"type":45,"tag":465,"props":3517,"children":3518},{},[3519],{"type":51,"value":3520},"Main risk scan → 1-3 full-review subagents + risk-focused → merge → ≤3 rounds",{"type":45,"tag":465,"props":3522,"children":3523},{},[3524],{"type":51,"value":3525},"Identical",{"type":45,"tag":443,"props":3527,"children":3528},{},[3529,3540],{"type":45,"tag":465,"props":3530,"children":3531},{},[3532,3538],{"type":45,"tag":80,"props":3533,"children":3535},{"className":3534},[],[3536],{"type":51,"value":3537},"gh pr review",{"type":51,"value":3539}," \u002F Reviews API posting inline comments",{"type":45,"tag":465,"props":3541,"children":3542},{},[3543],{"type":45,"tag":60,"props":3544,"children":3545},{},[3546,3548,3553,3555,3560],{"type":51,"value":3547},"Two ",{"type":45,"tag":80,"props":3549,"children":3551},{"className":3550},[],[3552],{"type":51,"value":110},{"type":51,"value":3554}," documents (EN + ZH) with ",{"type":45,"tag":80,"props":3556,"children":3558},{"className":3557},[],[3559],{"type":51,"value":518},{"type":51,"value":3561}," anchors",{"type":45,"tag":443,"props":3563,"children":3564},{},[3565,3570],{"type":45,"tag":465,"props":3566,"children":3567},{},[3568],{"type":51,"value":3569},"CI's review verdict is visible on the PR itself",{"type":45,"tag":465,"props":3571,"children":3572},{},[3573,3575,3580],{"type":51,"value":3574},"On a pass, one ",{"type":45,"tag":80,"props":3576,"children":3578},{"className":3577},[],[3579],{"type":51,"value":594},{"type":51,"value":3581}," comment from the local account (commit sha, timestamp, model, findings, notes); on REQUEST_CHANGES, nothing - the documents stay local",{"type":45,"tag":443,"props":3583,"children":3584},{},[3585,3590],{"type":45,"tag":465,"props":3586,"children":3587},{},[3588],{"type":51,"value":3589},"60-minute timeout",{"type":45,"tag":465,"props":3591,"children":3592},{},[3593],{"type":51,"value":3594},"No hard timeout, but likewise do not let one round turn into unbounded digging",{"type":45,"tag":642,"props":3596,"children":3597},{},[],{"type":45,"tag":646,"props":3599,"children":3601},{"id":3600},"_13-common-traps",[3602],{"type":51,"value":3603},"13. Common traps",{"type":45,"tag":983,"props":3605,"children":3606},{},[3607,3630,3660,3692,3715,3725,3742,3752,3768,3778,3796],{"type":45,"tag":657,"props":3608,"children":3609},{},[3610,3615,3616,3622,3624,3628],{"type":45,"tag":60,"props":3611,"children":3612},{},[3613],{"type":51,"value":3614},"The local checkout is ahead of the PR head.",{"type":51,"value":3306},{"type":45,"tag":80,"props":3617,"children":3619},{"className":3618},[],[3620],{"type":51,"value":3621},"commit check: ahead:N",{"type":51,"value":3623}," means there are unpushed\ncommits and ",{"type":45,"tag":60,"props":3625,"children":3626},{},[3627],{"type":51,"value":1060},{"type":51,"value":3629},". This is the easiest thing for a reader to\nmisread, so state it both in the document header and in the closing report.",{"type":45,"tag":657,"props":3631,"children":3632},{},[3633,3638,3639,3644,3646,3651,3653,3659],{"type":45,"tag":60,"props":3634,"children":3635},{},[3636],{"type":51,"value":3637},"Anchoring to the wrong side.",{"type":51,"value":3306},{"type":45,"tag":80,"props":3640,"children":3642},{"className":3641},[],[3643],{"type":51,"value":1540},{"type":51,"value":3645}," gives ",{"type":45,"tag":60,"props":3647,"children":3648},{},[3649],{"type":51,"value":3650},"post-change",{"type":51,"value":3652}," line numbers,\nwhich match what a file read shows in the current directory. A pure deletion has no line on the\nnew side - anchor the seam line and note ",{"type":45,"tag":80,"props":3654,"children":3656},{"className":3655},[],[3657],{"type":51,"value":3658},"(deleted, base line N)",{"type":51,"value":1062},{"type":45,"tag":657,"props":3661,"children":3662},{},[3663,3668,3669,3675,3676,3682,3684,3690],{"type":45,"tag":60,"props":3664,"children":3665},{},[3666],{"type":51,"value":3667},"Switching invalidates the current directory's incremental build.",{"type":51,"value":3306},{"type":45,"tag":80,"props":3670,"children":3672},{"className":3671},[],[3673],{"type":51,"value":3674},"output\u002F",{"type":51,"value":787},{"type":45,"tag":80,"props":3677,"children":3679},{"className":3678},[],[3680],{"type":51,"value":3681},"be\u002Fbuild_*",{"type":51,"value":3683},", and\n",{"type":45,"tag":80,"props":3685,"children":3687},{"className":3686},[],[3688],{"type":51,"value":3689},"*\u002Ftarget\u002F",{"type":51,"value":3691}," are all built per commit, so switching to the PR head and back usually forces a\nrebuild. Warn the user before switching.",{"type":45,"tag":657,"props":3693,"children":3694},{},[3695,3700,3701,3706,3708,3713],{"type":45,"tag":60,"props":3696,"children":3697},{},[3698],{"type":51,"value":3699},"Uncommitted changes are not in the diff.",{"type":51,"value":3306},{"type":45,"tag":80,"props":3702,"children":3704},{"className":3703},[],[3705],{"type":51,"value":3408},{"type":51,"value":3707}," only sees committed content.\nAlignment already guarantees no dirty tracked files at switch time, but the user may have edited\nsomething afterwards; anything in ",{"type":45,"tag":80,"props":3709,"children":3711},{"className":3710},[],[3712],{"type":51,"value":1651},{"type":51,"value":3714}," must be named in the documents' \"Coverage\nand Limits\".",{"type":45,"tag":657,"props":3716,"children":3717},{},[3718,3723],{"type":45,"tag":60,"props":3719,"children":3720},{},[3721],{"type":51,"value":3722},"Subagents racing on the ledger.",{"type":51,"value":3724}," One file per owner exists precisely for this; always hard-code\neach subagent's own file name in its prompt.",{"type":45,"tag":657,"props":3726,"children":3727},{},[3728,3733,3735,3740],{"type":45,"tag":60,"props":3729,"children":3730},{},[3731],{"type":51,"value":3732},"Re-raising an issue that already has a comment.",{"type":51,"value":3734}," CI forbids it, and so does a local run - read\n",{"type":45,"tag":80,"props":3736,"children":3738},{"className":3737},[],[3739],{"type":51,"value":1593},{"type":51,"value":3741}," first.",{"type":45,"tag":657,"props":3743,"children":3744},{},[3745,3750],{"type":45,"tag":60,"props":3746,"children":3747},{},[3748],{"type":51,"value":3749},"A subagent's conclusion cannot be taken at face value.",{"type":51,"value":3751}," The main agent must read the code back\nto confirm it; any \"if A then B\" that cannot name a concrete scenario for A is downgraded or\ndropped.",{"type":45,"tag":657,"props":3753,"children":3754},{},[3755,3766],{"type":45,"tag":60,"props":3756,"children":3757},{},[3758,3760,3765],{"type":51,"value":3759},"Do not commit ",{"type":45,"tag":80,"props":3761,"children":3763},{"className":3762},[],[3764],{"type":51,"value":110},{"type":51,"value":1062},{"type":51,"value":3767}," The doris repository does not ignore it, and an automatic commit\nwould slip it into the PR.",{"type":45,"tag":657,"props":3769,"children":3770},{},[3771,3776],{"type":45,"tag":60,"props":3772,"children":3773},{},[3774],{"type":51,"value":3775},"The PASS comment is public and signed with the user's name.",{"type":51,"value":3777}," It goes to a public Apache PR\nfrom their GitHub account, so it is posted only after they have seen the exact body. Treat a\nscript refusal as final rather than something to route around, and never \"tidy up\" the rendered\nbody by hand - a program reads it.",{"type":45,"tag":657,"props":3779,"children":3780},{},[3781,3786,3788,3794],{"type":45,"tag":60,"props":3782,"children":3783},{},[3784],{"type":51,"value":3785},"A pass is not a merge approval.",{"type":51,"value":3787}," The comment states that a local pipeline review found no\nBlocker and no Major on one specific commit. It carries no CI signal and no Apache sign-off, and\nthe ",{"type":45,"tag":80,"props":3789,"children":3791},{"className":3790},[],[3792],{"type":51,"value":3793},"\u003Csub>",{"type":51,"value":3795}," disclaimer line says exactly that - keep it.",{"type":45,"tag":657,"props":3797,"children":3798},{},[3799,3804,3805,3811],{"type":45,"tag":60,"props":3800,"children":3801},{},[3802],{"type":51,"value":3803},"Counts drift between the documents and the comment.",{"type":51,"value":3306},{"type":45,"tag":80,"props":3806,"children":3808},{"className":3807},[],[3809],{"type":51,"value":3810},"--findings",{"type":51,"value":3812}," must be the accepted\nfindings of step 9, not the candidate count from the ledger; re-count from the written documents\nbefore posting.",{"type":45,"tag":3814,"props":3815,"children":3816},"style",{},[3817],{"type":51,"value":3818},"html .light .shiki span {color: var(--shiki-light);background: var(--shiki-light-bg);font-style: var(--shiki-light-font-style);font-weight: var(--shiki-light-font-weight);text-decoration: var(--shiki-light-text-decoration);}html.light .shiki span {color: var(--shiki-light);background: var(--shiki-light-bg);font-style: var(--shiki-light-font-style);font-weight: var(--shiki-light-font-weight);text-decoration: var(--shiki-light-text-decoration);}html .default .shiki span {color: var(--shiki-default);background: var(--shiki-default-bg);font-style: var(--shiki-default-font-style);font-weight: var(--shiki-default-font-weight);text-decoration: var(--shiki-default-text-decoration);}html .shiki span {color: var(--shiki-default);background: var(--shiki-default-bg);font-style: var(--shiki-default-font-style);font-weight: var(--shiki-default-font-weight);text-decoration: var(--shiki-default-text-decoration);}html .dark .shiki span {color: var(--shiki-dark);background: var(--shiki-dark-bg);font-style: var(--shiki-dark-font-style);font-weight: var(--shiki-dark-font-weight);text-decoration: var(--shiki-dark-text-decoration);}html.dark .shiki span {color: var(--shiki-dark);background: var(--shiki-dark-bg);font-style: var(--shiki-dark-font-style);font-weight: var(--shiki-dark-font-weight);text-decoration: var(--shiki-dark-text-decoration);}",{"items":3820,"total":3911},[3821,3839,3851,3867,3879,3889,3899],{"slug":3822,"name":3822,"fn":3823,"description":3824,"org":3825,"tags":3826,"stars":22,"repoUrl":23,"updatedAt":3838},"doris-architecture-advisor","design Apache Doris data architectures","Workload-aware architecture design for Apache Doris. MUST USE when designing data architectures, choosing between data models, planning ingestion strategies, sizing clusters, or translating business requirements into Apache Doris system designs. Complements doris-best-practices with decision frameworks and sizing-first workflow. Use when user describes a workload involving: IoT, sensor data, telemetry, real-time analytics, dashboard, log analysis, log search, CDC sync, time-series, device monitoring, point query service, ad-hoc analytics, lakehouse federation, ETL\u002FELT pipeline, report analytics, clickstream, user behavior, observability, metrics, fleet tracking, or any OLAP workload requiring table design from scratch. Also triggers on prompts like: \"design a table for...\", \"how should I store...\", \"build an architecture for...\", \"we have X devices sending data every Y seconds\", \"recommend a cluster size for...\", \"what data model should I use for...\", \"we need to ingest X GB\u002Fday\", \"migrate from MySQL\u002FPostgreSQL to Apache Doris\". Also use for legacy analytics\u002Fsearch\u002Fserving stack consolidation prompts even when Apache Doris is not named explicitly, including replacing or migrating from Impala, Kudu, Elasticsearch\u002FES, Greenplum, Presto, HBase, Hive, Hadoop, Redis, or Lambda-style multi-engine data platforms.\n",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3827,3830,3833,3836],{"name":3828,"slug":3829,"type":15},"Architecture","architecture",{"name":3831,"slug":3832,"type":15},"Data Engineering","data-engineering",{"name":3834,"slug":3835,"type":15},"Data Pipeline","data-pipeline",{"name":3837,"slug":30,"type":15},"Database","2026-07-12T08:35:33.88758",{"slug":3840,"name":3840,"fn":3841,"description":3842,"org":3843,"tags":3844,"stars":22,"repoUrl":23,"updatedAt":3850},"doris-best-practices","optimize Apache Doris table design and clusters","Apache Doris table design and cluster sizing best practices. MUST USE when writing, reviewing, or optimizing Doris CREATE TABLE statements, partition\u002Fbucket strategies, data models, or cluster configurations. ALSO MUST USE whenever the doris-architecture-advisor skill produces DDL — apply the Pre-Flight Checklist to every CREATE TABLE before output. Also triggers on any workload design involving: IoT, analytics, dashboard, CDC, time-series, log analysis, real-time warehouse, point query, data platform, or any scenario where table design decisions are being made. Also triggers on replacing or migrating from legacy analytics\u002Fsearch\u002Fserving stacks such as Impala, Kudu, Elasticsearch\u002FES, Greenplum, Presto, HBase, Hive, Hadoop, Redis, or Lambda-style multi-engine data platforms, even when Apache Doris is not named explicitly. Also use when user provides an Apache Doris connection string or asks to get started. Also triggers on slow query investigation, query profiling, runtime performance diagnosis, tablet skew analysis, and table health checks — any scenario where runtime evidence (profile output, tablet distribution) informs optimization. Cluster lifecycle, billing, and networking are managed-service operations, out of scope here — use your platform's cluster-management console for those.\n",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3845,3846,3847],{"name":3837,"slug":30,"type":15},{"name":20,"slug":21,"type":15},{"name":3848,"slug":3849,"type":15},"Performance","performance","2026-07-12T08:35:32.619438",{"slug":3852,"name":3852,"fn":3853,"description":3854,"org":3855,"tags":3856,"stars":22,"repoUrl":23,"updatedAt":3866},"doris-debug","diagnose Apache Doris production issues","Apache Doris production diagnostics router. Use when Apache Doris queries are slow, imports are failing or timing out, compaction is raising -235 errors, nodes are OOM or crashing, materialized views are not rewriting, or tablet\u002Freplica health is degraded. Routes to the appropriate doris-debug-* skill. Covers shared-nothing and cloud (storage-compute separation) deployments.\n",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3857,3860,3863],{"name":3858,"slug":3859,"type":15},"Debugging","debugging",{"name":3861,"slug":3862,"type":15},"Monitoring","monitoring",{"name":3864,"slug":3865,"type":15},"Operations","operations","2026-07-25T05:56:31.330361",{"slug":3868,"name":3868,"fn":3869,"description":3870,"org":3871,"tags":3872,"stars":22,"repoUrl":23,"updatedAt":3878},"doris-debug-cloud","debug Apache Doris cloud mode issues","Use for Doris storage-compute separation (cloud mode) issues: meta-service latency, cache miss storms, object store throughput, and shared-nothing config conflicts in compute groups.\n",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3873,3876,3877],{"name":3874,"slug":3875,"type":15},"Cloud","cloud",{"name":3858,"slug":3859,"type":15},{"name":3848,"slug":3849,"type":15},"2026-07-25T05:56:24.239866",{"slug":3880,"name":3880,"fn":3881,"description":3882,"org":3883,"tags":3884,"stars":22,"repoUrl":23,"updatedAt":3888},"doris-debug-compaction","resolve Apache Doris compaction lag","Use for Doris -235 \u002F too many versions and compaction lag. Error raised in RowsetBuilder::check_tablet_version_count when version_count > max_tablet_version_num (default 2000) or meta serialize size limit.\n",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3885,3886,3887],{"name":3837,"slug":30,"type":15},{"name":3858,"slug":3859,"type":15},{"name":3848,"slug":3849,"type":15},"2026-07-25T05:56:31.974921",{"slug":3890,"name":3890,"fn":3891,"description":3892,"org":3893,"tags":3894,"stars":22,"repoUrl":23,"updatedAt":3898},"doris-debug-data-lake","debug Apache Doris data lake connectivity","Use for Doris external catalog issues: Hive\u002FIceberg\u002FPaimon\u002FHudi query failures, metadata refresh, filesystem S3\u002FHDFS connectivity, external MV rewrite misses.\n",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3895,3896,3897],{"name":3831,"slug":3832,"type":15},{"name":3837,"slug":30,"type":15},{"name":3858,"slug":3859,"type":15},"2026-07-25T05:56:23.920304",{"slug":3900,"name":3900,"fn":3901,"description":3902,"org":3903,"tags":3904,"stars":22,"repoUrl":23,"updatedAt":3910},"doris-debug-deployment","troubleshoot Apache Doris deployment failures","Use for Doris FE\u002FBE startup failures, port conflicts, priority_networks misrouting, meta_dir corruption, and ADD\u002FDROP BACKEND issues.\n",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3905,3906,3909],{"name":3858,"slug":3859,"type":15},{"name":3907,"slug":3908,"type":15},"Deployment","deployment",{"name":3864,"slug":3865,"type":15},"2026-07-25T05:56:21.494401",15,{"items":3913,"total":4073},[3914,3932,3950,3964,3978,3989,4000,4013,4031,4042,4052,4063],{"slug":3915,"name":3916,"fn":3917,"description":3918,"org":3919,"tags":3920,"stars":3929,"repoUrl":3930,"updatedAt":3931},"computer-use","Computer Use","operate local desktop applications","Use when the user asks to inspect or operate a local desktop application UI, including reading windows, clicking controls, filling forms, using menus, scrolling lists, moving windows, or waiting for dialogs. Trigger for requests such as \"operate this app\", \"do this in TextEdit\u002FCalculator\u002FSettings\", \"look at the current window\", or \"click\u002Ftype\u002Fscroll\"; prefer Browser tools for web pages and non-GUI tools for files or terminal work.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3921,3924,3927],{"name":3922,"slug":3923,"type":15},"Automation","automation",{"name":3925,"slug":3926,"type":15},"Desktop","desktop",{"name":3928,"slug":32,"type":15},"MCP",1580,"https:\u002F\u002Fgithub.com\u002Fapache\u002Fmaka","2026-08-20T03:52:55.588112",{"slug":3933,"name":3933,"fn":3934,"description":3935,"org":3936,"tags":3937,"stars":3947,"repoUrl":3948,"updatedAt":3949},"datafusion-python","write Apache DataFusion Python code","Use when the user is writing datafusion-python (Apache DataFusion Python bindings) DataFrame or SQL code. Covers imports, data loading, DataFrame operations, expression building, SQL-to-DataFrame mappings, idiomatic patterns, and common pitfalls.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3938,3941,3944],{"name":3939,"slug":3940,"type":15},"Data Analysis","data-analysis",{"name":3942,"slug":3943,"type":15},"Python","python",{"name":3945,"slug":3946,"type":15},"SQL","sql",593,"https:\u002F\u002Fgithub.com\u002Fapache\u002Fdatafusion-python","2026-07-12T08:36:04.957626",{"slug":3951,"name":3951,"fn":3952,"description":3953,"org":3954,"tags":3955,"stars":3961,"repoUrl":3962,"updatedAt":3963},"flink-agents-dev","build and debug Apache Flink agents","Use when building, scaffolding, modifying, debugging, converting, or verifying Apache Flink Agents applications, including Flink Agents YAML, Workflow Agent, ReAct Agent, Actions, Resources, MCP servers, vector stores, runtime skills, Python, or Java. Do not use for ordinary Flink jobs that do not use Flink Agents.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3956,3959,3960],{"name":3957,"slug":3958,"type":15},"Agents","agents",{"name":3922,"slug":3923,"type":15},{"name":3928,"slug":32,"type":15},421,"https:\u002F\u002Fgithub.com\u002Fapache\u002Fflink-agents","2026-08-07T05:03:37.315408",{"slug":3965,"name":3965,"fn":3966,"description":3967,"org":3968,"tags":3969,"stars":3975,"repoUrl":3976,"updatedAt":3977},"bydbql","generate and execute BanyanDB BydbQL queries","Generate, validate, and optionally execute read-only BanyanDB BydbQL for STREAM, MEASURE, TRACE, and PROPERTY resources. Use when the user asks to query BanyanDB, translate natural language to BydbQL, inspect BanyanDB schema or data, validate BydbQL, or fetch raw BanyanDB records.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3970,3973,3974],{"name":3971,"slug":3972,"type":15},"Analytics","analytics",{"name":3837,"slug":30,"type":15},{"name":3945,"slug":3946,"type":15},344,"https:\u002F\u002Fgithub.com\u002Fapache\u002Fskywalking-banyandb","2026-07-12T08:31:01.294423",{"slug":3979,"name":3979,"fn":3980,"description":3981,"org":3982,"tags":3983,"stars":3975,"repoUrl":3976,"updatedAt":3988},"compiling","compile and build BanyanDB projects","Compile and build the SkyWalking BanyanDB project. Use when the user asks to compile, build, or generate code for this project.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3984,3987],{"name":3985,"slug":3986,"type":15},"Build","build",{"name":20,"slug":21,"type":15},"2026-07-12T08:31:06.373309",{"slug":3990,"name":3990,"fn":3991,"description":3992,"org":3993,"tags":3994,"stars":3975,"repoUrl":3976,"updatedAt":3999},"gh-pull-request","create GitHub pull requests for BanyanDB","Create a GitHub pull request for SkyWalking BanyanDB. Use when the user asks to create a PR, submit changes, or open a pull request.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[3995,3996],{"name":13,"slug":14,"type":15},{"name":3997,"slug":3998,"type":15},"Pull Requests","pull-requests","2026-07-12T08:31:03.792415",{"slug":4001,"name":4001,"fn":4002,"description":4003,"org":4004,"tags":4005,"stars":3975,"repoUrl":3976,"updatedAt":4012},"vendor-update","update Go and Node.js vendor dependencies","Upgrade Go\u002FNode.js vendor dependencies and sync tool versions. Use whenever the user says \"upgrade dependencies\", \"update vendors\", \"vendor update\", \"run vendor-upgrade\", \"bump dependencies\", \"update packages\", or asks to run the `vendor-update` Make target. This skill also checks `scripts\u002Fbuild\u002Fversion.mk` after upgrading to see if any tracked tool versions need updating too, and removes stale binaries from `bin\u002F` when versions change.\n",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[4006,4009],{"name":4007,"slug":4008,"type":15},"Go","go",{"name":4010,"slug":4011,"type":15},"Node.js","node-js","2026-07-12T08:31:02.555555",{"slug":4014,"name":4014,"fn":4015,"description":4016,"org":4017,"tags":4018,"stars":4028,"repoUrl":4029,"updatedAt":4030},"cayenne-cgen","generate Cayenne entity Java classes","Use this skill whenever the user wants to (re)generate Cayenne entity Java classes from a DataMap. Trigger on phrases like 'generate Java classes', 'regenerate entities', 'run cgen', 'create the entity classes', 'why is the Artist class missing fields', 'where did the `_Abstract*` classes come from', 'sync the entity classes with the model', or any request to materialize Java from the DataMap. Also trigger as a follow-up after modeling changes (someone added an entity, attribute, or relationship and now the Java side is stale). This skill exclusively uses the `mcp__cayenne__cgen_run` MCP tool — it does NOT use `mvn cayenne:cgen` or the Gradle cgen task.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[4019,4022,4025],{"name":4020,"slug":4021,"type":15},"Data Modeling","data-modeling",{"name":4023,"slug":4024,"type":15},"Java","java",{"name":4026,"slug":4027,"type":15},"ORM","orm",343,"https:\u002F\u002Fgithub.com\u002Fapache\u002Fcayenne","2026-08-10T04:19:27.571948",{"slug":4032,"name":4032,"fn":4033,"description":4034,"org":4035,"tags":4036,"stars":4028,"repoUrl":4029,"updatedAt":4041},"cayenne-db-import","import database schema into Cayenne DataMaps","Use this skill when the user wants to import database schema metadata into a Cayenne DataMap — the *model\u002Fmapping only*, not names or Java classes. Trigger on phrases like 'reverse engineer the database', 'import the schema', 'generate a DataMap from my DB', 'add the new tables from the DB into the model', 'import the customer table', 'create entities from these tables', or any request to read database metadata to populate or update a DataMap's XML. This is for *full schema* or *bulk table* import; one-off a-la-carte entity additions belong in the cayenne-modeling skill. IMPORTANT — scope: this imports the mapping ONLY; it does not clean up the Object-layer names or (re)generate Java classes. When the user wants their whole project brought in line with the DB ('sync my project with the database', 'my schema changed, update everything', 'update my entities\u002Fclasses from the DB'), that is the end-to-end `cayenne-full-db-sync` skill, which runs this import and then name cleanup and class generation. To regenerate classes alone use `cayenne-cgen`. The skill runs reverse engineering directly via the `mcp__cayenne__dbimport_run` MCP tool when a DBConnector is already configured; otherwise it opens the CayenneModeler GUI via `mcp__cayenne__open_project` to configure the connection first.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[4037,4038,4039,4040],{"name":3837,"slug":30,"type":15},{"name":4023,"slug":4024,"type":15},{"name":4026,"slug":4027,"type":15},{"name":3945,"slug":3946,"type":15},"2026-07-19T05:40:33.655062",{"slug":4043,"name":4043,"fn":4044,"description":4045,"org":4046,"tags":4047,"stars":4028,"repoUrl":4029,"updatedAt":4051},"cayenne-full-db-sync","synchronize Cayenne projects with database","Use this skill when the user wants to bring their WHOLE Cayenne project in line with the database in one shot — the mapping, the Object-layer names, and the generated Java classes together. This is the end-to-end 'sync with the DB' workflow, and it orchestrates three skills in order: `cayenne-db-import` (import schema metadata into the DataMap) → `cayenne-model-naming` (polish the just-imported names) → `cayenne-cgen` (regenerate Java classes). Trigger on holistic phrases like 'sync my project with the database', 'sync with the DB', 'my schema changed, update everything', 'update my entities\u002Fclasses from the database', 'reverse engineer and regenerate the classes', 'import the new tables and rebuild the entities', 'full DB sync', 'bring the model and classes up to date with the DB'. The distinguishing signal is scope: the user wants the whole project (mapping + names + Java code), not just one stage. For the *model\u002Fmapping only* (no name cleanup, no class generation) use `cayenne-db-import`; to (re)generate classes alone use `cayenne-cgen`; to clean names alone use `cayenne-model-naming`. Uses the `mcp__cayenne__dbimport_run` and `mcp__cayenne__cgen_run` MCP tools via the sub-skills; does NOT use Maven or Gradle goals.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[4048,4049,4050],{"name":3837,"slug":30,"type":15},{"name":4023,"slug":4024,"type":15},{"name":4026,"slug":4027,"type":15},"2026-07-19T06:03:49.112969",{"slug":4053,"name":4053,"fn":4054,"description":4055,"org":4056,"tags":4057,"stars":4028,"repoUrl":4029,"updatedAt":4062},"cayenne-model-naming","clean up Cayenne object-layer names","Use this skill to clean up Object-layer names in a Cayenne DataMap — ObjEntity, ObjAttribute, and ObjRelationship names, plus DbRelationship names (the first-class unit of relationship cleanup — every FK has one whether or not an ObjRelationship was generated; the ObjRelationship name is synced to it when one exists) — so they read as descriptive, consistent Java. Trigger on phrases like 'clean up the model names', 'fix the entity names', 'these names look ugly', 'make the names descriptive', 'normalize the ObjEntity\u002Fattribute\u002Frelationship names', 'why is this relationship called team1', 'rename entities to be consistent', 'the import produced Gametype instead of GameType'. Invoke it on an explicit user request, or as a manual follow-up after a `cayenne-db-import` to polish the just-imported additions — it is never triggered automatically. IMPORTANT: this is a LIGHT polish pass — CayenneModeler's reverse-engineering already produces good names for the common case; only improve the specific things its deterministic algorithm cannot (run-together names with no separators like `gametype`, meaningless numbered names like `team1` from multiple relationships between two tables, and a common entity prefix that leaks into relationship names like `aaOrders`). Do NOT rewrite names that are already correct. This is Obj-layer naming polish; for structural model edits use `cayenne-modeling`, and for regenerating classes afterward use `cayenne-cgen`.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[4058,4059,4060,4061],{"name":4020,"slug":4021,"type":15},{"name":3837,"slug":30,"type":15},{"name":4023,"slug":4024,"type":15},{"name":4026,"slug":4027,"type":15},"2026-07-22T05:35:32.342548",{"slug":4064,"name":4064,"fn":4065,"description":4066,"org":4067,"tags":4068,"stars":4028,"repoUrl":4029,"updatedAt":4072},"cayenne-modeler","manage Cayenne projects with CayenneModeler","Use this skill when the user explicitly wants to open CayenneModeler (the GUI) on a Cayenne project, or when the modeling task is inherently visual — reverse engineering (delegated to cayenne-db-import), bulk relationship layout, multi-entity visual refactoring. Trigger on phrases like 'open the Modeler', 'open in CayenneModeler', 'launch the GUI', 'edit visually', 'show me the project in the Modeler'. Do NOT trigger as a fallback for ordinary a-la-carte XML edits — those belong in the cayenne-modeling skill, which is faster and doesn't require the user to context-switch.",{"slug":8,"name":9,"logoUrl":10,"githubOrg":8},[4069,4070,4071],{"name":4020,"slug":4021,"type":15},{"name":4023,"slug":4024,"type":15},{"name":4026,"slug":4027,"type":15},"2026-07-12T08:32:37.199428",112]