@zat-design/sisyphus-scene 4.6.3-beta.2 → 4.6.3

package/skills/sisyphus-scene/references/approved-examples.yaml

@@ -7,7 +7,7 @@ examples: []
 #   status: approved
 #   application: xk-wjs-claim-web
 #   capabilities: [search-form, data-table, table-tabs, drawer-form]
-#   sisyphus_version: "4.6.3-beta.2"
+#   sisyphus_version: "4.6.3"
 #   antd_range: ">=6 <7"
 #   paths:
 #     - src/pages/litigationManagement/index.tsx

package/skills/sisyphus-scene/references/platform-claude.md

@@ -7,3 +7,10 @@
 - Ant Design MCP 不可用时，通过 Context7 查询 antd 6 官方文档。
 - 读取项目 `CLAUDE.md`，同时显式检查 `AGENTS.md` 和 `.cursor/rules`；同级冲突时停止。
 - 不使用 Claude 专属 frontmatter 驱动共享核心行为，保证同一 SKILL.md 可被其他平台读取。
+
+## 规范门卫
+
+- 明确“审查 + 路径”时直接进入 `path-review`，不调用 Scene 生成和 MCP 链路；仍使用平台原生独立只读子代理。
+- `validate-scaffold.mjs` 通过后，使用平台原生的子代理能力创建独立只读审查子代理，并加载 `references/review-guard.md`。
+- 向子代理传入用户需求与豁免摘要、`SceneSpec`、本次文件清单、项目根目录、目标目录、`ProjectProfile`、`ComponentDecision` 和审查轮次；不传入修改权限。
+- 无法调用平台原生子代理时阻断交付，不得降级为主 Agent 自审。

package/skills/sisyphus-scene/references/platform-codex.md

@@ -8,3 +8,10 @@
 - `agents/openai.yaml` 只承载 UI 元数据。当前 Sisyphus MCP 是本地 stdio 命令，没有可验证的 streamable HTTP URL，因此不伪造 Codex MCP dependency。
 - 读取最近层级的 `AGENTS.md`，并显式检查 `CLAUDE.md` 和 `.cursor/rules`。
 - 修改已有代码符号前遵循项目 GitNexus impact 约束，完成后执行 detect changes。
+
+## 规范门卫
+
+- 明确“审查 + 路径”时直接进入 `path-review`，不调用 Scene 生成和 MCP 链路；仍使用平台原生独立只读子代理。
+- `validate-scaffold.mjs` 通过后，使用平台原生的子代理能力创建独立只读审查子代理，并加载 `references/review-guard.md`。
+- 向子代理传入用户需求与豁免摘要、`SceneSpec`、本次文件清单、项目根目录、目标目录、`ProjectProfile`、`ComponentDecision` 和审查轮次；明确其只读职责，不授予修改权限。
+- 无法调用平台原生子代理时阻断交付，不得降级为主 Agent 自审。

package/skills/sisyphus-scene/references/platform-cursor.md

@@ -7,3 +7,10 @@
 - 读取并应用 `.cursor/rules/*.mdc`，同时检查项目根 `AGENTS.md` 与 `CLAUDE.md`。
 - Cursor Rules 是持久项目规范，Skill 是按需工作流；不要把整份团队规范复制进 SKILL.md。
 - MCP 工具未出现或缓存版本与目标包不一致时，停止并要求刷新 MCP，不使用旧缓存继续生成。
+
+## 规范门卫
+
+- 明确“审查 + 路径”时直接进入 `path-review`，不调用 Scene 生成和 MCP 链路；仍使用平台原生独立只读子代理。
+- `validate-scaffold.mjs` 通过后，使用平台原生的子代理能力创建独立只读审查子代理，并加载 `references/review-guard.md`。
+- 向子代理传入用户需求与豁免摘要、`SceneSpec`、本次文件清单、项目根目录、目标目录、`ProjectProfile`、`ComponentDecision` 和审查轮次；门卫只返回审查证据和修复计划。
+- 无法调用平台原生子代理时阻断交付，不得降级为主 Agent 自审。

package/skills/sisyphus-scene/references/review-guard.md

@@ -0,0 +1,151 @@
+# Sisyphus CR 规范门卫
+
+## 来源与定位
+
+- 来源：`sisyphus-cr@1.5.0`，快照日期为 2026-08-20；源 `SKILL.md` SHA-256 为 `1b9730edad78a754de2320938dc7969d0f0b8084781933d802a75af5a793c34d`，源 `CHANGELOG.md` SHA-256 为 `ed2a11215d7894e70584cd383828d1cf2d6d5764631b624c198d12a90a2afae0`，源审查基线 SHA-256 为 `361e3d818a7ed07677db5c9415d244bd5cc895ffccf6d262ef51a671a49f38e3`。
+- 本文件是 Scene 包内可独立分发的规则副本，不在运行时查找或调用用户全局 `sisyphus-cr`。
+- `references/review-standard.md` 是同一快照的完整通用基线；门卫必须完整读取 `references/review-standard.md`，不能只依赖本文件的核心摘要。
+- 门卫必须由独立只读子代理执行。门卫只审查并给出修复计划；主 Scene Agent 才能修改代码。
+- 门卫补充 `validate-scaffold.mjs` 难以判断的语义和调用链问题，不重复替代版本、组件证据、目录和函数语法等确定性校验。
+
+## 输入契约
+
+主 Scene Agent 必须先声明 `reviewMode`。两种模式共享项目根目录、目标路径、本次文件清单、用户需求与豁免摘要、完整 `ProjectProfile` 和规则来源。
+
+- `generation-gate`：额外提供本轮 `SceneSpec`、全部 `ComponentDecision`、当前审查轮次，以及复审时的上一轮问题和修复摘要。
+- `path-review`：提供用户明确指定的文件或目录绝对路径；不建立 `SceneSpec`，不需要 `ComponentDecision`，审查轮次固定为 `0`。
+
+输入缺失、路径不存在或不可读时，结论为“无法判断”。不得通过扩大范围或猜测内容补齐证据。
+
+## 范围与权限
+
+- 只报告本次生成或修改的文件清单内的问题。
+- `path-review` 的文件目标只报告该文件问题；目录目标递归审查目录内源码，并应用相同忽略目录规则。
+- 可以读取清单中文件的直接 import、类型、配置、项目指令、相邻 service 和统一请求层，以证明规则、API 或调用关系；不得把问题扩大到清单之外。
+- 忽略 `.git`、`node_modules`、`dist`、`build`、`coverage`、`.umi`、缓存和生成目录。
+- 严禁编辑、创建或删除文件，严禁生成补丁，严禁运行格式化、`lint --fix`、安装、构建或浏览器验收。
+- 不得自动进入修复阶段，不得把审查任务转交回主 Agent，也不得降级为主 Agent 自审。
+
+## 规则优先级
+
+冲突时按以下顺序决定适用规则：
+
+1. 用户本次明确提出的需求和豁免。
+2. 本门卫的 Sisyphus 核心边界。
+3. 目标路径最近的 `AGENTS.md`、`CLAUDE.md`、`.cursor/rules` 和它们引用的完整规范。
+4. 目标项目 `package.json`、TypeScript 配置，以及实际安装版本的 Sisyphus/Ant Design 类型与 API。
+5. `references/review-standard.md` 的完整通用审查基线。
+
+至少通过以下一项确认目标属于 Sisyphus 项目：最近 `package.json` 声明 `@zat-design/sisyphus-react`、清单内源码直接导入该包，或项目规范明确要求使用该组件库。证据不足时结论为“无法判断”。
+
+## 核心审查规则
+
+### 目录与职责
+
+- 先枚举页面或组件根目录中的非入口源码，沿导出、直接依赖和调用方判定职责。纯函数工具、DTO/payload 转换、解析与序列化、克隆、布局和历史计算进入 `utils/`；类型、配置/注册表、服务、组件和 Context 进入各自责任层。不得以根目录 `utils.ts` 或业务化文件名规避。
+- React 组件、状态 Hook、请求函数以及项目明确独立的协议/基础设施模块不机械移入 `utils/`；混合文件只拆分工具职责。
+- 普通页面或组件模块不建立模块私有 `constants` 或 `constants.ts`。固定选项、业务码值和静态配置并入 `config`；只在一个实现中使用的常量留在责任文件；只有被多个模块真实复用时才上收到共享常量层。
+- `index.tsx` 保持编排职责，不承载大段静态配置、传输定义或 DTO 清洗。
+- 页面独占接口进入所属 `services`，纯 payload 转换进入 `utils`；不为形式完整创建空目录。
+- 子组件、样式、类型文件和 barrel 遵循 `ProjectProfile.fileStructureAdapter` 及其证据文件。
+
+### 状态、loading 与自有组件 API
+
+- ahooks `useRequest`、`useAntdTable` 等已经提供的请求 loading 必须直接消费，不得复制到本地 state 后手工切换。
+- loading 或其他非函数业务输入跨自有组件传递时必须收敛到对象型 `dataSource`；接收组件再把它传给内部第三方组件的公开 loading API。
+- antd、Sisyphus 等第三方组件保持公开的 `loading`、`spinning`、`confirmLoading` API，不机械包成 `dataSource`。
+- `form`、`mode`、`open`、`disabled`、`value` 等控制属性及 `onXxx` 回调保持平铺。
+- 非请求业务 loading 允许存在，但仍需根据来源、切换者和消费点判断，不得只按变量名判错。
+
+### 表单、表格与配置
+
+- Sisyphus columns 的字段值变化回调使用 `fieldProps.onFieldChange`；直接控件的公开 `onChange` 保持不变。
+- 简单必填只配置 `required: true`；自定义校验器、正则、长度和跨字段约束等复杂校验继续使用 `rules`。
+- columns 字段已有 `label` 时不重复配置默认 placeholder；用户要求、项目规则或特殊业务语义明确时保留。
+- 最终由 `ProForm`、`ProDrawerForm` 等 Sisyphus 表单消费的实例必须由 `ProForm.useForm()` 创建；只由纯 antd Form 消费时保留 `Form.useForm()`。
+- 表单字段不在页面 state 中镜像第二份值；可派生数据保持为派生值。
+- 含 JSX、操作按钮、页面回调、权限或运行时 loading 的表格列留在消费页面；`config` 只保存可复用的基础列，操作列不得进入配置工厂。
+- 固定配置和动态配置工厂遵循项目命名及类型规则，不因个人偏好制造问题。
+
+### 服务、类型与数据适配
+
+- 请求参数、响应和 DTO 使用明确类型，不用 `any` 绕过契约；纯类型使用 `import type`。
+- service 只表达传输契约，不引用 Form、message 或页面状态，不吞异常或伪装成功。
+- 项目要求 service 注释时，JSDoc、导出声明和相邻 service 块分别独占行且块间保留空行。
+- 动态 URL 片段使用 `encodeURIComponent`；标识字段按后端契约传递，不因类型相同而混用。
+- payload 转换移除界面临时字段，并清理当前条件下不应提交的互斥字段。
+
+### 错误处理与异步安全
+
+- 接口失败统一展示响应中的 `message`。统一请求层已经反馈时，页面或组件不得再追加 `message.error`、`notification.error` 等重复失败提示。
+- 统一请求层没有负责错误反馈时，页面保留可观察的真实失败反馈是合理场景，不得误报。
+- 保存、删除或状态更新只在真实成功后关闭界面、提示成功并刷新。
+- 弹层可快速关闭或切换记录时，旧请求不得覆盖新视图；连续提交或状态切换必须防止重复请求。
+- 禁止空 catch、静默失败、裸 `void promise()` 掩盖异步结果，以及失败后伪装成功。
+
+### 导入、样式与测试
+
+- import 分组、路径别名、类型导入、类名风格和测试目录以目标项目最近规范为准。
+- 测试覆盖 payload 适配、动态列条件、service 契约、竞态保护和高风险操作门禁等可观察行为；只匹配标题、注释或实现措辞不能证明行为正确。
+
+## 证据与误报控制
+
+- 每个问题必须包含清单内真实的 `文件:行号`、适用规则、证据和可观察影响。
+- 文本命中只能证明结构事实；loading、表单、错误提示和异步问题必须沿实际调用链确认来源与消费点。
+- 同一根因的多个现象合并为一个问题，不按命中次数重复计数。
+- 不推广样板页面的字段、接口、字典、文案、分页、固定宽度或弹层数量。
+- 第三方 loading、纯 antd Form、真实共享常量、非请求业务 loading，以及没有全局错误反馈时的页面失败提示都是必须保留的合理反例。
+
+## 问题分级
+
+- `P0`：可能造成错误提交、数据丢失、竞态覆盖或高风险操作失控。
+- `P1`：违反明确项目规范、实际组件 API、传输契约或职责边界。
+- `P2`：不影响当前正确性，但偏离适用的 Sisyphus 标杆并增加维护成本。
+
+不得把纯个人偏好列为问题。没有明确规则或基线不适用时标记为不适用，不进入问题清单。
+
+## 唯一输出格式
+
+最终回答只能包含以下三个一级部分，不增加其他一级标题：
+
+### 规范审查
+
+先以独占一行的 `门卫元数据：<JSON>` 输出机器元数据，再写审查说明，并以独占一行的 `结论：<值>` 输出结论。值只能是：`通过`、`部分通过`、`不通过`、`无法判断`。
+
+元数据必须符合：
+
+```text
+门卫元数据：{"reviewId":"CRG-<唯一编号>","reviewer":"sisyphus-cr-readonly-subagent","reviewMode":"<generation-gate|path-review>","ruleVersion":"sisyphus-cr@1.5.0","reviewRound":<0|1|2>,"targetPath":"<目标绝对路径>","files":["<本次文件绝对路径>"],"ruleSources":["<规则来源>"],"sisyphusEvidence":"<Sisyphus 识别证据>"}
+```
+
+`reviewRound` 必须与当前轮次一致；`targetPath` 和 `files` 必须位于项目根目录内，文件清单不能为空。不得输出缺少真实目标、规则来源或 Sisyphus 证据的空壳“通过”报告。
+
+### 问题清单
+
+按 `P0 → P1 → P2` 排序：
+
+| 编号 | 优先级 | 文件:行号 | 违反规则 | 证据与影响 |
+| --- | --- | --- | --- | --- |
+
+问题使用稳定编号 `CR-001`、`CR-002`。未发现问题时只写“未发现规范问题”。
+
+### 修复计划
+
+按执行依赖排序，每一步关联问题编号，写明修改目标、调整方向和验证方式。计划必须保持现有路由、接口、权限、公开组件 API 和无关业务行为。通过时写“无需修复”；无法判断时写“证据不足，暂不制定修复计划”。
+
+## 主 Agent 修复闭环
+
+1. `validate-scaffold.mjs` 通过后，主 Agent 创建独立只读子代理并传入上述完整输入。
+2. 把门卫三段式输出保存到任务临时目录，并运行 `node scripts/evaluate-review-gate.mjs --mode generation-gate --root <项目根目录> --report-root <任务临时目录> --report <报告文件> --round <0|1|2>`。
+3. 只有门禁脚本返回 `status: passed` 才算门卫通过；输出自相矛盾、缺少固定段落或缺少结论都会阻断。
+4. 门禁脚本返回 `status: repair` 时，主 Agent 按问题编号做最小修复；修复权限只覆盖本次文件清单，并使用脚本返回的 `nextRound`。
+5. 每次修复后重新运行 `validate-scaffold.mjs`，再创建新的独立只读子代理复审，不能复用旧结论。
+6. 自动修复最多 2 轮。门禁脚本返回 `status: blocked` 时立即停止；任何 P0、P1、P2 遗留问题和“无法判断”都会进入该状态。
+7. 修复若需要改动文件清单之外的源码、公共 API、路由、权限、依赖或全局布局，立即停止并报告，不自动扩大权限。
+8. 门卫通过后才运行目标项目 typecheck、lint、相关测试、页面验收和 GitNexus `detect_changes`。
+
+三种允许的闭环结果：首次审查通过；第一或第二轮修复后复审通过；两轮后仍不通过并停止报告。不存在带 P2 问题继续交付或平台无法创建子代理时降级自审的路径。
+
+## 指定路径只读审查
+
+`path-review` 只执行一次独立只读审查：解析并验证用户指定路径，读取最近项目规则、依赖、类型和必要调用链，输出三段式报告，再运行 `evaluate-review-gate.mjs --mode path-review --round 0` 校验报告身份、规则版本和范围。合法报告返回 `status: reviewed`；无论发现 P0、P1 或 P2，都不进入 repair、不修改文件。路径缺失、不可读或缺少 Sisyphus 识别证据时输出“无法判断”。

package/skills/sisyphus-scene/references/review-standard.md

@@ -0,0 +1,130 @@
+# Sisyphus 代码审查完整基线
+
+本文件是 `sisyphus-cr@1.5.0` 在 2026-08-20 的包内规则快照，源 `sisyphus-review-standard.md` SHA-256 为 `361e3d818a7ed07677db5c9415d244bd5cc895ffccf6d262ef51a671a49f38e3`。目标项目没有更具体规则时使用；它不替代目标路径最近的项目指令，也不把某个样板页面的业务字段、接口或交互细节推广为通用规范。
+
+## 规则等级
+
+- **项目硬规则**：目标项目文档、类型声明或 API 明确要求。违反时通常为 `P1`，影响数据正确性时为 `P0`。
+- **适用基线**：目标代码具备对应职责且项目没有相反约定时使用。违反时按实际影响评为 `P1` 或 `P2`。
+- **标杆建议**：仅改善一致性或维护性，最多为 `P2`。
+- **不适用**：目标没有对应职责，或项目明确采用其他合理方案，不进入问题清单。
+
+## 1. 项目识别与审查边界
+
+- 必须先确认目标使用 `@zat-design/sisyphus-react`，并读取最近项目规则和实际依赖版本。
+- 组件 API 以目标项目的类型声明和文档为准，不能凭记忆套用旧版 Sisyphus 或 Ant Design API。
+- 只报告文件清单内的问题。清单外文件可用于证明调用关系、类型契约或规则来源，不扩大整改范围。
+- 生成目录、构建产物、依赖目录和缓存不参与源码规范审查。
+
+## 2. 页面目录与职责分层
+
+- 查询列表页按职责拆分入口、配置、服务、页面类型和测试；不为形式完整创建未使用的空目录。
+- `index.tsx` 是编排层，不承载大段静态配置、传输定义或 DTO 清洗。
+- 页面或组件根目录中的非入口源码必须按实际内容归位：纯函数工具、DTO/payload 转换、解析与序列化、克隆、布局、历史计算等进入 `utils/`；页面共享类型进入最近项目约定的类型层；静态配置、选项和注册表进入 `config`；请求进入 `services`；React 组件与 Context 分别进入组件和上下文责任层。不得用根目录 `utils.ts`、`flowHelper.ts` 或业务化文件名平铺工具实现，也不得因文件只有一个调用方而豁免归位。
+- 普通页面或组件模块不建立模块私有 `constants`。本模块固定选项、业务码值和静态配置并入 `config`；只在单一实现使用的常量留在责任文件；被多个模块真实共享时才上收到项目共享层。基础设施、协议、全局 API 路径和统一码表不适用此限制。
+- 请求函数放在所属页面或领域的 `services`；页面独占接口不被其他页面跨私有目录引用，两处以上真实复用时再上收。
+- ProForm、ProTable、ProEditTable 的基础配置放在 `config`；纯 payload 适配和业务判定放在 `utils`；页面共享类型放在统一类型文件。
+- 新增或重构的自有子组件使用同名目录和 `index.tsx`，样式就近放置。组件聚合出口使用显式命名导出；项目要求场景注释时，每个导出紧邻真实用途说明，不使用无说明的 `export *`。
+- 目录名或文件长度不能单独证明问题，必须结合模块职责和引用关系判断。
+- 只在目标确实是业务页面或发生重构时应用完整目录基线；小型叶子组件不要求页面全套目录。
+- 先枚举模块根目录文件，再读取其导出、直接依赖和调用方判定职责；文件名只能发现候选，不能单独证明它是入口、配置或工具。业务化命名文件若主要导出无副作用纯函数，仍按工具文件审查。
+- 不把 React 组件、状态 Hook、请求函数或项目明确要求独立存在的协议/基础设施模块机械移动到 `utils/`；混合文件只拆分其中的工具职责，其余内容保留在对应责任层。
+- “文件较长”本身不是问题，只有出现职责混杂、跨层依赖或难以独立验证时才列项。
+
+## 3. 自有组件 API 与职责边界
+
+- 自有组件的非函数业务数据收敛到对象型 `dataSource`。
+- ahooks `useRequest`、`useAntdTable` 等已经提供的请求 loading 必须直接消费，不得复制进本地 state 后手工切换。
+- loading 作为非函数输入跨自有组件传递时，必须随关联数据收敛到 `dataSource`；接收组件可以再传给第三方公开 loading API。
+- antd、Sisyphus 等第三方组件保持其 `loading`、`spinning`、`confirmLoading` API。
+- 不由请求 Hook 管理的本地计算、复合流程或逐行操作锁仍需按真实来源审查，不机械归类为请求 loading。
+- `form`、`mode`、`open`、`disabled`、`value`、`defaultValue` 等控制属性保持平铺；回调以 `onXxx` 命名并平铺。
+- 页面拥有请求、保存、删除、反馈、刷新和跨弹层协调；子组件拥有展示或表单装配并通过事件通知页面。
+- 含 JSX、操作按钮、页面回调、权限或运行时 loading 的列留在消费页面；`config` 只保留可复用基础列，操作列不得进入配置工厂。
+- 先追溯 loading 的真实来源和切换者，并检查所有消费点，不能只按变量名或声明位置判断；镜像字段和手工切换代码合并为同一根因。
+- 根据导入来源区分自有组件与第三方组件：相对路径、项目别名或项目内部模块按自有组件审查，外部依赖包按第三方公开 API 审查。
+- 直接把 Hook loading 传给第三方组件不列问题；作为平铺 prop 传给自有组件则违反组件 API 边界，即使没有镜像本地 state。
+- 非请求业务忙碌状态跨自有组件时仍收敛到 `dataSource`，但不得机械改成请求 Hook loading。
+- 新增等列表级动作优先放入 ProTable 工具栏能力，避免页面标题区和表格区重复表达同一动作。
+- 编辑表单适合抽屉时优先使用 ProDrawerForm 的标准提交与取消能力；只读详情按信息层级选择容器，不强制所有页面使用同一种弹层。
+
+## 4. 状态、表单与副作用
+
+- 同一生命周期内两个及以上相关本地状态使用项目约定的合并状态方案；采用 `useSetState` 时使用能表达页面职责的状态类型。
+- 单一 primitive、第三方专用 hook、Form 实例和不驱动渲染的 ref 不为形式统一强行合并。
+- ProTable 列表、分页与 loading 由 `useAntdTable` 或等价方案管理；详情和辅助查询使用 `useRequest({ manual: true })` 或等价方案。
+- 同一请求不得同时使用 Hook loading 和页面 state 中手工切换的 loading；请求生命周期由 Hook 收敛，页面只处理业务数据、竞态结果和必要界面状态。
+- 表单字段由 Form 管理，不在页面 state 中镜像第二份字段值。
+- 最终由 Sisyphus 表单消费的实例使用 `ProForm.useForm()`；只有最终由纯 antd Form 消费时才保留 `Form.useForm()`。确认误用且无高优先级豁免时按 `P2` 报告并计划改为 `ProForm.useForm()`。
+- 不驱动渲染的请求序号、重复提交锁和进行中 key 集合放在 ref 或等价非渲染状态中。
+- effect 集中在组件返回之前，不散落在业务函数之间；由现有值可计算的数据保持派生值。
+- 不使用裸 `void promise()` 掩盖异步结果，不写无意义裸表达式。
+- 弹层可快速关闭或切换记录、旧请求可能覆盖新视图时，必须废弃或取消过期响应，但不限定必须使用请求 id；连续提交或状态切换必须阻止重复请求并反馈失败。
+
+## 5. 表单、表格与编辑表格配置
+
+- 固定配置按组件类型命名为 `formColumns`、`tableColumns`、`editTableColumns`；同文件多组同类配置增加用途前缀。
+- 动态工厂命名为 `create<Business><Form|Table|EditTable>Columns`，参数使用对象解构和具名 interface，并标注返回类型。
+- 即使只有一个运行时参数，也避免位置参数导致后续扩展产生顺序耦合。
+- ProForm 使用 Sisyphus 提供的 columns 配置能力；能用组件原生 `valueType`、`component` 或 `viewRender` 表达时，不重复手写低层渲染。
+- Sisyphus columns 表单字段需要响应值变化时，必须在 `fieldProps` 中使用 `onFieldChange`，不得使用 `onChange`。直接渲染的 antd、React 或其他第三方控件仍保持其公开 `onChange` API。
+- 简单必填字段只配置 `required: true`，不得再重复配置仅表达简单必填的 `rules`。只有字段存在自定义校验器、正则、长度或跨字段约束时才使用 `rules`，且不得为覆盖必填文案重复建模同一约束。
+- Sisyphus columns 表单字段已有 `label` 且没有特别说明时，不配置 `fieldProps.placeholder`，由组件按字段类型和 `label` 自动生成默认提示。只有用户明确要求、最近项目规范规定，或默认文案不能表达必要业务语义时才保留自定义 placeholder。
+- 基础数据列不随意固定宽度；操作列、固定列或有布局证据的列才配置宽度。
+- `config` 只保留不依赖 JSX、页面回调、权限或运行时 loading 的基础列；页面展开基础列后追加业务渲染列。
+- 不机械改名第三方 `columns` props、`fieldProps.columns`、遍历参数或测试抽象集合。
+- 只使用组件原生 `valueType` 等声明式能力的基础列不因视觉格式化被移出 `config`；包含 JSX、操作按钮或页面运行时交互依赖的 `render`、`viewRender` 等配置必须留在消费页面。
+- 发现 Sisyphus columns 的 `fieldProps.onChange` 时，必须确认回调用于字段值变化；满足条件时按 `P1` 报告并计划替换为 `onFieldChange`，不能只凭文本命中判错。
+- 字段同时配置 `required: true` 和仅含简单必填的 `rules` 时，按 `P1` 合并为一个重复校验问题；复杂规则只移除重复必填部分，不删除自定义校验。
+- `fieldProps.placeholder` 仅重复“请输入/请选择 + label”时按 `P2` 报告并计划删除；存在特殊格式、业务含义或交互指引时不列问题。
+
+## 6. 类型、服务与数据适配
+
+- 请求参数、响应和 DTO 使用明确类型，不以 `any` 绕过契约；纯类型使用 `import type`。
+- service 使用项目统一 request，只表达传输契约，不引用 Form、message 或页面状态。
+- 动态 URL 片段使用 `encodeURIComponent`；id、业务 key 和其他标识按后端契约分别传递，不能因类型相同而混用。
+- 项目要求 service 注释时，注释包含业务动作、全大写 HTTP method、路径与消费语义。
+- 每个 service 的 JSDoc 与 `export` 声明各自独占一行，相邻 service 块之间保留空行；项目没有更严格格式规则时按 `P2` 报告。
+- service 排版修复只调整布局，不改请求方法、路径、参数、返回类型或调用关系。
+- DTO 到 payload 的纯转换放在工具层，移除界面临时字段并清理不应提交的互斥字段。
+- 成功、失败、确认、关闭和刷新反馈由编排层处理；service 不吞异常或伪装成功。
+- 删除、停用或修改会影响已有引用时，先获取影响范围并阻止或确认操作；影响查询失败时不继续高风险动作。
+- 状态接口优先表达目标状态而不是简单取反，防止重试导致结果反转；最终以实际后端契约为准。
+
+## 7. 空值、错误与异步安全
+
+- 可选数据通过可选链、空值合并或入口默认值兜底；数组默认空数组，展示文案提供可读回退。
+- `0`、空字符串和 `false` 有业务意义时，不用 `||` 错误覆盖。
+- 异步失败必须向上抛出或提供可观察反馈，禁止空 catch 和静默伪成功。
+- 接口失败以响应 `message` 为准。统一请求层已经展示时，页面不得再调用 `message.error`、`notification.error` 等追加固定失败提示。
+- 页面可以恢复 loading、保留弹层或执行必要清理，但不得用固定文案覆盖接口 `message`，也不得为消除重复提示而吞异常。
+- 保存、删除和状态更新只在真实成功后关闭界面、提示成功并刷新。
+- 二次确认的异步保存准确返回成功、取消和失败，避免表单容器误判保存结果；竞态保护覆盖主详情及其关联分页、引用和辅助数据，而不是只保护一个请求。
+- 文本搜索不能单独证明重复错误提示或 loading 镜像，必须沿请求链确认来源与消费点。
+- 已确认统一请求层负责错误反馈后，页面或组件追加固定失败文案按 `P1` 报告。
+- 已确认请求由 Hook 承载后，手工 loading state、状态类型字段和置位代码合并作为一个 `P1` 根因；自有组件平铺 loading prop 及其 Props 声明合并作为另一个 `P1` 根因；第三方公开 loading API 不列问题。
+
+## 8. 导入、样式与测试
+
+- import 分组、相对路径深度、类型导入、类名风格和测试目录以目标项目最近规范为准。
+- 项目无具体规则时，建议按 React、antd、其他第三方、项目别名、当前目录、父目录、样式分组。
+- 深层相对路径优先使用项目别名；运行时导入与类型导入保持清晰。
+- 自定义类名保持项目风格；采用 kebab-case 的项目中不混入 camelCase、snake_case 或 BEM 拼接。
+- 聚焦测试覆盖 payload 适配、动态列条件、service 方法与路径、竞态保护和高风险操作门禁。
+- 测试证明可观察行为或契约，不只匹配标题、注释或实现措辞。
+
+## 9. 不得推广的样板细节
+
+- 不推广样板的业务字段、字典 code、接口地址、文案、分页大小和固定宽度。
+- 不要求所有页面具有相同数量的抽屉、弹窗、请求或状态字段。
+- 不强制所有字典使用同一种组件，也不强制所有详情使用同一种展示组件。
+- 不要求所有状态变更都二次确认；只有存在引用、权限、数据破坏或相似风险时才需要保护。
+- 仅因为实现与样板不同不能判错，必须说明适用规则和实际影响。
+
+## 10. 证据与修复计划质量
+
+- 每条问题至少包含一个文件清单内的 `文件:行号`；清单外证据只用于证明规则或调用关系。
+- 文本搜索只证明结构事实。运行时、类型或数据问题需要调用链、类型声明、接口契约或测试证据。
+- 同一根因导致的多个现象合并为一个问题。
+- 修复计划关联问题编号，保持现有路由、接口、权限、公开组件 API 和无关业务行为。
+- 修复顺序为正确性风险、职责与契约、可维护性，并为每步指定聚焦验证；门卫不在计划中实现代码。

package/skills/sisyphus-scene/references/validation-rules.md

@@ -40,10 +40,11 @@
 ## 验证命令
 
 1. `validate-scaffold.mjs --profile <ProjectProfile JSON>`
-2. 目标项目 typecheck
-3. 目标项目 lint
-4. 相关单元/集成测试
-5. 有截图时进行页面视觉核对
-6. GitNexus detect changes
+2. 按 `references/review-guard.md` 创建独立只读子代理，并使用 `scripts/evaluate-review-gate.mjs` 判定通过、修复或阻断；最多 2 轮自动修复，每轮后重新执行第 1、2 步
+3. 目标项目 typecheck
+4. 目标项目 lint
+5. 相关单元/集成测试
+6. 有截图时进行页面视觉核对
+7. GitNexus detect changes
 
 任一检查未执行都必须在交付报告中标注“未验证”，不能省略。

package/skills/sisyphus-scene/references/workflow.md

@@ -8,6 +8,7 @@
 - Sisyphus 校准
 - 能力缺口检查
 - Ant Design 与项目组件补全
+- 确定性校验与规范门卫
 - 停止条件
 
 ## 输入归一化
@@ -91,6 +92,25 @@ Sisyphus 证据：<查询过的组件/API/demo>
 
 Ant Design 仍无法覆盖时检索项目公共组件。只使用有源码、调用示例和目标项目兼容证据的组件。修改已有符号前执行 GitNexus impact。
 
+## 确定性校验与规范门卫
+
+代码生成后先运行 `validate-scaffold.mjs`，传入本轮使用的同一份 `ProjectProfile` 和全部 `ComponentDecision`。确定性校验失败时先修复对应错误，不创建规范门卫。
+
+确定性校验通过后读取 `references/review-guard.md` 和 `references/review-standard.md`，并通过平台原生能力创建独立只读规范门卫子代理。输入严格限定为用户需求与豁免摘要、`SceneSpec`、项目根目录、目标目录、本次生成或修改的文件清单、`ProjectProfile`、`ComponentDecision`、审查轮次，以及复审时的上一轮问题和修复摘要。门卫可读取直接依赖和项目规则取证，但只能报告清单内问题，不能编辑文件。
+
+门卫输出保存到任务临时目录，再通过 `scripts/evaluate-review-gate.mjs --root <项目根目录> --report-root <任务临时目录>` 校验门卫身份、真实目标与文件清单、规则/Sisyphus 证据、固定段落、结论、问题等级和轮次。主 Agent 不自行解释或放宽不一致的报告。
+
+门卫结论的处理规则：
+
+- `通过` 且问题清单为“未发现规范问题”：进入项目 typecheck、lint、相关测试、页面验收和 GitNexus 检查。
+- `部分通过` 或 `不通过`：主 Agent 按问题编号在文件清单内执行最小修复，然后重新运行确定性校验并创建新的门卫子代理复审。
+- `无法判断`：立即阻断，不猜测缺失证据。
+- 自动修复最多 2 轮；两轮后 P0、P1、P2 任一遗留问题都阻断交付，不允许忽略 P2。
+- 修复需要变更文件清单之外的源码、公共 API、路由、权限、依赖或全局布局时停止并报告，不扩大范围。
+- 平台无法创建独立只读子代理时阻断；不得降级为主 Agent 自审。
+
+每次复审必须创建新子代理并基于当前文件重新取证，不能复用上一轮“通过”或问题清单。
+
 ## 停止条件
 
 出现任一情况即停止代码生成：
@@ -101,5 +121,9 @@ Ant Design 仍无法覆盖时检索项目公共组件。只使用有源码、调
 - 项目指令文件存在同级冲突
 - Sisyphus、Ant Design 和项目组件均无法覆盖必需能力
 - 无法确认会改变公共接口或业务数据流的关键需求
+- 平台无法创建独立只读规范门卫子代理
+- 门卫结论为“无法判断”
+- 最多 2 轮自动修复后仍存在 P0、P1 或 P2 问题
+- 门卫修复要求超出本次生成或修改的文件清单
 
 停止时输出已完成的证据、未解决能力、建议的独立前置任务，不用占位代码伪装完成。

package/skills/sisyphus-scene/scripts/evaluate-review-gate.mjs

@@ -0,0 +1,222 @@
+#!/usr/bin/env node
+
+import fs from 'node:fs';
+import path from 'node:path';
+import process from 'node:process';
+
+const parseArgs = (argv) => {
+  const args = {};
+  for (let index = 0; index < argv.length; index += 1) {
+    const token = argv[index];
+    if (!token.startsWith('--')) continue;
+    const key = token.slice(2);
+    const value = argv[index + 1];
+    if (!value || value.startsWith('--')) args[key] = true;
+    else {
+      args[key] = value;
+      index += 1;
+    }
+  }
+  return args;
+};
+
+const args = parseArgs(process.argv.slice(2));
+const mode = String(args.mode || 'generation-gate');
+const root = path.resolve(String(args.root || process.cwd()));
+const reportRoot = path.resolve(String(args['report-root'] || root));
+const round = Number(args.round);
+const reasons = [];
+const isWithin = (parent, target) => {
+  const relative = path.relative(parent, target);
+  return relative === '' || (!relative.startsWith('..') && !path.isAbsolute(relative));
+};
+
+if (!args.report) reasons.push('missing-report');
+if (!['generation-gate', 'path-review'].includes(mode)) reasons.push('invalid-mode');
+if (
+  !Number.isInteger(round) ||
+  round < 0 ||
+  round > 2 ||
+  (mode === 'path-review' && round !== 0)
+)
+  reasons.push('invalid-round');
+
+let report = '';
+if (args.report) {
+  const reportPath = path.resolve(reportRoot, String(args.report));
+  if (!isWithin(reportRoot, reportPath)) reasons.push('report-outside-root');
+  else if (!fs.existsSync(reportPath)) reasons.push('report-not-found');
+  else {
+    try {
+      const realReportRoot = fs.realpathSync(reportRoot);
+      const realReportPath = fs.realpathSync(reportPath);
+      const reportStat = fs.statSync(realReportPath);
+      if (!isWithin(realReportRoot, realReportPath)) reasons.push('report-outside-root');
+      else if (!reportStat.isFile()) reasons.push('report-unreadable');
+      else if (reportStat.size > 1024 * 1024) reasons.push('report-too-large');
+      else report = fs.readFileSync(realReportPath, 'utf8');
+    } catch {
+      reasons.push('report-unreadable');
+    }
+  }
+}
+
+const headings = ['### 规范审查', '### 问题清单', '### 修复计划'];
+const headingIndexes = headings.map((heading) => report.indexOf(heading));
+if (
+  headingIndexes.some((index) => index < 0) ||
+  !(headingIndexes[0] < headingIndexes[1] && headingIndexes[1] < headingIndexes[2])
+)
+  reasons.push('invalid-report-sections');
+
+const allHeadings = [...report.matchAll(/^#{1,6}\s+.+$/gm)].map((match) => match[0]);
+if (allHeadings.length !== headings.length || allHeadings.some((heading, index) => heading !== headings[index]))
+  reasons.push('unexpected-report-sections');
+
+const hasValidSections = headingIndexes.every((index) => index >= 0);
+const reviewSection = hasValidSections
+  ? report.slice(headingIndexes[0] + headings[0].length, headingIndexes[1])
+  : '';
+const findingsSection = hasValidSections
+  ? report.slice(headingIndexes[1] + headings[1].length, headingIndexes[2])
+  : '';
+const conclusionMatch = reviewSection.match(
+  /^结论[：:]\s*(通过|部分通过|不通过|无法判断)\s*$/m,
+);
+const conclusion = conclusionMatch?.[1] || null;
+if (!conclusion) reasons.push('missing-conclusion');
+
+let metadata = null;
+let realTargetPath = null;
+let targetIsFile = false;
+const reviewedFilePaths = new Set();
+const metadataMatch = reviewSection.match(/^门卫元数据[：:]\s*(\{.+\})\s*$/m);
+if (!metadataMatch) reasons.push('missing-review-metadata');
+else {
+  try {
+    metadata = JSON.parse(metadataMatch[1]);
+  } catch {
+    reasons.push('invalid-review-metadata');
+  }
+}
+
+if (metadata) {
+  if (!/^CRG-[A-Za-z0-9_-]+$/.test(String(metadata.reviewId || '')))
+    reasons.push('invalid-review-id');
+  if (metadata.reviewer !== 'sisyphus-cr-readonly-subagent')
+    reasons.push('invalid-reviewer');
+  if (metadata.reviewMode !== mode) reasons.push('review-mode-mismatch');
+  if (metadata.ruleVersion !== 'sisyphus-cr@1.5.0')
+    reasons.push('review-rule-version-mismatch');
+  if (metadata.reviewRound !== round) reasons.push('review-round-mismatch');
+  if (!path.isAbsolute(String(metadata.targetPath || '')))
+    reasons.push('invalid-target-path');
+  else {
+    try {
+      const realRoot = fs.realpathSync(root);
+      const realTarget = fs.realpathSync(metadata.targetPath);
+      if (!isWithin(realRoot, realTarget)) reasons.push('target-outside-root');
+      else {
+        const targetStat = fs.statSync(realTarget);
+        if (!targetStat.isFile() && !targetStat.isDirectory())
+          reasons.push('invalid-target-path');
+        else {
+          realTargetPath = realTarget;
+          targetIsFile = targetStat.isFile();
+        }
+      }
+    } catch {
+      reasons.push('invalid-target-path');
+    }
+  }
+  if (!Array.isArray(metadata.files) || metadata.files.length === 0)
+    reasons.push('missing-reviewed-files');
+  else {
+    for (const filePath of metadata.files) {
+      if (!path.isAbsolute(String(filePath || ''))) {
+        reasons.push('invalid-reviewed-file');
+        continue;
+      }
+      try {
+        const realRoot = fs.realpathSync(root);
+        const realFile = fs.realpathSync(filePath);
+        if (!isWithin(realRoot, realFile) || !fs.statSync(realFile).isFile())
+          reasons.push('invalid-reviewed-file');
+        else if (
+          realTargetPath &&
+          (targetIsFile ? realFile !== realTargetPath : !isWithin(realTargetPath, realFile))
+        )
+          reasons.push('reviewed-file-outside-target');
+        else reviewedFilePaths.add(realFile);
+      } catch {
+        reasons.push('invalid-reviewed-file');
+      }
+    }
+  }
+  if (
+    !Array.isArray(metadata.ruleSources) ||
+    metadata.ruleSources.length === 0 ||
+    metadata.ruleSources.some((source) => typeof source !== 'string' || source.length === 0)
+  )
+    reasons.push('missing-rule-sources');
+  if (typeof metadata.sisyphusEvidence !== 'string' || metadata.sisyphusEvidence.length === 0)
+    reasons.push('missing-sisyphus-evidence');
+}
+
+const findingRows = [
+  ...findingsSection.matchAll(
+    /^\|\s*(CR-\d{3})\s*\|\s*(P[012])\s*\|\s*([^|]+?)\s*\|/gm,
+  ),
+];
+const priorities = [...new Set(findingRows.map((match) => match[2]))].sort();
+const issueCount = findingRows.length;
+for (const finding of findingRows) {
+  const location = finding[3].trim();
+  const locationMatch = location.match(/^(.+):(\d+)(?::\d+)?$/);
+  if (!locationMatch || !path.isAbsolute(locationMatch[1])) {
+    reasons.push('invalid-finding-location');
+    continue;
+  }
+  try {
+    const realFindingFile = fs.realpathSync(locationMatch[1]);
+    if (!reviewedFilePaths.has(realFindingFile))
+      reasons.push('finding-outside-reviewed-files');
+  } catch {
+    reasons.push('invalid-finding-location');
+  }
+}
+const noFindings = /(?:^|\n)未发现规范问题\s*(?:\n|$)/.test(findingsSection);
+
+let status = 'blocked';
+let nextRound = null;
+if (reasons.length === 0) {
+  if (mode === 'path-review') {
+    if (conclusion === '通过' && issueCount === 0 && noFindings) status = 'reviewed';
+    else if (conclusion === '通过') reasons.push('inconsistent-passed-report');
+    else if (conclusion === '无法判断' && issueCount === 0) status = 'reviewed';
+    else if (issueCount > 0) status = 'reviewed';
+    else reasons.push('missing-findings');
+  } else if (conclusion === '无法判断') reasons.push('unable-to-determine');
+  else if (conclusion === '通过' && issueCount === 0 && noFindings) status = 'passed';
+  else if (conclusion === '通过') reasons.push('inconsistent-passed-report');
+  else if (issueCount === 0) reasons.push('missing-findings');
+  else if (round < 2) {
+    status = 'repair';
+    nextRound = round + 1;
+  } else reasons.push('repair-rounds-exhausted');
+}
+
+const output = {
+  status,
+  mode,
+  ruleVersion: metadata?.ruleVersion || null,
+  round: Number.isInteger(round) ? round : null,
+  nextRound,
+  conclusion,
+  issueCount,
+  priorities,
+  reasons,
+  reviewId: metadata?.reviewId || null,
+};
+process.stdout.write(`${JSON.stringify(output, null, 2)}\n`);
+if (status === 'blocked') process.exitCode = 2;