dock Claude's task list above the composer (T-308)
Claude's TodoWrite checklist was invisible. Add a TaskItem/TaskStatus model + a latest-wins parser (taskListFrom) that reads the most recent TodoWrite tool call (it replaces the whole list each time), and a compact display-only ClaudeTaskDock pinned between the conversation and the composer: collapsed to "N tasks · M done" + the current in-progress item, expandable to the full checklist with per-item status glyphs + a11y labels. Hidden when there are no tasks. Parser + widget tests. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -0,0 +1,72 @@
|
||||
/// Widget coverage for the docked task list (T-308): hidden when empty,
|
||||
/// collapsed summary with the current in-progress item, expand/collapse, and
|
||||
/// per-item status semantics.
|
||||
library;
|
||||
|
||||
import 'package:clide/builtin/claude/src/claude_task_dock.dart';
|
||||
import 'package:clide/builtin/claude/src/task_list.dart';
|
||||
import 'package:clide/widgets/widgets.dart';
|
||||
import 'package:flutter/widgets.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
|
||||
import '../../helpers/kernel_fixture.dart';
|
||||
import '../../helpers/widget_harness.dart';
|
||||
|
||||
void main() {
|
||||
late KernelFixture f;
|
||||
setUp(() async => f = await KernelFixture.create());
|
||||
tearDown(() => f.dispose());
|
||||
|
||||
const tasks = [
|
||||
TaskItem(text: 'wire the dock', status: TaskStatus.completed),
|
||||
TaskItem(text: 'render the rows', status: TaskStatus.inProgress),
|
||||
TaskItem(text: 'write the tests', status: TaskStatus.pending),
|
||||
];
|
||||
|
||||
Future<void> pump(WidgetTester tester, List<TaskItem> items) async {
|
||||
await tester.pumpWidget(harness(
|
||||
f,
|
||||
Align(alignment: Alignment.topLeft, child: SizedBox(width: 400, child: ClaudeTaskDock(tasks: items))),
|
||||
));
|
||||
await tester.pump();
|
||||
}
|
||||
|
||||
testWidgets('renders nothing when there are no tasks', (tester) async {
|
||||
await pump(tester, const []);
|
||||
expect(find.byType(ClideText), findsNothing);
|
||||
expect(tester.takeException(), isNull);
|
||||
});
|
||||
|
||||
testWidgets('collapsed: a one-line summary + the current in-progress item; rows hidden', (tester) async {
|
||||
final handle = tester.ensureSemantics();
|
||||
await pump(tester, tasks);
|
||||
expect(find.text('3 tasks · 1 done'), findsOneWidget);
|
||||
expect(find.text('render the rows'), findsOneWidget); // current in-progress in the summary
|
||||
// The other rows aren't shown while collapsed.
|
||||
expect(find.text('wire the dock'), findsNothing);
|
||||
expect(find.text('write the tests'), findsNothing);
|
||||
expect(find.bySemanticsLabel('Claude task list, 3 tasks · 1 done, collapsed'), findsOneWidget);
|
||||
handle.dispose();
|
||||
});
|
||||
|
||||
testWidgets('tapping expands to the full checklist with per-item status', (tester) async {
|
||||
final handle = tester.ensureSemantics();
|
||||
await pump(tester, tasks);
|
||||
await tester.tap(find.bySemanticsLabel('Claude task list, 3 tasks · 1 done, collapsed'));
|
||||
await tester.pump();
|
||||
|
||||
expect(find.text('wire the dock'), findsOneWidget);
|
||||
expect(find.text('render the rows'), findsOneWidget);
|
||||
expect(find.text('write the tests'), findsOneWidget);
|
||||
// Per-item status announced for AT.
|
||||
expect(find.bySemanticsLabel('wire the dock, done'), findsOneWidget);
|
||||
expect(find.bySemanticsLabel('render the rows, in progress'), findsOneWidget);
|
||||
expect(find.bySemanticsLabel('write the tests, pending'), findsOneWidget);
|
||||
handle.dispose();
|
||||
});
|
||||
|
||||
testWidgets('singular task count reads "1 task"', (tester) async {
|
||||
await pump(tester, const [TaskItem(text: 'lonely', status: TaskStatus.pending)]);
|
||||
expect(find.text('1 task · 0 done'), findsOneWidget);
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,68 @@
|
||||
/// Parser for Claude's TodoWrite task list (T-308): latest-wins, status
|
||||
/// mapping, content/activeForm fallback, and graceful handling of non-TodoWrite
|
||||
/// or malformed input.
|
||||
library;
|
||||
|
||||
import 'package:clide/builtin/claude/src/task_list.dart';
|
||||
import 'package:clide/builtin/claude/src/transcript_reader.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
|
||||
final _t = DateTime.utc(2026);
|
||||
|
||||
AssistantToolUse _todo(Object todos, {String id = 'x'}) =>
|
||||
AssistantToolUse(uuid: id, timestamp: _t, isSidechain: false, toolUseId: id, name: 'TodoWrite', input: {'todos': todos});
|
||||
|
||||
void main() {
|
||||
test('parses todos with their status', () {
|
||||
final tasks = taskListFrom([
|
||||
_todo([
|
||||
{'content': 'a', 'status': 'pending'},
|
||||
{'content': 'b', 'status': 'in_progress'},
|
||||
{'content': 'c', 'status': 'completed'},
|
||||
])
|
||||
]);
|
||||
expect(tasks, const [
|
||||
TaskItem(text: 'a', status: TaskStatus.pending),
|
||||
TaskItem(text: 'b', status: TaskStatus.inProgress),
|
||||
TaskItem(text: 'c', status: TaskStatus.completed),
|
||||
]);
|
||||
});
|
||||
|
||||
test('the latest TodoWrite wins — a snapshot, not an append log', () {
|
||||
final tasks = taskListFrom([
|
||||
_todo([
|
||||
{'content': 'old', 'status': 'pending'}
|
||||
], id: '1'),
|
||||
_todo([
|
||||
{'content': 'new', 'status': 'in_progress'}
|
||||
], id: '2'),
|
||||
]);
|
||||
expect(tasks, const [TaskItem(text: 'new', status: TaskStatus.inProgress)]);
|
||||
});
|
||||
|
||||
test('content falls back to activeForm then empty; unknown status → pending', () {
|
||||
final tasks = taskListFrom([
|
||||
_todo([
|
||||
{'activeForm': 'doing it', 'status': 'in_progress'},
|
||||
{'status': 'weird'},
|
||||
])
|
||||
]);
|
||||
expect(tasks[0].text, 'doing it');
|
||||
expect(tasks[1].text, '');
|
||||
expect(tasks[1].status, TaskStatus.pending);
|
||||
});
|
||||
|
||||
test('no TodoWrite → empty', () {
|
||||
expect(taskListFrom(const []), isEmpty);
|
||||
expect(
|
||||
taskListFrom([
|
||||
AssistantToolUse(uuid: 'b', timestamp: _t, isSidechain: false, toolUseId: 'b', name: 'Bash', input: const {'command': 'ls'})
|
||||
]),
|
||||
isEmpty,
|
||||
);
|
||||
});
|
||||
|
||||
test('malformed todos (not a list) → empty', () {
|
||||
expect(taskListFrom([_todo('nope')]), isEmpty);
|
||||
});
|
||||
}
|
||||
Reference in New Issue
Block a user