dsh-plugin 2.2.0 → 2.2.2

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

@@ -657,6 +657,11 @@ function spawnMutation(options: {
         task.status = result.timedOut || result.exitCode !== 0 ? 'failed' : 'done'
         if (task.status === 'done') {
           task.progress = 100
+          // 实际装上的版本：目录版本可能滞后，客户端账本必须记这个值，否则「有更新」永久失真
+          if (!isGlobalNpm && action !== 'remove') {
+            const installed = installedVersionOf(profile, target)
+            if (installed !== null) task.installedVersion = installed
+          }
           // 市场自更新：若 bundle 已被取消勾选，重启后市场不会出现，结果页要明确提示
           if ((action === 'add' || action === 'update') && (target === 'dsh-plugin' || target.startsWith('dsh-plugin@'))
             && !bundleIncludes(profile, 'dsh-plugin')) {
@@ -699,10 +704,38 @@ function spawnMutation(options: {
         pushLine(task, result.timedOut ? '[timed out]' : `[exit ${result.exitCode ?? '?'}]${printedNothing ? ' (no output captured)' : ''}`)
       }
     }
-    return result
+    // 带上最终生效的源：后续注解（如 [dsh-too-old]）读元数据必须与安装同源，
+    // 否则换过源之后会去另一个源查，查不到就静默漏报
+    return activeRegistry === undefined ? result : { ...result, registry: activeRegistry }
   })()
 }
 
+/**
+ * 安装成功后从 profile 的 `node_modules/<pkg>/package.json` 读**实际**版本；
+ * 定位不到包名或读不到返回 null。客户端拿它写版本账本 —— 目录里的版本可能滞后。
+ */
+export function installedVersionOf(profile: string, target: string): string | null {
+  const profileDir = profileDirectory(profile)
+  let deps: Record<string, string> = {}
+  try {
+    const pkg = JSON.parse(readFileSync(join(profileDir, 'package.json'), 'utf8')) as { dependencies?: Record<string, string> }
+    deps = pkg.dependencies ?? {}
+  } catch {
+    return null
+  }
+  const targetRepo = githubRepoOf(target)?.toLowerCase()
+  const name = targetRepo !== undefined
+    ? Object.keys(deps).find((k) => githubRepoOf(deps[k])?.toLowerCase() === targetRepo)
+    : deps[target] !== undefined ? target : Object.keys(deps).find((k) => deps[k] === target)
+  if (!name) return null
+  try {
+    const meta = JSON.parse(readFileSync(join(profileDir, 'node_modules', name, 'package.json'), 'utf8')) as { version?: unknown }
+    return typeof meta.version === 'string' && meta.version !== '' ? meta.version : null
+  } catch {
+    return null
+  }
+}
+
 /**
  * 安装后校验已装包里 package.json 声明的文件（main / exports["."].default 与
  * dsh.bundle.patch）是否真实存在。npm 包也可能漏发构建产物（lib/ 等），pnpm 装完没有
@@ -805,8 +838,9 @@ async function annotateDshTooOld(result: InstallResult, options: {
   if (options.action === 'remove' || result.exitCode === 0) return result
   // 全局 npm 装的是本机 CLI 工具，不进 profile，没有宿主版本要求
   if ((options.globalNpm ?? []).length > 0) return result
-  // 用安装实际使用的源读元数据：私有源/镜像里也能读到同一个包的声明
-  const line = await dshTooOldMarker(options.target, { registry: options.registries?.[0] })
+  // 用安装**实际**使用的源读元数据：换源重试后候选表首位可能并不是真正生效的那个，
+  // 用错源会读不到包而静默漏报；私有源/镜像里同样能读到包声明的 engines
+  const line = await dshTooOldMarker(options.target, { registry: result.registry ?? options.registries?.[0] })
   if (line === null) return result
   if (options.task && options.task.status !== 'cancelled') pushLine(options.task, line)
   return { ...result, stderr: `${result.stderr}\n${line}`, error: result.error ? `${result.error}\n${line}` : line }

package/src/server/services/npm-release-age.ts

@@ -25,8 +25,11 @@ export const RELEASE_AGE_TIMEOUT_MS = 8000
 const MAX_PACKUMENT_BYTES = 10 * 1024 * 1024
 export const DEFAULT_NPM_REGISTRY = 'https://registry.npmjs.org'
 
-/** 会话内缓存（包名 → 版本与发布时间）：只存成功结果，一次网络抖动不会把判断永久粘住。 */
-const releaseCache = new Map<string, NpmRelease>()
+/** 会话内缓存（源 + 包名 → 结果）：只存成功结果，一次网络抖动不会把判断永久粘住。
+ *  必须带过期时间 —— 插件中心可能连续开着几小时，作者发了新版本后不能一直沿用旧结果；
+ *  也必须把源算进键 —— 同一个包在官方源与镜像上的 latest / 发布时间不同，混用会算错门禁。 */
+const CACHE_TTL_MS = 10 * 60 * 1000
+const releaseCache = new Map<string, { at: number; release: NpmRelease }>()
 
 /** 从完整 packument 取 dist-tags.latest 对应版本及其发布时间；缺任一字段返回 null。 */
 export function latestReleaseOf(packument: unknown): NpmRelease | null {
@@ -87,21 +90,22 @@ async function fetchPackument(url: string, proxy: string, timeoutMs: number): Pr
   return getJson(url, timeoutMs)
 }
 
-/** 查询一个包的最新版本与发布时间（成功结果缓存，同会话内同包不重复请求）；registry 为空回退官方源，失败返回 null。 */
+/** 查询一个包的最新版本与发布时间（成功结果按源缓存 10 分钟）；registry 为空回退官方源，失败返回 null。 */
 export async function lookupReleaseAge(
   packageName: string,
   registry: string,
   proxy: string,
   timeoutMs = RELEASE_AGE_TIMEOUT_MS,
 ): Promise<NpmRelease | null> {
-  const cached = releaseCache.get(packageName)
-  if (cached !== undefined) return cached
   const base = registry.replace(/\/+$/, '') || DEFAULT_NPM_REGISTRY
+  const key = `${base}\n${packageName}`
+  const cached = releaseCache.get(key)
+  if (cached !== undefined && Date.now() - cached.at < CACHE_TTL_MS) return cached.release
   const body = await fetchPackument(`${base}/${encodeURIComponent(packageName)}`, proxy, timeoutMs)
   if (body === null) return null
   let parsed: unknown
   try { parsed = JSON.parse(body) } catch { return null }
   const release = latestReleaseOf(parsed)
-  if (release !== null) releaseCache.set(packageName, release)
+  if (release !== null) releaseCache.set(key, { at: Date.now(), release })
   return release
 }

package/src/server/services/profile/installed-versions.ts

@@ -17,7 +17,7 @@
  * 存储：`DSH_HOME/profiles/<profile>/gro.ngilp-hsd-versions.json`
  *   { "<owner>/<repo>"（小写）: { version, updatedAt, installedAt } }
  */
-import { readFileSync, writeFileSync, mkdirSync } from 'node:fs'
+import { readFileSync, writeFileSync, mkdirSync, renameSync } from 'node:fs'
 import { homedir } from 'node:os'
 import { dirname, join } from 'node:path'
 
@@ -38,10 +38,25 @@ export function versionsFilePath(profile: string): string {
   return join(process.env.DSH_HOME ?? join(homedir(), '.dsh'), 'profiles', profile, 'gro.ngilp-hsd-versions.json')
 }
 
+/** 读取某 profile 的版本账本；缺失/损坏/字段不全一律归一化成完整形状。
+ *  安装一开始只会写入 npm 包名映射（版本要等任务结束才知道），所以磁盘上可能存在
+ *  只有 npmPackage 的记录 —— 原样下发会让客户端的版本比较拿到 undefined 直接抛错。 */
 export function readInstalledVersions(profile: string): InstalledVersionTable {
   try {
     const raw = JSON.parse(readFileSync(versionsFilePath(profile), 'utf8')) as unknown
-    return raw !== null && typeof raw === 'object' ? raw as InstalledVersionTable : {}
+    if (raw === null || typeof raw !== 'object' || Array.isArray(raw)) return {}
+    const table: InstalledVersionTable = {}
+    for (const [key, value] of Object.entries(raw as Record<string, unknown>)) {
+      if (value === null || typeof value !== 'object') continue
+      const rec = value as Partial<InstalledVersionRecord>
+      table[key] = {
+        version: typeof rec.version === 'string' ? rec.version : '',
+        updatedAt: typeof rec.updatedAt === 'string' ? rec.updatedAt : '',
+        installedAt: typeof rec.installedAt === 'string' ? rec.installedAt : '',
+        ...(typeof rec.npmPackage === 'string' && rec.npmPackage !== '' ? { npmPackage: rec.npmPackage } : {}),
+      }
+    }
+    return table
   } catch {
     return {}
   }
@@ -64,7 +79,14 @@ export function recordInstalledVersion(profile: string, repo: string, version: s
 export function recordResolvedNpmPackage(profile: string, repo: string, npmPackage: string): void {
   const table = readInstalledVersions(profile)
   const key = repo.toLowerCase()
-  table[key] = { ...table[key], npmPackage }
+  // 显式写全字段：这一步发生在安装开始时，版本还不知道 —— 但落盘记录必须是完整形状，
+  // 否则只带 npmPackage 的残缺记录会在客户端版本比较时崩掉整块界面
+  table[key] = {
+    version: table[key]?.version ?? '',
+    updatedAt: table[key]?.updatedAt ?? '',
+    installedAt: table[key]?.installedAt ?? '',
+    npmPackage,
+  }
   writeVersions(profile, table)
 }
 
@@ -78,5 +100,9 @@ export function removeInstalledVersion(profile: string, repo: string): void {
 function writeVersions(profile: string, table: InstalledVersionTable): void {
   const file = versionsFilePath(profile)
   mkdirSync(dirname(file), { recursive: true })
-  writeFileSync(file, JSON.stringify(table, null, 2) + '\n', 'utf8')
+  // 先写临时文件再原子替换：直接覆盖时进程被杀/磁盘满会留下截断的 JSON，
+  // 下次读取整份账本都会失效（更新检测静默停摆），而用户看不到任何报错
+  const tmp = `${file}.tmp`
+  writeFileSync(tmp, JSON.stringify(table, null, 2) + '\n', 'utf8')
+  renameSync(tmp, file)
 }