📦 Architecture Critic — 建筑评论员

架构批评者 一个对抗性的预构建代理。其职责是在编写一行代码之前找出拟议的构建中有什么问题。这不是一个协作者。它不会改进计划或建议替代方案。它读取提案并返回具有具体、逐项发现的判决。没有结果的利益相关性。没有需要保护的关系。运行成本：~$0.05。跳过成本：一个重构周期。何时运行条件操作新API端点或路由运行模式更改（DB表、迁移）运行支付或收费流程触发运行Auth逻辑触发运行功能触及>3个文件运行新外部API集成运行任何带有信心的决策运行复制/样式更改，<3个文件，无逻辑跳过隔离的bug修复，无支付/认证触发跳过当怀疑时：运行它。门槛成本仅为重构周期的一小部分。如何运行# 1. 编写DONE_WHEN简要cat > /tmp/brief.md << 'EOF'目标：<此构建完成什么>范围：<哪些文件/系统被触及>完成时：<具体的验收标准>EOF# 2. 运行批评者bash ~/.openclaw/workspace/skills/architecture-critic/scripts/run-critic.sh \--task "构建简要描述" \--done-when /tmp/brief.md \--repo /path/to/your/repo或者让您的代理自动运行它 —— 参见下面的代理集成部分。判决判决含义操作批准计划合理。没有发现阻塞问题。继续构建。修订发现了具体问题。修复发现结果，重新运行。最多2个周期，然后升级。拒绝方法中的基本缺陷。停止。重新设计。不要构建。判决结果保存到您的工作空间中的specialists/critic-verdicts/YYYY-MM-DD-.md中。代理集成将以下内容添加到您的AGENTS.md或代理指令中：## 架构批评者 —— 强制门槛在任何重要的构建之前（新端点、模式更改、支付/认证触发、>3个文件）：1. 将DONE_WHEN简要写入临时文件2. 运行：bash ~/.openclaw/workspace/skills/architecture-critic/scripts/run-critic.sh \--task "<描述>" --done-when <文件> [--repo <路径>]3. 批准 → 继续4. 修订 → 修复发现结果，重新运行（最多2个周期）5. 拒绝 → 停止，升级到人类跳过仅适用于：复制/样式更改 <3个文件，无逻辑，隔离的bug修复，无支付/认证触发。工作原理批评者脚本：快照相关代码库状态（文件树 + 关键文件）将其与您的DONE_WHEN简要结合成结构化提示以温度0生成隔离代理 —— 无创造力，无鼓励代理运行检查清单对您的计划返回具有具体、逐项发现的判决结果将判决结果保存到您的工作空间中批评者代理永远不会看到您的对话历史、您的热情或提议者的推理。它只看到规格和代码。LLM兼容性与您的OpenClaw实例中配置的任何LLM一起使用：Anthropic（Claude Sonnet，Haiku）OpenAI（GPT-4o，GPT-4 Turbo）Google（Gemini）通过Ollama的本地模型脚本从您的OpenClaw配置中读取以找到活动模型和API密钥。无需单独配置。检查清单覆盖批评者根据特定于域的检查清单进行审查：Web（references/checklist-web.md）— 路由、缓存、部署配置、破坏性更改安全性（references/security.md）— 认证绕过、注入、密钥、CORS、OWASP Top 10支付流程（references/payment-flows.md）— Webhook、幂等性、竞争条件、PCI范围AI构建（references/ai-builds.md）— 提示注入、热路径、令牌成本、后备处理通用（references/checklist-general.md）— 范围蔓延、依赖项、回滚、数据迁移传递--checklist references/security.md以关注特定域，或让批评者根据您的简要自动选择。文件architecture-critic/├── SKILL.md # 本文件├── README.md # GitHub面向文档├── LICENSE # MIT├── scripts/│ └── run-critic.sh # 主运行脚本└── references/ ├── checklist-web.md # Web/API检查清单 ├── checklist-general.md # 通用架构检查清单 ├── security.md # 安全检查清单 ├── payment-flows.md # 支付/计费检查清单 └── ai-builds.md # AI/LLM特定检查清单