pabal-store-api-mcp 1.3.5 → 1.3.6

package/dist/src/core/services/google-play-service.js

@@ -196,26 +196,18 @@ export class GooglePlayService {
                             ((localeData.screenshots.phone &&
                                 localeData.screenshots.phone.length > 0) ||
                                 (localeData.screenshots.tablet &&
-                                    localeData.screenshots.tablet.length > 0) ||
-                                (localeData.screenshots.tablet7 &&
-                                    localeData.screenshots.tablet7.length > 0) ||
-                                (localeData.screenshots.tablet10 &&
-                                    localeData.screenshots.tablet10.length > 0));
+                                    localeData.screenshots.tablet.length > 0));
                         let screenshots;
                         if (hasScreenshotsInJson) {
                             // Use screenshots from aso-data.json (relative paths)
                             console.error(`[GooglePlay]   📋 Using screenshots from aso-data.json for ${locale}`);
                             const relativePaths = localeData.screenshots;
-                            // If 'tablet' array exists (generic), use it for both 7-inch and 10-inch
-                            const genericTablet = (relativePaths.tablet || []).map((p) => `${screenshotsBaseDir}/${p}`);
+                            // Google Play upload strategy:
+                            // - phone array → uploads to both phoneScreenshots AND sevenInchScreenshots
+                            // - tablet array → uploads to tenInchScreenshots only
                             screenshots = {
                                 phone: (relativePaths.phone || []).map((p) => `${screenshotsBaseDir}/${p}`),
-                                tablet7: relativePaths.tablet7?.length
-                                    ? relativePaths.tablet7.map((p) => `${screenshotsBaseDir}/${p}`)
-                                    : genericTablet,
-                                tablet10: relativePaths.tablet10?.length
-                                    ? relativePaths.tablet10.map((p) => `${screenshotsBaseDir}/${p}`)
-                                    : genericTablet,
+                                tablet: (relativePaths.tablet || []).map((p) => `${screenshotsBaseDir}/${p}`),
                                 featureGraphic: localeData.featureGraphic
                                     ? `${screenshotsBaseDir}/${localeData.featureGraphic}`
                                     : null,
@@ -230,29 +222,33 @@ export class GooglePlayService {
                                 continue;
                             }
                             console.error(`[GooglePlay]   📂 Parsing screenshots from file system for ${locale}`);
-                            screenshots = parseGooglePlayScreenshots(screenshotsFsDir, locale);
+                            const fsScreenshots = parseGooglePlayScreenshots(screenshotsFsDir, locale);
+                            // File system fallback: use tablet10 as tablet
+                            screenshots = {
+                                phone: fsScreenshots.phone,
+                                tablet: fsScreenshots.tablet10,
+                                featureGraphic: fsScreenshots.featureGraphic,
+                            };
                         }
                         // Google Play requires minimum 2 phone screenshots
-                        const phoneCount = screenshots.phone.length + screenshots.tablet7.length;
+                        const phoneCount = screenshots.phone.length;
                         if (phoneCount < 2) {
-                            console.error(`[GooglePlay]   ⚠️  Skipping ${locale} - needs at least 2 phone/tablet7 screenshots (found ${phoneCount})`);
+                            console.error(`[GooglePlay]   ⚠️  Skipping ${locale} - needs at least 2 phone screenshots (found ${phoneCount})`);
                             skippedLocales.push(locale);
                             continue;
                         }
                         console.error(`[GooglePlay]   📤 Uploading screenshots for ${locale} (batch mode - will replace existing)...`);
-                        // Use batch upload method - deletes existing and uploads new in single edit session
-                        // Combine phone and tablet7 screenshots as phoneScreenshots (Google Play accepts both sizes)
-                        const phoneScreenshotPaths = [
-                            ...screenshots.phone,
-                            ...screenshots.tablet7,
-                        ];
+                        // Google Play upload strategy:
+                        // - phone → uploads to phoneScreenshots AND sevenInchScreenshots (both use same images)
+                        // - tablet → uploads to tenInchScreenshots only
                         const uploadResult = await client.uploadScreenshotsForLocale({
                             language: locale,
-                            phoneScreenshots: phoneScreenshotPaths,
-                            tenInchScreenshots: screenshots.tablet10,
+                            phoneScreenshots: screenshots.phone,
+                            sevenInchScreenshots: screenshots.phone,
+                            tenInchScreenshots: screenshots.tablet,
                             featureGraphic: screenshots.featureGraphic || undefined,
                         });
-                        console.error(`[GooglePlay]   ✅ Screenshots uploaded for ${locale}: ${uploadResult.uploaded.phoneScreenshots} phone, ${uploadResult.uploaded.tenInchScreenshots} tablet`);
+                        console.error(`[GooglePlay]   ✅ Screenshots uploaded for ${locale}: ${uploadResult.uploaded.phoneScreenshots} phone, ${uploadResult.uploaded.sevenInchScreenshots} 7-inch, ${uploadResult.uploaded.tenInchScreenshots} 10-inch`);
                         uploadedLocales.push(locale);
                     }
                     catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pabal-store-api-mcp",
-  "version": "1.3.5",
+  "version": "1.3.6",
   "description": "MCP server for App Store / Play Store ASO workflows",
   "main": "dist/src/index.js",
   "types": "dist/src/index.d.ts",