claude-task-worker 0.111.0 → 0.113.0

package/dist/index.js

@@ -1486,6 +1486,27 @@ async function fetchPrRef(owner, name, prNumber) {
     return null;
   }
 }
+async function getPrDetail(prNumber) {
+  const { owner, name } = await getRepoInfo();
+  return fetchPrRef(owner, name, prNumber);
+}
+async function closeIssue(issueNumber, reason = "completed") {
+  const { owner, name } = await getRepoInfo();
+  await execGh([
+    "api",
+    "-X",
+    "PATCH",
+    `repos/${owner}/${name}/issues/${issueNumber}`,
+    "-f",
+    "state=closed",
+    "-f",
+    `state_reason=${reason}`
+  ]);
+}
+function parseClosingIssueNumbers(body) {
+  const pattern = /\b(?:close[sd]?|fix(?:e[sd])?|resolve[sd]?)\s*:?\s*#(\d+)\b(?!\d)/gi;
+  return [...new Set([...body.matchAll(pattern)].map((m) => Number(m[1])))];
+}
 function bodyClosesIssue(body, issueNumber) {
   const pattern = new RegExp(`\\b(close[sd]?|fix(?:e[sd])?|resolve[sd]?)\\s*:?\\s*#${issueNumber}\\b(?!\\d)`, "i");
   return pattern.test(body);
@@ -4389,11 +4410,27 @@ var triageCreatedIssueWorker = (opts = {}) => createIssuePollingWorker({
 })();
 
 // src/workers/triage-pr.ts
+function shouldCloseLinkedIssues(pr, defaultBranch) {
+  return pr !== null && pr.state === "MERGED" && pr.baseRefName !== defaultBranch;
+}
 var triagePrWorker = createPrPollingWorker({
   name: "triage-pr",
   command: "/claude-task-worker:triage-pr",
   triggerLabel: "cc-triage-scope",
-  excludeLabels: ["cc-fix-onetime", "cc-resolve-conflict", "cc-release-ready", "cc-need-human-check"]
+  excludeLabels: ["cc-fix-onetime", "cc-resolve-conflict", "cc-release-ready", "cc-need-human-check"],
+  onCompleted: async (pr) => {
+    const detail = await getPrDetail(pr.number);
+    const { defaultBranch } = await getRepoInfo();
+    if (!shouldCloseLinkedIssues(detail, defaultBranch)) return;
+    for (const issueNumber of parseClosingIssueNumbers(detail.body)) {
+      try {
+        await closeIssue(issueNumber);
+        console.log(`[triage-pr] closed issue #${issueNumber} (PR #${pr.number} merged into ${detail.baseRefName})`);
+      } catch (err) {
+        console.error(`[triage-pr] failed to close issue #${issueNumber} for PR #${pr.number}: ${err}`);
+      }
+    }
+  }
 });
 
 // src/workers/resolve-conflict.ts
@@ -4503,6 +4540,9 @@ function extractDesignFilePath(body) {
   if (path2.length === 0 || path2.includes("<") || path2.includes(">") || !path2.endsWith(".pen")) return null;
   return path2;
 }
+function shouldVerifyDesignFileExists(cloud) {
+  return !cloud;
+}
 function classifyDesignPr(pr) {
   if (pr === null) return "needs-human";
   if (pr.state === "MERGED" || pr.mergedAt !== null) return "proceed";
@@ -4537,10 +4577,10 @@ function designPrNotCreatedComment(issueNumber) {
     "- \u30C7\u30B6\u30A4\u30F3\u3092\u8AE6\u3081\u3066\u305D\u306E\u307E\u307E\u5B9F\u88C5\u3078\u9032\u3081\u308B\u5834\u5408: `cc-need-human-check` \u30E9\u30D9\u30EB\u3092\u5916\u3057\u3001`cc-exec-issue` \u30E9\u30D9\u30EB\u306E\u307F\u4ED8\u3051\u3066\u304F\u3060\u3055\u3044"
   ].join("\n");
 }
-function designReferenceMissingComment(issueNumber) {
+function designReferenceFailureComment(heading, cause) {
   return [
-    "## \u30C7\u30B6\u30A4\u30F3\u53C2\u7167\u306E\u66F8\u304D\u623B\u3057\u3092\u78BA\u8A8D\u3067\u304D\u307E\u305B\u3093\uFF08\u8981\u4EBA\u624B\u78BA\u8A8D\uFF09",
-    `apply-ui-design \u306E\u30BB\u30C3\u30B7\u30E7\u30F3\u306F\u6B63\u5E38\u7D42\u4E86\uFF08exit 0\uFF09\u3057\u307E\u3057\u305F\u304C\u3001Issue #${issueNumber} \u306E description \u306B \`${DESIGN_REFERENCE_HEADING}\` \u30BB\u30AF\u30B7\u30E7\u30F3\uFF08\`.pen\` \u306E\u30D1\u30B9\u3092\u542B\u3080\uFF09\u304C\u898B\u3064\u304B\u308A\u307E\u305B\u3093\u3067\u3057\u305F\u3002`,
+    heading,
+    cause,
     "",
     "## \u306A\u305C\u6B62\u3081\u3066\u3044\u308B\u304B",
     "- \u30C7\u30B6\u30A4\u30F3\u53C2\u7167\u304C description \u306B\u7121\u3044\u3068\u3001`exec-issue` \u306F\u30C7\u30B6\u30A4\u30F3\u3092\u5165\u529B\u306B\u3067\u304D\u305A\u3001\u5408\u610F\u6E08\u307F\u306E\u30C7\u30B6\u30A4\u30F3\u3068\u7121\u95A2\u4FC2\u306A\u5B9F\u88C5\u306B\u306A\u308A\u307E\u3059",
@@ -4550,6 +4590,18 @@ function designReferenceMissingComment(issueNumber) {
     "- \u624B\u52D5\u3067 description \u306B\u53C2\u7167\u3092\u8FFD\u8A18\u3057\u305F\u5834\u5408: `cc-need-human-check` \u30E9\u30D9\u30EB\u3092\u5916\u3057\u3001`cc-ui-design-ready` \u3068 `cc-exec-issue` \u30E9\u30D9\u30EB\u3092\u4ED8\u3051\u3066\u304F\u3060\u3055\u3044"
   ].join("\n");
 }
+function designReferenceMissingComment(issueNumber) {
+  return designReferenceFailureComment(
+    "## \u30C7\u30B6\u30A4\u30F3\u53C2\u7167\u306E\u66F8\u304D\u623B\u3057\u3092\u78BA\u8A8D\u3067\u304D\u307E\u305B\u3093\uFF08\u8981\u4EBA\u624B\u78BA\u8A8D\uFF09",
+    `apply-ui-design \u306E\u30BB\u30C3\u30B7\u30E7\u30F3\u306F\u6B63\u5E38\u7D42\u4E86\uFF08exit 0\uFF09\u3057\u307E\u3057\u305F\u304C\u3001Issue #${issueNumber} \u306E description \u306B \`${DESIGN_REFERENCE_HEADING}\` \u30BB\u30AF\u30B7\u30E7\u30F3\uFF08\`.pen\` \u306E\u30D1\u30B9\u3092\u542B\u3080\uFF09\u304C\u898B\u3064\u304B\u308A\u307E\u305B\u3093\u3067\u3057\u305F\u3002\u30BB\u30C3\u30B7\u30E7\u30F3\u304C\u66F8\u304D\u623B\u3057\u524D\u306B\u4E2D\u65AD\u3057\u305F\u304B\u3001\u66F8\u304D\u623B\u3057\u306B\u5931\u6557\u3057\u305F\u53EF\u80FD\u6027\u304C\u3042\u308A\u307E\u3059\u3002`
+  );
+}
+function designFileMissingComment(issueNumber, designFilePath) {
+  return designReferenceFailureComment(
+    "## \u30C7\u30B6\u30A4\u30F3\u53C2\u7167\u306E\u30D1\u30B9\u304C\u5B9F\u5728\u3057\u307E\u305B\u3093\uFF08\u8981\u4EBA\u624B\u78BA\u8A8D\uFF09",
+    `apply-ui-design \u306F Issue #${issueNumber} \u306E description \u306B \`${DESIGN_REFERENCE_HEADING}\` \u30BB\u30AF\u30B7\u30E7\u30F3\u3092\u66F8\u304D\u623B\u3057\u307E\u3057\u305F\u304C\u3001\u305D\u3053\u304C\u6307\u3059 \`${designFilePath}\` \u304C\u30D9\u30FC\u30B9\u30D6\u30E9\u30F3\u30C1\u306E\u4F5C\u696D\u30C4\u30EA\u30FC\u306B\u5B58\u5728\u3057\u307E\u305B\u3093\u3067\u3057\u305F\u3002\u30C7\u30B6\u30A4\u30F3PR\u306E\u30DE\u30FC\u30B8\u5148\u3068\u672CIssue\u306E\u30D9\u30FC\u30B9\u30D6\u30E9\u30F3\u30C1\u304C\u98DF\u3044\u9055\u3063\u3066\u3044\u308B\u304B\u3001\u30D1\u30B9\u306E\u7DB4\u308A\u304C\u8AA4\u3063\u3066\u3044\u308B\u53EF\u80FD\u6027\u304C\u3042\u308A\u307E\u3059\u3002`
+  );
+}
 
 // src/workers/create-ui-design.ts
 function designPrLabelingFailedComment(issueNumber, prNumber, error, yolo = false) {
@@ -4636,12 +4688,12 @@ import { join as join6 } from "node:path";
 function designFileExistsInWorktree(worktreeId, designFilePath) {
   return existsSync2(join6(getWorktreePath(worktreeId), designFilePath));
 }
-async function markDesignReferenceMissing(issueNumber, logMessage) {
+async function markDesignReferenceMissing(issueNumber, logMessage, comment) {
   console.error(logMessage);
   await addLabel("issue", issueNumber, "cc-need-human-check").catch(
     (err) => console.error(`[apply-ui-design] addLabel cc-need-human-check failed for #${issueNumber}: ${err}`)
   );
-  await commentOnIssue(issueNumber, designReferenceMissingComment(issueNumber)).catch(
+  await commentOnIssue(issueNumber, comment).catch(
     (err) => console.error(`[apply-ui-design] commentOnIssue failed for #${issueNumber}: ${err}`)
   );
 }
@@ -4683,7 +4735,7 @@ var applyUiDesignWorker = async (opts = {}) => {
       }
       return "skip";
     },
-    onCompleted: async (issueNumber, worktreeId) => {
+    onCompleted: async (issueNumber, worktreeId, _output, ctx) => {
       let body;
       try {
         body = await getIssueBody(issueNumber);
@@ -4695,14 +4747,16 @@ var applyUiDesignWorker = async (opts = {}) => {
       if (designFilePath === null) {
         await markDesignReferenceMissing(
           issueNumber,
-          `[apply-ui-design] #${issueNumber}: session exited without a design reference section; marking cc-need-human-check`
+          `[apply-ui-design] #${issueNumber}: session exited without a design reference section; marking cc-need-human-check`,
+          designReferenceMissingComment(issueNumber)
         );
         return false;
       }
-      if (!designFileExistsInWorktree(worktreeId, designFilePath)) {
+      if (shouldVerifyDesignFileExists(ctx.cloud) && !designFileExistsInWorktree(worktreeId, designFilePath)) {
         await markDesignReferenceMissing(
           issueNumber,
-          `[apply-ui-design] #${issueNumber}: design reference points to ${designFilePath}, which does not exist in worktree ${worktreeId}; marking cc-need-human-check`
+          `[apply-ui-design] #${issueNumber}: design reference points to ${designFilePath}, which does not exist in worktree ${worktreeId}; marking cc-need-human-check`,
+          designFileMissingComment(issueNumber, designFilePath)
         );
         return false;
       }
@@ -5353,6 +5407,7 @@ import { homedir as homedir6 } from "node:os";
 import { join as join8 } from "node:path";
 var LOG_PREFIX = "cloud-setup";
 var CLOUD_DEFAULT_PERMISSION_MODE = "auto";
+var CLOUD_PERMISSION_ALLOW = ["Bash(gh *)"];
 var CLOUD_SETTINGS_DEFAULTS = {
   outputStyle: "Proactive",
   language: "Japanese"
@@ -5388,6 +5443,13 @@ function withCloudDefaults(existing, force) {
     changed = true;
   };
   set(permissions, "defaultMode", CLOUD_DEFAULT_PERMISSION_MODE);
+  const allow = permissions["allow"] ?? [];
+  if (!Array.isArray(allow)) throw new Error("`permissions.allow` is not an array");
+  const missing = CLOUD_PERMISSION_ALLOW.filter((rule) => !allow.includes(rule));
+  if (missing.length > 0) {
+    permissions["allow"] = [...allow, ...missing];
+    changed = true;
+  }
   for (const [key, value] of Object.entries(CLOUD_SETTINGS_DEFAULTS)) set(settings, key, value);
   for (const [key, value] of Object.entries(CLOUD_SETTINGS_ENV)) set(env, key, value);
   if (!changed) return null;

package/package.json

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