diff --git a/packages/client/ui-agent-preset/src/client/locales.ts b/packages/client/ui-agent-preset/src/client/locales.ts
index 613ff59b08..3d323636ea 100644
--- a/packages/client/ui-agent-preset/src/client/locales.ts
+++ b/packages/client/ui-agent-preset/src/client/locales.ts
@@ -6,7 +6,7 @@ export type AgentPresetSettingsKey =
| 'nav' | 'sectionIntro' | 'builtIn' | 'defaultBadge' | 'setDefault' | 'edit' | 'view'
| 'duplicate' | 'delete' | 'newPreset' | 'presetId' | 'presetIdPlaceholder' | 'copyOf'
| 'displayName' | 'displayNamePlaceholder' | 'displayDescription' | 'displayDescriptionPlaceholder'
- | 'inUse' | 'noDescription'
+ | 'inUse' | 'noDescription' | 'builtInGroup' | 'customGroup'
| 'composition' | 'readOnlyNotice' | 'save' | 'saving' | 'cancel' | 'close' | 'retry'
| 'idRequired' | 'idInvalid' | 'idTaken'
| 'deleteTitle' | 'deleteDescription' | 'deleteConfirm' | 'deleting'
@@ -17,7 +17,7 @@ export const en: Record
= {
description: 'Applies to sessions you start from now on. Running sessions keep the preset they began with.',
loading: 'Loading presets…',
error: 'Could not load agent presets.',
- userTrust: 'Local',
+ userTrust: 'Custom',
seatHint: 'Agent preset for this session — switchable until you send the first message',
lockedHint: 'This session\'s agent preset is fixed once the conversation starts',
nav: 'Agent presets',
@@ -39,6 +39,8 @@ export const en: Record = {
displayDescription: 'Description',
displayDescriptionPlaceholder: 'One sentence on what this preset is for',
inUse: 'In use',
+ builtInGroup: 'Built-in',
+ customGroup: 'Custom',
noDescription: 'No description.',
copyOf: 'Copied from',
composition: 'Composition (cordis.yml)',
@@ -64,7 +66,7 @@ export const zh: Record = {
description: '对此后新建的会话生效。运行中的会话保持它开始时的 preset。',
loading: '正在加载预设…',
error: '无法加载 Agent 预设。',
- userTrust: '本地',
+ userTrust: '自定义',
seatHint: '本会话的 Agent 预设 —— 发送第一条消息前可切换',
lockedHint: '会话开始后，其 Agent 预设即固定',
nav: 'Agent 预设',
@@ -84,6 +86,8 @@ export const zh: Record = {
displayDescription: '描述',
displayDescriptionPlaceholder: '一句话说明这个预设做什么',
inUse: '当前使用',
+ builtInGroup: '内置',
+ customGroup: '自定义',
noDescription: '暂无描述。',
copyOf: '复制自',
composition: '组装（cordis.yml）',
diff --git a/packages/client/ui-agent-preset/tests/section.spec.tsx b/packages/client/ui-agent-preset/tests/section.spec.tsx
index 0ae4eb88cf..34e441cb1c 100644
--- a/packages/client/ui-agent-preset/tests/section.spec.tsx
+++ b/packages/client/ui-agent-preset/tests/section.spec.tsx
@@ -98,35 +98,65 @@ describe('the preset list', () => {
expect(within(rowFor('mine')).getByText(en.userTrust)).toBeTruthy()
})
+ it('separates built-in presets from custom ones', () => {
+ renderSection()
+
+ // Two different things: one set ships with the deployment and is
+ // read-only, the other is the user's own.
+ expect(screen.getByRole('heading', { name: en.builtInGroup })).toBeTruthy()
+ expect(screen.getByRole('heading', { name: en.customGroup })).toBeTruthy()
+ expect(within(rowFor('standard')).getByText(en.builtIn)).toBeTruthy()
+ expect(within(rowFor('mine')).getByText(en.userTrust)).toBeTruthy()
+ })
+
+ it('shows no group heading for a set nobody has', () => {
+ renderSection({ rows: [{ id: 'standard', trust: 'system', isDefault: true }] })
+
+ expect(screen.queryByRole('heading', { name: en.customGroup })).toBeNull()
+ })
+
+ it('picks a preset by clicking its card, and the one in use is inert', () => {
+ const actions = renderSection()
+
+ const inUse = within(rowFor('standard')).getByRole('button', { name: `${en.inUse}: 标准模式` })
+ expect(inUse).toHaveProperty('disabled', true)
+ fireEvent.click(inUse)
+
+ // Clicking the card IS the choice; the preset already in use cannot be
+ // re-picked, so the click reaches nothing.
+ expect(actions.makeDefault).not.toHaveBeenCalled()
+ })
+
it('offers Edit for a local preset and View for a shipped one', () => {
renderSection()
- expect(within(rowFor('mine')).getByText(en.edit)).toBeTruthy()
+ expect(within(rowFor('mine')).getByRole('button', { name: en.edit })).toBeTruthy()
// A shipped composition is readable but not editable, and the label is
// what says so before the editor opens.
- expect(within(rowFor('standard')).getByText(en.view)).toBeTruthy()
+ expect(within(rowFor('standard')).getByRole('button', { name: en.view })).toBeTruthy()
})
it('offers Delete only for a locally authored preset', () => {
renderSection()
- expect(within(rowFor('mine')).getByText(en.delete)).toBeTruthy()
- expect(within(rowFor('standard')).queryByText(en.delete)).toBeNull()
+ expect(within(rowFor('mine')).getByRole('button', { name: en.delete })).toBeTruthy()
+ expect(within(rowFor('standard')).queryByRole('button', { name: en.delete })).toBeNull()
})
it('hides duplication and disables creation when nothing is writable', () => {
renderSection({ authorable: false })
- expect(screen.queryByText(en.duplicate)).toBeNull()
+ expect(screen.queryByRole('button', { name: en.duplicate })).toBeNull()
expect(screen.getByText(`+ ${en.newPreset}`)).toHaveProperty('disabled', true)
})
it('routes the row actions to the controller', () => {
const actions = renderSection()
- fireEvent.click(within(rowFor('mine')).getByText(en.setDefault))
- fireEvent.click(within(rowFor('mine')).getByText(en.edit))
- fireEvent.click(within(rowFor('mine')).getByText(en.duplicate))
+ // The card body is the control that picks a preset.
+ fireEvent.click(within(rowFor('mine')).getByRole('button', { name: `${en.setDefault}: mine` }))
+ fireEvent.click(within(rowFor('mine')).getByRole('button', { name: en.edit }))
+ fireEvent.click(within(rowFor('mine')).getByRole('button', { name: en.duplicate }))
fireEvent.click(screen.getByText(`+ ${en.newPreset}`))
expect(actions.makeDefault).toHaveBeenCalledWith('mine')
@@ -259,7 +289,7 @@ describe('deleting a preset', () => {
it('asks before deleting', () => {
const actions = renderSection()
- fireEvent.click(within(rowFor('mine')).getByText(en.delete))
+ fireEvent.click(within(rowFor('mine')).getByRole('button', { name: en.delete }))
expect(actions.confirmDelete).toHaveBeenCalledWith('mine')
})