Apply CodeRabbit review suggestions

packages/sdk/package.json

@@ -46,7 +46,7 @@
     "@bufbuild/buf": "^1.28.1",
     "@bufbuild/protoc-gen-es": "^1.6.0",
     "@connectrpc/protoc-gen-connect-es": "^1.4.0",
-    "@types/express": "^5.0.0",
+    "@types/express": "^4.17.21",
     "@types/google-protobuf": "^3.15.5",
     "@types/long": "^4.0.1",
     "@typescript-eslint/eslint-plugin": "^6.21.0",

packages/sdk/src/api/converter.ts

@@ -803,26 +803,12 @@ function toChangePack(pack: ChangePack<Indexable>): PbChangePack {
   });
 }
 
-/**
- * `errorCodeOf` returns the error code of the given connect error.
- */
-export function errorCodeOf(error: ConnectError): string {
-  // NOTE(hackerwins): Currently, we only use the first detail to represent the
-  // error code.
-  const infos = error.findDetails(ErrorInfo);
-  for (const info of infos) {
-    if (info.metadata.code) {
-      return info.metadata.code;
-    }
-  }
-
-  return '';
-}
-
 /**
  * `errorMetadataOf` returns the error code of the given connect error.
  */
 export function errorMetadataOf(error: ConnectError): Record<string, string> {
+  // NOTE(chacha912): Currently, we only use the first detail to represent the
+  // error metadata.
   const infos = error.findDetails(ErrorInfo);
   for (const info of infos) {
     return info.metadata;
@@ -831,6 +817,13 @@ export function errorMetadataOf(error: ConnectError): Record<string, string> {
   return {};
 }
 
+/**
+ * `errorCodeOf` returns the error code of the given connect error.
+ */
+export function errorCodeOf(error: ConnectError): string {
+  return errorMetadataOf(error).code ?? '';
+}
+
 /**
  * `fromChangeID` converts the given Protobuf format to model format.
  */

packages/sdk/src/client/client.ts

@@ -1181,6 +1181,10 @@ export class Client {
           ],
         }),
       );
+      this.rpcClient.transport.interceptors = [
+        createAuthInterceptor(this.apiKey, token),
+        createMetricInterceptor(),
+      ];
       return true;
     }