@godv61/dsh-task-engine 0.24.0 → 0.25.0

package/lib/client.js

@@ -4557,6 +4557,14 @@ var workbenchTheme = `
 .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; }
+/* Search + filter row. The catalog grows with every install, so finding one skill
+   has to be a lookup rather than a scroll. */
+.te-binding-controls { display:flex; gap:8px; align-items:center; flex-wrap:wrap; margin:0 0 6px; }
+.te-binding-search { flex:1; min-width:120px; padding:6px 9px; font-size:12px; color:var(--dsw-alias-label-primary); background:var(--dsw-alias-bg-layer-1); border:1px solid var(--dsw-alias-border-l2); border-radius:7px; }
+.te-binding-search:focus-visible { outline:2px solid var(--dsw-alias-state-business-primary); outline-offset:1px; }
+/* Only the open skill's rules render, and inside a bounded region: expanding a
+   second skill must not push the first off screen or grow the page without limit. */
+.te-rule-list { display:flex; flex-direction:column; gap:4px; margin-top:6px; padding:8px; max-height:240px; overflow-y:auto; overscroll-behavior:contain; border:1px solid var(--dsw-alias-border-l2); border-radius:8px; background:var(--dsw-alias-bg-layer-1); 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); }
@@ -4695,6 +4703,10 @@ var styles = {
   },
   // ── flow preset picker ────────────────────────────────────────────────────
   flowGrid: { display: "grid", gridTemplateColumns: "repeat(auto-fit, minmax(230px, 1fr))", gap: 10 },
+  // The adoption row is deliberately separated from the flow picker: choosing a
+  // skeleton and adopting a recommended setup are different decisions, and the
+  // button is the only thing that writes method into the user's config.
+  adoptRow: { display: "flex", alignItems: "center", gap: 10, flexWrap: "wrap", marginBottom: 4 },
   flowOption: {
     display: "flex",
     flexDirection: "column",
@@ -4871,19 +4883,23 @@ var STANDARD = {
     { from: "\u4EA4\u4ED8", to: "\u4EE3\u7801\u5BA1\u6838", requires: ["verified"] },
     { from: "\u4EE3\u7801\u5BA1\u6838", to: "\u5B8C\u6210", requires: ["review_passed", "artifacts_present"] }
   ],
+  // No artifacts, no commit policy, no skills: the skeleton carries none of them.
+  artifacts: [],
+  commit: { policy: "task", message_pattern: "", message_hint: "\u65E0\u683C\u5F0F\u8981\u6C42", checkpoints: ["\u4EE3\u7801\u5BA1\u6838"], file_scope: true },
+  high_risk_requires_verification: true
+};
+var STANDARD_RECOMMENDATION = {
+  commit: {
+    // Only the TEXT conventions. Policy, checkpoints and scope stay on the
+    // skeleton: they decide when a gate fires, not how a message reads.
+    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\uFF08\u5982 GREET-001\uFF09\uFF0C\u5199\u7ED3\u679C\u4E0D\u5199\u7A7A\u6CDB\u52A8\u4F5C"
+  },
   artifacts: [
     { stage: "\u9700\u6C42\u8BC4\u5BA1", id: "requirement", name: "\u9700\u6C42\u8BF4\u660E", fields: ["scope", "acceptance_criteria"] },
     { stage: "\u8BBE\u8BA1", id: "design", name: "\u8BBE\u8BA1\u6587\u6863", fields: ["approach", "risks", "impact"] },
     { stage: "\u4EE3\u7801\u5BA1\u6838", id: "review", name: "\u8BC4\u5BA1\u8BB0\u5F55", fields: ["conclusion", "issues"] }
   ],
-  commit: {
-    policy: "task",
-    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\uFF08\u5982 GREET-001\uFF09\uFF0C\u5199\u7ED3\u679C\u4E0D\u5199\u7A7A\u6CDB\u52A8\u4F5C",
-    checkpoints: ["\u4EE3\u7801\u5BA1\u6838"],
-    file_scope: true
-  },
-  high_risk_requires_verification: true,
   stage_bindings: {
     "\u9700\u6C42\u8BC4\u5BA1": { skills: [bundled("requirement-analysis", "security-redlines")] },
     "\u8BBE\u8BA1": { skills: [bundled("solution-design")] },
@@ -4903,30 +4919,38 @@ var AGILE = {
   transitions: [
     { from: "\u9700\u6C42", to: "\u5F00\u53D1", requires: ["requirement_confirmation"] },
     { from: "\u5F00\u53D1", to: "\u4EA4\u4ED8", requires: ["todos_done"] },
+    // Entering 审查 is unconditional. A review guard on this edge would be
+    // circular — it would demand the verdict BEFORE the stage that produces it —
+    // and it would also block the commit at 交付, since a commit requires its
+    // stage's outgoing guards. What 审查 requires is declared as completion_guards
+    // instead, because a terminal stage has no outgoing edge to carry it.
     { from: "\u4EA4\u4ED8", to: "\u5BA1\u67E5", requires: [] }
   ],
+  artifacts: [],
+  commit: { policy: "item", message_pattern: "", message_hint: "\u65E0\u683C\u5F0F\u8981\u6C42", checkpoints: ["\u4EA4\u4ED8"], file_scope: true },
+  high_risk_requires_verification: false,
+  // 审查 is terminal and IS the review, so there is no outgoing edge to carry
+  // the guard. Declaring it here is what makes a blocked review prevent
+  // completion; without it the stage could be reached and finished with the
+  // review failing, which is what the assessment found.
+  completion_guards: ["review_passed"]
+};
+var AGILE_RECOMMENDATION = {
+  commit: {
+    // `item` policy emits two label shapes: `T<n>` mid-flow and `TASK` at the
+    // closing checkpoint. The pattern accepts BOTH, so the label the engine hands
+    // the model always satisfies the rule that validates it.
+    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"
+  },
   artifacts: [
     { 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.
+    // `record artifact=review`. Declaring it keeps that binding executable: the
+    // engine refuses an undeclared artifact, so a bound skill pointing at one would
+    // be a contract the recommendation 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\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: [bundled("requirement-analysis", "security-redlines")] },
     "\u5F00\u53D1": { skills: [bundled("code-implement", "coding-conventions", "security-redlines")] },
@@ -4941,18 +4965,13 @@ var MINIMAL = {
     { from: "\u5F00\u53D1", to: "\u4EA4\u4ED8", requires: ["todos_done"] }
   ],
   artifacts: [],
-  commit: {
-    policy: "task",
-    message_pattern: "",
-    message_hint: "\u65E0\u683C\u5F0F\u8981\u6C42",
-    checkpoints: ["\u4EA4\u4ED8"],
-    file_scope: false
-  },
-  high_risk_requires_verification: false,
+  commit: { policy: "task", message_pattern: "", message_hint: "\u65E0\u683C\u5F0F\u8981\u6C42", checkpoints: ["\u4EA4\u4ED8"], file_scope: false },
+  high_risk_requires_verification: false
+};
+var MINIMAL_RECOMMENDATION = {
   // 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.
+  // drops is the duplicated verdict, which is where the weight actually was.
   review_depth: "single",
   stage_bindings: {
     "\u5F00\u53D1": { skills: [bundled("code-implement", "coding-conventions", "security-redlines")] },
@@ -4968,54 +4987,81 @@ function deriveCapabilities(config) {
   if (config.commit.file_scope) caps.push("file_scope");
   return caps;
 }
-function preset(id, version, label2, description, config) {
-  return { id, version, label: label2, description, capabilities: deriveCapabilities(config), config };
+function preset(id, version, label2, description, config, recommendation) {
+  return {
+    id,
+    version,
+    label: label2,
+    description,
+    capabilities: deriveCapabilities(config),
+    config,
+    // exactOptionalPropertyTypes: omit rather than assign undefined.
+    ...recommendation !== void 0 ? { recommendation } : {}
+  };
 }
 var FLOW_PRESETS = {
-  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)
+  standard: preset("standard", 3, "\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, STANDARD_RECOMMENDATION),
+  // Version 4 for agile: 交付 → 审查 now requires review_passed, where it previously
+  // required nothing. That is a gate-semantics change, so a task created before it
+  // keeps its frozen version 3 config and behaves exactly as it did.
+  //
+  // Version 3 across all three: the preset is now a bare skeleton and its former
+  // built-in bindings, commit rule and artifacts are a separate recommendation a
+  // user adopts explicitly. Tasks created before this keep their frozen config and
+  // are unaffected; a project's existing config is likewise left exactly as it is.
+  agile: preset("agile", 4, "\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, AGILE_RECOMMENDATION),
+  minimal: preset("minimal", 3, "\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, MINIMAL_RECOMMENDATION)
 };
 function dedupe(names) {
   return [...new Set(names)];
 }
-function mergeBindings(base, override) {
-  if (override === void 0) return base;
-  let changed = false;
-  const merged = { ...base.stage_bindings };
-  for (const [stage, addition] of Object.entries(override)) {
-    const existing = merged[stage];
-    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 (legacy.length > 0) binding.legacy_rules = legacy;
-    merged[stage] = binding;
-    changed = true;
-  }
-  if (!changed) return base;
-  return { ...base, stage_bindings: merged };
+function applyProjectConfig(base, project) {
+  if (project === void 0) return base;
+  const next = { ...base };
+  if (project.stage_bindings !== void 0) {
+    const bindings = {};
+    for (const [stage, binding] of Object.entries(project.stage_bindings)) {
+      const entry = {};
+      if ((binding.skills ?? []).length > 0) entry.skills = [...binding.skills];
+      const legacy = dedupe(binding.legacy_rules ?? []);
+      if (legacy.length > 0) entry.legacy_rules = legacy;
+      bindings[stage] = entry;
+    }
+    next.stage_bindings = bindings;
+  }
+  if (project.commit !== void 0) next.commit = project.commit;
+  if (project.artifacts !== void 0) next.artifacts = project.artifacts;
+  if (project.review_depth !== void 0) next.review_depth = project.review_depth;
+  if (project.commit_required !== void 0) next.commit_required = project.commit_required;
+  return next;
+}
+function adoptRecommendation(presetId, current) {
+  const preset2 = FLOW_PRESETS[presetId];
+  if (preset2 === void 0) return void 0;
+  const rec = preset2.recommendation;
+  if (rec === void 0) return { flow: presetId, ...current };
+  const commit = {
+    ...preset2.config.commit,
+    ...rec.commit ?? {},
+    ...current?.commit ?? {}
+  };
+  const config = {
+    flow: presetId,
+    stage_bindings: current?.stage_bindings ?? rec.stage_bindings ?? {},
+    commit,
+    artifacts: current?.artifacts ?? rec.artifacts ?? preset2.config.artifacts,
+    ...current?.review_depth !== void 0 ? { review_depth: current.review_depth } : rec.review_depth !== void 0 ? { review_depth: rec.review_depth } : {},
+    ...current?.commit_required !== void 0 ? { commit_required: current.commit_required } : {}
+  };
+  return config;
 }
-function resolveFlow(flow, override) {
+function resolveFlow(flow, project) {
   const preset2 = FLOW_PRESETS[flow];
   if (preset2 === void 0) {
     return { ok: false, code: "UNKNOWN_FLOW", flow, knownFlows: Object.keys(FLOW_PRESETS) };
   }
-  return { ok: true, config: mergeBindings(preset2.config, override?.stage_bindings), preset: preset2 };
+  const { flow: _ignored, ...rest } = project ?? { flow };
+  return { ok: true, config: applyProjectConfig(preset2.config, { flow, ...rest }), preset: preset2 };
 }
 
 // src/client/shared.ts
@@ -5031,13 +5077,16 @@ function refFromText(text2) {
   return parseResourceRef(text2) ?? { source: "bundled", name: text2 };
 }
 function resolvedConfig(flow, stageBindings) {
-  const resolved = resolveFlow(flow, { stage_bindings: stageBindings });
+  const resolved = resolveFlow(flow, { flow, stage_bindings: stageBindings });
   return resolved.ok ? resolved.config : FLOW_PRESETS.standard.config;
 }
 function TaskEngineSection(props) {
   const { remote, workspace } = props;
   const [flow, setFlow] = (0, import_react.useState)("standard");
   const [stageBindings, setStageBindings] = (0, import_react.useState)({});
+  const [commitRule, setCommitRule] = (0, import_react.useState)(void 0);
+  const [artifacts, setArtifacts] = (0, import_react.useState)(void 0);
+  const [reviewDepth, setReviewDepth] = (0, import_react.useState)(void 0);
   const [source, setSource] = (0, import_react.useState)("default");
   const [savedAt, setSavedAt] = (0, import_react.useState)("");
   const [dirty, setDirty] = (0, import_react.useState)(false);
@@ -5059,6 +5108,9 @@ function TaskEngineSection(props) {
       if (result.ok) {
         setFlow(result.value.flow);
         setStageBindings(result.value.config.stage_bindings ?? {});
+        setCommitRule(result.value.config.commit);
+        setArtifacts(result.value.config.artifacts);
+        setReviewDepth(result.value.config.review_depth);
         setSource(result.value.source);
         setConfigProblems(result.value.problems);
         setSavedAt("");
@@ -5098,7 +5150,17 @@ function TaskEngineSection(props) {
     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("");
+  };
+  const adopt = () => {
+    const adopted = adoptRecommendation(flow);
+    if (adopted === void 0) return;
+    if (dirty && !window.confirm("\u91C7\u7528\u63A8\u8350\u914D\u7F6E\u4F1A\u66FF\u6362\u5F53\u524D\u7684\u6280\u80FD\u4E0E\u89C4\u5219\u8BBE\u7F6E\u3002\u7EE7\u7EED\uFF1F")) return;
+    setStageBindings({ ...adopted.stage_bindings ?? {} });
+    setCommitRule(adopted.commit);
+    setArtifacts(adopted.artifacts);
+    setReviewDepth(adopted.review_depth);
     setDirty(true);
     setSavedAt("");
   };
@@ -5124,7 +5186,15 @@ function TaskEngineSection(props) {
     ] }),
     /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.head, children: [
       /* @__PURE__ */ (0, import_jsx_runtime.jsx)("h2", { style: styles.title, children: "\u5DE5\u7A0B\u6D41\u7A0B\u914D\u7F6E" }),
-      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("p", { style: styles.muted, children: "\u9009\u4E00\u5957\u6D41\u7A0B\uFF0C\u518D\u7ED9\u6BCF\u4E2A\u8282\u70B9\u6302\u4E0A skill / rule\u3002\u9636\u6BB5\u6D41\u8F6C\u3001\u5B88\u536B\u3001\u63D0\u4EA4\u89C4\u5219\u90FD\u7531\u6D41\u7A0B\u9884\u8BBE\u56FA\u5316\uFF0C\u65E0\u9700\u624B\u5DE5\u914D\u7F6E\u3002" })
+      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("p", { style: styles.muted, children: "\u6D41\u7A0B\u51B3\u5B9A\u5DE5\u4F5C\u600E\u4E48\u6D41\u8F6C\uFF08\u9636\u6BB5\u4E0E\u95E8\u7981\uFF09\uFF1B\u6280\u80FD\u3001\u89C4\u5219\u3001\u63D0\u4EA4\u683C\u5F0F\u548C\u4EA7\u7269\u5B57\u6BB5\u7531\u4F60\u81EA\u5DF1\u914D\u7F6E\u3002 \u9700\u8981\u4E00\u4EFD\u73B0\u6210\u7684\u8D77\u70B9\u65F6\uFF0C\u53EF\u4EE5\u300C\u91C7\u7528\u63A8\u8350\u914D\u7F6E\u300D\u2014\u2014\u91C7\u7528\u540E\u8FD9\u4E9B\u5C31\u5B8C\u5168\u5C5E\u4E8E\u4F60\uFF0C\u53EF\u968F\u610F\u4FEE\u6539\u6216\u5220\u9664\u3002" })
+    ] }),
+    /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.adoptRow, children: [
+      /* @__PURE__ */ (0, import_jsx_runtime.jsxs)(import_dsh_client_ui_primitives.Button, { size: "sm", onClick: adopt, children: [
+        "\u91C7\u7528\u300C",
+        FLOW_PRESETS[flow]?.label ?? flow,
+        "\u300D\u7684\u63A8\u8350\u914D\u7F6E"
+      ] }),
+      /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { style: styles.muted, children: "\u5199\u5165\u63A8\u8350\u6280\u80FD\u3001\u63D0\u4EA4\u4FE1\u606F\u683C\u5F0F\u4E0E\u4EA7\u7269\u5B57\u6BB5\u3002\u4EC5\u5728\u4F60\u70B9\u51FB\u65F6\u53D1\u751F\u4E00\u6B21\uFF1B\u5220\u9664\u540E\u4E0D\u4F1A\u81EA\u52A8\u8865\u56DE\u3002" })
     ] }),
     /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { style: styles.sourceRow, children: [
       /* @__PURE__ */ (0, import_jsx_runtime.jsx)(import_dsh_client_ui_primitives.StateDot, { state: source === "project" ? "done" : source === "invalid" ? "warning" : "ongoing" }),
@@ -5182,7 +5252,7 @@ function TaskEngineSection(props) {
           disabled: problems.length > 0 || saving,
           onClick: () => {
             setSaving(true);
-            void save(remote, flow, stageBindings, workspace, setSavedAt, setSource).then((ok) => {
+            void save(remote, { flow, stage_bindings: stageBindings, ...commitRule !== void 0 ? { commit: commitRule } : {}, ...artifacts !== void 0 ? { artifacts } : {}, ...reviewDepth !== void 0 ? { review_depth: reviewDepth } : {} }, workspace, setSavedAt, setSource).then((ok) => {
               if (ok) {
                 setConfigProblems([]);
                 setDirty(false);
@@ -5234,10 +5304,36 @@ function SectionCard({ icon, title, hint, children }) {
 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 [query, setQuery] = (0, import_react.useState)("");
+  const [onlySelected, setOnlySelected] = (0, import_react.useState)(false);
+  const [ruleQuery, setRuleQuery] = (0, import_react.useState)("");
+  const [ruleOnlySelected, setRuleOnlySelected] = (0, import_react.useState)(false);
   const currentStage = stages.includes(stage) ? stage : stages[0] ?? "";
   const bindingFor = (name) => stageBindings[name] ?? {};
   const presetSkills = (name) => defaults[name]?.skills ?? [];
   const isPresetSkill = (name, raw) => presetSkills(name).some((entry) => formatResourceRef(entry.skill) === raw);
+  const visibleSkills = () => {
+    const bound = bindingFor(currentStage).skills ?? [];
+    const boundKeys = new Set(bound.map((entry) => formatResourceRef(entry.skill)));
+    const needle = query.trim().toLowerCase();
+    return skills.filter((entry) => {
+      const raw = formatResourceRef(entry.ref);
+      const inForce = boundKeys.has(raw) || isPresetSkill(currentStage, raw);
+      if (onlySelected && !inForce) return false;
+      if (needle === "") return true;
+      return inForce || entry.name.toLowerCase().includes(needle) || entry.description.toLowerCase().includes(needle) || entry.sourceLabel.toLowerCase().includes(needle);
+    });
+  };
+  const visibleRulesFor = (bound) => {
+    const chosen = new Set(bound.rules.map((rule) => formatResourceRef(rule)));
+    const needle = ruleQuery.trim().toLowerCase();
+    return rules.filter((rule) => {
+      const raw = formatResourceRef(rule.ref);
+      if (ruleOnlySelected && !chosen.has(raw)) return false;
+      if (needle === "") return true;
+      return chosen.has(raw) || rule.name.toLowerCase().includes(needle) || rule.sourceLabel.toLowerCase().includes(needle);
+    });
+  };
   const setSkills = (name, next) => {
     setStageBindings({ ...stageBindings, [name]: { ...bindingFor(name), skills: next } });
   };
@@ -5284,12 +5380,32 @@ function BindingEditor({ stages, defaults, stageBindings, setStageBindings, skil
         /* @__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` : ""
+          presetCount > 0 ? `\uFF08\u542B ${presetCount} \u9879\u9884\u8BBE\uFF09` : "",
+          visibleSkills().length !== skills.length ? ` \xB7 \u663E\u793A ${visibleSkills().length}/${skills.length}` : ""
+        ] })
+      ] }),
+      /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("div", { className: "te-binding-controls", children: [
+        /* @__PURE__ */ (0, import_jsx_runtime.jsx)(
+          "input",
+          {
+            className: "te-binding-search",
+            type: "search",
+            value: query,
+            placeholder: "\u641C\u7D22\u6280\u80FD\u540D\u6216\u8BF4\u660E\u2026",
+            "aria-label": "\u641C\u7D22\u6280\u80FD",
+            onChange: (event) => setQuery(event.target.value)
+          }
+        ),
+        /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("label", { className: "te-binding-filter", children: [
+          /* @__PURE__ */ (0, import_jsx_runtime.jsx)("input", { type: "checkbox", checked: onlySelected, onChange: () => setOnlySelected((value) => !value) }),
+          " ",
+          "\u4EC5\u770B\u5DF2\u9009"
         ] })
       ] }),
       /* @__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) => {
+        skills.length > 0 && visibleSkills().length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("p", { className: "te-binding-empty", children: "\u6CA1\u6709\u5339\u914D\u7684\u6280\u80FD\u3002" }) : null,
+        visibleSkills().map((entry) => {
           const raw = formatResourceRef(entry.ref);
           const bound = boundSkills.find((candidate) => formatResourceRef(candidate.skill) === raw);
           const preset2 = isPresetSkill(currentStage, raw);
@@ -5317,8 +5433,29 @@ function BindingEditor({ stages, defaults, stageBindings, setStageBindings, skil
               ] }),
               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: [
+                ruleQuery !== "" || ruleOnlySelected ? null : /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("span", { className: "te-binding-meta", children: [
+                  rules.length,
+                  " \u6761\u53EF\u7528\u89C4\u5219"
+                ] }),
+                /* @__PURE__ */ (0, import_jsx_runtime.jsx)(
+                  "input",
+                  {
+                    className: "te-binding-search",
+                    type: "search",
+                    value: ruleQuery,
+                    placeholder: "\u8FC7\u6EE4\u89C4\u5219\u2026",
+                    "aria-label": `\u8FC7\u6EE4 ${entry.name} \u7684\u89C4\u5219`,
+                    onChange: (event) => setRuleQuery(event.target.value)
+                  }
+                ),
+                /* @__PURE__ */ (0, import_jsx_runtime.jsxs)("label", { className: "te-binding-filter", children: [
+                  /* @__PURE__ */ (0, import_jsx_runtime.jsx)("input", { type: "checkbox", checked: ruleOnlySelected, onChange: () => setRuleOnlySelected((value) => !value) }),
+                  " ",
+                  "\u4EC5\u770B\u5DF2\u9009"
+                ] }),
                 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) => {
+                rules.length > 0 && visibleRulesFor(bound).length === 0 ? /* @__PURE__ */ (0, import_jsx_runtime.jsx)("span", { className: "te-binding-meta", children: "\u6CA1\u6709\u5339\u914D\u7684\u89C4\u5219\u3002" }) : null,
+                visibleRulesFor(bound).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: [
@@ -5340,9 +5477,9 @@ function BindingEditor({ stages, defaults, stageBindings, setStageBindings, skil
     ] }) }, currentStage)
   ] });
 }
-async function save(remote, flow, stageBindings, workspace, setSavedAt, setSource) {
+async function save(remote, config, workspace, setSavedAt, setSource) {
   try {
-    const result = await remote.write({ path: workspace, flow, stage_bindings: stageBindings });
+    const result = await remote.write({ path: workspace, ...config });
     if (!result.ok) {
       setSavedAt("\u4FDD\u5B58\u5931\u8D25\uFF1A" + describeError(result.error));
       return false;