dsh-life-pack 0.3.1 → 0.3.2

package/dist/client.js

@@ -310,7 +310,13 @@ window.__ModuleLoader__.load({
 				version
 			});
 		}
-		/** 装上更新：查新版 → 提交安装 → 有界轮询查状态到任务收尾；凭证过期重查一次再提交。 */
+		/** 装上更新：查新版 → 提交安装 → 有界轮询查状态到任务收尾；凭证过期重查一次再提交。
+		*
+		* **第一句那次 `checkTarget` 不是多余的**（票 #740 第四轮）：凭证是「这一刻的安装态」的凭据，
+		* 更新包提交时拿它跟当时读到的安装态比（`service.js:309`／`:325`），不一样就拒。
+		* 而面板**从来只提交这一刻现签的凭证**——这就是「刚在本面板装过别家，这一家照样点得动
+		* 「装上更新」」的全部依据：旧快照只用来显示版本，不参与提交。（第四轮以前面板还把别家整行
+		* 换成「重新检查」——那是白挡一次本来会成功的点击，已删。） */
 		async function updateInstalled(call, target, pollMs) {
 			if (!target.phones) return failure("bad-request", { target: target.key });
 			const first = await checkTarget(call, target);
@@ -418,25 +424,14 @@ window.__ModuleLoader__.load({
 			if (verdict?.action) return verdict.action;
 			return phase === "failed" ? "recheck" : null;
 		}
-		/** 装成一家之后，其余各家的读数一律作废（票 #740）：七家共用同一份使用范围清单，
-		*  刚那次安装已经改写了它，旧快照再拿去装必然被守卫拦下。这句话就是那条守卫的人话。 */
-		function staleVerdict() {
-			return {
-				kind: "blocked",
-				text: "刚在本面板装过别的插件，共用的安装清单变了，这一家要先重读一次。点「重新检查」，再点「装上更新」。",
-				action: "recheck"
-			};
-		}
-		/** 把除 `keepKey` 之外的行全标成过期（票 #740）：装成一家 ⇒ 七家共用的安装清单被改写，
-		*  其余各家的旧快照再拿去装必然被守卫拦下。纯函数：状态源与单测共用同一份判据，
-		*  不关心行里还有什么（泛型只要求有 `stale` 这一格）。 */
-		function markStaleOthers(rows, keepKey) {
-			const next = {};
-			for (const [key, row] of Object.entries(rows)) next[key] = key === keepKey ? row : {
-				...row,
-				stale: true
-			};
-			return next;
+		/** 这一行要不要摊出那条手工命令（票 #740 第三轮定的形状，第四轮补的这一格）。
+		*
+		*  三种情形摊：① 上一次装失败（`failed`）；② 这句话自己指着「下面这条命令」（`manualHint`）。
+		*  **待重启**与 `incompatible-node` 不摊：它们那时该做的是**重启**（或升级 Node），不是重装一次。
+		*  （第四轮维护者真机截图逮到：待重启那张卡下面挂着一条「dsh plugin … add」——点它等于把刚装好的
+		*  那一版再装一遍，纯噪声，还会让人以为状态不对。） */
+		function showManualOf(verdict, phase) {
+			return phase === "failed" || verdict?.manualHint === true;
 		}
 		/** 把快照翻成一行结论：还没查 / 已是最新 / 有新版 / 待重启 / 装不了（附人话原因与下一步）。 */
 		function verdictOf(target, snapshot) {
@@ -823,10 +818,6 @@ window.__ModuleLoader__.load({
 					[key]: next
 				}));
 			}, []);
-			/** 装成一家 ⇒ 其余各家的旧快照一律作废（票 #740）：判据见 `markStaleOthers`。 */
-			const staleOthers = react.useCallback((keepKey) => {
-				setRows((previous) => markStaleOthers(previous, keepKey));
-			}, []);
 			return {
 				targets,
 				rows,
@@ -842,21 +833,18 @@ window.__ModuleLoader__.load({
 					for (const target of list) patch(target.key, {
 						phase: "checking",
 						outcome: rows[target.key]?.outcome ?? null,
-						failure: null,
-						stale: false
+						failure: null
 					});
 					Promise.all(list.map(async (target) => {
 						const result = await checkTarget(getCall(), target);
 						patch(target.key, result.ok ? {
 							phase: "ready",
 							outcome: result.value,
-							failure: null,
-							stale: false
+							failure: null
 						} : {
 							phase: "failed",
 							outcome: null,
-							failure: result,
-							stale: false
+							failure: result
 						});
 					})).finally(() => setChecking(false));
 				}, [
@@ -871,21 +859,18 @@ window.__ModuleLoader__.load({
 						patch(target.key, {
 							phase: "checking",
 							outcome: current.outcome,
-							failure: null,
-							stale: false
+							failure: null
 						});
 						(async () => {
 							const checked = await checkTarget(getCall(), target);
 							patch(target.key, checked.ok ? {
 								phase: "ready",
 								outcome: checked.value,
-								failure: null,
-								stale: false
+								failure: null
 							} : {
 								phase: "failed",
 								outcome: null,
-								failure: checked,
-								stale: false
+								failure: checked
 							});
 						})();
 						return;
@@ -893,8 +878,7 @@ window.__ModuleLoader__.load({
 					patch(target.key, {
 						...current,
 						phase: "installing",
-						failure: null,
-						stale: false
+						failure: null
 					});
 					(async () => {
 						const absent = isAbsent(target);
@@ -905,36 +889,29 @@ window.__ModuleLoader__.load({
 								patch(target.key, {
 									phase: "failed",
 									outcome: null,
-									failure: checked,
-									stale: false
+									failure: checked
 								});
 								return;
 							}
 							patch(target.key, {
 								phase: "installing",
 								outcome: checked.value,
-								failure: null,
-								stale: false
+								failure: null
 							});
 							version = checked.value.snapshot.latestVersion;
 						}
 						const result = absent ? await installAbsent(getCall(), target, version ?? "") : await updateInstalled(getCall(), target, pollMs);
 						if (result.ok) {
-							const before = current.outcome?.snapshot.installedVersion ?? null;
 							const refreshed = await checkTarget(getCall(), target, target.phones?.status);
-							const after = refreshed.ok ? refreshed.value.snapshot.installedVersion ?? null : null;
 							patch(target.key, refreshed.ok ? {
 								phase: "ready",
 								outcome: refreshed.value,
-								failure: null,
-								stale: false
+								failure: null
 							} : {
 								phase: "failed",
 								outcome: null,
-								failure: refreshed,
-								stale: false
+								failure: refreshed
 							});
-							if (after !== null && after !== before) staleOthers(target.key);
 							await reload();
 							return;
 						}
@@ -944,8 +921,7 @@ window.__ModuleLoader__.load({
 							failure: {
 								...result,
 								manual: result.manual ?? current.outcome?.manual ?? null
-							},
-							stale: false
+							}
 						});
 					})();
 				}, [
@@ -953,8 +929,7 @@ window.__ModuleLoader__.load({
 					patch,
 					pollMs,
 					reload,
-					rows,
-					staleOthers
+					rows
 				])
 			};
 		}
@@ -1007,10 +982,10 @@ window.__ModuleLoader__.load({
 			const body = shown.map((target) => {
 				const row = rows[target.key];
 				const snapshot = row.outcome?.snapshot ?? null;
-				const verdict = row.stale === true ? staleVerdict() : snapshot ? verdictOf(target, snapshot) : null;
+				const verdict = snapshot ? verdictOf(target, snapshot) : null;
 				const manual = manualForDisplay(target, row.failure?.manual ?? row.outcome?.manual ?? null);
 				const busy = row.phase === "installing";
-				const showManual = manual !== null && (row.phase === "failed" || verdict?.manualHint === true || verdict?.kind === "blocked" && verdict.action === null);
+				const showManual = manual !== null && showManualOf(verdict, row.phase);
 				const buttonAction = cardActionOf(verdict, row.phase);
 				const tone = rowToneOf(row, verdict);
 				const paint = ROW_TONE[tone];