feat: restructure logger module and update build configuration for improved browser compatibility

Author: hotlong

packages/client/src/index.ts

@@ -87,7 +87,7 @@ import {
   WellKnownCapabilities,
   ApiRoutes,
 } from '@objectstack/spec/api';
-import { Logger, createLogger } from '@objectstack/core';
+import { Logger, createLogger } from '@objectstack/core/logger';
 import { RealtimeAPI } from './realtime-api';
 
 /**

packages/core/package.json

@@ -11,10 +11,15 @@
       "types": "./dist/index.d.ts",
       "import": "./dist/index.js",
       "require": "./dist/index.cjs"
+    },
+    "./logger": {
+      "types": "./dist/logger.d.ts",
+      "import": "./dist/logger.js",
+      "require": "./dist/logger.cjs"
     }
   },
   "scripts": {
-    "build": "tsup --config ../../tsup.config.ts",
+    "build": "tsup",
     "test": "vitest run",
     "test:watch": "vitest"
   },

packages/core/src/logger.ts

@@ -3,6 +3,11 @@
 import type { LoggerConfig, LogLevel } from '@objectstack/spec/system';
 import type { Logger } from '@objectstack/spec/contracts';
 
+// Re-export the contract type so consumers can do
+// `import type { Logger } from '@objectstack/core/logger'` without also
+// pulling `@objectstack/spec` into their bundle graph manually.
+export type { Logger };
+
 const LEVEL_ORDER: Record<LogLevel, number> = {
     debug: 0,
     info: 1,
@@ -115,10 +120,23 @@ export class ObjectLogger implements Logger {
 
         const out = line + '\n';
 
-        if (level === 'error' || level === 'fatal') {
-            process.stderr?.write(out);
-        } else {
-            process.stdout?.write(out);
+        // Browser-safe output: prefer process streams when available, otherwise
+        // fall back to console. The previous unguarded `process.stderr?.write`
+        // throws `ReferenceError: process is not defined` in browsers because
+        // `process` itself is the missing global, not just its `stderr` field.
+        if (typeof process !== 'undefined' && (process as any).stderr) {
+            if (level === 'error' || level === 'fatal') {
+                (process as any).stderr.write(out);
+            } else {
+                (process as any).stdout?.write(out);
+            }
+        } else if (typeof console !== 'undefined') {
+            const fn =
+                level === 'error' || level === 'fatal' ? console.error
+                : level === 'warn' ? console.warn
+                : level === 'debug' ? console.debug
+                : console.log;
+            fn(line);
         }
 
         if (this.fileStream) {

packages/core/tsup.config.ts

@@ -0,0 +1,19 @@
+import { defineConfig } from 'tsup';
+
+/**
+ * `@objectstack/core` ships two entry points:
+ * - `index.ts` — full microkernel (Node-only; pulls in plugin sandbox,
+ *   filesystem helpers, etc).
+ * - `logger.ts` — tiny browser-safe logger reused by `@objectstack/client`
+ *   so importing the client SDK in a browser bundle does NOT drag in
+ *   Node-only kernel internals (e.g. `process.cpuUsage()`).
+ */
+export default defineConfig({
+  entry: ['src/index.ts', 'src/logger.ts'],
+  splitting: false,
+  sourcemap: true,
+  clean: true,
+  dts: true,
+  format: ['esm', 'cjs'],
+  target: 'es2020',
+});