pabal-web-mcp 1.4.3 → 1.4.4

package/dist/bin/mcp-server.js

@@ -517,6 +517,19 @@ function convertToMultilingual(data, locale) {
 // src/tools/public-to-aso.ts
 import fs4 from "fs";
 var FIELD_LIMITS_DOC_PATH = "docs/aso/ASO_FIELD_LIMITS.md";
+var APP_STORE_LIMITS = {
+  name: 30,
+  subtitle: 30,
+  keywords: 100,
+  promotionalText: 170,
+  description: 4e3,
+  whatsNew: 4e3
+};
+var GOOGLE_PLAY_LIMITS = {
+  title: 50,
+  shortDescription: 80,
+  fullDescription: 4e3
+};
 var toJsonSchema2 = zodToJsonSchema2;
 var publicToAsoInputSchema = z2.object({
   slug: z2.string().describe("Product slug"),
@@ -527,6 +540,51 @@ var jsonSchema2 = toJsonSchema2(publicToAsoInputSchema, {
   $refStrategy: "none"
 });
 var inputSchema2 = jsonSchema2.definitions?.PublicToAsoInput || jsonSchema2;
+function validateFieldLimits(configData) {
+  const issues = [];
+  if (configData.appStore) {
+    const appStoreData = configData.appStore;
+    const locales = isAppStoreMultilingual(appStoreData) ? appStoreData.locales : { [appStoreData.locale || DEFAULT_LOCALE]: appStoreData };
+    for (const [locale, data] of Object.entries(locales)) {
+      const { name, subtitle, keywords, promotionalText, description, whatsNew } = data;
+      const push = (field, value, limit) => {
+        if (typeof value === "string" && value.length > limit) {
+          issues.push(
+            `App Store [${locale}].${field}: ${value.length}/${limit}`
+          );
+        }
+      };
+      push("name", name, APP_STORE_LIMITS.name);
+      push("subtitle", subtitle, APP_STORE_LIMITS.subtitle);
+      push("keywords", keywords, APP_STORE_LIMITS.keywords);
+      push("promotionalText", promotionalText, APP_STORE_LIMITS.promotionalText);
+      push("description", description, APP_STORE_LIMITS.description);
+      push("whatsNew", whatsNew, APP_STORE_LIMITS.whatsNew);
+    }
+  }
+  if (configData.googlePlay) {
+    const googlePlayData = configData.googlePlay;
+    const locales = isGooglePlayMultilingual(googlePlayData) ? googlePlayData.locales : { [googlePlayData.defaultLanguage || DEFAULT_LOCALE]: googlePlayData };
+    for (const [locale, data] of Object.entries(locales)) {
+      const { title, shortDescription, fullDescription } = data;
+      const push = (field, value, limit) => {
+        if (typeof value === "string" && value.length > limit) {
+          issues.push(
+            `Google Play [${locale}].${field}: ${value.length}/${limit}`
+          );
+        }
+      };
+      push("title", title, GOOGLE_PLAY_LIMITS.title);
+      push(
+        "shortDescription",
+        shortDescription,
+        GOOGLE_PLAY_LIMITS.shortDescription
+      );
+      push("fullDescription", fullDescription, GOOGLE_PLAY_LIMITS.fullDescription);
+    }
+  }
+  return issues;
+}
 async function downloadScreenshotsToAsoDir(slug, asoData) {
   const rootDir = getPushDataDir();
   const productStoreRoot = path4.join(rootDir, "products", slug, "store");
@@ -672,6 +730,7 @@ This tool:
 2. Converts to store-compatible format (removes screenshots from metadata, sets contactWebsite/marketingUrl)
 3. Saves metadata to .aso/pushData/products/[slug]/store/ (path from ~/.config/pabal-mcp/config.json dataDir)
 4. Copies/downloads screenshots to .aso/pushData/products/[slug]/store/screenshots/
+5. Validates text field lengths against ${FIELD_LIMITS_DOC_PATH} (fails if over limits)
 
 Before running, review ${FIELD_LIMITS_DOC_PATH} for per-store limits. This prepares data for pushing to stores without actually uploading.`,
   inputSchema: inputSchema2
@@ -761,6 +820,11 @@ Possible causes:
     );
   }
   const storeData = prepareAsoDataForPush(slug, configData);
+  const validationIssues = validateFieldLimits(configData);
+  const validationMessage = validationIssues.length > 0 ? `\u26A0\uFE0F Field limit issues (see ${FIELD_LIMITS_DOC_PATH}):
+- ${validationIssues.join(
+    "\n- "
+  )}` : `Field limits OK (checked against ${FIELD_LIMITS_DOC_PATH})`;
   const pushDataRoot = getPushDataDir();
   if (dryRun) {
     return {
@@ -773,11 +837,19 @@ ${JSON.stringify(
             storeData,
             null,
             2
-          )}`
+          )}
+
+${validationMessage}`
         }
       ]
     };
   }
+  if (validationIssues.length > 0) {
+    throw new Error(
+      `Field limit violations detected. Fix before pushing.
+${validationMessage}`
+    );
+  }
   saveRawAsoData(slug, storeData);
   await downloadScreenshotsToAsoDir(slug, configData);
   const localeCounts = {};
@@ -1314,10 +1386,10 @@ ${researchSections.join("\n")}
   prompt += `**CRITICAL**: You MUST include the complete \`landing\` object in your optimized JSON output.
 
 `;
-  prompt += `## Step 3: Validate
+  prompt += `## Step 3: Validate (after applying all keywords)
 
 `;
-  prompt += `Check all limits: title \u226430, subtitle \u226430, shortDescription \u226480, keywords \u2264100, intro \u2264300, outro \u2264200
+  prompt += `Check all limits using ${FIELD_LIMITS_DOC_PATH2}: title \u226430, subtitle \u226430, shortDescription \u226480, keywords \u2264100, intro \u2264300, outro \u2264200
 `;
   prompt += `- Remove keyword duplicates (unique list; avoid repeating title/subtitle terms verbatim)
 `;
@@ -1436,7 +1508,7 @@ function generateKeywordLocalizationPrompt(args) {
 `;
   prompt += `2. **Replace ONLY keywords with optimized keywords** - keep ALL existing content, structure, tone, and context unchanged. Only swap keywords for better ASO keywords.
 `;
-  prompt += `3. Validate character limits + store rules (${FIELD_LIMITS_DOC_PATH2}) + keyword duplication
+  prompt += `3. After all keywords are applied, validate character limits + store rules (${FIELD_LIMITS_DOC_PATH2}) + keyword duplication
 `;
   prompt += `4. **SAVE the updated JSON to file** using the save-locale-file tool (only if file exists)
 
@@ -1633,7 +1705,7 @@ ${researchSections.join(
 `;
   prompt += `3. **CRITICAL**: Ensure ALL landing fields are translated (not English)
 `;
-  prompt += `4. Validate limits + store rules (${FIELD_LIMITS_DOC_PATH2}) + keyword duplication (unique list; avoid repeating title/subtitle terms verbatim)
+  prompt += `4. After swapping keywords, validate limits + store rules (${FIELD_LIMITS_DOC_PATH2}) + keyword duplication (unique list; avoid repeating title/subtitle terms verbatim)
 `;
   prompt += `5. **SAVE the updated JSON to file** using save-locale-file tool
 `;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pabal-web-mcp",
-  "version": "1.4.3",
+  "version": "1.4.4",
   "type": "module",
   "description": "MCP server for ASO data management with shared types and utilities",
   "author": "skyu",