refactor(sdk): add reader subpath exports and narrow utils logging deps (#8616)

Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>

Author: paulbalaji

.changeset/sdk-subpaths-utils-logging.md

@@ -0,0 +1,6 @@
+---
+'@hyperlane-xyz/sdk': patch
+'@hyperlane-xyz/utils': patch
+---
+
+Added `@hyperlane-xyz/sdk` subpath exports for `core`, `hook`, and `ism` modules. Hardened `@hyperlane-xyz/utils` pretty-mode logging with a graceful fallback when `pino-pretty` is not installed.

pnpm-lock.yaml

@@ -24,6 +24,7 @@
     "format": "oxfmt --write ./src"
   },
   "dependencies": {
+    "@google-cloud/pino-logging-gcp-config": "catalog:",
     "@hyperlane-xyz/registry": "catalog:",
     "@hyperlane-xyz/sdk": "workspace:*",
     "@hyperlane-xyz/utils": "workspace:*",

typescript/fee-quoting/package.json

@@ -25,6 +25,7 @@
     "test:unit": "mocha --config .mocharc.json './test/**/*.ts' --exit"
   },
   "dependencies": {
+    "@google-cloud/pino-logging-gcp-config": "catalog:",
     "@hyperlane-xyz/registry": "catalog:",
     "@hyperlane-xyz/sdk": "workspace:*",
     "@hyperlane-xyz/utils": "workspace:*",

typescript/http-registry-server/package.json

@@ -21,6 +21,18 @@
       "types": "./dist/index.d.ts",
       "default": "./dist/index.js"
     },
+    "./core/*": {
+      "types": "./dist/core/*.d.ts",
+      "default": "./dist/core/*.js"
+    },
+    "./hook/*": {
+      "types": "./dist/hook/*.d.ts",
+      "default": "./dist/hook/*.js"
+    },
+    "./ism/*": {
+      "types": "./dist/ism/*.d.ts",
+      "default": "./dist/ism/*.js"
+    },
     "./metadata/*": {
       "types": "./dist/metadata/*.d.ts",
       "default": "./dist/metadata/*.js"

typescript/sdk/package.json

@@ -75,52 +75,64 @@ export function createHyperlanePinoLogger(
   logLevel: LevelWithSilent,
   logFormat: LogFormat,
 ) {
+  const createFallbackLogger = () =>
+    pino({
+      level: logLevel,
+      name: 'hyperlane',
+      formatters: {
+        // Remove pino's default bindings of hostname but keep pid
+        bindings: (defaultBindings) => ({ pid: defaultBindings.pid }),
+      },
+      hooks: {
+        logMethod(inputArgs, method, level) {
+          // Pino has no simple way of setting custom log shapes and they
+          // recommend against using pino-pretty in production so when
+          // pretty is enabled we circumvent pino and log directly to console
+          if (
+            logFormat === LogFormat.Pretty &&
+            level >= pino.levels.values[logLevel]
+          ) {
+            // eslint-disable-next-line no-console
+            console.log(...inputArgs);
+            // Then return null to prevent pino from logging
+            return null;
+          }
+          return method.apply(this, inputArgs);
+        },
+      },
+    });
+
   // In development, pino-pretty is used for a better dev experience,
   // but only if the log format is 'pretty'. This allows for JSON logs
   // in development as well if explicitly configured.
   if (
     process.env.NODE_ENV === 'development' &&
     logFormat === LogFormat.Pretty
   ) {
-    return pino({
-      level: logLevel,
-      transport: {
-        target: 'pino-pretty',
-        options: {
-          colorize: true,
-          translateTime: 'SYS:standard',
-          ignore: 'pid,hostname',
+    try {
+      return pino({
+        level: logLevel,
+        transport: {
+          target: 'pino-pretty',
+          options: {
+            colorize: true,
+            translateTime: 'SYS:standard',
+            ignore: 'pid,hostname',
+          },
         },
-      },
-    });
+      });
+    } catch (err) {
+      const fallbackLogger = createFallbackLogger();
+      fallbackLogger.warn(
+        err,
+        'Could not initialize pino-pretty, falling back to built-in pretty logger',
+      );
+      return fallbackLogger;
+    }
   }
 
   // In production (or other envs), use the original hook-based logger
-  return pino({
-    level: logLevel,
-    name: 'hyperlane',
-    formatters: {
-      // Remove pino's default bindings of hostname but keep pid
-      bindings: (defaultBindings) => ({ pid: defaultBindings.pid }),
-    },
-    hooks: {
-      logMethod(inputArgs, method, level) {
-        // Pino has no simple way of setting custom log shapes and they
-        // recommend against using pino-pretty in production so when
-        // pretty is enabled we circumvent pino and log directly to console
-        if (
-          logFormat === LogFormat.Pretty &&
-          level >= pino.levels.values[logLevel]
-        ) {
-          // eslint-disable-next-line no-console
-          console.log(...inputArgs);
-          // Then return null to prevent pino from logging
-          return null;
-        }
-        return method.apply(this, inputArgs);
-      },
-    },
-  });
+  return createFallbackLogger();
 }
 
 export function ethersBigNumberSerializer(key: string, value: any): any {