菜单
Pre-merge review checklist based on recurring AI reviewer feedback patterns
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
Use when adding or reviewing tests for Reborn behavior — choosing a test tier, covering a bug fix, testing model/tool-choice behavior, touching tests/integration or tests/fixtures/llm_traces, or when a test needs Postgres, Docker, or a live LLM.
Navigate building a user-facing feature in the Reborn stack (a capability that crosses product_workflow → composition → webui_v2 → runtime/serve → frontend). Use when planning or implementing any new Reborn settings page, endpoint, facade method, or runtime-backed capability — especially before writing code, to avoid rebuilding what already exists and to wire it in one pass instead of layer-by-layer.
Use when asked to "review the open PRs", review a batch or stack of pull requests, or run a recurring PR-review pass on a repo — especially with many PRs, stacked branches, conflicts, or security-sensitive changes. Covers grouping, fan-out to review subagents, verdict synthesis, and posting.
Generate or update the IronClaw architecture overview video using Remotion. Use when asked to update, regenerate, or modify the architecture video, add/remove scenes, or reflect codebase changes in the video.
Use when writing or reviewing a change in crates/ that adds a trait, a crate, a dependency edge, a re-export, or code in ironclaw_reborn_composition — or when deciding whether an abstraction, layer, or crate boundary is justified in the IronClaw Reborn stack.
Use when starting work in the IronClaw repo, deciding where a feature/fix/prompt/doc belongs, tracing how a request flows, looking up which crate owns a subsystem, or when repo docs, the knowledge graph, or component names seem stale, missing, or contradictory.
| name | review-checklist |
| version | 0.1.0 |
| description | Pre-merge review checklist based on recurring AI reviewer feedback patterns |
| activation | {"patterns":["review.*checklist","ready to merge","pre-merge check","check.*before.*merge"],"keywords":["review","checklist","merge","pre-merge"],"max_context_tokens":1500} |
Before merging, verify these items. They represent the most common issues caught by automated code reviewers (Copilot, Gemini) on IronClaw PRs.
redact_params() before logging or SSE/WebSocket broadcastrequires_approval() returning Always or UnlessAutoApproved&s[..n]) on external/user strings -- use is_char_boundary() or char_indices().to_ascii_lowercase() before matching)LlmProvider trait methods are delegated in ALL wrapper types (grep impl LlmProvider for)tempfile crate, no hardcoded /tmp/ pathsserial_test)