@linxin666/dsh-remote-web-ui 0.3.23 → 0.4.0

package/lib/types/update.js

@@ -11,6 +11,8 @@
 import { readFileSync } from 'node:fs';
 import { dirname, join } from 'node:path';
 import { spawn } from 'node:child_process';
+import { createOutputCapture } from "./console-output.js";
+import { withIdentityEncoding } from "./http.js";
 /** npm registry base used for version probes. */
 export const NPM_REGISTRY = 'https://registry.npmjs.org';
 /** The family scope every dsh-web package is published under. */
@@ -229,7 +231,7 @@ export async function fetchLatestVersion(name, fetchImpl, timeoutMs = 10_000) {
         const controller = new AbortController();
         const timer = setTimeout(() => { controller.abort(); }, timeoutMs);
         try {
-            const response = await fetchImpl(NPM_REGISTRY + '/' + name.replace('/', '%2F') + '/latest', { signal: controller.signal });
+            const response = await fetchImpl(NPM_REGISTRY + '/' + name.replace('/', '%2F') + '/latest', withIdentityEncoding({ signal: controller.signal }));
             if (!response.ok)
                 return undefined;
             const body = await response.json();
@@ -312,10 +314,10 @@ export async function fetchGitHubReleaseNotes(version, fetchImpl = fetch, timeou
     const controller = new AbortController();
     const timer = setTimeout(() => { controller.abort(); }, timeoutMs);
     try {
-        const response = await fetchImpl(`https://api.github.com/repos/${UPDATE_RELEASE_REPO}/releases/tags/v${version}`, {
+        const response = await fetchImpl(`https://api.github.com/repos/${UPDATE_RELEASE_REPO}/releases/tags/v${version}`, withIdentityEncoding({
             headers: { accept: 'application/vnd.github+json', 'user-agent': 'dsh-web-update' },
             signal: controller.signal,
-        });
+        }));
         if (!response.ok)
             return undefined;
         const body = await response.json();
@@ -424,6 +426,13 @@ export async function checkUpdates(deps) {
 }
 /** Cap on captured pnpm output (keeps error payloads bounded). */
 const OUTPUT_CAP = 16 * 1024;
+/**
+ * Characters a package name may contain before it is handed to the update
+ * command line. Deliberately looser than the strict family shape (the tests
+ * and future callers use short names) but it admits no whitespace, quotes,
+ * shell metacharacters, or path separators beyond the scope slash.
+ */
+const SHELL_SAFE_PACKAGE_RE = /^@?[A-Za-z0-9][A-Za-z0-9._/-]*$/;
 /**
  * Windows cmd command-not-found stderr. With shell:true a missing shim
  * exits with code 1 or 9009 (cmd cannot report ENOENT), so the fallback chain
@@ -432,22 +441,14 @@ const OUTPUT_CAP = 16 * 1024;
  */
 const WIN_CMD_MISSING_RE = /not recognized as an internal or external command|不是内部或外部命令/i;
 /**
- * Decode process output with tolerant decoding:
- * Defaults to UTF-8; if invalid byte sequences are encountered (common on
- * Windows consoles using code page 936/GBK), falls back to GBK via TextDecoder.
+ * Fold one decoded candidate output into the running total under the display
+ * cap (the tail carries the failure reason).
+ * @param output - the accumulated text.
+ * @param text - the candidate's decoded text.
  */
-function decodeProcessChunk(chunk) {
-    try {
-        return new TextDecoder('utf-8', { fatal: true }).decode(chunk);
-    }
-    catch {
-        try {
-            return new TextDecoder('gbk').decode(chunk);
-        }
-        catch {
-            return chunk.toString('utf8');
-        }
-    }
+function capOutput(output, text) {
+    const combined = output + text;
+    return combined.length > OUTPUT_CAP ? combined.slice(combined.length - OUTPUT_CAP) : combined;
 }
 /**
  * Bypass for pnpm 11's supply-chain gate: `minimumReleaseAge` (default 24 h)
@@ -469,16 +470,24 @@ const MIN_RELEASE_AGE_OVERRIDE = '--config.minimumReleaseAge=0';
 export function runUpdate(deps) {
     return new Promise((resolve) => {
         const spawnImpl = deps.spawnImpl ?? spawn;
-        const packages = deps.packages;
         const platform = deps.platform ?? process.platform;
+        // The names come from the profile/anchor manifests and the win32 candidate
+        // runs through a shell, so anything outside the npm name shape is dropped
+        // instead of being concatenated into a command line (a dependency key
+        // containing shell metacharacters would otherwise run as a second command).
+        const packages = deps.packages.filter(name => SHELL_SAFE_PACKAGE_RE.test(name));
         // Windows ships pnpm/corepack/npx as .cmd shims; Node's spawn cannot
         // start .cmd files directly (ENOENT even when installed), so route
         // them through cmd.exe there. POSIX spawns stay shell-free.
+        // POSIX: the child leads its own process group (detached) so the timeout
+        // can signal pnpm's whole tree — pnpm spawns lifecycle-script children that
+        // a direct SIGTERM to the wrapper leaves running against the profile.
         const spawnOptions = {
             cwd: deps.profileDir,
             stdio: ['ignore', 'pipe', 'pipe'],
-            ...(platform === 'win32' ? { shell: true } : {}),
+            ...(platform === 'win32' ? { shell: true } : { detached: true }),
         };
+        const killImpl = deps.killImpl ?? process.kill;
         // Ordered fallback chain: each is tried only when the previous one is
         // missing on PATH (ENOENT on the spawn error event, never on close).
         // `--latest` is required: dsh plugin writes exact-version specs (e.g.
@@ -499,16 +508,22 @@ export function runUpdate(deps) {
         // (its tail is subject to OUTPUT_CAP). The win32 missing-command test must
         // run against `currentOutput` so a previous candidate's 'not recognized'
         // stderr cannot misclassify a real failure in the next one.
+        // Each candidate accumulates raw bytes and is decoded exactly once
+        // (console-output.ts): decoding per `data` event splits a multi-byte
+        // character that straddles two reads into replacement characters, which is
+        // what the panel then shows as garbled pnpm diagnostics.
         let output = '';
-        let currentOutput = '';
-        const append = (chunk) => {
-            const text = decodeProcessChunk(chunk);
-            output += text;
-            if (output.length > OUTPUT_CAP)
-                output = output.slice(output.length - OUTPUT_CAP);
-            currentOutput += text;
-            if (currentOutput.length > OUTPUT_CAP)
-                currentOutput = currentOutput.slice(currentOutput.length - OUTPUT_CAP);
+        let capture = createOutputCapture(OUTPUT_CAP);
+        let candidateFolded = false;
+        const append = (chunk) => { capture.push(chunk); };
+        /** This candidate's decoded text (idempotent). */
+        const candidateText = () => capture.read();
+        /** Fold this candidate's text into the running output exactly once. */
+        const foldCandidate = () => {
+            if (candidateFolded)
+                return;
+            candidateFolded = true;
+            output = capOutput(output, capture.read());
         };
         let currentChild;
         // Terminal guard: once a result is produced the promise is settled and no
@@ -531,7 +546,11 @@ export function runUpdate(deps) {
                 const pid = currentChild?.pid;
                 if (pid !== undefined && pid > 0) {
                     try {
-                        spawnImpl('taskkill', ['/pid', String(pid), '/t', '/f'], { stdio: 'ignore' });
+                        // A spawn failure arrives as an asynchronous 'error' event, which
+                        // the try/catch cannot see; an emitter with no error listener throws
+                        // and would take the host process down on this path.
+                        const killer = spawnImpl('taskkill', ['/pid', String(pid), '/t', '/f'], { stdio: 'ignore' });
+                        killer.on?.('error', () => { });
                     }
                     catch {
                         // Best-effort kill; fall through to the timeout result.
@@ -539,8 +558,26 @@ export function runUpdate(deps) {
                 }
             }
             else {
-                currentChild?.kill('SIGTERM');
+                // Detached child: its process-group id is its pid, so the negative pid
+                // reaches pnpm's grandchildren too. A group kill throws when the child
+                // already exited; fall back to the direct kill.
+                const pid = currentChild?.pid;
+                if (pid !== undefined && pid > 0) {
+                    try {
+                        killImpl(-pid, 'SIGTERM');
+                    }
+                    catch {
+                        try {
+                            currentChild?.kill('SIGTERM');
+                        }
+                        catch { /* already gone */ }
+                    }
+                }
+                else {
+                    currentChild?.kill('SIGTERM');
+                }
             }
+            foldCandidate();
             finish({ ok: false, exitCode: null, output, error: 'update timed out; install process killed', errorCode: 'timeout' });
         }, deps.timeoutMs ?? 10 * 60_000);
         const runCandidate = (index) => {
@@ -557,7 +594,8 @@ export function runUpdate(deps) {
                 return;
             }
             const candidate = candidates[index];
-            currentOutput = '';
+            capture = createOutputCapture(OUTPUT_CAP);
+            candidateFolded = false;
             const child = spawnImpl(candidate.command, candidate.args, spawnOptions);
             currentChild = child;
             let settled = false;
@@ -571,6 +609,7 @@ export function runUpdate(deps) {
             child.stderr?.on('data', append);
             child.on('error', (error) => {
                 once(() => {
+                    foldCandidate();
                     if (error.code === 'ENOENT') {
                         // Command missing: try the next candidate in the chain.
                         runCandidate(index + 1);
@@ -587,11 +626,13 @@ export function runUpdate(deps) {
                 // 'not recognized' or localized message instead of ENOENT — keep the
                 // chain going. Checked against this candidate's own output so a prior
                 // fallback's stderr cannot misclassify a real failure here.
-                if (platform === 'win32' && (code === 9009 || (code !== 0 && WIN_CMD_MISSING_RE.test(currentOutput)))) {
+                if (platform === 'win32' && (code === 9009 || (code !== 0 && WIN_CMD_MISSING_RE.test(candidateText())))) {
+                    foldCandidate();
                     runCandidate(index + 1);
                     return;
                 }
                 settled = true;
+                foldCandidate();
                 finish({
                     ok: code === 0,
                     exitCode: code,

package/package.json

@@ -1,7 +1,8 @@
 {
   "name": "@linxin666/dsh-remote-web-ui",
   "description": "Scan-to-pair remote access for the dsh web GUI that shares one official interface: a QR beside the settings button pairs phones and PCs into the same Web GUI (a portrait-touch adaptation layer for phones, full desktop on PCs) through one-time tokens and revocable device sessions, with a LAN bind toggle, optional Cloudflare tunnel, and one-click family self-update",
-  "version": "0.3.23",
+  "version": "0.4.0",
+  "icon": "icon.svg",
   "type": "module",
   "engines": {
     "node": "^22.19.0 || >=24.0.0"
@@ -31,12 +32,13 @@
     "lib/**/*.d.ts.map",
     "src",
     "assets",
-    "cordis.patch.yml"
+    "cordis.patch.yml",
+    "icon.svg"
   ],
   "license": "Apache-2.0",
   "dsh": {
     "engines": {
-      "dsh": ">=0.1.5-rc.1"
+      "dsh": ">=0.1.7-rc.1"
     },
     "bundle": {
       "patch": "./cordis.patch.yml"
@@ -58,27 +60,28 @@
     "zod": "^4.4.3"
   },
   "peerDependencies": {
+    "@deepseek-ai/dsh": ">=0.1.7-rc.1",
     "react": "^18.2.0",
     "react-dom": "^18.2.0"
   },
   "devDependencies": {
-    "@deepseek-ai/cordis": "^4.0.2",
-    "@deepseek-ai/dsh-agent": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-api-remotes": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-api-session-controller": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-api-workspace-controller": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-client-connection": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-client-locale": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-client-store": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-client-ui-primitives": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-client-ui-renderer": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-client-ui-settings": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-client-ui-sidebar": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-client-ui-slots": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-commands": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-host-webserver": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-invariants": "^0.1.5-rc.1",
-    "@deepseek-ai/dsh-settings": "^0.1.5-rc.1",
+    "@deepseek-ai/cordis": "^4.0.4",
+    "@deepseek-ai/dsh-agent": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-api-remotes": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-api-session-controller": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-api-workspace-controller": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-client-connection": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-client-locale": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-client-store": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-client-ui-primitives": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-client-ui-renderer": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-client-ui-settings": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-client-ui-sidebar": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-client-ui-slots": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-commands": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-host-webserver": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-invariants": "^0.1.7-rc.1",
+    "@deepseek-ai/dsh-settings": "^0.1.7-rc.1",
     "@testing-library/dom": "^10.4.1",
     "@testing-library/react": "^16.3.2",
     "@types/node": "^22.20.0",

package/src/client/FenceNotice.tsx

@@ -62,8 +62,6 @@ export function FenceNotice({ t, onRetry, onAccept = acceptPair }: FenceNoticePr
       }
       if (result.code === 'invalid') {
         setErrorMsg(t('fence.unpaired.tokenInvalid'))
-      } else if (result.code === 'used') {
-        setErrorMsg(t('fence.unpaired.tokenUsed'))
       } else {
         setErrorMsg(t('fence.unpaired.tokenFailed'))
       }

package/src/client/PluginSettingsCard.tsx

@@ -64,6 +64,10 @@ export interface PluginSettingsCardProps<TKey extends string = string> {
    * edits, so the footer would sit there permanently disabled.
    */
   hideFooter?: boolean
+  /** Render children even when the settings namespace is not exposed by the host. */
+  renderChildrenWhenNotExposed?: boolean
+  /** Hide the 'not exposed' warning banner when the settings namespace is not exposed. */
+  hideNotExposedNotice?: boolean
   /** The plugin's controls. */
   children: ReactNode
 }
@@ -126,14 +130,15 @@ export function PluginSettingsCard<TKey extends string = string>(props: PluginSe
   // official settings allowlist omits third-party namespaces): show a card
   // that explains the gap instead of vanishing, so a missing card never
   // reads as a missing plugin.
-  if (!state.exposed) {
+  if (!state.exposed && props.renderChildrenWhenNotExposed !== true) {
+    const showNotice = props.hideNotExposedNotice !== true
     return (
       <li className={cardClass}>
         {header}
         {expanded
           ? (
             <div className={css.body}>
-              <p className={css.notExposed} role="status">{props.t('settings.notExposed')}</p>
+              {showNotice ? <p className={css.notExposed} role="status">{props.t('settings.notExposed')}</p> : null}
             </div>
           )
           : null}

package/src/client/RemoteEntry.tsx

@@ -181,18 +181,23 @@ export function RemoteEntry({ wide, t }: RemoteEntryProps) {
   useEffect(() => closeEventSource, [closeEventSource])
 
   const handleStop = useCallback(() => {
-    // A failed stop request is harmless: the optimistic phase flip below
-    // keeps the UI honest, and the status stream confirms the stopped phase.
-    void stopPair().catch(() => {})
-    // Optimistic fallback; the status stream confirms with the stopped phase.
-    setState(previous => previous.kind === 'ready' ? { ...previous, phase: 'stopped' as PairingPhase, devices: [] } : previous)
+    // The phase flip lands only once the server accepted the stop. A refused
+    // stop changes nothing server-side and the service emits no state change
+    // (notify dedupes identical snapshots), so an optimistic flip would stick
+    // and claim a revocation that never happened.
+    void stopPair().then(() => {
+      setState(previous => previous.kind === 'ready' ? { ...previous, phase: 'stopped' as PairingPhase, devices: [] } : previous)
+    }).catch(() => {})
   }, [])
 
   const handleRevoke = useCallback((deviceId: string) => {
-    void revokePair(deviceId).catch(() => {})
-    setState(previous => previous.kind === 'ready'
-      ? { ...previous, devices: previous.devices.filter(device => device.id !== deviceId) }
-      : previous)
+    // Same rule as stop: a refused unpair must leave the row, or the panel
+    // claims a still-live session is gone.
+    void revokePair(deviceId).then(() => {
+      setState(previous => previous.kind === 'ready'
+        ? { ...previous, devices: previous.devices.filter(device => device.id !== deviceId) }
+        : previous)
+    }).catch(() => {})
   }, [])
 
   const handleRefresh = useCallback(() => {

package/src/client/RemotePanel.tsx

@@ -5,9 +5,10 @@
  * actions arrive through props from the entry's behavior component.
  */
 import clsx from 'clsx'
+import { memo } from 'react'
 import { QRCodeSVG } from 'qrcode.react'
 import {
-  IconCloseOutline16, IconCopyOutline16, IconRefreshOutline16, IconStopFill16,
+  IconCloseOutlineRegular, IconCopyOutlineRegular, IconRefreshOutlineRegular, IconStopFillRegular,
 } from '@deepseek-ai/dsh-client-ui-primitives'
 import type { TranslateNS } from '@deepseek-ai/dsh-client-ui-slots'
 import type { PairingPhase } from '../pairing.ts'
@@ -22,6 +23,15 @@ import {
 import { deviceNameFromUserAgent } from './device-name.ts'
 import css from './remote.module.css'
 
+/**
+ * The QR symbol, memoized on the link: the panel re-renders on every SSE state
+ * frame (a paired phone heartbeats every 10s), and qrcode.react rebuilds the
+ * SVG path and element tree on each render.
+ */
+const PairQrCode = memo(function PairQrCode({ url, className }: { url: string; className: string }) {
+  return <QRCodeSVG value={url} size={184} level="M" marginSize={1} className={className} />
+})
+
 /** The panel's view state, owned by the entry component. */
 export type PanelState =
   | { kind: 'lan-required'; tunnel?: TunnelStatusFrame }
@@ -115,7 +125,7 @@ export function RemotePanel({
           <p className={css.subtitle}>{t('subtitle')}</p>
         </div>
         <button type="button" className={css.close} aria-label={t('close.label')} onClick={onClose}>
-          <IconCloseOutline16 size={14} />
+          <IconCloseOutlineRegular size={14} />
         </button>
       </div>
 
@@ -155,7 +165,7 @@ export function RemotePanel({
               </span>
             </div>
             <div className={css.qrWrap} data-testid="remote-qr">
-              <QRCodeSVG value={state.url} size={184} level="M" marginSize={1} className={css.qr} />
+              <PairQrCode url={state.url} className={css.qr} />
             </div>
             {state.expired
               ? <p className={css.expired}>{t('pair.expired')}</p>
@@ -170,7 +180,7 @@ export function RemotePanel({
                 <code className={css.link} title={state.url}>{state.url}</code>
               </div>
               <button type="button" className={css.copyLink} onClick={() => onCopy(state.url)}>
-                <IconCopyOutline16 size={14} />
+                <IconCopyOutlineRegular size={14} />
                 {copied ? t('action.copied') : t('action.copyLink')}
               </button>
             </div>
@@ -185,7 +195,7 @@ export function RemotePanel({
                   className={css.copyLink}
                   onClick={() => onCopyToken ? onCopyToken(state.token!) : onCopy(state.token!)}
                 >
-                  <IconCopyOutline16 size={14} />
+                  <IconCopyOutlineRegular size={14} />
                   {copiedToken ? t('action.copiedToken') : t('action.copyToken')}
                 </button>
               </div>
@@ -246,11 +256,11 @@ export function RemotePanel({
 
           <div className={css.actions}>
             <button type="button" className={css.action} onClick={onStop}>
-              <IconStopFill16 size={14} />
+              <IconStopFillRegular size={14} />
               {t('action.stop')}
             </button>
             <button type="button" className={css.action} onClick={onRefresh}>
-              <IconRefreshOutline16 size={14} />
+              <IconRefreshOutlineRegular size={14} />
               {t('action.refresh')}
             </button>
           </div>

package/src/client/RemoteSettingsCard.tsx

@@ -1,16 +1,16 @@
 /**
  * The remote-control settings card: pairing security and device limits.
  * Registers into the `web-ui.plugin.item` child slot the Web UI plugin group
- * renders, bound to the `remote-web-ui` settings namespace.
+ * renders, bound to the `remote-web-ui` profile entry's configuration form.
  */
 
 import { useEffect, useState } from 'react'
 import type { InjectFace, PropsLocale, PropsRuntime } from '@deepseek-ai/dsh-client-ui-slots'
-import type { SettingsScope } from '@deepseek-ai/dsh-client-ui-settings/client'
+import type { ConfigForm } from '@deepseek-ai/dsh-client-ui-settings/client'
 import type { SnapshotStore } from '@deepseek-ai/dsh-client-store'
 import { PluginSettingsCard, ValueField, BooleanField } from './PluginSettingsCard.tsx'
 import { CardForm, booleanField, numberField, secretField, textField, type CardActions, type CardShell, type FieldState as CardFieldState } from './settings-form.ts'
-import { readLanBindStatus, type LanBindFrame } from './pair-api.ts'
+import { LanBindStatusError, readLanBindStatus, shouldStopLanBindPoll, type LanBindFrame } from './pair-api.ts'
 import type { TranslateNS } from '@deepseek-ai/dsh-client-ui-slots'
 
 /** The remote-control fields this card edits (the namespace's full schema). */
@@ -88,13 +88,13 @@ export interface RemoteSettingsCardFace extends CardActions {
   }
 }
 
-/** Bridges the `remote-web-ui` scope onto the card's staged form. */
+/** Bridges the `remote-web-ui` form onto the card's staged form. */
 export class RemoteSettingsCardController {
   private readonly form: CardForm<RemoteSettings>
   private readonly store: SnapshotStore<RemoteSettingsCardState>
 
-  /** @param scope - the bound settings scope for the `remote-web-ui` namespace. */
-  constructor(scope: SettingsScope<RemoteSettings>) {
+  /** @param scope - the configuration form for the `remote-web-ui` entry. */
+  constructor(scope: ConfigForm<RemoteSettings>) {
     this.form = new CardForm(scope, [
       booleanField('enabled'),
       numberField('tokenTtlMs'),
@@ -139,7 +139,7 @@ export class RemoteSettingsCardController {
   }
 
   /**
-   * Release the card's scope subscription and bound stores; the slot
+   * Release the card's form subscription and bound stores; the slot
    * disposer calls this on teardown.
    */
   dispose(): void {
@@ -175,6 +175,8 @@ export function RemoteSettingsCard(props: RemoteSettingsCardProps) {
       descriptionKey="settings.description"
       defaultOpen={false}
       state={state}
+      renderChildrenWhenNotExposed
+      hideNotExposedNotice
       onSave={props.save}
       onDiscard={props.discard}
     >
@@ -325,10 +327,15 @@ function LanBindStatus({ t }: { t: TranslateNS<'remote'> }) {
     const read = (): void => {
       void readLanBindStatus().then((value) => {
         if (alive) setFrame(value)
-      }).catch(() => {})
+      }).catch((error: unknown) => {
+        // The endpoint is loopback-only: a 401/403 means this origin (a paired
+        // phone) can never read it, so the 10s poll stops instead of retrying a
+        // known refusal for as long as the card stays expanded.
+        if (error instanceof LanBindStatusError && shouldStopLanBindPoll(error.status)) window.clearInterval(timer)
+      })
     }
-    read()
     const timer = window.setInterval(read, 10_000)
+    read()
     return () => {
       alive = false
       window.clearInterval(timer)

package/src/client/UpdateEntry.tsx

@@ -9,7 +9,7 @@
 import { useCallback, useEffect, useRef, useState } from "react"
 import { createPortal } from "react-dom"
 import type { TranslateNS } from '@deepseek-ai/dsh-client-ui-slots'
-import { IconDownloadOutline16 } from '@deepseek-ai/dsh-client-ui-primitives'
+import { IconDownloadOutlineRegular } from '@deepseek-ai/dsh-client-ui-primitives'
 import type { UpdateStatus } from '../update.ts'
 import { fetchUpdateStatus, runUpdate, UpdateStatusError } from "./update-api.ts"
 import { UpdatePanel, type UpdateView } from "./UpdatePanel.tsx"
@@ -128,7 +128,7 @@ export function UpdateEntry({ wide, t }: UpdateEntryProps) {
         title={updateLabel}
         onClick={openPanel}
       >
-        <IconDownloadOutline16 size={wide ? 16 : 18} />
+        <IconDownloadOutlineRegular size={wide ? 16 : 18} />
         {updateAvailable && wide && (
           <span className={css.updateBadgeText}>{t("update.badge")}</span>
         )}

package/src/client/UpdatePanel.tsx

@@ -5,7 +5,7 @@
  * through props from the entry behavior component.
  */
 import type { TranslateNS } from '@deepseek-ai/dsh-client-ui-slots'
-import { IconCloseOutline16, IconRefreshOutline16 } from '@deepseek-ai/dsh-client-ui-primitives'
+import { IconCloseOutlineRegular, IconRefreshOutlineRegular } from '@deepseek-ai/dsh-client-ui-primitives'
 import type { UpdateReleaseNotes, UpdateRunResult, UpdateStatus } from '../update.ts'
 import css from "./remote.module.css"
 
@@ -55,7 +55,7 @@ export function UpdatePanel({ t, view, onClose, onRecheck, onStartUpdate }: Upda
           {subtitle !== undefined && <p className={css.subtitle}>{subtitle}</p>}
         </div>
         <button type="button" className={css.close} aria-label={t("update.close")} onClick={onClose}>
-          <IconCloseOutline16 />
+          <IconCloseOutlineRegular />
         </button>
       </div>
       {view.kind === "checking" && <p className={css.updateStatus}>{t("update.checking")}</p>}
@@ -86,7 +86,7 @@ export function UpdatePanel({ t, view, onClose, onRecheck, onStartUpdate }: Upda
       {(view.kind === "done" || view.kind === "error") && (
         <div className={css.updateActions}>
           <button type="button" className={css.updateRetry} onClick={onRecheck}>
-            <IconRefreshOutline16 /> {t("update.retry")}
+            <IconRefreshOutlineRegular /> {t("update.retry")}
           </button>
         </div>
       )}

package/src/client/deep-link.ts

@@ -70,8 +70,8 @@ async function runAccept(token: string, page: PageSurface): Promise<void> {
   } catch {
     sessionStorage.setItem(PAIR_FAILED_MARKER, 'failed')
   }
-  // Drop the token from the URL either way: an accepted token is consumed
-  // (a re-scan would 409), and a failed one must not loop.
+  // Drop the token from the URL either way: it is spent for this tab (the
+  // session is established), and a failed one must not loop.
   const url = new URL(page.href)
   url.searchParams.delete('pair')
   page.replaceState(`${url.pathname}${url.search}${url.hash}`)