pabal-store-api-mcp 1.3.5 → 1.3.7

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/dist/src/packages/configs/aso-config/utils.js

@@ -76,14 +76,14 @@ export function saveAsoData(slug, asoData, options) {
             ensureDir(googlePlayDir);
             const filePath = join(googlePlayDir, "aso-data.json");
             writeFileSync(filePath, JSON.stringify({ googlePlay: asoData.googlePlay }, null, 2));
-            console.log(`💾 Google Play data saved to ${filePath}`);
+            console.error(`💾 Google Play data saved to ${filePath}`);
         }
         if (asoData.appStore) {
             const appStoreDir = join(productStoreDir, "app-store");
             ensureDir(appStoreDir);
             const filePath = join(appStoreDir, "aso-data.json");
             writeFileSync(filePath, JSON.stringify({ appStore: asoData.appStore }, null, 2));
-            console.log(`💾 App Store data saved to ${filePath}`);
+            console.error(`💾 App Store data saved to ${filePath}`);
         }
     }
     catch (error) {

package/dist/src/packages/stores/app-store/client.js

@@ -167,7 +167,7 @@ export class AppStoreClient {
             throw new Error("App Store version not found.");
         const localizationsResponse = await this.listAppStoreVersionLocalizations(version.id);
         const allLocalizations = localizationsResponse.data || [];
-        console.log(`🌍 Found ${allLocalizations.length} App Store localizations: ${allLocalizations
+        console.error(`🌍 Found ${allLocalizations.length} App Store localizations: ${allLocalizations
             .map((l) => l.attributes?.locale)
             .join(", ")}`);
         const locales = {};
@@ -290,11 +290,11 @@ export class AppStoreClient {
         });
         const existing = versionsResponse.data?.find((v) => v.attributes?.versionString === versionString);
         if (existing) {
-            console.log(`⚠️  Version ${versionString} already exists.`);
+            console.error(`⚠️  Version ${versionString} already exists.`);
             return existing;
         }
         const createResponse = await this.createAppStoreVersion(appId, versionString, APP_STORE_PLATFORM);
-        console.log(`✅ Created new version: ${versionString}`);
+        console.error(`✅ Created new version: ${versionString}`);
         return createResponse.data;
     }
     async createNewVersionWithAutoIncrement(baseVersion) {
@@ -306,12 +306,12 @@ export class AppStoreClient {
             const latestVersion = await this.getLatestVersion();
             if (!latestVersion) {
                 newVersionString = "1.0.0";
-                console.log(`📦 No existing versions. Starting with ${newVersionString}`);
+                console.error(`📦 No existing versions. Starting with ${newVersionString}`);
             }
             else {
                 const latest = latestVersion.attributes?.versionString ?? "0.0.0";
                 newVersionString = this.incrementVersion(latest);
-                console.log(`📦 Latest: ${latest} → New: ${newVersionString}`);
+                console.error(`📦 Latest: ${latest} → New: ${newVersionString}`);
             }
         }
         return this.createNewVersion(newVersionString);
@@ -330,7 +330,7 @@ export class AppStoreClient {
                 whatsNew,
             });
         }
-        console.log(`✅ Updated What's New for ${locale}`);
+        console.error(`✅ Updated What's New for ${locale}`);
     }
     async pullReleaseNotes() {
         const appId = await this.findAppId();

package/dist/src/packages/stores/play-store/client.js

@@ -81,7 +81,7 @@ export class GooglePlayClient {
             const appDetails = await this.getAppDetails(session);
             const listingsResponse = await this.listListings(session);
             const allListings = listingsResponse.data.listings || [];
-            console.log(`🌍 Found ${allListings.length} Google Play languages: ${allListings
+            console.error(`🌍 Found ${allListings.length} Google Play languages: ${allListings
                 .map((l) => l.language)
                 .join(", ")}`);
             const locales = {};

package/package.json

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