From 6e6167f29cee4ebd613b91fbb8c0803dde2a046f Mon Sep 17 00:00:00 2001 From: jackwener Date: Tue, 11 Aug 2026 22:38:57 +0800 Subject: [PATCH] test: remove trivial parser and compatibility suites --- .../main/__tests__/side-chat-command.test.ts | 17 ---------- .../src/__tests__/tui-terminal-mock.test.ts | 12 ------- .../core/src/__tests__/graph-command.test.ts | 14 --------- .../permission-compatibility.test.ts | 31 ------------------- .../core/src/__tests__/swarm-command.test.ts | 14 --------- 5 files changed, 88 deletions(-) delete mode 100644 apps/desktop/src/main/__tests__/side-chat-command.test.ts delete mode 100644 packages/cli/src/__tests__/tui-terminal-mock.test.ts delete mode 100644 packages/core/src/__tests__/graph-command.test.ts delete mode 100644 packages/core/src/__tests__/permission-compatibility.test.ts delete mode 100644 packages/core/src/__tests__/swarm-command.test.ts diff --git a/apps/desktop/src/main/__tests__/side-chat-command.test.ts b/apps/desktop/src/main/__tests__/side-chat-command.test.ts deleted file mode 100644 index 38916c48a25..00000000000 --- a/apps/desktop/src/main/__tests__/side-chat-command.test.ts +++ /dev/null @@ -1,17 +0,0 @@ -import assert from 'node:assert/strict'; -import { describe, it } from 'node:test'; -import { - parseSideChatCommand, -} from '../../renderer/side-chat-command.js'; - -describe('side chat slash command', () => { - it('matches only /side and preserves an optional multiline prompt', () => { - assert.deepEqual(parseSideChatCommand('/side'), { prompt: '' }); - assert.deepEqual(parseSideChatCommand(' /side inspect this\ncarefully '), { - prompt: 'inspect this\ncarefully', - }); - assert.equal(parseSideChatCommand('/sidebar'), null); - assert.equal(parseSideChatCommand('before /side'), null); - }); - -}); diff --git a/packages/cli/src/__tests__/tui-terminal-mock.test.ts b/packages/cli/src/__tests__/tui-terminal-mock.test.ts deleted file mode 100644 index 95692dfc770..00000000000 --- a/packages/cli/src/__tests__/tui-terminal-mock.test.ts +++ /dev/null @@ -1,12 +0,0 @@ -import assert from 'node:assert/strict'; -import { test } from 'node:test'; -import { findInputSurfaceRows, inputSurfaceRows } from './tui-terminal-mock.js'; - -test('input surface lookup distinguishes an intermediate frame from a settled frame', () => { - const intermediate = ['────────────────']; - const settled = ['────────────────', 'draft', '────────────────']; - - assert.equal(findInputSurfaceRows(intermediate), undefined); - assert.deepEqual(findInputSurfaceRows(settled), [0, 2]); - assert.throws(() => inputSurfaceRows(intermediate)); -}); diff --git a/packages/core/src/__tests__/graph-command.test.ts b/packages/core/src/__tests__/graph-command.test.ts deleted file mode 100644 index 3cc2b2fcaa6..00000000000 --- a/packages/core/src/__tests__/graph-command.test.ts +++ /dev/null @@ -1,14 +0,0 @@ -import assert from 'node:assert/strict'; -import { describe, test } from 'node:test'; -import { parseGraphCommand } from '../graph-command.js'; - -describe('/graph parser', () => { - test('parses persistent mode changes', () => { - assert.deepEqual(parseGraphCommand('/graph on'), { kind: 'set_mode', mode: 'graph' }); - assert.deepEqual(parseGraphCommand('/graph off'), { kind: 'set_mode', mode: 'default' }); - }); - - test('does not claim lookalike slash commands', () => { - assert.equal(parseGraphCommand('/graphing now'), null); - }); -}); diff --git a/packages/core/src/__tests__/permission-compatibility.test.ts b/packages/core/src/__tests__/permission-compatibility.test.ts deleted file mode 100644 index 1092a4f5e34..00000000000 --- a/packages/core/src/__tests__/permission-compatibility.test.ts +++ /dev/null @@ -1,31 +0,0 @@ -import { describe, test } from 'node:test'; -import { expect } from '../test-helpers.js'; -import { - categorizeBash, - classifyToolUse, - isToolCategory, - permissionReasonForCategory, -} from '../permission.js'; - -describe('legacy permission payload classification', () => { - test('keeps historical command reviews conservatively categorized', () => { - expect(categorizeBash('ls -la')).toBe('shell_unsafe'); - expect(categorizeBash('rm -rf /tmp/example')).toBe('fs_destructive'); - expect(categorizeBash('git reset --hard')).toBe('git_destructive'); - expect(categorizeBash('sudo true')).toBe('privileged'); - }); - - test('keeps plan-mode tool availability classification independent of authorization', () => { - expect(classifyToolUse({ toolName: 'Read', args: {} })).toBe('read'); - expect(classifyToolUse({ toolName: 'Write', args: {} })).toBe('file_write'); - expect(classifyToolUse({ toolName: 'apply_patch', args: {} })).toBe('file_write'); - expect(classifyToolUse({ toolName: 'ExploreAgent', args: {}, categoryHint: 'subagent' })).toBe( - 'subagent', - ); - }); - - test('keeps Client Capabilities behind a conservative permission floor', () => { - expect(isToolCategory('client_capability')).toBe(true); - expect(permissionReasonForCategory('client_capability')).toBe('custom'); - }); -}); diff --git a/packages/core/src/__tests__/swarm-command.test.ts b/packages/core/src/__tests__/swarm-command.test.ts deleted file mode 100644 index f83c6ea07e0..00000000000 --- a/packages/core/src/__tests__/swarm-command.test.ts +++ /dev/null @@ -1,14 +0,0 @@ -import assert from 'node:assert/strict'; -import { describe, test } from 'node:test'; -import { parseSwarmCommand } from '../swarm-command.js'; - -describe('/swarm parser', () => { - test('parses explicit mode changes', () => { - assert.deepEqual(parseSwarmCommand('/swarm on'), { kind: 'set_mode', mode: 'swarm' }); - assert.deepEqual(parseSwarmCommand('/swarm off'), { kind: 'set_mode', mode: 'default' }); - }); - - test('does not claim lookalike slash commands', () => { - assert.equal(parseSwarmCommand('/swarming now'), null); - }); -});