ework-daemon 0.4.56 → 0.4.58

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ework-daemon",
-  "version": "0.4.56",
+  "version": "0.4.58",
   "description": "Issue-driven AI development daemon. Spawns opencode subprocesses to resolve Gitea issues.",
   "module": "src/index.ts",
   "type": "module",

package/src/config.ts

@@ -72,6 +72,7 @@ export const configSchema = z.object({
   stuck: z.object({
     thresholdMs: z.coerce.number().positive(),
     maxNudges: z.coerce.number().int().nonnegative(),
+    maxRuntimeMs: z.coerce.number().positive(),
   }).optional(),
   file: z.object({
     roots: z.array(z.string()).default([]),
@@ -170,9 +171,10 @@ export function loadConfig(): Config {
         baseURL: process.env.COMPLETION_CHECK_BASE_URL ?? "",
         model: process.env.COMPLETION_CHECK_MODEL ?? "",
       } : undefined,
-      stuck: process.env.DAEMON_STUCK_THRESHOLD_MS || process.env.DAEMON_MAX_STUCK_NUDGES ? {
+      stuck: process.env.DAEMON_STUCK_THRESHOLD_MS || process.env.DAEMON_MAX_STUCK_NUDGES || process.env.DAEMON_STUCK_MAX_RUNTIME_MS ? {
         thresholdMs: Number(process.env.DAEMON_STUCK_THRESHOLD_MS) || 30 * 60 * 1000,
         maxNudges: Number(process.env.DAEMON_MAX_STUCK_NUDGES) || 1,
+        maxRuntimeMs: Number(process.env.DAEMON_STUCK_MAX_RUNTIME_MS) || 3 * 60 * 60 * 1000,
       } : undefined,
       childEnvDeny: (process.env.WORK_CHILD_ENV_DENY ?? "").split(",").map((s) => s.trim()).filter(Boolean),
     });
@@ -217,9 +219,10 @@ export function loadConfig(): Config {
       baseURL: process.env.COMPLETION_CHECK_BASE_URL ?? "",
       model: process.env.COMPLETION_CHECK_MODEL ?? "",
     } : undefined,
-    stuck: process.env.DAEMON_STUCK_THRESHOLD_MS || process.env.DAEMON_MAX_STUCK_NUDGES ? {
+    stuck: process.env.DAEMON_STUCK_THRESHOLD_MS || process.env.DAEMON_MAX_STUCK_NUDGES || process.env.DAEMON_STUCK_MAX_RUNTIME_MS ? {
       thresholdMs: Number(process.env.DAEMON_STUCK_THRESHOLD_MS) || 30 * 60 * 1000,
       maxNudges: Number(process.env.DAEMON_MAX_STUCK_NUDGES) || 1,
+      maxRuntimeMs: Number(process.env.DAEMON_STUCK_MAX_RUNTIME_MS) || 3 * 60 * 60 * 1000,
     } : undefined,
     file: {
       roots: (process.env.WORK_FILE_ROOTS ?? "").split(":").filter(Boolean).length > 0

package/src/opencode.ts

@@ -332,6 +332,7 @@ export class Engine {
   private startedAt = new Map<string, number>();
   private progressCommentId = new Map<string, string>();
   private pickupCommentId = new Map<string, string>();
+  private forwardCommentId = new Map<string, string>();
 
   private nudgeRounds = new Map<string, number>();
   private emptyResponseRounds = new Map<string, number>();
@@ -358,6 +359,7 @@ export class Engine {
   private static MAX_NUDGE_ROUNDS = 1;
   private static MAX_EMPTY_RESPONSE_ROUNDS = 1;
   private static MAX_STUCK_NUDGE_ROUNDS = 1;
+  private static MAX_RUNTIME_MS = 3 * 60 * 60 * 1000;
   private static OBSERVER_INTERVAL_MS = 5 * 60 * 1000;
   private static STUCK_THRESHOLD_MS = 30 * 60 * 1000;
   private static MAX_PROCESS_EXIT_NUDGE_ROUNDS = 1;
@@ -556,6 +558,10 @@ export class Engine {
     return this.cfg.stuck?.maxNudges ?? Engine.MAX_STUCK_NUDGE_ROUNDS;
   }
 
+  private get maxRuntimeMs(): number {
+    return this.cfg.stuck?.maxRuntimeMs ?? Engine.MAX_RUNTIME_MS;
+  }
+
   private getTracker(type: string): IssueTracker {
     const tracker = this.trackers.get(type);
     if (!tracker) throw new Error(`Unknown tracker type: ${type}`);
@@ -894,7 +900,10 @@ export class Engine {
         );
 
         // Immediate ack
-        await tracker.createComment(ref, `[system] ✓ Message forwarded to **${session.name}**${this.running.has(this.sessionKey(session, issue)) ? " (running)" : ""}.\n> session: ${this.sessionRef(session)} | workdir: ${this.workdirLink(workdir)}`);
+        {
+          const c = await tracker.createComment(ref, `[system] ✓ Message forwarded to **${session.name}**${this.running.has(this.sessionKey(session, issue)) ? " (running)" : ""}.\n> session: ${this.sessionRef(session)} | workdir: ${this.workdirLink(workdir)}`);
+          if (!session.opencodeSessionId) this.forwardCommentId.set(this.sessionKey(session, issue), c.id);
+        }
 
         await this.enqueueOrRun(session, issue, prompt, comment.id, model);
       } else {
@@ -1230,10 +1239,17 @@ export class Engine {
               const pickupId = this.pickupCommentId.get(k);
               if (pickupId) {
                 this.pickupCommentId.delete(k);
+    this.forwardCommentId.delete(k);
                 await tracker
                   .editComment(ref, pickupId, `[system] 🔄 **${session.name}** picked up this issue.\n> session: ${this.sessionRef(session)} | workdir: ${this.workdirLink(workdir)}`)
                   .catch((err) => log.error(`engine: failed to rewrite pickup comment for ${k}:`, (err as Error).message));
               }
+              const forwardId = this.forwardCommentId.get(k);
+              if (forwardId) {
+                this.forwardCommentId.delete(k);
+                const body = `[system] ✓ Message forwarded to **${session.name}**${this.running.has(k) ? " (running)" : ""}.\n> session: ${this.sessionRef(session)} | workdir: ${this.workdirLink(workdir)}`;
+                await tracker.editComment(ref, forwardId, body).catch(() => { /* cosmetic rewrite */ });
+              }
             }
           },
         },
@@ -1738,6 +1754,19 @@ export class Engine {
           continue;
         }
 
+        // Process alive — cap total run time. Output-silence detection cannot
+        // catch loops that keep emitting (observed: 6h of failing compress calls
+        // every ~5s), so any single run is hard-stopped after maxRuntimeMs.
+        const started = this.startedAt.get(k);
+        if (started && Date.now() - started >= this.maxRuntimeMs) {
+          const hrs = (this.maxRuntimeMs / 3600000).toFixed(1);
+          log.warn(`engine: run exceeded max runtime (${hrs}h) on ${k}, stopping`);
+          await tracker.createComment(this.sessionToRef(session, issue), `[system] ⏹ **${session.name}** run exceeded ${hrs}h — stopped. Reply again on the issue to continue.`).catch(() => { /* best-effort */ });
+          this.nudgeRounds.set(k, Engine.MAX_NUDGE_ROUNDS);
+          this.forceStop(k);
+          continue;
+        }
+
         // Process alive — check stuck
         if (lastTs && Date.now() - lastTs >= this.stuckThresholdMs) {
           const minutes = Math.round((Date.now() - lastTs) / 60000);