fix(desktop): harden cleanup fallback and runtime version detection

Author: zouyonghe

astrbot/core/utils/pip_installer.py

@@ -439,9 +439,17 @@ async def install(
         wheel_only: bool = False,
     ) -> None:
         args = ["install"]
-        pip_install_args = (
-            shlex.split(self.pip_install_arg) if self.pip_install_arg else []
-        )
+        pip_install_args: list[str] = []
+        if self.pip_install_arg:
+            try:
+                pip_install_args = shlex.split(self.pip_install_arg)
+            except ValueError as exc:
+                logger.warning(
+                    "Failed to parse pip_install_arg with shlex (%s). "
+                    "Falling back to legacy whitespace split.",
+                    exc,
+                )
+                pip_install_args = self.pip_install_arg.split()
         requested_requirements: set[str] = set()
         if package_name:
             args.append(package_name)

desktop/lib/backend-manager.js

@@ -858,16 +858,20 @@ class BackendManager {
   }
 
   buildWindowsUnmanagedBackendMatcher(backendConfig) {
-    const expectedImageName = path.basename(backendConfig.cmd || 'python.exe').toLowerCase();
+    const safeBackendConfig =
+      backendConfig && typeof backendConfig === 'object' ? backendConfig : {};
+    const expectedImageName = path
+      .basename(safeBackendConfig.cmd || 'python.exe')
+      .toLowerCase();
     const requireStrictCommandLineCheck =
       this.isGenericWindowsPythonImage(expectedImageName);
     const expectedCommandLineMarkers = [];
-    if (Array.isArray(backendConfig.args) && backendConfig.args.length > 0) {
-      const primaryArg = backendConfig.args[0];
+    if (Array.isArray(safeBackendConfig.args) && safeBackendConfig.args.length > 0) {
+      const primaryArg = safeBackendConfig.args[0];
       if (typeof primaryArg === 'string' && primaryArg) {
         const resolvedPrimaryArg = path.isAbsolute(primaryArg)
           ? primaryArg
-          : path.resolve(backendConfig.cwd || process.cwd(), primaryArg);
+          : path.resolve(safeBackendConfig.cwd || process.cwd(), primaryArg);
         expectedCommandLineMarkers.push(
           this.normalizeWindowsPathForMatch(resolvedPrimaryArg),
         );
@@ -884,6 +888,18 @@ class BackendManager {
     };
   }
 
+  buildFallbackWindowsUnmanagedBackendMatcher() {
+    const fallbackCmdRaw = process.env.ASTRBOT_BACKEND_CMD || 'python.exe';
+    const fallbackCmd = String(fallbackCmdRaw).trim().split(/\s+/, 1)[0] || 'python.exe';
+    return {
+      expectedImageName: path.basename(fallbackCmd).toLowerCase(),
+      // Fallback mode only checks image name to avoid false negatives
+      // when backend config is unavailable in current session.
+      requireStrictCommandLineCheck: false,
+      expectedCommandLineMarkers: [],
+    };
+  }
+
   shouldKillUnmanagedBackendProcess(pid, processInfo, processMatcher) {
     const actualImageName = processInfo.imageName.toLowerCase();
     if (actualImageName !== processMatcher.expectedImageName) {
@@ -944,8 +960,25 @@ class BackendManager {
       `Attempting unmanaged backend cleanup by port=${port} pids=${pids.join(',')}`,
     );
 
-    const backendConfig = this.getBackendConfig();
-    const processMatcher = this.buildWindowsUnmanagedBackendMatcher(backendConfig);
+    let backendConfig = null;
+    try {
+      backendConfig = this.getBackendConfig();
+    } catch (error) {
+      this.log(
+        `Failed to resolve backend config during unmanaged cleanup: ${
+          error instanceof Error ? error.message : String(error)
+        }`,
+      );
+    }
+    const hasBackendConfig = backendConfig && typeof backendConfig === 'object';
+    const processMatcher = hasBackendConfig
+      ? this.buildWindowsUnmanagedBackendMatcher(backendConfig)
+      : this.buildFallbackWindowsUnmanagedBackendMatcher();
+    if (!hasBackendConfig) {
+      this.log(
+        'Backend config is unavailable during unmanaged cleanup; falling back to image-name-only matching.',
+      );
+    }
 
     for (const pid of pids) {
       const processInfo = this.getWindowsProcessInfo(pid);