dsh-plugin 1.6.7 → 1.6.9

package/src/server/services/install/operation-log.ts

@@ -0,0 +1,46 @@
+/**
+ * DSH Plugin Hub — the community plugin marketplace for DeepSeek Harness.
+ * Website: https://dsh-plugin.org
+ * GitHub: https://github.com/dshplugin/dsh-plugin-hub
+ *
+ * 失败回捞：DSH 有时只把一行摘要（`dsh: plugin command failed; diagnostics: <path>`）
+ * 回显到控制台，真正的 pnpm 报错写进该路径的 operation 日志。这里从输出里解析出该路径
+ * 并读取其尾部，补进失败信息 —— 否则前端拿不到任何错误特征，只能把失败归为「未知」。
+ * 只允许读 DSH 自己的 operation 日志（路径含 .plugin-manager 且以 pnpm.log 结尾），
+ * 防止插件把任意路径打印进日志造成越权读取；任何异常都静默降级（返回 null），
+ * 不改变原有失败处理。
+ */
+import { readFileSync } from 'node:fs'
+import { CAPTURE_LIMIT_BYTES } from './install-types.ts'
+
+/**
+ * 从 DSH 输出里解析 operation 日志路径（`diagnostics: <path>`）。
+ * 只认 DSH 的 operation 日志：路径含 `.plugin-manager` 且以 `pnpm.log` 结尾；
+ * 不是这种路径（或压根没有该行）返回 null，调用方据此跳过回捞。
+ */
+export function operationLogPathOf(output: string): string | null {
+  for (const raw of output.split(/\r?\n/)) {
+    const m = /diagnostics:\s*"?(.+?)"?\s*$/i.exec(raw.trim())
+    if (m === null) continue
+    const path = m[1].trim()
+    if (!path.includes('.plugin-manager') || !path.toLowerCase().endsWith('pnpm.log')) continue
+    return path
+  }
+  return null
+}
+
+/**
+ * 回捞 operation 日志尾部，作为失败详情返回；拿不到（无路径 / 文件不存在 / 读失败 /
+ * 内容为空）返回 null。尾部截到与子进程输出相同的上限，避免超大日志撑爆内存与消息。
+ */
+export function backfillOperationLog(output: string): string | null {
+  const path = operationLogPathOf(output)
+  if (path === null) return null
+  try {
+    const text = readFileSync(path, 'utf8').trim()
+    if (text === '') return null
+    return text.length > CAPTURE_LIMIT_BYTES ? text.slice(-CAPTURE_LIMIT_BYTES) : text
+  } catch {
+    return null
+  }
+}

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

@@ -14,6 +14,7 @@ import { dirname, join, resolve } from 'node:path'
 import type { InstallResult, InstallTask, Invocation, QueueItem } from './install-types.ts'
 import { CAPTURE_LIMIT_BYTES, MAX_TASKS, MAX_TASK_LINES } from './install-types.ts'
 import { npmTooLowMarker } from './npm-check.ts'
+import { backfillOperationLog } from './operation-log.ts'
 import { cleanLine, estimateProgress } from '../profile/progress.ts'
 import type { LoaderHandle } from '../loader.ts'
 import { hasClientUi, isDshPlugin, mountLoadedEntry, removeLoadedEntry } from '../loader.ts'
@@ -566,6 +567,9 @@ function spawnMutation(options: {
   })
 
   return (async () => {
+    // 超时复核用基线：本次任务开始前目标是否已在 profile 里。已安装的话，超时后核对无法区分
+    // 「旧版本还在」与「刚装上」，那种情形（更新超时）不冒认成功。
+    const wasInstalled = action !== 'remove' && !isGlobalNpm && verifyInstalledEntry(profile, target).name !== null
     let result = await spawnOnce(args)
     // npm 11 的 arborist peer-set bug（`Cannot read properties of null (reading 'children')`）
     // 会让 `npm install -g` 在解析阶段直接崩溃。首次失败后自动带 `--legacy-peer-deps` 重试一次：
@@ -577,6 +581,16 @@ function spawnMutation(options: {
       if (task) pushLine(task, '[npm] peer-set resolution crashed npm (known npm bug) — retrying with --legacy-peer-deps')
       result = await spawnOnce(retryArgs)
     }
+    // 超时被强杀 ≠ 没装上：命中的安装可能已经落盘（日志里 pnpm 已打印 Done，只是 dsh 进程迟迟不退出
+    // 而被我们超时掐掉）。判失败之前先复核 profile —— 目标已在依赖里、且声明分发的入口文件都在，
+    // 就按成功收尾，避免「其实装好了却报安装失败」并被自动归档成未知失败。
+    if (result.timedOut && !wasInstalled && action !== 'remove' && !isGlobalNpm && task?.status !== 'cancelled') {
+      const verified = verifyInstalledEntry(profile, target)
+      if (verified.name !== null && verified.missing === null) {
+        result = { ...result, timedOut: false, exitCode: 0, error: null }
+        if (task) pushLine(task, '[timeout] the install had already completed — treating it as a success')
+      }
+    }
     // 统一终态后处理：exitCode/status/needsRestart 以最后一次尝试为准
     if (task) {
       task.exitCode = result.exitCode
@@ -776,6 +790,20 @@ export async function runPluginMutation(options: {
       return { exitCode: 1, timedOut: false, error: `declared file missing: ${verified.missing}`, stdout: result.stdout, stderr: `${result.stderr}\n${detail}` }
     }
   }
+  // DSH 只把一行摘要（`dsh: plugin command failed; diagnostics: <path>`）回显到控制台时，
+  // 控制台里没有任何错误特征，前端只能把失败归为「未知」（issue #192）。这里按该路径回捞
+  // operation 日志尾部补进失败信息：用户直接看到真实报错，自动归类也有据可依。
+  // 只在本就不含任何错误码时回捞 —— pnpm 已把报错回显出来时无需多读一次文件。
+  if (result.exitCode !== 0 && !result.timedOut && !/\bERR_[A-Z0-9_]+\b/.test(`${result.stderr}\n${result.stdout}`)) {
+    const backfilled = backfillOperationLog(`${result.stderr}\n${result.stdout}`)
+    if (backfilled !== null) {
+      if (options.task && options.task.status !== 'cancelled') {
+        pushLine(options.task, '[pnpm log] DSH did not echo pnpm output — recovered the operation log from disk')
+        for (const line of backfilled.split(/\r?\n/)) if (line.trim() !== '') pushLine(options.task, line.trimEnd())
+      }
+      result = { ...result, stderr: `${result.stderr}\n${backfilled}` }
+    }
+  }
   // npm arborist edgesOut 崩溃 + 本机 npm 版本过低 → 追加 [npm-too-low] 标记，前端据此给准确原因
   return annotateNpmTooLow(result, options)
 }