fix(core): use SdkError(NotConnected) in request() to match notification path

Author: felixweinberger

packages/core/src/shared/protocol.ts

@@ -811,7 +811,7 @@ export abstract class Protocol<ContextT extends BaseContext> {
             };
 
             if (!this._transport) {
-                earlyReject(new Error('Not connected'));
+                earlyReject(new SdkError(SdkErrorCode.NotConnected, 'Not connected'));
                 return;
             }
 

packages/core/test/shared/customMethods.test.ts

@@ -169,9 +169,11 @@ describe('custom notification handlers', () => {
 });
 
 describe('sendCustomRequest', () => {
-    test('not connected -> rejects', async () => {
+    test('not connected -> throws SdkError NotConnected', async () => {
         const proto = new TestProtocol();
-        await expect(proto.sendCustomRequest('acme/x', {}, z.object({}))).rejects.toThrow(/Not connected/);
+        await expect(proto.sendCustomRequest('acme/x', {}, z.object({}))).rejects.toSatisfy(
+            (e: unknown) => e instanceof SdkError && e.code === SdkErrorCode.NotConnected
+        );
     });
 
     test('undefined params accepted', async () => {