@godv61/dsh-task-engine 0.23.8 → 0.24.0

package/lib/client.js

@@ -4547,6 +4547,27 @@ var workbenchTheme = `
 .te-workbench button:focus-visible,.te-modal button:focus-visible,.te-input:focus-visible { outline:2px solid var(--dsw-alias-state-business-primary); outline-offset:3px; }
 .te-workbench h2 { margin:0 0 6px; font-size:24px; letter-spacing:-.5px; }
 .te-workbench h3,.te-modal h3 { margin:0; font-size:16px; overflow-wrap:anywhere; }
+.te-workbench .te-disclosure-title { font-size:16px; font-weight:700; color:var(--dsw-alias-label-primary); }
+.te-binding-grid { display:grid; grid-template-columns:repeat(auto-fit,minmax(min(100%,320px),1fr)); gap:14px; align-items:start; }
+.te-binding-picker { min-width:0; display:flex; flex-direction:column; gap:10px; border:1px solid var(--dsw-alias-border-l2); border-radius:12px; padding:14px; background:var(--dsw-alias-bg-layer-1); }
+.te-binding-heading { display:flex; align-items:center; justify-content:space-between; gap:8px; }
+.te-binding-heading h4 { margin:0; font-size:14px; font-weight:650; }
+.te-binding-count,.te-binding-meta { color:var(--dsw-alias-label-secondary); font-size:12px; }
+.te-binding-tools { display:flex; gap:10px; align-items:center; flex-wrap:wrap; }
+.te-binding-tools .te-input { flex:1; width:100%; min-width:120px; padding:8px 10px; font-size:13px; }
+.te-binding-filter { display:flex; align-items:center; gap:5px; font-size:12px; white-space:nowrap; cursor:pointer; }
+.te-binding-list { max-height:280px; overflow-y:auto; overscroll-behavior:contain; display:flex; flex-direction:column; gap:4px; scrollbar-gutter:stable; }
+.te-binding-option { display:flex; gap:9px; align-items:flex-start; padding:9px; border:1px solid transparent; border-radius:8px; cursor:pointer; }
+.te-binding-option:hover { background:var(--dsw-alias-bg-layer-2); }
+.te-binding-option.is-selected { background:var(--dsw-alias-bg-layer-2); border-color:var(--dsw-alias-border-l2); }
+.te-binding-option.is-fixed { cursor:default; }
+.te-binding-option input { flex:none; margin:3px 0 0; }
+.te-binding-picker input { accent-color:var(--dsw-alias-state-business-primary); }
+.te-binding-picker :focus-visible { outline:2px solid var(--dsw-alias-state-business-primary); outline-offset:2px; }
+.te-binding-detail { min-width:0; display:flex; flex-direction:column; gap:3px; }
+.te-binding-name { font-size:13px; font-weight:600; overflow-wrap:anywhere; }
+.te-binding-description { color:var(--dsw-alias-label-secondary); font-size:12px; line-height:1.4; display:-webkit-box; -webkit-line-clamp:2; -webkit-box-orient:vertical; overflow:hidden; overflow-wrap:anywhere; }
+.te-binding-empty { margin:0; padding:18px 8px; font-size:13px; }
 .te-workbench p,.te-modal p { color:var(--dsw-alias-label-secondary); line-height:1.65; }
 .te-content { max-width:1120px; margin:auto; width:100%; }
 .te-resource { display:flex; flex-direction:column; gap:20px; }
@@ -4613,7 +4634,7 @@ var styles = {
   section: { display: "flex", flexDirection: "column", gap: 12 },
   cardBody: { padding: "4px 0 8px", display: "flex", flexDirection: "column", gap: 12 },
   row: { display: "flex", alignItems: "center", gap: 8, flexWrap: "wrap" },
-  field: { display: "flex", flexDirection: "column", gap: 4, flex: "1 1 220px", minWidth: 180 },
+  field: { display: "flex", flexDirection: "column", gap: 6, minWidth: 0 },
   fieldLabel: { color: textSecondary, fontSize: 13 },
   hint: { color: textTertiary, fontSize: 12, margin: 0, lineHeight: 1.4 },
   chips: { display: "flex", flexWrap: "wrap", gap: 6, alignItems: "center" },
@@ -4719,6 +4740,14 @@ var import_react = require("react");
 var import_dsh_client_ui_primitives = require("@deepseek-ai/dsh-client-ui-primitives");
 
 // src/engine.ts
+function formatResourceRef(ref) {
+  return `${ref.source}:${ref.name}`;
+}
+function parseResourceRef(text2) {
+  const match = /^(bundled|project|user):(.+)$/u.exec(text2);
+  if (match === null) return void 0;
+  return { source: match[1], name: match[2] };
+}
 var GUARD_NAMES = [
   "requirement_confirmation",
   "solution_confirmation",
@@ -4779,15 +4808,26 @@ function validateWorkflow(config) {
     if (!config.stages.includes(stage)) {
       problems.push(`stage_bindings: stage "${stage}" is not a stage`);
     }
-    for (const skill of binding.skills ?? []) {
-      if (typeof skill !== "string" || skill.trim() === "") {
-        problems.push(`stage_bindings "${stage}": empty skill name`);
+    for (const [index, entry] of (binding.skills ?? []).entries()) {
+      if (typeof entry?.skill?.name !== "string" || entry.skill.name.trim() === "") {
+        problems.push(`stage_bindings "${stage}": skill ${index + 1} has no name`);
+        continue;
       }
-    }
-    for (const rule of binding.rules ?? []) {
-      if (typeof rule !== "string" || rule.trim() === "") {
-        problems.push(`stage_bindings "${stage}": empty rule name`);
+      if (!["bundled", "project", "user"].includes(entry.skill.source)) {
+        problems.push(`stage_bindings "${stage}": skill "${entry.skill.name}" has unknown source "${String(entry.skill.source)}"`);
       }
+      for (const rule of entry.rules ?? []) {
+        if (typeof rule?.name !== "string" || rule.name.trim() === "") {
+          problems.push(`stage_bindings "${stage}": skill "${entry.skill.name}" has a rule with no name`);
+          continue;
+        }
+        if (!["bundled", "project", "user"].includes(rule.source)) {
+          problems.push(`stage_bindings "${stage}": skill "${entry.skill.name}" rule "${rule.name}" has unknown source "${String(rule.source)}"`);
+        }
+      }
+    }
+    if ((binding.legacy_rules ?? []).length > 0) {
+      problems.push(`stage_bindings "${stage}": ${binding.legacy_rules.length} legacy stage-level rule(s) are unassigned and still in force; assign each to the skill that should carry it`);
     }
   }
   if (!["task", "item", "manual"].includes(config.commit.policy)) {
@@ -4805,14 +4845,21 @@ function validateWorkflow(config) {
       problems.push("commit.message_pattern is not a valid regular expression");
     }
   }
+  if (config.review_depth !== void 0 && config.review_depth !== "two-stage" && config.review_depth !== "single") {
+    problems.push(`review_depth "${String(config.review_depth)}" is not one of: two-stage, single`);
+  }
   return problems;
 }
 
 // src/workflows.ts
+function bundled(skill, ...rules) {
+  const ref = (name) => ({ source: "bundled", name });
+  return { skill: ref(skill), rules: rules.map(ref) };
+}
 var FLOW_OPTIONS = [
-  { id: "standard", label: "\u6807\u51C6\u7814\u53D1", description: "\u9700\u6C42\u8BC4\u5BA1 \u2192 \u8BBE\u8BA1 \u2192 \u5F00\u53D1 \u2192 \u4EA4\u4ED8 \u2192 \u4EE3\u7801\u5BA1\u6838\uFF0C\u542B\u4EA7\u7269\u95E8 + \u786E\u8BA4\u95E8" },
-  { id: "agile", label: "\u654F\u6377\u8F7B\u91CF", description: "\u9700\u6C42 \u2192 \u5F00\u53D1 \u2192 \u4EA4\u4ED8 \u2192 \u5BA1\u67E5\uFF0C\u56DB\u9636\u6BB5\u3001\u5C11\u4EA7\u7269" },
-  { id: "minimal", label: "\u7EAF\u4EE3\u7801", description: "\u5F00\u53D1 \u2192 \u4EA4\u4ED8\uFF0C\u4E24\u9636\u6BB5\uFF0C\u53EA\u7559\u63D0\u4EA4\u95E8\u7981" }
+  { id: "standard", label: "\u5B8C\u6574\u7814\u53D1", description: "\u65B0\u529F\u80FD\u3001\u67B6\u6784\u6216\u8DE8\u6A21\u5757\u6539\u52A8\u3001\u9AD8\u98CE\u9669\u4EFB\u52A1\uFF1A\u9700\u6C42\u786E\u8BA4 \u2192 \u65B9\u6848\u786E\u8BA4 \u2192 \u5B9E\u73B0 \u2192 \u9A8C\u8BC1 \u2192 \u5BA1\u6838 \u2192 \u63D0\u4EA4" },
+  { id: "agile", label: "\u65E5\u5E38\u8FED\u4EE3", description: "\u76EE\u6807\u660E\u786E\u7684\u5E38\u89C4\u529F\u80FD\u4E0E\u7F3A\u9677\u4FEE\u590D\uFF1A\u76EE\u6807\u4E0E\u9A8C\u6536 \u2192 \u5B9E\u73B0 \u2192 \u9A8C\u6536\u4E0E\u5BA1\u67E5 \u2192 \u63D0\u4EA4" },
+  { id: "minimal", label: "\u5FEB\u901F\u4FEE\u6539", description: "\u5C40\u90E8\u3001\u4F4E\u98CE\u9669\u3001\u65B9\u6848\u660E\u786E\u7684\u6539\u52A8\uFF1A\u4FEE\u6539 \u2192 \u68C0\u67E5\u4E0E\u63D0\u4EA4" }
 ];
 var STANDARD = {
   stages: ["\u9700\u6C42\u8BC4\u5BA1", "\u8BBE\u8BA1", "\u5F00\u53D1", "\u4EA4\u4ED8", "\u4EE3\u7801\u5BA1\u6838", "\u5B8C\u6210"],
@@ -4838,11 +4885,16 @@ var STANDARD = {
   },
   high_risk_requires_verification: true,
   stage_bindings: {
-    "\u9700\u6C42\u8BC4\u5BA1": { skills: ["requirement-analysis"], rules: ["security-redlines"] },
-    "\u8BBE\u8BA1": { skills: ["solution-design"] },
-    "\u5F00\u53D1": { skills: ["code-implement"], rules: ["coding-conventions"] },
-    "\u4EA4\u4ED8": { skills: ["code-verify"], rules: ["coding-conventions"] },
-    "\u4EE3\u7801\u5BA1\u6838": { skills: ["code-review", "code-commit"], rules: ["security-redlines", "commit-conventions"] }
+    "\u9700\u6C42\u8BC4\u5BA1": { skills: [bundled("requirement-analysis", "security-redlines")] },
+    "\u8BBE\u8BA1": { skills: [bundled("solution-design")] },
+    "\u5F00\u53D1": { skills: [bundled("code-implement", "coding-conventions", "security-redlines")] },
+    "\u4EA4\u4ED8": { skills: [bundled("code-verify")] },
+    "\u4EE3\u7801\u5BA1\u6838": {
+      skills: [
+        bundled("code-review", "coding-conventions", "security-redlines"),
+        bundled("code-commit", "commit-conventions")
+      ]
+    }
   }
 };
 var AGILE = {
@@ -4854,21 +4906,32 @@ var AGILE = {
     { from: "\u4EA4\u4ED8", to: "\u5BA1\u67E5", requires: [] }
   ],
   artifacts: [
-    { stage: "\u9700\u6C42", id: "requirement", name: "\u9700\u6C42\u8BF4\u660E", fields: ["scope"] }
+    { stage: "\u9700\u6C42", id: "requirement", name: "\u9700\u6C42\u8BF4\u660E", fields: ["scope"] },
+    // The 审查 stage binds code-review, whose instructions write this artifact with
+    // `record artifact=review`. Declaring it here keeps that binding executable:
+    // the engine refuses an undeclared artifact, so a bound skill pointing at one
+    // would be a contract the preset itself made impossible to satisfy.
+    { stage: "\u5BA1\u67E5", id: "review", name: "\u8BC4\u5BA1\u8BB0\u5F55", fields: ["conclusion", "issues"] }
   ],
   commit: {
+    // `item` policy emits two label shapes: `T<n>` for a mid-flow item commit and
+    // `TASK` for the closing commit at the checkpoint. The pattern below accepts
+    // BOTH, so the label the engine hands the model always satisfies the preset's
+    // own rule. Previously it accepted only `T\d+` while the checkpoint stage
+    // returned `TASK`, which made the closing commit impossible to phrase: the
+    // status line and the validator disagreed about the contract.
     policy: "item",
-    message_pattern: "^\u3010(\\S+)\u3011\u3010T\\d+\u3011.+",
-    message_hint: "\u3010<task_id>\u3011\u3010T1\u3011\u8BF4\u660E \u2014\u2014 \u7B2C\u4E00\u6BB5\u586B\u672C\u4EFB\u52A1 id",
+    message_pattern: "^\u3010(\\S+)\u3011\u3010(?:TASK|T\\d+)\u3011.+",
+    message_hint: "\u3010<task_id>\u3011\u3010TASK/T1\u3011\u8BF4\u660E \u2014\u2014 \u7B2C\u4E00\u6BB5\u586B\u672C\u4EFB\u52A1 id\uFF1B\u5B9E\u65BD\u9879\u63D0\u4EA4\u7528 T1\u3001T2\uFF0C\u6536\u5C3E\u63D0\u4EA4\u7528 TASK",
     checkpoints: ["\u4EA4\u4ED8"],
     file_scope: true
   },
   high_risk_requires_verification: false,
   stage_bindings: {
-    "\u9700\u6C42": { skills: ["requirement-analysis"] },
-    "\u5F00\u53D1": { skills: ["code-implement"], rules: ["coding-conventions"] },
-    "\u4EA4\u4ED8": { skills: ["code-verify", "code-commit"], rules: ["commit-conventions"] },
-    "\u5BA1\u67E5": { skills: ["code-review"] }
+    "\u9700\u6C42": { skills: [bundled("requirement-analysis", "security-redlines")] },
+    "\u5F00\u53D1": { skills: [bundled("code-implement", "coding-conventions", "security-redlines")] },
+    "\u4EA4\u4ED8": { skills: [bundled("code-verify"), bundled("code-commit", "commit-conventions")] },
+    "\u5BA1\u67E5": { skills: [bundled("code-review", "coding-conventions", "security-redlines")] }
   }
 };
 var MINIMAL = {
@@ -4886,9 +4949,14 @@ var MINIMAL = {
     file_scope: false
   },
   high_risk_requires_verification: false,
+  // A fast-change flow checks each change once, not twice under two headings a
+  // short change rarely distinguishes. The item still has to be reviewed — what
+  // drops is the duplicated verdict, which is where the weight actually was: this
+  // flow used to carry the same per-item audit burden as the full one.
+  review_depth: "single",
   stage_bindings: {
-    "\u5F00\u53D1": { skills: ["code-implement"] },
-    "\u4EA4\u4ED8": { skills: ["code-commit"], rules: ["commit-conventions"] }
+    "\u5F00\u53D1": { skills: [bundled("code-implement", "coding-conventions", "security-redlines")] },
+    "\u4EA4\u4ED8": { skills: [bundled("code-commit", "commit-conventions")] }
   }
 };
 function deriveCapabilities(config) {
@@ -4904,9 +4972,16 @@ function preset(id, version, label2, description, config) {
   return { id, version, label: label2, description, capabilities: deriveCapabilities(config), config };
 }
 var FLOW_PRESETS = {
-  standard: preset("standard", 2, "\u6807\u51C6\u7814\u53D1", "\u9700\u6C42\u8BC4\u5BA1 \u2192 \u8BBE\u8BA1 \u2192 \u5F00\u53D1 \u2192 \u4EA4\u4ED8 \u2192 \u4EE3\u7801\u5BA1\u6838\uFF0C\u5BA1\u6838\u540E\u63D0\u4EA4", STANDARD),
-  agile: preset("agile", 1, "\u654F\u6377\u8F7B\u91CF", "\u9700\u6C42 \u2192 \u5F00\u53D1 \u2192 \u4EA4\u4ED8 \u2192 \u5BA1\u67E5\uFF0C\u56DB\u9636\u6BB5\u3001\u5C11\u4EA7\u7269", AGILE),
-  minimal: preset("minimal", 1, "\u7EAF\u4EE3\u7801", "\u5F00\u53D1 \u2192 \u4EA4\u4ED8\uFF0C\u4E24\u9636\u6BB5\uFF0C\u53EA\u7559\u63D0\u4EA4\u95E8\u7981", MINIMAL)
+  standard: preset("standard", 2, "\u5B8C\u6574\u7814\u53D1", "\u65B0\u529F\u80FD\u3001\u67B6\u6784\u6216\u8DE8\u6A21\u5757\u6539\u52A8\u3001\u9AD8\u98CE\u9669\u4EFB\u52A1\uFF1A\u9700\u6C42\u786E\u8BA4 \u2192 \u65B9\u6848\u786E\u8BA4 \u2192 \u5B9E\u73B0 \u2192 \u9A8C\u8BC1 \u2192 \u5BA1\u6838 \u2192 \u63D0\u4EA4", STANDARD),
+  // Version 2: the closing-commit label shape was unified with the message pattern,
+  // and the `review` artifact the bound code-review skill writes was declared. Tasks
+  // created before this keep their frozen version 1 config and are unaffected.
+  agile: preset("agile", 2, "\u65E5\u5E38\u8FED\u4EE3", "\u76EE\u6807\u660E\u786E\u7684\u5E38\u89C4\u529F\u80FD\u4E0E\u7F3A\u9677\u4FEE\u590D\uFF1A\u76EE\u6807\u4E0E\u9A8C\u6536 \u2192 \u5B9E\u73B0 \u2192 \u9A8C\u6536\u4E0E\u5BA1\u67E5 \u2192 \u63D0\u4EA4", AGILE),
+  // Version 2: per-item review depth is declared as `single` instead of inheriting
+  // the full flow's two verdicts per item. Tasks created before this keep their
+  // frozen version 1 config, which has no `review_depth` and therefore reads as
+  // two-stage exactly as it behaved.
+  minimal: preset("minimal", 2, "\u5FEB\u901F\u4FEE\u6539", "\u5C40\u90E8\u3001\u4F4E\u98CE\u9669\u3001\u65B9\u6848\u660E\u786E\u7684\u6539\u52A8\uFF1A\u4FEE\u6539 \u2192 \u68C0\u67E5\u4E0E\u63D0\u4EA4\uFF0C\u6BCF\u9879\u4E00\u6B21\u68C0\u67E5", MINIMAL)
 };
 function dedupe(names) {
   return [...new Set(names)];
@@ -4917,11 +4992,18 @@ function mergeBindings(base, override) {
   const merged = { ...base.stage_bindings };
   for (const [stage, addition] of Object.entries(override)) {
     const existing = merged[stage];
-    const skills = dedupe([...existing?.skills ?? [], ...addition.skills ?? []]);
-    const rules = dedupe([...existing?.rules ?? [], ...addition.rules ?? []]);
+    const skills = [...existing?.skills ?? []];
+    const seen = new Set(skills.map((skill) => formatResourceRef(skill.skill)));
+    for (const skill of addition.skills ?? []) {
+      const key = formatResourceRef(skill.skill);
+      if (seen.has(key)) continue;
+      seen.add(key);
+      skills.push(skill);
+    }
+    const legacy = dedupe([...existing?.legacy_rules ?? [], ...addition.legacy_rules ?? []]);
     const binding = {};
     if (skills.length > 0) binding.skills = skills;
-    if (rules.length > 0) binding.rules = rules;
+    if (legacy.length > 0) binding.legacy_rules = legacy;
     merged[stage] = binding;
     changed = true;
   }
@@ -4937,24 +5019,6 @@ function resolveFlow(flow, override) {
 }
 
 // src/client/shared.ts
-function sourceLabel(source) {
-  switch (source) {
-    case "bundled":
-      return "\u5185\u7F6E";
-    case "project":
-    case "project-dsh":
-    case "project-agents":
-      return "\u9879\u76EE";
-    case "user":
-    case "user-dsh":
-    case "user-agents":
-      return "\u7528\u6237";
-    case "custom":
-      return "\u81EA\u5B9A\u4E49";
-    default:
-      return source;
-  }
-}
 function describeError(error) {
   if (error instanceof Error) return error.message;
   if (typeof error === "string") return error;
@@ -4963,24 +5027,20 @@ function describeError(error) {
 
 // src/client/TaskEngineSection.tsx
 var import_jsx_runtime = require("react/jsx-runtime");
+function refFromText(text2) {
+  return parseResourceRef(text2) ?? { source: "bundled", name: text2 };
+}
 function resolvedConfig(flow, stageBindings) {
   const resolved = resolveFlow(flow, { stage_bindings: stageBindings });
   return resolved.ok ? resolved.config : FLOW_PRESETS.standard.config;
 }
-var cardV = {
-  display: "flex",
-  flexDirection: "column",
-  gap: 8,
-  border: `1px solid ${border}`,
-  borderRadius: 8,
-  padding: "10px 12px"
-};
 function TaskEngineSection(props) {
   const { remote, workspace } = props;
   const [flow, setFlow] = (0, import_react.useState)("standard");
   const [stageBindings, setStageBindings] = (0, import_react.useState)({});
   const [source, setSource] = (0, import_react.useState)("default");
   const [savedAt, setSavedAt] = (0, import_react.useState)("");
+  const [dirty, setDirty] = (0, import_react.useState)(false);
   const [loadError, setLoadError] = (0, import_react.useState)("");
   const [saving, setSaving] = (0, import_react.useState)(false);
   const [loading, setLoading] = (0, import_react.useState)(true);
@@ -5035,8 +5095,12 @@ function TaskEngineSection(props) {
     [flow, stageBindings]
   );
   const selectFlow = (id) => {
+    if (id === flow) return;
+    if (dirty && !window.confirm("\u5F53\u524D\u6D41\u7A0B\u914D\u7F6E\u6709\u672A\u4FDD\u5B58\u7684\u4FEE\u6539\uFF0C\u5207\u6362\u6D41\u7A0B\u4F1A\u4E22\u5F03\u5B83\u4EEC\u3002\u7EE7\u7EED\u5207\u6362\uFF1F")) return;
     setFlow(id);
     setStageBindings({ ...FLOW_PRESETS[id]?.config.stage_bindings ?? {} });
+    setDirty(true);
+    setSavedAt("");
   };
   if (loadError !== "") {
     return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.wrap, children: [
@@ -5067,7 +5131,7 @@ function TaskEngineSection(props) {
       /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { style: styles.sourceText, children: sourceText(source) })
     ] }),
     /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.field, children: [
-      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { style: styles.fieldLabel, children: "\u6D41\u7A0B\u9884\u8BBE" }),
+      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("h3", { children: "\u6D41\u7A0B\u9884\u8BBE" }),
       /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { style: styles.flowGrid, children: FLOW_OPTIONS.map((option) => {
         const active = flow === option.id;
         return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)(
@@ -5089,11 +5153,13 @@ function TaskEngineSection(props) {
           },
           option.id
         );
-      }) }),
-      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("p", { style: styles.hint, children: FLOW_OPTIONS.find((option) => option.id === flow)?.description })
+      }) })
     ] }),
-    /* @__PURE__ */ (0, import_jsx_runtime.jsx)(SectionCard, { icon: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(import_dsh_client_ui_primitives.IconBranchOutline16, { size: 16 }), title: "\u6D41\u7A0B\u8282\u70B9", hint: "\u8BE5\u6D41\u7A0B\u9884\u8BBE\u56FA\u5B9A\u7684\u9636\u6BB5\u987A\u5E8F\uFF1B\u6539\u53D8\u6D41\u7A0B\u8BF7\u5207\u6362\u4E0A\u9762\u7684\u9884\u8BBE\u3002", children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { style: styles.chips, children: stages.map((stage) => /* @__PURE__ */ (0, import_jsx_runtime.jsx)(import_dsh_client_ui_primitives.Pill, { active: stage === (FLOW_PRESETS[flow]?.config.start_stage ?? ""), title: stage === FLOW_PRESETS[flow]?.config.start_stage ? "\u8D77\u59CB\u9636\u6BB5" : void 0, children: stage }, stage)) }) }),
-    /* @__PURE__ */ (0, import_jsx_runtime.jsx)(SectionCard, { icon: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(import_dsh_client_ui_primitives.IconSettingsOutline16, { size: 16 }), title: "\u9636\u6BB5\u6280\u80FD / \u89C4\u5219", hint: "\u9009\u4E00\u4E2A\u8282\u70B9\uFF0C\u52FE\u9009\u5B83\u300C\u7528\u4EC0\u4E48 skill\uFF08\u6280\u80FD\uFF09\u300D\u548C\u300C\u5B88\u54EA\u6761 rule\uFF08\u89C4\u5219\uFF09\u300D\u3002\u6D41\u7A0B\u81EA\u5E26\u7684\u9ED8\u8BA4\u7ED1\u5B9A\u4E0D\u4F1A\u56E0\u53D6\u6D88\u52FE\u9009\u800C\u79FB\u9664\uFF08\u53EA\u4F1A\u4FDD\u7559\u5E76\u8FFD\u52A0\u4F60\u65B0\u52FE\u9009\u7684\uFF09\uFF1B\u8981\u65B0\u5EFA\u6216\u6539\u6B63\u6587\uFF0C\u5207\u5230\u4E0A\u65B9\u7684\u300C\u6280\u80FD\u300D/\u300C\u89C4\u5219\u300D\u6807\u7B7E\u9875\u3002", children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(BindingEditor, { stages, stageBindings, setStageBindings, skills, rules }) }),
+    /* @__PURE__ */ (0, import_jsx_runtime.jsx)(SectionCard, { icon: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(import_dsh_client_ui_primitives.IconSettingsOutline16, { size: 16 }), title: "\u9636\u6BB5\u6280\u80FD / \u89C4\u5219", hint: "\u9009\u62E9\u8282\u70B9\uFF0C\u518D\u52FE\u9009\u9644\u52A0\u6280\u80FD\u4E0E\u89C4\u5219\u3002\u9884\u8BBE\u9ED8\u8BA4\u7ED1\u5B9A\u56FA\u5B9A\u4FDD\u7559\uFF1B\u65B0\u5EFA\u6216\u7F16\u8F91\u5185\u5BB9\u8BF7\u524D\u5F80\u4E0A\u65B9\u201C\u6280\u80FD\u201D\u6216\u201C\u89C4\u5219\u201D\u9875\u3002", children: /* @__PURE__ */ (0, import_jsx_runtime.jsx)(BindingEditor, { stages, defaults: (FLOW_PRESETS[flow] ?? FLOW_PRESETS.standard).config.stage_bindings ?? {}, stageBindings, setStageBindings: (next) => {
+      setStageBindings(next);
+      setDirty(true);
+      setSavedAt("");
+    }, skills, rules }) }),
     problems.length > 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.problems, children: [
       /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { style: styles.problemsTitle, children: [
         /* @__PURE__ */ (0, import_jsx_runtime.jsx)(import_dsh_client_ui_primitives.StateDot, { state: "error" }),
@@ -5117,10 +5183,13 @@ function TaskEngineSection(props) {
           onClick: () => {
             setSaving(true);
             void save(remote, flow, stageBindings, workspace, setSavedAt, setSource).then((ok) => {
-              if (ok) setConfigProblems([]);
+              if (ok) {
+                setConfigProblems([]);
+                setDirty(false);
+              }
             }).finally(() => setSaving(false));
           },
-          children: saving ? "\u6B63\u5728\u4FDD\u5B58\u2026" : "\u4FDD\u5B58\u5230 .dsh/eng.json"
+          children: saving ? "\u6B63\u5728\u4FDD\u5B58\u2026" : dirty ? "\u4FDD\u5B58\u5230 .dsh/eng.json" : "\u5DF2\u4FDD\u5B58 \xB7 \u65E0\u9700\u4FDD\u5B58"
         }
       ),
       savedAt !== "" ? /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { style: styles.okText, children: [
@@ -5147,8 +5216,10 @@ function SectionCard({ icon, title, hint, children }) {
     {
       icon,
       title,
+      titleClassName: "te-disclosure-title",
       open,
       expandable: true,
+      expandOnRowClick: true,
       onToggle: () => {
         setOpen(!open);
       },
@@ -5160,63 +5231,113 @@ function SectionCard({ icon, title, hint, children }) {
     }
   ) });
 }
-function BindingEditor({ stages, stageBindings, setStageBindings, skills, rules }) {
+function BindingEditor({ stages, defaults, stageBindings, setStageBindings, skills, rules }) {
   const [stage, setStage] = (0, import_react.useState)(stages[0] ?? "");
+  const [expanded, setExpanded] = (0, import_react.useState)(null);
   const currentStage = stages.includes(stage) ? stage : stages[0] ?? "";
-  const toggleSkill = (skill) => {
-    if (currentStage === "") return;
-    const current = stageBindings[currentStage]?.skills ?? [];
-    const next = current.includes(skill) ? current.filter((s) => s !== skill) : [...current, skill].sort();
-    setStageBindings({ ...stageBindings, [currentStage]: { ...stageBindings[currentStage] ?? {}, skills: next } });
+  const bindingFor = (name) => stageBindings[name] ?? {};
+  const presetSkills = (name) => defaults[name]?.skills ?? [];
+  const isPresetSkill = (name, raw) => presetSkills(name).some((entry) => formatResourceRef(entry.skill) === raw);
+  const setSkills = (name, next) => {
+    setStageBindings({ ...stageBindings, [name]: { ...bindingFor(name), skills: next } });
+  };
+  const toggleSkill = (raw) => {
+    if (currentStage === "" || isPresetSkill(currentStage, raw)) return;
+    const current = bindingFor(currentStage).skills ?? [];
+    const next = current.some((entry) => formatResourceRef(entry.skill) === raw) ? current.filter((entry) => formatResourceRef(entry.skill) !== raw) : [...current, { skill: refFromText(raw), rules: [] }].sort((a, b) => formatResourceRef(a.skill).localeCompare(formatResourceRef(b.skill)));
+    setSkills(currentStage, next);
   };
-  const toggleRule = (rule) => {
+  const toggleRule = (skillRaw, ruleRaw) => {
     if (currentStage === "") return;
-    const current = stageBindings[currentStage]?.rules ?? [];
-    const next = current.includes(rule) ? current.filter((s) => s !== rule) : [...current, rule].sort();
-    setStageBindings({ ...stageBindings, [currentStage]: { ...stageBindings[currentStage] ?? {}, rules: next } });
+    const current = bindingFor(currentStage).skills ?? [];
+    setSkills(currentStage, current.map((entry) => {
+      if (formatResourceRef(entry.skill) !== skillRaw) return entry;
+      const has = entry.rules.some((rule) => formatResourceRef(rule) === ruleRaw);
+      const next = has ? entry.rules.filter((rule) => formatResourceRef(rule) !== ruleRaw) : [...entry.rules, refFromText(ruleRaw)].sort((a, b) => formatResourceRef(a).localeCompare(formatResourceRef(b)));
+      return { ...entry, rules: next };
+    }));
   };
   if (currentStage === "") {
     return /* @__PURE__ */ (0, import_jsx_runtime.jsx)("p", { style: styles.muted, children: "\u5F53\u524D\u6D41\u7A0B\u6CA1\u6709\u8282\u70B9\u3002" });
   }
-  const boundSkills = stageBindings[currentStage]?.skills ?? [];
-  const boundRules = stageBindings[currentStage]?.rules ?? [];
+  const boundSkills = bindingFor(currentStage).skills ?? [];
+  const unassigned = bindingFor(currentStage).legacy_rules ?? [];
+  const presetCount = presetSkills(currentStage).length;
   return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.section, children: [
-    /* @__PURE__ */ (0, import_jsx_runtime.jsx)("p", { style: styles.hint, children: "\u7ED1\u5B9A\u6309\u540D\u79F0\u751F\u6548\uFF1B\u540C\u540D\u8D44\u6E90\u53EF\u5728\u8D44\u6E90\u7BA1\u7406\u4E2D\u6309\u9879\u76EE\uFF0F\u4E2A\u4EBA\u5206\u522B\u67E5\u770B\u3002\u5B8C\u6210\u8282\u70B9\u7684\u6280\u80FD\u5FC5\u987B\u5728\u8FDB\u5165\u5B8C\u6210\u524D\u6267\u884C\uFF0C\u6D4B\u8BD5\u6280\u80FD\u5EFA\u8BAE\u653E\u5728\u4EA4\u4ED8\u8282\u70B9\u3002" }),
+    /* @__PURE__ */ (0, import_jsx_runtime.jsx)("p", { style: styles.hint, children: "\u8282\u70B9\u53EA\u9009\u62E9\u6280\u80FD\uFF1B\u89C4\u5219\u5C5E\u4E8E\u6280\u80FD\u2014\u2014\u70B9\u5F00\u4E00\u4E2A\u6280\u80FD\u5C31\u80FD\u770B\u5230\u5B83\u5B8C\u6574\u7684\u89C4\u5219\u5217\u8868\uFF0C\u6302\u5230\u4EFB\u4F55\u8282\u70B9\u90FD\u662F\u540C\u4E00\u5957\u89C4\u5219\u3002 \u9700\u8981\u540C\u4E00\u6280\u80FD\u7528\u4E0D\u540C\u89C4\u5219\u65F6\uFF0C\u590D\u5236\u6210\u53E6\u4E00\u4E2A\u6280\u80FD\u518D\u5206\u522B\u914D\u7F6E\u3002" }),
     /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.field, children: [
-      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { style: styles.fieldLabel, children: "\u9009\u62E9\u8282\u70B9\uFF08\u6539\u54EA\u4E2A\u8282\u70B9\u7684\u6302\u8F7D\uFF09" }),
+      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { style: styles.bindingLabel, children: "\u9009\u62E9\u8282\u70B9" }),
       /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { style: styles.chips, children: stages.map((s) => /* @__PURE__ */ (0, import_jsx_runtime.jsx)(import_dsh_client_ui_primitives.Pill, { active: s === currentStage, onClick: () => {
         setStage(s);
       }, children: s }, s)) })
     ] }),
-    /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: cardV, children: [
-      /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.bindingStage, children: [
-        /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { style: styles.fieldLabel, children: "skill\uFF08\u8FD9\u4E2A\u8282\u70B9\u505A\u4EC0\u4E48\uFF09" }),
-        /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { style: styles.chips, children: skills.length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { style: styles.sourceBadge, children: "\u6682\u65E0\u53EF\u7528 skill \u2014\u2014 \u5207\u5230\u4E0A\u65B9\u300C\u6280\u80FD\u300D\u6807\u7B7E\u9875\u65B0\u5EFA" }) : skills.filter((sk, index, all) => all.findIndex((other) => other.name === sk.name) === index).map((sk) => /* @__PURE__ */ (0, import_jsx_runtime.jsxs)(import_dsh_client_ui_primitives.Pill, { active: boundSkills.includes(sk.name), onClick: () => {
-          toggleSkill(sk.name);
-        }, title: `${sk.description}\uFF08${sourceLabel(sk.source)}\uFF09`, children: [
-          sk.name,
-          sk.source !== "bundled" ? /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { style: styles.sourceBadge, children: [
-            "\uFF08",
-            sourceLabel(sk.source),
-            "\uFF09"
-          ] }) : null
-        ] }, sk.name)) })
-      ] }),
-      /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.bindingStage, children: [
-        /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { style: styles.fieldLabel, children: "rule\uFF08\u8FD9\u4E2A\u8282\u70B9\u5B88\u4EC0\u4E48\uFF09" }),
-        /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { style: styles.chips, children: rules.length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { style: styles.sourceBadge, children: "\u6682\u65E0\u53EF\u7528 rule \u2014\u2014 \u5207\u5230\u4E0A\u65B9\u300C\u89C4\u5219\u300D\u6807\u7B7E\u9875\u65B0\u5EFA" }) : rules.filter((rule, index, all) => all.findIndex((other) => other.name === rule.name) === index).map((rule) => /* @__PURE__ */ (0, import_jsx_runtime.jsxs)(import_dsh_client_ui_primitives.Pill, { active: boundRules.includes(rule.name), onClick: () => {
-          toggleRule(rule.name);
-        }, title: `\u6765\u6E90\uFF1A${sourceLabel(rule.source)}`, children: [
-          rule.name,
-          rule.source !== "bundled" ? /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { style: styles.sourceBadge, children: [
-            "\uFF08",
-            sourceLabel(rule.source),
-            "\uFF09"
-          ] }) : null
-        ] }, rule.name)) })
+    unassigned.length > 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { className: "te-alert", children: [
+      "\u4EE5\u4E0B\u89C4\u5219\u5C5E\u4E8E",
+      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("strong", { children: "\u8282\u70B9" }),
+      "\u800C\u4E0D\u662F\u4EFB\u4F55\u6280\u80FD\uFF08\u65E7\u914D\u7F6E\uFF09\uFF1A",
+      unassigned.join("\u3001"),
+      "\u3002 \u5B83\u4EEC\u4ECD\u7136\u751F\u6548\uFF0C\u4F46\u9700\u8981\u6307\u5B9A\u5F52\u5C5E\u2014\u2014\u628A\u6BCF\u6761\u89C4\u5219\u52A0\u5230\u4F60\u5E0C\u671B\u627F\u8F7D\u5B83\u7684\u6280\u80FD\u4E0B\uFF0C\u7136\u540E\u4ECE\u8FD9\u91CC\u79FB\u9664\uFF1B \u82E5\u540C\u4E00\u6280\u80FD\u5728\u4E0D\u540C\u8282\u70B9\u9700\u8981\u4E0D\u540C\u89C4\u5219\uFF0C\u8BF7\u590D\u5236\u8BE5\u6280\u80FD\u518D\u5206\u522B\u914D\u7F6E\u3002"
+    ] }) : null,
+    /* @__PURE__ */ (0, import_jsx_runtime.jsx)("div", { className: "te-binding-grid", children: /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("section", { className: "te-binding-picker", "aria-label": "\u6280\u80FD\u7ED1\u5B9A", children: [
+      /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { className: "te-binding-heading", children: [
+        /* @__PURE__ */ (0, import_jsx_runtime.jsx)("h4", { children: "\u6280\u80FD" }),
+        /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { className: "te-binding-count", children: [
+          "\u5DF2\u9009 ",
+          boundSkills.length,
+          presetCount > 0 ? `\uFF08\u542B ${presetCount} \u9879\u9884\u8BBE\uFF09` : ""
+        ] })
       ] }),
-      boundSkills.length === 0 && boundRules.length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("p", { style: styles.hint, children: "\u8FD9\u4E2A\u8282\u70B9\u8FD8\u6CA1\u6302\u4EFB\u4F55 skill / rule \u2014\u2014 \u70B9\u4E0A\u9762\u7684\u540D\u79F0\u52FE\u9009\u3002" }) : null
-    ] })
+      /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { className: "te-binding-list", role: "group", "aria-label": "\u6280\u80FD\u5217\u8868", tabIndex: 0, children: [
+        skills.length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("p", { className: "te-binding-empty", children: "\u6682\u65E0\u6280\u80FD\uFF0C\u8BF7\u5728\u4E0A\u65B9\u201C\u6280\u80FD\u201D\u9875\u6DFB\u52A0\u3002" }) : null,
+        skills.map((entry) => {
+          const raw = formatResourceRef(entry.ref);
+          const bound = boundSkills.find((candidate) => formatResourceRef(candidate.skill) === raw);
+          const preset2 = isPresetSkill(currentStage, raw);
+          const open = expanded === raw;
+          return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { className: `te-binding-option${bound !== void 0 ? " is-selected" : ""}`, children: [
+            /* @__PURE__ */ (0, import_jsx_runtime.jsx)(
+              "input",
+              {
+                type: "checkbox",
+                "aria-label": raw,
+                checked: bound !== void 0,
+                disabled: preset2,
+                onChange: () => toggleSkill(raw)
+              }
+            ),
+            /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { className: "te-binding-detail", children: [
+              /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { className: "te-binding-name", children: [
+                entry.name,
+                preset2 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { className: "te-source-badge", children: "\u9884\u8BBE" }) : null
+              ] }),
+              /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { className: "te-binding-meta", children: [
+                entry.sourceLabel,
+                " \xB7 ",
+                bound === void 0 ? "\u672A\u7ED1\u5B9A" : `${bound.rules.length} \u6761\u89C4\u5219`
+              ] }),
+              bound !== void 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("button", { type: "button", className: "te-link", onClick: () => setExpanded(open ? null : raw), children: open ? "\u6536\u8D77\u89C4\u5219" : "\u89C4\u5219\u8BBE\u7F6E" }) : null,
+              open && bound !== void 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { className: "te-rule-list", children: [
+                rules.length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { className: "te-binding-meta", children: "\u89C4\u5219\u5E93\u4E3A\u7A7A\u3002" }) : null,
+                rules.map((rule) => {
+                  const ruleRaw = formatResourceRef(rule.ref);
+                  const on = bound.rules.some((candidate) => formatResourceRef(candidate) === ruleRaw);
+                  return /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("label", { className: "te-binding-filter", children: [
+                    /* @__PURE__ */ (0, import_jsx_runtime.jsx)("input", { type: "checkbox", checked: on, onChange: () => toggleRule(raw, ruleRaw) }),
+                    " ",
+                    rule.name,
+                    /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { className: "te-binding-meta", children: [
+                      "\uFF08",
+                      rule.sourceLabel,
+                      "\uFF09"
+                    ] })
+                  ] }, ruleRaw);
+                })
+              ] }) : null
+            ] })
+          ] }, raw);
+        })
+      ] })
+    ] }) }, currentStage)
   ] });
 }
 async function save(remote, flow, stageBindings, workspace, setSavedAt, setSource) {