pixiv-cli-review

pixiv-cli Review

以 finding-first 方式审查本仓库 worktree、commit range 或 GitHub PR。先读取 AGENTS.md 与本 SKILL 的 核对清单，再按受影响领域补读 docs/zh-CN/maintainers/architecture.md、docs/zh-CN/maintainers/development.md、 PR 模板和对应 locale contract。

审查边界

• 默认只读：不要擅自修复代码、改 PR body、提交 review、resolve thread、merge 或 push；用户明确要求后才执行对应写操作。
• 先确定范围，不把工作区中与目标无关的既有改动当作本次问题。审查 PR 时优先使用 gh pr view、gh pr diff；审查本地改动时使用 git status --short、git diff --stat、git diff --name-status 和目标 base 的 diff。
• 代码、构建定义和结构化配置优先使用可用 LSP 查定义、调用方、引用和诊断；删除公开符号、跨文件逻辑或测试前先确认影响范围。
• 不只看风格：优先检查架构边界、错误语义、secret/token 泄露、无依据 timeout/重试/截断/fallback、CLI/MCP/SDK 契约、发布/工作流安全和测试覆盖。
• 用户可见行为、兼容性、废弃/移除或安全变化必须同步所需文档；普通 PR 不预写最终双语版本说明。

流程

1. 收集并声明审查范围、base/ref、已有用户改动和相关文件。
2. 读取维护清单，按"边界 → 行为/安全风险 → CLI/MCP/SDK 契约 → 文档 → 测试/构建"顺序核对；对 workflow 变化同时运行对应 policy command。
3. 对每个可复现问题定位到具体文件和紧凑行号，说明触发条件、影响和最小修复方向；不能由仓库证据确认的内容放入 Open Questions，不猜测。
4. 运行与范围匹配的聚焦测试、git diff --check 和必要的 lint/vet/build；记录未运行的真实 API、跨平台或凭据环境测试及剩余风险。
5. 若没有 finding，明确写"未发现阻塞问题"，并说明仍未覆盖的风险；不要用"LGTM"掩盖未完成检查。

输出

按严重程度降序输出，不先写泛泛总结：

Findings
- [P1] path:line 问题。触发条件。影响。建议的最小修复。

Open Questions
- 需要用户或外部证据确认的事项；没有则写 None。

Summary
审查范围；已运行/未运行的测试、policy 和构建；剩余风险。

优先级含义：P0 为立即阻断发布或高危安全/数据损害，P1 为应在合并前修复的高影响问题，P2 为应修复但不阻断的正确性/契约问题，P3 为低风险可维护性问题。发现问题时不要把修复直接混入审查结果；若用户要求修复，再切换到实现流程并重新验证。

核对清单

代码审查时先看行为风险，再看风格。输出 finding-first，按严重程度排序，并给出文件/行号。

[!IMPORTANT] 边界规则以 AGENTS.md 与 docs/zh-CN/maintainers/architecture.md 为准；以下为审查时的快速核对项，发生冲突时以仓库源码与架构文档为权威。

边界

• internal/cli 的 command owner 是否只处理 Cobra、TTY、OAuth loopback、presenter 与 owner-local 窄端口；internal/cli/root.go 是否统一全局 flag、启动生命周期与退出码。
• 是否重新引入 composition root、Runtime、service locator 或 CLI/MCP constructor；MCP stdio 是否由 CLI MCP 命令启动，而不是恢复独立 internal/mcpserver/stdio 包。
• CLI/MCP 是否经账号服务与窄端口调用 public SDK；不得直连 internal/services/{pixiv,fanbox} 协议适配包。
• internal/config/settings 是否只维护 config.toml schema、defaults、effective runtime config、immutable snapshots 和 sparse writes。
• internal/utils/* 是否保持协议无关；Pixiv/MCP/config 协议值不要搬进 generic utils/common。