browser-extension-manager 1.2.12 → 1.2.13

package/dist/gulp/config/locales.js

@@ -0,0 +1,30 @@
+// Locale field limits (Chrome Web Store requirements)
+// https://developer.chrome.com/docs/webstore/i18n
+module.exports = {
+  // Field character limits
+  limits: {
+    appName: 50,
+    appNameShort: 25,
+    appDescription: 200,
+  },
+
+  // Languages to translate (code: name)
+  languages: {
+    zh: 'Chinese (Simplified)',
+    es: 'Spanish',
+    hi: 'Hindi',
+    ar: 'Arabic',
+    pt: 'Portuguese',
+    ru: 'Russian',
+    ja: 'Japanese',
+    de: 'German',
+    fr: 'French',
+    ko: 'Korean',
+    ur: 'Urdu',
+    id: 'Indonesian',
+    bn: 'Bengali',
+    tl: 'Tagalog/Filipino',
+    vi: 'Vietnamese',
+    it: 'Italian',
+  },
+};

package/dist/gulp/tasks/audit.js

@@ -15,6 +15,7 @@ const rootPathProject = Manager.getRootPath('project');
 // Audit results tracker
 const auditResults = {
   externalScripts: [],
+  localeWarnings: [],
 };
 
 // Patterns to detect external script references
@@ -38,6 +39,44 @@ const EXTERNAL_SCRIPT_PATTERNS = [
   /["'](https?:\/\/(?:www\.)?(?:google\.com\/recaptcha|apis\.google\.com)[^"']*)["']/gi,
 ];
 
+// Locale config (shared with translate.js)
+const { limits: LOCALE_LIMITS } = require('../config/locales.js');
+
+// Check locale files for warnings
+function checkLocaleFiles(packagedDir) {
+  const localesDir = path.join(packagedDir, '_locales');
+
+  if (!jetpack.exists(localesDir)) {
+    return;
+  }
+
+  const localeFiles = jetpack.find(localesDir, { matching: '*/messages.json' });
+
+  localeFiles.forEach(filePath => {
+    try {
+      const content = jetpack.read(filePath);
+      const messages = JSON.parse(content);
+
+      // Check each field against its limit
+      Object.entries(LOCALE_LIMITS).forEach(([field, limit]) => {
+        const message = messages[field]?.message;
+
+        if (message && message.length > limit) {
+          auditResults.localeWarnings.push({
+            file: path.relative(rootPathProject, filePath),
+            field: field,
+            length: message.length,
+            limit: limit,
+            value: message,
+          });
+        }
+      });
+    } catch (e) {
+      logger.warn(`Error parsing locale file ${filePath}: ${e.message}`);
+    }
+  });
+}
+
 // Check a single file for external script references
 function checkFileForExternalScripts(filePath) {
   try {
@@ -86,6 +125,7 @@ async function auditFn(complete) {
 
   // Reset results
   auditResults.externalScripts = [];
+  auditResults.localeWarnings = [];
 
   try {
     // Find all files in packaged directory (JS, HTML, etc.)
@@ -100,7 +140,7 @@ async function auditFn(complete) {
 
     logger.log(`Auditing ${files.length} files (JS, HTML)...`);
 
-    // Check each file
+    // Check each file for external scripts
     files.forEach(filePath => {
       const externalScripts = checkFileForExternalScripts(filePath);
       if (externalScripts.length > 0) {
@@ -108,6 +148,9 @@ async function auditFn(complete) {
       }
     });
 
+    // Check locale files
+    checkLocaleFiles(packagedDir);
+
     // Display results
     displayAuditResults();
 
@@ -142,6 +185,23 @@ function displayAuditResults() {
     console.log(chalk.green('✅ No external scripts detected'));
   }
 
+  console.log('');
+
+  // Locale Warnings
+  if (auditResults.localeWarnings.length > 0) {
+    console.log(chalk.yellow.bold('⚠️  LOCALE FIELD LENGTH WARNINGS'));
+    console.log(chalk.gray('Some locale fields exceed recommended character limits.\n'));
+
+    auditResults.localeWarnings.forEach((item, index) => {
+      console.log(chalk.yellow(`  ${index + 1}. ${item.file}`));
+      console.log(chalk.gray(`     Field: ${item.field} (${item.length}/${item.limit} chars)`));
+      console.log(chalk.gray(`     Value: "${item.value}"`));
+      console.log('');
+    });
+  } else {
+    console.log(chalk.green('✅ All locale fields within limits'));
+  }
+
   // Summary
   console.log(chalk.bold('\n───────────────────────────────────────────────────'));
   console.log(chalk.bold('SUMMARY'));
@@ -151,6 +211,10 @@ function displayAuditResults() {
   const externalScriptColor = externalScriptCount > 0 ? chalk.red : chalk.green;
   console.log(externalScriptColor(`External Scripts: ${externalScriptCount}`));
 
+  const localeWarningCount = auditResults.localeWarnings.length;
+  const localeWarningColor = localeWarningCount > 0 ? chalk.yellow : chalk.green;
+  console.log(localeWarningColor(`Locale Warnings: ${localeWarningCount}`));
+
   console.log(chalk.bold('═══════════════════════════════════════════════════\n'));
 }
 

package/dist/gulp/tasks/translate.js

@@ -7,25 +7,8 @@ const path = require('path');
 const { execute } = require('node-powertools');
 const JSON5 = require('json5');
 
-// Languages to translate
-const LANGUAGES = [
-  'zh',
-  'es',
-  'hi',
-  'ar',
-  'pt',
-  'ru',
-  'ja',
-  'de',
-  'fr',
-  'ko',
-  'ur',
-  'id',
-  'bn',
-  'tl',
-  'vi',
-  'it',
-];
+// Locale config (shared with audit.js)
+const { limits: LOCALE_LIMITS, languages: LANGUAGES } = require('../config/locales.js');
 
 // Paths
 const localesDir = path.join(process.cwd(), 'src', '_locales');
@@ -80,7 +63,7 @@ async function translate(complete) {
   // Check which languages need translation
   const languagesToTranslate = [];
 
-  for (const lang of LANGUAGES) {
+  for (const lang of Object.keys(LANGUAGES)) {
     const langDir = path.join(localesDir, lang);
     const langMessagesPath = path.join(langDir, 'messages.json');
 
@@ -156,21 +139,30 @@ async function translate(complete) {
 async function translateAllWithClaude(enMessages, languagesToTranslate) {
   // Build language info for prompt
   const languageList = languagesToTranslate.map(({ lang }) =>
-    `- "${lang}": ${getLanguageName(lang)}`
+    `- "${lang}": ${LANGUAGES[lang]}`
   ).join('\n');
 
+  // Build character limits info
+  const limitsInfo = Object.entries(LOCALE_LIMITS)
+    .map(([field, limit]) => `- ${field}: max ${limit} characters`)
+    .join('\n');
+
   const prompt = `Translate the following Chrome extension messages.json content from English to multiple languages.
 
 TARGET LANGUAGES:
 ${languageList}
 
+CHARACTER LIMITS (Chrome Web Store requirements):
+${limitsInfo}
+
 IMPORTANT RULES:
 1. Only translate the "message" field values
 2. Keep the "description" field values in English (they are for developers)
 3. Keep all JSON keys exactly as they are
 4. Return ONLY valid JSON, no markdown, no explanation
 5. Preserve any placeholders like $1, $2, etc.
-6. Return a JSON object where each key is the language code and the value is the translated messages object
+6. IMPORTANT: Respect the character limits above for each field
+7. Return a JSON object where each key is the language code and the value is the translated messages object
 
 INPUT (English):
 ${JSON.stringify(enMessages, null, 2)}
@@ -225,29 +217,5 @@ Output the translated JSON:`;
   }
 }
 
-// Get full language name
-function getLanguageName(code) {
-  const names = {
-    zh: 'Chinese (Simplified)',
-    es: 'Spanish',
-    hi: 'Hindi',
-    ar: 'Arabic',
-    pt: 'Portuguese',
-    ru: 'Russian',
-    ja: 'Japanese',
-    de: 'German',
-    fr: 'French',
-    ko: 'Korean',
-    ur: 'Urdu',
-    id: 'Indonesian',
-    bn: 'Bengali',
-    tl: 'Tagalog/Filipino',
-    vi: 'Vietnamese',
-    it: 'Italian',
-  };
-
-  return names[code] || code;
-}
-
 // Export task
 module.exports = series(translate);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "browser-extension-manager",
-  "version": "1.2.12",
+  "version": "1.2.13",
   "description": "Browser Extension Manager dependency manager",
   "main": "dist/index.js",
   "exports": {