meno-core 1.0.53 → 1.1.0

package/lib/shared/richtext/htmlToTiptap.ts

@@ -250,7 +250,7 @@ function parseParagraph(element: HTMLElement): TiptapNode {
 }
 
 function parseHeading(element: HTMLElement): TiptapNode {
-  const level = parseInt(element.tagName[1], 10);
+  const level = parseInt(element.tagName[1] ?? '', 10);
   const content = parseInlineChildren(element);
   const node: TiptapNode = {
     type: 'heading',
@@ -403,9 +403,7 @@ function parseTable(element: HTMLElement): TiptapNode {
   }
 
   // Also check direct tr children
-  const directRows = Array.from(element.children).filter(
-    (child) => child.tagName.toLowerCase() === 'tr'
-  );
+  const directRows = Array.from(element.children).filter((child) => child.tagName.toLowerCase() === 'tr');
   for (const tr of directRows) {
     const row = parseTableRow(tr as HTMLElement);
     if (row) rows.push(row);

package/lib/shared/richtext/tiptapToHtml.test.ts

@@ -220,11 +220,7 @@ describe('tiptapToHtml', () => {
         content: [
           {
             type: 'paragraph',
-            content: [
-              { type: 'text', text: 'Line 1' },
-              { type: 'hardBreak' },
-              { type: 'text', text: 'Line 2' },
-            ],
+            content: [{ type: 'text', text: 'Line 1' }, { type: 'hardBreak' }, { type: 'text', text: 'Line 2' }],
           },
         ],
       };
@@ -243,7 +239,10 @@ describe('tiptapToHtml', () => {
               {
                 type: 'tableRow',
                 content: [
-                  { type: 'tableHeader', content: [{ type: 'paragraph', content: [{ type: 'text', text: 'Header' }] }] },
+                  {
+                    type: 'tableHeader',
+                    content: [{ type: 'paragraph', content: [{ type: 'text', text: 'Header' }] }],
+                  },
                 ],
               },
               {

package/lib/shared/richtext/types.ts

@@ -76,11 +76,4 @@ export type TiptapNodeType =
 /**
  * Known Tiptap mark types
  */
-export type TiptapMarkType =
-  | 'bold'
-  | 'italic'
-  | 'underline'
-  | 'strike'
-  | 'code'
-  | 'link'
-  | 'textStyle';
+export type TiptapMarkType = 'bold' | 'italic' | 'underline' | 'strike' | 'code' | 'link' | 'textStyle';

package/lib/shared/slugTranslator.test.ts

@@ -251,23 +251,23 @@ describe('slugTranslator', () => {
 
     test('should include correct locale codes', () => {
       const links = getLocaleLinks('/about', 'en', i18nConfig, slugIndex);
-      const codes = links.map(link => link.locale);
+      const codes = links.map((link) => link.locale);
       expect(codes).toEqual(['en', 'pl', 'de']);
     });
 
     test('should include correct native names', () => {
       const links = getLocaleLinks('/about', 'en', i18nConfig, slugIndex);
-      const names = links.map(link => link.nativeName);
+      const names = links.map((link) => link.nativeName);
       expect(names).toEqual(['English', 'Polski', 'Deutsch']);
     });
 
     test('should mark current locale correctly', () => {
       const links = getLocaleLinks('/about', 'pl', i18nConfig, slugIndex);
-      const currentLink = links.find(link => link.locale === 'pl');
+      const currentLink = links.find((link) => link.locale === 'pl');
       expect(currentLink?.isCurrent).toBe(true);
 
-      const otherLinks = links.filter(link => link.locale !== 'pl');
-      otherLinks.forEach(link => {
+      const otherLinks = links.filter((link) => link.locale !== 'pl');
+      otherLinks.forEach((link) => {
         expect(link.isCurrent).toBe(false);
       });
     });
@@ -275,36 +275,36 @@ describe('slugTranslator', () => {
     test('should translate paths correctly for each locale', () => {
       const links = getLocaleLinks('/about', 'en', i18nConfig, slugIndex);
 
-      const enLink = links.find(link => link.locale === 'en');
+      const enLink = links.find((link) => link.locale === 'en');
       expect(enLink?.path).toBe('/about');
 
-      const plLink = links.find(link => link.locale === 'pl');
+      const plLink = links.find((link) => link.locale === 'pl');
       expect(plLink?.path).toBe('/pl/o-nas');
 
-      const deLink = links.find(link => link.locale === 'de');
+      const deLink = links.find((link) => link.locale === 'de');
       expect(deLink?.path).toBe('/de/uber');
     });
 
     test('should handle root path', () => {
       const links = getLocaleLinks('/', 'en', i18nConfig, slugIndex);
 
-      const enLink = links.find(link => link.locale === 'en');
+      const enLink = links.find((link) => link.locale === 'en');
       expect(enLink?.path).toBe('/');
 
-      const plLink = links.find(link => link.locale === 'pl');
+      const plLink = links.find((link) => link.locale === 'pl');
       expect(plLink?.path).toBe('/pl');
 
-      const deLink = links.find(link => link.locale === 'de');
+      const deLink = links.find((link) => link.locale === 'de');
       expect(deLink?.path).toBe('/de');
     });
 
     test('should handle Polish path', () => {
       const links = getLocaleLinks('/pl/o-nas', 'pl', i18nConfig, slugIndex);
 
-      const enLink = links.find(link => link.locale === 'en');
+      const enLink = links.find((link) => link.locale === 'en');
       expect(enLink?.path).toBe('/about');
 
-      const plLink = links.find(link => link.locale === 'pl');
+      const plLink = links.find((link) => link.locale === 'pl');
       expect(plLink?.path).toBe('/pl/o-nas');
     });
   });

package/lib/shared/slugTranslator.ts

@@ -49,7 +49,7 @@ export function buildSlugIndex(mappings: SlugMap[]): Map<string, SlugIndexEntry>
 export function findPageBySlug(
   slug: string,
   locale: string,
-  index: Map<string, SlugIndexEntry>
+  index: Map<string, SlugIndexEntry>,
 ): SlugIndexEntry | undefined {
   const key = `${locale}:${slug}`;
   return index.get(key);
@@ -70,7 +70,7 @@ export function translatePath(
   targetLocale: string,
   currentLocale: string,
   defaultLocale: string,
-  index: Map<string, SlugIndexEntry>
+  index: Map<string, SlugIndexEntry>,
 ): string {
   // Extract slug from current path (remove locale prefix if present)
   let slug = currentPath;
@@ -127,12 +127,12 @@ export function translatePath(
  * Locale link information for locale switcher
  */
 export interface LocaleLink {
-  locale: string;      // Locale code (e.g., "pl")
-  langTag: string;     // BCP 47 language tag (e.g., "pl-PL")
-  name: string;        // English name (e.g., "Polish")
-  nativeName: string;  // Display name in native language (e.g., "Polski")
-  path: string;        // Translated path for this locale
-  isCurrent: boolean;  // Whether this is the current locale
+  locale: string; // Locale code (e.g., "pl")
+  langTag: string; // BCP 47 language tag (e.g., "pl-PL")
+  name: string; // English name (e.g., "Polish")
+  nativeName: string; // Display name in native language (e.g., "Polski")
+  path: string; // Translated path for this locale
+  isCurrent: boolean; // Whether this is the current locale
 }
 
 /**
@@ -149,9 +149,9 @@ export function getLocaleLinks(
   currentPath: string,
   currentLocale: string,
   i18nConfig: I18nConfig,
-  index: Map<string, SlugIndexEntry>
+  index: Map<string, SlugIndexEntry>,
 ): LocaleLink[] {
-  return i18nConfig.locales.map(localeConfig => ({
+  return i18nConfig.locales.map((localeConfig) => ({
     locale: localeConfig.code,
     langTag: localeConfig.langTag,
     name: localeConfig.name,
@@ -173,11 +173,7 @@ export function getLocaleLinks(
  * Used by the slug/page-rename flow to compute the (old, new) href pair
  * for cascade rewrites in incoming links.
  */
-export function buildPageUrlForLocale(
-  slug: string,
-  locale: string,
-  defaultLocale: string,
-): string {
+export function buildPageUrlForLocale(slug: string, locale: string, defaultLocale: string): string {
   const cleanSlug = slug.replace(/^\/+/, '');
   if (locale === defaultLocale) {
     return cleanSlug === '' ? '/' : `/${cleanSlug}`;
@@ -196,7 +192,7 @@ export function buildPageUrlForLocale(
 export function resolveSlugToPageId(
   slug: string,
   locale: string,
-  index: Map<string, SlugIndexEntry>
+  index: Map<string, SlugIndexEntry>,
 ): string | undefined {
   const entry = findPageBySlug(slug, locale, index);
   return entry?.pageId;

package/lib/shared/slugify.ts

@@ -17,13 +17,13 @@ export function slugify(text: string): string {
 
   return text
     .toLowerCase()
-    .normalize('NFD')                    // Decompose accented chars (é -> e + ́)
-    .replace(/[\u0300-\u036f]/g, '')     // Remove diacritics
-    .replace(/[^a-z0-9\s-]/g, '')        // Remove special chars
+    .normalize('NFD') // Decompose accented chars (é -> e + ́)
+    .replace(/[\u0300-\u036f]/g, '') // Remove diacritics
+    .replace(/[^a-z0-9\s-]/g, '') // Remove special chars
     .trim()
-    .replace(/\s+/g, '-')                // Replace spaces with hyphens
-    .replace(/-+/g, '-')                 // Collapse multiple hyphens
-    .replace(/^-|-$/g, '');              // Remove leading/trailing hyphens
+    .replace(/\s+/g, '-') // Replace spaces with hyphens
+    .replace(/-+/g, '-') // Collapse multiple hyphens
+    .replace(/^-|-$/g, ''); // Remove leading/trailing hyphens
 }
 
 /**
@@ -34,10 +34,7 @@ export function slugify(text: string): string {
  * @param defaultLocale - Locale to use for i18n extraction (default: 'en')
  * @returns Extracted string value
  */
-export function extractStringValue(
-  value: unknown,
-  defaultLocale: string = 'en'
-): string {
+export function extractStringValue(value: unknown, defaultLocale: string = 'en'): string {
   // Direct string
   if (typeof value === 'string') {
     return value;
@@ -69,10 +66,7 @@ export function extractStringValue(
  * @param existingFilenames - Array of existing filenames to check against
  * @returns Unique filename
  */
-export function generateUniqueFilename(
-  baseSlug: string,
-  existingFilenames: string[]
-): string {
+export function generateUniqueFilename(baseSlug: string, existingFilenames: string[]): string {
   if (!baseSlug) {
     baseSlug = 'untitled';
   }
@@ -107,7 +101,7 @@ export function generateFilenameFromItem(
   item: Record<string, unknown>,
   titleField: string = 'title',
   existingFilenames: string[] = [],
-  defaultLocale: string = 'en'
+  defaultLocale: string = 'en',
 ): string {
   // Try to get title from specified field
   const titleValue = item[titleField];

package/lib/shared/styleNodeUtils.test.ts

@@ -52,7 +52,7 @@ describe('styleNodeUtils', () => {
         type: NODE_TYPE.NODE,
         tag: 'div',
         children: [],
-        style: { color: 'blue' }
+        style: { color: 'blue' },
       };
       const instanceStyles = { fontSize: '16px' };
       const result = mergeNodeStyles(node, instanceStyles);
@@ -63,7 +63,7 @@ describe('styleNodeUtils', () => {
       const node: ComponentInstanceNode = {
         type: NODE_TYPE.COMPONENT,
         component: 'Button',
-        props: { style: { color: 'blue' } }
+        props: { style: { color: 'blue' } },
       };
       const instanceStyles = { padding: '10px' };
       const result = mergeNodeStyles(node, instanceStyles);
@@ -75,7 +75,7 @@ describe('styleNodeUtils', () => {
         type: NODE_TYPE.NODE,
         tag: 'div',
         children: [],
-        style: { color: 'blue', fontSize: '14px' }
+        style: { color: 'blue', fontSize: '14px' },
       };
       const instanceStyles = { fontSize: '16px' };
       const result = mergeNodeStyles(node, instanceStyles);
@@ -100,7 +100,7 @@ describe('styleNodeUtils', () => {
         type: NODE_TYPE.LINK,
         href: '/foo',
         children: [],
-        style: { color: '#000000', backgroundColor: '#ffffff' }
+        style: { color: '#000000', backgroundColor: '#ffffff' },
       };
       const instanceStyles = { backgroundColor: '#252525', color: '#ffffff' };
       const result = mergeNodeStyles(node, instanceStyles);
@@ -112,7 +112,7 @@ describe('styleNodeUtils', () => {
         type: NODE_TYPE.LINK,
         href: '/foo',
         children: [],
-        style: { base: { color: '#000', padding: '8px' }, tablet: { fontSize: '14px' }, mobile: {} }
+        style: { base: { color: '#000', padding: '8px' }, tablet: { fontSize: '14px' }, mobile: {} },
       };
       const instanceStyles = { base: { backgroundColor: '#252525', color: '#fff' }, tablet: {}, mobile: {} };
       const result = mergeNodeStyles(node, instanceStyles);
@@ -130,7 +130,7 @@ describe('styleNodeUtils', () => {
         type: NODE_TYPE.NODE,
         tag: 'div',
         children: [],
-        style: { color: 'red', fontSize: '16px' }
+        style: { color: 'red', fontSize: '16px' },
       };
       const styles = extractStylesFromNode(node);
       expect(styles).toEqual({ color: 'red', fontSize: '16px' });
@@ -140,7 +140,7 @@ describe('styleNodeUtils', () => {
       const node: ComponentInstanceNode = {
         type: NODE_TYPE.COMPONENT,
         component: 'Button',
-        props: { style: { padding: '10px' } }
+        props: { style: { padding: '10px' } },
       };
       const styles = extractStylesFromNode(node);
       expect(styles).toEqual({ padding: '10px' });
@@ -169,7 +169,7 @@ describe('styleNodeUtils', () => {
         type: NODE_TYPE.LINK,
         href: '/foo',
         children: [],
-        style: { backgroundColor: '#252525' }
+        style: { backgroundColor: '#252525' },
       };
       const styles = extractStylesFromNode(node);
       expect(styles).toEqual({ backgroundColor: '#252525' });

package/lib/shared/styleNodeUtils.ts

@@ -3,7 +3,7 @@
  * Provides helper functions for applying and merging styles based on node type
  */
 
-import type { ComponentNode, HtmlNode, ComponentInstanceNode, StyleValue, ResponsiveStyleObject } from './types';
+import type { ComponentNode, StyleValue } from './types';
 import { isComponentNode, isHtmlNode, isEmbedNode, isListNode, isLinkNode } from './nodeUtils';
 import { isResponsiveStyle } from './styleUtils';
 
@@ -15,7 +15,7 @@ import { isResponsiveStyle } from './styleUtils';
 export function applyStylesToNode(
   node: ComponentNode,
   styles: StyleValue | Record<string, string | number> | null | undefined,
-  viewportWidth?: number
+  _viewportWidth?: number,
 ): ComponentNode {
   if (!styles) return node;
 
@@ -44,31 +44,33 @@ export function applyStylesToNode(
  * exporter can replicate SSR's instance-into-root style merge at build time.
  */
 export function deepMergeStyles(
-  existing: StyleValue | Record<string, any>,
-  instance: StyleValue | Record<string, any> | null | undefined
+  existing: StyleValue | Record<string, unknown>,
+  instance: StyleValue | Record<string, unknown> | null | undefined,
 ): StyleValue {
   if (!instance) return existing as StyleValue;
 
-  const bothResponsive =
-    isResponsiveStyle(existing as StyleValue) && isResponsiveStyle(instance as StyleValue);
+  const bothResponsive = isResponsiveStyle(existing as StyleValue) && isResponsiveStyle(instance as StyleValue);
 
   if (bothResponsive) {
-    const e = existing as Record<string, any>;
-    const i = instance as Record<string, any>;
-    const merged: Record<string, any> = {};
+    const e = existing as Record<string, unknown>;
+    const i = instance as Record<string, unknown>;
+    const merged: Record<string, unknown> = {};
     const keys = new Set([...Object.keys(e), ...Object.keys(i)]);
     for (const key of keys) {
-      if (e[key] && i[key] && typeof e[key] === 'object' && typeof i[key] === 'object') {
-        merged[key] = { ...e[key], ...i[key] };
+      // Capture the indexed values so `typeof` narrows them for the object spread.
+      const ev = e[key];
+      const iv = i[key];
+      if (ev && iv && typeof ev === 'object' && typeof iv === 'object') {
+        merged[key] = { ...(ev as object), ...(iv as object) };
       } else {
-        merged[key] = i[key] !== undefined ? i[key] : e[key];
+        merged[key] = iv !== undefined ? iv : ev;
       }
     }
     return merged as StyleValue;
   }
 
   // Flat styles: shallow merge (instance overrides)
-  return { ...(existing as Record<string, any>), ...(instance as Record<string, any>) } as StyleValue;
+  return { ...(existing as Record<string, unknown>), ...(instance as Record<string, unknown>) } as StyleValue;
 }
 
 /**
@@ -78,7 +80,7 @@ export function deepMergeStyles(
 export function mergeNodeStyles(
   node: ComponentNode,
   instanceStyles: StyleValue | Record<string, string | number> | null | undefined,
-  viewportWidth?: number
+  _viewportWidth?: number,
 ): ComponentNode {
   if (!instanceStyles) return node;
 
@@ -86,7 +88,7 @@ export function mergeNodeStyles(
     // For HTML nodes, Embed nodes, List nodes, and Link nodes: merge instance styles with existing top-level styles
     const existingStyle = node.style;
     if (existingStyle && typeof existingStyle === 'object') {
-      node.style = deepMergeStyles(existingStyle, instanceStyles);
+      node.style = deepMergeStyles(existingStyle as Record<string, unknown>, instanceStyles);
     } else {
       node.style = instanceStyles as StyleValue;
     }
@@ -110,10 +112,7 @@ export function mergeNodeStyles(
  * Extract styles from a ComponentNode based on its type
  * Returns a flat style object ready for React props
  */
-export function extractStylesFromNode(
-  node: ComponentNode,
-  viewportWidth?: number
-): Record<string, string | number> {
+export function extractStylesFromNode(node: ComponentNode, _viewportWidth?: number): Record<string, string | number> {
   if (isComponentNode(node)) {
     // Component instance: styles are in props.style
     const style = node.props?.style;
@@ -130,4 +129,3 @@ export function extractStylesFromNode(
 
   return {};
 }
-

package/lib/shared/styleUtils.test.ts

@@ -21,11 +21,13 @@ describe('styleUtils', () => {
     });
 
     test('should return true for object with multiple responsive keys', () => {
-      expect(isResponsiveStyle({
-        base: { color: 'red' },
-        tablet: { color: 'blue' },
-        mobile: { color: 'green' },
-      })).toBe(true);
+      expect(
+        isResponsiveStyle({
+          base: { color: 'red' },
+          tablet: { color: 'blue' },
+          mobile: { color: 'green' },
+        }),
+      ).toBe(true);
     });
 
     test('should return false for regular style object', () => {
@@ -58,10 +60,12 @@ describe('styleUtils', () => {
     });
 
     test('should handle object with base key and other properties', () => {
-      expect(isResponsiveStyle({
-        base: { color: 'red' },
-        customProp: 'value',
-      } as any)).toBe(true);
+      expect(
+        isResponsiveStyle({
+          base: { color: 'red' },
+          customProp: 'value',
+        } as any),
+      ).toBe(true);
     });
   });
 
@@ -71,41 +75,51 @@ describe('styleUtils', () => {
     });
 
     test('should return true for style object with string values', () => {
-      expect(isStyleObject({
-        color: 'red',
-        backgroundColor: 'blue',
-        padding: '10px',
-      })).toBe(true);
+      expect(
+        isStyleObject({
+          color: 'red',
+          backgroundColor: 'blue',
+          padding: '10px',
+        }),
+      ).toBe(true);
     });
 
     test('should return true for style object with number values', () => {
-      expect(isStyleObject({
-        width: 100,
-        height: 200,
-        opacity: 0.5,
-      })).toBe(true);
+      expect(
+        isStyleObject({
+          width: 100,
+          height: 200,
+          opacity: 0.5,
+        }),
+      ).toBe(true);
     });
 
     test('should return true for style object with mixed string and number', () => {
-      expect(isStyleObject({
-        color: 'red',
-        width: 100,
-        fontSize: '16px',
-        opacity: 0.8,
-      })).toBe(true);
+      expect(
+        isStyleObject({
+          color: 'red',
+          width: 100,
+          fontSize: '16px',
+          opacity: 0.8,
+        }),
+      ).toBe(true);
     });
 
     test('should return true for style object with _mapping property', () => {
-      expect(isStyleObject({
-        color: { _mapping: 'theme.colors.primary' },
-      })).toBe(true);
+      expect(
+        isStyleObject({
+          color: { _mapping: 'theme.colors.primary' },
+        }),
+      ).toBe(true);
     });
 
     test('should return false for responsive style object', () => {
-      expect(isStyleObject({
-        base: { color: 'red' },
-        tablet: { color: 'blue' },
-      })).toBe(false);
+      expect(
+        isStyleObject({
+          base: { color: 'red' },
+          tablet: { color: 'blue' },
+        }),
+      ).toBe(false);
     });
 
     test('should return false for object with only base key', () => {
@@ -138,25 +152,31 @@ describe('styleUtils', () => {
     });
 
     test('should return false for object with invalid value types', () => {
-      expect(isStyleObject({
-        color: 'red',
-        invalid: [],
-      })).toBe(false);
+      expect(
+        isStyleObject({
+          color: 'red',
+          invalid: [],
+        }),
+      ).toBe(false);
     });
 
     test('should return false for object with nested objects without _mapping', () => {
-      expect(isStyleObject({
-        color: 'red',
-        nested: { prop: 'value' },
-      })).toBe(false);
+      expect(
+        isStyleObject({
+          color: 'red',
+          nested: { prop: 'value' },
+        }),
+      ).toBe(false);
     });
 
     test('should handle complex StyleMapping objects', () => {
-      expect(isStyleObject({
-        color: { _mapping: 'colors.primary' },
-        fontSize: '16px',
-        width: 100,
-      })).toBe(true);
+      expect(
+        isStyleObject({
+          color: { _mapping: 'colors.primary' },
+          fontSize: '16px',
+          width: 100,
+        }),
+      ).toBe(true);
     });
   });
 
@@ -166,10 +186,12 @@ describe('styleUtils', () => {
     });
 
     test('should return true for responsive style object', () => {
-      expect(isStyleValue({
-        base: { color: 'red' },
-        tablet: { color: 'blue' },
-      })).toBe(true);
+      expect(
+        isStyleValue({
+          base: { color: 'red' },
+          tablet: { color: 'blue' },
+        }),
+      ).toBe(true);
     });
 
     test('should return true for object with base key', () => {
@@ -218,21 +240,25 @@ describe('styleUtils', () => {
     });
 
     test('should handle complex responsive styles', () => {
-      expect(isStyleValue({
-        base: { color: 'red', fontSize: '18px' },
-        tablet: { color: 'blue', fontSize: '16px' },
-        mobile: { color: 'green', fontSize: '14px' },
-      })).toBe(true);
+      expect(
+        isStyleValue({
+          base: { color: 'red', fontSize: '18px' },
+          tablet: { color: 'blue', fontSize: '16px' },
+          mobile: { color: 'green', fontSize: '14px' },
+        }),
+      ).toBe(true);
     });
 
     test('should handle complex style objects', () => {
-      expect(isStyleValue({
-        color: 'red',
-        fontSize: '16px',
-        width: 100,
-        height: 200,
-        opacity: 0.8,
-      })).toBe(true);
+      expect(
+        isStyleValue({
+          color: 'red',
+          fontSize: '16px',
+          width: 100,
+          height: 200,
+          opacity: 0.8,
+        }),
+      ).toBe(true);
     });
   });
 });