fix(js-sdk): stop waiting for stream close after command end event

On receiving the "end" event in iterateEvents(), call
handleDisconnect() to abort the transport controller, then let the
abort error propagate through the for-await loop. This allows
connectrpc's runStreamingCall to reach its finally block and clear
the deadline timer, avoiding resource leaks in Deno.

In handleEvents(), ignore iteration errors when a result has already
been captured since the error is the expected abort from disconnect.

Author: beran-t

packages/js-sdk/src/sandbox/commands/commandHandle.ts

@@ -216,6 +216,7 @@ export class CommandHandle
             stdout: this.stdout,
             stderr: this.stderr,
           }
+          this.handleDisconnect()
           break
       }
       // TODO: Handle empty events like in python SDK
@@ -234,7 +235,9 @@ export class CommandHandle
         }
       }
     } catch (e) {
-      this.iterationError = handleRpcError(e)
+      if (!this.result) {
+        this.iterationError = handleRpcError(e)
+      }
     }
   }
 }