@noob-stupid/dsh-plugin-console 0.5.44 → 0.5.45

package/lib/server/routes/framework.js

@@ -1,10 +1,10 @@
 // L2 · routes —— 框架（/framework-check · /check-update · /framework-upgrade · /framework-rollback · /framework-upgrade-status · /framework-relaunch · /compat-gate · /restart）
 // 分层 Step 8b：从 lib/index.js 的 handle() 原样搬出（只搬移未改逻辑；缩进保持原样）
 
-import { readFileSync, writeFileSync, existsSync, statSync, mkdirSync } from 'node:fs'
+import { readFileSync, writeFileSync, existsSync, statSync, mkdirSync, rmSync } from 'node:fs'
 import { writeFile } from 'node:fs/promises'
 import { execFile } from 'node:child_process'
-import { dirname, join, resolve } from 'node:path'
+import { dirname, join, resolve, basename } from 'node:path'
 import { tmpdir } from 'node:os'
 import { createRequire } from 'node:module'
 import { readCompatGate, writeCompatGate } from '../domain/compat.js'
@@ -460,14 +460,56 @@ async function routeRestart(req, res, rc) {
 }
 
 /** 清掉「最近一次升级/回滚记录」（2026-09-25 用户要求）：记录常驻是设计，但用户要能一键关掉。
- *  只删状态文件与心跳（不动日志，日志仍可在 ~/.dsh/plugin-console 里查）。 */
+ *  只删状态文件与心跳（不动日志，日志仍可在 ~/.dsh/plugin-console 里查）。
+ *  2026-10-11 改错：删除失败**如实上报**（500 + 每个文件的原因 + removed/failed 明细），
+ *  不再像旧实现那样吞掉异常还回 ok:true —— 静默失败一律不许（见函数内注释与
+ *  tests/test-framework-status-clear.mjs 的正控/负控）。 */
+/** 删文件失败时给用户看的一句话原因（code + 系统原文，便于判因）。 */
+function explainRemoveError(error) {
+  const code = typeof error?.code === 'string' && error.code !== '' ? error.code : 'UNKNOWN'
+  const text = (error instanceof Error ? error.message : String(error ?? '')).split('\n')[0].trim()
+  return text === '' ? code : `${code}: ${text}`
+}
+
+/** 删除时文件已不在 = 目标达成（不计失败）；其余错误一律算失败。 */
+function removeTargetGone(error) {
+  return error?.code === 'ENOENT'
+}
+
 async function routeFrameworkStatusClear(req, res, rc) {
   const dir = join(dshHome(), 'plugin-console')
   const state = join(dir, 'fw-upgrade-state.txt')
+  const io = rc.deps?.fs ?? { existsSync, rmSync } // 注入缝：负控要能造出真实的删除失败
   const removed = []
+  const failed = []
   for (const f of [state, `${state}.hb`]) {
-    try { if (existsSync(f)) { rmSync(f, { force: true }); removed.push(basename(f)) } } catch {}
+    let present = false
+    try { present = io.existsSync(f) === true } catch (error) { failed.push({ file: basename(f), reason: explainRemoveError(error) }); continue }
+    if (!present) continue
+    try {
+      io.rmSync(f, { force: true })
+      removed.push(basename(f))
+    } catch (error) {
+      // 2026-10-11 改错：这里原来是 `catch {}` + 无条件 ok:true —— 静默失败。
+      // 先前的真实现状更糟：rmSync/basename 没 import ⇒ 每轮都抛 ReferenceError、被空 catch 吞掉，
+      // 路由照样回 200 {"ok":true,"removed":[]}，标记文件一个都没删（用户以为清掉了）。
+      // 判据（用户红线）：宁可红，也不报假的绿 —— 删除失败必须如实上报原因。
+      if (removeTargetGone(error)) { removed.push(basename(f)); continue }
+      failed.push({ file: basename(f), reason: explainRemoveError(error) })
+    }
+  }
+  if (failed.length > 0) {
+    // 如实上报：500 + 每个文件的原因 + 与成功路径同形的 removed/status 字段（客户端不必猜）。
+    sendJson(res, 500, {
+      ok: false,
+      error: `清除升级记录失败：${failed.map((x) => `${x.file}（${x.reason}）`).join('、')}`,
+      removed,
+      failed,
+      status: 'clear-failed',
+      message: null,
+    })
+    return
   }
-  sendJson(res, 200, { ok: true, removed, status: 'idle', message: null })
+  sendJson(res, 200, { ok: true, removed, failed, status: 'idle', message: null })
 }
 export { refuseWhenShellHosted, refuseWhenPlatformUnsupported, routeFrameworkStatusClear, routeFrameworkUpgradeStatusGet, routeFrameworkRelaunch, routeFrameworkCheck, routeCompatGate, routeCheckUpdate, routeFrameworkRollback, routeRestart }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@noob-stupid/dsh-plugin-console",
-  "version": "0.5.44",
+  "version": "0.5.45",
   "description": "DSH 框架升级安全与插件升级门控：一键升级、失败自动回滚、升级后回滚上版、旧插件不适配自动禁用；内置多源插件市场为发现层，插件源全部可自定义，可指向公司内网私有源 / 私有索引 / 本地 Git 仓库，纯内网离线可用 | Framework upgrade safety & plugin version gating for DSH, with a customizable multi-source plugin market: point every source at internal mirrors or a local file:// repo for intranet-only, offline installs.",
   "repository": {
     "type": "git",