ai-engineering-init 1.16.2 → 1.16.3

package/.claude/skills/code-patterns/SKILL.md

@@ -101,120 +101,6 @@ feat(order): 新增订单导出功能
 Closes #123
 ```
 
-## 数据类型规范
-
-### 布尔语义字段必须使用 Boolean
-
-```java
-// ❌ 错误
-private Integer ifNarrow;
-private Integer isEnabled;
-
-// ✅ 正确
-private Boolean narrow;     // getter 自动生成 isNarrow()
-private Boolean enabled;    // getter 自动生成 isEnabled()
-```
-
-**规则**：
-- 语义为"是/否"的字段，类型必须为 `Boolean`
-- 字段名不加 `if`/`is`/`has` 前缀（JavaBean 规范中 `Boolean` 的 getter 自动生成 `isXxx()`）
-- 数据库字段使用 `TINYINT(1)` 或 `BIT(1)`
-
-### 枚举字段必须提供明确约束
-
-```java
-// ❌ 错误：调用方无法知道合法值
-@ApiModelProperty(value = "操作类型")
-private Integer tradeType;
-
-// ✅ 方案一：VO/DTO 层直接用枚举（推荐）
-@ApiModelProperty(value = "操作类型")
-private AccTradeTypeEnum tradeType;
-
-// ✅ 方案二：保留 Integer 但标注合法值
-@ApiModelProperty(value = "操作类型：1-充值 2-消费 3-退款", allowableValues = "1,2,3")
-private Integer tradeType;
-```
-
-### 金额字段禁止使用浮点类型
-
-```java
-// ❌ 错误
-private Double amount;
-private Float price;
-
-// ✅ 正确：Entity/Service 层用 Long（分），VO 层展示用 BigDecimal（元）
-private Long amountFen;
-private BigDecimal amountYuan;
-```
-
-### 原始类型 vs 包装类型
-
-| 场景 | 用原始类型 | 用包装类型 |
-|------|----------|----------|
-| Entity / VO / DTO 字段 | — | ✅ 统一用包装类型 |
-| 方法参数（不允许 null） | ✅ `int count` | — |
-| 方法参数（允许 null） | — | ✅ `Integer count` |
-| 局部变量 | ✅ `int i = 0` | — |
-
-## Optional 使用规范
-
-```java
-// ❌ 错误：of() 不接受 null，value 为 null 直接 NPE
-Optional.of(value).orElse(defaultValue);
-
-// ✅ 正确：ofNullable() 安全处理 null
-Optional.ofNullable(value).orElse(defaultValue);
-
-// ❌ 禁止：Optional 作为方法参数或类字段
-public void process(Optional<String> name) { ... }
-private Optional<String> name;
-
-// ✅ 允许：Optional 作为方法返回值、链式处理
-public Optional<Entity> findById(Long id) { ... }
-Optional.ofNullable(entity).map(Entity::getConfig).orElse(DEFAULT_VALUE);
-```
-
-## @Transactional 规范
-
-```java
-// ❌ 错误：默认只回滚 RuntimeException
-@Transactional
-public void createOrder() { ... }
-
-// ✅ 正确：显式指定回滚异常
-@Transactional(rollbackFor = Exception.class)
-public void createOrder() { ... }
-```
-
-- 所有 `@Transactional` 必须显式写 `rollbackFor = Exception.class`
-- 只读查询不加 `@Transactional`（或用 `readOnly = true`）
-- 事务方法不要 try-catch 吞掉异常，否则事务不回滚
-
-## TODO 管理规范
-
-```java
-// ❌ 错误：无负责人、无日期、无跟踪
-// TODO 修改一下
-
-// ✅ 正确：完整 TODO 格式
-// TODO(@陈沈杰, 2026-03-20, #TASK-1234): 移动端 AppId 赋值逻辑待产品确认
-```
-
-- 每个 TODO 必须有对应的任务号
-- 超过 2 个迭代未处理的 TODO 必须清理
-- 不用的代码直接删除，不要注释保留
-
-## 代码格式化规范
-
-| 项目 | 规范 |
-|------|------|
-| 缩进 | 4 个空格（不用 Tab） |
-| 行宽 | 120 字符 |
-| 大括号 | K&R 风格（同行开始） |
-| 空行 | 方法间 1 个空行，逻辑块间 1 个空行 |
-| import | 分组排序：java → jakarta → org → net → com，组间空行 |
-
 ## 代码示例
 
 ### 统一响应格式
@@ -275,8 +161,3 @@ public enum OrderStatusEnum {
 | Git 提交信息写"fix bug" | 写清楚修了什么：`fix(order): 修复金额计算精度丢失` |
 | Boolean 变量名：`flag` | 有意义的名字：`isActive`, `hasPermission` |
 | 缩写命名：`usr`, `mgr` | 完整命名：`user`, `manager` |
-| `Optional.of(可能null值)` | `Optional.ofNullable(value)` |
-| `@Transactional` 无 rollbackFor | `@Transactional(rollbackFor = Exception.class)` |
-| TODO 无负责人和日期 | `// TODO(@负责人, 日期, #任务号): 描述` |
-| 布尔字段用 `Integer` | 用 `Boolean` 类型 |
-| 枚举字段无合法值说明 | `@ApiModelProperty` 标注合法值或直接用枚举类型 |

package/.claude/skills/codex-code-review/SKILL.md

@@ -88,45 +88,6 @@ Grep pattern: "@Transactional\((?!.*rollbackFor)" path: [命中文件]
 ```
 - ❌ `@Transactional` → ✅ `@Transactional(rollbackFor = Exception.class)`
 
-#### 🔴 A7b. selectOne 无唯一保障
-```bash
-Grep pattern: "selectOne(" path: [目标目录] glob: "*.java"
-# 命中行检查：是否有 LIMIT 1 或注释说明唯一索引
-```
-- ❌ 无 LIMIT 且无唯一索引 → ✅ 加 `.last("LIMIT 1")` 或确保有唯一索引
-
-#### 🟡 A7c. selectCount 用于存在性判断
-```bash
-Grep pattern: "selectCount" path: [目标目录] glob: "*.java"
-# 检查命中行是否用于 > 0 判断（存在性），而非真正计数
-```
-- ❌ `selectCount(w) > 0` → ✅ `mapper.exists(w)` 或 `selectList(w.last("LIMIT 1"))`
-
-#### 🔴 A7d. Redis KEYS 命令
-```bash
-Grep pattern: "keysByPattern|\.keys\(" path: [目标目录] glob: "*.java"
-```
-- ❌ `redisTemplate.keys()` / `keysByPattern()` → ✅ 使用 Redisson `deleteByPattern()`（内部 SCAN + UNLINK）
-
-#### 🟡 A7e. Optional.of 误用
-```bash
-Grep pattern: "Optional\.of\(" path: [目标目录] glob: "*.java"
-# 排除 ofNullable 的命中
-```
-- ❌ `Optional.of(可能为null)` → ✅ `Optional.ofNullable(value)`
-
-#### 🟡 A7f. 布尔字段类型错误
-```bash
-Grep pattern: "private Integer (if|is|has)" path: [目标目录] glob: "*.java"
-```
-- ❌ `private Integer isEnabled` → ✅ `private Boolean enabled`
-
-#### 🟡 A7g. Wrapper 嵌套过深
-```bash
-# 人工审查：Read 文件时检查 Wrapper 嵌套是否超过 2 层
-```
-- ❌ Wrapper 嵌套 >2 层 → ✅ 迁移到 XML 原生 SQL
-
 #### 🟡 A8. 请求体封装
 ```bash
 Grep pattern: "@RequestBody [^L]" path: [目标目录] glob: "*Controller.java"

package/.claude/skills/fix-bug/SKILL.md

@@ -197,6 +197,62 @@ Agent(mysql-runner, "根据日志发现涉及 order_info 表，查询 id=xxx 的
 | Bug 描述 + DB 信息 | bug-analyzer + mysql-runner | — |
 | Bug 描述 + traceId + DB 信息 | 全部 | 日志中有新表/ID → mysql-runner 追加查询 |
 
+## SQL 报表 Bug 专项流程
+
+当 Bug 涉及 **SQL 查询逻辑错误**（SUM/GROUP BY/CASE WHEN/金额计算等），启用以下验证流程：
+
+### 修复前：建立基线
+
+```
+1. 读取当前 Mapper XML，理解现有 SQL 逻辑
+2. 如果有已修复的参考查询（如同模块的明细查询），先读取其 SQL 逻辑作为正确参考
+3. 查库获取当前错误结果作为"修复前基线"（可选，用户提供 DB 时执行）
+```
+
+### 修复后：交叉验证（必须执行）
+
+**两步验证法**（以本项目销售汇总修复为典型案例）：
+
+```
+步骤 1：直接执行修改后的 SQL，验证基本逻辑
+  - 检查各记录类型是否正确处理（称重/按份、正向/逆向、入库/出库）
+  - 检查特殊值：NULL 处理、除零保护、边界条件
+
+步骤 2：交叉对比验证
+  - 方法 A：用已修复的明细查询做 SUM 汇总
+  - 方法 B：执行修改后的汇总查询
+  - 对比 A vs B：总计必须完全一致
+  - 按维度对比（按商品/按门店等）：逐行 MATCH/DIFF
+```
+
+### 验证 SQL 模板
+
+```sql
+-- 交叉验证：明细汇总 vs 汇总查询
+SELECT '明细汇总' as source, SUM(saleNum), SUM(cost), ...
+FROM (/* 明细查询逐行 */) detail
+UNION ALL
+SELECT '汇总查询' as source, ...
+FROM /* 汇总查询 */;
+
+-- 按维度逐行对比
+SELECT COALESCE(d.name, s.name) as name,
+       d.detail_value, s.summary_value,
+       CASE WHEN d.detail_value = s.summary_value THEN 'MATCH' ELSE 'DIFF' END as result
+FROM (/* 明细按维度 GROUP BY */) d
+LEFT JOIN (/* 汇总按维度 GROUP BY */) s ON d.id = s.id;
+```
+
+### SQL 修改注意事项
+
+| 规则 | 说明 |
+|------|------|
+| **保留 WHERE 条件** | 新增/修改查询必须包含与现有查询相同的过滤条件 |
+| **GROUP BY 优先** | 优先添加字段到 GROUP BY，不用 ANY_VALUE() |
+| **JOIN 一致性** | 主查询和合计查询的 JOIN 必须一致（如 drp_unit） |
+| **IFNULL 保护** | 除法运算中的分母必须用 IFNULL 防止除零 |
+| **业务类型分支** | CASE WHEN 必须覆盖所有业务类型（称重/按份、入库/出库等） |
+
 ## 提交规则
 
 修复完成后，**必须调用 `Skill(git-workflow)` 再执行 git 操作**。
@@ -210,3 +266,4 @@ Agent(mysql-runner, "根据日志发现涉及 order_info 表，查询 id=xxx 的
 - 简单 Bug 不要过度编排，直接修就行（但仍需先报告再修复）
 - 如果用户没提供 DB/Loki 信息但 Bug 涉及数据问题，主动询问
 - 与 `bug-detective` 技能的区别：`bug-detective` 是排查指南，`fix-bug` 是全流程编排（包含排查+修复+提交）
+- SQL 报表 Bug 修复后**必须执行交叉验证**，不能只看"能跑"就提交

package/.claude/skills/leniu-code-patterns/SKILL.md

@@ -394,187 +394,10 @@ public class OrderService {
 }
 ```
 
-## 数据类型规范
-
-### 布尔字段命名
-
-```java
-// ❌ 错误：前缀冗余、类型错误
-private Integer ifNarrow;
-private Integer isEnabled;
-
-// ✅ 正确：Boolean 类型，无前缀
-private Boolean narrow;     // getter → isNarrow()
-private Boolean enabled;    // getter → isEnabled()
-```
-
-### 枚举字段标注
-
-```java
-// ❌ 错误：只靠注释 @see
-/** @see AccTradeTypeEnum */
-private Integer tradeType;
-
-// ✅ 方案一：VO/DTO 用枚举类型（配合 @JsonValue）
-private AccTradeTypeEnum tradeType;
-
-// ✅ 方案二：@ApiModelProperty 标注合法值
-@ApiModelProperty(value = "操作类型：1-充值 2-消费 3-退款", allowableValues = "1,2,3")
-private Integer tradeType;
-```
-
-## MyBatis-Plus 安全规范
-
-### selectOne 必须有唯一保障
-
-```java
-// ❌ 危险：多条记录时抛 TooManyResultsException
-Entity entity = mapper.selectOne(wrapper);
-
-// ✅ 方案一：LIMIT 1
-Entity entity = mapper.selectOne(wrapper.last("LIMIT 1"));
-
-// ✅ 方案二：selectList 取第一条
-List<Entity> list = mapper.selectList(wrapper);
-Entity entity = CollUtil.isNotEmpty(list) ? list.get(0) : null;
-
-// ✅ 方案三：确保有唯一索引（注释说明）
-// 唯一索引：UNIQUE KEY (order_no, del_flag)
-Entity entity = mapper.selectOne(wrapper);
-```
-
-### 存在性判断用 EXISTS，禁用 selectCount
-
-```java
-// ❌ 低效：100万行表 ~200ms
-Long count = mapper.selectCount(wrapper);
-if (count > 0) { ... }
-
-// ✅ 高效：~2ms（MyBatis-Plus 3.5.4+）
-boolean exists = mapper.exists(wrapper);
-
-// ✅ 或 selectList + LIMIT 1
-boolean exists = CollUtil.isNotEmpty(mapper.selectList(wrapper.last("LIMIT 1")));
-```
-
-### Wrapper 嵌套不超过 2 层
-
-```java
-// ❌ 过于复杂的 Wrapper
-wrapper.and(w -> w.eq(A::getType, type)
-    .or(q -> q.eq(A::getType, 0))
-    .or(!PersonTypeEnum.LABOUR.getKey().equals(type),
-        q -> q.eq(A::getType, PSN_TYPE_SHARED)));
-
-// ✅ 复杂查询写到 XML 中
-List<Entity> list = mapper.selectByTypeCondition(type, isLabour);
-```
-
-### 禁止 SELECT *
-
-```xml
-<!-- ❌ 禁止 -->
-<select id="selectAll">SELECT * FROM t_order WHERE del_flag = 2</select>
-
-<!-- ✅ 明确列出字段 -->
-<select id="selectAll">SELECT id, order_no, amount, status, crtime FROM t_order WHERE del_flag = 2</select>
-```
-
-## Redis 使用规范
-
-### 禁止 KEYS 命令
-
-```java
-// ❌ 严禁：KEYS 阻塞 Redis
-Set<Object> keys = keysByPattern(pattern);
-Set<Object> keys = redisTemplate.keys(pattern);
-
-// ✅ 使用 Redisson deleteByPattern（内部 SCAN + UNLINK）
-RedissonClient redisson = SpringUtil.getBean(RedissonClient.class);
-redisson.getKeys().deleteByPattern(keyPattern);
-```
-
-## Optional 使用规范
-
-```java
-// ❌ 错误：of() 不接受 null
-Optional.of(value).orElse(defaultValue);
-
-// ✅ 正确：ofNullable()
-Optional.ofNullable(value).orElse(defaultValue);
-
-// ✅ 链式安全转换
-Optional.ofNullable(model.getReserveRate())
-    .map(BigDecimal::new)
-    .orElse(BigDecimal.ZERO);
-
-// ❌ 禁止作为方法参数或类字段
-// ✅ 允许作为方法返回值
-```
-
-## @Transactional 规范
-
-```java
-// ❌ 默认只回滚 RuntimeException
-@Transactional
-public void createOrder() { ... }
-
-// ✅ 显式指定 rollbackFor
-@Transactional(rollbackFor = Exception.class)
-public void createOrder() { ... }
-```
-
-- 事务方法不要 try-catch 吞掉异常
-- 只读查询不加 `@Transactional`
-
-## 业务逻辑分层规范
-
-```java
-// ❌ 错误：业务判断混在数据操作中
-public void processOrder(Long orderId) {
-    OrderInfo order = orderMapper.selectById(orderId);
-    if (order.getStatus() == 1 && order.getPayTime() != null
-        && ChronoUnit.HOURS.between(order.getPayTime(), LocalDateTime.now()) < 24) {
-        order.setStatus(2);
-        orderMapper.updateById(order);
-        accWalletService.deduct(order.getCustId(), order.getAmount());
-    }
-}
-
-// ✅ 正确：分层清晰
-public void processOrder(Long orderId) {
-    OrderInfo order = orderMapper.selectById(orderId);
-    if (ObjectUtil.isNull(order)) {
-        throw new LeException(I18n.getMessage("order_not_found"));
-    }
-    checkCanProcess(order);           // 业务校验（独立方法）
-    order.markAsProcessed();          // 状态变更（Entity 方法封装）
-    orderMapper.updateById(order);
-    afterOrderProcessed(order);       // 后续动作（独立方法）
-}
-```
-
-| 层 | 职责 | 不应做的 |
-|----|------|---------|
-| Controller | 参数接收、格式转换 | 不含业务判断 |
-| Business | 业务编排、跨 Service 协调 | 不直接操作 Mapper |
-| Service | 单表 CRUD、单表事务 | 不含跨表业务逻辑 |
-| Mapper | SQL 映射 | 不含业务逻辑 |
-
-## TODO 管理规范
-
-```java
-// ❌ 错误
-// TODO 修改一下
-
-// ✅ 正确
-// TODO(@陈沈杰, 2026-03-20, #TASK-1234): 移动端 AppId 赋值逻辑待产品确认
-```
-
-- 不用的代码直接删除，不要注释保留
-
 ## 通用代码规范
 
+无论使用哪种项目架构，以下规范都是通用的：
+
 1. **禁止使用 `SELECT *`**：明确指定字段
 2. **使用参数化查询**：`#{}` 而非 `${}`
 3. **异常必须处理**：不能吞掉异常

package/.claude/skills/leniu-crud-development/SKILL.md

@@ -39,7 +39,6 @@ description: |
 | 架构 | Controller -> Business -> Service -> Mapper（四层） |
 | 无 DAO 层 | Service 直接注入 Mapper |
 | 对象转换 | `BeanUtil.copyProperties()` (Hutool) |
-| Service 模式 | **两种并存**：简单 CRUD 继承 `ServiceImpl`；业务聚合直接 `@Service` |
 | Entity 基类 | 无基类，自定义审计字段 |
 | 请求封装 | `LeRequest<T>` |
 | 响应封装 | `Page<T>` / `LeResponse<T>` / `void` |
@@ -111,38 +110,19 @@ private LocalDateTime uptime;
 private Integer delFlag;  // 1=删除, 2=正常
 ```
 
-### Service 两种模式
+### Service 注入模式
 
-项目中存在两种 Service 模式，根据业务复杂度选择：
-
-**模式 A：简单 CRUD Service**（适用于单表操作，利用 MyBatis-Plus 内置方法）
-```java
-// 接口
-public interface XxxService extends IService<XxxEntity> { }
-
-// 实现
-@Slf4j
-@Service
-public class XxxServiceImpl extends ServiceImpl<XxxMapper, XxxEntity> implements XxxService {
-    // 继承 ServiceImpl 获得 save/updateById/removeById/page 等内置方法
-    // 通过 this.baseMapper 访问 Mapper（父类提供）
-}
-```
-
-**模式 B：业务聚合 Service**（适用于跨表操作、复杂业务编排）
 ```java
 @Slf4j
 @Service
-public class XxxService {
-    @Autowired
+public class XxxServiceImpl implements XxxService {
+    @Resource
     private XxxMapper xxxMapper;  // 直接注入 Mapper，无 DAO 层
-    @Autowired
-    private YyyMapper yyyMapper;  // 可注入多个 Mapper
+
+    // 不继承 ServiceImpl，只实现接口
 }
 ```
 
-**选择建议**：新建简单单表 CRUD 用模式 A；涉及多表联查、报表、业务编排用模式 B。
-
 ### Controller 请求封装
 
 ```java
@@ -386,6 +366,7 @@ private String createBy;                  // -> crby
 entity.setDelFlag(0);                     // -> setDelFlag(2) 表示正常
 throw new ServiceException("...");        // -> throw new LeException("...")
 MapstructUtils.convert(src, Dst.class);   // -> BeanUtil.copyProperties(src, Dst.class)
+extends ServiceImpl<XxxMapper, Xxx>       // -> implements XxxService（不继承）
 @Resource private XxxDao xxxDao;          // -> @Resource private XxxMapper xxxMapper
 // XML 放 resources/mapper/              // -> 与 Java 同目录
 return null;                              // -> return Collections.emptyList()
@@ -396,7 +377,7 @@ return null;                              // -> return Collections.emptyList()
 ## 生成前检查清单
 
 - [ ] 包名 `net.xnzn.core.*`
-- [ ] Service 模式选择：简单 CRUD 用模式 A（继承 ServiceImpl）；业务聚合用模式 B（直接 @Service）
+- [ ] Service 只实现接口，不继承基类
 - [ ] Service 直接注入 Mapper（无 DAO）
 - [ ] 审计字段 crby/crtime/upby/uptime
 - [ ] delFlag: 1=删除, 2=正常