meno-core 1.0.53 → 1.1.0

package/lib/server/services/componentService.ts

@@ -1,13 +1,13 @@
 /**
  * Component Service
  * Handles component loading and management
- * 
+ *
  * Provides methods for loading components from the file system, caching them in memory,
  * and managing component definitions including saving JavaScript and CSS files.
  */
 
-import { join } from 'path';
-import { existsSync, mkdirSync, unlinkSync, readdirSync, rmdirSync } from 'fs';
+import { join } from 'node:path';
+import { existsSync, mkdirSync, readdirSync, rmdirSync } from 'node:fs';
 import { loadComponentDirectory, loadJSONFile, parseJSON } from '../jsonLoader';
 import type { ComponentWithCategory, ComponentDirectoryResult, ComponentLoadDiagnostic } from '../jsonLoader';
 import { projectPaths } from '../projectContext';
@@ -15,6 +15,7 @@ import type { ComponentDefinition, JSONPage } from '../../shared/types';
 import { readTextFile, fileExists } from '../runtime';
 import type { PageService } from './pageService';
 import { rewriteComponentRefs } from '../../shared/componentRefs';
+import { isSafePathSegment, isPathWithinRoot } from '../../shared/pathSecurity';
 
 // Re-export for callers that already imported the helper from this module.
 export { rewriteComponentRefs };
@@ -157,7 +158,10 @@ export class ComponentService {
 
     if (this.loader) {
       // Legacy loader interface returns plain Map
-      loadedComponents = await this.loader.loadDirectory(projectPaths.components()) as Map<string, ComponentWithCategory>;
+      loadedComponents = (await this.loader.loadDirectory(projectPaths.components())) as Map<
+        string,
+        ComponentWithCategory
+      >;
     } else {
       const result: ComponentDirectoryResult = await loadComponentDirectory(projectPaths.components());
       loadedComponents = result.components;
@@ -189,10 +193,10 @@ export class ComponentService {
 
   /**
    * Get component by name
-   * 
+   *
    * @param name - Component name (without .json extension)
    * @returns ComponentDefinition if found, undefined otherwise
-   * 
+   *
    * @example
    * ```typescript
    * const button = componentService.getComponent('Button');
@@ -207,10 +211,10 @@ export class ComponentService {
 
   /**
    * Check if component exists
-   * 
+   *
    * @param name - Component name to check
    * @returns True if component exists in cache, false otherwise
-   * 
+   *
    * @example
    * ```typescript
    * if (componentService.hasComponent('Button')) {
@@ -266,7 +270,7 @@ export class ComponentService {
     this.components.forEach((value, key) => {
       record[key] = {
         definition: value,
-        category: this.componentCategories.get(key)
+        category: this.componentCategories.get(key),
       };
     });
     return record;
@@ -308,10 +312,10 @@ export class ComponentService {
 
   /**
    * Validate that a component has a valid structure
-   * 
+   *
    * @param componentDef - ComponentDefinition to validate
    * @returns true if component has valid structure, false otherwise
-   * 
+   *
    * @example
    * ```typescript
    * const component = componentService.getComponent('Button');
@@ -324,35 +328,35 @@ export class ComponentService {
     if (!componentDef) {
       return false;
     }
-    
+
     // Handle ComponentDefinition in any format:
     // 1. Legacy: { type: string, component: { structure: {...} } }
     // 2. New: { component: { structure: {...} } }
     // 3. Direct: { component: { structure: {...} } } (already normalized)
-    
+
     // Access component field safely (handle both TypeScript types and runtime data)
     const component = (componentDef as any).component;
-    
+
     if (!component || typeof component !== 'object') {
       return false;
     }
-    
+
     // Check for structure in component
     const structure = component.structure;
     if (!structure) {
       return false;
     }
-    
+
     // Validate structure is a proper object with type property
     if (typeof structure !== 'object' || structure === null || Array.isArray(structure)) {
       return false;
     }
-    
+
     // Structure must have a type property that's a string
     if (!('type' in structure) || typeof structure.type !== 'string') {
       return false;
     }
-    
+
     return true;
   }
 
@@ -362,8 +366,15 @@ export class ComponentService {
    */
   private getComponentDir(name: string, category?: string): string {
     const componentsDir = this.componentsBaseDir();
+    // A component name becomes a filename — it must be a single safe segment (no
+    // `..`/separators). The resolved category dir must stay within the components
+    // root (nested categories are allowed, traversal is not). Mirrors the guards
+    // renameComponent/createFolder already apply; the save path previously lacked them.
+    if (!isSafePathSegment(name)) throw new Error(`unsafe component name: ${name}`);
     const cat = this.resolveComponentCategory(name, category);
-    return cat ? join(componentsDir, cat) : componentsDir;
+    const dir = cat ? join(componentsDir, cat) : componentsDir;
+    if (!isPathWithinRoot(dir, componentsDir)) throw new Error(`component category escapes components dir: ${cat}`);
+    return dir;
   }
 
   /**
@@ -447,7 +458,7 @@ export class ComponentService {
         return await readTextFile(jsFilePath);
       }
       return null;
-    } catch (error) {
+    } catch (_error) {
       return null;
     }
   }
@@ -480,11 +491,27 @@ export class ComponentService {
    * ```
    */
   async saveComponent(name: string, data: ComponentDefinition, category?: string): Promise<void> {
-    // Create a copy without the javascript field (JavaScript should only be in .js files)
+    // The `.json` PAYLOAD never carries javascript (it lives in the sibling `.js`
+    // file), so the JSON write below strips it. But the component's JS must NOT be
+    // lost in the process:
+    //   - single-file writers (the astro `.astro` format) re-emit the WHOLE file
+    //     from the def they're given — writing a js-less def silently deletes the
+    //     component's `<script>` from the source of truth;
+    //   - the in-memory cache previously took the stripped copy too, so even JSON
+    //     projects lost the component's JS from the live cache (SSR preview renders
+    //     without it) until a file-watcher reload.
+    // Preserve: the incoming def's javascript wins; absent that, the cached one.
     const dataWithoutJS = JSON.parse(JSON.stringify(data));
     if (dataWithoutJS?.component?.javascript !== undefined) {
       delete dataWithoutJS.component.javascript;
     }
+    const incomingJs = (data as { component?: { javascript?: string } })?.component?.javascript;
+    const cachedJs = this.components.get(name)?.component?.javascript;
+    const javascript = incomingJs !== undefined ? incomingJs : cachedJs;
+    const dataWithJS: ComponentDefinition =
+      javascript !== undefined
+        ? { ...dataWithoutJS, component: { ...dataWithoutJS.component, javascript } }
+        : dataWithoutJS;
 
     // Determine target directory. Resolver consults the explicit arg, then
     // the in-memory cache, then disk — so a save during the file-watcher
@@ -496,13 +523,13 @@ export class ComponentService {
 
     if (this.writer) {
       // Format-aware write (e.g. emit a `.astro` file). The writer owns directory
-      // creation + serialization. The category folder is part of `componentDir`.
-      await this.writer.writeComponent(componentDir, name, dataWithoutJS);
+      // creation + serialization, and for single-file formats the def must carry
+      // EVERYTHING — including javascript (see the ComponentWriter contract).
+      await this.writer.writeComponent(componentDir, name, dataWithJS);
     } else {
-      // JSON path — unchanged, byte-identical to legacy behavior.
-      const writeFile = this.fs
-        ? this.fs.writeFile.bind(this.fs)
-        : (await import('fs/promises')).writeFile;
+      // JSON path — unchanged, byte-identical to legacy behavior (js-less payload;
+      // the sibling `.js` file owns the script).
+      const writeFile = this.fs ? this.fs.writeFile.bind(this.fs) : (await import('node:fs/promises')).writeFile;
 
       // Create category directory if needed
       if (targetCategory && !existsSync(componentDir)) {
@@ -513,8 +540,9 @@ export class ComponentService {
       await writeFile(filePath, JSON.stringify(dataWithoutJS, null, 2), 'utf-8');
     }
 
-    // Update in-memory cache
-    this.components.set(name, dataWithoutJS);
+    // Update in-memory cache with the FULL def (js preserved) so the live preview
+    // keeps rendering the component's script between save and watcher reload.
+    this.components.set(name, dataWithJS);
     this.componentCategories.set(name, targetCategory);
   }
 
@@ -550,9 +578,7 @@ export class ComponentService {
       return;
     }
 
-    const writeFile = this.fs
-      ? this.fs.writeFile.bind(this.fs)
-      : (await import('fs/promises')).writeFile;
+    const writeFile = this.fs ? this.fs.writeFile.bind(this.fs) : (await import('node:fs/promises')).writeFile;
     const jsFilePath = join(componentDir, `${name}.js`);
     await writeFile(jsFilePath, javascript || '', 'utf-8');
 
@@ -569,7 +595,7 @@ export class ComponentService {
           }
           parsed.component.javascript = jsContent;
         }
-      } catch (error) {
+      } catch (_error) {
         // Silently handle JS file read errors
       }
       this.components.set(name, parsed);
@@ -608,9 +634,7 @@ export class ComponentService {
       return;
     }
 
-    const writeFile = this.fs
-      ? this.fs.writeFile.bind(this.fs)
-      : (await import('fs/promises')).writeFile;
+    const writeFile = this.fs ? this.fs.writeFile.bind(this.fs) : (await import('node:fs/promises')).writeFile;
     const cssFilePath = join(componentDir, `${name}.css`);
     await writeFile(cssFilePath, css || '', 'utf-8');
 
@@ -627,7 +651,7 @@ export class ComponentService {
           }
           parsed.component.css = cssContent;
         }
-      } catch (error) {
+      } catch (_error) {
         // Silently handle CSS file read errors
       }
       this.components.set(name, parsed);
@@ -657,8 +681,8 @@ export class ComponentService {
     try {
       const entries = readdirSync(componentsDir, { withFileTypes: true });
       return entries
-        .filter(entry => entry.isDirectory())
-        .map(entry => entry.name)
+        .filter((entry) => entry.isDirectory())
+        .map((entry) => entry.name)
         .sort();
     } catch {
       return [];
@@ -687,7 +711,10 @@ export class ComponentService {
     }
 
     // Validate folder name (alphanumeric, dashes, underscores only)
-    const sanitized = folderName.trim().toLowerCase().replace(/[^a-z0-9-_]/g, '-');
+    const sanitized = folderName
+      .trim()
+      .toLowerCase()
+      .replace(/[^a-z0-9-_]/g, '-');
     if (sanitized !== folderName.trim()) {
       throw new Error('Folder name can only contain lowercase letters, numbers, dashes, and underscores');
     }
@@ -748,7 +775,7 @@ export class ComponentService {
 
       // Move all component files (.json, .js, .css)
       const extensions = ['.json', '.js', '.css'];
-      const { rename } = await import('fs/promises');
+      const { rename } = await import('node:fs/promises');
 
       for (const ext of extensions) {
         const sourcePath = join(sourceDir, `${name}${ext}`);
@@ -843,7 +870,7 @@ export class ComponentService {
         }
       }
 
-      const { rename } = await import('fs/promises');
+      const { rename } = await import('node:fs/promises');
       for (const ext of ['.json', '.js', '.css'] as const) {
         const src = join(dir, `${oldName}${ext}`);
         const dst = join(dir, `${trimmedNew}${ext}`);
@@ -864,9 +891,7 @@ export class ComponentService {
     // each one that actually changed. The owning component's own structure
     // shouldn't reference itself, so we don't bother walking it.
     let componentRefs = 0;
-    const writeFile = this.fs
-      ? this.fs.writeFile.bind(this.fs)
-      : (await import('fs/promises')).writeFile;
+    const writeFile = this.fs ? this.fs.writeFile.bind(this.fs) : (await import('node:fs/promises')).writeFile;
     for (const [name, otherDef] of this.components.entries()) {
       if (name === trimmedNew) continue;
       const structure = otherDef.component?.structure;
@@ -912,4 +937,3 @@ export class ComponentService {
     };
   }
 }
-

package/lib/server/services/configService.test.ts

@@ -1,12 +1,11 @@
-import { describe, test, expect, beforeEach, afterEach, mock } from 'bun:test';
-import { join } from 'path';
-import { mkdirSync, rmSync, writeFileSync, existsSync } from 'fs';
+import { describe, test, expect, beforeEach, afterEach } from 'bun:test';
+import { join } from 'node:path';
+import { mkdirSync, rmSync, writeFileSync, existsSync } from 'node:fs';
 
 // We need to mock projectPaths.config() before importing ConfigService.
 // Since projectPaths is a module-level export, we mock the projectContext module.
 import { setProjectRoot, getProjectRoot } from '../../server/projectContext';
 import { ConfigService } from './configService';
-import type { IconsConfig } from './configService';
 import { DEFAULT_BREAKPOINTS } from '../../shared/breakpoints';
 import { DEFAULT_RESPONSIVE_SCALES } from '../../shared/responsiveScaling';
 import { DEFAULT_I18N_CONFIG } from '../../shared/i18n';
@@ -209,8 +208,8 @@ describe('ConfigService', () => {
       await service.load();
 
       const result = service.getBreakpoints();
-      expect(result.tablet.breakpoint).toBe(1024);
-      expect(result.tablet.previewPoint).toBe(1024);
+      expect(result.tablet!.breakpoint).toBe(1024);
+      expect(result.tablet!.previewPoint).toBe(1024);
     });
 
     test('should fall back to defaults for non-positive number values', async () => {
@@ -278,7 +277,7 @@ describe('ConfigService', () => {
       await service.load();
 
       const result = service.getBreakpoints();
-      expect(result.tablet.previewPoint).toBe(1024);
+      expect(result.tablet!.previewPoint).toBe(1024);
     });
 
     test('should support custom breakpoint names', async () => {
@@ -329,12 +328,12 @@ describe('ConfigService', () => {
       const result = service.getI18n();
       expect(result.defaultLocale).toBe('en');
       expect(result.locales).toHaveLength(2);
-      expect(result.locales[0].code).toBe('en');
-      expect(result.locales[1].code).toBe('pl');
+      expect(result.locales[0]!.code).toBe('en');
+      expect(result.locales[1]!.code).toBe('pl');
       // Migrated locales should have name, nativeName, langTag
-      expect(result.locales[1].name).toBe('PL');
-      expect(result.locales[1].nativeName).toBe('PL');
-      expect(result.locales[1].langTag).toBe('pl-PL');
+      expect(result.locales[1]!.name).toBe('PL');
+      expect(result.locales[1]!.nativeName).toBe('PL');
+      expect(result.locales[1]!.langTag).toBe('pl-PL');
     });
 
     test('should pass through new LocaleConfig[] format', async () => {
@@ -547,10 +546,7 @@ describe('ConfigService', () => {
     test('should normalize string URLs to object format for JS libraries', async () => {
       writeConfig({
         libraries: {
-          js: [
-            'https://cdn.example.com/lib.js',
-            'https://cdn.example.com/other.js',
-          ],
+          js: ['https://cdn.example.com/lib.js', 'https://cdn.example.com/other.js'],
         },
       });
       await service.load();
@@ -565,62 +561,44 @@ describe('ConfigService', () => {
     test('should normalize string URLs to object format for CSS libraries', async () => {
       writeConfig({
         libraries: {
-          css: [
-            'https://fonts.googleapis.com/css2?family=Inter',
-          ],
+          css: ['https://fonts.googleapis.com/css2?family=Inter'],
         },
       });
       await service.load();
 
       const result = service.getLibraries();
-      expect(result.css).toEqual([
-        { url: 'https://fonts.googleapis.com/css2?family=Inter' },
-      ]);
+      expect(result.css).toEqual([{ url: 'https://fonts.googleapis.com/css2?family=Inter' }]);
     });
 
     test('should pass through object format for JS libraries', async () => {
       writeConfig({
         libraries: {
-          js: [
-            { url: 'https://cdn.example.com/lib.js', mode: 'async', position: 'head' },
-          ],
+          js: [{ url: 'https://cdn.example.com/lib.js', mode: 'async', position: 'head' }],
         },
       });
       await service.load();
 
       const result = service.getLibraries();
-      expect(result.js).toEqual([
-        { url: 'https://cdn.example.com/lib.js', mode: 'async', position: 'head' },
-      ]);
+      expect(result.js).toEqual([{ url: 'https://cdn.example.com/lib.js', mode: 'async', position: 'head' }]);
     });
 
     test('should pass through object format for CSS libraries', async () => {
       writeConfig({
         libraries: {
-          css: [
-            { url: 'https://fonts.googleapis.com/style.css', media: 'screen' },
-          ],
+          css: [{ url: 'https://fonts.googleapis.com/style.css', media: 'screen' }],
         },
       });
       await service.load();
 
       const result = service.getLibraries();
-      expect(result.css).toEqual([
-        { url: 'https://fonts.googleapis.com/style.css', media: 'screen' },
-      ]);
+      expect(result.css).toEqual([{ url: 'https://fonts.googleapis.com/style.css', media: 'screen' }]);
     });
 
     test('should handle mixed string and object formats', async () => {
       writeConfig({
         libraries: {
-          js: [
-            'https://cdn.example.com/simple.js',
-            { url: 'https://cdn.example.com/complex.js', mode: 'defer' },
-          ],
-          css: [
-            'https://cdn.example.com/simple.css',
-            { url: 'https://cdn.example.com/complex.css', media: 'print' },
-          ],
+          js: ['https://cdn.example.com/simple.js', { url: 'https://cdn.example.com/complex.js', mode: 'defer' }],
+          css: ['https://cdn.example.com/simple.css', { url: 'https://cdn.example.com/complex.css', media: 'print' }],
         },
       });
       await service.load();
@@ -807,8 +785,8 @@ describe('ConfigService', () => {
 
       // Verify all getters work correctly together
       const breakpoints = service.getBreakpoints();
-      expect(breakpoints.tablet.breakpoint).toBe(1024);
-      expect(breakpoints.mobile.previewPoint).toBe(375);
+      expect(breakpoints.tablet!.breakpoint).toBe(1024);
+      expect(breakpoints.mobile!.previewPoint).toBe(375);
 
       const scales = service.getResponsiveScales();
       expect(scales.enabled).toBe(true);
@@ -827,7 +805,6 @@ describe('ConfigService', () => {
 
       const csp = service.getCSP();
       expect(csp.scriptSrc).toEqual(['https://cdn.example.com']);
-
     });
 
     test('should handle empty config object', async () => {

package/lib/server/services/configService.ts

@@ -19,6 +19,7 @@ import { DEFAULT_REM_CONFIG } from '../../shared/pxToRem';
 import { DEFAULT_I18N_CONFIG, migrateI18nConfig } from '../../shared/i18n';
 import { projectPaths } from '../projectContext';
 import { readTextFile, fileExists } from '../runtime';
+import { logRuntimeError } from '../../shared/errorLogger';
 
 /**
  * Icons configuration
@@ -36,6 +37,17 @@ export interface SocialConfig {
   twitterHandle?: string;
 }
 
+/**
+ * Sanity CMS integration config (read-only, public datasets).
+ * Connection info only — the runtime builds GROQ query URLs from this. No token (public).
+ */
+export interface SanityConfig {
+  projectId: string;
+  dataset: string;
+  apiVersion?: string;
+  useCdn?: boolean;
+}
+
 /**
  * Raw project config structure from project.config.json
  */
@@ -47,6 +59,7 @@ interface RawProjectConfig {
   i18n?: unknown;
   icons?: IconsConfig;
   social?: SocialConfig;
+  integrations?: { sanity?: SanityConfig };
   libraries?: LibrariesConfig;
   csp?: CSPConfig;
   baseComponent?: string;
@@ -78,8 +91,9 @@ export class ConfigService {
         const content = await readTextFile(projectPaths.config());
         this.config = JSON.parse(content);
       }
-    } catch {
+    } catch (error) {
       // Fall through to defaults
+      logRuntimeError('configService.load', error, { filePath: projectPaths.config() });
       this.config = null;
     }
 
@@ -123,9 +137,8 @@ export class ConfigService {
         if (typeof entry.breakpoint === 'number' && entry.breakpoint > 0) {
           validInput[key] = {
             breakpoint: entry.breakpoint,
-            previewPoint: typeof entry.previewPoint === 'number' && entry.previewPoint > 0
-              ? entry.previewPoint
-              : entry.breakpoint,
+            previewPoint:
+              typeof entry.previewPoint === 'number' && entry.previewPoint > 0 ? entry.previewPoint : entry.breakpoint,
           };
         }
       }
@@ -157,7 +170,7 @@ export class ConfigService {
    */
   private mergeScaleCategory(
     userScales: BreakpointScales | undefined,
-    defaultScales: BreakpointScales | undefined
+    defaultScales: BreakpointScales | undefined,
   ): BreakpointScales | undefined {
     if (!userScales && !defaultScales) return undefined;
     if (!userScales) return defaultScales ? { ...defaultScales } : undefined;
@@ -186,34 +199,30 @@ export class ConfigService {
       enabled: userScales.enabled ?? DEFAULT_RESPONSIVE_SCALES.enabled,
       mode: (userScales as { mode?: 'breakpoints' | 'fluid' }).mode ?? DEFAULT_RESPONSIVE_SCALES.mode,
       baseReference: userScales.baseReference ?? DEFAULT_RESPONSIVE_SCALES.baseReference,
-      fluidRange: (userScales as { fluidRange?: { min: number; max: number } }).fluidRange
-        ?? (DEFAULT_RESPONSIVE_SCALES.fluidRange ? { ...DEFAULT_RESPONSIVE_SCALES.fluidRange } : undefined),
-      siteMargin: (userScales as { siteMargin?: { min: number; max: number } }).siteMargin
-        ?? (DEFAULT_RESPONSIVE_SCALES.siteMargin ? { ...DEFAULT_RESPONSIVE_SCALES.siteMargin } : undefined),
+      fluidRange:
+        (userScales as { fluidRange?: { min: number; max: number } }).fluidRange ??
+        (DEFAULT_RESPONSIVE_SCALES.fluidRange ? { ...DEFAULT_RESPONSIVE_SCALES.fluidRange } : undefined),
+      siteMargin:
+        (userScales as { siteMargin?: { min: number; max: number } }).siteMargin ??
+        (DEFAULT_RESPONSIVE_SCALES.siteMargin ? { ...DEFAULT_RESPONSIVE_SCALES.siteMargin } : undefined),
       fontSize: this.mergeScaleCategory(
         userScales.fontSize as BreakpointScales | undefined,
-        DEFAULT_RESPONSIVE_SCALES.fontSize
+        DEFAULT_RESPONSIVE_SCALES.fontSize,
       ),
       padding: this.mergeScaleCategory(
         userScales.padding as BreakpointScales | undefined,
-        DEFAULT_RESPONSIVE_SCALES.padding
+        DEFAULT_RESPONSIVE_SCALES.padding,
       ),
       margin: this.mergeScaleCategory(
         userScales.margin as BreakpointScales | undefined,
-        DEFAULT_RESPONSIVE_SCALES.margin
-      ),
-      gap: this.mergeScaleCategory(
-        userScales.gap as BreakpointScales | undefined,
-        DEFAULT_RESPONSIVE_SCALES.gap
+        DEFAULT_RESPONSIVE_SCALES.margin,
       ),
+      gap: this.mergeScaleCategory(userScales.gap as BreakpointScales | undefined, DEFAULT_RESPONSIVE_SCALES.gap),
       borderRadius: this.mergeScaleCategory(
         userScales.borderRadius as BreakpointScales | undefined,
-        DEFAULT_RESPONSIVE_SCALES.borderRadius
-      ),
-      size: this.mergeScaleCategory(
-        userScales.size as BreakpointScales | undefined,
-        DEFAULT_RESPONSIVE_SCALES.size
+        DEFAULT_RESPONSIVE_SCALES.borderRadius,
       ),
+      size: this.mergeScaleCategory(userScales.size as BreakpointScales | undefined, DEFAULT_RESPONSIVE_SCALES.size),
     };
   }
 
@@ -254,6 +263,24 @@ export class ConfigService {
     return this.config.social;
   }
 
+  /**
+   * Get Sanity CMS integration config (read-only, public datasets).
+   * Returns undefined when not configured or missing required projectId/dataset.
+   */
+  getSanity(): SanityConfig | undefined {
+    const sanity = this.config?.integrations?.sanity;
+    if (!sanity || typeof sanity !== 'object') {
+      return undefined;
+    }
+    if (typeof sanity.projectId !== 'string' || typeof sanity.dataset !== 'string') {
+      return undefined;
+    }
+    if (sanity.projectId.trim() === '' || sanity.dataset.trim() === '') {
+      return undefined;
+    }
+    return sanity;
+  }
+
   /**
    * Get libraries configuration
    * Returns empty arrays if not configured
@@ -268,16 +295,12 @@ export class ConfigService {
 
     // Normalize JS libraries: support both string URLs and object format
     const normalizedJs = Array.isArray(libs.js)
-      ? libs.js.map((lib) =>
-          typeof lib === 'string' ? { url: lib } : lib
-        ) as JSLibraryConfig[]
+      ? (libs.js.map((lib) => (typeof lib === 'string' ? { url: lib } : lib)) as JSLibraryConfig[])
       : [];
 
     // Normalize CSS libraries: support both string URLs and object format
     const normalizedCss = Array.isArray(libs.css)
-      ? libs.css.map((lib) =>
-          typeof lib === 'string' ? { url: lib } : lib
-        ) as CSSLibraryConfig[]
+      ? (libs.css.map((lib) => (typeof lib === 'string' ? { url: lib } : lib)) as CSSLibraryConfig[])
       : [];
 
     return {