refactor(openclaw): remove findNpmPath and hardcoded fallback paths

- Inline findNpmPath into its two call sites (install, uninstall)
- Remove possiblePaths filesystem fallback from findOpenClawBinary —
  openclaw is installed via npm global, so it's always in PATH
- findOpenClawBinary now delegates entirely to findExecutableInEnv

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: DeJeune

src/main/services/OpenClawService.ts

@@ -228,15 +228,6 @@ class OpenClawService {
     return 'https://git-scm.com/downloads'
   }
 
-  /**
-   * Locate the npm binary in the user's shell environment.
-   * Returns the resolved path, or 'npm' as a fallback for spawn.
-   */
-  private async findNpmPath(): Promise<string> {
-    const { path: npmPath } = await findExecutableInEnv('npm')
-    return npmPath || 'npm'
-  }
-
   /**
    * Send install progress to renderer
    */
@@ -255,7 +246,8 @@ class OpenClawService {
     const packageName = inChina ? '@qingchencloud/openclaw-zh@latest' : 'openclaw@latest'
     const registryArg = inChina ? `--registry=${NPM_MIRROR_CN}` : ''
 
-    const npmPath = await this.findNpmPath()
+    const { path: npmPath_ } = await findExecutableInEnv('npm')
+    const npmPath = npmPath_ || 'npm'
 
     const npmArgs = ['install', '-g', packageName]
     if (registryArg) npmArgs.push(registryArg)
@@ -378,7 +370,8 @@ class OpenClawService {
       await this.stopGateway()
     }
 
-    const npmPath = await this.findNpmPath()
+    const { path: npmPath_ } = await findExecutableInEnv('npm')
+    const npmPath = npmPath_ || 'npm'
 
     const npmArgs = ['uninstall', '-g', 'openclaw', '@qingchencloud/openclaw-zh']
 
@@ -910,37 +903,9 @@ class OpenClawService {
     return []
   }
 
-  /**
-   * Find OpenClaw binary in PATH or common locations
-   * On Windows, npm global packages create .cmd wrapper scripts, not .exe files
-   */
   private async findOpenClawBinary(): Promise<string | null> {
-    const home = os.homedir()
-
     const { path: binaryPath } = await findExecutableInEnv('openclaw')
-    if (binaryPath) {
-      return binaryPath
-    }
-
-    // Check common filesystem locations as fallback
-    const binaryName = isWin ? 'openclaw.exe' : 'openclaw'
-    const possiblePaths = isWin
-      ? [path.join(home, 'AppData', 'Local', 'openclaw', binaryName), path.join(home, '.openclaw', 'bin', binaryName)]
-      : [
-          path.join(home, '.openclaw', 'bin', binaryName),
-          path.join(home, '.local', 'bin', binaryName),
-          `/usr/local/bin/${binaryName}`,
-          `/opt/homebrew/bin/${binaryName}`
-        ]
-
-    for (const p of possiblePaths) {
-      if (fs.existsSync(p)) {
-        logger.info('Found OpenClaw binary at: ' + p)
-        return p
-      }
-    }
-
-    return null
+    return binaryPath
   }
 
   /**