@bybrawe/opencode-loop 0.5.18 → 0.5.19

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.5.19
+
+- Made the asynchronous goal prompt smoke test wait for the observable SDK call instead of assuming a single event-loop tick is always sufficient under load.
+
 ## 0.5.18
 
 - Fixed package-based installs loading stale OpenCode cache entries after npm had installed a newer release. The installer now pins an existing package config entry to its exact installed version while preserving JSONC comments.

package/README.md

@@ -16,7 +16,7 @@ v0.5.11 includes a referenced heartbeat scheduler. This is important in OpenCode
 
 ## Current status
 
-**v0.5.18 hardens Goal Mode, package updates, and the background daemon.** Package installs are pinned to the installed version so OpenCode cannot keep loading an older cached release, scheduler-created goal messages no longer self-interrupt on delayed updates, finite daemon failures return nonzero, model/agent selection is supported, and Windows scheduled tasks use a short launcher that stays below the `/TR` limit.
+**v0.5.19 hardens Goal Mode, package updates, and the background daemon.** Package installs are pinned to the installed version so OpenCode cannot keep loading an older cached release, scheduler-created goal messages no longer self-interrupt on delayed updates, finite daemon failures return nonzero, model/agent selection is supported, Windows scheduled tasks use a short launcher that stays below the `/TR` limit, and asynchronous release verification is reliable under load.
 
 The known update-related symptoms from older builds are fixed:
 
@@ -32,7 +32,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.18 quick behavior guide
+## v0.5.19 quick behavior guide
 
 OpenCode Loop has two triggers now:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bybrawe/opencode-loop",
-  "version": "0.5.18",
+  "version": "0.5.19",
   "description": "Claude Code/Codex style /loop and experimental goal mode for OpenCode: heartbeat scheduler, idle-safe loops, scheduled commands, compact scheduling, verification, checkpoints, and persistent coding goals.",
   "type": "module",
   "main": "src/index.js",

package/scripts/smoke-test.mjs

@@ -12,6 +12,15 @@ const prompts = []
 const liveStatuses = new Map([[sessionID, "idle"]])
 let hooks
 
+async function waitFor(predicate, timeoutMs = 2_000) {
+  const deadline = Date.now() + timeoutMs
+  while (!predicate()) {
+    if (Date.now() >= deadline) return false
+    await new Promise((resolve) => setTimeout(resolve, 10))
+  }
+  return true
+}
+
 function legacyBody(name) {
   return async (args) => {
     attempts[name]++
@@ -86,8 +95,11 @@ try {
   assert.equal(output.parts.length, 0, "a locally handled slash command must not start a placeholder model turn")
 
   await hooks["command.execute.before"]({ command: "loop-now", sessionID, arguments: "goal" }, { parts: [] })
-  await new Promise((resolve) => setTimeout(resolve, 0))
-  assert.ok(prompts.some((prompt) => prompt.includes(path.resolve(directory))), "goal prompt must include the working directory")
+  assert.equal(
+    await waitFor(() => prompts.some((prompt) => prompt.includes(path.resolve(directory)))),
+    true,
+    "goal prompt must include the working directory",
+  )
   assert.ok(prompts.some((prompt) => prompt.includes("never turn a relative path into a root path")))
 
   const stateFile = path.join(directory, ".opencode", "opencode-loop", `${sessionID}.json`)
@@ -122,8 +134,11 @@ try {
 
   await hooks["tool.execute.after"]({ tool: "bash", sessionID, callID: "call_background", args: {} }, { title: "done", output: "", metadata: {} })
   await hooks.event({ event: { type: "session.idle", properties: { sessionID } } })
-  await new Promise((resolve) => setTimeout(resolve, 1_400))
-  assert.equal(prompts.length, promptCountBeforeBackgroundTool + 1, "the due loop must resume after the active tool finishes and the session becomes idle")
+  assert.equal(
+    await waitFor(() => prompts.length === promptCountBeforeBackgroundTool + 1, 5_000),
+    true,
+    "the due loop must resume after the active tool finishes and the session becomes idle",
+  )
 
   await hooks["command.execute.before"]({ command: "loop-clear", sessionID, arguments: "" }, { parts: [] })
 
@@ -143,8 +158,11 @@ try {
   liveStatuses.set(childSessionID, "idle")
   await hooks.event({ event: { type: "session.status", properties: { sessionID: childSessionID, status: { type: "idle" } } } })
   await hooks.event({ event: { type: "session.idle", properties: { sessionID } } })
-  await new Promise((resolve) => setTimeout(resolve, 1_400))
-  assert.equal(prompts.length, promptCountBeforeBackgroundChild + 1, "the parent loop must resume after its background child becomes idle")
+  assert.equal(
+    await waitFor(() => prompts.length === promptCountBeforeBackgroundChild + 1, 5_000),
+    true,
+    "the parent loop must resume after its background child becomes idle",
+  )
 
   await hooks["command.execute.before"]({ command: "loop-clear", sessionID, arguments: "" }, { parts: [] })
   assert.equal(attempts.log, 1)