@mestreyoda/fabrica 0.2.0 → 0.2.1

package/dist/index.js

@@ -111324,8 +111324,8 @@ import fsSync from "node:fs";
 import path5 from "node:path";
 import { fileURLToPath as fileURLToPath3 } from "node:url";
 function getCurrentVersion() {
-  if ("0.2.0") {
-    return "0.2.0";
+  if ("0.2.1") {
+    return "0.2.1";
   }
   try {
     const pkgPath = path5.join(THIS_DIR, "..", "..", "package.json");
@@ -112302,7 +112302,7 @@ function resolve2(config2, sourceLayers, trace2) {
     dispatchMs: config2.timeouts?.dispatchMs ?? 6e5,
     staleWorkerHours: config2.timeouts?.staleWorkerHours ?? 2,
     sessionContextBudget: config2.timeouts?.sessionContextBudget ?? 0.6,
-    stallTimeoutMinutes: config2.timeouts?.stallTimeoutMinutes ?? 15,
+    stallTimeoutMinutes: config2.timeouts?.stallTimeoutMinutes ?? 5,
     sessionConfirmAttempts: config2.timeouts?.sessionConfirmAttempts ?? 5,
     sessionConfirmDelayMs: config2.timeouts?.sessionConfirmDelayMs ?? 250,
     sessionLabelMaxLength: config2.timeouts?.sessionLabelMaxLength ?? 64,
@@ -124579,6 +124579,7 @@ import { jsonResult as jsonResult15 } from "openclaw/plugin-sdk";
 init_audit();
 
 // lib/services/heartbeat/diagnostic.ts
+import { execFileSync } from "child_process";
 async function diagnoseStall(input) {
   const { owner, repo, issueId, dispatchAttemptCount } = input;
   if ((dispatchAttemptCount ?? 0) >= 2) {
@@ -124591,9 +124592,9 @@ async function diagnoseStall(input) {
   if (pr) {
     const qaStatus = await checkPrQaStatus(owner, repo, pr.number);
     if (qaStatus === "pass") {
-      return { action: "transition_to_review", reason: "stall", evidence: `PR #${pr.number} exists, QA passing` };
+      return { action: "transition_to_review", reason: "stall", evidence: `PR #${pr.number} exists, QA passing`, prNumber: pr.number };
     }
-    return { action: "redispatch_same_level", reason: "stall", evidence: `PR #${pr.number} exists, QA ${qaStatus}` };
+    return { action: "redispatch_same_level", reason: "stall", evidence: `PR #${pr.number} exists, QA ${qaStatus}`, prNumber: pr.number };
   }
   const hasCommits = await checkForBranchCommits(owner, repo, issueId);
   if (hasCommits) {
@@ -124606,10 +124607,13 @@ async function diagnoseStall(input) {
   }
   return { action: "escalate_level", reason: "complexity", evidence: "Active session without commits" };
 }
+function ghSync(args) {
+  return execFileSync("gh", args, { encoding: "utf-8", timeout: 15e3 });
+}
 async function findPrForIssue(owner, repo, issueId) {
+  if (!owner || !repo) return null;
   try {
-    const { execa } = await import("execa");
-    const { stdout } = await execa("gh", [
+    const stdout = ghSync([
       "pr",
       "list",
       "--repo",
@@ -124628,29 +124632,31 @@ async function findPrForIssue(owner, repo, issueId) {
   }
 }
 async function checkPrQaStatus(owner, repo, prNumber) {
+  if (!owner || !repo) return "pending";
   try {
-    const { execa } = await import("execa");
-    const { stdout } = await execa("gh", [
+    const stdout = ghSync([
       "pr",
-      "checks",
+      "view",
       String(prNumber),
       "--repo",
       `${owner}/${repo}`,
       "--json",
-      "state"
+      "statusCheckRollup"
     ]);
-    const checks = JSON.parse(stdout);
-    if (checks.every((c) => c.state === "SUCCESS")) return "pass";
-    if (checks.some((c) => c.state === "FAILURE")) return "fail";
+    const data = JSON.parse(stdout);
+    const checks = data.statusCheckRollup ?? [];
+    if (checks.length === 0) return "pass";
+    if (checks.every((c) => c.conclusion === "SUCCESS")) return "pass";
+    if (checks.some((c) => c.conclusion === "FAILURE")) return "fail";
     return "pending";
   } catch {
-    return "pending";
+    return "pass";
   }
 }
 async function checkForBranchCommits(owner, repo, issueId) {
+  if (!owner || !repo) return false;
   try {
-    const { execa } = await import("execa");
-    const { stdout } = await execa("gh", [
+    const stdout = ghSync([
       "api",
       `repos/${owner}/${repo}/commits`,
       "--jq",
@@ -124678,6 +124684,7 @@ var GRACE_PERIOD_MS = 5 * 60 * 1e3;
 var DISPATCH_CONFIRMATION_TIMEOUT_MS = 2 * 60 * 1e3;
 var NUDGE_MESSAGE = `You appear to have stalled. Continue working on your current task. If you are blocked or unable to proceed, call work_finish with result "blocked".`;
 var MAX_STALL_NUDGES = 3;
+var MAX_DISPATCH_ATTEMPTS = 5;
 async function auditHealthFixApplied(workspaceDir, fix, details) {
   if (!fix.fixed) return;
   await withCorrelationContext({
@@ -125045,30 +125052,84 @@ async function checkWorkerHealth(opts) {
       }
       if (slot.active && sessionKey && sessions && !withinGracePeriod && isSessionAlive(sessionKey, sessions)) {
         const session = sessions.get(sessionKey);
-        const stallThresholdMs = (opts.stallTimeoutMinutes ?? 15) * 6e4;
+        const stallThresholdMs = (opts.stallTimeoutMinutes ?? 5) * 6e4;
         if (session.updatedAt == null) continue;
         const sessionIdleMs = Date.now() - session.updatedAt;
         if (sessionIdleMs > stallThresholdMs) {
           const idleMinutes = Math.round(sessionIdleMs / 6e4);
+          const repoRemote = project.repoRemote ?? "";
+          const remoteMatch = repoRemote.match(/github\.com\/([^/]+)\/([^/.]+)/);
+          const diagnostic = await diagnoseStall({
+            projectSlug,
+            owner: remoteMatch?.[1] ?? "",
+            repo: remoteMatch?.[2] ?? "",
+            issueId: issueIdNum ?? 0,
+            sessionKey,
+            slotStartTime: slot.startTime ? new Date(slot.startTime).getTime() : Date.now() - sessionIdleMs,
+            sessionUpdatedAt: session.updatedAt,
+            dispatchAttemptCount: issueRuntime?.dispatchAttemptCount ?? 0
+          });
+          const evidenceActions = ["transition_to_review", "nudge_open_pr"];
+          const hasActionableEvidence = evidenceActions.includes(diagnostic.action);
           const modelUnresponsiveMs = stallThresholdMs * MAX_STALL_NUDGES;
           const slotAgeMs = slot.startTime ? Date.now() - new Date(slot.startTime).getTime() : sessionIdleMs;
-          if (slotAgeMs > modelUnresponsiveMs) {
-            const diagnostic = await diagnoseStall({
+          const isModelUnresponsive = slotAgeMs > modelUnresponsiveMs;
+          const currentAttempts = issueRuntime?.dispatchAttemptCount ?? 0;
+          if (currentAttempts >= MAX_DISPATCH_ATTEMPTS && !hasActionableEvidence) {
+            const fix2 = {
+              issue: {
+                type: "diagnostic_needs_human_review",
+                severity: "critical",
+                project: project.name,
+                projectSlug,
+                role,
+                level,
+                sessionKey,
+                issueId: slot.issueId,
+                slotIndex,
+                message: `${role.toUpperCase()} ${level}[${slotIndex}] circuit breaker: ${currentAttempts} dispatch attempts without completion \u2014 moving to Refining`
+              },
+              fixed: false
+            };
+            if (autoFix) {
+              await revertLabel(fix2, expectedLabel, "Refining");
+              await deactivateSlot();
+              fix2.fixed = true;
+            }
+            await log(workspaceDir, "dispatch_circuit_breaker", {
+              project: project.name,
               projectSlug,
-              owner: project.remote?.owner ?? "",
-              repo: project.remote?.repo ?? "",
-              issueId: issueIdNum ?? 0,
+              role,
+              level,
               sessionKey,
-              slotStartTime: slot.startTime ? new Date(slot.startTime).getTime() : Date.now() - sessionIdleMs,
-              sessionUpdatedAt: session.updatedAt,
-              dispatchAttemptCount: issueRuntime?.dispatchAttemptCount ?? 0
+              issueId: slot.issueId,
+              slotIndex,
+              dispatchAttemptCount: currentAttempts,
+              diagnostic: diagnostic.action,
+              evidence: diagnostic.evidence
+            }).catch(() => {
             });
+            fixes.push(fix2);
+            continue;
+          }
+          if (hasActionableEvidence || isModelUnresponsive) {
             if (issueIdNum) {
-              await updateIssueRuntime(workspaceDir, projectSlug, String(issueIdNum), {
+              const runtimeUpdate = {
                 lastDiagnosticResult: diagnostic.evidence,
                 lastFailureReason: diagnostic.reason,
                 dispatchAttemptCount: (issueRuntime?.dispatchAttemptCount ?? 0) + 1
-              }).catch(() => {
+              };
+              if (diagnostic.prNumber) {
+                const prOwner = remoteMatch?.[1] ?? "";
+                const prRepo = remoteMatch?.[2] ?? "";
+                runtimeUpdate.currentPrNumber = diagnostic.prNumber;
+                runtimeUpdate.currentPrUrl = `https://github.com/${prOwner}/${prRepo}/pull/${diagnostic.prNumber}`;
+                runtimeUpdate.currentPrState = "open";
+                runtimeUpdate.bindingSource = "diagnostic";
+                runtimeUpdate.bindingConfidence = "high";
+                runtimeUpdate.boundAt = (/* @__PURE__ */ new Date()).toISOString();
+              }
+              await updateIssueRuntime(workspaceDir, projectSlug, String(issueIdNum), runtimeUpdate).catch(() => {
               });
             }
             const diagnosticType = `diagnostic_${diagnostic.action}`;
@@ -125089,13 +125150,19 @@ async function checkWorkerHealth(opts) {
             };
             if (autoFix) {
               switch (diagnostic.action) {
-                case "transition_to_review":
-                  await revertLabel(fix2, expectedLabel, "To Review");
+                case "transition_to_review": {
+                  const targetLabel = role === "tester" ? "Done" : role === "reviewer" ? "To Test" : "To Review";
+                  await revertLabel(fix2, expectedLabel, targetLabel);
                   if (!fix2.labelRevertFailed) {
                     await deactivateSlot();
+                    if (role === "tester" && issueIdNum) {
+                      await provider.closeIssue(issueIdNum).catch(() => {
+                      });
+                    }
                     fix2.fixed = true;
                   }
                   break;
+                }
                 case "redispatch_same_level":
                 case "nudge_open_pr":
                 case "retry_infra":
@@ -125113,10 +125180,12 @@ async function checkWorkerHealth(opts) {
                   }
                   break;
                 case "needs_human_review":
+                  await revertLabel(fix2, expectedLabel, "Refining");
                   await deactivateSlot();
                   fix2.fixed = true;
                   break;
                 case "log_infra":
+                  await revertLabel(fix2, expectedLabel, "Refining");
                   await deactivateSlot();
                   fix2.fixed = true;
                   break;
@@ -125134,6 +125203,7 @@ async function checkWorkerHealth(opts) {
               diagnostic: diagnostic.action,
               reason: diagnostic.reason,
               evidence: diagnostic.evidence,
+              fastPath: hasActionableEvidence,
               dispatchAttemptCount: (issueRuntime?.dispatchAttemptCount ?? 0) + 1
             }).catch(() => {
             });
@@ -125151,7 +125221,7 @@ async function checkWorkerHealth(opts) {
               sessionKey,
               issueId: slot.issueId,
               slotIndex,
-              message: `${role.toUpperCase()} ${level}[${slotIndex}] session idle ${idleMinutes}m \u2014 sending nudge`
+              message: `${role.toUpperCase()} ${level}[${slotIndex}] session idle ${idleMinutes}m \u2014 no deliverables yet, sending nudge`
             },
             fixed: false
           };
@@ -138727,16 +138797,32 @@ async function classifyDmIntent(ctx, content, _workspaceDir) {
       idempotencyKey: sessionKey
     });
     const waitResult = await runtime.subagent.waitForRun({ runId, timeoutMs: 15e3 });
-    if (waitResult.status !== "ok") return null;
-    const messages = await runtime.subagent.getSessionMessages({ sessionKey });
-    const lastAssistant = messages?.filter((m2) => m2.role === "assistant").pop();
-    const text = lastAssistant?.content ?? "";
-    if (!text.trim()) return null;
+    if (waitResult.status !== "ok") {
+      ctx.logger.warn(`[telegram-bootstrap] classify waitForRun status=${waitResult.status} (expected ok)`);
+      return null;
+    }
+    const messagesResult = await runtime.subagent.getSessionMessages({ sessionKey });
+    const messages = Array.isArray(messagesResult) ? messagesResult : Array.isArray(messagesResult?.messages) ? messagesResult.messages : [];
+    const lastAssistant = messages.filter((m2) => m2.role === "assistant").pop();
+    if (!lastAssistant) {
+      ctx.logger.warn(`[telegram-bootstrap] classify: no assistant message found (messages=${messages.length}, resultType=${typeof messagesResult})`);
+      return null;
+    }
+    const rawContent = lastAssistant.content;
+    const text = typeof rawContent === "string" ? rawContent : Array.isArray(rawContent) ? rawContent.find((b) => b.type === "text")?.text ?? "" : "";
+    if (!text.trim()) {
+      ctx.logger.warn(`[telegram-bootstrap] classify: empty text from assistant (contentType=${typeof rawContent}, isArray=${Array.isArray(rawContent)})`);
+      return null;
+    }
     const jsonStr = text.replace(/^```(json)?/gm, "").replace(/```$/gm, "").trim();
     const intentData = JSON.parse(jsonStr);
     const validated = DmIntentSchema.safeParse(intentData);
+    if (!validated.success) {
+      ctx.logger.warn(`[telegram-bootstrap] classify: schema validation failed: ${validated.error?.message}`);
+    }
     return validated.success ? validated.data : null;
-  } catch {
+  } catch (err) {
+    ctx.logger.warn(`[telegram-bootstrap] classify exception: ${err.message ?? err}`);
     return null;
   }
 }