@jjlmoya/utils-developer 1.30.0 → 1.31.0

package/src/tests/inverted_punctuation.test.ts

@@ -1,113 +1,113 @@
-import { describe, expect, it } from 'vitest';
-import { ALL_TOOLS } from '../tools';
-
-const INVERTED_PUNCTUATION_LOCALES = {
-  es: {
-    language: 'Spanish',
-    questionStart: '¿',
-    questionEnd: '?',
-    exclamationStart: '¡',
-    exclamationEnd: '!',
-  },
-} as const;
-
-type InvertedPunctuationLocale = keyof typeof INVERTED_PUNCTUATION_LOCALES;
-
-const TRANSLATABLE_KEYS = ['title', 'description', 'ui', 'seo', 'faq', 'howTo'] as const;
-
-function collectStrings(value: unknown): string[] {
-  if (typeof value === 'string') return [value];
-  if (!value || typeof value !== 'object') return [];
-  if (Array.isArray(value)) return value.flatMap(collectStrings);
-  return Object.values(value).flatMap(collectStrings);
-}
-
-function translatableStrings(content: Record<string, unknown>): string[] {
-  return TRANSLATABLE_KEYS.flatMap((key) => collectStrings(content[key]));
-}
-
-function sentenceStart(text: string, endIndex: number): string {
-  const beforeMark = text.slice(0, endIndex).trimEnd();
-
-  const boundaries = [
-    ...[...beforeMark.matchAll(/\.(?=\s|$)/g)].map((m) => m.index ?? 0),
-    ...[...beforeMark.matchAll(/:(?=\s|$)/g)].map((m) => m.index ?? 0),
-    ...[...beforeMark.matchAll(/;(?=\s|$)/g)].map((m) => m.index ?? 0),
-  ];
-
-  const lastBoundary = boundaries.length > 0
-    ? Math.max(...boundaries)
-    : -1;
-
-  const nlBoundary = beforeMark.lastIndexOf('\n');
-  const boundary = Math.max(lastBoundary, nlBoundary);
-
-  return beforeMark.slice(boundary + 1).trimStart();
-}
-
-function isCodeContext(text: string, index: number, mark: string): boolean {
-  if (mark === '!') {
-    const after = text.slice(index + 1, index + 10);
-    return /^important\b/.test(after);
-  }
-  if (mark === '?') {
-    const before = text.slice(Math.max(0, index - 2), index);
-    return /[,#\/&+=]/.test(before) || / y $/.test(text.slice(0, index));
-  }
-  return false;
-}
-
-function findMissingInvertedMarks(
-  text: string,
-  startMark: string,
-  endMark: string,
-  _toolId: string,
-): string[] {
-  const cleaned = text.replace(/<code>.*?<\/code>/g, '');
-
-  return [...cleaned.matchAll(new RegExp(`\\${endMark}`, 'g'))]
-    .filter((match) => !isCodeContext(cleaned, match.index ?? 0, endMark))
-    .map((match) => sentenceStart(cleaned, match.index ?? 0))
-    .filter((segment) => {
-      if (segment.length === 0) return false;
-      if (segment.includes(startMark)) return false;
-      if (endMark === '?' && /:\/\//.test(segment)) return false;
-      return true;
-    })
-    .map((segment) => `${segment}${endMark}`);
-}
-
-describe('Inverted punctuation validation', () => {
-  ALL_TOOLS.forEach((tool) => {
-    describe(`Tool: ${tool.entry.id}`, () => {
-      Object.keys(INVERTED_PUNCTUATION_LOCALES).forEach((locale) => {
-        it(`${locale} uses opening punctuation marks for questions and exclamations`, async () => {
-          const typedLocale = locale as InvertedPunctuationLocale;
-          const loader = tool.entry.i18n[typedLocale];
-          if (!loader) return;
-
-          const rule = INVERTED_PUNCTUATION_LOCALES[typedLocale];
-          const content = await loader();
-          const strings = translatableStrings(content as Record<string, unknown>);
-          const toolId = tool.entry.id;
-          const missingQuestions = strings.flatMap((text) =>
-            findMissingInvertedMarks(text, rule.questionStart, rule.questionEnd, toolId)
-          );
-          const missingExclamations = strings.flatMap((text) =>
-            findMissingInvertedMarks(text, rule.exclamationStart, rule.exclamationEnd, toolId)
-          );
-          const failures = [...missingQuestions, ...missingExclamations];
-
-          expect(
-            failures,
-            [
-              `Missing opening punctuation marks in ${tool.entry.id}/${typedLocale} (${rule.language}).`,
-              `Questions must use ${rule.questionStart}...${rule.questionEnd} and exclamations must use ${rule.exclamationStart}...${rule.exclamationEnd}.`,
-              `Examples: ${failures.slice(0, 5).join(' | ')}`,
-            ].join(' '),
-          ).toHaveLength(0);
-        });
-      });
-    });
-  });
-});
+import { describe, expect, it } from 'vitest';
+import { ALL_TOOLS } from '../tools';
+
+const INVERTED_PUNCTUATION_LOCALES = {
+  es: {
+    language: 'Spanish',
+    questionStart: '¿',
+    questionEnd: '?',
+    exclamationStart: '¡',
+    exclamationEnd: '!',
+  },
+} as const;
+
+type InvertedPunctuationLocale = keyof typeof INVERTED_PUNCTUATION_LOCALES;
+
+const TRANSLATABLE_KEYS = ['title', 'description', 'ui', 'seo', 'faq', 'howTo'] as const;
+
+function collectStrings(value: unknown): string[] {
+  if (typeof value === 'string') return [value];
+  if (!value || typeof value !== 'object') return [];
+  if (Array.isArray(value)) return value.flatMap(collectStrings);
+  return Object.values(value).flatMap(collectStrings);
+}
+
+function translatableStrings(content: Record<string, unknown>): string[] {
+  return TRANSLATABLE_KEYS.flatMap((key) => collectStrings(content[key]));
+}
+
+function sentenceStart(text: string, endIndex: number): string {
+  const beforeMark = text.slice(0, endIndex).trimEnd();
+
+  const boundaries = [
+    ...[...beforeMark.matchAll(/\.(?=\s|$)/g)].map((m) => m.index ?? 0),
+    ...[...beforeMark.matchAll(/:(?=\s|$)/g)].map((m) => m.index ?? 0),
+    ...[...beforeMark.matchAll(/;(?=\s|$)/g)].map((m) => m.index ?? 0),
+  ];
+
+  const lastBoundary = boundaries.length > 0
+    ? Math.max(...boundaries)
+    : -1;
+
+  const nlBoundary = beforeMark.lastIndexOf('\n');
+  const boundary = Math.max(lastBoundary, nlBoundary);
+
+  return beforeMark.slice(boundary + 1).trimStart();
+}
+
+function isCodeContext(text: string, index: number, mark: string): boolean {
+  if (mark === '!') {
+    const after = text.slice(index + 1, index + 10);
+    return /^important\b/.test(after);
+  }
+  if (mark === '?') {
+    const before = text.slice(Math.max(0, index - 2), index);
+    return /[,#\/&+=]/.test(before) || / y $/.test(text.slice(0, index));
+  }
+  return false;
+}
+
+function findMissingInvertedMarks(
+  text: string,
+  startMark: string,
+  endMark: string,
+  _toolId: string,
+): string[] {
+  const cleaned = text.replace(/<code>.*?<\/code>/g, '');
+
+  return [...cleaned.matchAll(new RegExp(`\\${endMark}`, 'g'))]
+    .filter((match) => !isCodeContext(cleaned, match.index ?? 0, endMark))
+    .map((match) => sentenceStart(cleaned, match.index ?? 0))
+    .filter((segment) => {
+      if (segment.length === 0) return false;
+      if (segment.includes(startMark)) return false;
+      if (endMark === '?' && /:\/\//.test(segment)) return false;
+      return true;
+    })
+    .map((segment) => `${segment}${endMark}`);
+}
+
+describe('Inverted punctuation validation', () => {
+  ALL_TOOLS.forEach((tool) => {
+    describe(`Tool: ${tool.entry.id}`, () => {
+      Object.keys(INVERTED_PUNCTUATION_LOCALES).forEach((locale) => {
+        it(`${locale} uses opening punctuation marks for questions and exclamations`, async () => {
+          const typedLocale = locale as InvertedPunctuationLocale;
+          const loader = tool.entry.i18n[typedLocale];
+          if (!loader) return;
+
+          const rule = INVERTED_PUNCTUATION_LOCALES[typedLocale];
+          const content = await loader();
+          const strings = translatableStrings(content as Record<string, unknown>);
+          const toolId = tool.entry.id;
+          const missingQuestions = strings.flatMap((text) =>
+            findMissingInvertedMarks(text, rule.questionStart, rule.questionEnd, toolId)
+          );
+          const missingExclamations = strings.flatMap((text) =>
+            findMissingInvertedMarks(text, rule.exclamationStart, rule.exclamationEnd, toolId)
+          );
+          const failures = [...missingQuestions, ...missingExclamations];
+
+          expect(
+            failures,
+            [
+              `Missing opening punctuation marks in ${tool.entry.id}/${typedLocale} (${rule.language}).`,
+              `Questions must use ${rule.questionStart}...${rule.questionEnd} and exclamations must use ${rule.exclamationStart}...${rule.exclamationEnd}.`,
+              `Examples: ${failures.slice(0, 5).join(' | ')}`,
+            ].join(' '),
+          ).toHaveLength(0);
+        });
+      });
+    });
+  });
+});

package/src/tests/mocks/astro_mock.js

@@ -1,2 +1,2 @@
 export default function () { return null; }
-
+

package/src/tests/no_h1_in_components.test.ts

@@ -45,4 +45,4 @@ describe('No H1 in Components', () => {
     });
   });
 });
-
+

package/src/tests/promote_this_website_i18n.test.ts

@@ -0,0 +1,33 @@
+import { describe, expect, it } from 'vitest';
+import { promoteThisWebsite } from '../tool/promoteThisWebsite/entry';
+
+const locales = [
+  'de', 'en', 'es', 'fr', 'id', 'it', 'ja', 'ko', 'nl', 'pl', 'pt', 'ru', 'sv', 'tr', 'zh',
+];
+
+describe('promote-this-website translations', () => {
+  it('provides every visible UI key in every locale', async () => {
+    const english = await promoteThisWebsite.i18n.en!();
+    const expectedKeys = Object.keys(english.ui).sort();
+
+    for (const locale of locales) {
+      const content = await promoteThisWebsite.i18n[locale as keyof typeof promoteThisWebsite.i18n]!();
+      expect(Object.keys(content.ui).sort(), `${locale} has an incomplete UI translation`).toEqual(expectedKeys);
+      expect(content.faq).toHaveLength(english.faq.length);
+      expect(content.howTo).toHaveLength(english.howTo.length);
+      expect(content.seo).toHaveLength(english.seo.length);
+    }
+  });
+
+  it('does not silently use English for the core translated controls', async () => {
+    const english = await promoteThisWebsite.i18n.en!();
+    const translatedControls = ['urlLabel', 'applyUrl', 'titleLabel', 'download', 'reset', 'titleStyleLabel', 'brandStyleLabel'];
+
+    for (const locale of locales.filter((locale) => locale !== 'en')) {
+      const content = await promoteThisWebsite.i18n[locale as keyof typeof promoteThisWebsite.i18n]!();
+      for (const key of translatedControls) {
+        expect(content.ui[key], `${locale}.${key} is still English`).not.toBe(english.ui[key]);
+      }
+    }
+  });
+});

package/src/tests/slug_language_code_format.test.ts

@@ -1,25 +1,25 @@
-import { describe, it, expect } from 'vitest';
-import { ALL_TOOLS } from '../tools';
-import type { ToolLocaleContent } from '../types';
-
-describe('Slug Language Code Format Validation', () => {
-
-
-  ALL_TOOLS.forEach((tool) => {
-    describe(`Tool: ${tool.entry.id}`, () => {
-      it('slug should not end with 2-letter language codes like -ja, -ru, -ko', async () => {
-        const locales = Object.keys(tool.entry.i18n);
-
-        for (const locale of locales) {
-          const loader = tool.entry.i18n[locale as keyof typeof tool.entry.i18n];
-          const content = (await loader?.()) as ToolLocaleContent;
-
-          expect(
-            content.slug,
-            `Tool "${tool.entry.id}" locale "${locale}" slug ("${content.slug}") cannot end with a 2-letter language code (e.g., -ja, -ru, -ko).`,
-          ).not.toMatch(/-[a-z]{2}$/);
-        }
-      });
-    });
-  });
-});
+import { describe, it, expect } from 'vitest';
+import { ALL_TOOLS } from '../tools';
+import type { ToolLocaleContent } from '../types';
+
+describe('Slug Language Code Format Validation', () => {
+
+
+  ALL_TOOLS.forEach((tool) => {
+    describe(`Tool: ${tool.entry.id}`, () => {
+      it('slug should not end with 2-letter language codes like -ja, -ru, -ko', async () => {
+        const locales = Object.keys(tool.entry.i18n);
+
+        for (const locale of locales) {
+          const loader = tool.entry.i18n[locale as keyof typeof tool.entry.i18n];
+          const content = (await loader?.()) as ToolLocaleContent;
+
+          expect(
+            content.slug,
+            `Tool "${tool.entry.id}" locale "${locale}" slug ("${content.slug}") cannot end with a 2-letter language code (e.g., -ja, -ru, -ko).`,
+          ).not.toMatch(/-[a-z]{2}$/);
+        }
+      });
+    });
+  });
+});

package/src/tests/slug_uniqueness.test.ts

@@ -1,87 +1,87 @@
-import { describe, it, expect } from 'vitest';
-import { ALL_TOOLS } from '../tools';
-import type { ToolLocaleContent } from '../types';
-
-const sharingLocales = ['ja', 'ko', 'zh'];
-const toolsWithSharedSpanishSlug = ['cron-generator', 'json-formatter', 'keycode'];
-
-interface ValidateParams {
-  toolId: string;
-  locale: string;
-  content: ToolLocaleContent;
-  enSlug: string;
-  slugs: Map<string, string>;
-}
-
-const validateLocaleSlug = ({
-  toolId,
-  locale,
-  content,
-  enSlug,
-  slugs,
-}: ValidateParams) => {
-  expect(
-    content.slug,
-    `Tool "${toolId}" locale "${locale}" has an invalid slug ("${content.slug}"). Slugs must be transliterated (only a-z, 0-9, and -).`,
-  ).toMatch(/^[a-z0-9-]+$/);
-
-  if (locale === 'en') {
-    return;
-  }
-
-  if (sharingLocales.includes(locale)) {
-    expect(
-      content.slug,
-      `Tool "${toolId}" locale "${locale}" must use the same slug as "en" ("${enSlug}").`,
-    ).toBe(enSlug);
-  } else if (locale === 'es' && toolsWithSharedSpanishSlug.includes(toolId)) {
-    expect(
-      content.slug,
-      `Tool "${toolId}" locale "${locale}" must use the same slug as "en" ("${enSlug}").`,
-    ).toBe(enSlug);
-  } else {
-    expect(
-      content.slug,
-      `Tool "${toolId}" locale "${locale}" has the same slug as "en" ("${enSlug}"). Cada slug tiene que estar en su propia idioma`,
-    ).not.toBe(enSlug);
-
-    if (slugs.has(content.slug)) {
-      const previousLocale = slugs.get(content.slug);
-      expect(
-        false,
-        `Tool "${toolId}" locales "${locale}" and "${previousLocale}" share the same slug ("${content.slug}"). Cada slug tiene que estar en su propia idioma`,
-      ).toBe(true);
-    }
-    slugs.set(content.slug, locale);
-  }
-};
-
-describe('Slug Localization and Uniqueness Validation', () => {
-  ALL_TOOLS.forEach((tool) => {
-    describe(`Tool: ${tool.entry.id}`, () => {
-      it('every locale should have a unique, translated slug', async () => {
-        const slugs = new Map<string, string>();
-        const locales = Object.keys(tool.entry.i18n);
-
-        let enSlug = '';
-        if (locales.includes('en')) {
-          const enLoader = tool.entry.i18n['en' as keyof typeof tool.entry.i18n];
-          const enContent = (await enLoader?.()) as ToolLocaleContent;
-          enSlug = enContent.slug;
-        }
-
-        for (const locale of locales) {
-          const loader = tool.entry.i18n[locale as keyof typeof tool.entry.i18n];
-          const content = (await loader?.()) as ToolLocaleContent;
-          validateLocaleSlug({
-            toolId: tool.entry.id,
-            locale,
-            content,
-            enSlug,
-            slugs,
-          });
-        }
-      });
-    });
-  });
+import { describe, it, expect } from 'vitest';
+import { ALL_TOOLS } from '../tools';
+import type { ToolLocaleContent } from '../types';
+
+const sharingLocales = ['ja', 'ko', 'zh'];
+const toolsWithSharedSpanishSlug = ['cron-generator', 'json-formatter', 'keycode'];
+
+interface ValidateParams {
+  toolId: string;
+  locale: string;
+  content: ToolLocaleContent;
+  enSlug: string;
+  slugs: Map<string, string>;
+}
+
+const validateLocaleSlug = ({
+  toolId,
+  locale,
+  content,
+  enSlug,
+  slugs,
+}: ValidateParams) => {
+  expect(
+    content.slug,
+    `Tool "${toolId}" locale "${locale}" has an invalid slug ("${content.slug}"). Slugs must be transliterated (only a-z, 0-9, and -).`,
+  ).toMatch(/^[a-z0-9-]+$/);
+
+  if (locale === 'en') {
+    return;
+  }
+
+  if (sharingLocales.includes(locale)) {
+    expect(
+      content.slug,
+      `Tool "${toolId}" locale "${locale}" must use the same slug as "en" ("${enSlug}").`,
+    ).toBe(enSlug);
+  } else if (locale === 'es' && toolsWithSharedSpanishSlug.includes(toolId)) {
+    expect(
+      content.slug,
+      `Tool "${toolId}" locale "${locale}" must use the same slug as "en" ("${enSlug}").`,
+    ).toBe(enSlug);
+  } else {
+    expect(
+      content.slug,
+      `Tool "${toolId}" locale "${locale}" has the same slug as "en" ("${enSlug}"). Cada slug tiene que estar en su propia idioma`,
+    ).not.toBe(enSlug);
+
+    if (slugs.has(content.slug)) {
+      const previousLocale = slugs.get(content.slug);
+      expect(
+        false,
+        `Tool "${toolId}" locales "${locale}" and "${previousLocale}" share the same slug ("${content.slug}"). Cada slug tiene que estar en su propia idioma`,
+      ).toBe(true);
+    }
+    slugs.set(content.slug, locale);
+  }
+};
+
+describe('Slug Localization and Uniqueness Validation', () => {
+  ALL_TOOLS.forEach((tool) => {
+    describe(`Tool: ${tool.entry.id}`, () => {
+      it('every locale should have a unique, translated slug', async () => {
+        const slugs = new Map<string, string>();
+        const locales = Object.keys(tool.entry.i18n);
+
+        let enSlug = '';
+        if (locales.includes('en')) {
+          const enLoader = tool.entry.i18n['en' as keyof typeof tool.entry.i18n];
+          const enContent = (await enLoader?.()) as ToolLocaleContent;
+          enSlug = enContent.slug;
+        }
+
+        for (const locale of locales) {
+          const loader = tool.entry.i18n[locale as keyof typeof tool.entry.i18n];
+          const content = (await loader?.()) as ToolLocaleContent;
+          validateLocaleSlug({
+            toolId: tool.entry.id,
+            locale,
+            content,
+            enSlug,
+            slugs,
+          });
+        }
+      });
+    });
+  });
 });

package/src/tests/tool_validation.test.ts

@@ -1,17 +1,17 @@
-import { describe, it, expect } from 'vitest';
-import { ALL_TOOLS } from '../tools';
-import { developerCategory } from '../data';
-
-describe('Tool Validation Suite', () => {
-  describe('Library Registration', () => {
-    it('should have 28 tools in ALL_TOOLS', () => {
+import { describe, it, expect } from 'vitest';
+import { ALL_TOOLS } from '../tools';
+import { developerCategory } from '../data';
+
+describe('Tool Validation Suite', () => {
+  describe('Library Registration', () => {
+    it('should have 28 tools in ALL_TOOLS', () => {
       expect(ALL_TOOLS.length).toBe(29);
     });
-
-    it('developerCategory should be defined', () => {
-      expect(developerCategory).toBeDefined();
-      expect(developerCategory.i18n).toBeDefined();
-    });
-  });
-});
-
+
+    it('developerCategory should be defined', () => {
+      expect(developerCategory).toBeDefined();
+      expect(developerCategory.i18n).toBeDefined();
+    });
+  });
+});
+