@happyvertical/smrt-svelte 0.37.7 → 0.37.9

package/AGENTS.md

@@ -233,7 +233,7 @@ await expectNoA11yViolations(container); // axe; color-contrast off (jsdom has n
 - `@happyvertical/smrt-types` (shared types) — includes the identity data contracts (`User`, `Role`, `Membership`, `Tenant`) the role/membership components type against, so no dependency on `smrt-users` / `smrt-profiles` is needed
 - `@happyvertical/smrt-ui` (UI runtime: primitives, theme system, i18n client, module registry) is a hard `dependency`. The agent-admin shells that used to type against `@happyvertical/smrt-agents/ui` moved to `@happyvertical/smrt-agents/svelte` (#1589), so `smrt-agents` is no longer a dependency here — this drops smrt-svelte below smrt-agents in the package DAG.
 - `@happyvertical/smrt-languages` is a hard `dependency` (not an optional peer): the Node-only `/i18n/server` subpath imports its resolver. The browser bundle still excludes it — the client `/i18n` layer never imports the languages root, so it tree-shakes out.
-- `@happyvertical/logger` (SDK) is a `dependency` — the browser-safe console logger used for voice/AI error reporting in the form components.
+- `@happyvertical/logger` (SDK) is a `dependency` — the console logger used for voice/AI error reporting in the form components. Consume it **only** through `src/internal/logger.ts`, never `createLogger()` at module scope: `createLogger()` reads `HAVE_LOGGER_LEVEL` from `process.env`, so a top-level call throws `ReferenceError: process is not defined` in the browser and kills client-side hydration under `vite dev` (prod builds tree-shake/define it away, so this only bites in dev). The `internal/logger` wrapper constructs the logger lazily and falls back to a bare `ConsoleLogger` when `process.env` is absent, keeping this browser-reachable module (imported by `Provider` + the form primitives) safe.
 - Peer (all optional): `svelte` >=5.18.2, plus the browser-AI engines (`@huggingface/transformers`, `@mlc-ai/web-llm`, `@remotion/whisper-web`, `@xenova/transformers`) and `chrono-node`.
 
 ## Workspace shell primitives

package/dist/internal/logger.d.ts

@@ -1,12 +1,19 @@
 /**
  * Shared package logger for `@happyvertical/smrt-svelte`.
  *
- * Browser-safe: the `@happyvertical/logger` console adapter pulls no Node
- * built-ins and its `@sentry/node` integration is an optional peer that is
- * never imported here. Centralising the instance keeps voice/AI error reporting
- * consistent across the form components (mic permission denials, STT init
- * failures) instead of swallowing them in empty `catch` blocks or scattering
- * raw `console.*` calls.
+ * This module is browser-reachable (Provider + the form components import it),
+ * so it must never touch Node globals at module scope. `createLogger()` reads
+ * `HAVE_LOGGER_LEVEL` from `process.env`, and `process` does not exist in
+ * browsers — calling it during module init threw
+ * `ReferenceError: process is not defined` under `vite dev` and killed
+ * client-side hydration for every consuming app. The logger is therefore
+ * created lazily on first use, and the env-config path is only consulted when
+ * a usable `process.env` actually exists; browsers get the plain console
+ * logger at the same level.
+ *
+ * Centralising the instance keeps voice/AI error reporting consistent across
+ * the form components (mic permission denials, STT init failures) instead of
+ * swallowing them in empty `catch` blocks or scattering raw `console.*` calls.
  */
 import { type Logger } from '@happyvertical/logger';
 export declare const logger: Logger;

package/dist/internal/logger.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"logger.d.ts","sourceRoot":"","sources":["../../src/internal/logger.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AACH,OAAO,EAAgB,KAAK,MAAM,EAAE,MAAM,uBAAuB,CAAC;AAElE,eAAO,MAAM,MAAM,EAAE,MAAwC,CAAC"}
+{"version":3,"file":"logger.d.ts","sourceRoot":"","sources":["../../src/internal/logger.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AACH,OAAO,EAGL,KAAK,MAAM,EACZ,MAAM,uBAAuB,CAAC;AAmB/B,eAAO,MAAM,MAAM,EAAE,MAKpB,CAAC"}

package/dist/internal/logger.js

@@ -1,12 +1,37 @@
 /**
  * Shared package logger for `@happyvertical/smrt-svelte`.
  *
- * Browser-safe: the `@happyvertical/logger` console adapter pulls no Node
- * built-ins and its `@sentry/node` integration is an optional peer that is
- * never imported here. Centralising the instance keeps voice/AI error reporting
- * consistent across the form components (mic permission denials, STT init
- * failures) instead of swallowing them in empty `catch` blocks or scattering
- * raw `console.*` calls.
+ * This module is browser-reachable (Provider + the form components import it),
+ * so it must never touch Node globals at module scope. `createLogger()` reads
+ * `HAVE_LOGGER_LEVEL` from `process.env`, and `process` does not exist in
+ * browsers — calling it during module init threw
+ * `ReferenceError: process is not defined` under `vite dev` and killed
+ * client-side hydration for every consuming app. The logger is therefore
+ * created lazily on first use, and the env-config path is only consulted when
+ * a usable `process.env` actually exists; browsers get the plain console
+ * logger at the same level.
+ *
+ * Centralising the instance keeps voice/AI error reporting consistent across
+ * the form components (mic permission denials, STT init failures) instead of
+ * swallowing them in empty `catch` blocks or scattering raw `console.*` calls.
  */
-import { createLogger } from '@happyvertical/logger';
-export const logger = createLogger({ level: 'warn' });
+import { ConsoleLogger, createLogger, } from '@happyvertical/logger';
+const LEVEL = 'warn';
+let instance;
+function resolveLogger() {
+    if (!instance) {
+        const hasProcessEnv = typeof process !== 'undefined' &&
+            typeof process.env === 'object' &&
+            process.env !== null;
+        instance = hasProcessEnv
+            ? createLogger({ level: LEVEL })
+            : new ConsoleLogger(LEVEL);
+    }
+    return instance;
+}
+export const logger = {
+    debug: (message, context) => resolveLogger().debug(message, context),
+    info: (message, context) => resolveLogger().info(message, context),
+    warn: (message, context) => resolveLogger().warn(message, context),
+    error: (message, context) => resolveLogger().error(message, context),
+};

package/dist/internal/logger.test.js

@@ -0,0 +1,54 @@
+/**
+ * Regression coverage for the "process is not defined" hydration killer.
+ *
+ * `internal/logger` is browser-reachable through Provider and the form
+ * components, and `@happyvertical/logger`'s `createLogger()` eagerly reads
+ * `process.env` (HAVE_LOGGER_LEVEL). Constructing the logger at module scope
+ * therefore threw during Vite dev pre-bundle evaluation in browsers and killed
+ * client-side hydration for every consuming app.
+ *
+ * vite-node's module runner itself needs `process.platform` to load inline
+ * modules, so the test cannot delete `process` outright the way a real browser
+ * lacks it. Stubbing it to an object without a usable `env` reproduces the
+ * same failure mode: the pre-fix module init still blows up at import time
+ * (`Object.keys(process.env)`), and the fixed guard must route to the plain
+ * console logger.
+ */
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+describe('internal/logger', () => {
+    beforeEach(() => {
+        vi.resetModules();
+    });
+    afterEach(() => {
+        vi.unstubAllGlobals();
+        vi.restoreAllMocks();
+    });
+    it('imports and logs without a usable process.env (browser)', async () => {
+        vi.stubGlobal('process', {});
+        const { logger } = await import('./logger.js');
+        const warn = vi.spyOn(console, 'warn').mockImplementation(() => { });
+        logger.warn('mic permission denied');
+        expect(warn).toHaveBeenCalledTimes(1);
+        expect(warn.mock.calls[0][0]).toContain('mic permission denied');
+    });
+    it('filters below the warn level in the browser fallback', async () => {
+        vi.stubGlobal('process', {});
+        const { logger } = await import('./logger.js');
+        const debug = vi.spyOn(console, 'debug').mockImplementation(() => { });
+        const info = vi.spyOn(console, 'info').mockImplementation(() => { });
+        const error = vi.spyOn(console, 'error').mockImplementation(() => { });
+        logger.debug('below threshold');
+        logger.info('below threshold');
+        logger.error('stt init failed');
+        expect(debug).not.toHaveBeenCalled();
+        expect(info).not.toHaveBeenCalled();
+        expect(error).toHaveBeenCalledTimes(1);
+    });
+    it('logs through the env-aware logger when process exists (node/SSR)', async () => {
+        const { logger } = await import('./logger.js');
+        const warn = vi.spyOn(console, 'warn').mockImplementation(() => { });
+        logger.warn('stt init failed', { adapter: 'whisper-cpp' });
+        expect(warn).toHaveBeenCalledTimes(1);
+        expect(warn.mock.calls[0][0]).toContain('stt init failed');
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happyvertical/smrt-svelte",
-  "version": "0.37.7",
+  "version": "0.37.9",
   "description": "Svelte 5 components for SMRT user management - auth, users, tenants, roles, permissions, groups",
   "type": "module",
   "smrtRawPrimitives": "strict",
@@ -78,9 +78,9 @@
   "dependencies": {
     "@happyvertical/logger": "^0.74.11",
     "esm-env": "^1.2.2",
-    "@happyvertical/smrt-languages": "0.37.7",
-    "@happyvertical/smrt-types": "0.37.7",
-    "@happyvertical/smrt-ui": "0.37.7"
+    "@happyvertical/smrt-languages": "0.37.9",
+    "@happyvertical/smrt-ui": "0.37.9",
+    "@happyvertical/smrt-types": "0.37.9"
   },
   "peerDependencies": {
     "@huggingface/transformers": ">=3.8.1",