@riceawa/dsh-lan-gateway 0.5.3 → 0.5.5

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@riceawa/dsh-lan-gateway",
   "description": "LAN/internet reverse-proxy gateway for the DeepSeek Harness web GUI: binds 0.0.0.0 and forwards to the loopback dsh web server. Default-deny: every source (loopback, LAN, internet) must sign in with an HMAC session cookie unless lanPasswordless is explicitly enabled; against dsh >= 0.1.2-rc.1 the gateway relays one shared upstream browser session, so the harness's own authorization still gates every request. Fail-closed start guard (password required, plaintext needs an explicit opt-in), session revocation by epoch (password changes and secret rotation kill cookies and live WebSockets), same-site/Origin fence on HTTP and WebSocket upgrades, optional TLS (auto self-signed or user-supplied certs), and a Settings → Plugins card for live adjustment of port, CIDRs, auth, and TLS. Includes an insecure-origin UUID shim client bundle: on gateway-served plain-HTTP origins browsers lack crypto.randomUUID, so the client half patches a getRandomValues-backed randomUUID onto the Crypto prototype, fixing workspace open over LAN without touching DSH source.",
-  "version": "0.5.3",
+  "version": "0.5.5",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",

package/skills/lan-gateway.md

@@ -9,8 +9,9 @@ user-invocable: true
 
 The `dsh-lan-gateway` plugin lets the DeepSeek Harness web GUI be reached from the
 LAN and the wider internet. dsh itself binds only to loopback (the web CLI
-hard-refuses `0.0.0.0`), so this plugin runs its own reverse-proxy gateway on
-`0.0.0.0` that forwards to the loopback web server while rewriting Host/Origin.
+hard-refuses `0.0.0.0`), so this plugin runs its own reverse-proxy gateway on the
+unspecified address — both families, so IPv6 clients reach it too — forwarding to
+the loopback web server while rewriting Host/Origin.
 
 Since v0.5.0 the model is **default-deny** (post-QVD-2026-57410 hardening):
 
@@ -31,8 +32,8 @@ Since v0.5.0 the model is **default-deny** (post-QVD-2026-57410 hardening):
 Do not edit state files by hand — use the `lan_gateway` tool.
 
 - `lan_gateway` with `command: "status"` — is it listening, on which port, toward
-  which dsh port, password set?, session epoch, upstream-session relay state,
-  ingress encryption, last error.
+  which dsh port, password set?, session epoch, how many signed-out sessions are
+  still held, upstream-session relay state, ingress encryption, last error.
 - `lan_gateway` with `command: "enable"` — start listening. If it refuses (no
   password, legacy `authRequired: false`, plaintext without opt-in, `lanPasswordless`
   without a session-capable base), the message tells you what to change.
@@ -72,3 +73,6 @@ once (passwords and sessions travel in clear).
   without an Origin) does not.
 - Sessions don't survive a password change / `rotate-secret`: that is by design —
   the revocation epoch advanced and all cookies (and live WebSockets) were revoked.
+  Signing out is narrower: it revokes only the session that signed out, so that
+  session's cookie is dead even if a copy of it was kept elsewhere, while the
+  user's other devices stay signed in.

package/src/auth.ts

@@ -77,6 +77,13 @@ function normalizeAddress(raw: string): string {
 
 /**
  * Classify a source address string into one of the three trust tiers.
+ *
+ * The input is a *socket* address — `req.socket.remoteAddress`, unwrapped from
+ * its `::ffff:` mapping — which is a different domain from the URL hostname
+ * `isLoopbackHost` in `request-policy.ts` judges. The two agree on the common
+ * inputs but are not interchangeable: this one never sees `[::1]`, and that one
+ * never sees a mapped form. Both spans are documented where each lives.
+ *
  * @param remoteAddress - the raw value of `req.socket.remoteAddress`.
  * @param lanCidrs - CIDR strings treated as trusted LAN space (IPv4).
  * @returns the classification. IPv4-mapped IPv6 addresses are unwrapped.
@@ -100,16 +107,40 @@ export function classifySource(
   }
 
   if (address === '::1') return 'loopback'
-  // Link-local IPv6 fe80::/10.
-  if (address.toLowerCase().startsWith('fe80:')) return 'lan'
+  if (inIpv6LinkLocal(address)) return 'lan'
   return 'internet'
 }
 
+/**
+ * Whether a textual IPv6 address falls inside fe80::/10. The first ten bits are
+ * `1111111010`, so the leading hextet spans fe80–febf; a `startsWith('fe80:')`
+ * test covers only fe80::/16 and misclassifies fe90::–febf:: as internet.
+ */
+function inIpv6LinkLocal(address: string): boolean {
+  const match = /^([0-9a-fA-F]{1,4}):/.exec(address)
+  if (match === null) return false
+  return (Number.parseInt(match[1]!, 16) & 0xffc0) === 0xfe80
+}
+
 /** Encode a byte buffer as URL-safe base64 without padding. */
 function base64url(input: Buffer): string {
   return input.toString('base64url')
 }
 
+/** The claims a verified session cookie carries. */
+export interface SessionClaims {
+  /** Epoch millis at which the session expires. */
+  exp: number
+  /** The revocation epoch the cookie was minted under. */
+  epoch: number
+  /**
+   * Per-session id. Present on cookies minted from 0.5.4 on, which is what
+   * lets one session be retired on its own (sign-out) instead of retiring
+   * every session the password authorized. Absent on older cookies.
+   */
+  sid?: string
+}
+
 /**
  * Issue a signed session cookie value.
  * @param secret - the HMAC signing secret (base64 string).
@@ -118,28 +149,29 @@ function base64url(input: Buffer): string {
  *   cookie whose epoch no longer matches the live state is rejected by
  *   {@link verifyCookie}. Defaults to 0 (epoch-less, legacy) for callers that
  *   do not participate in revocation.
+ * @param sid - optional per-session id (see {@link SessionClaims.sid}).
  * @returns a `payload.signature` string suitable for the cookie value.
  */
-export function signCookie(secret: string, expiresMs: number, epoch: number = 0): string {
-  const payload = base64url(Buffer.from(JSON.stringify({ exp: expiresMs, epoch })))
+export function signCookie(secret: string, expiresMs: number, epoch: number = 0, sid?: string): string {
+  const claims = sid === undefined ? { exp: expiresMs, epoch } : { exp: expiresMs, epoch, sid }
+  const payload = base64url(Buffer.from(JSON.stringify(claims)))
   const sig = createHmac('sha256', secret).update(payload).digest('base64url')
   return `${payload}.${sig}`
 }
 
 /**
- * Whether a cookie value is a valid, unexpired session signed with `secret`
- * and minted under `epoch`. Epoch-less cookies (legacy payloads) count as
- * epoch 0, so an upgrade from a pre-0.5.0 state does not log everyone out.
+ * Verify a cookie's signature, expiry and epoch.
+ * @returns the claims it carries, or undefined when it is not a valid session.
  */
-export function verifyCookie(
+export function verifySession(
   secret: string,
   value: string | undefined,
   now: number,
   epoch: number = 0,
-): boolean {
-  if (value === undefined) return false
+): SessionClaims | undefined {
+  if (value === undefined) return undefined
   const dot = value.indexOf('.')
-  if (dot === -1) return false
+  if (dot === -1) return undefined
   const payload = value.slice(0, dot)
   const sig = value.slice(dot + 1)
   const expected = createHmac('sha256', secret).update(payload).digest()
@@ -147,17 +179,22 @@ export function verifyCookie(
   try {
     actual = Buffer.from(sig, 'base64url')
   } catch {
-    return false
+    return undefined
   }
-  if (expected.length !== actual.length) return false
-  if (!timingSafeEqual(expected, actual)) return false
+  if (expected.length !== actual.length) return undefined
+  if (!timingSafeEqual(expected, actual)) return undefined
   try {
-    const decoded = JSON.parse(Buffer.from(payload, 'base64url').toString('utf8')) as { exp?: unknown; epoch?: unknown }
-    if (typeof decoded.exp !== 'number' || decoded.exp <= now) return false
+    const decoded = JSON.parse(Buffer.from(payload, 'base64url').toString('utf8')) as Partial<SessionClaims>
+    if (typeof decoded.exp !== 'number' || decoded.exp <= now) return undefined
     const cookieEpoch = typeof decoded.epoch === 'number' ? decoded.epoch : 0
-    return cookieEpoch === epoch
+    if (cookieEpoch !== epoch) return undefined
+    return {
+      exp: decoded.exp,
+      epoch: cookieEpoch,
+      ...(typeof decoded.sid === 'string' ? { sid: decoded.sid } : {}),
+    }
   } catch {
-    return false
+    return undefined
   }
 }
 
@@ -181,7 +218,15 @@ export function originMatchesHost(origin: string | undefined, host: string | und
 
 /** A token bucket limiter keyed by source address. */
 export class RateLimiter {
+  /**
+   * Hard ceiling on tracked sources. Expiry alone only reclaims a bucket when
+   * `prune` runs, and a spray from many distinct addresses inside one window
+   * outruns it, so the map also sheds its soonest-expiring entries past this.
+   */
+  private static readonly MAX_BUCKETS = 10_000
   private readonly buckets = new Map<string, { tokens: number; resetAt: number }>()
+  /** Epoch millis at which the next opportunistic sweep is due. */
+  private nextPruneAt = 0
   constructor(
     private readonly maxTokens: number,
     private readonly windowMs: number,
@@ -194,22 +239,42 @@ export class RateLimiter {
    */
   allow(key: string): boolean {
     const now = Date.now()
-    const bucket = this.buckets.get(key)
-    if (bucket === undefined || bucket.resetAt <= now) {
-      this.buckets.set(key, { tokens: this.maxTokens - 1, resetAt: now + this.windowMs })
-      return true
+    // Sweep on a rolling window. Without this, expired buckets are only
+    // replaced when their own key returns, so every address that ever posted
+    // to the login route keeps an entry for the life of the process.
+    if (now >= this.nextPruneAt) {
+      this.prune(now)
+      this.nextPruneAt = now + this.windowMs
     }
-    if (bucket.tokens > 0) {
-      bucket.tokens -= 1
+    const existing = this.buckets.get(key)
+    if (existing !== undefined && existing.resetAt > now) {
+      if (existing.tokens <= 0) return false
+      existing.tokens -= 1
       return true
     }
-    return false
+    // A new bucket. A spray of distinct addresses inside one window outruns
+    // expiry, so shed the closest-to-expiring entries first — they are the
+    // ones about to lapse anyway, so the eviction costs the least fidelity.
+    if (existing === undefined && this.buckets.size >= RateLimiter.MAX_BUCKETS) {
+      this.evictSoonestToExpire()
+    }
+    this.buckets.set(key, { tokens: this.maxTokens - 1, resetAt: now + this.windowMs })
+    return true
   }
 
-  /** Drop expired buckets to bound memory. */
+  /** Drop expired buckets to bound memory. Called from {@link allow}. */
   prune(now: number = Date.now()): void {
     for (const [key, bucket] of this.buckets) {
       if (bucket.resetAt <= now) this.buckets.delete(key)
     }
   }
+
+  /** Trim back to 90% of the ceiling, oldest expiry first. */
+  private evictSoonestToExpire(): void {
+    const target = Math.floor(RateLimiter.MAX_BUCKETS * 0.9)
+    const byExpiry = [...this.buckets.entries()].sort((a, b) => a[1].resetAt - b[1].resetAt)
+    for (const [key] of byExpiry.slice(0, Math.max(0, this.buckets.size - target))) {
+      this.buckets.delete(key)
+    }
+  }
 }

package/src/client/lan-gateway-card.tsx

@@ -12,6 +12,14 @@
 
 import { useEffect, useState, type ChangeEvent, type ReactNode } from 'react'
 import type { PropsRuntime } from '@deepseek-ai/dsh-client-ui-slots'
+import {
+  FIELDS,
+  TRISTATE_OPTIONS,
+  formatValue,
+  parseValue,
+  type FieldDef,
+  type LanGatewaySettings,
+} from '../config-fields.ts'
 
 /**
  * The official Settings → Plugins page declares the `settings.plugin.item`
@@ -27,27 +35,20 @@ declare module '@deepseek-ai/dsh-client-ui-slots' {
   }
 }
 
-/** Props the renderer binds for this card (unused — the card is self-loading). */
+/**
+ * Props the renderer binds for this card (unused — the card is self-loading).
+ */
 export type LanGatewayCardProps = PropsRuntime<'settings.plugin.item'>
 
-/** The wire shape of the `lan-gateway` config section. */
-export interface LanGatewaySettings {
-  enabled?: boolean
-  gatewayPort?: number
-  dshTargetPort?: number
-  lanCidrs?: string[]
-  lanPasswordless?: boolean
-  cookieMaxAgeDays?: number
-  tlsEnabled?: boolean
-  tlsMode?: 'self-signed' | 'custom'
-  tlsCertPath?: string
-  tlsKeyPath?: string
-  tlsSelfSignedHosts?: string
-  tlsCertMaxAgeDays?: number
-  allowInsecurePlaintext?: boolean
-  trustedTerminator?: string
-  secureCookies?: boolean
-}
+/**
+ * The card's field table and value codecs live in `config-fields.ts`, shared
+ * with the host: the host's config route decides which submitted keys are
+ * editable and which empty value means "clear", and a table duplicated here
+ * would let the two disagree about a field the card can render but the route
+ * would refuse. Re-exported so the existing tests keep their import path.
+ */
+export { FIELDS, TRISTATE_OPTIONS, formatValue, parseValue }
+export type { FieldDef, LanGatewaySettings }
 
 /** GET /lan-gateway/config response. */
 interface RouteState {
@@ -70,7 +71,6 @@ interface Labels {
   saving: string
   discard: string
   reset: string
-  overridden: string
   readOnly: string
   saveFailed: string
   loadFailed: string
@@ -93,10 +93,9 @@ const LABELS: Record<'zh' | 'en', Labels> = {
     saving: '保存中…',
     discard: '放弃',
     reset: '重置',
-    overridden: '已覆盖',
-    readOnly: '网关设置当前不可用（读不到配置路由）。',
+    readOnly: '网关设置只能在宿主机本机打开 dsh web 时修改：配置路由仅监听回环地址，经网关远程访问的浏览器会被拒绝。远程请改用 lan_gateway 工具。',
     saveFailed: '保存未生效，请检查输入后重试。',
-    loadFailed: '加载网关配置失败。',
+    loadFailed: '无法读取网关配置',
     emptyMeansClear: '留空 = 使用默认',
     running: '运行中',
     stopped: '已停止',
@@ -115,7 +114,7 @@ const LABELS: Record<'zh' | 'en', Labels> = {
     'field.allowInsecurePlaintext': '允许明文 HTTP',
     'hint.allowInsecurePlaintext': '危险：关闭 TLS 或受信终止代理时仍启动监听，密码与会话将以明文传输',
     'field.trustedTerminator': '受信 TLS 终止代理',
-    'hint.trustedTerminator': '可选：声明前置代理标识，视为加密入口（如 nginx）。留空 = 未声明',
+    'hint.trustedTerminator': '可选：声明前置代理标识，视为加密入口（如 nginx）。留空 = 未声明。注意：登录限流以 TCP 源地址为键，代理之后所有浏览器共用一个额度（5 次/分钟）',
     'field.secureCookies': '会话 cookie 的 Secure 属性',
     'hint.secureCookies': '自动 = TLS 或已声明受信终止代理时加 Secure。受信代理只做明文鉴权、浏览器走 http 访问时须设为 false，否则浏览器拒收 Secure cookie，登录会无限弹回登录页',
     'opt.auto': '自动',
@@ -128,13 +127,13 @@ const LABELS: Record<'zh' | 'en', Labels> = {
     'field.tlsMode': '证书来源',
     'hint.tlsMode': 'self-signed = 自动生成自签名证书；custom = 使用自己的证书',
     'field.tlsSelfSignedHosts': '自签名证书域名/IP',
-    'hint.tlsSelfSignedHosts': '逗号分隔，写入证书 SAN，如 localhost, 192.168.1.5',
+    'hint.tlsSelfSignedHosts': '逗号分隔，写入证书 SAN，如 localhost, 192.168.1.5。仅影响下次换发：已有证书沿用至到期，改动不会立刻生效',
     'field.tlsCertPath': '证书文件路径（custom）',
     'hint.tlsCertPath': 'PEM 格式证书（或证书链）的绝对路径',
     'field.tlsKeyPath': '私钥文件路径（custom）',
     'hint.tlsKeyPath': '与证书配套的 PEM 私钥绝对路径',
     'field.tlsCertMaxAgeDays': '自签名证书有效期（天）',
-    'hint.tlsCertMaxAgeDays': '默认 825（约 27 个月）',
+    'hint.tlsCertMaxAgeDays': '默认 825（约 27 个月）。仅影响下次换发：已有证书沿用至到期',
   },
   en: {
     title: 'LAN Gateway',
@@ -144,10 +143,9 @@ const LABELS: Record<'zh' | 'en', Labels> = {
     saving: 'Saving…',
     discard: 'Discard',
     reset: 'Reset',
-    overridden: 'overridden',
-    readOnly: 'Gateway settings unavailable (config route unreachable).',
+    readOnly: 'Gateway settings can only be changed where dsh web runs locally: the config route listens on loopback only, so a browser reaching dsh through the gateway is refused. Use the lan_gateway tool remotely.',
     saveFailed: 'The save did not land — check the inputs and retry.',
-    loadFailed: 'Failed to load gateway configuration.',
+    loadFailed: 'Cannot read the gateway configuration',
     emptyMeansClear: 'Empty = default',
     running: 'Running',
     stopped: 'Stopped',
@@ -166,7 +164,7 @@ const LABELS: Record<'zh' | 'en', Labels> = {
     'field.allowInsecurePlaintext': 'Allow plaintext HTTP',
     'hint.allowInsecurePlaintext': 'Dangerous: start the listener even without TLS or a trusted terminator; passwords and sessions travel in clear',
     'field.trustedTerminator': 'Trusted TLS terminator',
-    'hint.trustedTerminator': 'Optional identifier for a front proxy (e.g. nginx) treated as the encrypted ingress. Empty = none declared',
+    'hint.trustedTerminator': 'Optional identifier for a front proxy (e.g. nginx) treated as the encrypted ingress. Empty = none declared. Note: login rate limiting keys on the TCP source address, so behind a proxy every browser shares one budget (5/min)',
     'field.secureCookies': 'Session cookie Secure attribute',
     'hint.secureCookies': 'Auto = Secure when TLS or a trusted terminator is declared. Set false when the trusted proxy only authenticates over plaintext and browsers reach it over http — otherwise browsers drop the Secure cookie and every login bounces back to the login page',
     'opt.auto': 'Auto',
@@ -179,13 +177,13 @@ const LABELS: Record<'zh' | 'en', Labels> = {
     'field.tlsMode': 'Certificate source',
     'hint.tlsMode': 'self-signed = auto-generated certificate; custom = your own files',
     'field.tlsSelfSignedHosts': 'Self-signed hosts (SANs)',
-    'hint.tlsSelfSignedHosts': 'Comma separated DNS/IP names, e.g. localhost, 192.168.1.5',
+    'hint.tlsSelfSignedHosts': 'Comma separated DNS/IP names, e.g. localhost, 192.168.1.5. Applies to the next issuance only: an existing certificate is reused until it expires',
     'field.tlsCertPath': 'Certificate path (custom)',
     'hint.tlsCertPath': 'Absolute path to a PEM certificate (or chain)',
     'field.tlsKeyPath': 'Private key path (custom)',
     'hint.tlsKeyPath': 'Absolute path to the matching PEM private key',
     'field.tlsCertMaxAgeDays': 'Self-signed validity (days)',
-    'hint.tlsCertMaxAgeDays': 'Default 825 (about 27 months)',
+    'hint.tlsCertMaxAgeDays': 'Default 825 (about 27 months). Applies to the next issuance only: an existing certificate is reused until it expires',
   },
 }
 
@@ -194,93 +192,6 @@ function labels(): Labels {
   return lang.startsWith('zh') ? LABELS.zh : LABELS.en
 }
 
-/* ------------------------------------------------------------------ */
-/* Field model                                                         */
-/* ------------------------------------------------------------------ */
-
-type FieldKind = 'boolean' | 'number' | 'text' | 'cidrs' | 'select' | 'tristate'
-
-interface FieldDef {
-  field: keyof LanGatewaySettings
-  kind: FieldKind
-  optional?: boolean
-  options?: readonly string[]
-}
-
-/**
- * The card's field table and its two value codecs are exported for tests: the
- * tri-state codec is the load-bearing part of the settings round-trip (an
- * unset value must stay distinguishable from an explicit false, or the
- * plaintext-proxy escape hatch silently reverts).
- */
-export { TRISTATE_OPTIONS, FIELDS, formatValue, parseValue }
-export type { FieldDef, Write }
-
-/** The three states of a tri-state field, in display order. */
-const TRISTATE_OPTIONS = ['auto', 'true', 'false'] as const
-
-const FIELDS: readonly FieldDef[] = [
-  { field: 'enabled', kind: 'boolean' },
-  { field: 'gatewayPort', kind: 'number' },
-  { field: 'dshTargetPort', kind: 'number', optional: true },
-  { field: 'lanCidrs', kind: 'cidrs' },
-  { field: 'lanPasswordless', kind: 'boolean' },
-  { field: 'cookieMaxAgeDays', kind: 'number' },
-  { field: 'tlsEnabled', kind: 'boolean' },
-  { field: 'tlsMode', kind: 'select', options: ['self-signed', 'custom'] },
-  { field: 'tlsSelfSignedHosts', kind: 'text' },
-  { field: 'tlsCertPath', kind: 'text', optional: true },
-  { field: 'tlsKeyPath', kind: 'text', optional: true },
-  { field: 'tlsCertMaxAgeDays', kind: 'number' },
-  { field: 'allowInsecurePlaintext', kind: 'boolean' },
-  { field: 'trustedTerminator', kind: 'text', optional: true },
-  { field: 'secureCookies', kind: 'tristate' },
-]
-
-function formatValue(def: FieldDef, value: unknown): string {
-  switch (def.kind) {
-    case 'boolean': return value === true ? 'true' : 'false'
-    case 'number': return typeof value === 'number' ? String(value) : ''
-    case 'cidrs': return Array.isArray(value) ? value.join(', ') : ''
-    case 'select': return typeof value === 'string' ? value : (def.options?.[0] ?? '')
-    // Tri-state: an unset value is a distinct third state ("auto"), never "false".
-    case 'tristate': return value === true ? 'true' : value === false ? 'false' : 'auto'
-    case 'text': return typeof value === 'string' ? value : ''
-  }
-}
-
-type Write = { kind: 'set'; value: unknown } | { kind: 'clear' }
-
-/** Parse draft text into a value for the POST body; undefined blocks saving. */
-function parseValue(def: FieldDef, text: string): Write | undefined {
-  const trimmed = text.trim()
-  switch (def.kind) {
-    case 'boolean':
-      if (trimmed === 'true') return { kind: 'set', value: true }
-      if (trimmed === 'false') return { kind: 'set', value: false }
-      return undefined
-    case 'number':
-      if (trimmed === '') return def.optional ? { kind: 'clear' } : undefined
-      if (!/^\d+$/.test(trimmed)) return undefined
-      return { kind: 'set', value: Number(trimmed) }
-    case 'cidrs': {
-      const cidrs = trimmed.split(',').map(s => s.trim()).filter(s => s !== '')
-      return cidrs.length === 0 ? { kind: 'clear' } : { kind: 'set', value: cidrs }
-    }
-    case 'select':
-      return def.options?.includes(trimmed) ? { kind: 'set', value: trimmed } : undefined
-    case 'tristate':
-      // 'auto' clears the key so it re-inherits the composition layer (and the
-      // resolution rule), which is what an unset tri-state means.
-      if (trimmed === 'auto') return { kind: 'clear' }
-      if (trimmed === 'true') return { kind: 'set', value: true }
-      if (trimmed === 'false') return { kind: 'set', value: false }
-      return undefined
-    case 'text':
-      return trimmed === '' ? (def.optional ? { kind: 'clear' } : undefined) : { kind: 'set', value: trimmed }
-  }
-}
-
 /* ------------------------------------------------------------------ */
 /* Card                                                                */
 /* ------------------------------------------------------------------ */
@@ -314,7 +225,26 @@ export function LanGatewayCard(_props: LanGatewayCardProps): ReactNode {
     return () => { cancelled = true }
   }, [])
 
-  if (loadFailed) return null
+  // A remote browser reaches this card through the gateway, which answers 403
+  // for the plugin's own prefix by design, so the route is unreachable exactly
+  // where a user is most likely to go looking for the setting. Rendering
+  // nothing left them with a blank entry and no way to tell a missing card from
+  // a broken one; say what is wrong and where the card does work instead.
+  if (loadFailed) {
+    return (
+      <li style={styles.card}>
+        <div style={styles.header}>
+          <span style={styles.headerTop}>
+            <span style={styles.name}>{t.title}</span>
+          </span>
+          <span style={styles.description}>{t.loadFailed}</span>
+        </div>
+        <div style={styles.body}>
+          <p style={styles.hint}>{t.readOnly}</p>
+        </div>
+      </li>
+    )
+  }
   if (route === null) return null
 
   const { config } = route
@@ -352,18 +282,22 @@ export function LanGatewayCard(_props: LanGatewayCardProps): ReactNode {
     setSaving(true)
     setFailed(null)
     try {
-      // Build the next full config: the loaded one with drafts applied.
-      const next: Record<string, unknown> = {}
-      for (const def of FIELDS) {
-        const text = drafts[def.field] ?? formatValue(def, config[def.field])
-        const write = parseValue(def, text)
+      // A patch of the edited fields only, never the whole config: the card
+      // cannot express every key the section may hold (a custom `cookieName`,
+      // say), and posting a synthesized full config made the route treat those
+      // keys as submitted — resetting each one to its schema default.
+      const patch: Record<string, unknown> = {}
+      for (const [field, text] of Object.entries(drafts)) {
+        const def = FIELDS.find(f => f.field === field)
+        if (def === undefined) continue
+        const write = parseValue(def, text ?? '')
         if (write === undefined) continue
-        next[def.field] = write.kind === 'clear' ? null : write.value
+        patch[field] = write.kind === 'clear' ? null : write.value
       }
       const response = await fetch('/lan-gateway/config', {
         method: 'POST',
         headers: { 'content-type': 'application/json' },
-        body: JSON.stringify(next),
+        body: JSON.stringify(patch),
       })
       const body = await response.json().catch(() => ({})) as Partial<RouteState> & { error?: string }
       if (!response.ok) {

package/src/config-fields.ts

@@ -0,0 +1,137 @@
+/**
+ * The `lan-gateway` configuration field table: which settings keys exist, how
+ * each one is rendered, and how a draft string becomes a config value or a
+ * clear.
+ *
+ * The host and the browser card share this module. The host derives the key set
+ * its config route accepts and the set an empty value clears from it; the card
+ * renders its controls from it. A field therefore cannot be editable on one
+ * side and unknown on the other, which is what let the card silently rewrite
+ * keys it never showed.
+ *
+ * Because both halves import it, this module must stay dependency-free and
+ * side-effect-free: no schemastery (the host schema remains the validating
+ * authority, not this table), no node built-ins, no DOM, no I/O.
+ *
+ * @module @riceawa/dsh-lan-gateway/config-fields
+ */
+
+/**
+ * The wire shape of the `lan-gateway` config section, as the Settings card sees
+ * it. Deliberately narrower than the host's `Config`: `cookieName` and the
+ * retired `authRequired` are not card-editable. The config route applies a
+ * patch, so a key absent here is left untouched rather than reset.
+ */
+export interface LanGatewaySettings {
+  enabled?: boolean
+  gatewayPort?: number
+  dshTargetPort?: number
+  lanCidrs?: string[]
+  lanPasswordless?: boolean
+  cookieMaxAgeDays?: number
+  tlsEnabled?: boolean
+  tlsMode?: 'self-signed' | 'custom'
+  tlsCertPath?: string
+  tlsKeyPath?: string
+  tlsSelfSignedHosts?: string
+  tlsCertMaxAgeDays?: number
+  allowInsecurePlaintext?: boolean
+  trustedTerminator?: string
+  secureCookies?: boolean
+}
+
+/** A settings key this table knows how to edit. */
+export type ConfigFieldKey = keyof LanGatewaySettings
+
+/** How a field is rendered and parsed. */
+export type FieldKind = 'boolean' | 'number' | 'text' | 'cidrs' | 'select' | 'tristate'
+
+export interface FieldDef {
+  readonly field: ConfigFieldKey
+  readonly kind: FieldKind
+  /** An empty draft clears the key back to the composition layer. */
+  readonly optional?: boolean
+  /** The accepted values of a `select` field. */
+  readonly options?: readonly string[]
+}
+
+/** The three states of a tri-state field, in display order. */
+export const TRISTATE_OPTIONS = ['auto', 'true', 'false'] as const
+
+/**
+ * The editable settings, in display order. Adding a config key means adding it
+ * here (the host whitelist and the card's controls both follow), to the
+ * `Config` schema in `index.ts`, and to `listenerKey` when it changes listener
+ * behavior.
+ */
+export const FIELDS: readonly FieldDef[] = [
+  { field: 'enabled', kind: 'boolean' },
+  { field: 'gatewayPort', kind: 'number' },
+  { field: 'dshTargetPort', kind: 'number', optional: true },
+  { field: 'lanCidrs', kind: 'cidrs' },
+  { field: 'lanPasswordless', kind: 'boolean' },
+  { field: 'cookieMaxAgeDays', kind: 'number' },
+  { field: 'tlsEnabled', kind: 'boolean' },
+  { field: 'tlsMode', kind: 'select', options: ['self-signed', 'custom'] },
+  { field: 'tlsSelfSignedHosts', kind: 'text' },
+  { field: 'tlsCertPath', kind: 'text', optional: true },
+  { field: 'tlsKeyPath', kind: 'text', optional: true },
+  { field: 'tlsCertMaxAgeDays', kind: 'number' },
+  { field: 'allowInsecurePlaintext', kind: 'boolean' },
+  { field: 'trustedTerminator', kind: 'text', optional: true },
+  { field: 'secureCookies', kind: 'tristate' },
+]
+
+/** Every settings key the config route accepts; anything else is ignored. */
+export const CONFIG_FIELD_KEYS: ReadonlySet<string> = new Set(FIELDS.map(def => def.field as string))
+
+/** Keys an empty submitted value clears back to the composition layer. */
+export const OPTIONAL_CONFIG_KEYS: ReadonlySet<string> = new Set(
+  FIELDS.filter(def => def.optional === true).map(def => def.field as string),
+)
+
+/** Render a stored value as draft text. */
+export function formatValue(def: FieldDef, value: unknown): string {
+  switch (def.kind) {
+    case 'boolean': return value === true ? 'true' : 'false'
+    case 'number': return typeof value === 'number' ? String(value) : ''
+    case 'cidrs': return Array.isArray(value) ? value.join(', ') : ''
+    case 'select': return typeof value === 'string' ? value : (def.options?.[0] ?? '')
+    // Tri-state: an unset value is a distinct third state ("auto"), never "false".
+    case 'tristate': return value === true ? 'true' : value === false ? 'false' : 'auto'
+    case 'text': return typeof value === 'string' ? value : ''
+  }
+}
+
+/** One field's contribution to a config patch. */
+export type Write = { kind: 'set'; value: unknown } | { kind: 'clear' }
+
+/** Parse draft text into a value for the POST body; undefined blocks saving. */
+export function parseValue(def: FieldDef, text: string): Write | undefined {
+  const trimmed = text.trim()
+  switch (def.kind) {
+    case 'boolean':
+      if (trimmed === 'true') return { kind: 'set', value: true }
+      if (trimmed === 'false') return { kind: 'set', value: false }
+      return undefined
+    case 'number':
+      if (trimmed === '') return def.optional ? { kind: 'clear' } : undefined
+      if (!/^\d+$/.test(trimmed)) return undefined
+      return { kind: 'set', value: Number(trimmed) }
+    case 'cidrs': {
+      const cidrs = trimmed.split(',').map(s => s.trim()).filter(s => s !== '')
+      return cidrs.length === 0 ? { kind: 'clear' } : { kind: 'set', value: cidrs }
+    }
+    case 'select':
+      return def.options?.includes(trimmed) ? { kind: 'set', value: trimmed } : undefined
+    case 'tristate':
+      // 'auto' clears the key so it re-inherits the composition layer (and the
+      // resolution rule), which is what an unset tri-state means.
+      if (trimmed === 'auto') return { kind: 'clear' }
+      if (trimmed === 'true') return { kind: 'set', value: true }
+      if (trimmed === 'false') return { kind: 'set', value: false }
+      return undefined
+    case 'text':
+      return trimmed === '' ? (def.optional ? { kind: 'clear' } : undefined) : { kind: 'set', value: trimmed }
+  }
+}