Communitygithub.com

omc-review

评估已完成工作的缺陷、风险及简化空间,在交付前进行审查。

omc-review 是什么?

该技能用于对已存在的工作成果进行评估,而非直接修改。它作为核心审查工作流,`merge-readiness` 会路由至此,而 `ai-slop-cleaner` 是其内部的一个可选通道。审查的目标是按严重性排序,找出实际存在的问题,并提供足够详细的信息以便采取行动。标准流程包括:明确变更内容与意图、对照意图阅读变更、依次检查正确性、风险与简化、验证每个候选发现后再报告,最后按严重性从高到低输出结果。 审查覆盖的正确性(逻辑缺陷、边界情况、错误路径、并发问题)、风险(安全边界、破坏性操作、数据完整性)、复用(是否应使用现有工具或模式)、简化(可删除或合并的代码)以及覆盖率(未测试的行为)。其核心规则包括:审查者不得与作者处于同一活跃上下文;报告前必须验证,避免“听起来合理但无法复现”的噪音;诚实陈述严重性,不为了显得全面而夸大列表;当工作本身无误时,“无发现”也是有效结果。默认以建议形式输出,但硬性门禁(如发布、安全、破坏性操作)保持独立且默认关闭。

兼容平台~Claude Code~Codex CLI~Cursor
npx skills add https://github.com/Yeachan-Heo/oh-my-claudecode/tree/main/skills/review

在你喜欢的 AI 中提问

打开一个已预加载此 Agent Skill 的新对话。

文档

审查

使用此技能来评估已完成的工作。审查绝不会编写它正在评判的变更。

这是规范的审查工作流。merge-readiness 会路由至此,而 ai-slop-cleaner 是其内部的一个可选通道。

目标

找出实际存在的问题,按严重性排序,并提供足够详细的信息以便采取行动。

工作流程

  1. 明确变更内容及其意图。
  2. 对照该意图阅读变更。
  3. 首先检查正确性,然后检查风险,最后检查简化空间。
  4. 在报告每个候选发现之前进行验证。
  5. 首先报告最严重的发现。

检查内容

  • 正确性 — 逻辑缺陷、边界情况、错误路径、并发问题
  • 风险 — 安全边界、破坏性操作、数据完整性
  • 复用 — 变更本应使用的现有工具或模式
  • 简化 — 可删除或合并的代码
  • 覆盖率 — 未经测试即交付的行为

规则

  • 分离通道:审查者不得与作者处于同一活跃上下文。
  • 报告前验证。一个听起来合理但无法复现的发现是噪音。
  • 诚实陈述严重性;不要为了显得全面而夸大列表。
  • 当工作本身无误时,“无发现”是有效结果。
  • 默认为建议——审查提供信息,而非阻塞。硬性门禁(如发布、安全、破坏性操作)保持独立且默认关闭。

输出

  • 发现项,按最严重到最不严重排序,每个包含文件、行号以及具体的故障场景
  • 已检查且确认无误的内容
  • 任何无法评估的内容

Individual skills in this repo

This repo contains 20 individual skills — each has its own dedicated page.

agent-doc-discipline

Writing-time discipline for documents agents consume (the five surfaces, specs, tickets, .omc/skills/) — every rule checkable and carrying a why, steps before reference, one meaning in one home, no restating what the environment already says. Mandatory at drydock seed generation and the launch C5 sediment pass; opt-in for any other agent-facing doc edit. The companion of minimal-code-discipline: that one disciplines code, this one disciplines papers.

ai-slop-cleaner

Clean AI-generated code slop with a regression-safe, deletion-first workflow and optional reviewer-only mode

ask

Process-first advisor routing for Claude, Codex, Gemini, Antigravity, Grok, or Cursor via `omc ask`, with artifact capture and no raw CLI assembly

ask-navigator

Shipyard

autopilot

Full autonomous execution from idea to working code

autoresearch

Stateful single-mission improvement loop with strict evaluator contract, markdown decision logs, and max-runtime stop behavior

cancel

Cancel any active OMC mode (autopilot, ralph, ultragoal, swarm, ultrapilot, pipeline, team) and clean up retired legacy state

configure-notifications

Configure notification integrations (Telegram, Discord, Slack) via natural language

debug

Diagnose the current OMC session or repo state using logs, traces, state, and focused reproduction

deepinit

Deep codebase initialization with hierarchical AGENTS.md documentation

deep-interview

Socratic deep interview with mathematical ambiguity gating before explicit execution approval

drydock

Lay the keel of the shipyard harness in any repo — the 4-pillar shared environment (Context, Rules, Tools, Standards) across 5 surfaces (CLAUDE.md, skills, design-system, mcp/cli, shared context) so that every human and agent inherits the same design language and anyone can ship. Run once per repo; re-run with --check to audit drift.

execute

Carry an approved task through to working, verified code

external-context

Invoke parallel document-specialist agents for external web searches and documentation lookup

graph

Deterministic orchestration graph runtime - declarative DAG pipelines with journal-based crash recovery

harbor

Harbor intake for external work — the captain only handles unresolved decisions. Sweeps incoming issues and PRs, verifies every claim before disposition, reuses every decision already made, and hands the maintainer a docket whose pending items each carry one question with options, recommendation, impact and evidence. Agent-autonomous for facts and for actions covered by standing authorization; signed for every new judgment. Never merges.

hud

Configure HUD display options (layout, presets, display elements)

launch

Shipyard

loft

Loft the shape before cutting steel — answer a design question that prose cannot settle by building a throwaway artifact: a pure logic module in a clickable shell, or structurally different UI variants behind one route. The captain reacts to the artifact; the answer folds into the decision; the artifact never docks. Use when a design question stalls in words, when a navigator map carries a loft ticket, or when a spec discussion reaches

minimal-code-discipline

YAGNI-ladder coding discipline for writing changes — existence-first, reuse before writing, dependency ladder, shortest correct diff, with non-negotiables that must never be minimized away

相关技能