菜单
Review a PR through the CodeReviewYAGNI persona. Use when asked to check whether a PR stays focused on the task, avoids overengineering, avoids premature generalization, and does not defensively program around edges that are not expected to occur.
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
Run isolated Moraine dev sandbox validation. Use when testing ingest, MCP, monitor, ClickHouse schema, source formats, stack behavior, or any change where QA must not touch the developer's host ~/.moraine install; use before reporting completion for stack-facing changes.
Help prepare and, only with explicit user approval, file a sanitized Moraine bug report for reproducible defects in Moraine itself.
Use Moraine realtime session data to inspect active agent work across harnesses when the user asks what is happening now or what another agent is doing.
Use Moraine MCP tools to recover prior agent-session context when the user refers to earlier work, decisions, failures, branches, files, or other agents.
Cut and publish Moraine releases from a version argument such as /release X.Y.Z or a request to use $moraine-dev:release. Use when Codex is asked to run Moraine's release process, bump release-managed versions, create and merge the release PR, push the vX.Y.Z tag, update GitHub release notes for a general audience, verify the release-moraine workflow, and confirm the PyPI moraine-cli package. Start or continue a Codex goal for the release.
Use Moraine MCP tools to recover prior agent-session context when the user refers to earlier work, decisions, failures, branches, files, or other agents.
| name | code-review-yagni |
| description | Review a PR through the CodeReviewYAGNI persona. Use when asked to check whether a PR stays focused on the task, avoids overengineering, avoids premature generalization, and does not defensively program around edges that are not expected to occur. |
Review for "you are not gonna need it." Ask whether the PR solves today's task directly, or whether it adds abstractions, options, indirection, and defensive code for hypothetical futures.
Do not reject small abstractions that clearly reduce present complexity. Do not argue against validation or error handling for inputs that can actually occur.
Lead with unnecessary implementation surface and the simpler present-tense alternative:
- [P2] Remove the provider registry until a second provider exists
`crates/example/src/provider.rs:14`
The PR only supports ClickHouse, but it adds a generic registry, trait object,
and config dispatch. A direct `ClickHouseBackend` keeps the current behavior
smaller and can be generalized when another backend lands.
If there are no YAGNI findings, say the PR stays appropriately focused for the task.