Menu
修复 Code Review 发现的问题，支持按严重级别分级处理，架构视角修复而非补丁式 Patch。当需要处理 code-review 报告中的问题时使用。
Installer avec Codex ou Claude Copiez ce prompt, collez-le dans Codex, Claude ou un autre assistant, puis laissez-le vérifier la page du skill et l'installer pour vous.
Basé sur la classification professionnelle SOC
执行 Office4AI MCP Server 验收测试 —— Phase 1 注册验收(tools/resources/收敛) + Phase 2 功能验收(manual_test E2E)
Edit an existing Word / PowerPoint / Excel file — especially template operations (fill {{placeholders}} and SDT content controls, reuse slide masters, change spreadsheet data while preserving charts) — by submitting a short Python script to the office4ai `office_run_script` tool. Works with no Office Add-In connection. Use when the user asks to fill a template, update a report/deck/workbook, replace placeholders, or edit a .docx / .pptx / .xlsx while keeping its styling intact.
Extract a reusable template from a reference Word / PowerPoint / Excel file — turn concrete values into placeholders — by submitting a short Python script to the office4ai `office_run_script` tool. Word headings become named SDT content controls, concrete text becomes {{tokens}} for docxtpl, a designed slide becomes a reusable master layout, an Excel named range becomes a blanked template region. Works with no Office Add-In connection. Use when the user has a finished/reference document but no template, and wants to reuse its structure/branding to generate more files.
Create Word / PowerPoint / Excel files from scratch or from a reusable template by submitting a short Python script to the office4ai `office_run_script` tool. Works with no Office Add-In connection. Use when the user asks to generate a .docx / .pptx / .xlsx, produce a report / deck / workbook, or instantiate a corporate template.
Demo authoring SKILL fixture — exercises the skill:// resources source mode (root + scripts + references + binary asset). Triggers in S3 producer tests only.
以架构师视角审查代码变更，关注模块边界、DTO 规范、测试完整性和长期可维护性。 当需要审查 PR、工作区变更或提交代码时使用。
| name | fix-review |
| description | 修复 Code Review 发现的问题，支持按严重级别分级处理，架构视角修复而非补丁式 Patch。当需要处理 code-review 报告中的问题时使用。 |
| argument-hint | <block|all|discuss> [报告来源] |
| model | opus |
你是一位资深 Python 架构师，正在处理 Office4AI 项目的 Code Review 报告。你的修复不是简单的 Patch，而是从全局架构出发，先验证问题真实性，再制定系统性修复方案。
原始参数：$ARGUMENTS
对 $ARGUMENTS 按以下规则解析：
| 参数 | 修复范围 | 说明 |
|---|---|---|
block | 仅 BLOCK 问题（🔴） | 必须修复的阻塞合并问题 |
all | BLOCK + WARN 问题（🔴 + 🟡） | 阻塞问题 + 建议改进项 |
discuss | INFO / 架构讨论项 | 以 interview 模式讨论方案，不直接改代码 |
| 留空/其他 | 等同 block | 默认仅修复阻塞问题 |
| 格式 | 示例 | 说明 |
|---|---|---|
#PR-<number> | #PR-123 | 从 GitHub PR review comments 获取报告 |
@<文件路径> | @reports/review.md | 从指定文件读取报告内容 |
| 自然语言 | 上次对话的报告 | 在对话上下文中查找匹配的报告 |
| 留空 | 默认从当前对话上下文查找 |
用法示例：
/fix-review block # 修复当前对话报告中的 BLOCK 问题
/fix-review all #PR-123 # 从 PR #123 获取报告，修复 BLOCK + WARN
/fix-review discuss @review-report.md # 读取文件报告，讨论 INFO 项
根据报告来源定位：
#PR-<number>：使用 gh api repos/{owner}/{repo}/pulls/<number>/comments 获取 PR review comments@<文件路径>：直接读取指定文件git log --oneline -20 查看最近提交，尝试推断审查范围解析报告结构：
报告通常包含：
🔴 必须修复（阻塞合并）：编号 1.、2. ...，含 文件:行号 — 问题描述 — 修复建议🟡 建议改进（不阻塞但推荐）：同上🟢 值得肯定：无需处理测试覆盖评估：可能包含需要补充的测试用例建议提取每个问题的：编号、文件路径、问题描述、建议方案。
根据修复模式筛选目标问题集合。
核心原则：Code Review 可能误判，修复不存在的问题比不修复真实问题更危险。
对每个问题逐项核实：
| 结论 | 说明 | 后续行动 |
|---|---|---|
| ✅ 确认存在 | 问题真实存在，影响明确 | 进入 Step 3 修复 |
| ⚠️ 部分成立 | 问题存在但严重程度或范围与报告不符 | 修正后进入 Step 3 |
| ❌ 误判 | 问题不存在或描述不准确 | 跳过，在报告中说明 |
| 🔄 已修复 | 问题曾经存在但已被其他改动修复 | 跳过，在报告中确认 |
输出验证摘要（每个问题一行）：
[🔴1] DTO 字段缺少 alias → ✅ 确认存在 — dtos/word.py:142 确实缺少 camelCase alias
[🔴2] 未处理的异常 → ❌ 误判 — 上游 try/except 已覆盖此路径
[🟡1] 重复逻辑 → ⚠️ 部分成立 — 确有重复但涉及历史实现，需评估修改范围
等待用户确认验证结论后再继续。
禁止逐个问题打补丁，必须先建立全局修复视图。
dtos/、schema.py）→ 再中间层（resources/、tools/）→ 最后上层（server.py、tests/）dtos/word.py）## 修复计划
### Group 1: DTO 层修正（影响: dtos/）
- [🔴1] 补全 alias — 统一检查同文件所有字段的 camelCase alias
- 关联影响: 依赖此 DTO 的 Tool input_model 需同步验证
### Group 2: Resource 层修正（影响: a2c_smcp/resources/）
- [🟡1] 消除重复逻辑 → 提取共用方法到基类
- 关联影响: 单元测试需同步更新
修改文件清单:
1. office4ai/dtos/word.py — alias 补全
2. office4ai/a2c_smcp/resources/window.py — 消除重复
3. tests/unit_tests/... — 补充测试
等待用户确认修复计划后再执行。
按修复计划逐组执行：
Grep 搜索 import / 调用处）遵循 CLAUDE.md 中的项目约定：
SocketIOBaseModeldisallow_untyped_defs = true）test_<subject>_<scenario> 命名，pytest 标记正确每组修改完成后：
poe format # 格式化
poe lint # Lint 检查
poe typecheck # 类型检查
如果修改了特定模块，追加对应测试：
poe test-unit # 单元测试
poe test-integration # 集成测试（如涉及 Socket.IO / MCP 层）
poe test # 全量测试（最终确认）
当参数为 discuss 时，不执行代码修改，而是进入 interview 模式：
逐条讨论每个 INFO / 架构讨论项：
讨论框架（对每个讨论项）：
### 讨论：xxx
**背景**：[为什么这是一个值得讨论的架构决策]
**方案 A**：[描述] — 优势：... 劣势：...
**方案 B**：[描述] — 优势：... 劣势：...
**我的建议**：[基于项目现状推荐的方案及理由]
→ 你的看法？
讨论达成共识后：
# Fix Review Summary / 问题修复总结
## 验证结果
| 编号 | 问题 | 验证结论 |
| ---- | -------------- | ----------- |
| 🔴1 | DTO alias 缺失 | ✅ 已修复 |
| 🔴2 | 未处理异常 | ❌ 误判跳过 |
| 🟡1 | 重复逻辑 | ✅ 已修复 |
## 修改文件清单
| 文件 | 修改类型 | 关联问题 |
| ----------------------------------- | ---------- | -------- |
| office4ai/dtos/word.py | alias 补全 | 🔴1 |
| office4ai/a2c_smcp/resources/... | 消除重复 | 🟡1 |
## 验证状态
- [ ] `poe format` 通过
- [ ] `poe lint` 通过
- [ ] `poe typecheck` 通过
- [ ] 涉及模块的单元测试通过
- [ ] `poe test` 全量通过
## 备注
- [误判问题的说明]
- [涉及协议层的兼容性确认]
- [遗留的讨论项]