ework-daemon 0.4.20 → 0.4.22

package/package.json

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

package/src/config.ts

@@ -64,6 +64,7 @@ export const configSchema = z.object({
     maxLines: z.coerce.number().int().positive().default(2000),
     maxBytes: z.coerce.number().int().positive().default(524288),
   }).default({ roots: [], maxLines: 2000, maxBytes: 524288 }),
+  childEnvDeny: z.array(z.string()).default([]),
 });
 
 export type Config = z.infer<typeof configSchema>;
@@ -145,6 +146,7 @@ export function loadConfig(): Config {
         thresholdMs: Number(process.env.DAEMON_STUCK_THRESHOLD_MS) || 30 * 60 * 1000,
         maxNudges: Number(process.env.DAEMON_MAX_STUCK_NUDGES) || 1,
       } : undefined,
+      childEnvDeny: (process.env.WORK_CHILD_ENV_DENY ?? "").split(",").map((s) => s.trim()).filter(Boolean),
     });
   }
 
@@ -188,5 +190,6 @@ export function loadConfig(): Config {
       maxLines: Number(process.env.WORK_FILE_MAX_LINES) || 2000,
       maxBytes: Number(process.env.WORK_FILE_MAX_BYTES) || 524288,
     },
+    childEnvDeny: (process.env.WORK_CHILD_ENV_DENY ?? "").split(",").map((s) => s.trim()).filter(Boolean),
   });
 }

package/src/opencode.ts

@@ -308,6 +308,7 @@ export class Engine {
   private processExitNudgeRounds = new Map<string, number>();
   private stuckNudgeRounds = new Map<string, number>();
   private currentPrompt = new Map<string, string>();
+  private paused = false;
 
   // Generation counter per session key — incremented on every execProcess call.
   // finishRun captures the generation at start and checks it after each await.
@@ -372,6 +373,22 @@ export class Engine {
     return this.daemonId;
   }
 
+  async pause(): Promise<void> {
+    this.paused = true;
+    await this.store.markDaemonStatus(this.daemonId, "drained");
+    log.info(`engine: daemon ${this.daemonId} paused (drained) — new issues rejected, existing sessions continue`);
+  }
+
+  async resume(): Promise<void> {
+    this.paused = false;
+    await this.store.markDaemonStatus(this.daemonId, "active");
+    log.info(`engine: daemon ${this.daemonId} resumed (active)`);
+  }
+
+  isPaused(): boolean {
+    return this.paused;
+  }
+
   /**
    * Ensure this engine owns the issue before doing work on it. Returns true
    * if we own it (either already, or just claimed). Returns false if another
@@ -539,6 +556,11 @@ export class Engine {
     const tracker = this.getTracker(ref.trackerType);
     const scopeKey = tracker.formatScopeKey(ref.scope);
 
+    if (this.paused && event.type === "issue_opened") {
+      log.info(`engine: paused — skipping issue_opened for ${ref.trackerType}:${scopeKey}#${ref.issueId}`);
+      return;
+    }
+
     const issueMapKey = `${ref.trackerType}:${scopeKey}#${ref.issueId}`;
     if (groupConfig) {
       this.groupConfigs.set(issueMapKey, groupConfig);
@@ -941,14 +963,13 @@ export class Engine {
     // (the operator's explicit config) instead of arbitrary env pollution.
     // Provider keys / Gitea vars are preserved (opencode + its plugin need
     // them); only the explicit model-override var is neutralized.
-    const childEnv = { ...process.env };
+    const childEnv = { ...process.env, ...this.hookEnvFor(issue, session, workdir) };
     if (!model) delete childEnv.OPENCODE_MODEL;
-    // Strip parent-session env vars so the child opencode reads its own
-    // opencode.json instead of inheriting a parent session's model/provider.
     delete childEnv.OPENCODE;
     delete childEnv.OPENCODE_PID;
     delete childEnv.OPENCODE_RUN_ID;
     delete childEnv.OPENCODE_PROCESS_ROLE;
+    for (const k of this.cfg.childEnvDeny) delete childEnv[k];
 
     try {
       const proc = spawn({
@@ -960,6 +981,12 @@ export class Engine {
         stdin: "ignore",
       });
 
+      if (this.generation.get(k) !== gen) {
+        log.warn(`engine: spawned pid=${proc.pid} but generation superseded — killing orphan for ${k}`);
+        try { this.killProcessTree(proc.pid); } catch { /* already dead */ }
+        return;
+      }
+
       this.processes.set(k, proc);
       this.lastOutputAt.set(k, Date.now());
       if (!this.startedAt.has(k)) {

package/src/server.ts

@@ -205,6 +205,15 @@ export function createServer(
       }
     }
 
+    if (pathname === "/api/admin/pause" && req.method === "POST") {
+      await engine.pause();
+      return json({ ok: true, paused: true });
+    }
+    if (pathname === "/api/admin/resume" && req.method === "POST") {
+      await engine.resume();
+      return json({ ok: true, paused: false });
+    }
+
     return json({ error: "not found" }, 404);
   }