@gpc-cli/core 0.9.73 → 0.9.74

package/dist/index.js

@@ -5132,7 +5132,8 @@ async function loadPreflightConfig(configPath) {
 }
 
 // src/preflight/aab-reader.ts
-import { open as yauzlOpen } from "yauzl";
+import { readFile as readFile10 } from "fs/promises";
+import { fromBuffer } from "yauzl";
 
 // src/preflight/manifest-parser.ts
 import protobuf from "protobufjs";
@@ -5321,12 +5322,20 @@ var AAB_MANIFEST_PATH = "base/manifest/AndroidManifest.xml";
 var APK_MANIFEST_PATH = "AndroidManifest.xml";
 var SO_HEADER_BYTES = 4096;
 var SO_PATH_RE = /(?:^|\/)lib\/[^/]+\/[^/]+\.so$/;
+var SCAN_TIMEOUT_MS = 6e4;
 function detectManifestPath(filePath) {
   return filePath.toLowerCase().endsWith(".apk") ? APK_MANIFEST_PATH : AAB_MANIFEST_PATH;
 }
 async function readAab(aabPath) {
   const manifestPath = detectManifestPath(aabPath);
-  const { zipfile, entries, manifestBuf, soHeaders } = await openAndScan(aabPath, manifestPath);
+  let buffer;
+  try {
+    buffer = await readFile10(aabPath);
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : String(err);
+    throw new Error(`Could not read ${aabPath}: ${msg}`);
+  }
+  const { zipfile, entries, manifestBuf, soHeaders } = await openAndScan(buffer, manifestPath);
   zipfile.close();
   if (!manifestBuf) {
     const fileType = aabPath.toLowerCase().endsWith(".apk") ? "APK" : "AAB";
@@ -5363,98 +5372,125 @@ function createFallbackManifest() {
     providers: []
   };
 }
-function openAndScan(aabPath, manifestPath = AAB_MANIFEST_PATH) {
+function openAndScan(buffer, manifestPath = AAB_MANIFEST_PATH) {
   return new Promise((resolve2, reject) => {
-    yauzlOpen(aabPath, { lazyEntries: true, autoClose: false }, (err, zipfile) => {
-      if (err || !zipfile) {
-        reject(err ?? new Error("Failed to open AAB file"));
-        return;
+    let settled = false;
+    const timer = setTimeout(() => {
+      if (!settled) {
+        settled = true;
+        reject(new Error("Timed out reading the archive. The file may be corrupt or truncated."));
       }
-      const entries = [];
-      const soHeaders = /* @__PURE__ */ new Map();
-      let manifestBuf = null;
-      let rejected = false;
-      let pendingStreams = 0;
-      let entrysDone = false;
-      function fail(error) {
-        if (!rejected) {
-          rejected = true;
-          zipfile.close();
-          reject(error);
+    }, SCAN_TIMEOUT_MS);
+    function settleResolve(value) {
+      if (settled) return;
+      settled = true;
+      clearTimeout(timer);
+      resolve2(value);
+    }
+    function settleReject(error) {
+      if (settled) return;
+      settled = true;
+      clearTimeout(timer);
+      reject(error);
+    }
+    try {
+      fromBuffer(buffer, { lazyEntries: true }, (err, zipfile) => {
+        if (err || !zipfile) {
+          settleReject(err ?? new Error("Failed to open AAB file"));
+          return;
         }
-      }
-      function maybeResolve() {
-        if (!rejected && entrysDone && pendingStreams === 0) {
-          resolve2({ zipfile, entries, manifestBuf, soHeaders });
+        const entries = [];
+        const soHeaders = /* @__PURE__ */ new Map();
+        let manifestBuf = null;
+        let pendingStreams = 0;
+        let entrysDone = false;
+        function fail(error) {
+          if (settled) return;
+          try {
+            zipfile.close();
+          } catch {
+          }
+          settleReject(error);
         }
-      }
-      function readEntryStream(entry, onData, maxBytes) {
-        pendingStreams++;
-        zipfile.openReadStream(entry, (streamErr, stream) => {
-          if (streamErr || !stream) {
-            fail(streamErr ?? new Error(`Failed to read ${entry.fileName}`));
-            return;
+        function maybeResolve() {
+          if (!settled && entrysDone && pendingStreams === 0) {
+            settleResolve({ zipfile, entries, manifestBuf, soHeaders });
           }
-          const chunks = [];
-          let totalBytes = 0;
-          let done = false;
-          stream.on("data", (chunk) => {
-            chunks.push(chunk);
-            totalBytes += chunk.length;
-            if (maxBytes && totalBytes >= maxBytes) {
-              stream.destroy();
-            }
-          });
-          stream.on("error", (e) => {
-            if (maxBytes && totalBytes >= maxBytes) return;
-            fail(e);
-          });
-          const finish = () => {
-            if (done) return;
-            done = true;
-            const buf = Buffer.concat(chunks);
-            onData(maxBytes ? buf.subarray(0, maxBytes) : buf);
-            pendingStreams--;
-            maybeResolve();
-          };
-          stream.on("end", finish);
-          stream.on("close", () => {
-            if (totalBytes > 0 && maxBytes) finish();
-          });
-        });
-      }
-      zipfile.on("error", fail);
-      zipfile.on("entry", (entry) => {
-        if (rejected) return;
-        const path = entry.fileName;
-        if (!path.endsWith("/")) {
-          entries.push({
-            path,
-            compressedSize: entry.compressedSize,
-            uncompressedSize: entry.uncompressedSize
-          });
         }
-        if (path === manifestPath) {
-          readEntryStream(entry, (buf) => {
-            manifestBuf = buf;
+        function readEntryStream(entry, onData, maxBytes) {
+          pendingStreams++;
+          zipfile.openReadStream(entry, (streamErr, stream) => {
+            if (streamErr || !stream) {
+              fail(streamErr ?? new Error(`Failed to read ${entry.fileName}`));
+              return;
+            }
+            const chunks = [];
+            let totalBytes = 0;
+            let done = false;
+            stream.on("data", (chunk) => {
+              chunks.push(chunk);
+              totalBytes += chunk.length;
+              if (maxBytes && totalBytes >= maxBytes) {
+                stream.destroy();
+              }
+            });
+            stream.on("error", (e) => {
+              if (maxBytes && totalBytes >= maxBytes) return;
+              fail(e);
+            });
+            const finish = () => {
+              if (done) return;
+              done = true;
+              const buf = Buffer.concat(chunks);
+              onData(maxBytes ? buf.subarray(0, maxBytes) : buf);
+              pendingStreams--;
+              maybeResolve();
+            };
+            stream.on("end", finish);
+            stream.on("close", () => {
+              if (totalBytes > 0 && maxBytes) finish();
+            });
           });
-        } else if (SO_PATH_RE.test(path)) {
-          readEntryStream(
-            entry,
-            (buf) => {
-              soHeaders.set(path, buf);
-            },
-            SO_HEADER_BYTES
-          );
         }
+        zipfile.on("error", fail);
+        zipfile.on("entry", (entry) => {
+          if (settled) return;
+          try {
+            const path = entry.fileName;
+            if (!path.endsWith("/")) {
+              entries.push({
+                path,
+                compressedSize: entry.compressedSize,
+                uncompressedSize: entry.uncompressedSize
+              });
+            }
+            if (path === manifestPath) {
+              readEntryStream(entry, (buf) => {
+                manifestBuf = buf;
+              });
+            } else if (SO_PATH_RE.test(path)) {
+              readEntryStream(
+                entry,
+                (buf) => {
+                  soHeaders.set(path, buf);
+                },
+                SO_HEADER_BYTES
+              );
+            }
+            zipfile.readEntry();
+          } catch (e) {
+            fail(e instanceof Error ? e : new Error(String(e)));
+          }
+        });
+        zipfile.on("end", () => {
+          entrysDone = true;
+          maybeResolve();
+        });
         zipfile.readEntry();
       });
-      zipfile.on("end", () => {
-        entrysDone = true;
-        maybeResolve();
-      });
-      zipfile.readEntry();
-    });
+    } catch (err) {
+      settleReject(err instanceof Error ? err : new Error(String(err)));
+    }
   });
 }
 
@@ -6018,7 +6054,7 @@ ${fileList}` + (hasPageSizeCompat ? "\nandroid:pageSizeCompat is set, so the app
 }
 
 // src/preflight/scanners/metadata-scanner.ts
-import { readdir as readdir4, stat as stat6, readFile as readFile10 } from "fs/promises";
+import { readdir as readdir4, stat as stat6, readFile as readFile11 } from "fs/promises";
 import { join as join8 } from "path";
 var SAFE_LANG = /^[a-zA-Z]{2,3}(-[a-zA-Z0-9]{2,8})*$/;
 var FILE_MAP2 = {
@@ -6077,7 +6113,7 @@ var metadataScanner = {
       for (const [fileName, field] of Object.entries(FILE_MAP2)) {
         const filePath = join8(langDir, fileName);
         try {
-          const content = await readFile10(filePath, "utf-8");
+          const content = await readFile11(filePath, "utf-8");
           fields[field] = content.trimEnd();
         } catch {
         }
@@ -6150,7 +6186,7 @@ var metadataScanner = {
     const defaultLang = locales.includes("en-US") ? "en-US" : locales[0];
     const privacyPath = join8(dir, defaultLang, "privacy_policy_url.txt");
     try {
-      const url = await readFile10(privacyPath, "utf-8");
+      const url = await readFile11(privacyPath, "utf-8");
       if (!url.trim()) throw new Error("empty");
     } catch {
       findings.push({
@@ -6175,7 +6211,7 @@ var metadataScanner = {
 };
 
 // src/preflight/scanners/secrets-scanner.ts
-import { readFile as readFile11 } from "fs/promises";
+import { readFile as readFile12 } from "fs/promises";
 
 // src/preflight/scan-files.ts
 import { readdir as readdir5, stat as stat7 } from "fs/promises";
@@ -6295,7 +6331,7 @@ var secretsScanner = {
     for (const filePath of files) {
       let content;
       try {
-        content = await readFile11(filePath, "utf-8");
+        content = await readFile12(filePath, "utf-8");
       } catch {
         continue;
       }
@@ -6323,7 +6359,7 @@ var secretsScanner = {
 };
 
 // src/preflight/scanners/billing-scanner.ts
-import { readFile as readFile12 } from "fs/promises";
+import { readFile as readFile13 } from "fs/promises";
 var BILLING_PATTERNS = [
   {
     ruleId: "billing-stripe-sdk",
@@ -6384,7 +6420,7 @@ var billingScanner = {
     for (const filePath of files) {
       let content;
       try {
-        content = await readFile12(filePath, "utf-8");
+        content = await readFile13(filePath, "utf-8");
       } catch {
         continue;
       }
@@ -6410,7 +6446,7 @@ var billingScanner = {
 };
 
 // src/preflight/scanners/privacy-scanner.ts
-import { readFile as readFile13 } from "fs/promises";
+import { readFile as readFile14 } from "fs/promises";
 var TRACKING_SDKS = [
   {
     name: "Facebook SDK",
@@ -6450,7 +6486,7 @@ var privacyScanner = {
     for (const filePath of files) {
       let content;
       try {
-        content = await readFile13(filePath, "utf-8");
+        content = await readFile14(filePath, "utf-8");
       } catch {
         continue;
       }
@@ -7150,7 +7186,7 @@ async function downloadGeneratedApk(client, packageName, versionCode, apkId, out
 }
 
 // src/commands/bundle-analysis.ts
-import { readFile as readFile14, stat as stat8 } from "fs/promises";
+import { readFile as readFile15, stat as stat8 } from "fs/promises";
 var EOCD_SIGNATURE = 101010256;
 var CD_SIGNATURE = 33639248;
 var MODULE_SUBDIRS = /* @__PURE__ */ new Set(["dex", "manifest", "res", "assets", "lib", "resources.pb", "root"]);
@@ -7230,7 +7266,7 @@ async function analyzeBundle(filePath) {
   if (!fileInfo || !fileInfo.isFile()) {
     throw new Error(`File not found: ${filePath}`);
   }
-  const buf = await readFile14(filePath);
+  const buf = await readFile15(filePath);
   const cdEntries = parseCentralDirectory(buf);
   const fileType = detectFileType2(filePath);
   const isAab = fileType === "aab";
@@ -7316,7 +7352,7 @@ function topFiles(analysis, n = 20) {
 async function checkBundleSize(analysis, configPath = ".bundlesize.json") {
   let config;
   try {
-    const raw = await readFile14(configPath, "utf-8");
+    const raw = await readFile15(configPath, "utf-8");
     config = JSON.parse(raw);
   } catch {
     return { passed: true, violations: [] };