Stats
Actions
Tags
How this skill is triggered — by the user, by Claude, or both
Slash command
/issuesmith:issuesmith-code-reviewThe summary Claude sees in its skill listing — used to decide when to auto-load this skill
在问题蔓延之前做 review。Review 的是产物（代码），不是思路（聊天记录）。
在问题蔓延之前做 review。Review 的是产物（代码），不是思路（聊天记录）。
核心原则： Review 早于合并，每次提交后、合并前。
必须 review：
/ism:finish 的提交后自检阶段建议 review：
1. 确定变更范围：
BASE_SHA=$(git merge-base origin/main HEAD)
HEAD_SHA=$(git rev-parse HEAD)
git diff --stat $BASE_SHA..$HEAD_SHA
git diff $BASE_SHA..$HEAD_SHA
2. 按以下维度审查变更：
以下是你应该检查的维度：
3. 分级输出：
按实际严重程度分级。别把所有问题都标 Critical。在列出问题前先肯定做得好的地方 — 准确的肯定让反馈更可信。
### 优点
[做得好的地方。要具体。]
### 问题
#### Critical（必须修复）
[Bug、安全问题、数据丢失风险、功能缺失]
#### Important（应该修复）
[架构问题、遗漏功能、错误处理不足、测试缺口]
#### Minor（可选修复）
[代码风格、优化机会、文档完善]
对每个问题提供：
- 文件:行号
- 什么问题
- 为什么重要
- 如何修复（如果不明显）
### 建议
[代码质量、架构或流程的改进建议]
### 评估
**可以合并？** [是 | 否 | 修复后]
**理由：** [1-2 句技术评估]
### 优点
- 数据库 schema 干净，迁移文件完整（db.ts:15-42）
- 测试覆盖全面（18 个测试，含所有边界情况）
- 错误处理稳健，有 fallback 机制（summarizer.ts:85-92）
### 问题
#### Important
1. CLI 包装器缺少帮助文本
- 文件: index-conversations.ts:1-31
- 问题: 没有 --help 标志，用户无法发现 --concurrency 选项
- 修复: 添加 --help case，展示使用示例
2. 日期校验缺失
- 文件: search.ts:25-27
- 问题: 无效日期静默返回空结果
- 修复: 验证 ISO 格式，无效时抛出带示例的错误
#### Minor
1. 进度指示器
- 文件: indexer.ts:130
- 问题: 长时间操作没有 "X / Y" 计数器
- 影响: 用户不知道要等多久
### 建议
- 为长时间运行的操作添加进度报告
- 考虑为被排除的项目添加配置文件
### 评估
**可以合并：修复后**
**理由：** 核心实现扎实，架构和测试都不错。Important 问题（帮助文本、日期校验）修复简单，不影响核心功能。
必须做：
不要做：
npx claudepluginhub membphis/issuesmith --plugin issuesmithGuides creation, editing, and verification of skills for AI coding agents using test-driven development with subagent scenarios. Use when authoring or debugging skills.