6aspec 3.0.0-dev.16 → 3.0.0-dev.19

package/.6aspec/rules/brown/brown_implement_sop.md

@@ -40,12 +40,29 @@
    - 各里程碑完成度（M0~Mn：已完成/总数）
    - 下一个可执行任务
 
+   **建立任务列表（外部状态锚点）**：
+
+   为每个待执行的 TASK，依次调用 `TaskCreate` 建立两个子任务，顺序为：
+   ```
+   TASK-<序号>: 实现
+   TASK-<序号>: Compliance Review
+   ```
+   例如有 TASK-001、TASK-002 待执行，则建立：
+   - `TASK-001: 实现`
+   - `TASK-001: Compliance Review`
+   - `TASK-002: 实现`
+   - `TASK-002: Compliance Review`
+
+   > 目的：任务列表作为外部状态锚点，防止 implementer subagent 返回后因"完成感"跳过 compliance review 步骤。
+
 4. **执行任务（循环直到完成或阻塞）**
 
    对每个待完成的任务，按以下流程执行：
 
    **a. 派发 implementer agent**
 
+   调用前：将对应的 `TASK-<序号>: 实现` 任务标记为 `in_progress`。
+
    调用 `6aspec-brown-implementer` agent，传入以下内容：
 
    ```
@@ -61,14 +78,16 @@
    ```
 
    **处理 agent 状态**：
-   - `DONE`：进入 spec compliance review
-   - `DONE_WITH_CONCERNS`：读取关注点，判断是否影响正确性；若影响则先处理，否则进入 review
+   - `DONE`：将 `TASK-<序号>: 实现` 标记为 `completed`，进入 spec compliance review
+   - `DONE_WITH_CONCERNS`：读取关注点，判断是否影响正确性；若影响则先处理，否则将 `TASK-<序号>: 实现` 标记为 `completed`，进入 review
    - `NEEDS_CONTEXT`：补充缺失信息后重新调用
    - `BLOCKED`：暂停，向用户报告阻塞原因和选项
    - **agent 开始前提问**：若 agent 在实现前提出疑问，主 agent 回答后重新调用（携带原有上下文 + 补充答案）
 
    **b. 派发 spec compliance reviewer agent**
 
+   调用前：将对应的 `TASK-<序号>: Compliance Review` 任务标记为 `in_progress`。
+
    调用 `6aspec-brown-spec-compliance-reviewer` agent，传入以下内容：
 
    ```
@@ -84,7 +103,7 @@
    ```
 
    **处理 review 结果**：
-   - `✅ 通过`：标记任务完成，继续下一个任务
+   - `✅ 通过`：将 `TASK-<序号>: Compliance Review` 标记为 `completed`，标记任务完成，继续下一个任务
    - `❌ 有问题`：重新调用 6aspec-brown-implementer agent 修复，传入**相同的 `skipTest：true|false` 行** + 原有上下文 + reviewer 的完整反馈（具体缺失/多余/偏差及文件路径）；修复后再次调用 reviewer；**最多循环 3 次**
    - **超过 3 次仍未通过**：暂停，向用户报告每轮的问题和修复情况，让用户决定如何处理
 

package/.6aspec/rules/brown/subagents/implementer.md

@@ -42,6 +42,10 @@
      - 若本任务引入/修改**可执行业务逻辑**（如 Service/Domain/校验/计算/流程编排）或修复缺陷：在项目有测试框架时补充/修改对应单元测试，覆盖任务关联的 AC/Scenario；若无测试框架则说明验证方式与回归点  
      - 若本任务主要为纯数据结构、**SQL/DDL/迁移脚本**、或**薄持久化**（见上）：默认不强制单元测试；改为提供可审查的验证清单，并在报告中标注风险  
      - **禁止**：把「本任务动到了 Repository/Mapper」**单独**当成必须写单测的理由；是否写单测取决于是否出现**非平凡逻辑**（分支、规则、补偿、复杂 SQL 行为），以及 TASK/specs 是否点名要自动化测试
+
+   **🚨 测试红线（绝对禁止）**：
+   - **禁止在测试代码中重新实现被测逻辑**：不得在测试方法里自己写 stream filter、循环、条件判断等来"模拟"业务行为后再断言结果——这类测试无论业务代码是否正确都会通过，没有任何验证价值。
+   - **测试必须调用被测类的真实方法**：通过断言其返回值，或通过 mock verify 断言其对依赖的调用行为来验证。合法结构示例：`given(mock).thenReturn(x)` → 调用 `service.realMethod(...)` → `assertEquals(expected, result)` 或 `verify(mock).someMethod(...)`。
 7. 完成后自我审查，报告结果
 
 ## 自我审查
@@ -54,7 +58,10 @@
 
 **一致性**：是否遵循了现有代码风格、`code.md` 与 design.md 的技术决策？
 
-**分层**：若本次改动涉及 Controller/API、事件订阅、消息消费或调度入口，是否符合 `code.md` **架构红线**中关于薄入口、不直连 DAO 与委托 Service 的约定（有 TASK/design 例外除外）？
+**分层**：若本次改动涉及 Controller/API、事件订阅、消息消费或调度入口，执行以下客观计数检查（有 TASK/design 明确例外除外）：
+- 数一数入口类核心方法中对**外部系统**（Gateway、Repository、其他 Service）的调用次数：**超过 1 次即违规**，说明编排逻辑未下沉到 Service。
+- 检查入口类是否存在**私有业务方法**（含 stream 聚合、多分支判断、远程查询封装等）：有则违规，这些逻辑应在 Service 中实现。
+- 合法的入口方法结构：解析入参 → 日志 → 调一次 Service → 返回结果。不符合此结构须重构后再报告。
 
 **测试/验证**：
 - 若 **`skipTest=true`**：是否已说明验证方式、回归点与残余风险？若 TASK/specs 强制要求测试而与参数冲突，是否已上报或已按主 agent 取舍执行？

package/.6aspec/rules/common/code-reviewer-agent.md

@@ -35,15 +35,26 @@ git diff {BASE_SHA}..{HEAD_SHA}
 ```
 
 3) **否则，如果未提供 SHA 但 git 可用**：
+
+收集工作区所有未提交的变更（不包含已 commit 的代码）：
+
 ```bash
-BASE_SHA=$(git rev-parse HEAD~1)
-HEAD_SHA=$(git rev-parse HEAD)
-git diff --stat ${BASE_SHA}..${HEAD_SHA}
-git diff ${BASE_SHA}..${HEAD_SHA}
+# 已追踪文件的修改（未 add）
+git diff
+
+# 已 add 但未 commit 的变更
+git diff --cached
+
+# 未追踪的新文件（未 add），转成 unified diff 格式
+git ls-files --others --exclude-standard | while read f; do
+  diff -u /dev/null "$f" || true
+done
 ```
 
-4) **否则（既没有 DIFF，也无法使用 git）**：
-- 返回 `NEEDS_CONTEXT`，并要求调用方提供 `DIFF`（unified diff），或提供明确的变更文件列表 + 变更片段。
+将以上三部分合并为完整 diff 进行审查。如果三部分均为空，提示用户"当前工作区没有任何未提交的变更"。
+
+4) **否则（git 不可用）**：
+- 提示用户"当前目录不是 git 仓库，请通过 --diff 参数提供 unified diff 文件"。
 
 ### 2. 读取需求（如果提供了文件路径）
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "6aspec",
-  "version": "3.0.0-dev.16",
+  "version": "3.0.0-dev.19",
   "description": "6Aspec - 轻量级 spec 驱动开发框架，支持 Cursor 和 Claude Code",
   "main": "lib/installer.js",
   "bin": {