From d154914db74627bb70bbaf1842a9eb937e67a103 Mon Sep 17 00:00:00 2001
From: root <root@CPC-xiaof-9ATXB.>
Date: Tue, 6 May 2025 10:13:09 +0000
Subject: [PATCH] fix: change pid return type from undefined to be null

---
 src/client/stdio.test.ts | 4 ++--
 src/client/stdio.ts      | 9 +++++++--
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/src/client/stdio.test.ts b/src/client/stdio.test.ts
index 96dd5648..b2132446 100644
--- a/src/client/stdio.test.ts
+++ b/src/client/stdio.test.ts
@@ -64,7 +64,7 @@ test("should return child process pid", async () => {
   const client = new StdioClientTransport(serverParameters);
 
   await client.start();
-  expect(client.pid).toBeDefined();
+  expect(client.pid).not.toBeNull();
   await client.close();
-  expect(client.pid).toBeUndefined();
+  expect(client.pid).toBeNull();
 });
diff --git a/src/client/stdio.ts b/src/client/stdio.ts
index 4f7f5f1b..af29614e 100644
--- a/src/client/stdio.ts
+++ b/src/client/stdio.ts
@@ -184,8 +184,13 @@ export class StdioClientTransport implements Transport {
     return this._process?.stderr ?? null;
   }
 
-  get pid(): number | undefined {
-    return this._process?.pid;
+  /**
+   * The child process pid spawned by this transport.
+   *
+   * This is only available after the transport has been started.
+   */
+  get pid(): number | null {
+    return this._process?.pid ?? null;
   }
 
   private processReadBuffer() {