dshmarket 1.66.43 → 1.66.44

package/client/client.js

@@ -1993,8 +1993,9 @@ window.__ModuleLoader__.load({ id: "dshmarket", factory: (require) => {
 		}
 		/** Heuristic: plugins that target a terminal surface rather than the web UI. */
 		function looksTerminal(plugin, lang) {
+			if (/\b(tui|cli|tty|terminal)\b|终端|命令行/i.test(plugin.name)) return true;
 			const positiveDesc = (plugin.description && (plugin.description[lang] || plugin.description.en) || "").replace(/\b(?:no|without)\b[^.!?;:，。！？；\n]{0,80}\b(?:tui|cli|tty|terminal)\b/gi, "").replace(/(?:无需|无须|不需要|不用)[^。！？；\n]{0,48}(?:tui|cli|tty|terminal|终端|命令行)/gi, "");
-			return /\b(tui|cli|tty|terminal)\b|终端|命令行/i.test(plugin.name + " " + positiveDesc);
+			return /\b(?:terminal|cli)[ -]only\b|\btui (?:app|plugin|client)\b|\bterminal (?:ui|app|plugin|client)\b|\bcommand[- ]line (?:tool|interface|app|client|plugin)\b|命令行(?:工具|插件|应用|界面|客户端)|终端(?:工具|插件|应用|界面|客户端)|只在终端|仅在终端|需要(?:一个)?终端|在终端(?:里|中)运行/i.test(positiveDesc);
 		}
 		/** Days per TimeRange (`all` has no cutoff and is handled by the caller). */
 		const TIME_RANGE_DAYS = {

package/lib/routes.js

@@ -1855,6 +1855,12 @@ export function mountMarketRoutes(host, config, commandRuntime, agentsLookup, ho
                 version,
                 requirement: verdict.requirement,
                 hostVersion: host?.version ?? null,
+                // #923: the client's "find a compatible version" search is keyed by
+                // the npm name, and the install branch could fall back to the catalog
+                // entry's `npm` while the update branch had nothing. The value is
+                // already here — it is this function's first argument and the key the
+                // registry lookup above used — so publish it.
+                npmName,
             },
             error: nothingWasInstalled
                 ? `${displayName} ${version ?? ''} 要求的 DSH 版本是 ${verdict.requirement ?? '未知'}，而当前运行的是 ${host?.version ?? '未知版本'}，装上多半会直接报错。已停止，没有安装任何东西。 / ${displayName} ${version ?? ''} declares it needs DSH ${verdict.requirement ?? '(unknown)'}, and this host is ${host?.version ?? 'unknown'}; installing it would most likely break the plugin. Nothing was installed.`
@@ -4478,6 +4484,12 @@ export function mountMarketRoutes(host, config, commandRuntime, agentsLookup, ho
                         if (pinnedForIdentification) {
                             declareProfileDependency(config.profile, name, pinValue, activeProfileDir);
                         }
+                        // #928: an update is remove-then-add, and a host killed between
+                        // the two leaves a package that is simply gone — with no record
+                        // of what was being attempted, because the completion log never
+                        // runs. Write the intent first, so the log can answer "where did
+                        // my plugin go" after the fact.
+                        logEvent('info', 'update-start', `${name} -> ${target} (profile ${config.profile})`);
                         const result = await runPlugin(config.profile, addArgs);
                         const cancelled = result.cancelled;
                         // The pin above is the market's own write, so it is undone whenever
@@ -5745,6 +5757,10 @@ export function mountMarketRoutes(host, config, commandRuntime, agentsLookup, ho
                         // it is meant to clean up after, and the rows outlive the package
                         // as boot-time orphans (#799).
                         const ownedRows = rowIdsForPackage(host, activeProfileDir, name);
+                        // #928: the same intent record as the update path — an uninstall
+                        // interrupted mid-remove leaves the same "where did it go" with no
+                        // entry saying what was running.
+                        logEvent('info', 'uninstall-start', `${name} (profile ${config.profile})`);
                         const result = await runPlugin(config.profile, ['remove', name]);
                         const cancelled = result.cancelled;
                         const ok = result.exitCode === 0 && !result.timedOut && !cancelled;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "dshmarket",
   "description": "Visual plugin market inside DeepSeek Harness — browse, search, and one-click install community plugins. · DSH 可视化插件市场：逛一逛，点一下，装好。",
-  "version": "1.66.43",
+  "version": "1.66.44",
   "icon": "assets/logo.svg",
   "type": "module",
   "main": "lib/index.js",

package/src/client/market-data.ts

@@ -430,15 +430,21 @@ export function readSession(key: string): any {
 
 /** Heuristic: plugins that target a terminal surface rather than the web UI. */
 export function looksTerminal(plugin: RegistryPlugin, lang: string): boolean {
+  // The package NAME is a claim about the package itself: `dsh-tui` is a
+  // terminal surface. Descriptions are not — they mention the terminal for
+  // many reasons, including a companion CLI that ships from the same
+  // repository (#931). A bare "CLI"/"terminal" mention in prose used to be
+  // enough to warn that a web plugin "may not work in the web app", and a
+  // warning that fires on ordinary plugins is one users learn to ignore.
+  if (/\b(tui|cli|tty|terminal)\b|终端|命令行/i.test(plugin.name)) return true
   const desc = (plugin.description && (plugin.description[lang] || plugin.description.en)) || ''
-  // A description can mention a CLI only to say it is NOT required. Treating
-  // that as positive evidence labels web plugins as terminal-only. Strip
-  // bounded negated clauses before applying the deliberately broad heuristic;
-  // the package name remains untouched and therefore stays strong evidence.
+  // A description can also mention a CLI only to say it is NOT required
+  // (#237). Strip bounded negated clauses first.
   const positiveDesc = desc
     .replace(/\b(?:no|without)\b[^.!?;:，。！？；\n]{0,80}\b(?:tui|cli|tty|terminal)\b/gi, '')
     .replace(/(?:无需|无须|不需要|不用)[^。！？；\n]{0,48}(?:tui|cli|tty|terminal|终端|命令行)/gi, '')
-  return /\b(tui|cli|tty|terminal)\b|终端|命令行/i.test(plugin.name + ' ' + positiveDesc)
+  // Only phrases that assert THIS plugin's own surface count.
+  return /\b(?:terminal|cli)[ -]only\b|\btui (?:app|plugin|client)\b|\bterminal (?:ui|app|plugin|client)\b|\bcommand[- ]line (?:tool|interface|app|client|plugin)\b|命令行(?:工具|插件|应用|界面|客户端)|终端(?:工具|插件|应用|界面|客户端)|只在终端|仅在终端|需要(?:一个)?终端|在终端(?:里|中)运行/i.test(positiveDesc)
 }
 
 /** Sortable field for the Discover list. */

package/src/routes.ts

@@ -2050,6 +2050,12 @@ export function mountMarketRoutes(
         version,
         requirement: verdict.requirement,
         hostVersion: host?.version ?? null,
+        // #923: the client's "find a compatible version" search is keyed by
+        // the npm name, and the install branch could fall back to the catalog
+        // entry's `npm` while the update branch had nothing. The value is
+        // already here — it is this function's first argument and the key the
+        // registry lookup above used — so publish it.
+        npmName,
       },
       error: nothingWasInstalled
         ? `${displayName} ${version ?? ''} 要求的 DSH 版本是 ${verdict.requirement ?? '未知'}，而当前运行的是 ${host?.version ?? '未知版本'}，装上多半会直接报错。已停止，没有安装任何东西。 / ${displayName} ${version ?? ''} declares it needs DSH ${verdict.requirement ?? '(unknown)'}, and this host is ${host?.version ?? 'unknown'}; installing it would most likely break the plugin. Nothing was installed.`
@@ -4688,6 +4694,12 @@ sendJson(response, 200, { updates })
             if (pinnedForIdentification) {
               declareProfileDependency(config.profile, name, pinValue, activeProfileDir)
             }
+            // #928: an update is remove-then-add, and a host killed between
+            // the two leaves a package that is simply gone — with no record
+            // of what was being attempted, because the completion log never
+            // runs. Write the intent first, so the log can answer "where did
+            // my plugin go" after the fact.
+            logEvent('info', 'update-start', `${name} -> ${target} (profile ${config.profile})`)
             const result = await runPlugin(config.profile, addArgs)
             const cancelled = result.cancelled
             // The pin above is the market's own write, so it is undone whenever
@@ -5979,6 +5991,10 @@ sendJson(response, 200, { updates })
             // it is meant to clean up after, and the rows outlive the package
             // as boot-time orphans (#799).
             const ownedRows = rowIdsForPackage(host, activeProfileDir, name)
+            // #928: the same intent record as the update path — an uninstall
+            // interrupted mid-remove leaves the same "where did it go" with no
+            // entry saying what was running.
+            logEvent('info', 'uninstall-start', `${name} (profile ${config.profile})`)
             const result = await runPlugin(config.profile, ['remove', name])
             const cancelled = result.cancelled
             const ok = result.exitCode === 0 && !result.timedOut && !cancelled