refactor: improve resource directory handling for macOS

.changeset/three-foxes-confess.md

@@ -0,0 +1,5 @@
+---
+"app-builder-lib": patch
+---
+
+refactor: improve resource directory handling for macOS

packages/app-builder-lib/src/electron/ElectronFramework.ts

@@ -97,27 +97,29 @@ async function removeUnusedLanguagesIfNeeded(options: BeforeCopyExtraFilesOption
     return
   }
 
-  const { dir, langFileExt } = getLocalesConfig(options)
+  const { dirs, langFileExt } = getLocalesConfig(options)
   // noinspection SpellCheckingInspection
-  await asyncPool(MAX_FILE_REQUESTS, await readdir(dir), async file => {
-    if (!file.endsWith(langFileExt)) {
-      return
-    }
+  const deletedFiles = async (dir: string) => {
+    await asyncPool(MAX_FILE_REQUESTS, await readdir(dir), async file => {
+      if (!file.endsWith(langFileExt)) {
+        return
+      }
 
-    const language = file.substring(0, file.length - langFileExt.length)
-    if (!wantedLanguages.includes(language)) {
-      return fs.rm(path.join(dir, file), { recursive: true, force: true })
-    }
-    return
-  })
+      const language = file.substring(0, file.length - langFileExt.length)
+      if (!wantedLanguages.includes(language)) {
+        return fs.rm(path.join(dir, file), { recursive: true, force: true })
+      }
+      return
+    })
+  }
+  await Promise.all(dirs.map(deletedFiles))
 
   function getLocalesConfig(options: BeforeCopyExtraFilesOptions) {
     const { appOutDir, packager } = options
     if (packager.platform === Platform.MAC) {
-      return { dir: packager.getResourcesDir(appOutDir), langFileExt: ".lproj" }
-    } else {
-      return { dir: path.join(packager.getResourcesDir(appOutDir), "..", "locales"), langFileExt: ".pak" }
+      return { dirs: [packager.getResourcesDir(appOutDir), packager.getMacOsElectronFrameworkResourcesDir(appOutDir)], langFileExt: ".lproj" }
     }
+    return { dirs: [path.join(packager.getResourcesDir(appOutDir), "..", "locales")], langFileExt: ".pak" }
   }
 }
 

packages/app-builder-lib/src/platformPackager.ts

@@ -608,13 +608,16 @@ export abstract class PlatformPackager<DC extends PlatformSpecificBuildOptions>
   getResourcesDir(appOutDir: string): string {
     if (this.platform === Platform.MAC) {
       return this.getMacOsResourcesDir(appOutDir)
-    } else if (isElectronBased(this.info.framework)) {
+    }
+    if (isElectronBased(this.info.framework)) {
       return path.join(appOutDir, "resources")
-    } else {
-      return appOutDir
     }
+    return appOutDir
   }
 
+  public getMacOsElectronFrameworkResourcesDir(appOutDir: string): string {
+    return path.join(appOutDir, `${this.appInfo.productFilename}.app`, "Contents", "Frameworks", "Electron Framework.framework", "Resources")
+  }
   public getMacOsResourcesDir(appOutDir: string): string {
     return path.join(appOutDir, `${this.appInfo.productFilename}.app`, "Contents", "Resources")
   }

test/snapshots/mac/macPackagerTest.js.snap

@@ -437,6 +437,20 @@ exports[`two-package 3`] = `
 `;
 
 exports[`two-package 4`] = `
+[
+  "Info.plist",
+  "MainMenu.nib",
+  "bn.lproj",
+  "chrome_100_percent.pak",
+  "chrome_200_percent.pak",
+  "en.lproj",
+  "icudtl.dat",
+  "resources.pak",
+  "v8_context_snapshot.x86_64.bin",
+]
+`;
+
+exports[`two-package 5`] = `
 {
   "CFBundleDisplayName": "TestApp",
   "CFBundleExecutable": "TestApp",
@@ -483,7 +497,7 @@ exports[`two-package 4`] = `
 }
 `;
 
-exports[`two-package 5`] = `
+exports[`two-package 6`] = `
 [
   "app-update.yml",
   "app.asar",
@@ -493,7 +507,21 @@ exports[`two-package 5`] = `
 ]
 `;
 
-exports[`two-package 6`] = `
+exports[`two-package 7`] = `
+[
+  "Info.plist",
+  "MainMenu.nib",
+  "bn.lproj",
+  "chrome_100_percent.pak",
+  "chrome_200_percent.pak",
+  "en.lproj",
+  "icudtl.dat",
+  "resources.pak",
+  "v8_context_snapshot.arm64.bin",
+]
+`;
+
+exports[`two-package 8`] = `
 {
   "CFBundleDisplayName": "TestApp",
   "CFBundleExecutable": "TestApp",
@@ -540,7 +568,7 @@ exports[`two-package 6`] = `
 }
 `;
 
-exports[`two-package 7`] = `
+exports[`two-package 9`] = `
 [
   "app-update.yml",
   "app.asar",
@@ -550,6 +578,21 @@ exports[`two-package 7`] = `
 ]
 `;
 
+exports[`two-package 10`] = `
+[
+  "Info.plist",
+  "MainMenu.nib",
+  "bn.lproj",
+  "chrome_100_percent.pak",
+  "chrome_200_percent.pak",
+  "en.lproj",
+  "icudtl.dat",
+  "resources.pak",
+  "v8_context_snapshot.arm64.bin",
+  "v8_context_snapshot.x86_64.bin",
+]
+`;
+
 exports[`yarn two package.json w/ native module 1`] = `
 {
   "mac": [

test/src/mac/macPackagerTest.ts

@@ -45,6 +45,9 @@ test.ifMac("two-package", ({ expect }) =>
       checkMacApp: async appDir => {
         const resources = await fs.readdir(path.join(appDir, "Contents", "Resources"))
         expect(resources.filter(it => !it.startsWith(".")).sort()).toMatchSnapshot()
+
+        const ElectronFrameworkResources = await fs.readdir(path.join(appDir, "Contents", "Frameworks", "Electron Framework.framework", "Resources"))
+        expect(ElectronFrameworkResources.filter(it => !it.startsWith(".")).sort()).toMatchSnapshot()
       },
     }
   )