🧹 Nitpick comments (1)

packages/serverless-hono/src/utils/wait-until-wrapper.ts (1)
`35-46`: **LGTM — correctly guards global state and swallows invocation errors.** Binding before wrapping avoids `Illegal invocation`, and the `try/catch` inside the wrapper (rather than around `.bind`) correctly catches per-call failures like `DataCloneError`. Removing the `else` branch preserves any outer-scope global, matching the fix for `#1203`. One optional consideration: fully silent swallowing can hide real bugs during development. The sibling `flushOnFinish` in `volt-agent-observability.ts` logs a `console.warn` when `waitUntil` throws; you may want a similar debug-level log here for symmetry, though it's not required for correctness.

Optional: log swallowed errors for observability parity
```diff globals.___voltagent_wait_until = (promise: Promise) => { try { boundWaitUntil(promise); - } catch { - // Swallow errors to avoid breaking the caller + } catch (error) { + // Swallow errors (e.g. DataCloneError) to avoid breaking the caller + // eslint-disable-next-line no-console + console.warn("[voltagent] waitUntil invocation failed", error); } }; ```

🤖 Prompt for AI Agents
``` Verify each finding against the current code and only fix it if needed. In `@packages/serverless-hono/src/utils/wait-until-wrapper.ts` around lines 35 - 46, The wrapper for globals.___voltagent_wait_until currently swallows errors silently; update the catch block in wait-until-wrapper.ts (inside the if checking currentWaitUntil and after binding via currentWaitUntil.bind(context)) to emit a debug/console.warn log including the caught error and a short context message (e.g., that waitUntil invocation failed) before swallowing, referencing globals.___voltagent_wait_until, boundWaitUntil and context so logs mirror the observability behavior used by flushOnFinish in volt-agent-observability.ts. ```

🤖 Prompt for all review comments with AI agents

``` Verify each finding against the current code and only fix it if needed. Nitpick comments: In `@packages/serverless-hono/src/utils/wait-until-wrapper.ts`: - Around line 35-46: The wrapper for globals.___voltagent_wait_until currently swallows errors silently; update the catch block in wait-until-wrapper.ts (inside the if checking currentWaitUntil and after binding via currentWaitUntil.bind(context)) to emit a debug/console.warn log including the caught error and a short context message (e.g., that waitUntil invocation failed) before swallowing, referencing globals.___voltagent_wait_until, boundWaitUntil and context so logs mirror the observability behavior used by flushOnFinish in volt-agent-observability.ts. ```

---

ℹ️ Review info

⚙️ Run configuration

**Configuration used**: defaults **Review profile**: CHILL **Plan**: Pro **Run ID**: `150ec58a-50a5-4ffe-a24a-3a0f9901c2e0`

📥 Commits

Reviewing files that changed from the base of the PR and between 71c9f84a9f58b591c453900c7c0a4eda53a08196 and 3491ff5836f4dcdd4ac37dc7e0d50c487d2422e3.

📒 Files selected for processing (1)

* `packages/serverless-hono/src/utils/wait-until-wrapper.ts`