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.