starpc 0.50.0 → 0.52.0

package/integration/cross-language/run.bash

@@ -111,27 +111,6 @@ stop_server() {
 
 # run_pair <test_name> <server_args...> -- <client_args...>
 # The client receives $SERVER_ADDR as its last argument.
-# check_receipt_events verifies terminal markers after the receipt server
-# naturally completes, so receipt synchronization is event/process-backed.
-check_receipt_events() {
-    local expected="$1"
-    local event_log="$2"
-    awk -v expected="$expected" '
-        $0 == "SERVER_RECEIPT_TERMINAL " expected { terminal = NR; next }
-        $0 == "SERVER_RECEIPT_ACK_WRITE " expected { ack_write = NR; next }
-        $0 == "CLIENT_RECEIPT_RESOLVED " expected { client = NR; next }
-        END {
-            if (!terminal || !client) {
-                exit 1
-            }
-            if (expected == "committed" &&
-                (!ack_write || terminal >= ack_write || ack_write >= client)) {
-                exit 1
-            }
-        }
-    ' "$event_log"
-}
-
 run_pair() {
     local test_name="$1"
     shift
@@ -140,7 +119,6 @@ run_pair() {
         return
     fi
 
-    # Split args on "--".
     local srv_args=()
     local cli_args=()
     local in_client=false
@@ -156,103 +134,33 @@ run_pair() {
         fi
     done
 
-    local expected_terminal=""
-    case "$test_name" in
-        *"receipt commit") expected_terminal="committed" ;;
-        *"receipt abort") expected_terminal="canceled" ;;
-        *"receipt loss") expected_terminal="transportLost" ;;
-        *"receipt bare-close") expected_terminal="closed" ;;
-    esac
-
     echo -n "  ${test_name}... "
-
-    local event_fifo=""
-    local event_log=""
-    local event_reader=""
-    if [ -n "$expected_terminal" ]; then
-        event_fifo=$(mktemp)
-        rm -f "$event_fifo"
-        mkfifo "$event_fifo"
-        event_log=$(mktemp)
-        exec 3<>"$event_fifo"
-        (
-            exec 3>&-
-            cat "$event_fifo" > "$event_log"
-        ) &
-        event_reader=$!
-        export RECEIPT_EVENT_FIFO="$event_fifo"
-    else
-        unset RECEIPT_EVENT_FIFO
-    fi
-
     if ! start_server "${srv_args[@]}"; then
         echo "FAILED (server start)"
         FAILED=$((FAILED + 1))
         ERRORS="${ERRORS}\n  ${test_name} (server start failed)"
-        if [ -n "$event_reader" ]; then
-            exec 3>&-
-            kill "$event_reader" 2>/dev/null || true
-            wait "$event_reader" 2>/dev/null || true
-            rm -f "$event_fifo" "$event_log"
-        fi
-        unset RECEIPT_EVENT_FIFO
         return
     fi
 
     local client_out
     client_out=$(mktemp)
     local client_ok=false
-    local server_ok=true
     if "${cli_args[@]}" "$SERVER_ADDR" > "$client_out" 2>&1; then
         client_ok=true
     fi
+    stop_server
 
-    if [ -n "$event_reader" ]; then
-        if $client_ok; then
-            if ! wait "$SERVER_PID"; then
-                server_ok=false
-            fi
-        else
-            stop_server
-            wait "$SERVER_PID" 2>/dev/null || true
-            server_ok=false
-        fi
-        exec 3>&-
-        wait "$event_reader" 2>/dev/null || true
-    else
-        stop_server
-    fi
-    unset RECEIPT_EVENT_FIFO
-
-    local terminal_ok=true
-    if [ -n "$expected_terminal" ] &&
-        ! check_receipt_events "$expected_terminal" "$event_log"; then
-        terminal_ok=false
-    fi
-
-    if $client_ok && $server_ok && $terminal_ok; then
+    if $client_ok; then
         echo "PASSED"
-        if [ -n "$expected_terminal" ]; then
-            echo "    receipt events:"
-            sed 's/^/      /' "$event_log"
-        fi
         PASSED=$((PASSED + 1))
     else
         echo "FAILED"
         FAILED=$((FAILED + 1))
         ERRORS="${ERRORS}\n  ${test_name}"
-        if ! $client_ok; then
-            echo "    client output:"
-            sed 's/^/    /' "$client_out"
-        fi
-        if ! $terminal_ok; then
-            echo "    receipt events:"
-            sed 's/^/    /' "$event_log"
-            echo "    server output:"
-            sed 's/^/    /' "$SERVER_LOG"
-        fi
+        echo "    client output:"
+        sed 's/^/    /' "$client_out"
     fi
-    rm -f "$client_out" "$SERVER_LOG" "$event_fifo" "$event_log"
+    rm -f "$client_out" "$SERVER_LOG"
     SERVER_LOG=""
 }
 
@@ -264,10 +172,6 @@ echo ""
 run_pair "go-server + go-client"   "$GO_SERVER" -- "$GO_CLIENT"
 run_pair "go-server + rust-client" "$GO_SERVER" -- "$RUST_CLIENT"
 run_pair "go-server + ts-client"   "$GO_SERVER" -- node "$TS_CLIENT"
-run_pair "go-server + ts-client receipt commit" "$GO_SERVER" receipt commit -- node "$TS_CLIENT" receipt commit
-run_pair "go-server + ts-client receipt abort" "$GO_SERVER" receipt abort -- node "$TS_CLIENT" receipt abort
-run_pair "go-server + ts-client receipt loss" "$GO_SERVER" receipt loss -- node "$TS_CLIENT" receipt loss
-run_pair "go-server + ts-client receipt bare-close" "$GO_SERVER" receipt bare-close -- node "$TS_CLIENT" receipt bare-close
 run_pair "go-server + cpp-client"  "$GO_SERVER" -- "$CPP_CLIENT"
 
 # Rust server combinations
@@ -278,10 +182,6 @@ run_pair "rust-server + cpp-client"  "$RUST_SERVER" -- "$CPP_CLIENT"
 
 # TypeScript server combinations
 run_pair "ts-server + go-client"   node "$TS_SERVER" -- "$GO_CLIENT"
-run_pair "ts-server + go-client receipt commit" node "$TS_SERVER" receipt commit -- "$GO_CLIENT" receipt commit
-run_pair "ts-server + go-client receipt abort" node "$TS_SERVER" receipt abort -- "$GO_CLIENT" receipt abort
-run_pair "ts-server + go-client receipt loss" node "$TS_SERVER" receipt loss -- "$GO_CLIENT" receipt loss
-run_pair "ts-server + go-client receipt bare-close" node "$TS_SERVER" receipt bare-close -- "$GO_CLIENT" receipt bare-close
 run_pair "ts-server + rust-client" node "$TS_SERVER" -- "$RUST_CLIENT"
 run_pair "ts-server + ts-client"   node "$TS_SERVER" -- node "$TS_CLIENT"
 run_pair "ts-server + cpp-client"  node "$TS_SERVER" -- "$CPP_CLIENT"

package/integration/cross-language/ts-client.ts

@@ -1,5 +1,4 @@
 import net from 'net'
-import { closeSync, openSync, writeSync } from 'node:fs'
 import { pipe } from 'it-pipe'
 import { pushable } from 'it-pushable'
 import { Client } from '../../srpc/client.js'
@@ -8,22 +7,9 @@ import {
   prependLengthPrefixTransform,
 } from '../../srpc/packet.js'
 import { combineUint8ArrayListTransform } from '../../srpc/array-list.js'
-import { EchoMsg, runClientTest } from '../../echo/index.js'
+import { runClientTest } from '../../echo/index.js'
 import type { OpenStreamFunc, PacketStream } from '../../srpc/stream.js'
 import type { Source } from 'it-stream-types'
-function emitReceiptEvent(line: string): void {
-  console.log(line)
-  const fifo = process.env.RECEIPT_EVENT_FIFO
-  if (!fifo) {
-    return
-  }
-  const fd = openSync(fifo, 'w')
-  try {
-    writeSync(fd, `${line}\n`)
-  } finally {
-    closeSync(fd)
-  }
-}
 
 // tcpSocketToPacketStream wraps a Node.js TCP socket into a PacketStream.
 function tcpSocketToPacketStream(socket: net.Socket): PacketStream {
@@ -60,28 +46,17 @@ function tcpSocketToPacketStream(socket: net.Socket): PacketStream {
 }
 
 async function main() {
-  const receiptMode = process.argv[2] === 'receipt'
-  const receiptCase = receiptMode ? process.argv[3] : undefined
-  const addr = receiptMode ? process.argv[4] : process.argv[2]
+  const addr = process.argv[2]
   if (!addr) {
-    console.error('usage: ts-client [receipt <case>] <host:port>')
+    console.error('usage: ts-client <host:port>')
     process.exit(1)
   }
-  if (
-    receiptMode &&
-    !['commit', 'abort', 'loss', 'bare-close'].includes(receiptCase ?? '')
-  ) {
-    throw new Error(`unknown receipt case: ${receiptCase}`)
-  }
 
   const [host, portStr] = addr.split(':')
   const port = parseInt(portStr, 10)
-  let activeSocket: net.Socket | undefined
-
   const openStream: OpenStreamFunc = async (): Promise<PacketStream> => {
     const { promise, resolve, reject } = Promise.withResolvers<PacketStream>()
     const socket = net.connect(port, host, () => {
-      activeSocket = socket
       resolve(tcpSocketToPacketStream(socket))
     })
     socket.on('error', reject)
@@ -89,71 +64,11 @@ async function main() {
   }
 
   const client = new Client(openStream)
-
-  if (receiptMode) {
-    console.log(`Running held receipt test via TCP (${receiptCase})...`)
-    const request = EchoMsg.create({ body: 'held receipt' })
-    const held = await client.requestWithReceipt(
-      'echo.Echoer',
-      'Echo',
-      EchoMsg.toBinary(request),
-    )
-    const result = EchoMsg.fromBinary(held.response)
-    if (result.body !== request.body) {
-      throw new Error(`expected ${request.body}, got ${result.body}`)
-    }
-    switch (receiptCase) {
-      case 'commit':
-        await held.receipt.commit()
-        break
-      case 'abort':
-        await held.receipt.abort()
-        break
-      case 'loss':
-        activeSocket?.resetAndDestroy()
-        await expectReceiptFailure(held.receipt)
-        break
-      case 'bare-close':
-        activeSocket?.end()
-        await expectReceiptFailure(held.receipt)
-        break
-    }
-    emitReceiptEvent(
-      `CLIENT_RECEIPT_RESOLVED ${receiptTerminalName(receiptCase ?? '')}`,
-    )
-  } else {
-    console.log('Running client test via TCP...')
-    await runClientTest(client)
-  }
+  console.log('Running client test via TCP...')
+  await runClientTest(client)
   console.log('All tests passed.')
 }
 
-function receiptTerminalName(receiptCase: string): string {
-  switch (receiptCase) {
-    case 'commit':
-      return 'committed'
-    case 'abort':
-      return 'canceled'
-    case 'loss':
-      return 'transportLost'
-    case 'bare-close':
-      return 'closed'
-    default:
-      return 'unknown'
-  }
-}
-
-async function expectReceiptFailure(receipt: {
-  commit(): Promise<void>
-}): Promise<void> {
-  try {
-    await receipt.commit()
-  } catch {
-    return
-  }
-  throw new Error('receipt commit unexpectedly succeeded')
-}
-
 process.on('unhandledRejection', (ev) => {
   console.error('Unhandled rejection', ev)
   process.exit(1)

package/integration/cross-language/ts-server.ts

@@ -1,5 +1,4 @@
 import net from 'net'
-import { closeSync, openSync, writeSync } from 'node:fs'
 import { pipe } from 'it-pipe'
 import { pushable } from 'it-pushable'
 import type { Source } from 'it-stream-types'
@@ -10,25 +9,10 @@ import {
   prependLengthPrefixTransform,
 } from '../../srpc/packet.js'
 import { combineUint8ArrayListTransform } from '../../srpc/array-list.js'
-import { EchoerServer, EchoMsg } from '../../echo/index.js'
+import { EchoerServer } from '../../echo/index.js'
 import { EchoerDefinition } from '../../echo/echo_srpc.pb.js'
-import { Packet, TerminalKind } from '../../srpc/rpcproto.pb.js'
 import type { PacketStream } from '../../srpc/stream.js'
 
-function emitReceiptEvent(line: string): void {
-  console.log(line)
-  const fifo = process.env.RECEIPT_EVENT_FIFO
-  if (!fifo) {
-    return
-  }
-  const fd = openSync(fifo, 'w')
-  try {
-    writeSync(fd, `${line}\n`)
-  } finally {
-    closeSync(fd)
-  }
-}
-
 // tcpSocketToPacketStream wraps a Node.js TCP socket into a PacketStream.
 // Each Uint8Array in source/sink is one packet (no length prefix).
 function tcpSocketToPacketStream(socket: net.Socket): PacketStream {
@@ -53,25 +37,12 @@ function tcpSocketToPacketStream(socket: net.Socket): PacketStream {
     sink: async (source: Source<Uint8Array>): Promise<void> => {
       for await (const chunk of pipe(source, prependLengthPrefixTransform())) {
         const data = chunk instanceof Uint8Array ? chunk : chunk.subarray()
-        const packet = Packet.fromBinary(data.subarray(4))
-        const receiptCompletion =
-          receiptMode &&
-          packet.body?.case === 'callData' &&
-          packet.body.value.complete &&
-          !packet.body.value.error
-        if (receiptCompletion) {
-          emitReceiptEvent('SERVER_RECEIPT_ACK_WRITE committed')
-        }
-        const writeDone = new Promise<void>((resolve, reject) => {
+        await new Promise<void>((resolve, reject) => {
           socket.write(data, (err) => {
             if (err) reject(err)
             else resolve()
           })
         })
-        await writeDone
-        if (receiptCompletion) {
-          finishReceiptServer()
-        }
       }
       socket.end()
     },
@@ -79,82 +50,9 @@ function tcpSocketToPacketStream(socket: net.Socket): PacketStream {
 }
 
 const mux = createMux()
-const receiptMode = process.argv[2] === 'receipt'
-const receiptCase = receiptMode ? process.argv[3] : undefined
-if (
-  receiptMode &&
-  !['commit', 'abort', 'loss', 'bare-close'].includes(receiptCase ?? '')
-) {
-  console.error(`unknown receipt case: ${receiptCase}`)
-  process.exit(1)
-}
-
-const receiptServerDone = Promise.withResolvers<void>()
-let receiptServerFinished = false
-function finishReceiptServer(): void {
-  if (!receiptServerFinished) {
-    receiptServerFinished = true
-    receiptServerDone.resolve()
-  }
-}
-
-if (receiptMode) {
-  mux.registerLookupMethod(async (serviceID, methodID) => {
-    if (serviceID !== 'echo.Echoer' || methodID !== 'Echo') {
-      return null
-    }
-    return async (dataSource, dataSink, invocation) => {
-      let requestData: Uint8Array | undefined
-      for await (const data of dataSource) {
-        requestData = data
-        break
-      }
-      if (!requestData) {
-        throw new Error('receipt request was empty')
-      }
-      const requestMsg = EchoMsg.fromBinary(requestData)
-      await dataSink(
-        (async function* () {
-          yield EchoMsg.toBinary(requestMsg)
-          if (!invocation) {
-            throw new Error('receipt invocation was missing')
-          }
-          emitReceiptEvent('SERVER_RECEIPT_WAITING')
-          const terminal = await invocation.waitTerminal(
-            new AbortController().signal,
-          )
-          emitReceiptEvent(`SERVER_RECEIPT_TERMINAL ${terminalName(terminal)}`)
-          if (terminal !== TerminalKind.COMMITTED) {
-            finishReceiptServer()
-          }
-        })(),
-      )
-    }
-  })
-}
-
-function terminalName(terminal: TerminalKind): string {
-  switch (terminal) {
-    case TerminalKind.COMMITTED:
-      return 'committed'
-    case TerminalKind.CANCELED:
-      return 'canceled'
-    case TerminalKind.TRANSPORT_LOST:
-      return 'transportLost'
-    case TerminalKind.CLOSED:
-      return 'closed'
-    case TerminalKind.ABANDONED:
-      return 'abandoned'
-    default:
-      return 'unknown'
-  }
-}
-
 const server = new Server(mux.lookupMethod)
-if (!receiptMode) {
-  const echoer = new EchoerServer(server)
-  mux.register(createHandler(EchoerDefinition, echoer))
-}
+const echoer = new EchoerServer(server)
+mux.register(createHandler(EchoerDefinition, echoer))
 
 const tcpServer = net.createServer((socket) => {
   const stream = tcpSocketToPacketStream(socket)
@@ -166,12 +64,6 @@ tcpServer.listen(0, '127.0.0.1', () => {
   console.log(`LISTENING ${addr.address}:${addr.port}`)
 })
 
-if (receiptMode) {
-  void receiptServerDone.promise.then(() => {
-    tcpServer.close()
-  })
-}
-
 process.on('SIGINT', () => {
   tcpServer.close()
   process.exit(0)

package/mock/mock.go

@@ -26,4 +26,4 @@ func (e *MockServer) MockRequest(ctx context.Context, msg *MockMsg) (*MockMsg, e
 }
 
 // _ is a type assertion
-var _ SRPCMockServer = ((*MockServer)(nil))
+var _ SRPCMockServer = (*MockServer)(nil)

package/mock/mock.pb.go

@@ -1,5 +1,5 @@
 // Code generated by protoc-gen-go-lite. DO NOT EDIT.
-// protoc-gen-go-lite version: v0.13.0
+// protoc-gen-go-lite version: v0.16.0
 // source: github.com/aperturerobotics/starpc/mock/mock.proto
 
 package e2e_mock
@@ -8,8 +8,6 @@ import (
 	fmt "fmt"
 	io "io"
 	slices "slices"
-	strconv "strconv"
-	strings "strings"
 
 	protobuf_go_lite "github.com/aperturerobotics/protobuf-go-lite"
 	json "github.com/aperturerobotics/protobuf-go-lite/json"
@@ -139,13 +137,10 @@ func (m *MockMsg) MarshalToSizedBufferVT(dAtA []byte) (int, error) {
 	var l int
 	_ = l
 	if m.unknownFields != nil {
-		i -= len(m.unknownFields)
-		copy(dAtA[i:], m.unknownFields)
+		i = protobuf_go_lite.EncodeRawBytes(dAtA, i, m.unknownFields)
 	}
 	if len(m.Body) > 0 {
-		i -= len(m.Body)
-		copy(dAtA[i:], m.Body)
-		i = protobuf_go_lite.EncodeVarint(dAtA, i, uint64(len(m.Body)))
+		i = protobuf_go_lite.EncodeString(dAtA, i, m.Body)
 		i--
 		dAtA[i] = 0xa
 	}
@@ -158,26 +153,19 @@ func (m *MockMsg) SizeVT() (n int) {
 	}
 	var l int
 	_ = l
-	l = len(m.Body)
-	if l > 0 {
-		n += 1 + l + protobuf_go_lite.SizeOfVarint(uint64(l))
-	}
+	n += protobuf_go_lite.SizeStringNonEmpty(1, m.Body)
 	n += len(m.unknownFields)
 	return n
 }
 
 func (x *MockMsg) MarshalProtoText() string {
-	var sb strings.Builder
-	sb.WriteString("MockMsg {")
+	var sb protobuf_go_lite.TextBuilder
+	initialLen := protobuf_go_lite.TextStartMessage(&sb, "MockMsg")
 	if x.Body != "" {
-		if sb.Len() > 9 {
-			sb.WriteString(" ")
-		}
-		sb.WriteString("body: ")
-		sb.WriteString(strconv.Quote(x.Body))
+		protobuf_go_lite.TextWriteFieldPrefix(&sb, initialLen, "body")
+		protobuf_go_lite.TextWriteString(&sb, x.Body)
 	}
-	sb.WriteString("}")
-	return sb.String()
+	return protobuf_go_lite.TextFinishMessage(&sb)
 }
 
 func (x *MockMsg) String() string {
@@ -208,24 +196,12 @@ func (m *MockMsg) UnmarshalVT(dAtA []byte) error {
 			if wireType != 2 {
 				return fmt.Errorf("proto: wrong wireType = %d for field Body", wireType)
 			}
-			var stringLen uint64
-			stringLen, iNdEx, err = protobuf_go_lite.DecodeVarint(dAtA, iNdEx)
+			var v string
+			v, iNdEx, err = protobuf_go_lite.DecodeString(dAtA, iNdEx)
 			if err != nil {
 				return err
 			}
-			intStringLen := int(stringLen)
-			if intStringLen < 0 {
-				return protobuf_go_lite.ErrInvalidLength
-			}
-			postIndex := iNdEx + intStringLen
-			if postIndex < 0 {
-				return protobuf_go_lite.ErrInvalidLength
-			}
-			if postIndex > l {
-				return io.ErrUnexpectedEOF
-			}
-			m.Body = string(dAtA[iNdEx:postIndex])
-			iNdEx = postIndex
+			m.Body = v
 		default:
 			iNdEx = preIndex
 			skippy, err := protobuf_go_lite.Skip(dAtA[iNdEx:])

package/mock/mock_srpc.pb.cpp

@@ -1,7 +1,7 @@
 //go:build deps_only
 
 // Code generated by protoc-gen-starpc-cpp. DO NOT EDIT.
-// protoc-gen-starpc-cpp version: v0.49.7
+// protoc-gen-starpc-cpp version: v0.49.21
 // source: github.com/aperturerobotics/starpc/mock/mock.proto
 
 #include "mock_srpc.pb.hpp"

package/mock/mock_srpc.pb.go

@@ -1,5 +1,5 @@
 // Code generated by protoc-gen-srpc. DO NOT EDIT.
-// protoc-gen-srpc version: v0.49.7
+// protoc-gen-srpc version: v0.49.21
 // source: github.com/aperturerobotics/starpc/mock/mock.proto
 
 package e2e_mock

package/mock/mock_srpc.pb.hpp

@@ -1,7 +1,7 @@
 //go:build deps_only && cgo
 
 // Code generated by protoc-gen-starpc-cpp. DO NOT EDIT.
-// protoc-gen-starpc-cpp version: v0.49.7
+// protoc-gen-starpc-cpp version: v0.49.21
 // source: github.com/aperturerobotics/starpc/mock/mock.proto
 
 #pragma once

package/mock/mock_srpc.pb.rs

@@ -1,5 +1,5 @@
 // Code generated by protoc-gen-starpc-rust. DO NOT EDIT.
-// protoc-gen-starpc-rust version: v0.49.7
+// protoc-gen-starpc-rust version: v0.49.21
 // source: github.com/aperturerobotics/starpc/mock/mock.proto
 
 #[allow(unused_imports)]

package/mock/mock_srpc.pb.ts

@@ -4,7 +4,7 @@
 
 import { MockMsg } from './mock.pb.js'
 import { MethodKind } from '@aptre/protobuf-es-lite'
-import { ProtoRpc } from 'starpc'
+import { ProtoRpc, ServerContext } from 'starpc'
 
 /**
  * Mock service mocks some RPCs for the e2e tests.
@@ -42,6 +42,24 @@ export interface Mock {
   MockRequest(request: MockMsg, abortSignal?: AbortSignal): Promise<MockMsg>
 }
 
+/**
+ * Mock service mocks some RPCs for the e2e tests.
+ *
+ * @generated from service e2e.mock.Mock
+ */
+export interface MockHandler {
+  /**
+   * MockRequest runs a mock unary request.
+   *
+   * @generated from rpc e2e.mock.Mock.MockRequest
+   */
+  MockRequest(
+    request: MockMsg,
+    abortSignal: AbortSignal,
+    context: ServerContext,
+  ): Promise<MockMsg>
+}
+
 export const MockServiceName = MockDefinition.typeName
 
 export class MockClient implements Mock {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "starpc",
-  "version": "0.50.0",
+  "version": "0.52.0",
   "description": "Streaming protobuf RPC service protocol over any two-way channel.",
   "license": "MIT",
   "author": {
@@ -114,7 +114,7 @@
     "happy-dom": "^20.9.0",
     "husky": "^9.1.7",
     "lint-staged": "^17.0.0",
-    "oxfmt": "0.61.0",
+    "oxfmt": "0.62.0",
     "oxlint": "^1.76.0",
     "rimraf": "^6.1.3",
     "tsx": "^4.20.4",

package/srpc/channel.test.ts

@@ -1,9 +1,29 @@
-import { describe, expect, it } from 'vitest'
+import { describe, expect, it, vi } from 'vitest'
 import { pushable } from 'it-pushable'
 
 import { ChannelStream } from './channel.js'
 
 describe('ChannelStream', () => {
+  it('recognizes MessagePort implementations from another realm', () => {
+    const port = {
+      close: vi.fn(),
+      onmessage: null,
+      postMessage: vi.fn(),
+      start: vi.fn(),
+    } as unknown as MessagePort
+
+    const stream = new ChannelStream<Uint8Array>('client', port)
+    try {
+      expect(port.start).toHaveBeenCalledOnce()
+      expect(port.postMessage).toHaveBeenCalledWith({
+        ack: true,
+        from: 'client',
+      })
+    } finally {
+      stream.close()
+    }
+  })
+
   it('keeps MessagePort peer writes open after local source completes normally', async () => {
     const { port1, port2 } = new MessageChannel()
     const client = new ChannelStream<Uint8Array>('client', port1)