dsh-plugin 2.0.5 → 2.0.6

package/lib/services/install/preflight.js

@@ -0,0 +1,36 @@
+/**
+ * 安装前置探测：命令是否存在、安装目标是否需要 git。
+ *
+ * 命令缺失时各平台报错形态不同（Windows 只回 GBK 乱码、POSIX 才抛 spawn ENOENT），
+ * 统一在 spawn 前用 which/where 探测，缺失则打 ASCII 标记，前端归类不受编码影响。
+ */
+import { spawnSync } from 'node:child_process';
+/** 缺 git 的失败标记：前端按 `[git-missing]` 归类并给「安装 git」指引（乱码免疫）。 */
+export const GIT_MISSING_DETAIL = '[git-missing] the git CLI was not found on this machine (install git and add it to the PATH, then retry)';
+/** 命令存在性探测：which/where 找 PATH 里的命令。
+ *  探测自身异常（which/where 不可用等）返回 true，不拦截安装，交给装后校验兜底。 */
+export function commandOnPath(cmd, env) {
+    try {
+        const probe = process.platform === 'win32' ? 'where' : 'which';
+        const res = spawnSync(probe, [cmd], { env, encoding: 'utf8', timeout: 3000, stdio: ['ignore', 'pipe', 'pipe'] });
+        return res.status === 0 && (res.stdout ?? '').trim().length > 0;
+    }
+    catch {
+        return true;
+    }
+}
+/** 该安装目标是否需要 git：github: / git+ / git@ 与 owner/repo 简写都要先 clone；
+ *  release 直链（https://…releases/download/…tgz）由 pnpm 直接下载，不需要 git。
+ *  owner/repo 段首必须是字母数字：`./x`、`../x` 这类相对路径是本地目录，不能误判。 */
+export function needsGit(target) {
+    const v = target.trim();
+    if (/^(github:|git\+|git@)/i.test(v))
+        return true;
+    return /^[A-Za-z0-9][A-Za-z0-9._-]*\/[A-Za-z0-9._-]+$/.test(v);
+}
+/** git 源安装前置检查：需要 git 但本机 PATH 里没有 → 返回 [git-missing] 文本；否则 null。 */
+export function gitMissingDetail(target, env) {
+    if (!needsGit(target))
+        return null;
+    return commandOnPath('git', env) ? null : GIT_MISSING_DETAIL;
+}

package/lib/services/install/task-queue.js

@@ -7,7 +7,7 @@
  * 每个任务一次 `dsh plugin --profile <profile> <action> <target>` 子进程，
  * 输出流式收集到任务上；卸载成功时尝试从运行中 loader 即时停用（见 loader.ts）。
  */
-import { spawn, spawnSync } from 'node:child_process';
+import { spawn } from 'node:child_process';
 import { requiredDistributionFiles, resolveBundlePatchFiles } from './package-entry.js';
 import { existsSync, readFileSync, statSync } from 'node:fs';
 import { dirname, join, resolve } from 'node:path';
@@ -20,6 +20,7 @@ import { hasClientUi, isDshPlugin, mountLoadedEntry, removeLoadedEntry } from '.
 import { addPendingRestart, clearPendingRestart } from '../profile/pending-restart.js';
 import { addAllowBuildsKey, bundleIncludes, githubRepoOf, parseAllowBuildsKey, profileDirectory } from '../profile/profile.js';
 import { appendLog } from '../log.js';
+import { commandOnPath, gitMissingDetail } from './preflight.js';
 /** 内存任务注册表，按任务 id 索引。 */
 const tasks = new Map();
 let nextTaskId = 1;
@@ -194,22 +195,6 @@ export function desktopCliBootstrap(entry) {
     }
     return null;
 }
-/**
- * dsh 命令存在性探测：which/where 找 PATH 里的 dsh。
- * 命令缺失时各平台报错形态不同（Windows shell 只回 GBK 乱码、POSIX 才抛 spawn ENOENT），
- * 统一在 spawn 前用 which/where 探测，命令不存在则直接失败并打 ASCII 标记 [dsh-missing]（乱码免疫）。
- * 探测自身异常（which/where 不可用等）不拦截安装，交给装后校验兜底。
- */
-function dshOnPath(env) {
-    try {
-        const probe = process.platform === 'win32' ? 'where' : 'which';
-        const res = spawnSync(probe, ['dsh'], { env, encoding: 'utf8', timeout: 3000, stdio: ['ignore', 'pipe', 'pipe'] });
-        return res.status === 0 && (res.stdout ?? '').trim().length > 0;
-    }
-    catch {
-        return true;
-    }
-}
 function stopChild(child) {
     if (process.platform === 'win32' && child.pid !== undefined) {
         const killer = spawn('taskkill', ['/pid', String(child.pid), '/t', '/f'], { stdio: 'ignore' });
@@ -400,7 +385,7 @@ function spawnMutation(options) {
             // 独立 CLI 场景（file: 'dsh'，宿主加载器走 node bin 不经过这里）：先探测 dsh 是否在 PATH。
             // 命令缺失时各平台报错形态不同（GBK 乱码 / spawn ENOENT），统一前置拦截并打 ASCII 标记
             // [dsh-missing]，前端据此精准提示「dsh 未加入 PATH」而不是笼统的插件失败。
-            if (invocation.file === 'dsh' && !dshOnPath(env)) {
+            if (invocation.file === 'dsh' && !commandOnPath('dsh', env)) {
                 const detail = '[dsh-missing] the dsh CLI was not found on this machine (DeepSeek Harness not installed, or dsh not on the PATH)';
                 if (task) {
                     task.status = 'failed';
@@ -411,6 +396,21 @@ function spawnMutation(options) {
                 resolveOnce({ exitCode: 1, timedOut: false, error: detail, stdout: '', stderr: detail });
                 return;
             }
+            // git 源安装：pnpm 需要调用 git 解析仓库；Windows 缺 git 只回 GBK 乱码，前端认不出来。
+            // 前置探测并打 ASCII 标记 [git-missing]，给出装 git 的指引而不是笼统的安装失败。
+            if (!isGlobalNpm && action !== 'remove') {
+                const gitMissing = gitMissingDetail(target, env);
+                if (gitMissing !== null) {
+                    if (task) {
+                        task.status = 'failed';
+                        task.exitCode = null;
+                        task.progress = 0;
+                        pushLine(task, gitMissing);
+                    }
+                    resolveOnce({ exitCode: 1, timedOut: false, error: gitMissing, stdout: '', stderr: gitMissing });
+                    return;
+                }
+            }
             child = spawn(invocation.file, attemptArgs, {
                 cwd: invocation.cwd,
                 // invocation.env 是「必须叠加」的变量（打包桌面端的 ELECTRON_RUN_AS_NODE）：

package/lib/types/services/install/preflight.d.ts

@@ -0,0 +1,11 @@
+/** 缺 git 的失败标记：前端按 `[git-missing]` 归类并给「安装 git」指引（乱码免疫）。 */
+export declare const GIT_MISSING_DETAIL = "[git-missing] the git CLI was not found on this machine (install git and add it to the PATH, then retry)";
+/** 命令存在性探测：which/where 找 PATH 里的命令。
+ *  探测自身异常（which/where 不可用等）返回 true，不拦截安装，交给装后校验兜底。 */
+export declare function commandOnPath(cmd: string, env?: NodeJS.ProcessEnv): boolean;
+/** 该安装目标是否需要 git：github: / git+ / git@ 与 owner/repo 简写都要先 clone；
+ *  release 直链（https://…releases/download/…tgz）由 pnpm 直接下载，不需要 git。
+ *  owner/repo 段首必须是字母数字：`./x`、`../x` 这类相对路径是本地目录，不能误判。 */
+export declare function needsGit(target: string): boolean;
+/** git 源安装前置检查：需要 git 但本机 PATH 里没有 → 返回 [git-missing] 文本；否则 null。 */
+export declare function gitMissingDetail(target: string, env?: NodeJS.ProcessEnv): string | null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin",
-  "version": "2.0.5",
+  "version": "2.0.6",
   "description": "A community plugin marketplace for DeepSeek Harness, built to the official plugin spec — browse, search and install 10000+ human-curated community plugins without leaving the app. · DeepSeek Harness 社区插件市场（遵循官方开发规范）：10000+ 人工精选社区插件，每日更新。",
   "type": "module",
   "main": "lib/index.js",

package/src/client/logic/installed.ts

@@ -48,6 +48,12 @@ export interface InstalledItem {
   dshCapable: boolean
 }
 
+/** 本地路径安装（file:/link:/绝对或相对路径）：身份在本机，不能按 npm 包名认领目录条目。 */
+function isLocalSpec(value: string): boolean {
+  const v = value.trim()
+  return /^(file|link):/i.test(v) || v.startsWith('.') || v.startsWith('/') || /^[A-Za-z]:[\\/]/.test(v)
+}
+
 /** 安装时记录的目录信号（按仓库小写 key 查表）。 */
 function signalOf(repo: string | null, versions: Record<string, InstalledVersionSignal>): InstalledVersionSignal | null {
   if (!repo) return null
@@ -66,15 +72,13 @@ export function installedNameOf(
   for (const [name, spec] of Object.entries(installed)) {
     if (repoFromInstallTarget(spec).toLowerCase() === needle) return name
   }
-  // npm 安装：profile 依赖 key 直接是 npm 包名。包名来源两处——目录数据（npmPackage），
-  // 或服务端安装时持久化的映射（versions[repo].npmPackage，覆盖组织 scope 与 GitHub
-  // 用户名不一致、目录未下发包名的场景）；命中任一并依赖 key 真实存在即视为已安装。
-  // 只认 npm 安装的依赖（spec 无法解析出仓库身份）：git/本地安装的依赖身份由仓库匹配决定，
-  // 包名同名不代表同仓库——否则「git 装 A 仓库」会被「npm 包名同名但仓库不同」的目录条目
-  // 误认领成已安装（假阳性），installedItems 里出现同 key 双条目，排序一变化就渲染出幽灵行。
+  // 只认 npm 安装的依赖（spec 无法解析出仓库身份、也不是本地路径）：git 安装的依赖身份由仓库
+  // 匹配决定，file:/link: 安装的本地插件身份在本机 —— 包名同名不代表同仓库，否则「本地装 A」
+  // 会被目录里同名的 B 认领，已安装页显示成 B 的信息。
   const pkg = ((plugin.source?.npmPackage || versions[repo.toLowerCase()]?.npmPackage) ?? '').toLowerCase()
   if (pkg) {
     for (const [name, spec] of Object.entries(installed)) {
+      if (isLocalSpec(spec)) continue
       if (isRepoLike(repoFromInstallTarget(spec))) continue
       if (name.toLowerCase() === pkg) return name
     }

package/src/server/services/install/preflight.ts

@@ -0,0 +1,37 @@
+/**
+ * 安装前置探测：命令是否存在、安装目标是否需要 git。
+ *
+ * 命令缺失时各平台报错形态不同（Windows 只回 GBK 乱码、POSIX 才抛 spawn ENOENT），
+ * 统一在 spawn 前用 which/where 探测，缺失则打 ASCII 标记，前端归类不受编码影响。
+ */
+import { spawnSync } from 'node:child_process'
+
+/** 缺 git 的失败标记：前端按 `[git-missing]` 归类并给「安装 git」指引（乱码免疫）。 */
+export const GIT_MISSING_DETAIL = '[git-missing] the git CLI was not found on this machine (install git and add it to the PATH, then retry)'
+
+/** 命令存在性探测：which/where 找 PATH 里的命令。
+ *  探测自身异常（which/where 不可用等）返回 true，不拦截安装，交给装后校验兜底。 */
+export function commandOnPath(cmd: string, env?: NodeJS.ProcessEnv): boolean {
+  try {
+    const probe = process.platform === 'win32' ? 'where' : 'which'
+    const res = spawnSync(probe, [cmd], { env, encoding: 'utf8', timeout: 3000, stdio: ['ignore', 'pipe', 'pipe'] })
+    return res.status === 0 && (res.stdout ?? '').trim().length > 0
+  } catch {
+    return true
+  }
+}
+
+/** 该安装目标是否需要 git：github: / git+ / git@ 与 owner/repo 简写都要先 clone；
+ *  release 直链（https://…releases/download/…tgz）由 pnpm 直接下载，不需要 git。
+ *  owner/repo 段首必须是字母数字：`./x`、`../x` 这类相对路径是本地目录，不能误判。 */
+export function needsGit(target: string): boolean {
+  const v = target.trim()
+  if (/^(github:|git\+|git@)/i.test(v)) return true
+  return /^[A-Za-z0-9][A-Za-z0-9._-]*\/[A-Za-z0-9._-]+$/.test(v)
+}
+
+/** git 源安装前置检查：需要 git 但本机 PATH 里没有 → 返回 [git-missing] 文本；否则 null。 */
+export function gitMissingDetail(target: string, env?: NodeJS.ProcessEnv): string | null {
+  if (!needsGit(target)) return null
+  return commandOnPath('git', env) ? null : GIT_MISSING_DETAIL
+}

package/src/server/services/install/task-queue.ts

@@ -22,6 +22,7 @@ import { hasClientUi, isDshPlugin, mountLoadedEntry, removeLoadedEntry } from '.
 import { addPendingRestart, clearPendingRestart } from '../profile/pending-restart.ts'
 import { addAllowBuildsKey, bundleIncludes, githubRepoOf, parseAllowBuildsKey, profileDirectory } from '../profile/profile.ts'
 import { appendLog } from '../log.ts'
+import { commandOnPath, gitMissingDetail } from './preflight.ts'
 
 /** 内存任务注册表，按任务 id 索引。 */
 const tasks = new Map<number, InstallTask>()
@@ -212,22 +213,6 @@ export function desktopCliBootstrap(entry: string): string | null {
   return null
 }
 
-/**
- * dsh 命令存在性探测：which/where 找 PATH 里的 dsh。
- * 命令缺失时各平台报错形态不同（Windows shell 只回 GBK 乱码、POSIX 才抛 spawn ENOENT），
- * 统一在 spawn 前用 which/where 探测，命令不存在则直接失败并打 ASCII 标记 [dsh-missing]（乱码免疫）。
- * 探测自身异常（which/where 不可用等）不拦截安装，交给装后校验兜底。
- */
-function dshOnPath(env?: NodeJS.ProcessEnv): boolean {
-  try {
-    const probe = process.platform === 'win32' ? 'where' : 'which'
-    const res = spawnSync(probe, ['dsh'], { env, encoding: 'utf8', timeout: 3000, stdio: ['ignore', 'pipe', 'pipe'] })
-    return res.status === 0 && (res.stdout ?? '').trim().length > 0
-  } catch {
-    return true
-  }
-}
-
 function stopChild(child: ReturnType<typeof spawn>): void {
   if (process.platform === 'win32' && child.pid !== undefined) {
     const killer = spawn('taskkill', ['/pid', String(child.pid), '/t', '/f'], { stdio: 'ignore' })
@@ -452,7 +437,7 @@ function spawnMutation(options: {
       // 独立 CLI 场景（file: 'dsh'，宿主加载器走 node bin 不经过这里）：先探测 dsh 是否在 PATH。
       // 命令缺失时各平台报错形态不同（GBK 乱码 / spawn ENOENT），统一前置拦截并打 ASCII 标记
       // [dsh-missing]，前端据此精准提示「dsh 未加入 PATH」而不是笼统的插件失败。
-      if (invocation.file === 'dsh' && !dshOnPath(env)) {
+      if (invocation.file === 'dsh' && !commandOnPath('dsh', env)) {
         const detail = '[dsh-missing] the dsh CLI was not found on this machine (DeepSeek Harness not installed, or dsh not on the PATH)'
         if (task) {
           task.status = 'failed'
@@ -463,6 +448,21 @@ function spawnMutation(options: {
         resolveOnce({ exitCode: 1, timedOut: false, error: detail, stdout: '', stderr: detail })
         return
       }
+      // git 源安装：pnpm 需要调用 git 解析仓库；Windows 缺 git 只回 GBK 乱码，前端认不出来。
+      // 前置探测并打 ASCII 标记 [git-missing]，给出装 git 的指引而不是笼统的安装失败。
+      if (!isGlobalNpm && action !== 'remove') {
+        const gitMissing = gitMissingDetail(target, env)
+        if (gitMissing !== null) {
+          if (task) {
+            task.status = 'failed'
+            task.exitCode = null
+            task.progress = 0
+            pushLine(task, gitMissing)
+          }
+          resolveOnce({ exitCode: 1, timedOut: false, error: gitMissing, stdout: '', stderr: gitMissing })
+          return
+        }
+      }
       child = spawn(invocation.file, attemptArgs, {
         cwd: invocation.cwd,
         // invocation.env 是「必须叠加」的变量（打包桌面端的 ELECTRON_RUN_AS_NODE）：