fix(client): focus only overflowing command text

This commit is contained in:
Tianyi Cui committed 2026-10-07 02:04:33 +08:00
1 parent e8b9e119da
commit 88311637f8
30 files changed
+329 -68

No files matched your search

@@ -6,7 +6,9 @@
- button "收起安装详情" [expanded]
- paragraph: 安装位置:{{cwd}}/.dsh-home/profiles/scaffold
- paragraph: 第 1 次 · 中国大陆镜像源
- text: 失败 $ pnpm add mirrored-package --registry=https://registry.npmmirror.com/ 退出码 1
- text: 失败 $
- group "命令第 1 行": pnpm add mirrored-package --registry=https://registry.npmmirror.com/
- text: 退出码 1
- button "复制"
- text: "ERR_PNPM_META_FETCH_FAIL GET https://registry.npmmirror.com/mirrored-package: socket hang up"
- paragraph: 第 2 次 · npm 官方源
@@ -1,3 +1,5 @@
- button "Bash Start the diagnostic worker" [expanded]
- text: "IN { \"command\": \"node ./scripts/start-worker.mjs --workspace=/tmp/diagnostics/long-workspace-name-for-background-command-details --log-level=debug --trace-tag=background-shell-full-command-selection --include=worker,lifecycle,transport,subscriptions --output=/tmp/diagnostics/background-worker.log\", \"description\": \"Start the diagnostic worker\", \"run_in_background\": true } OUT started background job bash-1"
- text: IN
- group "Command arguments": "{ \"command\": \"node ./scripts/start-worker.mjs --workspace=/tmp/diagnostics/long-workspace-name-for-background-command-details --log-level=debug --trace-tag=background-shell-full-command-selection --include=worker,lifecycle,transport,subscriptions --output=/tmp/diagnostics/background-worker.log\", \"description\": \"Start the diagnostic worker\", \"run_in_background\": true }"
- text: OUT started background job bash-1
- button "Inspect"
@@ -119,7 +119,7 @@ it.each(['https://registry.npmjs.org/', MIRROR])('selects the fastest responding
await dialog.getByText('第 1 次 · 中国大陆镜像源', { exact: true }).waitFor()
await dialog.getByText('第 2 次 · npm 官方源', { exact: true }).waitFor()
await expectSelectableTerminalCommand(page, dialog.locator('[data-terminal]').first(),
`pnpm add mirrored-package --registry=${MIRROR}`, 380)
`pnpm add mirrored-package --registry=${MIRROR}`, 380, '命令第 1 行')
expect(JSON.parse(await readFile(manifestPath, 'utf8'))).toMatchObject({ dependencies: { 'mirrored-package': '2.0.0' } })
const installed = (await captureStableAria(page, '[role="dialog"]', scaffold.workspaceCwd))
.split(process.execPath).join('{{node}}')
+4 -3
View File
@@ -8,7 +8,7 @@ import {
seedSession, watchConsole, webSnapshotMode,
} from './scaffold.ts'
import { expandOwningTurnProcess, newEnglishPage } from './support.ts'
import { expectSelectableCommandText } from './terminal-command-browser.ts'
import { expectCommandFocusAfterResize, expectSelectableCommandText } from './terminal-command-browser.ts'
const COMMAND = 'node ./scripts/start-worker.mjs --workspace=/tmp/diagnostics/long-workspace-name-for-background-command-details --log-level=debug --trace-tag=background-shell-full-command-selection --include=worker,lifecycle,transport,subscriptions --output=/tmp/diagnostics/background-worker.log'
const ARGS = { command: COMMAND, description: 'Start the diagnostic worker', run_in_background: true }
@@ -76,7 +76,7 @@ it('expands a background Bash launch to selectable arguments and its original ac
expect(await row.getAttribute('aria-expanded')).toBe('false')
await row.click()
const card = row.locator('..')
const input = card.locator('[class*="commandInput"]')
const input = card.locator('[data-command-text]')
await input.waitFor()
expect(await input.textContent()).toBe(JSON.stringify(ARGS, null, 2))
expect(await card.getByText(OUTPUT, { exact: true }).count()).toBe(1)
@@ -88,7 +88,8 @@ it('expands a background Bash launch to selectable arguments and its original ac
for (const width of ['', '380px']) {
await card.evaluate((element, value) => { element.style.width = value }, width)
await input.scrollIntoViewIfNeeded()
await expectSelectableCommandText(page, input, COMMAND)
await expectSelectableCommandText(page, input, COMMAND, 'Command arguments')
await expectCommandFocusAfterResize(card, input)
}
await card.evaluate((element) => { element.style.width = '' })
}
+88 -25
View File
@@ -7,11 +7,12 @@ import { expect } from 'vitest'
* @param card - Terminal card rendered by the application.
* @param expected - Complete single-line command.
* @param width - Narrow card width that leaves part of the command offscreen.
* @param label - Localized accessible name for the command line.
* @returns Resolves when the tail is reachable and native copying retains the full command.
*/
export async function expectSelectableTerminalCommand(page: Page, card: Locator, expected: string, width: number): Promise<void> {
export async function expectSelectableTerminalCommand(page: Page, card: Locator, expected: string, width: number, label = 'Command line 1'): Promise<void> {
const originalWidth = await card.evaluate(element => element.style.width)
const command = card.locator('[class*="_command"]').first()
const command = card.locator('[data-command-text]').first()
try {
for (const colorScheme of ['light', 'dark'] as const) {
await page.emulateMedia({ colorScheme })
@@ -31,11 +32,30 @@ export async function expectSelectableTerminalCommand(page: Page, card: Locator,
const { x, y, width: w, height } = element.getBoundingClientRect()
return { x, y, width: w, height }
}))
await expectSelectableCommandText(page, command, expected)
await expectSelectableCommandText(page, command, expected, label)
expect(await controls.evaluateAll(elements => elements.map((element) => {
const { x, y, width: w, height } = element.getBoundingClientRect()
return { x, y, width: w, height }
}))).toEqual(before)
const originalWhitespace = await card.evaluate((element) => {
const property = '--dsl-terminal-command-whitespace'
const previous = { value: element.style.getPropertyValue(property), priority: element.style.getPropertyPriority(property) }
element.style.setProperty(property, 'pre-wrap')
return previous
})
try {
await expectNoCommandTabStop(command)
expect(await command.evaluate((element) => {
const range = document.createRange()
range.selectNodeContents(element)
return range.getClientRects().length
})).toBeGreaterThan(1)
} finally {
await card.evaluate((element, previous) => {
element.style.setProperty('--dsl-terminal-command-whitespace', previous.value, previous.priority)
}, originalWhitespace)
}
await expectCommandFocusAfterResize(card, command)
await card.evaluate((element, value) => { element.style.width = value }, originalWidth)
}
} finally {
@@ -44,14 +64,42 @@ export async function expectSelectableTerminalCommand(page: Page, card: Locator,
}
}
/**
* Verify a resized command drops its explicit keyboard stop when all text fits.
* @param container - Width owner of the command.
* @param command - Mounted command text.
* @returns Resolves after widening removes the overflow-only accessibility attributes.
*/
export async function expectCommandFocusAfterResize(container: Locator, command: Locator): Promise<void> {
const originalWidth = await container.evaluate(element => element.style.width)
const extra = await command.evaluate(element => element.scrollWidth - element.clientWidth)
const width = await container.evaluate(element => element.getBoundingClientRect().width)
try {
await container.evaluate((element, value) => { element.style.width = `${value}px` }, width + Math.max(extra, 0) + 20)
await expectNoCommandTabStop(command)
} finally {
await container.evaluate((element, value) => { element.style.width = value }, originalWidth)
}
}
async function expectNoCommandTabStop(command: Locator): Promise<void> {
await expect.poll(() => command.evaluate(element => ({
overflow: element.scrollWidth > element.clientWidth + 1,
tabIndex: element.getAttribute('tabindex'),
role: element.getAttribute('role'),
label: element.getAttribute('aria-label'),
}))).toEqual({ overflow: false, tabIndex: null, role: null, label: null })
}
/**
* Verify native command selection within a text scrollport.
* @param page - Browser page that owns the text.
* @param command - Text scrollport containing the command, optionally within argument JSON.
* @param expected - Complete command, including its offscreen suffix.
* @param label - Localized accessible name for the overflowing text.
* @returns Resolves after horizontal navigation and native selection/copy succeed.
*/
export async function expectSelectableCommandText(page: Page, command: Locator, expected: string): Promise<void> {
export async function expectSelectableCommandText(page: Page, command: Locator, expected: string, label: string): Promise<void> {
const geometry = async () => command.evaluate((element, value) => {
const text = element.firstChild
if (!(text instanceof Text) || text.length === 0) throw new Error('Command has no text node')
@@ -81,30 +129,44 @@ export async function expectSelectableCommandText(page: Page, command: Locator,
expect(start.scrollWidth).toBeGreaterThan(start.clientWidth)
expect(start.tail.y).toBeCloseTo(start.head.y, 1)
expect(start.tail.x).toBeGreaterThan(start.right)
await command.hover()
await page.mouse.wheel(start.scrollWidth, 0)
await expect.poll(async () => {
const end = await geometry()
return end.tail.x <= end.right + 1 && end.tail.x >= end.left && end.scrollLeft > 0
}).toBe(true)
await page.mouse.wheel(-start.scrollWidth, 0)
await expect.poll(() => command.evaluate(element => element.scrollLeft)).toBe(0)
const head = await geometry()
await page.mouse.move(head.head.x + 0.1, head.head.y + head.head.height / 2)
await page.mouse.down()
try {
await page.mouse.move(head.right - 1, head.head.y + head.head.height / 2, { steps: 5 })
await expect.poll(() => command.evaluate(element => ({
tabIndex: element.getAttribute('tabindex'),
role: element.getAttribute('role'),
label: element.getAttribute('aria-label'),
}))).toEqual({ tabIndex: '0', role: 'group', label })
await command.focus()
await page.keyboard.press('Shift+Tab')
expect(await command.evaluate(element => document.activeElement === element)).toBe(false)
await page.keyboard.press('Tab')
expect(await command.evaluate(element => document.activeElement === element)).toBe(true)
await page.keyboard.press('ArrowRight')
await expect.poll(() => command.evaluate(element => element.scrollLeft)).toBeGreaterThan(0)
await page.keyboard.press('ArrowLeft')
await expect.poll(() => command.evaluate(element => element.scrollLeft)).toBe(0)
await command.hover()
await page.mouse.wheel(start.scrollWidth, 0)
await expect.poll(async () => {
const value = await geometry()
return value.tail.x <= head.right + 1 ? 'visible' : JSON.stringify({ before: head, after: value })
}).toBe('visible')
const end = await geometry()
await page.mouse.move(end.tail.x - 0.1, head.head.y + head.head.height / 2)
} finally {
await page.mouse.up()
}
try {
const end = await geometry()
return end.tail.x <= end.right + 1 && end.tail.x >= end.left && end.scrollLeft > 0
}).toBe(true)
await page.mouse.wheel(-start.scrollWidth, 0)
await expect.poll(() => command.evaluate(element => element.scrollLeft)).toBe(0)
const head = await geometry()
await page.mouse.move(head.head.x + 0.1, head.head.y + head.head.height / 2)
await page.mouse.down()
try {
await page.mouse.move(head.right - 1, head.head.y + head.head.height / 2, { steps: 5 })
await page.mouse.wheel(start.scrollWidth, 0)
await expect.poll(async () => {
const value = await geometry()
return value.tail.x <= head.right + 1 ? 'visible' : JSON.stringify({ before: head, after: value })
}).toBe('visible')
const end = await geometry()
await page.mouse.move(end.tail.x - 0.1, head.head.y + head.head.height / 2)
} finally {
await page.mouse.up()
}
await expect.poll(() => page.evaluate(() => document.getSelection()?.toString())).toBe(expected)
await page.context().grantPermissions(['clipboard-read', 'clipboard-write'], { origin: new URL(page.url()).origin })
await page.keyboard.press('ControlOrMeta+C')
@@ -112,6 +174,7 @@ export async function expectSelectableCommandText(page: Page, command: Locator,
} finally {
await command.evaluate((element) => {
element.scrollLeft = 0
element.blur()
document.getSelection()?.removeAllRanges()
})
}
@@ -0,0 +1,16 @@
# Bilingual-pair consistency record for guide.md (docs/i18n/README.md): per heading
# section, a hash of its English and Chinese blocks outside code blocks and generated regions.
# After editing either side, bring the other along and re-record with:
# pnpm run verify-translation-pairing --write docs/upgrade-guide/v0.2.1-alpha.1/terminal-command-labels/guide.md
/:
en: 0722f826c1c12360
zh: 9b48ce197b6091f2
/terminal-command-labels:
en: a75e371670f4ebc3
zh: cbad7dd538376e70
/terminal-command-labels/change:
en: 0e386cd5b5d349dc
zh: c456641a3aa938b3
/terminal-command-labels/migration:
en: 2044b801804a40aa
zh: 0f98e7da80e2f066
@@ -0,0 +1,18 @@
---
kind: upgrade-guide
description: "Custom TerminalBlock consumers must provide localized accessible names for overflowing command lines."
---
# Terminal command labels
English | [中文](guide.zh.md)
## Change
`TerminalBlockLabels` from `@deepseek-ai/dsh-client-ui-primitives` requires a `commandLine(line: number): string` formatter. Previously, callers supplied no accessible name for command lines. Each horizontally overflowing line now uses this name for its keyboard-focusable group; lines that fit, including wrapped lines, add no Tab stop. This affects custom UI plugins that render `TerminalBlock`.
## Migration
1. Add `commandLine` to each labels object passed to `TerminalBlock`. The argument is the authored line number, starting at one.
2. Return a localized name, such as “Command line 1”, through the plugin's locale dictionary. Keep the original command text separate from this name.
3. Rebuild the plugin. Verify that a long command can receive keyboard focus and scroll with the arrow keys, and that a short or wrapped command adds no Tab stop.
@@ -0,0 +1,18 @@
---
kind: upgrade-guide
description: "自定义 TerminalBlock 消费者必须为横向溢出的命令行提供本地化的无障碍名称。"
---
# 终端命令标签
[English](guide.md) | 中文
## 变更
`@deepseek-ai/dsh-client-ui-primitives` 的 `TerminalBlockLabels` 要求提供 `commandLine(line: number): string` 格式化函数。此前,调用者不提供命令行的无障碍名称。现在,每条横向溢出的命令行使用这个名称标识可通过键盘聚焦的分组;能够完整显示的行(包括折行后的行)不会增加 Tab 停靠点。这会影响渲染 `TerminalBlock` 的自定义 UI 插件。
## 迁移
1. 为传给 `TerminalBlock` 的每个 labels 对象添加 `commandLine`。参数是原始命令文本的行号,从一开始。
2. 通过插件的语言字典返回本地化名称,例如“命令第 1 行”。名称与原始命令文本分开保留。
3. 重新构建插件。确认长命令可通过键盘聚焦并使用方向键滚动,短命令或折行后的命令则不会增加 Tab 停靠点。
@@ -308,6 +308,7 @@ export const zh = {
'row.preparing': '正在准备调用',
'row.failed': '失败',
'row.stopped': '已停止',
'row.commandArguments': '命令参数',
'row.input': '输入',
'row.output': '输出',
'row.inspect': '查看',
@@ -369,6 +370,7 @@ export const zh = {
'terminal.signal': '信号 {signal}',
'terminal.exitCode': '退出码 {code}',
'terminal.noExitCode': '未正常退出',
'terminal.commandLine': '命令第 {n} 行',
'terminal.running': '运行中',
'terminal.failed': '失败',
'terminal.done': '已完成',
@@ -684,6 +686,7 @@ export const en = {
'row.preparing': 'Preparing tool call',
'row.failed': 'Failed',
'row.stopped': 'Stopped',
'row.commandArguments': 'Command arguments',
'row.input': 'IN',
'row.output': 'OUT',
'row.inspect': 'Inspect',
@@ -745,6 +748,7 @@ export const en = {
'terminal.signal': 'signal {signal}',
'terminal.exitCode': 'exit code {code}',
'terminal.noExitCode': 'no exit code',
'terminal.commandLine': 'Command line {n}',
'terminal.running': 'Running',
'terminal.failed': 'Failed',
'terminal.done': 'Done',
@@ -126,6 +126,7 @@ function formatDuration(elapsedMs: number, t: TranslateNS<typeof NS>): string {
/** Localized display copy for the embedded terminal panel. */
function terminalLabels(t: TranslateNS<typeof NS>): TerminalBlockLabels {
return {
commandLine: line => t('terminal.commandLine', { n: line }),
// The labels contract requires exit-fact formatters, but this panel never
// passes exit facts, so TerminalBlock never invokes them.
/* v8 ignore next */
@@ -34,6 +34,7 @@ export const zh = {
'terminal.signal': '信号 {signal}',
'terminal.exitCode': '退出码 {code}',
'terminal.noExitCode': '未正常退出',
'terminal.commandLine': '命令第 {n} 行',
'terminal.running': '运行中',
'terminal.failed': '已失败',
'terminal.done': '已完成',
@@ -77,6 +78,7 @@ export const en: Record<JobKey, string> = {
'terminal.signal': 'signal {signal}',
'terminal.exitCode': 'exit {code}',
'terminal.noExitCode': 'no exit code',
'terminal.commandLine': 'Command line {n}',
'terminal.running': 'running',
'terminal.failed': 'failed',
'terminal.done': 'done',
@@ -732,6 +732,7 @@ const INSTALL_TERMINAL_LINES = 12
/** The install terminal's display copy, from the tab's dictionary. */
function terminalLabels(t: Translate): TerminalBlockLabels {
return {
commandLine: line => t('terminalCommandLine', { n: String(line) }),
/* v8 ignore next -- the Host reports a killed pnpm as a null exit code, never a signal name; the label interface needs one */
signal: signal => t('terminalSignal', { signal }),
exitCode: code => t('terminalExitCode', { code: String(code) }),
@@ -165,6 +165,7 @@ export const zh = {
installFailureTimeout: '安装超时',
installFailurePnpmMissing: '没有找到 pnpm,无法安装',
installFailureGeneric: '安装过程中出错,原因见安装详情',
terminalCommandLine: '命令第 {n} 行',
terminalRunning: '运行中',
terminalFailed: '失败',
terminalDone: '已完成',
@@ -379,6 +380,7 @@ export const en = {
installFailureTimeout: 'The install timed out',
installFailurePnpmMissing: 'pnpm was not found, so nothing can be installed',
installFailureGeneric: 'Something went wrong during the install; the details say what',
terminalCommandLine: 'Command line {n}',
terminalRunning: 'Running',
terminalFailed: 'Failed',
terminalDone: 'Done',
@@ -18,8 +18,8 @@
en: 7bae72c97d0449f5
zh: 4589ecc2b216edc5
/deepseek-ai-dsh-client-ui-primitives/use-this-package/component-catalog:
en: f093268cd20a1c14
zh: 861c1fae527264fb
en: 66d83132d97e56c6
zh: fb47139ce1f742fe
/deepseek-ai-dsh-client-ui-primitives/use-this-package/controls-and-icons:
en: f0e54915b896af5a
zh: 2047520fd321be90
@@ -42,8 +42,8 @@
en: 8bbfa14c27f3bfd7
zh: 88f7548ac3a114e1
/deepseek-ai-dsh-client-ui-primitives/understand-the-implementation/geometry-and-overflow:
en: c5981989617c687b
zh: ec15382e560a43a3
en: e283bf2366b43879
zh: d089076d7834e853
/deepseek-ai-dsh-client-ui-primitives/further-exploration:
en: e82b713cdb2f954b
zh: a877f34b148f433f
+2 -1
View File
@@ -66,6 +66,7 @@ Check this table before writing a control in a feature package. A plugin cannot
| `SettingsFormModel`, `settingsNumberField`, `settingsTextField` | The staged-edit model behind such a page over a settings scope: drafts are staged and written on save, a field is overridden by its presence in the user layer, and a save that did not land keeps its drafts. |
| `JsonTree`, `JsonBlock` | Read-only JSON inspection. |
| `MarkdownText`, `MarkdownDelegateProvider`, `CodeBlock` | Untrusted GFM with TeX math, owner-delegated HTTP(S) navigation, and highlighted code. `CodeBlock` accepts opt-in `lineNumbers`; copied source excludes the gutter, and `contentRef` exposes its stable source wrapper to an owner that uses it as a scrollport. Set `showHeader={false}` when the owner supplies its own language and copy toolbar. |
| `CommandText` | Selectable command or argument text; only horizontal overflow adds a named keyboard focus target. |
| `TerminalBlock`, `ReadBlock`, `DiffBlock`, `SearchBlock`, `WebBlock` | The agent-output card matching each tool-result intent. |
| `icons/*`, `FishLogo`, `BrandWordmark`, `ReferenceIconRegular`/`ReferenceIconMedium`, `LinkIconRegular`/`LinkIconMedium` | Glyphs and brand marks. The thinking glyph keeps an inset orbit within its 16px view box. Use `LinkIconMedium` for 14px clickable-link categories and known-site marks. |
| `PermissionIconReadOnlyRegular`/`Medium`, `PermissionIconWorkspaceWriteRegular`/`Medium`, `PermissionIconFullAccessRegular`/`Medium` | Permission-mode glyphs for read-only, workspace-write, and full-access choices. |
@@ -160,7 +161,7 @@ While a reply streams, `MarkdownText` parses incrementally: all but the trailing
### Geometry and overflow
The output cards share one geometry model: `white-space: pre` with horizontal scrolling so column-aligned content keeps its alignment, and a head-plus-tail slice behind an expand button past `maxLines` (default 16) so a long body never stretches the card. `TerminalBlock` keeps each authored command line selectable and horizontally scrollable, with keyboard focus for scrolling; the prompt and status stay fixed. `TerminalBlock` parses ANSI into React spans with a per-line column buffer for cursor movement, honoring erase-in-line, tab stops, and character width. Hosts opt out of the shared geometry per surface: rebinding `--dsl-terminal-command-whitespace` / `--dsl-terminal-line-whitespace` to `pre-wrap` wraps commands and output in full with no sideways scroll, `maxLines: Infinity` disables the fold for hosts capping height through `--dsl-terminal-output-max-height` instead, `copyText` overrides the copy payload (and keeps the control rendered before any output), and `runStateDot: false` omits the run-state dot when the surrounding row already carries the state, reclaiming its gutter via `--dsl-terminal-gutter`. The banner divider follows the rendered body, so a running card that streams live output separates its command from the text like a settled card.
The output cards share one geometry model: `white-space: pre` with horizontal scrolling so column-aligned content keeps its alignment, and a head-plus-tail slice behind an expand button past `maxLines` (default 16) so a long body never stretches the card. `TerminalBlock` uses `CommandText` to keep each authored command line selectable and horizontally scrollable; the prompt and status stay fixed. Only horizontally overflowing text enters the Tab order as a named group, using the owner’s localized `commandLine(line)` label. Text that fits, including wrapped Jobs commands, adds no Tab stop. Size, text, and font changes update focusability without changing the scroll position. `TerminalBlock` parses ANSI into React spans with a per-line column buffer for cursor movement, honoring erase-in-line, tab stops, and character width. Hosts opt out of the shared geometry per surface: rebinding `--dsl-terminal-command-whitespace` / `--dsl-terminal-line-whitespace` to `pre-wrap` wraps commands and output in full with no sideways scroll, `maxLines: Infinity` disables the fold for hosts capping height through `--dsl-terminal-output-max-height` instead, `copyText` overrides the copy payload (and keeps the control rendered before any output), and `runStateDot: false` omits the run-state dot when the surrounding row already carries the state, reclaiming its gutter via `--dsl-terminal-gutter`. The banner divider follows the rendered body, so a running card that streams live output separates its command from the text like a settled card.
</details>
+2 -1
View File
@@ -66,6 +66,7 @@ kind: "package-library"
| `SettingsFormModel`、`settingsNumberField`、`settingsTextField` | 这类页面背后基于设置 scope 的暂存编辑模型:草稿先暂存、保存时写入,字段是否被覆盖看用户层是否含有它,未落地的保存保留草稿。 |
| `JsonTree`、`JsonBlock` | 只读 JSON 查看。 |
| `MarkdownText`、`MarkdownDelegateProvider`、`CodeBlock` | 不可信 GFM 与 TeX 数学、owner 委托的 HTTP(S) 导航,以及高亮代码。`CodeBlock` 可通过 `lineNumbers` 开启行号;复制的源码不含行号栏,`contentRef` 则向需要把稳定源码包装节点用作滚动区的 owner 提供该节点。调用方提供自己的语言与复制工具栏时,设置 `showHeader={false}`。 |
| `CommandText` | 可选中的命令或参数文本;仅横向溢出时添加具名的键盘聚焦目标。 |
| `TerminalBlock`、`ReadBlock`、`DiffBlock`、`SearchBlock`、`WebBlock` | 与各类工具结果意图对应的 agent 输出卡片。 |
| `icons/*`、`FishLogo`、`BrandWordmark`、`ReferenceIconRegular`/`ReferenceIconMedium`、`LinkIconRegular`/`LinkIconMedium` | 字形与品牌标识。思考图标的轨道在 16px 视口内保留内边距。`LinkIconMedium` 用于 14px 的可点击链接分类及已知站点标记。 |
| `PermissionIconReadOnlyRegular`/`Medium`、`PermissionIconWorkspaceWriteRegular`/`Medium`、`PermissionIconFullAccessRegular`/`Medium` | 只读、工作区写入与完全访问选项使用的权限模式图形。 |
@@ -160,7 +161,7 @@ kind: "package-library"
### 几何与溢出
输出卡片共享同一套几何模型:`white-space: pre` 并横向滚动,让按列对齐的内容保持对齐;超过 `maxLines`(默认 16)时折叠为头部切片加尾部切片,由展开按钮控制,长正文不会撑高卡片。`TerminalBlock` 的每条原始命令行都可选择并横向滚动,也可通过键盘聚焦后滚动;提示符和状态保持固定。`TerminalBlock` 把 ANSI 解析为 React span,并带逐行列缓冲处理光标移动,遵循行内擦除、制表位与字符宽度。宿主可按表层选择退出共享几何:把 `--dsl-terminal-command-whitespace` / `--dsl-terminal-line-whitespace` 重绑为 `pre-wrap` 让命令与输出完整换行且不横向滚动;`maxLines: Infinity` 为改用 `--dsl-terminal-output-max-height` 限高滚动的宿主禁用折叠;`copyText` 覆盖复制载荷(并让控件在任何输出出现之前就保持渲染);`runStateDot: false` 在外围行已携带同一状态时省去状态点,并经 `--dsl-terminal-gutter` 收回其落区。横幅分割线跟随渲染出的正文:正在流式输出的 running 卡片像已结束卡片一样把命令与文本分隔开。
输出卡片共享同一套几何模型:`white-space: pre` 并横向滚动,让按列对齐的内容保持对齐;超过 `maxLines`(默认 16)时折叠为头部切片加尾部切片,由展开按钮控制,长正文不会撑高卡片。`TerminalBlock` 通过 `CommandText` 让每条原始命令行可选中并横向滚动;提示符和状态保持固定。只有横向溢出的文本以具名 group 进入 Tab 顺序,名称使用 owner 的本地化 `commandLine(line)` 标签。能完整显示的文本(包括 Jobs 中已换行的命令)不会添加 Tab 停靠点。尺寸、文本与字体变化会更新可聚焦状态,不改变滚动位置。`TerminalBlock` 把 ANSI 解析为 React span,并带逐行列缓冲处理光标移动,遵循行内擦除、制表位与字符宽度。宿主可按表层选择退出共享几何:把 `--dsl-terminal-command-whitespace` / `--dsl-terminal-line-whitespace` 重绑为 `pre-wrap` 让命令与输出完整换行且不横向滚动;`maxLines: Infinity` 为改用 `--dsl-terminal-output-max-height` 限高滚动的宿主禁用折叠;`copyText` 覆盖复制载荷(并让控件在任何输出出现之前就保持渲染);`runStateDot: false` 在外围行已携带同一状态时省去状态点,并经 `--dsl-terminal-gutter` 收回其落区。横幅分割线跟随渲染出的正文:正在流式输出的 running 卡片像已结束卡片一样把命令与文本分隔开。
</details>
@@ -0,0 +1,12 @@
.text {
display: block;
min-width: 0;
overflow-x: auto;
white-space: var(--dsl-command-whitespace, pre);
overflow-wrap: anywhere;
}
.text:focus-visible {
outline: var(--dsw-focus-ring-width) solid var(--dsw-focus-ring-color, var(--dsw-alias-state-business-primary));
outline-offset: -2px;
}
@@ -0,0 +1,46 @@
/** Selectable command text with keyboard access while horizontally overflowing. */
import { useLayoutEffect, useRef, useState } from 'react'
import clsx from 'clsx'
import css from './CommandText.module.css'
/** Display text and owner-localized identification for a command scrollport. */
export interface CommandTextProps {
/** Original command or formatted command arguments, without truncation. */
text: string
/** Accessible name while the text can scroll horizontally. */
label: string
/** Owner typography and positioning. */
className?: string | undefined
}
/**
* Render command text without adding Tab stops for content that fits.
* @param props - Text, localized scrollport name, and owner styling.
* @returns A horizontal text scrollport, named and keyboard-focusable only when overflowing.
*/
export function CommandText({ text, label, className }: CommandTextProps) {
const ref = useRef<HTMLSpanElement>(null)
const [overflow, setOverflow] = useState(false)
useLayoutEffect(() => {
const element = ref.current
/* v8 ignore next -- the span is rendered unconditionally. */
if (element === null) return undefined
const measure = () => { setOverflow(element.scrollWidth > element.clientWidth) }
measure()
const observer = typeof ResizeObserver === 'undefined' ? undefined : new ResizeObserver(measure)
observer?.observe(element)
const fonts = 'fonts' in element.ownerDocument ? element.ownerDocument.fonts : undefined
fonts?.addEventListener('loadingdone', measure)
return () => {
observer?.disconnect()
fonts?.removeEventListener('loadingdone', measure)
}
}, [text])
return (
<span ref={ref} className={clsx(css.text, className)} data-command-text
tabIndex={overflow ? 0 : undefined}
role={overflow ? 'group' : undefined}
aria-label={overflow ? label : undefined}
>{text}</span>
)
}
@@ -113,18 +113,8 @@
/* Each authored command line scrolls independently of its prompt and status.
`pre` preserves whitespace; hosts can opt into wrapping with `pre-wrap`. */
.command {
min-width: 0;
--dsl-command-whitespace: var(--dsl-terminal-command-whitespace, pre);
color: var(--dsw-alias-label-primary);
overflow-x: auto;
white-space: var(--dsl-terminal-command-whitespace, pre);
overflow-wrap: anywhere;
user-select: text;
cursor: text;
}
.command:focus-visible {
outline: var(--dsw-focus-ring-width) solid var(--dsw-focus-ring-color, var(--dsw-alias-state-business-primary));
outline-offset: -2px;
}
/* Capped to the prompt's line height (Pill's own 24px height would exceed a
@@ -6,6 +6,7 @@ import { useCopyFeedback } from './use-copy-feedback.ts'
import { Pill } from './Pill.tsx'
import { StateDot, type StateDotState } from './StateDot.tsx'
import css from './TerminalBlock.module.css'
import { CommandText } from './CommandText.tsx'
/** Output lines shown before the height cap collapses the middle. */
export const DEFAULT_TERMINAL_MAX_LINES = 16
@@ -15,6 +16,8 @@ export const DEFAULT_TERMINAL_MAX_LINES = 16
* (this package is cordis-free, so copy arrives via props).
*/
export interface TerminalBlockLabels {
/** Accessible name for an overflowing authored command line, numbered from one. */
commandLine: (line: number) => string
/** Status pill text for a signal-terminated command. */
signal: (signal: string) => string
/** Status pill text for a non-zero exit code. */
@@ -244,7 +247,7 @@ export function TerminalBlock({
<span className={css.cwd}>
{index > 0 || cwd === undefined ? '$' : promptLabel(cwd, home)}
</span>
<span className={css.command} tabIndex={0}>{line}</span>
<CommandText className={css.command} text={line} label={copy.commandLine(index + 1)} />
</div>
))}
</div>
@@ -83,6 +83,8 @@ export type { RelativeTime, RelativeTimeUnit } from './relative-time.ts'
export { JsonTree } from './JsonTree.tsx'
export type { JsonTreeProps, JsonTreeLabels } from './JsonTree.tsx'
export { TerminalBlock, DEFAULT_TERMINAL_MAX_LINES } from './TerminalBlock.tsx'
export { CommandText } from './CommandText.tsx'
export type { CommandTextProps } from './CommandText.tsx'
export type { TerminalBlockProps, TerminalBlockLabels } from './TerminalBlock.tsx'
export { ReadBlock, DEFAULT_READ_MAX_LINES } from './ReadBlock.tsx'
export type { ReadBlockProps, ReadBlockLine, ReadBlockLabels } from './ReadBlock.tsx'
@@ -0,0 +1,80 @@
// @vitest-environment jsdom
import { act, cleanup, render } from '@testing-library/react'
import { afterEach, beforeEach, expect, it, vi } from 'vitest'
import { CommandText } from '../src/CommandText.tsx'
import { TerminalBlock } from '../src/TerminalBlock.tsx'
import { terminalBlockLabels } from './labels.client.ts'
const originalFonts = Object.getOwnPropertyDescriptor(document, 'fonts')
beforeEach(() => {
vi.stubGlobal('ResizeObserver', undefined)
vi.spyOn(HTMLElement.prototype, 'clientWidth', 'get').mockReturnValue(100)
vi.spyOn(HTMLElement.prototype, 'scrollWidth', 'get').mockImplementation(function (this: HTMLElement) {
return (this.textContent?.length ?? 0) * 10
})
})
afterEach(() => {
cleanup()
vi.unstubAllGlobals()
vi.restoreAllMocks()
if (originalFonts === undefined) Reflect.deleteProperty(document, 'fonts')
else Object.defineProperty(document, 'fonts', originalFonts)
})
it('names and focuses overflowing text as it changes, without changing its scroll position', () => {
const view = render(<CommandText text="echo ok" label="Command line 1" />)
const command = view.getByText('echo ok')
expect(command.hasAttribute('tabindex')).toBe(false)
expect(command.hasAttribute('role')).toBe(false)
expect(command.hasAttribute('aria-label')).toBe(false)
view.rerender(<CommandText text="echo a-long-command-argument" label="Command line 1" />)
expect(view.getByRole('group', { name: 'Command line 1' })).toBe(command)
expect(command.tabIndex).toBe(0)
command.scrollLeft = 40
view.rerender(<CommandText text="echo another-long-command-argument" label="Command line 1" />)
expect(command.scrollLeft).toBe(40)
view.rerender(<CommandText text="echo ok" label="Command line 1" />)
expect(command.hasAttribute('tabindex')).toBe(false)
expect(command.hasAttribute('aria-label')).toBe(false)
})
it('updates focusability after resizing and font loading, and releases both observers', () => {
let resize: (() => void) | undefined
const observe = vi.fn()
const disconnect = vi.fn()
vi.stubGlobal('ResizeObserver', class {
constructor(callback: () => void) { resize = callback }
observe = observe
disconnect = disconnect
})
const fonts = new EventTarget()
Object.defineProperty(document, 'fonts', { configurable: true, value: fonts })
const removeListener = vi.spyOn(fonts, 'removeEventListener')
const width = vi.spyOn(HTMLElement.prototype, 'clientWidth', 'get').mockReturnValue(300)
const view = render(<CommandText text="echo wider command" label="Command line 1" />)
const command = view.getByText('echo wider command')
expect(observe).toHaveBeenCalledWith(command)
expect(command.hasAttribute('tabindex')).toBe(false)
if (resize === undefined) throw new Error('Missing resize observer')
width.mockReturnValue(100)
act(resize)
expect(command.tabIndex).toBe(0)
width.mockReturnValue(300)
act(() => { fonts.dispatchEvent(new Event('loadingdone')) })
expect(command.hasAttribute('tabindex')).toBe(false)
view.unmount()
expect(disconnect).toHaveBeenCalledOnce()
expect(removeListener).toHaveBeenCalledWith('loadingdone', expect.any(Function))
width.mockClear()
fonts.dispatchEvent(new Event('loadingdone'))
expect(width).not.toHaveBeenCalled()
})
it('adds one named Tab stop for the overflowing line of a multiline command', () => {
const view = render(<TerminalBlock command={'echo ok\necho a-long-command-argument\necho done'} labels={terminalBlockLabels} />)
const commands = [...view.container.querySelectorAll('[data-command-text]')]
expect(commands.map(command => command.hasAttribute('tabindex'))).toEqual([false, true, false])
expect(view.getByRole('group', { name: '命令第 2 行' })).toBe(commands[1])
})
@@ -42,6 +42,7 @@ export const searchBlockLabels: SearchBlockLabels = {
}
export const terminalBlockLabels: TerminalBlockLabels = {
commandLine: line => `命令第 ${line} 行`,
signal: signal => `信号 ${signal}`,
exitCode: code => `退出码 ${code}`,
noExitCode: '未正常退出',
+2 -2
View File
@@ -21,8 +21,8 @@
en: 2e3c306b13d3bc25
zh: 77127f01763b3976
/deepseek-ai-dsh-client-ui-tool/use-this-package/built-in-views:
en: 0a8434ddc1606496
zh: 598a6dd0852f505b
en: 4d885fc693b497f4
zh: 2f68d9ea92f2ebd6
/deepseek-ai-dsh-client-ui-tool/understand-the-implementation:
en: 00bc7b0c84edc603
zh: 1cfcf5b435248a91
+1 -1
View File
@@ -51,7 +51,7 @@ Every registered view receives the explicit `preparing`, `start`, and `result` p
File-tool rows prefer a complete `file_path`; other generic rows, including search and unregistered tools, prefer a non-empty `description` before their variant-specific argument fallback. Read `args` during rendering or view construction: a preparing reader grows in place, and only previously observed answer changes trigger subsequent block publication.
This package owns the generic fallback and the built-in shell/pwsh, read, read_image, write/edit, running `str_replace_editor` `create`/`str_replace`, grep/glob, web, todo, question, and PTC dispatch presentations. Structured cards derive directly from first-party raw event fields; Host `presentCall` and `presentResult` values never enter the Client. Running atomic rows share one left-to-right highlight across their titles, separators, summaries, and suffixes, while settled rows stay static. Running and settled foreground standard `bash`/`pwsh` and `terminal_send` calls use terminal cards at the root and in PTC dispatch children, subject to the same argument, result, and error checks. Persistent `bash`/`pwsh` calls use terminal cards only while running. Shell output ending in a recognized spill-policy notice uses expandable generic output in shell rows and generic output in Details; a displaced or omitted exit marker cannot establish success. Settled persistent-shell results stay generic because reset and partial-output diagnostics do not always describe one process exit status; root persistent results are expandable. Background Bash calls expand to their original arguments and job acknowledgement. Their input scrolls horizontally without soft wrapping and remains selectable; Inspect is available when provided. The acknowledgement does not report the job's exit status. A native or PTC dispatch failure carrying `AUTO_REVIEW_DENIED` shows the Auto review verdict in its collapsed row and one normalized not-executed reason when expanded; a missing or whitespace-only reason uses localized fallback text. `ui-skill` demonstrates a business-owned registration for `skill`.
This package owns the generic fallback and the built-in shell/pwsh, read, read_image, write/edit, running `str_replace_editor` `create`/`str_replace`, grep/glob, web, todo, question, and PTC dispatch presentations. Structured cards derive directly from first-party raw event fields; Host `presentCall` and `presentResult` values never enter the Client. Running atomic rows share one left-to-right highlight across their titles, separators, summaries, and suffixes, while settled rows stay static. Running and settled foreground standard `bash`/`pwsh` and `terminal_send` calls use terminal cards at the root and in PTC dispatch children, subject to the same argument, result, and error checks. Persistent `bash`/`pwsh` calls use terminal cards only while running. Shell output ending in a recognized spill-policy notice uses expandable generic output in shell rows and generic output in Details; a displaced or omitted exit marker cannot establish success. Settled persistent-shell results stay generic because reset and partial-output diagnostics do not always describe one process exit status; root persistent results are expandable. Background Bash calls expand to their original arguments and job acknowledgement. Their input scrolls horizontally without soft wrapping and remains selectable; only horizontal overflow adds a named keyboard focus target. Inspect is available when provided. The acknowledgement does not report the job's exit status. A native or PTC dispatch failure carrying `AUTO_REVIEW_DENIED` shows the Auto review verdict in its collapsed row and one normalized not-executed reason when expanded; a missing or whitespace-only reason uses localized fallback text. `ui-skill` demonstrates a business-owned registration for `skill`.
Question rows pair answers with questions by id, using the tool result or the `userQuestions` projection for late replies. An answerable call has a button to reopen its panel. Once a late reply is in the agent Inbox, the row shows its submitted answers; “View answers” expands its read-only transcript while admission is pending. A settled call with answers opens the panel read-only. Closed calls without answers and invalid records offer no button. Cancelled and interrupted rows show their verdict without inventing answers. If no panel provider accepts the button action, it expands the row. Unsupported or ambiguous input falls back to flattened Tool text.
+1 -1
View File
@@ -51,7 +51,7 @@ owner 载荷为 `ToolCallOwnerProps`:`callId`、`toolName`、`phase` 判别字
文件工具行优先使用完整的 `file_path`;其他通用行(包括搜索和未注册的工具)优先使用非空 `description`,再回退到各自类别的参数摘要。应在渲染或构造视图时读取 `args`:准备态读器原地增长,后续仅在已观察答案变化时重新发布块。
本包拥有 generic fallback,以及 shell/pwsh、read、read_image、write/edit、运行中的 `str_replace_editor` `create`/`str_replace`、grep/glob、web、todo、question 与 PTC dispatch 的内置展示。结构化卡片直接从第一方原始 event 字段派生;Host `presentCall` 与 `presentResult` 值不会进入 Client。运行中的原子工具行以同一道高光从左到右扫过标题、分隔符、摘要与后缀;已完成行保持静态。运行中与已完成的前台标准 `bash`/`pwsh` 和 `terminal_send` 调用,无论位于根还是 PTC dispatch 子调用中,都在通过相同的参数、结果和错误检查后使用 terminal 卡片。持久 `bash`/`pwsh` 调用仅在运行中使用 terminal 卡片。以已识别的 spill 策略提示结尾的 shell 输出,在 shell 行中使用可展开的 generic 输出,在 Details 中使用 generic 输出;位置被改变或被省略的退出标记无法证明成功。已完成的持久 shell 结果保持 generic 展示,因为 reset 与部分输出诊断不一定描述单个进程的退出状态;根调用的持久 shell 结果可展开。后台 Bash 调用可展开查看原始参数与后台任务回执。输入内容可横向滚动,不自动折行,并可选中;提供 Inspect 时也可使用。回执不表示后台任务的退出状态。带有 `AUTO_REVIEW_DENIED` 的原生或 PTC dispatch 失败会在折叠行显示 Auto review 裁决,展开时显示一行归一化后的“未执行”原因;原因缺失或只有空白时使用本地化 fallback 文案。`ui-skill` 展示了业务包自行拥有的 `skill` 注册项。
本包拥有 generic fallback,以及 shell/pwsh、read、read_image、write/edit、运行中的 `str_replace_editor` `create`/`str_replace`、grep/glob、web、todo、question 与 PTC dispatch 的内置展示。结构化卡片直接从第一方原始 event 字段派生;Host `presentCall` 与 `presentResult` 值不会进入 Client。运行中的原子工具行以同一道高光从左到右扫过标题、分隔符、摘要与后缀;已完成行保持静态。运行中与已完成的前台标准 `bash`/`pwsh` 和 `terminal_send` 调用,无论位于根还是 PTC dispatch 子调用中,都在通过相同的参数、结果和错误检查后使用 terminal 卡片。持久 `bash`/`pwsh` 调用仅在运行中使用 terminal 卡片。以已识别的 spill 策略提示结尾的 shell 输出,在 shell 行中使用可展开的 generic 输出,在 Details 中使用 generic 输出;位置被改变或被省略的退出标记无法证明成功。已完成的持久 shell 结果保持 generic 展示,因为 reset 与部分输出诊断不一定描述单个进程的退出状态;根调用的持久 shell 结果可展开。后台 Bash 调用可展开查看原始参数与后台任务回执。输入内容可横向滚动,不自动折行,并可选中;仅横向溢出时添加具名的键盘聚焦目标。提供 Inspect 时也可使用。回执不表示后台任务的退出状态。带有 `AUTO_REVIEW_DENIED` 的原生或 PTC dispatch 失败会在折叠行显示 Auto review 裁决,展开时显示一行归一化后的“未执行”原因;原因缺失或只有空白时使用本地化 fallback 文案。`ui-skill` 展示了业务包自行拥有的 `skill` 注册项。
问题行按 id 配对问题与答案;答案来自工具结果,迟到回答则来自 `userQuestions` 投影。可回答的调用提供按钮以重新打开面板。迟到回复进入 agent Inbox 后,问题行显示已提交的答案;等待准入期间,「查看回答」展开只读的行内记录。带有答案的已结算调用则以只读方式打开面板。没有答案的已关闭调用和无效记录不提供按钮。已取消和已中断的行显示裁决,不虚构答案。若面板提供方不接受按钮操作,点击会展开该行。不受支持或含糊的输入回退为压平的工具文本。
@@ -17,6 +17,7 @@ import { recordedAbsolutePath } from './recorded-path.ts'
*/
export function terminalBlockLabels(t: TranslateNS<'conversation'>): TerminalBlockLabels {
return {
commandLine: line => t('terminal.commandLine', { n: line }),
signal: signal => t('terminal.signal', { signal }),
exitCode: code => t('terminal.exitCode', { code }),
noExitCode: t('terminal.noExitCode'),
@@ -78,15 +78,7 @@
}
.commandInput {
white-space: pre;
overflow-x: auto;
user-select: text;
cursor: text;
}
.commandInput:focus-visible {
outline: var(--dsw-focus-ring-width) solid var(--dsw-focus-ring-color, var(--dsw-alias-state-business-primary));
outline-offset: -2px;
color: var(--dsw-alias-label-secondary);
}
/* ToolRow's unified expand interaction, replicated per the registrant
@@ -3,7 +3,7 @@ import type { Context } from '@deepseek-ai/cordis'
import clsx from 'clsx'
import {
IconApiOutlineRegular, IconChevronDownOutlineRegular, IconChevronUpOutlineRegular, IconInspectOutlineRegular,
TerminalBlock, TextShimmer,
CommandText, TerminalBlock, TextShimmer,
} from '@deepseek-ai/dsh-client-ui-primitives'
import type { PropsLocale } from '@deepseek-ai/dsh-client-ui-slots'
import type { ToolCallViewProps } from '../../contract/slots.ts'
@@ -135,7 +135,9 @@ export const BashRow = memo(function BashRow({ toolName, block, sessionId, useSe
{body !== null && (
<div className={css.ioSection}>
<span className={css.ioLabel}>{t('row.input')}</span>
<span className={clsx(css.ioText, background && css.commandInput)} tabIndex={background ? 0 : undefined}>{body}</span>
{background
? <CommandText className={css.commandInput} text={body} label={t('row.commandArguments')} />
: <span className={css.ioText}>{body}</span>}
</div>
)}
{body !== null && model.output !== null && (
@@ -541,9 +541,9 @@ describe('BashRow terminal card', () => {
fireEvent.keyDown(row, { key: 'Enter' })
expect(row.getAttribute('aria-expanded')).toBe('true')
expect(view.container.querySelector('[class*="_ioText_"]')?.textContent)
expect(view.container.querySelector('[data-command-text]')?.textContent)
.toBe(JSON.stringify(args, null, 2))
expect(view.container.querySelector('[class*="_commandInput_"]')?.getAttribute('tabindex')).toBe('0')
expect(view.container.querySelector('[data-command-text]')?.hasAttribute('tabindex')).toBe(false)
expect(view.queryByText('输出')).toBeNull()
expect(view.container.querySelector('[data-terminal]')).toBeNull()
@@ -554,7 +554,7 @@ describe('BashRow terminal card', () => {
}))} inspect={inspect} />)
expect(row.getAttribute('aria-expanded')).toBe('true')
expect(view.container.querySelector('[class*="_ioText_"]')?.textContent)
expect(view.container.querySelector('[data-command-text]')?.textContent)
.toBe(JSON.stringify(args, null, 2))
expect(view.getByText(acknowledgement)).toBeTruthy()
expect(view.container.querySelector('[data-terminal], [data-state="done"]')).toBeNull()
@@ -566,7 +566,7 @@ describe('BashRow terminal card', () => {
expect(view.queryByText(acknowledgement)).toBeNull()
})
it('keeps a background result without a valid command collapsed', () => {
it('keeps malformed shell arguments collapsed even when background execution is requested', () => {
const view = render(<BashRow {...rowProps(settled({
call: { name: 'bash', argsRaw: JSON.stringify({ description: 'Wait', run_in_background: true }) },
content: [{ type: 'text', text: 'started background job job-1' }],