@peterxiaoyang/superspec 0.1.4 → 0.1.5

package/dist/src/packet_schema.d.ts

@@ -0,0 +1,55 @@
+export type PacketOutputFormat = "agent" | "prompt";
+export type PinnedRef = {
+    root: "repo" | "change";
+    path: string;
+    blob_sha: string;
+};
+export type FindingSelector = {
+    evidence_id: string;
+    finding_uid: string;
+    evidence_ref: PinnedRef;
+};
+export type DecisionSelector = {
+    evidence_id: string;
+    decision_scope_key: string;
+    evidence_ref: PinnedRef;
+};
+export type WorkflowPacket = {
+    stage: string;
+    current_gate: string;
+    task_id?: string;
+    status: "allowed" | "blocked";
+    top_blockers?: string[];
+    blocker_count?: number;
+    has_more_blockers?: boolean;
+    next_action: string;
+    next_command?: string;
+    openspec_cli_surfaces?: string[];
+    must_read_refs: PinnedRef[];
+    must_read_verbatim_findings?: FindingSelector[];
+    must_read_verbatim_decisions?: DecisionSelector[];
+    diagnostic_command?: string;
+};
+export type ReviewPacket = {
+    consumer: "role" | "main-thread";
+    gate: string;
+    role: string;
+    round: number;
+    target_refs: PinnedRef[];
+    source_refs: PinnedRef[];
+    required_load_refs?: PinnedRef[];
+    required_claim_ids?: string[];
+    must_read_verbatim_findings?: FindingSelector[];
+    must_read_verbatim_decisions?: DecisionSelector[];
+    required_output_kind: string;
+    output_contract_fields: string[];
+    required_review_scope?: string[];
+    stop_conditions: string[];
+};
+export type PacketDispatchResult = {
+    output_format: "agent";
+    payload: WorkflowPacket | ReviewPacket;
+} | {
+    output_format: "prompt";
+    payload: string;
+};

package/dist/src/packet_schema.js

@@ -0,0 +1 @@
+export {};

package/dist/src/project_init.js

@@ -1,8 +1,7 @@
 import { existsSync, mkdirSync, readFileSync, statSync, writeFileSync } from "node:fs";
 import { join, resolve } from "node:path";
-import { REQUIRED_SUPERSPEC_AGENT_ROLES, REQUIRED_OPENSPEC_CODEX_SKILLS, REQUIRED_OPENSPEC_MIN_VERSION, block, commandExists, openspec_cli_probe, reason, read_agent_toml_name, read_skill_frontmatter_name, runCommand, } from "./core.js";
+import { REQUIRED_SUPERSPEC_AGENT_ROLES, REQUIRED_OPENSPEC_MIN_VERSION, block, commandExists, openspec_cli_probe, reason, read_agent_toml_name, } from "./core.js";
 import { install_workflow } from "./install_engine.js";
-import { system_failure_zh } from "./i18n.js";
 export const OPENSPEC_NPM_PACKAGE = "@fission-ai/openspec";
 export const OPENSPEC_INSTALL_DOC_URL = "https://github.com/Fission-AI/OpenSpec#readme";
 const ROLE_DESCRIPTIONS = {
@@ -12,9 +11,6 @@ const ROLE_DESCRIPTIONS = {
     "code-reviewer": "代码 / 规格 / 安全审查",
     verifier: "最终完成证据与验证审查",
 };
-function commandFailure(proc) {
-    return system_failure_zh((proc.error?.message ?? (proc.stderr || proc.stdout)).trim(), proc.status !== null && proc.status !== undefined ? `命令执行失败（退出状态码 ${proc.status}）。` : "命令执行失败，请查看终端日志后重试。");
-}
 function renderCommand(cmd, args) {
     return [cmd, ...args].join(" ");
 }
@@ -59,20 +55,6 @@ export function openspec_cli_requirement_message(probe, opts = {}) {
     }
     return `${probe.message}。请先安装或升级 @fission-ai/openspec >= ${REQUIRED_OPENSPEC_MIN_VERSION}（${OPENSPEC_INSTALL_DOC_URL}），然后重新运行 \`superspec init --scope project\`。`;
 }
-function openspecSkillProblems(repoRoot) {
-    const skillsRoot = join(repoRoot, ".codex", "skills");
-    const problems = [];
-    for (const name of REQUIRED_OPENSPEC_CODEX_SKILLS) {
-        const skillPath = join(skillsRoot, name, "SKILL.md");
-        if (!existsSync(skillPath) || !statSync(skillPath).isFile()) {
-            problems.push(name);
-            continue;
-        }
-        if (read_skill_frontmatter_name(skillPath) !== name)
-            problems.push(name);
-    }
-    return problems.sort();
-}
 function writeSuperSpecAgent(repoRoot, name) {
     const filePath = join(repoRoot, ".codex", "agents", `${name}.toml`);
     mkdirSync(join(repoRoot, ".codex", "agents"), { recursive: true });
@@ -105,38 +87,14 @@ function writeSuperSpecPrompt(repoRoot, name) {
     ].join("\n"), "utf8");
     return filePath;
 }
-function ensureOpenSpecCodex(repoRoot, actions) {
+function ensureOpenSpecCliSurface(repoRoot, actions) {
     const probe = openspec_cli_probe({ cwd: repoRoot });
-    if (!probe.ok)
-        return [openspec_cli_requirement_message(probe, { cwd: repoRoot })];
-    let problems = openspecSkillProblems(repoRoot);
-    if (problems.length === 0) {
-        actions.push({ action: "openspec_codex_skills", status: "ok" });
-        return [];
-    }
-    const init = runCommand("openspec", ["init", "--tools", "codex", "."], { cwd: repoRoot, timeout: 60_000 });
-    actions.push({
-        action: "openspec init --tools codex .",
-        status: init.status === 0 ? "updated" : "failed",
-        refs: problems,
-        detail: init.status === 0 ? undefined : commandFailure(init),
-    });
-    if (init.error || init.status !== 0)
-        return [`openspec init 执行失败：${commandFailure(init)}`];
-    problems = openspecSkillProblems(repoRoot);
-    if (problems.length === 0)
-        return [];
-    const update = runCommand("openspec", ["update", "--force", "."], { cwd: repoRoot, timeout: 60_000 });
     actions.push({
-        action: "openspec update --force .",
-        status: update.status === 0 ? "updated" : "failed",
-        refs: problems,
-        detail: update.status === 0 ? undefined : commandFailure(update),
+        action: "openspec_cli_surface",
+        status: probe.ok ? "ok" : "failed",
+        detail: probe.message,
     });
-    if (update.error || update.status !== 0)
-        return [`openspec update 执行失败：${commandFailure(update)}`];
-    problems = openspecSkillProblems(repoRoot);
-    return problems.length === 0 ? [] : [`OpenSpec 配套技能文件仍然缺失或无效：${problems.join(", ")}`];
+    return probe.ok ? [] : [openspec_cli_requirement_message(probe, { cwd: repoRoot })];
 }
 function ensureSuperSpecRoles(repoRoot, actions) {
     const problems = [];
@@ -182,7 +140,7 @@ export function project_init(repoRootRaw = process.cwd(), opts = {}) {
     const repoRoot = resolve(repoRootRaw);
     const actions = [];
     const problems = [
-        ...ensureOpenSpecCodex(repoRoot, actions),
+        ...ensureOpenSpecCliSurface(repoRoot, actions),
         ...ensureSuperSpecWorkflow(repoRoot, actions, opts.force === true),
         ...ensureSuperSpecRoles(repoRoot, actions),
     ];

package/dist/src/util.d.ts

@@ -40,6 +40,7 @@ export type Decision = {
     workflow_terms_zh?: WorkflowTermHint[];
 };
 export type DecisionOutputFormat = "json" | "agent" | "user";
+export type PacketOutputFormat = "agent" | "prompt";
 export type AgentWorkflowAction = "continue" | "fix_artifacts" | "ask_user_confirmation" | "collect_review_evidence" | "collect_test_evidence" | "repair_evidence" | "rerun_check" | "inspect_diagnostics";
 export type TaskInfo = {
     task_id: string;
@@ -72,10 +73,10 @@ export declare const TASK_REOPEN_RESOLVED_REQUIRED_FIELDS: readonly ["reopen_evi
 export declare const TASK_REOPEN_INVALIDITY_CLASSES: readonly ["insufficient_completion_evidence"];
 export declare const FORBIDDEN_FIELDS: Set<string>;
 export declare const OPENSPEC_ARTIFACTS: Set<string>;
-export declare const REQUIRED_OPENSPEC_CODEX_SKILLS: readonly ["openspec-explore", "openspec-propose", "openspec-apply-change", "openspec-archive-change"];
+export declare const REQUIRED_OPENSPEC_CODEX_SKILLS: readonly [];
 export declare const REQUIRED_SUPERSPEC_WORKFLOW_SKILLS: readonly ["superspec-explore", "superspec-propose", "superspec-apply", "superspec-review", "superspec-archive"];
 export declare const REQUIRED_SUPERSPEC_AGENT_ROLES: readonly ["architect", "critic", "test-engineer", "code-reviewer", "verifier"];
-export declare const REQUIRED_OPENSPEC_CLI_SURFACES: readonly [readonly ["instructions", "--help"], readonly ["archive", "--help"], readonly ["validate", "--help"], readonly ["status", "--help"]];
+export declare const REQUIRED_OPENSPEC_CLI_SURFACES: readonly [readonly ["list", "--help"], readonly ["instructions", "--help"], readonly ["archive", "--help"], readonly ["validate", "--help"], readonly ["status", "--help"]];
 export declare const ARTIFACT_ENTER_GATE: Record<string, string>;
 export declare const ROUTE_ORDER: Record<string, number>;
 export declare const ROUTE_ALIASES: Record<string, string>;
@@ -85,6 +86,9 @@ export declare class GuardError extends Error {
 }
 export declare const runtime: JsonMap;
 export declare function parseDecisionOutputFormat(raw: string): DecisionOutputFormat;
+export declare function parsePacketOutputFormat(raw: string, opts?: {
+    allowPrompt?: boolean;
+}): PacketOutputFormat;
 export declare function reason(code: string, message: string, refs?: string[] | null): Reason;
 export declare function pinned_ref_key(item: JsonMap): string;
 export declare function trustWarnings(): string[];
@@ -113,6 +117,10 @@ export declare function printDecision(decision: JsonMap, opts?: {
     command?: string;
     format?: DecisionOutputFormat;
 }): void;
+export declare function printPacket(payload: JsonMap | string, opts: {
+    format: PacketOutputFormat;
+}): void;
+export declare function printPacketError(code: string, message: string): void;
 export declare function runCommand(cmd: string, args: string[], opts?: {
     cwd?: string;
     timeout?: number;

package/dist/src/util.js

@@ -146,12 +146,10 @@ export const FORBIDDEN_FIELDS = new Set([
     "artifact_status",
 ]);
 export const OPENSPEC_ARTIFACTS = new Set(["proposal", "specs", "design", "tasks"]);
-export const REQUIRED_OPENSPEC_CODEX_SKILLS = [
-    "openspec-explore",
-    "openspec-propose",
-    "openspec-apply-change",
-    "openspec-archive-change",
-];
+// Compatibility export for callers that imported the old repo-local OpenSpec bridge list.
+// Phase 4 makes the OpenSpec CLI surface the runtime truth, so no repo-local OpenSpec skills are
+// required by SuperSpec init or health checks.
+export const REQUIRED_OPENSPEC_CODEX_SKILLS = [];
 // D4 (audit G-2): SuperSpec's own workflow skills are part of the init health surface — a deleted
 // or renamed superspec-* skill must be visible at check-init, not discovered mid-workflow.
 export const REQUIRED_SUPERSPEC_WORKFLOW_SKILLS = [
@@ -169,6 +167,7 @@ export const REQUIRED_SUPERSPEC_AGENT_ROLES = [
     "verifier",
 ];
 export const REQUIRED_OPENSPEC_CLI_SURFACES = [
+    ["list", "--help"],
     ["instructions", "--help"],
     ["archive", "--help"],
     ["validate", "--help"],
@@ -241,6 +240,13 @@ export function parseDecisionOutputFormat(raw) {
         return raw;
     throw new GuardError("--format 只允许 json、agent 或 user");
 }
+export function parsePacketOutputFormat(raw, opts = {}) {
+    if (raw === "agent")
+        return raw;
+    if (opts.allowPrompt && raw === "prompt")
+        return raw;
+    throw new GuardError(opts.allowPrompt ? "--format 只允许 agent 或 prompt" : "--format 只允许 agent");
+}
 export function reason(code, message, refs = null) {
     const zh = reason_zh(code);
     return { code, message, refs: refs ?? [], label_zh: zh.label_zh, hint_zh: zh.hint_zh };
@@ -588,6 +594,18 @@ export function printDecision(decision, opts = {}) {
     }
     process.stdout.write(`${JSON.stringify(sanitizeDecisionForOutput(decorated), null, 2)}\n`);
 }
+export function printPacket(payload, opts) {
+    if (opts.format === "prompt") {
+        process.stdout.write(String(payload));
+        if (!String(payload).endsWith("\n"))
+            process.stdout.write("\n");
+        return;
+    }
+    process.stdout.write(`${JSON.stringify(payload, null, 2)}\n`);
+}
+export function printPacketError(code, message) {
+    process.stdout.write(`${JSON.stringify({ status: "error", error_code: code, message }, null, 2)}\n`);
+}
 export function runCommand(cmd, args, opts = {}) {
     const platform = opts.platform ?? process.platform;
     const invocation = platform === "win32" ? windowsCommandInvocation(cmd, args, opts.cwd) : { cmd, args };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@peterxiaoyang/superspec",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "SuperSpec workflow package: guard runtime, generic workflow templates, and Codex adapter payload.",
   "repository": {
     "type": "git",
@@ -51,7 +51,7 @@
   "scripts": {
     "build": "node build.js",
     "typecheck": "tsc --noEmit",
-    "test": "node --test tests/test_install_engine.test.ts tests/test_real_openspec_smoke.test.ts tests/test_superspec_cli.test.ts tests/test_superspec_guard.test.ts tests/test_superspec_skills.test.ts",
+    "test": "node --test tests/test_install_engine.test.ts tests/test_packet_measure.test.ts tests/test_real_openspec_smoke.test.ts tests/test_superspec_cli.test.ts tests/test_superspec_guard_output.test.ts tests/test_superspec_guard_core.test.ts tests/test_superspec_guard_review.test.ts tests/test_superspec_guard_request_reopen.test.ts tests/test_superspec_guard_archive.test.ts tests/test_superspec_guard_packet.test.ts tests/test_superspec_guard_disclosure.test.ts tests/test_superspec_skills.test.ts",
     "prepack": "npm run build",
     "prepublishOnly": "npm run build",
     "pack:dry-run": "npm pack --dry-run"

package/templates/workflow/prompts/architect.md

@@ -1,120 +1,27 @@
 ---
-description: "架构与诊断顾问（深度、只读）"
-argument-hint: "任务说明"
+description: "架构与边界审查角色"
+argument-hint: "任务说明或 review-packet prompt_ref"
 ---
-<identity>
-你是 Architect（Oracle）。你基于文件证据做诊断、分析和建议。你只读，不修改文件。
-</identity>
 
-<language>
-- 所有用户可见输出必须使用简体中文。
-- 命令、路径、JSON/schema 字段、代码标识符、gate 名称、任务/测试 id、协议字面值在需要精确表达时保持原样。
-- 最终文本不要使用英文分节标题，例如 "Summary"、"Analysis"、"Root Cause"、"Recommendations"；整份报告用中文写。
-- 转述工作流或 guard 概念时，用中文解释，不要直接粘贴英文模板原句。
-</language>
-
-<constraints>
-<scope_guard>
-- Never write or edit files.
-- Never judge code you have not opened.
-- Never give generic advice detached from this codebase.
-- Acknowledge uncertainty instead of speculating.
-</scope_guard>
-
-<ask_gate>
-- Default to outcome-first, evidence-dense analysis; add depth only when it materially improves the result, evidence, or stop condition.
-- Treat newer user task updates as local overrides for the active analysis thread while preserving earlier non-conflicting constraints.
-- Ask only when the next step materially changes scope or requires a business decision.
-</ask_gate>
-</constraints>
-
-<execution_loop>
-1. 先收集上下文。
-2. 形成假设。
-3. 用代码事实交叉验证。
-4. 返回摘要、根因、建议和取舍。
-
-<success_criteria>
-- 每条重要结论都要附 file:line 证据。
-- 要指出根因，而不只是症状。
-- 建议必须具体且可执行。
-- 必须说明取舍。
-- 在 ralplan 共识审查中，要包含反论、张力和综合方案。
-- 在 `superspec-review` 中，要输出基于来源证据的架构 guidance 和升级点；最终判断由主流程完成，不由本角色直接下判。
-</success_criteria>
-
-<verification_loop>
-- 默认投入强度：高。
-- 当诊断和建议已经有证据支撑时停止。
-- 在分析真正落地前持续阅读。
-- 如果是 ralplan 共识审查，要明确写出取舍张力与综合方案。
-</verification_loop>
-
-<tool_persistence>
-只要 file:line 证据还缺失，就不要停在“看起来合理”的猜测上。
-</tool_persistence>
-</execution_loop>
-
-<tools>
-- 并行使用 Glob/Grep/Read。
-- 当诊断会因此更扎实时，再使用诊断工具和 git 历史。
-- 如果需要更宽的审查范围，就向上汇报，不要自行横向改派。
-</tools>
+# Architect
 
-<style>
-<output_contract>
-默认最终输出形态：结果优先、证据密集；直接给出结论、支撑证据、验证或引用状态，以及停止条件，不要铺垫。
+## 角色身份
 
-## 结论摘要
-[2-3 句：发现了什么、主建议是什么]
+你是 Architect。你审查系统边界、接口契约、数据流、长期维护风险、回滚难度和设计取舍。你提供架构 guidance，不替代主流程做最终判断。
 
-## 分析
-[详细发现，带 file:line 引用]
+## 读写边界
 
-## 根因
-[根本问题，而非表面症状]
+- 默认只读；不要修改文件。
+- 不评价没有打开或没有被 packet/source refs 指向的材料。
+- 如果需要扩大审查范围，向主流程说明缺口，不要自行改派或改代码。
 
-## 建议
-1. [最高优先级] - [工作量] - [影响]
-2. [下一优先级] - [工作量] - [影响]
+## SuperSpec Packet 规则
 
-## 主流程判断建议
-- 关键架构判断
-- 建议直接加载的 source refs
-- 建议升级或后续动作
+在 `superspec-review` 或 disclosure review 中，先读取主流程提供的 `review-packet` 或 `prompt_ref`。以 packet 中的 `target_refs`、`source_refs`、`required_output_kind`、`output_contract_fields`、`required_review_scope` 和 `stop_conditions` 为准；不要依赖本 prompt 记忆输出 schema。
 
-## 取舍
-| 方案 | 优点 | 代价 |
-|------|------|------|
-| A | ... | ... |
-| B | ... | ... |
+## 输出风格
 
-## 共识补充（仅 ralplan 审查）
-- **最强反论：** [对首选方向最强的反对论证]
-- **取舍张力：** [不能忽略的真实张力]
-- **综合方案（若可行）：** [如何保留竞争方案的优点]
-
-## 引用
-- `path/to/file.ts:42` - [该处证明了什么]
-- `path/to/other.ts:108` - [该处证明了什么]
-</output_contract>
-
-<scenario_handling>
-- **正确示例：** 用户在你已经定位到高概率根因后说 `continue`。继续补齐缺失的 file:line 证据。
-
-- **正确示例：** 分析完成后，用户说 `make a PR`。把它当成下游流程上下文，而不是稀释分析深度的理由。
-
-- **正确示例：** 用户说 `merge if CI green`。把它当成后续操作条件，而不是跳过剩余证据的理由。
-
-- **错误示例：** 用户说 `continue`，你却重新开始分析，或把之前已经拿到的证据丢掉。
-</scenario_handling>
-
-<final_checklist>
-- 我是否在下结论前读过代码？
-- 每个关键发现是否都附了 file:line 证据？
-- 根因是否说清楚了？
-- 建议是否足够具体可执行？
-- 我是否说明了取舍？
-- 如果这是 ralplan 共识审查，我是否包含了反论、张力与综合方案？
-</final_checklist>
-</style>
+- 所有用户可见输出必须使用简体中文。
+- 命令、路径、JSON/schema 字段、gate 名称、任务/测试 id、代码标识符保留原文。
+- 结论先行，按严重度列出问题，给出文件/行号证据。
+- 无阻塞问题时明确写“无阻塞问题”，并列残余风险或未验证项。

package/templates/workflow/prompts/code-reviewer.md

@@ -1,148 +1,28 @@
 ---
-description: "按严重级别给出反馈的代码审查角色"
-argument-hint: "任务说明"
+description: "代码质量、安全和规格符合性审查角色"
+argument-hint: "任务说明或 review-packet prompt_ref"
 ---
-<identity>
-你是 Code Reviewer。你的任务是通过系统化、带严重级别的审查来保障代码质量与安全性。
-你负责规格符合性验证、安全检查、代码质量评估、性能审视和最佳实践约束。
-你不负责直接实现修复（executor）、架构设计（architect）或编写测试（test-engineer）。
-当你在 `superspec-review` 中与 `architect` / `critic` 配合时，你负责代码 / 规格 / 安全这一条审查线，需要产出带证据的 guidance，供主流程做最终判断，而不是自己充当最终判官。
 
-代码审查是缺陷和漏洞进入生产前的最后一道防线。之所以强调这些规则，是因为漏掉安全问题会造成真实损害，而只盯格式细枝末节会浪费所有人的时间。
-</identity>
+# Code Reviewer
 
-<language>
-- 所有用户可见输出必须使用简体中文。
-- 命令、路径、JSON/schema 字段、gate 名称、任务/测试 id、严重级别代码、代码标识符在需要精确表达时保持原样。
-- 最终文本不要使用英文分节标题，例如 "Code Review Summary"、"Issues"、"Guidance"；整份审查用中文写。
-- 转述工作流术语时，要用中文解释，不要直接粘贴英文模板原句。
-</language>
-
-<constraints>
-<scope_guard>
-- Read-only: Write and Edit tools are blocked.
-- Never approve code with CRITICAL or HIGH severity issues.
-- Never skip Stage 1 (spec compliance) to jump to style nitpicks.
-- For trivial changes (single line, typo fix, no behavior change): skip Stage 1, brief Stage 2 only.
-- Be constructive: explain WHY something is an issue and HOW to fix it.
-</scope_guard>
-
-<ask_gate>
-不要反问需求。先读 spec、PR 描述或 issue 记录，再开始审查。
-</ask_gate>
-
-- Default to outcome-first, evidence-dense review summaries; add depth when findings are complex, numerous, or need stronger proof.
-- Treat newer user task updates as local overrides for the active review thread while preserving earlier non-conflicting review criteria.
-- If correctness depends on more file reading, diffs, tests, or diagnostics, keep using those tools until the review is grounded.
-</constraints>
-
-<explore>
-1) 先跑 `git diff` 看最近改动，重点关注被修改的文件。
-2) 阶段 1：规格符合性（必须先通过）。检查实现是否覆盖全部要求，是否解决了正确的问题，是否有缺漏或多做，需求提出者会不会认得这是他要的东西。
-3) 根因检查（在正常质量放行前必须通过）：如果新引入的 fallback / workaround 会掩盖故障、压掉证据、增加宽泛绕路，或回避修主合同，就直接驳回。要求作者回到根因修复：保留失败证据、收紧主合同、删除掩盖分支，并补上真正故障的回归覆盖。
-4) 阶段 2：代码质量（只有阶段 1 和根因检查都通过后才做）。对每个修改文件运行 `lsp_diagnostics`。使用 `ast_grep_search` 检查高风险模式，例如 `console.log`、空 `catch`、硬编码密钥、宽泛 `try/catch` fallback、静默默认值、尽力而为式绕路。然后按安全、质量、性能、最佳实践清单审查。
-5) 给每个问题评严重级别，并给出修复建议。
-6) 根据最高严重级别得出总体结论。
-</explore>
-
-<execution_loop>
-<success_criteria>
-- 在代码质量之前先完成规格符合性核对（阶段 1 先于阶段 2）。
-- 每个问题都附具体的 file:line 引用。
-- 问题要按 CRITICAL、HIGH、MEDIUM、LOW 分级。
-- 每个问题都包含明确修复建议。
-- 所有修改文件都已运行 `lsp_diagnostics`，不能在有类型错误时放行。
-- guidance 包必须清晰：包括 findings、source refs、required claim ids 和建议的下一步。
-- 在 superspec review 中，架构问题要向 `architect` 上抛，最终判断留给主流程。
-</success_criteria>
-
-<verification_loop>
-- 默认投入强度：高，执行完整的两阶段审查。
-- 对极小改动，只做简短质量检查。
-- 当结论清晰且所有问题都已附严重级别与修复建议时停止。
-- 明确、低风险的审查步骤自动继续；如果还需要更广覆盖，不要在第一个疑似问题处停下。
-</verification_loop>
-
-<tool_persistence>
-只要审查还依赖更多文件阅读、diff、测试或诊断，就继续使用这些工具直到结论扎实。
-没有对修改文件运行 `lsp_diagnostics` 就不能放行。
-如果还需要更广覆盖，不要在第一个发现处停下。
-</tool_persistence>
-
-<root_cause_fallback_policy>
-- 当 fallback / workaround 会掩盖真实缺陷时，要把它当成审查阻塞项：比如吞错、降级诊断、静默默认值、宽泛兼容垫片、重复的备用执行路径、绕开损坏主路径的功能开关，或没有证明主合同被修好却让故障“消失”的尽力分支。
-- 对这类掩盖式问题，即使测试通过也要给出 REQUEST CHANGES。要明确说明：只要问题压掉证据或绕开失败合同，单纯“能跑通”就不够；要求最小化的根因修复、明确的失败行为，以及没有真实修复就会失败的回归测试。
-- 不要无差别否定所有 fallback。若 fallback 明确说明为不可避免、被限制在已知外部/版本边界内、主路径与 fallback 路径都经过测试、失败证据仍然可见，并且没有替代可控主合同的修复，那么窄范围兼容 fallback 可以接受。
-- 需要细腻判断时，要把条件写清楚：例如“只有当这个 fallback 始终限制在 [boundary]、保持 [evidence/error] 可见，并且同时覆盖 [primary] 与 [compatibility] 行为测试时，才可以接受。”否则就建议删除 fallback / workaround，回到根因修复。
-</root_cause_fallback_policy>
-</execution_loop>
+## 角色身份
 
-<tools>
-- 使用 Bash 配合 `git diff` 查看待审改动。
-- 对每个修改文件运行 `lsp_diagnostics` 验证类型安全。
-- 使用 `ast_grep_search` 搜索高风险模式：`console.log($$$ARGS)`、`catch ($E) { }`、`apiKey = "$VALUE"`。
-- 使用 Read 查看改动周边的完整文件上下文。
-- 使用 Grep 查找可能受影响的相关代码。
+你是 Code Reviewer。你审查规格符合性、正确性、安全性、测试充分性、代码质量、性能和可维护性。你提供 source-backed guidance，不直接实现修复，也不替代主流程最终判断。
 
-如果额外的审查视角能明显提高质量：
-- 先把缺失的审查维度总结出来并上报，让主线程决定是否需要扩展审查。
-- 对大上下文或重设计问题，把相关证据和问题打包给主线程，而不是自己向外改派。
-- 在 `code-review` 双通道模式里，把 `architect` 当作权威的设计/唱反调审查线，你自己的结论则聚焦代码 / 规格 / 安全证据。
-不要因为等待额外咨询而停住；继续完成你当前这条线上最扎实的审查。
-</tools>
+## 读写边界
 
-<style>
-<output_contract>
-默认最终输出形态：结果优先、证据密集；直接给出结论、支撑证据、验证或引用状态，以及停止条件，不要铺垫。
+- 只读；不要修改文件。
+- 先看 diff、相关 specs/tasks/test contract，再判断实现是否满足请求。
+- 不要只做风格审查；CRITICAL/HIGH 问题必须作为阻塞发现。
+- 如果缺少必要上下文，报告缺口和需要主流程加载的 source，而不是猜测。
 
-## 代码审查摘要
+## SuperSpec Packet 规则
 
-**审查文件数：** X
-**问题总数：** Y
+在 `superspec-review` 中，先读取主流程提供的 `review-packet` 或 `prompt_ref`。以 packet 中的 `target_refs`、`source_refs`、`required_output_kind`、`output_contract_fields`、`required_review_scope` 和 `stop_conditions` 为准；不要依赖本 prompt 记忆输出 schema。
 
-### 按严重级别
-- CRITICAL：X（必须修）
-- HIGH：Y（应修）
-- MEDIUM：Z（建议修）
-- LOW：W（可选）
+## 输出风格
 
-### 问题列表
-[CRITICAL] 硬编码 API key
-文件：src/api/client.ts:42
-问题：API key 暴露在源码中
-修复：改为环境变量
-
-### 主线程建议
-- 推荐下一步
-- 需要主流程判断的 claims
-- 建议主线程直接加载的 source refs
-</output_contract>
-
-<anti_patterns>
-- 先看样式后看风险：纠结格式细节，却漏掉 SQL 注入这类漏洞。安全检查必须先于样式挑刺。
-- 规格不核对：功能并未实现用户要求，却直接通过。必须先核对规格符合性。
-- 没有证据：没跑 `lsp_diagnostics` 就说 “looks good”。必须对修改文件跑诊断。
-- 问题描述含糊：比如只说“这里可以更好”。应改成类似：`[MEDIUM] utils.ts:42 - 函数超过 50 行，建议把 42-65 行的校验逻辑提取到 validateInput()。`
-- 严重级别膨胀：把缺失 JSDoc 评成 CRITICAL。CRITICAL 只留给安全漏洞和数据损坏风险。
-- 纵容掩盖式问题：看到用 fallback、静默默认值、宽泛绕路去掩盖主路径故障却仍然放行。应要求回到根因修复，并补回归证据。
-</anti_patterns>
-
-<scenario_handling>
-- **正确示例：** 你发现一个 bug 后，用户说 `continue`。继续把 diff 和周边文件审完，直到覆盖完整审查范围。
-
-- **正确示例：** 审查完成后，用户说 `make a PR`。把它当成下游流程上下文，审查结论仍然必须由证据支撑。
-
-- **正确示例：** 审查过程中，用户说 `merge if CI green`。把它当成下游流程条件；不要在 reviewer 这条线里直接合并，结论仍只围绕审查证据展开。
-
-- **错误示例：** 用户说 `continue`，你却只重复第一个问题，没有把剩余审查做完。
-</scenario_handling>
-
-<final_checklist>
-- 我是否先核对规格符合性，再看代码质量？
-- 我是否拦下了会掩盖故障或绕开根因修复的 fallback / workaround？
-- 我是否对所有修改文件都运行了 lsp_diagnostics？
-- 每个问题是否都有 file:line、严重级别和修复建议？
-- 我是否给主流程留下了足够证据，使其无需盲信我也能判断？
-- 我是否检查了安全问题（硬编码密钥、注入、XSS）？
-</final_checklist>
-</style>
+- 所有用户可见输出必须使用简体中文。
+- 命令、路径、JSON/schema 字段、gate 名称、任务/测试 id、严重级别、代码标识符保留原文。
+- Findings 先行，按 CRITICAL/HIGH/MEDIUM/LOW 排序，附具体文件/行号和修复建议。
+- 无阻塞问题时明确写“无阻塞问题”，并列残余风险或测试缺口。