@hyzyn/dsh-docker 0.7.0 → 0.7.2

package/client-src/docker.css

@@ -1981,6 +1981,19 @@
 /* 删除按钮靠右，不跟着列宽被拉长 */
 .dk_targetRow > .dk_btnDanger { justify-self: end; }
 
+/*
+ * 失效的连接簿引用（引用的条目名不在 tty 连接簿里）：标黄整行 + 左侧色条。
+ * 为什么必须可见：book 下拉的候选项来自 ttyBooks，失效的名字没有对应 option，
+ * 下拉会渲染成**空白**——界面上看不出错，直到真去连才报「引用的连接簿条目不存在」。
+ */
+.dk_targetRow[data-stale] {
+  border-color: var(--dk-warn);
+  box-shadow: inset 2px 0 0 var(--dk-warn);
+}
+.dk_hintWarn {
+  color: var(--dk-warn);
+}
+
 .dk_targetInline {
   display: grid;
   grid-column: 1 / -1;

package/client-src/index.js

@@ -17,7 +17,7 @@
  * （宿主只回 passwordSet / passphraseSet 布尔）。
  */
 import dockerCss from './docker.css'
-import { bookSessionHost, pickTargetByHost, sessionHostPort } from './session-target.js'
+import { bookSessionHost, pickTargetByHost, sessionHostPort, staleBookRef } from './session-target.js'
 import { currentSessionIdOf } from './current-session.js'
 
 const API = '/api/dsh-docker'
@@ -5958,7 +5958,15 @@ window.__ModuleLoader__.load({
         jsx('span', { className: 'dk_hint', children: 'docker socket 等价于目标主机的 root 权限。开启后，浏览器面板与 agent 都能执行对应操作，请只在可信环境下打开。' }),
 
         sectionTitle('目标'),
-        ...form.targets.map((item, index) => jsxs('div', { className: 'dk_targetRow', children: [
+        ...form.targets.map((item, index) => {
+          /*
+           * 失效引用（引用的条目名不在 tty 连接簿里）：下拉会**渲染成空白**——没有任何
+           * option 与 item.book 匹配，界面上看不出"这里引用错了"，要等真去连才报错。
+           * 这里两件事一起做：给那一行标记 data-stale（标黄），并把失效的名字补成一个
+           * 显式 option，免得用户看到一个空白下拉、以为是"没选"。
+           */
+          const stale = staleBookRef(item, form.ttyBooks)
+          return jsxs('div', { className: 'dk_targetRow', 'data-stale': stale !== undefined ? '1' : undefined, children: [
           jsx('input', { className: 'dk_input', value: item.name, placeholder: '目标名', onChange: (event) => patchTarget(index, { name: event.target.value }) }),
           jsx('select', { className: 'dk_select', value: item.kind, onChange: (event) => patchTarget(index, { kind: event.target.value }), children: [
             jsx('option', { value: 'local', children: '本机' }),
@@ -5971,11 +5979,17 @@ window.__ModuleLoader__.load({
                 className: 'dk_select',
                 value: item.book ?? '',
                 onChange: (event) => patchTarget(index, { book: event.target.value }),
+                title: stale !== undefined ? `引用的连接簿条目「${stale}」不存在——请改选一个已有条目，或清空改为手填` : undefined,
                 children: [
                   jsx('option', { value: '', children: form.ttyBooks.length === 0 ? '（无 tty 连接簿，请填内联信息）' : '（不用连接簿，手填）' }),
+                  // 失效的名字排在最前并显式标注：否则它没有对应 option，下拉显示空白
+                  ...(stale !== undefined ? [jsx('option', { value: stale, children: '⚠ 条目已不存在：' + stale }, stale)] : []),
                   ...form.ttyBooks.map((name) => jsx('option', { value: name, children: '连接簿：' + name }, name)),
                 ],
               }),
+              stale !== undefined
+                ? jsx('span', { className: 'dk_hint dk_hintWarn', children: `引用的条目「${stale}」不在 tty 连接簿里——请改选，或清空后手填` })
+                : null,
             ] }),
           jsx('button', { type: 'button', className: 'dk_btn dk_btnDanger', onClick: () => removeTarget(index), children: '删除' }),
           /*
@@ -5995,7 +6009,8 @@ window.__ModuleLoader__.load({
             (item.auth ?? 'agent') === 'password' ? jsx('input', { className: 'dk_input dk_credential', type: 'password', placeholder: item.passwordSet === true ? '（已设置，留空保持不变）' : 'env:SSH_PASSWORD', value: item.password ?? '', onChange: (event) => patchTarget(index, { password: event.target.value }) }) : null,
             jsx('label', { className: 'dk_check', children: [jsx('input', { type: 'checkbox', checked: item.agentForward === true, onChange: (event) => patchTarget(index, { agentForward: event.target.checked }) }), 'agent forwarding'] }),
           ] }) : null,
-        ] }, String(index) + item.name)),
+          ] }, String(index))
+        }),
         jsxs('div', { className: 'dk_row', children: [
           jsx('button', { type: 'button', className: 'dk_btn', onClick: addTarget, children: '添加目标' }),
           jsx('span', { className: 'dk_hint', children: 'SSH 目标推荐直接选 tty 终端面板的连接簿条目（凭证只需维护一处）；手填时密码 / 口令建议写 env:NAME（凭据引用：由官方凭据存储解析，缺失时退回环境变量）。' }),
@@ -6627,7 +6642,7 @@ window.__ModuleLoader__.load({
            * 宿主回显的实际连接（tty 挂在 tab.target 上的 `user@host[:port]`）。
            * 从连接簿打开的 SSH 标签，spec 里只有条目名、没有 host；少了这一份，
            * 「按 host 匹配目标」就整条失效——哪怕用户早就按同一台主机配过目标，
-           * 只是那条目标引用的是**另一条**连接簿条目（如 HS-248 vs 192.168.80.248）。
+           * 只是那条目标引用的是**另一条**连接簿条目（如 lab-a vs 192.0.2.10）。
            */
           const liveTarget = typeof payload?.tab?.target === 'string' ? payload.tab.target : ''
           const matched = matchTargetForSession(spec, bookName, liveTarget)

package/client-src/session-target.js

@@ -12,7 +12,7 @@
  * 两个来源共用它：宿主 `/targets` 里每个目标的 `label`，以及宿主在 ready 帧里
  * 回显、由 tty 挂在 `tab.target` 上的实际连接（`user@host[:port]`）。
  *
- * @param text 形如 `root@192.168.80.248` 或 `root@host:2222`；其它形态返回 undefined。
+ * @param text 形如 `root@192.0.2.10` 或 `root@host:2222`；其它形态返回 undefined。
  */
 export function parseUserHostPort(text) {
   const parsed = /^([^@]+)@(.+?)(?::(\d+))?$/.exec(typeof text === 'string' ? text : '')
@@ -27,8 +27,8 @@ export function parseUserHostPort(text) {
  * 退回宿主回显的 `tab.target` —— **从连接簿打开的 SSH 标签，spec 里只有条目名**
  * （tty 侧构造：`{t:'ssh', name: bookName, command}`），host 是空串。没有这层兜底，
  * 按 host 的匹配整条失效：会话主机显示为空、面板判成「该主机尚未配置为 Docker 目标」，
- * 哪怕用户早就按同一个主机配了目标（只是连接簿条目名不同，如 `HS-248` vs
- * `192.168.80.248`）。
+ * 哪怕用户早就按同一个主机配了目标（只是连接簿条目名不同，如 `lab-a` vs
+ * `192.0.2.10`）。
  *
  * @param spec tty 传过来的会话规格。
  * @param liveTarget 宿主回显的实际连接（tty 的 `tab.target`）。
@@ -71,7 +71,7 @@ export function pickTargetByHost(rows, session) {
  *     「该主机没配目标」，只能沿用上一次选的目标 → 屏幕上出现**另一台主机**的容器。
  *
  * 有了这一层，只要「会话走的那条连接簿」和「某条 docker 目标」指向同一台主机
- * （哪怕两条目标的 book 字段引用的是**不同**条目，如 HS-248 vs 192.168.80.248），
+ * （哪怕两条目标的 book 字段引用的是**不同**条目，如 lab-a vs 192.0.2.10），
  * 连接失败时也能对上目标——而那正是最需要面板的时候。
  *
  * @param bookName 会话所属的连接簿条目名（可能为空，如内联连接）。
@@ -90,3 +90,33 @@ export function bookSessionHost(bookName, rows) {
   }
   return undefined
 }
+
+/**
+ * 目标引用的连接簿条目是否**已失效**（引用的名字在 tty 连接簿里不存在）。
+ *
+ * 为什么需要它：`book` 那个下拉的候选项来自 `ttyBooks`，一旦引用的名字不在其中，
+ * 下拉会**渲染成空白**（没有任何 option 与之匹配）——界面上完全看不出"这里引用错了"，
+ * 直到真的去连才报「引用的连接簿条目不存在」。用户改了 tty 侧条目名（改名 / 重装 /
+ * 换机器）之后就会落到这个状态，且无从发现。
+ *
+ * 判定只针对 **ssh + 有 book** 的目标：本机目标不看连接簿；没填 book（走内联
+ * host/username）也不算失效——那是另一种合法配置。
+ *
+ * @param target 配置里的目标条目
+ * @param ttyBooks `/config.ttyBooks`：tty 连接簿的条目名数组
+ * @returns 引用的条目名（失效时），否则 undefined
+ */
+export function staleBookRef(target, ttyBooks) {
+  if (target === null || typeof target !== 'object') return undefined
+  if (target.kind !== 'ssh') return undefined
+  const book = typeof target.book === 'string' ? target.book.trim() : ''
+  if (book === '') return undefined
+  const books = Array.isArray(ttyBooks) ? ttyBooks : []
+  // tty 未安装 / 连接簿为空时**不算**引用失效：那是"宿主没装 tty"或"条目还没建"，
+  // 与"引用了一个不存在的名字"是两回事，标黄会误导（此时整体提示已在卡片顶部给过）。
+  if (books.length === 0) return undefined
+  for (const name of books) {
+    if (name === book) return undefined
+  }
+  return book
+}