fix(bash): drain output fiber before scope closes to prevent data loss

Fast-exiting commands (e.g. echo test) could produce (no output)
because Effect.forkScoped interrupted the stream consumer fiber
before it had a chance to process chunks buffered in the Node.js
readable pipe.

Fix by joining the fiber after the exit/abort/timeout race so all
buffered stdout/stderr is flushed into the accumulator list before
Effect.scoped disposes the scope.

Author: lgarceau768

packages/opencode/src/tool/bash.ts

@@ -17,7 +17,7 @@ import { Shell } from "@/shell/shell"
 import { BashArity } from "@/permission/arity"
 import * as Truncate from "./truncate"
 import { Plugin } from "@/plugin"
-import { Effect, Stream } from "effect"
+import { Effect, Fiber, Stream } from "effect"
 import { ChildProcess } from "effect/unstable/process"
 import { ChildProcessSpawner } from "effect/unstable/process/ChildProcessSpawner"
 
@@ -444,7 +444,7 @@ export const BashTool = Tool.define(
         Effect.gen(function* () {
           const handle = yield* spawner.spawn(cmd(input.shell, input.name, input.command, input.cwd, input.env))
 
-          yield* Effect.forkScoped(
+          const outputFiber = yield* Effect.forkScoped(
             Stream.runForEach(Stream.decodeText(handle.all), (chunk) => {
               const size = Buffer.byteLength(chunk, "utf-8")
               list.push({ text: chunk, size })
@@ -517,6 +517,11 @@ export const BashTool = Tool.define(
             yield* handle.kill({ forceKillAfter: "3 seconds" }).pipe(Effect.orDie)
           }
 
+          // Drain any remaining buffered output before the scope closes.
+          // Without this, the stream consumer fiber can be interrupted before
+          // it processes chunks that were buffered when the process exited.
+          yield* Fiber.join(outputFiber).pipe(Effect.ignore)
+
           return exit.kind === "exit" ? exit.code : null
         }),
       ).pipe(Effect.orDie)