fix(cli): don't block opencode tool results on the trace toast

Show the trace toast without awaiting it, so a slow or missing TUI never
holds back the tool result, and only append the model relay when the
shell tool returned string output, since an aborted tool may carry none.

Assisted-by: Claude Code
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>

Chainloop-Trace-Sessions: a3931907-c590-40a1-90c6-2c29aa204404

Author: migmartri

.opencode/plugins/chainloop-trace.ts

@@ -76,7 +76,7 @@ export const ChainloopTrace: Plugin = async ({ $, client }) => {
       if (event.type === "session.created") {
         const sessionID = event.properties?.info?.id ?? ""
         const res = await fire("session-start", { session_id: sessionID, hook_event_name: "session.created" })
-        if (res.message) await toast(res.message)
+        if (res.message) void toast(res.message)
       }
       if (event.type === "session.deleted") {
         const sessionID = event.properties?.info?.id ?? ""
@@ -112,9 +112,13 @@ export const ChainloopTrace: Plugin = async ({ $, client }) => {
         // A shell command may have been a git push, whose pre-push hook
         // attested the session and left a link to it. Show it on both
         // channels: the toast reaches the user now, the tool output reaches
-        // the model, whose reply outlives the toast.
-        if (res.message) await toast(res.message)
-        if (res.relayToModel) output.output = output.output + "\n\n" + res.relayToModel
+        // the model, whose reply outlives the toast. The toast is not awaited
+        // so a slow TUI never holds back the tool result, and an aborted tool
+        // may carry no output to append to.
+        if (res.message) void toast(res.message)
+        if (res.relayToModel && typeof output?.output === "string") {
+          output.output = output.output + "\n\n" + res.relayToModel
+        }
         return
       }
       if (!fileWritingTools.includes(input.tool)) return

app/cli/internal/trace/opencode/hooks.go

@@ -135,7 +135,7 @@ export const ChainloopTrace: Plugin = async ({ $, client }) => {
       if (event.type === "session.created") {
         const sessionID = event.properties?.info?.id ?? ""
         const res = await fire("session-start", { session_id: sessionID, hook_event_name: "session.created" })
-        if (res.message) await toast(res.message)
+        if (res.message) void toast(res.message)
       }
 {{SessionEndBlock}}
     },
@@ -168,9 +168,13 @@ export const ChainloopTrace: Plugin = async ({ $, client }) => {
         // A shell command may have been a git push, whose pre-push hook
         // attested the session and left a link to it. Show it on both
         // channels: the toast reaches the user now, the tool output reaches
-        // the model, whose reply outlives the toast.
-        if (res.message) await toast(res.message)
-        if (res.relayToModel) output.output = output.output + "\n\n" + res.relayToModel
+        // the model, whose reply outlives the toast. The toast is not awaited
+        // so a slow TUI never holds back the tool result, and an aborted tool
+        // may carry no output to append to.
+        if (res.message) void toast(res.message)
+        if (res.relayToModel && typeof output?.output === "string") {
+          output.output = output.output + "\n\n" + res.relayToModel
+        }
         return
       }
       if (!fileWritingTools.includes(input.tool)) return

app/cli/internal/trace/opencode/testdata/plugin_full.ts

@@ -76,7 +76,7 @@ export const ChainloopTrace: Plugin = async ({ $, client }) => {
       if (event.type === "session.created") {
         const sessionID = event.properties?.info?.id ?? ""
         const res = await fire("session-start", { session_id: sessionID, hook_event_name: "session.created" })
-        if (res.message) await toast(res.message)
+        if (res.message) void toast(res.message)
       }
       if (event.type === "session.deleted") {
         const sessionID = event.properties?.info?.id ?? ""
@@ -112,9 +112,13 @@ export const ChainloopTrace: Plugin = async ({ $, client }) => {
         // A shell command may have been a git push, whose pre-push hook
         // attested the session and left a link to it. Show it on both
         // channels: the toast reaches the user now, the tool output reaches
-        // the model, whose reply outlives the toast.
-        if (res.message) await toast(res.message)
-        if (res.relayToModel) output.output = output.output + "\n\n" + res.relayToModel
+        // the model, whose reply outlives the toast. The toast is not awaited
+        // so a slow TUI never holds back the tool result, and an aborted tool
+        // may carry no output to append to.
+        if (res.message) void toast(res.message)
+        if (res.relayToModel && typeof output?.output === "string") {
+          output.output = output.output + "\n\n" + res.relayToModel
+        }
         return
       }
       if (!fileWritingTools.includes(input.tool)) return

app/cli/internal/trace/opencode/testdata/plugin_tracerun.ts

@@ -76,7 +76,7 @@ export const ChainloopTrace: Plugin = async ({ $, client }) => {
       if (event.type === "session.created") {
         const sessionID = event.properties?.info?.id ?? ""
         const res = await fire("session-start", { session_id: sessionID, hook_event_name: "session.created" })
-        if (res.message) await toast(res.message)
+        if (res.message) void toast(res.message)
       }
     },
     "tool.execute.before": async (input, output) => {
@@ -108,9 +108,13 @@ export const ChainloopTrace: Plugin = async ({ $, client }) => {
         // A shell command may have been a git push, whose pre-push hook
         // attested the session and left a link to it. Show it on both
         // channels: the toast reaches the user now, the tool output reaches
-        // the model, whose reply outlives the toast.
-        if (res.message) await toast(res.message)
-        if (res.relayToModel) output.output = output.output + "\n\n" + res.relayToModel
+        // the model, whose reply outlives the toast. The toast is not awaited
+        // so a slow TUI never holds back the tool result, and an aborted tool
+        // may carry no output to append to.
+        if (res.message) void toast(res.message)
+        if (res.relayToModel && typeof output?.output === "string") {
+          output.output = output.output + "\n\n" + res.relayToModel
+        }
         return
       }
       if (!fileWritingTools.includes(input.tool)) return