dsh-plugin 1.6.7 → 1.6.8

package/lib/http/install.js

@@ -208,16 +208,15 @@ export function registerInstallRoutes(webServer, profile, loader) {
                             return;
                         }
                     }
-                    // 重复安装防护：非更新请求命中已安装目标时直接拒绝，不重复跑 CLI
-                    // （否则 CLI 会因「已存在」失败，且用户看到的是莫名其妙的报错）；
-                    // 更新请求（mode: 'update'）放行，CLI 的 add 对已存在依赖是 pnpm 原位覆盖重装。
-                    // 命中判定双通道：npm 包名按 dependencies 键直接命中，GitHub 目标按仓库身份匹配。
+                    // 命中已安装目标时一律按「更新」语义执行（pnpm add 对已存在依赖本就是原位覆盖重装），
+                    // 不再抛 409：客户端的已装列表与服务端实读可能不同步（如页面列表滞后），此时用户点
+                    // 「安装」会撞上重复安装守卫，直接覆盖重装比让用户看到一句看不懂的 already installed
+                    // 报错友好，最终状态也一致（都是已安装）。命中判定双通道：npm 包名按 dependencies
+                    // 键直接命中，GitHub 目标按仓库身份匹配。
                     const installed = readInstalled(profile);
                     const already = installedByRepo(installed, target) ?? (installed[target] !== undefined ? target : null);
-                    if (already !== null && mode !== 'update') {
-                        sendJson(response, 409, { error: `already installed: ${already}` });
-                        return;
-                    }
+                    if (already !== null)
+                        mode = 'update';
                     // 队列级去重：同一目标已在排队/执行中时拒绝，避免客户端防重竞态导致重复入队
                     if (hasQueuedTarget(target)) {
                         sendJson(response, 409, { error: `already queued: ${target}` });

package/lib/services/install/operation-log.js

@@ -0,0 +1,49 @@
+/**
+ * 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.js';
+/**
+ * 从 DSH 输出里解析 operation 日志路径（`diagnostics: <path>`）。
+ * 只认 DSH 的 operation 日志：路径含 `.plugin-manager` 且以 `pnpm.log` 结尾；
+ * 不是这种路径（或压根没有该行）返回 null，调用方据此跳过回捞。
+ */
+export function operationLogPathOf(output) {
+    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) {
+    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/lib/services/install/task-queue.js

@@ -13,6 +13,7 @@ import { existsSync, readFileSync, statSync } from 'node:fs';
 import { dirname, join, resolve } from 'node:path';
 import { CAPTURE_LIMIT_BYTES, MAX_TASKS, MAX_TASK_LINES } from './install-types.js';
 import { npmTooLowMarker } from './npm-check.js';
+import { backfillOperationLog } from './operation-log.js';
 import { cleanLine, estimateProgress } from '../profile/progress.js';
 import { hasClientUi, isDshPlugin, mountLoadedEntry, removeLoadedEntry } from '../loader.js';
 import { addPendingRestart, clearPendingRestart } from '../profile/pending-restart.js';
@@ -734,6 +735,22 @@ 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);
 }

package/lib/types/services/install/operation-log.d.ts

@@ -0,0 +1,11 @@
+/**
+ * 从 DSH 输出里解析 operation 日志路径（`diagnostics: <path>`）。
+ * 只认 DSH 的 operation 日志：路径含 `.plugin-manager` 且以 `pnpm.log` 结尾；
+ * 不是这种路径（或压根没有该行）返回 null，调用方据此跳过回捞。
+ */
+export declare function operationLogPathOf(output: string): string | null;
+/**
+ * 回捞 operation 日志尾部，作为失败详情返回；拿不到（无路径 / 文件不存在 / 读失败 /
+ * 内容为空）返回 null。尾部截到与子进程输出相同的上限，避免超大日志撑爆内存与消息。
+ */
+export declare function backfillOperationLog(output: string): string | null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin",
-  "version": "1.6.7",
+  "version": "1.6.8",
   "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/components/modals/NotificationsModal.tsx

@@ -377,6 +377,12 @@ export function NotificationsModal({ records, tasks, pendingRestarts, t, env, on
                         }, t('failIssueBig')) : null,
                       ])
                     }
+                    if (kind === 'hubState') {
+                      // 插件中心的列表状态已过期（安装器自身守卫消息）：非插件问题、也不是 bug，只给可行动提示，不引导提 Issue
+                      return h('div', null, [
+                        h('div', { className: styles.failPrepareHint }, t('failHubStateHint')),
+                      ])
+                    }
                     // pnpm allowBuilds 拦截与未知失败：不归到插件作者，改为引导到插件中心仓库排查
                     return h('div', null, [
                       kind === 'pnpmIgnoredBuild'

package/src/client/components/modals/modals.tsx

@@ -497,6 +497,11 @@ export function ErrorModal({ message, repo, kind, command, attempts, t, env, onC
                   title: t('failIssueHint'),
                 }, t('failIssueBig')) : null,
               ])
+              : failureKind === 'hubState'
+              ? h('div', null, [
+                // 插件中心的列表状态已过期（安装器自身守卫消息）：不是插件问题、也不是 bug，只给可行动提示，不引导提 Issue
+                h('div', { className: styles.failPrepareHint }, t('failHubStateHint')),
+              ])
               : h('div', null, [
                 // pnpm allowBuilds 拦截与未知失败：不归到插件作者，改为引导到插件中心仓库排查
                 failureKind === 'pnpmIgnoredBuild'

package/src/client/locales.ts

@@ -315,6 +315,8 @@ export const zh = {
   failHubIssueBig: '反馈给插件中心排查',
   // 未知失败兜底：不归到插件作者，引导到插件中心仓库排查
   failUnclassifiedHint: '安装失败，原因未能自动识别。这通常不是插件自身的问题（可能与本机环境或插件中心有关），请将错误信息反馈给插件中心，由维护者排查。',
+  // 插件中心自身状态类消息（插件未安装 / 不在此处卸载 / 已在队列中）：已安装列表与实读快照不同步，非插件问题，不引导提 Issue
+  failHubStateHint: '插件列表状态已过期或与本地不一致：该插件可能已经装过了（无需重复安装，要升级请点「更新」），也可能已不在已安装列表中，或已在队列中。这不是插件本身的问题 —— 请刷新页面后重试。',
   // prepare 构建脚本实际执行失败（git 分发缺子模块/构建产物）：插件打包分发问题，应提 Issue
   failPrepareHint: '该插件安装时的构建脚本执行失败（git 分发常缺失子模块或构建产物），属插件打包分发问题。请向作者仓库提交 Issue 反馈。',
   // 安装前预检 / 装后校验拦截（[packaging]）：git 分发缺插件需要的文件（入口 / 补丁 mount 的入口 / bundle 补丁）= 插件未适配官方默认安装方式
@@ -713,6 +715,8 @@ export const en = {
   failHubIssueBig: 'Report to the Plugin Hub for triage',
   // Unknown-failure fallback: not routed to the plugin author, but to the hub repo for triage
   failUnclassifiedHint: 'The install failed and the cause could not be identified automatically. This is usually not the plugin itself (it may relate to your local environment or the Plugin Hub). Please report the error to the Plugin Hub so the maintainers can triage it.',
+  // The hub's own state-guard messages (plugin not installed / cannot be uninstalled here / already queued / already installed): the installed list is out of sync with the live snapshot — not a plugin issue, no issue prompt
+  failHubStateHint: 'The plugin list is out of date or out of sync: this plugin may already be installed (no need to reinstall — use "Update" to upgrade), or may no longer be in the installed list, or may already be queued. This is not a plugin problem — refresh the page and retry.',
   // prepare script actually failed (git tarball misses submodules/build output): a packaging issue of the plugin itself — file an issue
   failPrepareHint: 'This plugin failed while running its build scripts during install (git tarballs often miss submodules or build output). This is a packaging issue of the plugin itself — please report it to the author repository.',
   // Pre-install / post-install guard hit ([packaging]): git distribution lacks a file the package needs (entry / entry a patch mounts / bundle patch) = not adapted to the official default install method

package/src/client/logic/failures.ts

@@ -151,7 +151,7 @@ export function removeNotification(id: number): NotificationRecord[] {
   return next
 }
 
-export type FailureKind = 'npmTooOld' | 'dshMissing' | 'gitMissing' | 'pnpmMissing' | 'npmMissing' | 'pnpmStore' | 'pnpmWorkspace' | 'originRejected' | 'pnpmPolicy' | 'pnpmUnusedPatch' | 'pnpmMissingDep' | 'registryVersionStale' | 'pnpmLinkedDirMissing' | 'fileLocked' | 'accessDenied' | 'fsUnavailable' | 'pnpmIgnoredBuild' | 'pluginPrepare' | 'network' | 'hub'
+export type FailureKind = 'npmTooOld' | 'dshMissing' | 'gitMissing' | 'pnpmMissing' | 'npmMissing' | 'pnpmStore' | 'pnpmWorkspace' | 'originRejected' | 'pnpmPolicy' | 'pnpmUnusedPatch' | 'pnpmMissingDep' | 'registryVersionStale' | 'pnpmLinkedDirMissing' | 'fileLocked' | 'accessDenied' | 'fsUnavailable' | 'pnpmIgnoredBuild' | 'pluginPrepare' | 'network' | 'hubState' | 'hub'
 
 /** Match the pnpm error code, not a generic mention of checksum or integrity. */
 function missingTarballIntegrity(message: string): boolean {
@@ -238,6 +238,12 @@ export function classifyFailure(message: string): FailureKind {
   // 构建脚本真的跑挂了才算插件问题。Command failed 区分大小写：只匹配 pnpm 的包装，
   // 不匹配宿主那句小写 dsh: plugin command failed
   if (/ERR_PNPM_PREPARE_PACKAGE|ELIFECYCLE|prepare-guard/i.test(message) || /Command failed/.test(message)) return 'pluginPrepare'
+  // 安装器自身产生的状态类消息（未安装 / 不在此处卸载 / 已在队列中 / 已安装）：这几句由本服务的
+  // 守卫发出，不是 CLI 输出、也不是插件问题，而是插件中心的已安装列表与实读快照不同步。
+  // 单独归类，让前端给「刷新后重试」这类可行动提示，而不是当作未知失败引导去提 Issue。
+  // already installed 只认「整行就是它」的短形态（旧版本守卫留下的历史记录）；带「——」说明的
+  // 包名冲突另有处置，不能按状态不同步提示。
+  if (/plugin is not installed|plugin cannot be uninstalled here|already queued:|^already installed: \S+$/im.test(message)) return 'hubState'
   // 其余未知失败：引导到插件中心仓库排查
   return 'hub'
 }

package/src/client/logic/urls.ts

@@ -63,6 +63,7 @@ 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 'hub': return 'install failed (cause unclassified)'
     default: return 'plugin install failed'
   }

package/src/server/http/install.ts

@@ -206,16 +206,14 @@ export function registerInstallRoutes(webServer: WebServerService, profile: stri
               return
             }
           }
-          // 重复安装防护：非更新请求命中已安装目标时直接拒绝，不重复跑 CLI
-          // （否则 CLI 会因「已存在」失败，且用户看到的是莫名其妙的报错）；
-          // 更新请求（mode: 'update'）放行，CLI 的 add 对已存在依赖是 pnpm 原位覆盖重装。
-          // 命中判定双通道：npm 包名按 dependencies 键直接命中，GitHub 目标按仓库身份匹配。
+          // 命中已安装目标时一律按「更新」语义执行（pnpm add 对已存在依赖本就是原位覆盖重装），
+          // 不再抛 409：客户端的已装列表与服务端实读可能不同步（如页面列表滞后），此时用户点
+          // 「安装」会撞上重复安装守卫，直接覆盖重装比让用户看到一句看不懂的 already installed
+          // 报错友好，最终状态也一致（都是已安装）。命中判定双通道：npm 包名按 dependencies
+          // 键直接命中，GitHub 目标按仓库身份匹配。
           const installed = readInstalled(profile)
           const already = installedByRepo(installed, target) ?? (installed[target] !== undefined ? target : null)
-          if (already !== null && mode !== 'update') {
-            sendJson(response, 409, { error: `already installed: ${already}` })
-            return
-          }
+          if (already !== null) mode = 'update'
           // 队列级去重：同一目标已在排队/执行中时拒绝，避免客户端防重竞态导致重复入队
           if (hasQueuedTarget(target)) {
             sendJson(response, 409, { error: `already queued: ${target}` })

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'
@@ -776,6 +777,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)
 }