ework-daemon 0.4.57 → 0.4.59

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ework-daemon",
-  "version": "0.4.57",
+  "version": "0.4.59",
   "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

@@ -309,6 +309,13 @@ function createDefaultBackend(cfg: Config): RuntimeBackend {
   return new OpencodeBackend(cfg.opencode.binary, cfg.opencode.dbPath, cfg.childEnvDeny);
 }
 
+function createBackendFor(cfg: Config, runtime: string): RuntimeBackend {
+  if (runtime === "pi" && cfg.pi) {
+    return new PiBackend(cfg.pi.binary, cfg.pi.provider, cfg.pi.defaultModel, cfg.childEnvDeny);
+  }
+  return new OpencodeBackend(cfg.opencode.binary, cfg.opencode.dbPath, cfg.childEnvDeny);
+}
+
 export class Engine {
   private cfg: Config;
   private store: Store;
@@ -352,6 +359,11 @@ export class Engine {
 
   private groupConfigs = new Map<string, GroupConfig>();
   private cloneUrls = new Map<string, string>();
+  // Per-issue runtime override ("opencode"|"pi") from webhook payloads.
+  // Existing sessions stay pinned to their original backend via the
+  // opencodeSessionId prefix (ses_=opencode, bare uuid=pi) in backendFor().
+  private issueRuntimes = new Map<string, string>();
+  private altBackend?: RuntimeBackend;
   private senders = new Map<string, string>();
   private envInitialized = new Set<string>();
 
@@ -359,6 +371,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;
@@ -377,6 +390,25 @@ export class Engine {
     void this.recover();
   }
 
+  // An existing session must keep the backend that owns it: opencode session
+  // ids are "ses_..." while pi ids are bare uuids, so the prefix outvotes the
+  // per-issue override. New sessions follow the issue's runtime setting.
+  private backendFor(k: string, opencodeSessionId?: string): RuntimeBackend {
+    if (opencodeSessionId) {
+      const wants = opencodeSessionId.startsWith("ses_") ? "opencode" : "pi";
+      if (wants !== this.cfg.runtime) return this.altBackendFor(wants);
+      return this.backend;
+    }
+    const runtime = this.issueRuntimes.get(k);
+    if (!runtime || runtime === this.cfg.runtime) return this.backend;
+    return this.altBackendFor(runtime);
+  }
+
+  private altBackendFor(runtime: string): RuntimeBackend {
+    if (!this.altBackend) this.altBackend = createBackendFor(this.cfg, runtime);
+    return this.altBackend;
+  }
+
   private workdirLink(workdir: string): string {
     const p = encodeURIComponent(workdir);
     return `[${workdir}](/file?path=${p}&daemon_id=${this.daemonId})`;
@@ -557,6 +589,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}`);
@@ -738,6 +774,9 @@ export class Engine {
     if (event.cloneUrl) {
       this.cloneUrls.set(issueMapKey, event.cloneUrl);
     }
+    if (event.runtime === "pi" || event.runtime === "opencode") {
+      this.issueRuntimes.set(issueMapKey, event.runtime);
+    }
     if (event.sender) {
       this.senders.set(issueMapKey, event.sender);
     }
@@ -1198,13 +1237,14 @@ export class Engine {
       const fromStrategy = await this.takeover.resumeOpenCodeSession(session);
       if (fromStrategy) resumeSessionId = fromStrategy;
     }
-    if (resumeSessionId && !(await this.backend.sessionExists(resumeSessionId))) {
+    if (resumeSessionId && !(await this.backendFor(k, resumeSessionId).sessionExists(resumeSessionId))) {
       log.warn(`stale session ${resumeSessionId} not found in db, starting fresh`);
       resumeSessionId = undefined;
       await this.store.updateSession(session.id, { opencodeSessionId: undefined });
     }
 
-    const model = msg.model || (this.cfg.runtime === "pi" && this.cfg.pi ? this.cfg.pi.defaultModel : this.cfg.opencode.defaultModel);
+    const backend = this.backendFor(k, resumeSessionId);
+    const model = msg.model || (backend instanceof PiBackend && this.cfg.pi ? this.cfg.pi.defaultModel : this.cfg.opencode.defaultModel);
     this.currentModel.set(k, model);
 
     if (msg.sourceCommentId) {
@@ -1216,7 +1256,7 @@ export class Engine {
     let exitCode: number | null = null;
 
     try {
-      const handle = await this.backend.spawn(
+      const handle = await backend.spawn(
         {
           workdir,
           prompt: msg.content,
@@ -1264,7 +1304,7 @@ export class Engine {
       await this.store.updateSession(session.id, { opencodePid: handle.pid });
       await this.persistRuntimeState(session.id);
 
-      log.info(`engine: spawned pid=${handle.pid} for ${k} (backend=${this.backend.name})`);
+      log.info(`engine: spawned pid=${handle.pid} for ${k} (backend=${backend.name})`);
 
       exitCode = await handle.exited;
       const stderr = await handle.stderrText;
@@ -1377,7 +1417,7 @@ export class Engine {
       });
       log.info(`engine: [bot] reply found for ${k} after prompt (comment ${matched?.id ?? "?"} createdAt ${matched?.createdAt ?? "?"}), marking done`);
       if (matched && !usedModel) {
-        const fromSession = await this.backend.lastSessionModel(session.opencodeSessionId).catch(() => ({ model: "" }));
+        const fromSession = await this.backendFor(k, session.opencodeSessionId).lastSessionModel(session.opencodeSessionId).catch(() => ({ model: "" }));
         if (fromSession.model) {
           void tracker.setCommentModel(ref, matched.id, fromSession.model).catch(() => { /* display-only */ });
         }
@@ -1387,7 +1427,7 @@ export class Engine {
       await this.persistRuntimeState(session.id);
       void tracker.updateStatus(ref, "");
     } else {
-      const sessionOutput = await this.backend.getSessionOutputTokens(session.opencodeSessionId);
+      const sessionOutput = await this.backendFor(k, session.opencodeSessionId).getSessionOutputTokens(session.opencodeSessionId);
       const emptyRound = this.emptyResponseRounds.get(k) ?? 0;
 
       if (!sessionOutput.hasOutput && emptyRound < Engine.MAX_EMPTY_RESPONSE_ROUNDS) {
@@ -1749,6 +1789,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);

package/src/trackers/gitea-tracker.ts

@@ -138,6 +138,8 @@ export class GiteaTracker implements IssueTracker {
     // Empty/missing = no model override; engine omits --model.
     const modelRaw = repository.ework_model;
     const model = typeof modelRaw === "string" && modelRaw.trim() ? modelRaw.trim() : undefined;
+    const runtimeRaw = repository.ework_runtime;
+    const runtime = runtimeRaw === "pi" || runtimeRaw === "opencode" ? runtimeRaw : undefined;
 
     const ref: TrackerRef = {
       trackerType: "gitea",
@@ -162,6 +164,7 @@ export class GiteaTracker implements IssueTracker {
           ai_status: aiStatus,
         },
         model,
+        runtime,
         cloneUrl,
         sender,
       };
@@ -186,6 +189,7 @@ export class GiteaTracker implements IssueTracker {
           authorKind: (comment as Record<string, unknown>).author_kind as string | undefined,
         },
         model,
+        runtime,
         cloneUrl,
         sender,
       };
@@ -203,6 +207,7 @@ export class GiteaTracker implements IssueTracker {
           ai_status: aiStatus,
         },
         model,
+        runtime,
         cloneUrl,
         sender,
       };

package/src/trackers/types.ts

@@ -36,6 +36,8 @@ export interface TrackerEvent {
   // global default). Empty/undefined = no override; engine omits --model
   // and lets opencode pick per its own opencode.json + env.
   model?: string;
+  // Runtime backend override for this issue from ework-web ("opencode"|"pi").
+  runtime?: string;
   // Real clone URL from the upstream tracker (e.g. Gitea repository.clone_url).
   // When present, RecloneStrategy uses this instead of the ework shim URL.
   cloneUrl?: string;