mirror of
https://github.com/ggml-org/llama.cpp.git
synced 2026-07-30 14:40:48 -05:00
* feat: Add shimmer text animation for processing state indicators * feat: Redesign CollapsibleContentBlock component with improved UX * feat: Add conditional setting display support with dependsOn field * feat: Add showAgenticTurnStats setting for per-turn statistics * feat: Update ChatMessageAgenticContent with improved UI and new features * feat: Enhance file read tool UI/UX * feat: Refine styling of collapsible content and code preview blocks * feat: add terminal variant to CollapsibleContentBlock * feat: add built-in tools UI registry * feat: extract ChatMessageReasoningBlock and ChatMessageToolCallBlock * refactor: simplify ChatMessageAgenticContent to use extracted blocks * fix: correct markdown content block margin spacing * fix: reorganize SettingsChatFields layout and reset button positioning * fix: use direct map access in agentic store session methods * refactor: remove reasoning preview/throttle system from CollapsibleContentBlock * feat: add auto-scroll to reasoning block and remove showThoughtInProgress * feat: add ChatMessageToolCallDateTime component and support for new tool types * feat: improve auto-scroll reliability in reasoning block with RAF coalescing and MutationObserver * feat: show MCP server favicon for tools without a built-in icon * feat: add search-results parsing utilities and tests * feat: add ChatMessageToolCallSearchResults component * feat: integrate search results rendering into ChatMessageAgenticContent * feat: display tool call input alongside output in ChatMessageToolCallBlock * style: use muted foreground color in reasoning block content * chore: Format * feat: Refine reasoning block layout and make pending thoughts display configurable * feat: Stream tool call code blocks with auto-scroll and handle partial JSON * feat: add streaming permission gate infrastructure * feat: wire permission gate into the agentic loop * fix: bail out on abort and skip already-approved tool calls * fix: clear partial tool calls on abort and savePartialResponse * test: cover partial tool call cleanup end-to-end * refactor: Remove streaming permission gate logic * fix: Correct autoscroll and streaming gates for tool calls and reasoning blocks * refactor: Chat Message Assistant componentization * fix: Show health metadata for disabled MCP servers and promote connections on enable * fix: Inherit global enabled state for missing MCP per-chat overrides * refactor: Cleanup * refactor: Split ChatMessageToolCallBlock into dedicated components * feat: Add live streaming and auto-scroll for tool execution output * feat: Add line numbers and change markers to file edit diffs * chore: Formatting * feat: Add type definitions and utilities for recommended MCP servers * feat: Add recommended MCP servers configuration and storage key * feat: Add McpServerCardCompact component for recommended servers * feat: Add recommended servers section to Add New Server dialog * feat: Update McpServerForm to support authorization requirements * feat: Add select-none classes for text selection prevention * feat: Add recommended MCP server icon assets * refactor: Store dismissed MCP recommendations as a boolean flag * feat: Render tool results as JSON or Markdown based on detected content type * feat: UI improvement * feat: Render search block early and update heading to show execution state * fix: Prevent non-web-search tools from triggering the search UI block * refactor: Cleanup * refactor: Extract hardcoded icon size classes into shared constants * refactor: Extract hardcoded tool result separator into a shared constant * refactor: Tool Calls UI/logic * refactor: Cleanup * refactor: Cleanup * refactor: Cleanup
137 lines
4.7 KiB
TypeScript
137 lines
4.7 KiB
TypeScript
import { describe, expect, it } from 'vitest';
|
|
import { DiffLineKind } from '$lib/enums';
|
|
import { computeLineDiff, renderUnifiedDiff, type DiffLine } from '$lib/utils';
|
|
|
|
describe('computeLineDiff', () => {
|
|
it('returns empty for two empty inputs', () => {
|
|
expect(computeLineDiff('', '')).toEqual([]);
|
|
});
|
|
|
|
it('marks every line as removed for an empty new text', () => {
|
|
expect(computeLineDiff('a\nb\nc', '')).toEqual([
|
|
{ kind: 'remove', text: 'a', oldLine: 1 },
|
|
{ kind: 'remove', text: 'b', oldLine: 2 },
|
|
{ kind: 'remove', text: 'c', oldLine: 3 }
|
|
]);
|
|
});
|
|
|
|
it('marks every line as added for an empty old text', () => {
|
|
expect(computeLineDiff('', 'a\nb')).toEqual([
|
|
{ kind: 'add', text: 'a', newLine: 1 },
|
|
{ kind: 'add', text: 'b', newLine: 2 }
|
|
]);
|
|
});
|
|
|
|
it('detects a single-line replace', () => {
|
|
expect(computeLineDiff('old', 'new')).toEqual([
|
|
{ kind: 'add', text: 'new', newLine: 1 },
|
|
{ kind: 'remove', text: 'old', oldLine: 1 }
|
|
]);
|
|
});
|
|
|
|
it('preserves interleaved context around additions', () => {
|
|
const oldText = ['a', 'b', 'c'].join('\n');
|
|
const newText = ['a', 'b', 'B', 'c'].join('\n');
|
|
expect(computeLineDiff(oldText, newText)).toEqual([
|
|
{ kind: 'context', text: 'a', oldLine: 1, newLine: 1 },
|
|
{ kind: 'context', text: 'b', oldLine: 2, newLine: 2 },
|
|
{ kind: 'add', text: 'B', newLine: 3 },
|
|
{ kind: 'context', text: 'c', oldLine: 3, newLine: 4 }
|
|
]);
|
|
});
|
|
|
|
it('preserves interleaved context around an isolated replace', () => {
|
|
// Multi-line context around a one-line change -> the diff should
|
|
// show context flanking the changed line at its natural position.
|
|
const oldText = ['a', 'b', 'c', 'd'].join('\n');
|
|
const newText = ['a', 'b', 'X', 'd'].join('\n');
|
|
expect(computeLineDiff(oldText, newText)).toEqual([
|
|
{ kind: 'context', text: 'a', oldLine: 1, newLine: 1 },
|
|
{ kind: 'context', text: 'b', oldLine: 2, newLine: 2 },
|
|
{ kind: 'add', text: 'X', newLine: 3 },
|
|
{ kind: 'remove', text: 'c', oldLine: 3 },
|
|
{ kind: 'context', text: 'd', oldLine: 4, newLine: 4 }
|
|
]);
|
|
});
|
|
|
|
it('preserves interleaved context around removals', () => {
|
|
const oldText = ['a', 'b', 'c', 'd'].join('\n');
|
|
const newText = ['a', 'c', 'd'].join('\n');
|
|
expect(computeLineDiff(oldText, newText)).toEqual([
|
|
{ kind: 'context', text: 'a', oldLine: 1, newLine: 1 },
|
|
{ kind: 'remove', text: 'b', oldLine: 2 },
|
|
{ kind: 'context', text: 'c', oldLine: 3, newLine: 2 },
|
|
{ kind: 'context', text: 'd', oldLine: 4, newLine: 3 }
|
|
]);
|
|
});
|
|
|
|
it('handles purely identical inputs', () => {
|
|
const text = 'x\ny\nz';
|
|
const result = computeLineDiff(text, text);
|
|
expect(result).toEqual([
|
|
{ kind: 'context', text: 'x', oldLine: 1, newLine: 1 },
|
|
{ kind: 'context', text: 'y', oldLine: 2, newLine: 2 },
|
|
{ kind: 'context', text: 'z', oldLine: 3, newLine: 3 }
|
|
]);
|
|
});
|
|
|
|
it('strips a trailing newline on the old/new inputs', () => {
|
|
expect(computeLineDiff('a\n', 'a\nb\n')).toEqual([
|
|
{ kind: 'context', text: 'a', oldLine: 1, newLine: 1 },
|
|
{ kind: 'add', text: 'b', newLine: 2 }
|
|
]);
|
|
});
|
|
|
|
it('normalizes trailing CR on each line', () => {
|
|
expect(computeLineDiff('a\r\nb\r\n', 'a\nb')).toEqual([
|
|
{ kind: 'context', text: 'a', oldLine: 1, newLine: 1 },
|
|
{ kind: 'context', text: 'b', oldLine: 2, newLine: 2 }
|
|
]);
|
|
});
|
|
|
|
it('keeps line numbers monotonic across mixed add/remove/context', () => {
|
|
const oldText = ['l1', 'l2', 'l3', 'l4', 'l5'].join('\n');
|
|
const newText = ['l1', 'l2-EDIT', 'l3', 'l4-NEW', 'l5'].join('\n');
|
|
const diff = computeLineDiff(oldText, newText);
|
|
|
|
// Walk the diff: every oldLine must increase strictly, and every
|
|
// newLine must increase strictly. Lines missing one side (add or
|
|
// remove) carry no number on that side.
|
|
let lastOld = 0;
|
|
let lastNew = 0;
|
|
for (const line of diff) {
|
|
if (line.oldLine !== undefined) {
|
|
expect(line.oldLine).toBeGreaterThan(lastOld);
|
|
lastOld = line.oldLine;
|
|
}
|
|
if (line.newLine !== undefined) {
|
|
expect(line.newLine).toBeGreaterThan(lastNew);
|
|
lastNew = line.newLine;
|
|
}
|
|
}
|
|
});
|
|
});
|
|
|
|
describe('renderUnifiedDiff', () => {
|
|
it('returns empty string for empty diff', () => {
|
|
expect(renderUnifiedDiff([])).toBe('');
|
|
});
|
|
|
|
it('prefixes each line with `+`, `-`, or a single space', () => {
|
|
const lines: DiffLine[] = [
|
|
{ kind: DiffLineKind.CONTEXT, text: 'ctx' },
|
|
{ kind: DiffLineKind.ADD, text: 'plus' },
|
|
{ kind: DiffLineKind.REMOVE, text: 'minus' }
|
|
];
|
|
expect(renderUnifiedDiff(lines)).toBe(' ctx\n+plus\n-minus');
|
|
});
|
|
|
|
it('ignores oldLine/newLine metadata when emitting prefixes', () => {
|
|
const lines: DiffLine[] = [
|
|
{ kind: DiffLineKind.CONTEXT, text: 'a', oldLine: 1, newLine: 1 },
|
|
{ kind: DiffLineKind.ADD, text: 'b', newLine: 2 }
|
|
];
|
|
expect(renderUnifiedDiff(lines)).toBe(' a\n+b');
|
|
});
|
|
});
|