@bybrawe/opencode-loop 0.5.8 → 0.5.9

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.5.9
+
+- Fixed stale-busy recovery for TUI sessions. Cached `busy` / `retry` status is now short-lived, so due jobs no longer get stuck at `due in every idle` after `/loop-status` or other short custom-command turns.
+- Improved due-timer behavior when OpenCode misses or delays the final idle event after a command turn.
+- Fixed `/compact` routing for current OpenCode TUI builds by trying `session_compact` first, then `session.compact`, then the session summarize fallback.
+- Kept the command markdown templates short; OpenCode still creates a tiny `OK` turn for custom commands because markdown commands are prompt templates.
+
 ## 0.5.8
 
 - Added a real due timer for TUI loop jobs, so delayed jobs can wake without waiting for another OpenCode event.

package/README.md

@@ -11,7 +11,7 @@ NPM package name: **@bybrawe/opencode-loop**
 
 ## Current status
 
-**v0.5.8 is a reliability hotfix for the new loop types and Experimental Goal Mode.** It adds a real due timer in addition to OpenCode idle/status events, so delayed jobs such as `/loop 1m --no-now ...`, `/loop-command 1m --no-now /compact`, `/loop-ask 1h ...`, and `/loop-shell 10m ...` do not depend on another user action to wake the scheduler. The TUI loop still waits for the session to become idle before starting work, debounces idle events, and avoids starting a new run while OpenCode is busy or queued.
+**v0.5.9 is a reliability hotfix for stale-busy status and `/compact` routing.** It keeps the real due timer from v0.5.8, but no longer trusts a cached `busy` / `retry` session status forever. This fixes the case where `/loop-status` or another short custom-command turn could leave a loop showing `due in every idle` without firing. It also routes scheduled `/compact` through OpenCode's current TUI alias first, with fallback paths for older builds.
 
 The known update-related symptoms from older builds are fixed:
 
@@ -27,7 +27,7 @@ The known update-related symptoms from older builds are fixed:
 
 The TUI loop is still intentionally session-bound: it runs while OpenCode is open and the current session emits status/idle events. For long-running background work after closing the terminal or OpenCode, use `opencode-loopd`.
 
-## v0.5.8 quick behavior guide
+## v0.5.9 quick behavior guide
 
 OpenCode Loop has two triggers now:
 
@@ -74,7 +74,7 @@ Run a shell command every 10 minutes when idle.
 
 Experimental persistent goal mode: keep working until the goal is complete, blocked, paused, cleared, or a safety limit is reached.
 
-> Note: OpenCode custom command markdown still creates a tiny assistant turn for slash commands. v0.5.8 keeps these command templates short and asks the model to reply `OK`. The actual loop scheduling is handled locally by the plugin.
+> Note: OpenCode custom command markdown still creates a tiny assistant turn for slash commands. v0.5.9 keeps these command templates short and asks the model to reply `OK`. The actual loop scheduling is handled locally by the plugin. If that short turn briefly makes OpenCode look busy, the loop now recovers by refreshing stale busy status on the next due retry.
 
 ## Why this exists
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@bybrawe/opencode-loop",
-  "version": "0.5.8",
-  "description": "Claude Code/Codex style /loop and experimental goal mode for OpenCode: reliable timers, idle-safe loops, scheduled commands, compact scheduling, verification, checkpoints, and persistent coding goals.",
+  "version": "0.5.9",
+  "description": "Claude Code/Codex style /loop and experimental goal mode for OpenCode: reliable timers, stale-busy recovery, idle-safe loops, scheduled commands, compact scheduling, verification, checkpoints, and persistent coding goals.",
   "type": "module",
   "main": "src/index.js",
   "exports": {

package/src/index.js

@@ -7,6 +7,7 @@ const STATE_DIR = ".opencode/opencode-loop"
 const DEFAULT_ACTIVE_GUARD_MS = 60_000
 const IDLE_DEBOUNCE_MS = 1_200
 const BUSY_RETRY_MS = 5_000
+const SESSION_STATUS_CACHE_MS = 1_500
 const MIN_DUE_TIMER_MS = 250
 const MAX_DUE_TIMER_MS = 2_147_000_000
 const MAX_SCAN_FILES = 200
@@ -19,6 +20,7 @@ const handledCommands = new Map()
 const idleTimers = new Map()
 const dueTimers = new Map()
 const sessionStatuses = new Map()
+const sessionStatusSeenAt = new Map()
 
 const DEFAULT_PROGRESS_MD = `# Progress
 
@@ -389,16 +391,22 @@ async function executeTuiCommand(client, command) {
 
 function compactTuiCommandName(command = "compact") {
   const normalized = String(command || "compact").replace(/^\/+/, "").trim().toLowerCase()
-  if (normalized === "compact" || normalized === "summarize") return "session.compact"
+  if (normalized === "compact" || normalized === "summarize") return "session_compact"
   return undefined
 }
 
 async function compactSession(client, sessionID) {
-  try {
-    await executeTuiCommand(client, "session.compact")
-    return true
-  } catch (error) {
-    await log(client, "warn", "tui session.compact failed", { error: sdkErrorMessage(error) })
+  // OpenCode's TUI API accepts legacy keybind aliases (session_compact) in
+  // current builds, while some older docs/examples mention the event value
+  // (session.compact). Try the alias first, then the event value, then the
+  // session summarize endpoint as a last resort.
+  for (const command of ["session_compact", "session.compact"]) {
+    try {
+      await executeTuiCommand(client, command)
+      return true
+    } catch (error) {
+      await log(client, "warn", `tui ${command} failed`, { error: sdkErrorMessage(error) })
+    }
   }
   try {
     await sdkCall(client.session.summarize.bind(client.session), { sessionID }, { path: { id: sessionID }, body: {} })
@@ -734,12 +742,16 @@ function updateSessionStatusFromEvent(event) {
   if (typeof sessionID !== "string") return undefined
   if (event?.type === "session.idle") {
     sessionStatuses.set(sessionID, "idle")
+    sessionStatusSeenAt.set(sessionID, now())
     return { sessionID, idle: true }
   }
   if (event?.type === "session.status") {
     const status = event?.properties?.status
     const type = status && typeof status === "object" ? status.type : undefined
-    if (typeof type === "string") sessionStatuses.set(sessionID, type)
+    if (typeof type === "string") {
+      sessionStatuses.set(sessionID, type)
+      sessionStatusSeenAt.set(sessionID, now())
+    }
     return { sessionID, idle: type === "idle" }
   }
   return undefined
@@ -747,18 +759,30 @@ function updateSessionStatusFromEvent(event) {
 
 async function sessionStatusType(client, sessionID) {
   const cached = sessionStatuses.get(sessionID)
-  if (cached && cached !== "idle") return cached
+  const seenAt = sessionStatusSeenAt.get(sessionID) || 0
+
+  // Idle is safe to trust until OpenCode tells us otherwise. Busy/retry is only
+  // trusted briefly: OpenCode custom commands such as /loop-status create their
+  // own short assistant turn, and some TUI builds do not always emit the final
+  // idle event after that turn. If we cache busy forever, due loop work can get
+  // stuck at "due in every idle" until the user types another command.
+  if (cached === "idle") return cached
+  if (cached && now() - seenAt < SESSION_STATUS_CACHE_MS) return cached
+
   try {
     const data = await sdkCall(client.session.status.bind(client.session), {}, undefined)
     const status = data?.[sessionID]
     const type = status && typeof status === "object" ? status.type : undefined
     if (typeof type === "string") {
       sessionStatuses.set(sessionID, type)
+      sessionStatusSeenAt.set(sessionID, now())
       return type
     }
   } catch {}
+
   const fallback = activeRuns.has(sessionID) ? "busy" : "idle"
   sessionStatuses.set(sessionID, fallback)
+  sessionStatusSeenAt.set(sessionID, now())
   return fallback
 }
 
@@ -1077,7 +1101,7 @@ async function fireAction(directory, client, sessionID, job) {
     }
     const tuiCommand = compactTuiCommandName(command)
     if (tuiCommand) {
-      await executeTuiCommand(client, tuiCommand)
+      await compactSession(client, sessionID)
       return { startsAssistantTurn: true }
     }
     await sdkCall(client.session.command.bind(client.session), { sessionID, command, arguments: argumentsText }, { path: { id: sessionID }, body: { command, arguments: argumentsText } })