refactor: extract iteration code from linking logic

This change attempts to separate the Xcframework linking logic from the directory iteration by extracting the "glob-like" function out. This change also opens the door for easier migration to a proper glob() function.

Author: Mariusz Pasinski

packages/host/src/node/cli/apple.ts

@@ -64,73 +64,73 @@ export async function linkXcframework({
     await fs.promises.rm(outputPath, { recursive: true, force: true });
     await fs.promises.cp(modulePath, tempPath, { recursive: true });
 
-    const frameworkPaths = await Promise.all(
-      fs
-        .readdirSync(tempPath, {
-          withFileTypes: true,
-        })
+    // Following extracted function mimics `glob("*/*.framework/")`
+    function globFrameworkDirs<T>(
+      startPath: string,
+      fn: (parentPath: string, name: string) => Promise<T>
+    ) {
+      return fs
+        .readdirSync(startPath, { withFileTypes: true })
         .filter((tripletEntry) => tripletEntry.isDirectory())
         .flatMap((tripletEntry) => {
-          const tripletPath = path.join(tempPath, tripletEntry.name);
+          const tripletPath = path.join(startPath, tripletEntry.name);
           return fs
-            .readdirSync(tripletPath, {
-              withFileTypes: true,
-            })
+            .readdirSync(tripletPath, { withFileTypes: true })
             .filter(
               (frameworkEntry) =>
                 frameworkEntry.isDirectory() &&
                 path.extname(frameworkEntry.name) === ".framework"
             )
-            .flatMap(async (frameworkEntry) => {
-              const frameworkPath = path.join(tripletPath, frameworkEntry.name);
-              const oldLibraryName = path.basename(
-                frameworkEntry.name,
-                ".framework"
-              );
-              const oldLibraryPath = path.join(frameworkPath, oldLibraryName);
-              const newFrameworkPath = path.join(
-                tripletPath,
-                `${newLibraryName}.framework`
-              );
-              const newLibraryPath = path.join(
-                newFrameworkPath,
-                newLibraryName
-              );
-              assert(
-                fs.existsSync(oldLibraryPath),
-                `Expected a library at '${oldLibraryPath}'`
-              );
-              // Rename the library
-              await fs.promises.rename(
-                oldLibraryPath,
-                // Cannot use newLibraryPath here, because the framework isn't renamed yet
-                path.join(frameworkPath, newLibraryName)
-              );
-              // Rename the framework
-              await fs.promises.rename(frameworkPath, newFrameworkPath);
-              // Expect the library in the new location
-              assert(fs.existsSync(newLibraryPath));
-              // Update the binary
-              await spawn(
-                "install_name_tool",
-                [
-                  "-id",
-                  `@rpath/${newLibraryName}.framework/${newLibraryName}`,
-                  newLibraryPath,
-                ],
-                {
-                  outputMode: "buffered",
-                }
-              );
-              // Update the Info.plist file for the framework
-              await updateInfoPlist({
-                filePath: path.join(newFrameworkPath, "Info.plist"),
-                oldLibraryName,
-                newLibraryName,
-              });
-              return newFrameworkPath;
-            });
-        })
+            .flatMap(async (frameworkEntry) =>
+              await fn(tripletPath, frameworkEntry.name)
+            );
+        });
+    }
+
+    const frameworkPaths = await Promise.all(
+      globFrameworkDirs(tempPath, async (tripletPath, frameworkEntryName) => {
+        const frameworkPath = path.join(tripletPath, frameworkEntryName);
+        const oldLibraryName = path.basename(frameworkEntryName, ".framework");
+        const oldLibraryPath = path.join(frameworkPath, oldLibraryName);
+        const newFrameworkPath = path.join(
+          tripletPath,
+          `${newLibraryName}.framework`
+        );
+        const newLibraryPath = path.join(newFrameworkPath, newLibraryName);
+        assert(
+          fs.existsSync(oldLibraryPath),
+          `Expected a library at '${oldLibraryPath}'`
+        );
+        // Rename the library
+        await fs.promises.rename(
+          oldLibraryPath,
+          // Cannot use newLibraryPath here, because the framework isn't renamed yet
+          path.join(frameworkPath, newLibraryName)
+        );
+        // Rename the framework
+        await fs.promises.rename(frameworkPath, newFrameworkPath);
+        // Expect the library in the new location
+        assert(fs.existsSync(newLibraryPath));
+        // Update the binary
+        await spawn(
+          "install_name_tool",
+          [
+            "-id",
+            `@rpath/${newLibraryName}.framework/${newLibraryName}`,
+            newLibraryPath,
+          ],
+          {
+            outputMode: "buffered",
+          }
+        );
+        // Update the Info.plist file for the framework
+        await updateInfoPlist({
+          filePath: path.join(newFrameworkPath, "Info.plist"),
+          oldLibraryName,
+          newLibraryName,
+        });
+        return newFrameworkPath;
+      })
     );
 
     // Create a new xcframework from the renamed frameworks