claude-task-worker 0.46.0 → 0.48.0

package/README.md

@@ -74,7 +74,8 @@ UI実装Issueについて、実装の前に Pencil（`.pen`）でデザインを
 {
   "uiDesign": {
     "enabled": true,
-    "designDir": "designs"
+    "designDir": "designs",
+    "yolo": false
   }
 }
 ```
@@ -83,6 +84,7 @@ UI実装Issueについて、実装の前に Pencil（`.pen`）でデザインを
 |---|---|---|
 | `uiDesign.enabled` | `false` | 本ワークフローの有効化。`false` の間は `triage-created-issue` がUI判定を行わず、2つのワーカーも起動しない |
 | `uiDesign.designDir` | `"designs"` | `.pen` とスナップショットの配置先（リポジトリルートからの相対パス） |
+| `uiDesign.yolo` | `false` | デザインPRを自動レビュー・自動マージへ流すか。`true` のときだけデザインPRに `cc-triage-scope` を付与する。`false`（既定）ではデザインPRは `cc-ui-design` のみが付いた状態で止まり、人がレビュー・マージするまで `apply-ui-design` は先へ進まない |
 
 フローは以下のとおり。
 
@@ -93,8 +95,9 @@ cc-issue-created + triage-created-issue（ルーティング）
         → create-ui-design ワーカー
            ・.pen を作成/更新 + snapshots/ に PNG 出力
            ・ブランチ cc-ui-design-<N> を push しデザインPRを作成（Refs #N。closing keyword は使わない）
-           ・PR に cc-triage-scope + cc-ui-design、Issue に cc-ui-design-pr-created を付与
-        → triage-pr / fix-review-point / resolve-conflict（既存フローでレビュー・マージ）
+           ・PR に cc-ui-design（uiDesign.yolo が true なら cc-triage-scope も）、Issue に cc-ui-design-pr-created を付与
+        → uiDesign.yolo: true  → triage-pr / fix-review-point / resolve-conflict（既存フローでレビュー・マージ）
+           uiDesign.yolo: false → 人がデザインPRをレビュー・マージ
         → apply-ui-design ワーカー
            ・デザインPRが MERGED になるまで skip
            ・Issue description に「## UIデザイン」セクションを追記
@@ -102,7 +105,7 @@ cc-issue-created + triage-created-issue（ルーティング）
         → exec-issue（デザインを参照元として実装）
 ```
 
-デザインPRは Epic PR ではないため `cc-release-ready` によるマージ保留の対象外で、`triage-pr` が通常どおりレビュー・マージする。`triage-pr` は `cc-ui-design` ラベル付きPRに対してコードレビューではなくデザイン向けの観点（差分が `.pen` とPNGに限定されているか、スナップショットとデザイン意図が読み取れるか、Issue要件を満たしているか）で評価する。
+デザインPRのレビュー・マージ経路は `uiDesign.yolo` で切り替わる。`true` のときはデザインPRに `cc-triage-scope` が付き、Epic PR ではないため `cc-release-ready` によるマージ保留の対象外で、`triage-pr` が通常どおりレビュー・マージする。`false`（既定）では `cc-triage-scope` を付けないためどのワーカーもデザインPRを拾わず、人がレビューしてマージするまで待つ（`apply-ui-design` はデザインPRが `MERGED` になるまでスキップし続ける）。デザインに人が介在したい場合は既定のままにする。`triage-pr` は `cc-ui-design` ラベル付きPRに対してコードレビューではなくデザイン向けの観点（差分が `.pen` とPNGに限定されているか、スナップショットとデザイン意図が読み取れるか、Issue要件を満たしているか）で評価する。
 
 デザインが不要と判明した場合は `create-ui-design` が理由をコメントして `cc-ui-design-ready` + `cc-exec-issue` を付与し、人手を介さず実装へ復帰する。Pencil が使えない環境やデザインPRが却下された場合は `cc-need-human-check` に落ちて停止する（共通除外ラベルのため無限リトライしない）。
 
@@ -465,6 +468,7 @@ advisor は main モデル以上の能力を持つモデルである必要があ
 `cc-issue-created` と `cc-triage-scope` の両方のラベルが付いたIssueを定期取得し、Claude Codeでトリアージを実行する。（1分間隔）
 
 - `cc-pr-created` / `cc-update-issue` / `cc-answer-issue-questions` / `cc-exec-issue` のいずれかが付いているIssueは除外
+- `cc-create-ui-design` / `cc-ui-design-pr-created`（UIデザイン先行フローの進行中マーカー）が付いているIssueも除外。トリガーラベルはトリアージ完了後に付き直されるため、除外しないとデザインPRのマージまで再トリアージが走り続け、パターンE-1の前提ゲートが満たされない再実行が `cc-exec-issue` を付けてデザイン合意前に実装を始めてしまう
 - 確認事項の有無に応じて `cc-answer-issue-questions` または `cc-exec-issue` ラベルを付与（または不要ならクローズ）
 
 ### triage-pr
@@ -505,7 +509,7 @@ advisor は main モデル以上の能力を持つモデルである必要があ
 - `claude-task-worker.json` の `uiDesign.enabled` が `true` の場合のみ起動する（`false` なら1行ログを出して即終了）
 - `cc-ui-design-pr-created` / `cc-ui-design-ready` が付いているIssueは除外
 - デザインPRは `cc-ui-design-<Issue番号>` の固定名ブランチを head とする（後段の `apply-ui-design` が一意に特定するため）
-- 完了後、デザインPRの実在を確認できた場合のみ、PRに `cc-ui-design` と `cc-triage-scope`、Issueに `cc-ui-design-pr-created` を付与する
+- 完了後、デザインPRの実在を確認できた場合のみ、PRに `cc-ui-design`（`uiDesign.yolo` が `true` の場合は `cc-triage-scope` も）、Issueに `cc-ui-design-pr-created` を付与する
 - スキルが「デザイン不要」と判定した場合（`cc-ui-design-ready` 付与済み）は完了扱いとし、そのまま実装フェーズへ流す
 - PRの実在を確認できない場合は `cc-need-human-check` を付与してIssueにコメントを残す
 
@@ -580,7 +584,7 @@ claude-task-worker -v
 | キー | 型 | デフォルト | 説明 |
 |---|---|---|---|
 | `fixReviewPointCallbackCommentMessage` | string | - | fix-review-point 完了時にPRへ投稿するコメント（未設定の場合は投稿しない） |
-| `uiDesign` | object | `{ "enabled": false, "designDir": "designs" }` | UIデザイン先行ワークフローの設定（詳細は「[UIデザイン先行ワークフロー](#uiデザイン先行ワークフロー)」） |
+| `uiDesign` | object | `{ "enabled": false, "designDir": "designs", "yolo": false }` | UIデザイン先行ワークフローの設定（詳細は「[UIデザイン先行ワークフロー](#uiデザイン先行ワークフロー)」） |
 | `workers` | object | `{}` | ワーカーごとに Claude CLI に渡すスキル、`--model` / `--advisor` / `--effort`、ポーリング間隔、クールダウン時間、最大同時実行数を上書きする設定（詳細は下記） |
 
 ### ワーカーごとの設定

package/dist/index.js

@@ -1597,7 +1597,8 @@ import { readFileSync } from "node:fs";
 import { isAbsolute, join, normalize, sep as SEP } from "node:path";
 var DEFAULT_UI_DESIGN_CONFIG = {
   enabled: false,
-  designDir: "designs"
+  designDir: "designs",
+  yolo: false
 };
 var DEFAULT_WORKER_CONFIG = {
   skill: "",
@@ -1813,6 +1814,15 @@ function parseUiDesignEntry(val) {
       );
     }
   }
+  if ("yolo" in entry) {
+    if (typeof entry.yolo === "boolean") {
+      result.yolo = entry.yolo;
+    } else {
+      console.warn(
+        `[config] invalid uiDesign.yolo: ${String(entry.yolo)}, using default ${DEFAULT_UI_DESIGN_CONFIG.yolo}`
+      );
+    }
+  }
   if ("designDir" in entry) {
     const normalized = typeof entry.designDir === "string" && entry.designDir.length > 0 ? normalize(entry.designDir) : null;
     const isContained = normalized !== null && !isAbsolute(normalized) && normalized !== ".." && !normalized.startsWith(`..${SEP}`);
@@ -3369,7 +3379,22 @@ var triageCreatedIssueWorker = (opts = {}) => createIssuePollingWorker({
   name: "triage-created-issue",
   command: "/claude-task-worker:triage-created-issue",
   triggerLabels: ["cc-issue-created", "cc-triage-scope"],
-  excludeLabels: ["cc-pr-created", "cc-update-issue", "cc-answer-issue-questions", "cc-exec-issue"],
+  // cc-create-ui-design / cc-ui-design-pr-created はUIデザイン先行フローの進行中マーカー。
+  // 除外しないと、トリガーラベル（cc-issue-created / cc-triage-scope）はトリアージ完了後も
+  // 付き直されるため、デザインPRの作成〜レビュー〜マージが終わるまでの間ずっと再トリアージが
+  // 走り続ける。しかもスキルのパターンE-1は「cc-create-ui-design / cc-ui-design-pr-created が
+  // 付いていない」ことを前提ゲートにしているため、再実行では E-1 がスキップされてパターンE
+  // （cc-exec-issue 付与）へ落ち、デザイン合意前に実装フェーズが始まってしまう。
+  // cc-ui-design-ready はデザイン確定後のマーカーで、同時に付く cc-exec-issue 側で除外されるため
+  // ここには含めない（実装完了後の再トリアージ可能性を残す）。
+  excludeLabels: [
+    "cc-pr-created",
+    "cc-update-issue",
+    "cc-answer-issue-questions",
+    "cc-exec-issue",
+    "cc-create-ui-design",
+    "cc-ui-design-pr-created"
+  ],
   epicFilters: opts.epicFilters,
   labelFilters: opts.labelFilters,
   onCompleted: async (issueNumber) => {
@@ -3518,11 +3543,12 @@ function designReferenceMissingComment(issueNumber) {
 }
 
 // src/workers/create-ui-design.ts
-function designPrLabelingFailedComment(issueNumber, prNumber, error) {
+function designPrLabelingFailedComment(issueNumber, prNumber, error, yolo = false) {
   const message = error instanceof Error ? error.message : String(error);
+  const prLabels = yolo ? "`cc-ui-design` / `cc-triage-scope`" : "`cc-ui-design`";
   return [
     "## \u30C7\u30B6\u30A4\u30F3PR\u3078\u306E\u30E9\u30D9\u30EB\u4ED8\u4E0E\u306B\u5931\u6557\u3057\u307E\u3057\u305F\uFF08\u8981\u4EBA\u624B\u78BA\u8A8D\uFF09",
-    `\u30C7\u30B6\u30A4\u30F3PR #${prNumber}\uFF08\u30D6\u30E9\u30F3\u30C1 \`${designBranchName(issueNumber)}\`\uFF09\u306F\u4F5C\u6210\u3055\u308C\u307E\u3057\u305F\u304C\u3001\u5F8C\u7D9A\u30E9\u30D9\u30EB\uFF08\`cc-ui-design\` / \`cc-triage-scope\` / \`cc-ui-design-pr-created\`\uFF09\u306E\u4ED8\u4E0E\u306B\u5931\u6557\u3057\u307E\u3057\u305F\u3002`,
+    `\u30C7\u30B6\u30A4\u30F3PR #${prNumber}\uFF08\u30D6\u30E9\u30F3\u30C1 \`${designBranchName(issueNumber)}\`\uFF09\u306F\u4F5C\u6210\u3055\u308C\u307E\u3057\u305F\u304C\u3001\u5F8C\u7D9A\u30E9\u30D9\u30EB\uFF08${prLabels} / \`cc-ui-design-pr-created\`\uFF09\u306E\u4ED8\u4E0E\u306B\u5931\u6557\u3057\u307E\u3057\u305F\u3002`,
     "",
     "## \u8D77\u3053\u308A\u3046\u308B\u539F\u56E0",
     "- \u672C\u30EF\u30FC\u30AF\u30D5\u30ED\u30FC\u8FFD\u52A0\u6642\u306E\u30E9\u30D9\u30EB\u304C\u672A\u4F5C\u6210\u306E\u53EF\u80FD\u6027\u304C\u3042\u308A\u307E\u3059\u3002`claude-task-worker init` \u3092\u518D\u5B9F\u884C\u3057\u3066\u30E9\u30D9\u30EB\u3092\u4F5C\u6210\u3057\u3066\u304F\u3060\u3055\u3044",
@@ -3533,14 +3559,16 @@ function designPrLabelingFailedComment(issueNumber, prNumber, error) {
     "```",
     "",
     "## \u5BFE\u5FDC\u5F8C\u306E\u9032\u3081\u65B9",
-    "- \u30E9\u30D9\u30EB\u4F5C\u6210\u5F8C\u306B\u3084\u308A\u76F4\u3059\u5834\u5408: `cc-need-human-check` \u30E9\u30D9\u30EB\u3092\u5916\u3057\u3001`cc-ui-design`\u30FB`cc-triage-scope`\uFF08PR\u5074\uFF09\u3068 `cc-ui-design-pr-created`\uFF08Issue\u5074\uFF09\u3092\u624B\u52D5\u3067\u4ED8\u3051\u3066\u304F\u3060\u3055\u3044"
+    `- \u30E9\u30D9\u30EB\u4F5C\u6210\u5F8C\u306B\u3084\u308A\u76F4\u3059\u5834\u5408: \`cc-need-human-check\` \u30E9\u30D9\u30EB\u3092\u5916\u3057\u3001${yolo ? "`cc-ui-design`\u30FB`cc-triage-scope`" : "`cc-ui-design`"}\uFF08PR\u5074\uFF09\u3068 \`cc-ui-design-pr-created\`\uFF08Issue\u5074\uFF09\u3092\u624B\u52D5\u3067\u4ED8\u3051\u3066\u304F\u3060\u3055\u3044`
   ].join("\n");
 }
 var createUiDesignWorker = async (opts = {}) => {
-  if (!getUiDesignConfig().enabled) {
+  const uiDesignConfig = getUiDesignConfig();
+  if (!uiDesignConfig.enabled) {
     console.log("[create-ui-design] uiDesign.enabled is false, skipping");
     return;
   }
+  const yolo = uiDesignConfig.yolo;
   await createIssuePollingWorker({
     name: "create-ui-design",
     command: "/claude-task-worker:create-ui-design",
@@ -3571,12 +3599,18 @@ var createUiDesignWorker = async (opts = {}) => {
       }
       try {
         await addLabel("pr", prNumber, "cc-ui-design");
-        await addLabel("pr", prNumber, "cc-triage-scope");
+        if (yolo) {
+          await addLabel("pr", prNumber, "cc-triage-scope");
+        } else {
+          console.log(
+            `[create-ui-design] #${issueNumber}: uiDesign.yolo is false, leaving design PR #${prNumber} for human review (no cc-triage-scope)`
+          );
+        }
         await addLabel("issue", issueNumber, "cc-ui-design-pr-created");
       } catch (err) {
         console.error(`[create-ui-design] #${issueNumber}: labeling design PR #${prNumber} failed: ${err}`);
         await addLabel("issue", issueNumber, "cc-need-human-check");
-        await commentOnIssue(issueNumber, designPrLabelingFailedComment(issueNumber, prNumber, err)).catch(
+        await commentOnIssue(issueNumber, designPrLabelingFailedComment(issueNumber, prNumber, err, yolo)).catch(
           (commentErr) => console.error(`[create-ui-design] commentOnIssue failed for #${issueNumber}: ${commentErr}`)
         );
         return false;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-task-worker",
-  "version": "0.46.0",
+  "version": "0.48.0",
   "description": "CLI tool that polls GitHub Issues/PRs and delegates work to Claude CLI",
   "type": "module",
   "main": "dist/index.js",