perf: read package info entries in a single zip scan

- zip.ts: add getEntriesFromHarmonyApp to read multiple entries (and do
  the nested .hap extraction) in one pass; getEntryFromHarmonyApp now
  delegates to it
- utils/index.ts: getApkInfo (2 scans -> 1), getIpaInfo (4 -> 1) and
  getAppInfo (3 scans + 3 nested .hap extractions -> 1) batch their
  entry lookups instead of enumerating the archive once per entry
- tests: cover batch harmony reads and case-sensitive regex entry match

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: sunnylqm

src/utils/app-info-parser/zip.ts

@@ -37,20 +37,33 @@ export class Zip {
   async getEntryFromHarmonyApp(
     regex: RegExp,
   ): Promise<Buffer | Blob | undefined> {
+    return (await this.getEntriesFromHarmonyApp([regex]))[0];
+  }
+
+  /**
+   * Read multiple entries in a single pass over the archive (including
+   * nested .hap extraction), instead of one full scan per entry.
+   * Returns buffers in the same order as the given regexps.
+   */
+  async getEntriesFromHarmonyApp(
+    regexps: RegExp[],
+  ): Promise<(Buffer | undefined)[]> {
+    const results: (Buffer | undefined)[] = new Array(regexps.length);
     try {
-      let originSource: Buffer | Blob | undefined;
       if (typeof this.file !== 'string') {
         throw new Error('Param error: [file] must be file path in Node.');
       }
       await enumZipEntries(this.file, async (entry, zipFile) => {
-        if (regex.test(entry.fileName)) {
-          originSource = await readEntry(entry, zipFile);
+        for (let i = 0; i < regexps.length; i++) {
+          if (regexps[i].test(entry.fileName)) {
+            results[i] = await readEntry(entry, zipFile);
+          }
         }
       });
-      return originSource;
     } catch (error) {
-      console.error('Error in getEntryFromHarmonyApp:', error);
+      console.error('Error in getEntriesFromHarmonyApp:', error);
     }
+    return results;
   }
 
   private async readEntries(

src/utils/index.ts

@@ -139,11 +139,17 @@ export function translateOptions<T extends Record<string, unknown>>(
   return result as T;
 }
 
+const ApkBundleFileName = /assets\/index.android.bundle/;
+const ApkUpdateJsonName = /res\/raw\/update.json/;
+
 export async function getApkInfo(fn: string) {
   const appInfoParser = new AppInfoParser(fn);
-  const bundleFile = await appInfoParser.parser.getEntry(
-    /assets\/index.android.bundle/,
-  );
+  // read both entries in a single scan over the archive
+  const entries = await appInfoParser.parser.getEntries([
+    ApkBundleFileName,
+    ApkUpdateJsonName,
+  ]);
+  const bundleFile = entries[String(ApkBundleFileName)];
   if (!bundleFile) {
     throw new Error(
       t('bundleNotFound', {
@@ -152,9 +158,7 @@ export async function getApkInfo(fn: string) {
       }),
     );
   }
-  const updateJsonFile = await appInfoParser.parser.getEntry(
-    /res\/raw\/update.json/,
-  );
+  const updateJsonFile = entries[String(ApkUpdateJsonName)];
   let appCredential = {};
   if (updateJsonFile) {
     appCredential = JSON.parse(updateJsonFile.toString()).android;
@@ -182,9 +186,13 @@ export async function getApkInfo(fn: string) {
 
 export async function getAppInfo(fn: string) {
   const appInfoParser = new AppInfoParser(fn);
-  const bundleFile = await appInfoParser.parser.getEntryFromHarmonyApp(
-    /rawfile\/bundle.harmony.js/,
-  );
+  // single scan (and single nested .hap extraction) for all three entries
+  const [bundleFile, updateJsonFile, metaJsonFile] =
+    await appInfoParser.parser.getEntriesFromHarmonyApp([
+      /rawfile\/bundle.harmony.js/,
+      /rawfile\/update.json/,
+      /rawfile\/meta.json/,
+    ]);
   if (!bundleFile) {
     throw new Error(
       t('bundleNotFound', {
@@ -193,14 +201,10 @@ export async function getAppInfo(fn: string) {
       }),
     );
   }
-  const updateJsonFile =
-    await appInfoParser.parser.getEntryFromHarmonyApp(/rawfile\/update.json/);
   let appCredential = {};
   if (updateJsonFile) {
     appCredential = JSON.parse(updateJsonFile.toString()).harmony;
   }
-  const metaJsonFile =
-    await appInfoParser.parser.getEntryFromHarmonyApp(/rawfile\/meta.json/);
   let metaData: ParsedAppMetaInfo = {};
   if (metaJsonFile) {
     metaData = JSON.parse(metaJsonFile.toString()) as ParsedAppMetaInfo;
@@ -216,11 +220,23 @@ export async function getAppInfo(fn: string) {
   return { versionName, buildTime, ...appCredential };
 }
 
+const IpaBundleFileName = /payload\/.+?\.app\/main.jsbundle/;
+const IpaUpdateJsonName = /payload\/.+?\.app\/assets\/update.json/;
+const IpaBuildTimeName = /payload\/.+?\.app\/pushy_build_time.txt/;
+// Not in root bundle when use `use_frameworks`
+const IpaBuildTimeFrameworkName =
+  /payload\/.+?\.app\/frameworks\/react_native_update.framework\/pushy_build_time.txt/;
+
 export async function getIpaInfo(fn: string) {
   const appInfoParser = new AppInfoParser(fn);
-  const bundleFile = await appInfoParser.parser.getEntry(
-    /payload\/.+?\.app\/main.jsbundle/,
-  );
+  // read all four entries in a single scan over the archive
+  const entries = await appInfoParser.parser.getEntries([
+    IpaBundleFileName,
+    IpaUpdateJsonName,
+    IpaBuildTimeName,
+    IpaBuildTimeFrameworkName,
+  ]);
+  const bundleFile = entries[String(IpaBundleFileName)];
   if (!bundleFile) {
     throw new Error(
       t('bundleNotFound', {
@@ -229,24 +245,16 @@ export async function getIpaInfo(fn: string) {
       }),
     );
   }
-  const updateJsonFile = await appInfoParser.parser.getEntry(
-    /payload\/.+?\.app\/assets\/update.json/,
-  );
+  const updateJsonFile = entries[String(IpaUpdateJsonName)];
   let appCredential = {};
   if (updateJsonFile) {
     appCredential = JSON.parse(updateJsonFile.toString()).ios;
   }
   const { CFBundleShortVersionString: versionName } =
     await appInfoParser.parse<ParsedIpaInfo>();
-  let buildTimeTxtBuffer = await appInfoParser.parser.getEntry(
-    /payload\/.+?\.app\/pushy_build_time.txt/,
-  );
-  if (!buildTimeTxtBuffer) {
-    // Not in root bundle when use `use_frameworks`
-    buildTimeTxtBuffer = await appInfoParser.parser.getEntry(
-      /payload\/.+?\.app\/frameworks\/react_native_update.framework\/pushy_build_time.txt/,
-    );
-  }
+  const buildTimeTxtBuffer =
+    entries[String(IpaBuildTimeName)] ??
+    entries[String(IpaBuildTimeFrameworkName)];
   if (!buildTimeTxtBuffer) {
     throw new Error(t('buildTimeNotFound'));
   }

tests/app-info-zip.test.ts

@@ -145,6 +145,40 @@ suffix`,
     expect((buf as Buffer).toString()).toBe('{"harmony": true}');
   });
 
+  test('getEntriesFromHarmonyApp reads multiple entries in one pass', async () => {
+    const zipPath = path.join(tempRoot, 'app.zip');
+    await writeZip(zipPath, {
+      'rawfile/bundle.harmony.js': 'bundle',
+      'rawfile/update.json': '{"harmony":{}}',
+      'other.txt': 'ignore',
+    });
+
+    const zip = new Zip(zipPath);
+
+    const [bundle, updateJson, missing] = await zip.getEntriesFromHarmonyApp([
+      /rawfile\/bundle.harmony.js/,
+      /rawfile\/update.json/,
+      /rawfile\/meta.json/,
+    ]);
+    expect((bundle as Buffer).toString()).toBe('bundle');
+    expect((updateJson as Buffer).toString()).toBe('{"harmony":{}}');
+    expect(missing).toBeUndefined();
+  });
+
+  test('getEntry matches case-sensitive regex against original entry name', async () => {
+    const zipPath = path.join(tempRoot, 'app.zip');
+    await writeZip(zipPath, {
+      'base/manifest/AndroidManifest.xml': 'aab manifest',
+    });
+
+    // regex with uppercase letters used to never match because only the
+    // lowercased entry name was tested
+    const buf = await new Zip(zipPath).getEntry(
+      /^base\/manifest\/AndroidManifest\.xml$/,
+    );
+    expect((buf as Buffer).toString()).toBe('aab manifest');
+  });
+
   test('constructor resolves path for string input', () => {
     const relPath = 'some/relative/path.zip';
     const zip = new Zip(relPath);