@sorenllm/opencode-forge 0.8.0 → 0.9.1

package/README.md

@@ -265,7 +265,7 @@ the stdio-EOF bug):
 | --- | --- | --- | --- |
 | process exit | — | final; exit code + output tail returned | **terminal evidence** — output delivered in full; the job self-clears from the registry (no wake, no clear needed) |
 | `success_pattern` regex matches new output | opt-in | completes as success; process kept alive by default (server semantics), `keep_alive: false` kills its tree | kept alive → later exit still wakes (the server dying is real news); killed inline → self-clears |
-| `idle_ms` with no new output | default 60000 | early return `still-running` + `jobId`; process stays alive | later exit queues a wake; entry stays listed until cleared |
+| `idle_ms` with no new output | default 60000 | early return `still-running` + `jobId`; process stays alive | later exit queues a wake — unless a poll has already read the terminal state (a terminal poll consumes the entry); stays listed until then |
 | `max_wait_ms` hard cap | default 120000, max 600000 | early return `still-running`; never kills | same as idle |
 
 `run_in_background: true` skips all waiting and returns
@@ -292,7 +292,11 @@ idle (exactly once; `notify: false` opts out per job). A foreground call
 resolved on the exit event is terminal evidence: the caller already holds
 the complete output, so no wake fires and the registry entry self-clears —
 a stale `poll`/`kill`/`clear` on such an id answers "already consumed"
-instead of "unknown job".
+instead of "unknown job". The same consumption applies to a poll that
+observes a finished job: the poll itself delivered the exit status and the
+drained output, so the queued wake is dropped and the entry self-clears —
+no duplicate notification, no manual clear. Wakes fire only for
+completions that were never read.
 
 **Ownership.** Jobs belong to the session that created them. Session
 deleted → its live session-scoped jobs are killed and the event is recorded

package/dist/index.js

@@ -13419,6 +13419,9 @@ function createJobManager(opts = {}) {
     if (job.notify && job.wakeState === "none" && !job.terminalEvidence) {
       job.wakeState = "queued";
       job.wakeQueuedAt = now();
+      try {
+        opts.onWakeQueued?.(job);
+      } catch {}
     }
     if (job.terminalEvidence) {
       jobs.delete(job.id);
@@ -13449,8 +13452,11 @@ function createJobManager(opts = {}) {
     markTerminal(job, "killed", null);
   }
   function poll(job) {
-    if (isTerminal2(job))
+    if (isTerminal2(job)) {
       job.readAfterEnd = true;
+      jobs.delete(job.id);
+      rememberConsumed(job.id);
+    }
     const cursor = job.outLen;
     const start = job.pollCursor;
     const newOutput = sliceFromCursor(job, start);
@@ -15600,8 +15606,15 @@ function jobLedgerSink(entry) {
 `);
   } catch {}
 }
-var jobManager = createJobManager({ sink: jobLedgerSink });
+var jobManager = createJobManager({
+  sink: jobLedgerSink,
+  onWakeQueued: (job) => {
+    if (idleJobSessions.has(job.ownerSession))
+      deliverJobWakes(job.ownerSession);
+  }
+});
 var jobClient = null;
+var idleJobSessions = new Set;
 function jobWakeText(job) {
   return [
     `[forge:job-complete] Background job ${job.id} finished (${job.state}${job.exitCode !== null ? `, exit ${job.exitCode}` : ""}).`,
@@ -16242,7 +16255,7 @@ ${r.outputTail || "(none yet)"}`,
 });
 var FORGE_JOBS_ACTIONS = ["list", "poll", "log", "kill", "clear", "handoff"];
 var forgeJobsTool = tool({
-  description: "Manage forge_shell jobs. Actions: list (all jobs, newest first); poll {jobId, waitMs<=30000} — bounded wait for NEW output or exit, drains it; log {jobId, offset?, limit?} — line paging over the on-disk log (omitted offset = tail window, default 200 lines); kill {jobId} — terminate the job's whole process tree; clear {jobId} — drop a finished job from the registry (only early-returned/background jobs need this — synchronously consumed jobs self-clear at completion); handoff {jobId} — rebind ownership to the root session so the job survives this (sub)session's end. A delegated agent MUST poll its jobs before yielding its conclusion.",
+  description: "Manage forge_shell jobs. Actions: list (all jobs, newest first); poll {jobId, waitMs<=30000} — bounded wait for NEW output or exit, drains it (polling a finished job consumes it: the entry self-clears and no wake fires); log {jobId, offset?, limit?} — line paging over the on-disk log (omitted offset = tail window, default 200 lines); kill {jobId} — terminate the job's whole process tree; clear {jobId} — drop a finished job from the registry (only un-polled early-returned/background jobs need this — synchronously consumed jobs self-clear at completion, and a terminal poll consumes the entry); handoff {jobId} — rebind ownership to the root session so the job survives this (sub)session's end. A delegated agent MUST poll its jobs before yielding its conclusion.",
   args: {
     action: tool.schema.string().describe(`One of: ${FORGE_JOBS_ACTIONS.join(", ")}`),
     jobId: tool.schema.string().optional().describe("Job id from forge_shell (required for every action except list)"),
@@ -16943,8 +16956,11 @@ var server = async (input, options) => {
       if (forgeDisabled)
         return;
       const raw = input2;
-      if (typeof raw.sessionID === "string" && raw.sessionID && typeof raw.agent === "string" && raw.agent) {
-        sessionAgents.set(raw.sessionID, raw.agent);
+      if (typeof raw.sessionID === "string" && raw.sessionID) {
+        idleJobSessions.delete(raw.sessionID);
+        if (typeof raw.agent === "string" && raw.agent) {
+          sessionAgents.set(raw.sessionID, raw.agent);
+        }
       }
     },
     "chat.params": async (input2, output) => {
@@ -17030,6 +17046,7 @@ var server = async (input, options) => {
         const sessionID = event.properties.sessionID;
         if (typeof sessionID === "string" && sessionID) {
           goalProbe(`idle event session=${sessionID}`);
+          idleJobSessions.add(sessionID);
           scheduleIdleContinuation(client, sessionID);
           deliverJobWakes(sessionID);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sorenllm/opencode-forge",
-  "version": "0.8.0",
+  "version": "0.9.1",
   "description": "Single general-purpose forge agent for opencode: static worker subagents defined in forge.json (pinned model + thoughtLevel pair, or an Auto worker inheriting the parent model; dispatched through the native task tool — TUI-visible), a plan harness (file-backed plans with approve/close confirmation gates and a hard draft write-ban), a goal harness (autonomous, host-verified objectives), a /crew orchestration command (declared-plan waves with an evidence-checked close gate), a job supervisor (forge_shell with exit-bound completion, idle/success early return, background jobs with poll/log/kill/handoff), and a builtin-shell hang watchdog.",
   "type": "module",
   "main": "./dist/index.js",