doris-repo-review

doris-repo-review is a skill for Codex from apache/doris-skills. It costs 180 tokens per session (9,290 once invoked), scanned A, original, Apache-2.0.

A local review workflow for pull requests in the Apache Doris source repository. A pull request is a proposed change to a code project for review before it is merged.

In plain words
What is it for?
Use it to review a Doris pull request from a local clone, align the worktree when safe, and produce English and Chinese review documents.
Why use it?
It checks that the local code matches the requested change and organizes the review around required checks, risks, and conclusions.

Skill for Codex ✓ vendor

Written for Codex: runs codex exec. Also seen: reads .claude/ paths; mentions subagents; mentions Claude Code.

Good fit Use it to review a Doris pull request from a local clone, align the worktree when safe, and produce English and Chinese review documents.

Compare 6 skills from other repositories ↓
Install with agentmods
npx agentmods add skills/apache/doris-skills/doris-repo-review
About the project

Apache Doris Skills is a collection of agent instructions containing Apache Doris database knowledge for designing, sizing, operating, diagnosing, and contributing to Doris clusters. It serves developers and operators using Claude Code, Cursor, Codex, Cline, Amp, and other agent tools, with the skills providing decision logic and evidence-based workflows.

apache/doris-skills · 41 stars · on GitHub · doris.apache.org

Install

Getting it into your agent

One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.

Any agent
npx skills add apache/doris-skills --skill doris-repo-review
Clone the repo
git clone --depth 1 https://github.com/apache/doris-skills

Made for: Codex.

Wrote this? Show the measurements

A badge with what this costs and how it scanned, read live from this page, so it follows the numbers instead of freezing them. Markdown for a README, HTML for a documentation site or a project page.

agentmods badge for doris-repo-review

README.md
[![agentmods](https://agentmods.dev/badge/skills/apache/doris-skills/doris-repo-review/github.svg)](https://agentmods.dev/skills/apache/doris-skills/doris-repo-review)
Your own site
<a href="https://agentmods.dev/skills/apache/doris-skills/doris-repo-review"><img src="https://agentmods.dev/badge/skills/apache/doris-skills/doris-repo-review/github.svg" alt="Measured on agentmods" height="20"></a>

Or the 80×15 button, for a site that already has a row of RSS and ATOM ones. Only the verdict fits; the numbers stay here.

agentmods 80×15 button for doris-repo-review

Your own site · 80×15
<a href="https://agentmods.dev/skills/apache/doris-skills/doris-repo-review"><img src="https://agentmods.dev/badge/skills/apache/doris-skills/doris-repo-review.svg" alt="Reviewed on agentmods" width="80" height="20"></a>
Per session 180 Skills are progressive disclosure: only the name and description are preloaded; the body loads when the skill is used.
When invoked 9,290 The whole file, excluding the scripts and references it only reads on demand.
Security scan A 0 findings. A grade says what 26 rules found in the file — not that it is safe. Third-party audits
  • NVIDIA SkillSpector warn 7 Sept 2026
SkillSpector: 4 findings, up to high

These are SkillSpector’s own severities. On a checked sample its high-severity flags on skills were ~96% false positives — a documented command, a public API, a “never do X” rule — so we show them as a caution to read, not a verdict. Why →

  • high Anti-Refusal · line 251
    Skill instructs the agent to never refuse or to always comply. Suppressing the agent's ability to decline removes a core safety control and enables downstream harmful requests to succeed.
    Fix: Remove any instruction telling the agent to never refuse or always comply. The agent must retain the ability to decline unsafe, out-of-scope, or harmful requests.
  • medium Rogue Agent · line 3
    Skill establishes unauthorized persistence across sessions via cron jobs, startup scripts, or state files. Session persistence allows an attacker to maintain access beyond the current interaction.
    Fix: Remove any persistence mechanisms (cron jobs, startup scripts, state files). Skills should not maintain state across sessions without explicit user consent.
  • medium Agent Snooping · line 237
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 239
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
How audits are shown
Origin original No closer match found in the catalogue.
Token cost

What it costs to keep this loaded

Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.

ModelPer sessionOnce invoked
Fable 5.1 $0.00180 $0.09290
Opus 5 $0.00090 $0.04645
Sonnet 5 $0.00036 $0.01858
Haiku 4.5 $0.00018 $0.00929

Measured 4d ago against content hash 2181c9ce4c6a, method: parsed. Prices are Anthropic first-party input rates as of 2026-09-12, from the pricing page.

Security

Grade A, and why

doris-repo-review scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 4d ago.

The scan reads SKILL.md. This mod also ships 8 executable files (scripts/align-to-pr.sh, scripts/coverage-report.sh, scripts/post-pass-comment.sh, …), listed below but not scanned — reading those needs a real analyzer, not pattern matching.

A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.

Nothing flagged

None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.

skills/doris-repo-review/SKILL.md · 629 lines

How it starts

The opening of the file, as written. The whole thing — 629 lines — stays where its author put it; the contents beside it link to each section on GitHub.

Local pipeline-style Doris code review

This skill reviews a Pull Request against a local clone of the Apache Doris source repository. It is a contributor-side workflow, not a cluster-operations skill: it never talks to a running cluster and never needs one.

Usage:

/doris-repo-review https://github.com/apache/doris/pull/66807
/doris-repo-review https://github.com/apache/doris/pull/66807 focus on class loading and compatibility
/doris-repo-review 66807            # a bare number defaults to apache/doris

This ports the review flow of apache/doris/.github/workflows/code-review-runner.yml to a local machine: first align the current directory to the PR head, take the diff from local git, and produce one English and one Chinese document under the current directory's review-docs/. Everything else - required reading, main-agent risk scan, subagent split, shared ledger, at most 3 convergence rounds, an explicit conclusion per checkpoint - matches CI.

The scope is the one worktree you are standing in. No other worktree is scanned, chosen, or created; no environment variable selects a different directory; nothing is assumed about the machine's directory layout.

Let $S be this skill's own scripts/ directory (with a default Claude Code install that is ~/.claude/skills/doris-repo-review/scripts):

$S/align-to-pr.sh <PR> --check                      # diagnosis only, changes nothing
$S/align-to-pr.sh <PR> --out "$CTX/align.env"       # step 1: align the current directory
$S/prepare-review-context.sh --ctx "$CTX" --align "$CTX/align.env"   # step 2: gather context
$S/coverage-report.sh --ctx "$CTX"                  # step 6a: what has nobody read yet
python3 $S/verify-review-docs.py --ctx "$CTX" --doc <en> --doc <zh>  # step 9: verify documents
$S/record-review-runtime.sh --ctx "$CTX" --model <id> --effort <effort>
$S/post-pass-comment.sh --ctx "$CTX"                                 # step 10: auto-post PASS
$S/review-runtime-policy.sh check <model> <effort>                    # reviewer eligibility
$S/save-run-state.sh --ctx "$CTX" --verdict ... --findings b,m,mi,n  # step 11: leave state for next time
File Purpose
scripts/align-to-pr.sh Resolve the PR, diagnose how the current directory relates to it, align it to the PR head
scripts/prepare-review-context.sh Produce the authoritative diff, new-side line ranges, required AGENTS.md list, existing comments, ledger skeleton
scripts/coverage-report.sh Mechanical check of which changed files no ledger file has mentioned yet
scripts/verify-review-docs.py Validate commit, anchors, EN/ZH agreement, verdict, findings, rounds, and convergence
scripts/record-review-runtime.sh Record the qualified reviewer model, effort, and exact commit
scripts/review-runtime-policy.sh The exact model and effort allowlist for a pipeline-equivalent review
scripts/post-pass-comment.sh Render and post the machine-readable PASS comment; refuses everything that is not a pass
scripts/save-run-state.sh Persist this run's merged ledger under the stable per-PR state directory, so the next review inherits its dismissals
references/prompts.md Subagent prompt templates (CI wording, carried over verbatim)
references/doc-templates.md Templates for both documents, anchor format, verdict rule
references/pr-comment-format.md The doris-repo-review/v1 comment schema, field meanings, and how a program reads it back
references/qualified-runtime.md How to select or delegate to a qualified lead and coverage reviewers

Requirements: git, an authenticated gh CLI, jq, and python3. The clone must have full history (git fetch --unshallow on a shallow one), because the authoritative diff is a three-dot diff from the merge base.


Read the full file on GitHub · 629 lines

Changes

What this file has done since we first saw it

Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.

  1. 4d ago Changed · +3 tokens per session 2181c9ce4c6a
  2. 8d ago Changed · -5 lines · -136 tokens per session 3388dc1e0800
  3. 12d ago First seen · 634 lines · 313 tokens per session scan A e2180c83a245

Subscribe to this mod's changes

doris-repo-review is a skill published in the GitHub repository apache/doris-skills (41 stars, last pushed 5d ago), licensed Apache-2.0. It adds 180 tokens to every session and 9,290 once invoked, about $0.0009 per session on Opus 5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.

Related

Other skills, from other repositories

git

Git and code hosting platform management — GitHub, GitLab, Codeberg, Forgejo. Zero barrier — install x-cmd and manage repos from the terminal. Use for "git", "github", "gitlab", "repo", "pull request", "code hosting".

x-cmd/x-cmd · 58 tokens

issue-work-loop

Run Herdr loops for one open GitHub issue (resolve→review→fix) or an existing PR (review→lazy fixer) until CLEAN. Don't use for plain resolution without review, review-only/no-fix requests, backlog automation, or merging.

luongnv89/skills · 55 tokens

gh-address-comments

Address review and issue comments on the open GitHub PR for the current branch using gh CLI. Use when user says "address PR comments", "fix review feedback", "respond to PR review", or "handle PR comments". Verifies gh auth first and prompts to authenticate if not logged in. Do NOT use for creating PRs, CI debugging…

tech-leads-club/agent-skills · 85 tokens

azure-repos

Expert knowledge for Azure Repos development including troubleshooting, best practices, decision making, architecture & design patterns, limits & quotas, security, configuration, and integrations & coding patterns. Use when managing Git/TFVC repos, branch/PR policies, CLI/IDE integrations, CodeQL/scanning, or…

MicrosoftDocs/Agent-Skills · 115 tokens

hashiiiii-pull-request

Use this skill to create or edit a GitHub pull request with the repository template or the default structure.

hashiiiii/rules-for-ai · 28 tokens

cf-ship-custom

This is a version bump + ship + release operation. Run these steps BEFORE the standard cf-ship workflow.

dinhanhthi/coding-friend · 0 tokens