dsh-plugin 1.6.11 → 1.6.12

package/src/client/logic/urls.ts

@@ -9,9 +9,9 @@
  */
 import type { EnvInfo, HubPlugin, LocaleId } from '../types.ts'
 import { GITHUB_URL, HUB_REPO, PLUGIN_VERSION, SITE_URL } from './constants.ts'
-import { MAX_CORE_CHARS, classifyFailure, coreErrorCode, pnpmPolicyHintOf, summarizeError } from './failures.ts'
+import { MAX_CORE_CHARS, actionWordEn, actionWordZh, classifyFailure, coreErrorCode, pnpmPolicyHintOf, summarizeError } from './failures.ts'
 import { executedCommandOf } from './install-command.ts'
-import type { FailureKind } from './failures.ts'
+import type { FailureAction, FailureKind } from './failures.ts'
 
 /** dsh-plugin.org 中文页挂在 /zh/ 前缀下，英文在根路径。 */
 export function langPathOf(lang: LocaleId): string {
@@ -41,18 +41,18 @@ export function pluginSiteUrl(repo: string): string {
   return siteUrl(`plugins/${repo}`)
 }
 
-/** 按失败类型给出简洁的错误原因标题（对外用英文）：
- *  标题直接点明问题出在哪一侧（构建白名单/分发物/本机 npm/网络/插件侧），
- *  让作者与用户扫一眼列表就能分流，而不是笼统的 Install/Remove 动作词。 */
-function reasonTitleOf(kind: FailureKind): string {
+/** 错误原因标题（对外英文）：点明问题出在哪一侧，便于 issue 列表分流；
+ *  动作级原因（未分类/超时…）带动作词，卸载失败不写成 install failed。 */
+function reasonTitleOf(kind: FailureKind, action: FailureAction): string {
+  const word = actionWordEn(action)
   switch (kind) {
-    case 'npmTooOld': return 'npm too old to install'
+    case 'npmTooOld': return `npm too old for this ${word}`
     case 'dshMissing': return 'dsh command not found on the user machine'
     case 'pnpmMissing': return 'pnpm command not found on the user machine'
     case 'pnpmStore': return 'pnpm store version mismatch on the user machine'
-    case 'pnpmWorkspace': return 'pnpm workspace-root check blocked the install on the user machine'
+    case 'pnpmWorkspace': return `pnpm workspace-root check blocked the ${word} on the user machine`
     case 'originRejected': return 'request origin rejected by the local service on the user machine'
-    case 'pnpmPolicy': return 'pnpm supply-chain policy blocked the install on the user machine'
+    case 'pnpmPolicy': return `pnpm supply-chain policy blocked the ${word} on the user machine`
     case 'pnpmUnusedPatch': return 'stale pnpm patch entry on the user machine'
     case 'pnpmMissingDep': return 'dependency not found on the npm registry on the user machine'
     case 'registryVersionStale': return 'requested version not available on the npm registry in use on the user machine'
@@ -63,12 +63,13 @@ function reasonTitleOf(kind: FailureKind): string {
     case 'pnpmIgnoredBuild': return 'build scripts blocked by pnpm allowlist'
     case 'pluginPrepare': return 'plugin distribution incomplete'
     case 'network': return 'network failure on the user side'
-    case 'hubState': return 'stale install state in the plugin hub'
-    case 'timeout': return 'install timed out'
+    case 'hostCliSpawn': return 'the DSH host could not be launched on the user machine'
+    case 'hubState': return `stale ${word} state in the plugin hub`
+    case 'timeout': return `${word} timed out`
     case 'profileLock': return 'DSH profile is locked by another process'
     case 'pluginMissingDep': return 'a dependency of the plugin is not published on npm'
-    case 'hub': return 'install failed (cause unclassified)'
-    default: return 'plugin install failed'
+    case 'hub': return `${word} failed (cause unclassified)`
+    default: return `plugin ${word} failed`
   }
 }
 
@@ -81,14 +82,16 @@ function reasonTitleOf(kind: FailureKind): string {
  * attempts（尝试过的安装方式，npm 反查 + 实际执行命令）：作者看到我们查过/试过的命令，
  * 组织 scope 与 GitHub 用户名不一致时也能直接指认正确的 npm 包名。
  */
-export function pluginIssueUrl(repo: string, message: string, env?: EnvInfo | null, command?: string, attempts?: string[]): string {
+export function pluginIssueUrl(repo: string, message: string, env?: EnvInfo | null, command?: string, attempts?: string[], action: FailureAction = 'install'): string {
   // 原因判定：按失败类型归类，便于作者判断问题是否出在插件侧。
   // [packaging]（预检/装后校验拦截）单独列：无论走 npm 还是 git 通道，都是分发物不完整
   // （package.json 声明的文件在发布物里缺失），作者照此补齐即可。
   const kind = classifyFailure(message)
   const executed = executedCommandOf(command, attempts)
+  // 动作词：标题与正文的「安装/更新/卸载」统一取自这里
+  const zhWord = actionWordZh(action)
   // 标题 = 错误原因（非动作词），让 issue 列表一眼可分流
-  const title = `[dsh-plugin.org | dsh-plugin-hub] ${reasonTitleOf(kind)}: ${repo}`
+  const title = `[dsh-plugin.org | dsh-plugin-hub] ${reasonTitleOf(kind, action)}: ${repo}`
   const reason = /\[packaging\]/i.test(message)
     ? 'plugin distribution is incomplete — a file the package needs (its entry from main / exports, an entry a bundled patch mounts, or the patch declared by dsh.bundle.patch) is missing from the published package (github tarball or npm package); please commit build output or publish a complete package'
       : /\[registry-missing\]/i.test(message)
@@ -123,16 +126,15 @@ export function pluginIssueUrl(repo: string, message: string, env?: EnvInfo | nu
       `- Cause: ${reason}`,
       ...(code ? [`- Key error: \`${code}\``] : []),
       '',
-      // 来源说明：标题（链官网）+ 一句来源（链仓库）+ 实际执行的安装命令（含 --profile）与执行结果，链接由常量动态拼接
-      `## [DSH Plugin 插件市场](${SITE_URL}) · 安装 Plugin 失败错误信息`,
-      `本错误信息由 [dsh-plugin-hub](${GITHUB_URL}) 插件市场的安装程序自动生成，随本次安装失败一并提交。`,
-      `- 实际执行的安装命令：${executed ? `\`${executed}\`` : 'unknown (no executed command recorded)'}`,
-      `- 执行结果：安装失败，未能安装该插件。`,
-      // 尝试过的安装方式（npm 反查 + 实际执行命令，按先后顺序）：作者据此反推正确的
-      // npm 包名 —— 组织 scope 与 GitHub 用户名不一致时仅凭仓库名猜不到，作者看到我们
-      // 查过/试过的命令就能直接指认
+      // 来源说明：标题（链官网）+ 一句来源（链仓库）+ 实际执行的命令（含 --profile）与执行结果，链接由常量动态拼接
+      `## [DSH Plugin 插件市场](${SITE_URL}) · ${zhWord} Plugin 失败错误信息`,
+      `本错误信息由 [dsh-plugin-hub](${GITHUB_URL}) 插件市场的安装程序自动生成，随本次${zhWord}失败一并提交。`,
+      `- 实际执行的${zhWord}命令：${executed ? `\`${executed}\`` : 'unknown (no executed command recorded)'}`,
+      `- 执行结果：${zhWord}失败，未能${zhWord}该插件。`,
+      // 尝试过的通道（npm 反查 + 实际执行命令，按先后顺序）：作者据此指认正确的 npm 包名。
+      // 不写「安装」：卸载失败也走这条正文。
       ...(attempts && attempts.length > 0
-        ? ['', '## Attempted install channels（已尝试的安装方式）', ...attempts.map((a) => `- ${a}`)]
+        ? ['', '## Attempted channels（已尝试的方式）', ...attempts.map((a) => `- ${a}`)]
         : []),
       // [packaging] 场景：说明三种官方安装通道（npm 包 / git 直装 / 预构建 release .tgz），当前插件在对应通道下分发不完整
       ...(/\[packaging\]/i.test(message)
@@ -210,19 +212,22 @@ export function hubIssueUrl(
   attempts?: string[],
   /** 见 HubIssueVariant；缺省 'triage'，既有正文保持不变 */
   variant: HubIssueVariant = 'triage',
+  /** 上报动作：标题与正文按它取词 */
+  action: FailureAction = 'install',
 ): string {
   const kind = classifyFailure(message)
   const executed = executedCommandOf(command, attempts)
+  const zhWord = actionWordZh(action)
   const repo = pluginRepo && pluginRepo.trim() !== '' ? pluginRepo.trim() : 'unknown'
   const title = variant === 'misjudge'
     // 标题就点明「可能是误判」，维护者一眼能区分它和未分类兜底
     ? `[dsh-plugin-hub] possible misjudgment: ${repo}`
-    : `[dsh-plugin-hub] ${reasonTitleOf(kind)}: ${repo}`
+    : `[dsh-plugin-hub] ${reasonTitleOf(kind, action)}: ${repo}`
   const code = coreErrorCode(message)
   const build = (coreChars: number): string => {
     const body = [
       '## Summary',
-      `- Auto-classified cause: ${reasonTitleOf(kind)}`,
+      `- Auto-classified cause: ${reasonTitleOf(kind, action)}`,
       `- Target plugin: \`${repo}\``,
       ...(code ? [`- Key error: \`${code}\``] : []),
       // 为什么投递到这里而非插件仓库：只有确有证据是插件自身构建失败（pluginPrepare）才投插件仓库
@@ -236,12 +241,12 @@ export function hubIssueUrl(
         ]
         : ['- Why filed here: the installer only routes a failure to the plugin repo when there is positive evidence the plugin build itself failed (pluginPrepare). This failure does not match that, so it is routed here for triage.']),
       '',
-      `## [DSH Plugin 插件市场](${SITE_URL}) · 安装 Plugin 失败错误信息`,
-      `本错误信息由 [dsh-plugin-hub](${GITHUB_URL}) 插件市场的安装程序自动生成，随本次安装失败一并提交。`,
-      `- 实际执行的安装命令：${executed ? `\`${executed}\`` : 'unknown (no executed command recorded)'}`,
-      `- 执行结果：安装失败，未能安装该插件。`,
+      `## [DSH Plugin 插件市场](${SITE_URL}) · ${zhWord} Plugin 失败错误信息`,
+      `本错误信息由 [dsh-plugin-hub](${GITHUB_URL}) 插件市场的安装程序自动生成，随本次${zhWord}失败一并提交。`,
+      `- 实际执行的${zhWord}命令：${executed ? `\`${executed}\`` : 'unknown (no executed command recorded)'}`,
+      `- 执行结果：${zhWord}失败，未能${zhWord}该插件。`,
       ...(attempts && attempts.length > 0
-        ? ['', '## Attempted install channels（已尝试的安装方式）', ...attempts.map((a) => `- ${a}`)]
+        ? ['', '## Attempted channels（已尝试的方式）', ...attempts.map((a) => `- ${a}`)]
         : []),
       '',
       '## Environment',

package/src/server/services/install/registry-check.ts

@@ -3,20 +3,9 @@
  * Website: https://dsh-plugin.org
  * GitHub: https://github.com/dshplugin/dsh-plugin-hub
  *
- * 官方 registry 存在性复核：把「包在 registry 上不存在」这类失败的责任方分开。
- *
- * 背景：安装失败日志出现 `ERR_PNPM_FETCH_404` / `Not Found - 404` 时，光看日志分不清三种情形：
- *   ① 缺的包是 **profile 自己声明的依赖**（历史残留条目，指向一个从未发布的包）—— 用户环境问题：
- *      pnpm 每次解析依赖都会先卡在它上面，任何插件都装不进来（issue #73），提示删掉该条目即可；
- *   ② 缺的包来自**插件的依赖树**，且该包在官方 registry 上也不存在 —— 作者侧问题（漏声明依赖、
- *      依赖被删除或从未发布），应引导用户投插件作者仓库；
- *   ③ 缺的包来自插件依赖树，但官方 registry 上有 —— 用户本机 registry 指向的镜像/私有源未同步
- *      该包，属用户环境问题，提示换源即可。
- * 故本模块先排除 ①（读 profile 的 package.json），再用**官方** registry（registry.npmjs.org，
- * 绕开本机可能配置的镜像）复查存在性，产出 ② / ③ 的标记行。
- *
- * 与 npm-check.ts 同构：本模块只产出「标记行」，由 task-queue 追加到任务输出，前端按标记分类。
- * fetch 与 profile 目录都可注入，便于单测覆盖全部结论而不依赖网络。
+ * 官方 registry 存在性复核：用官方源（绕开本机镜像）复查 404 的包，产出标记行区分
+ * 「profile 残留依赖」「插件依赖在官方源也不存在（作者侧）」「本机镜像未同步」三种情形；
+ * 标记由 task-queue 追加到任务输出，前端按标记归类。fetch 与 profile 目录可注入，便于单测。
  */
 import { readFileSync } from 'node:fs'
 import { join } from 'node:path'

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

@@ -149,6 +149,13 @@ function pushLine(task: InstallTask, line: string): void {
   }
 }
 
+/** spawn 的 cwd：profile 目录（asar 路径不能作 cwd）；目录不存在时回退进程工作目录，
+ *  否则 Node 报的是 `spawn <file> ENOENT`，与「可执行文件不存在」无法区分。 */
+function spawnCwdFor(profile: string): string {
+  const dir = profileDirectory(profile)
+  return existsSync(dir) ? dir : process.cwd()
+}
+
 function cliInvocation(profile: string): Invocation {
   const entry = process.argv[1]
   if (entry !== undefined) {
@@ -162,8 +169,8 @@ function cliInvocation(profile: string): Invocation {
       return {
         file: process.execPath,
         prefixArgs: ['--expose-internals', bootstrap],
-        // 工作目录用 profile 目录（真实存在的目录）：asar 内的路径不能作为 spawn 的 cwd
-        cwd: profileDirectory(profile),
+        // cwd 用 profile 目录（asar 路径不能作 cwd），不存在时由 spawnCwdFor 回退
+        cwd: spawnCwdFor(profile),
         useShell: false,
         // Electron 可执行文件只有带这个变量才按 Node 运行引导脚本
         env: { ELECTRON_RUN_AS_NODE: '1' },
@@ -475,15 +482,19 @@ function spawnMutation(options: {
         stdio: ['ignore', 'pipe', 'pipe'],
       })
     } catch (error) {
+      const detail = error instanceof Error ? error.message : String(error)
+      // 宿主自身入口起不来 → 补标记，前端据此归因
+      const marker = hostSpawnMarker(invocation, error)
       if (task) {
         task.status = 'failed'
         task.exitCode = null
-        pushLine(task, `[error] ${error instanceof Error ? error.message : String(error)}`)
+        pushLine(task, `[error] ${detail}`)
+        if (marker !== null) pushLine(task, marker)
       }
       resolveOnce({
         exitCode: null,
         timedOut: false,
-        error: error instanceof Error ? error.message : String(error),
+        error: marker === null ? detail : `${detail}\n${marker}`,
         stdout,
         stderr,
       })
@@ -554,12 +565,15 @@ function spawnMutation(options: {
     })
 
     child.once('error', (error) => {
+      // 同同步抛出那条路径：宿主自身入口起不来 → 补标记
+      const marker = hostSpawnMarker(invocation, error)
       if (task && task.status !== 'cancelled') {
         task.status = 'failed'
         task.exitCode = null
         pushLine(task, `[error] ${error.message}`)
+        if (marker !== null) pushLine(task, marker)
       }
-      settleOnce({ exitCode: null, timedOut: false, error: error.message, stdout, stderr })
+      settleOnce({ exitCode: null, timedOut: false, error: marker === null ? error.message : `${error.message}\n${marker}`, stdout, stderr })
     })
 
     child.once('close', (code) => {
@@ -712,10 +726,8 @@ function annotateNpmTooLow(result: InstallResult, options: {
   return { ...result, stderr: `${result.stderr}\n${line}`, error: result.error ? `${result.error}\n${line}` : line }
 }
 
-/** 包在 registry 不存在（404）→ 用官方源复查一次，把「作者依赖缺失/包被删」与「本机镜像源滞后」
- *  分开（两者的提示与去向不同：前者引导投作者、后者只提示换源）。检测逻辑在独立的
- *  registry-check.ts；这里之所以是异步，是因为要联网查官方 registry —— 只在安装失败且日志含 404
- *  时发生，代价可忽略。 */
+/** 安装失败且日志含 404 时，用官方源复查一次并注入标记（逻辑见 registry-check.ts）。
+ *  异步是因为要联网；只在失败路径上发生。 */
 async function annotateRegistryMissing(result: InstallResult, options: {
   action: 'add' | 'remove' | 'update'
   profile: string
@@ -730,6 +742,17 @@ async function annotateRegistryMissing(result: InstallResult, options: {
   return { ...result, stderr: `${result.stderr}\n${line}`, error: result.error ? `${result.error}\n${line}` : line }
 }
 
+/**
+ * 宿主自身 CLI 调不起来时的 ASCII 标记行：spawn 我们解析出的宿主入口报 ENOENT 时返回
+ * `[host-spawn]`，供前端单独归因、不引导提 Issue。PATH 上的 dsh/npm 各有自己的标记，返回 null。
+ * 导出供单测覆盖判据。
+ */
+export function hostSpawnMarker(invocation: Invocation, error: unknown): string | null {
+  if ((error as NodeJS.ErrnoException | null)?.code !== 'ENOENT') return null
+  if (invocation.file === 'dsh' || invocation.file === 'npm') return null
+  return `[host-spawn] the DSH host could not be launched (${invocation.file}: ENOENT)`
+}
+
 /**
  * Run a plugin mutation with one recovery path: when `add` fails because
  * pnpm's allowBuilds gate blocks a build script — either the git-hosted