[Feat] - Improve compatibility layer initialization for Mac and Linux (#1214)

* [Feat] - Refactor compatibility layer initialization for Mac and Linux support

- Updated `initializeCompatibilityLayer` to include Linux in the compatibility checks.
- Renamed `setGPTKDefaultOnMacOS` to `setDefaultCompatibilityLayer` for broader applicability.
- Enhanced wine version selection logic to accommodate Proton-GE for Linux and improved compatibility checks for MacOS.
- Added early return conditions to streamline the process based on platform compatibility.

This refactor improves the overall compatibility layer setup for both Mac and Linux environments.

* [Refactor] Update WineManager settings and version fetching logic

- Changed the default count for wine version info fetching from 50 to 15 to optimize performance.
- Renamed Wine-GE settings to Proton-GE in the WineManager component for clarity and consistency.
- Updated the state initialization to reflect the new Proton-GE settings.
- Added a condition to refresh wine version info if no versions are available, enhancing user experience.

* chore: comments

* chore: simplify checks

* chore: rosetta check

---------

Co-authored-by: Flavio F Lima <flavioislima@users.noreply.github.com>

Author: flavioislima

src/backend/utils/compatibility_layers.ts

@@ -597,9 +597,11 @@ export async function initializeCompatibilityLayer() {
     initializationTasks.push(downloadDefaultWine())
   }
 
-  if (isMac) {
-    initializationTasks.push(checkRosettaInstall())
-    initializationTasks.push(setGPTKDefaultOnMacOS())
+  if (isMac || isLinux) {
+    if (isMac) {
+      initializationTasks.push(checkRosettaInstall())
+    }
+    initializationTasks.push(setDefaultCompatibilityLayer())
   }
 
   try {
@@ -616,21 +618,17 @@ export async function downloadDefaultWine() {
   if (isWindows) return null
 
   try {
-    // Refresh wine list
     await updateWineVersionInfos(true)
 
     // Get list of available wine versions
     const availableWine = wineDownloaderInfoStore.get('wine-releases', [])
 
-    // use Wine-GE type if on Linux and GPTK or Wine-Crossover if on Mac
+    // use Proton-GE type if on Linux and GPTK or Wine-Crossover if on Mac
     const isGPTKCompatible = isMac ? await isMacSonomaOrHigher() : false
     const results = await Promise.all(
       availableWine.map(async (version) => {
         if (isLinux) {
-          return (
-            version.type === 'Wine-GE' &&
-            version.version.includes('Wine-GE-Proton')
-          )
+          return version.type === 'Proton-GE'
         }
 
         if (isMac) {
@@ -705,32 +703,46 @@ export async function downloadDefaultWine() {
   }
 }
 
-export async function setGPTKDefaultOnMacOS() {
-  const isGPTKCompatible = await isMacSonomaOrHigher()
-  if (!isGPTKCompatible) {
+export async function setDefaultCompatibilityLayer() {
+  if (!isMac && !isLinux) {
     return
   }
 
+  // For MacOS, check GPTK compatibility
+  if (isMac) {
+    const isGPTKCompatible = await isMacSonomaOrHigher()
+    if (!isGPTKCompatible) {
+      return
+    }
+  }
+
   const { wineVersion: defaultWine } = GlobalConfig.get().getSettings()
 
-  const ignoreList = ['crossover', 'toolkit']
+  // Get target wine type and prefix name based on platform
+  const targetType = isMac ? 'toolkit' : 'proton'
+  const prefixName = isMac ? 'GPTK' : 'Proton'
 
+  // Early return if already using target type
+  const ignoreList = ['crossover', 'toolkit', 'proton']
   if (
     ignoreList.includes(defaultWine.type.toLowerCase()) ||
     defaultWine.name.includes('Toolkit')
   ) {
     return
   }
 
+  // Find target wine version
   const wineList = await GlobalConfig.get().getAlternativeWine()
-  const gptk = wineList.find((wine) => wine.type === 'toolkit')
+  const targetWine = wineList.find((wine) => wine.type === targetType)
+
+  // Set as default if found and valid
+  if (targetWine && existsSync(targetWine.bin)) {
+    logInfo(`Changing wine version to ${targetWine.name}`)
+    GlobalConfig.get().setSetting('wineVersion', targetWine)
 
-  if (gptk && existsSync(gptk.bin)) {
-    logInfo(`Changing wine version to ${gptk.name}`)
-    GlobalConfig.get().setSetting('wineVersion', gptk)
-    // update prefix to use the new one as well
+    // Update prefix path
     const installPath = GlobalConfig.get().getSettings().defaultInstallPath
-    const newPrefix = join(installPath, 'Prefixes', 'GPTK')
+    const newPrefix = join(installPath, 'Prefixes', prefixName)
     GlobalConfig.get().setSetting('winePrefix', newPrefix)
   }
   return

src/backend/wine/manager/utils.ts

@@ -28,7 +28,7 @@ export const wineDownloaderInfoStore = new TypeCheckedStoreBackend(
 
 async function updateWineVersionInfos(
   fetch = false,
-  count = 50
+  count = 15
 ): Promise<WineVersionInfo[]> {
   if (isWindows) {
     return []

src/frontend/screens/WineManager/index.tsx

@@ -28,9 +28,9 @@ export default React.memo(function WineManager(): JSX.Element | null {
   const { refreshWineVersionInfo, platform } = useContext(ContextProvider)
   const isLinux = platform === 'linux'
 
-  const winege: WineManagerUISettings = {
-    type: 'Wine-GE',
-    value: 'winege',
+  const protonge: WineManagerUISettings = {
+    type: 'Proton-GE',
+    value: 'protonge',
     enabled: isLinux
   }
 
@@ -41,13 +41,13 @@ export default React.memo(function WineManager(): JSX.Element | null {
   }
 
   const [repository, setRepository] = useState<WineManagerUISettings>(
-    isLinux ? winege : gamePortingToolkit
+    isLinux ? protonge : gamePortingToolkit
   )
   const [wineManagerSettings, setWineManagerSettings] = useState<
     WineManagerUISettings[]
   >([
-    { type: 'Wine-GE', value: 'winege', enabled: isLinux },
     { type: 'Proton-GE', value: 'protonge', enabled: isLinux },
+    { type: 'Wine-GE', value: 'winege', enabled: isLinux },
     { type: 'Game-Porting-Toolkit', value: 'toolkit', enabled: !isLinux },
     { type: 'Wine-Crossover', value: 'winecrossover', enabled: !isLinux }
   ])
@@ -69,6 +69,10 @@ export default React.memo(function WineManager(): JSX.Element | null {
   // Track the screen view once
   useEffect(() => {
     window.api.trackScreen('Compatibility Layer ')
+
+    if (wineVersions.length === 0) {
+      refreshWineVersionInfo(true)
+    }
   }, [])
 
   useEffect(() => {