Skip to content

Commit fa1c2be

Browse files
authored
Always use the failures-only reporter for testing (#160)
The default reporter has verbose output which is not useful but eats up a lot of input tokens once it's in the history. Always run tests with `--reporter=failures-only` to reduce the output size. This output format still includes a count of passing tests. Update changelog to focus on SDK version in the header and merge the updates.
1 parent 66a152f commit fa1c2be

File tree

4 files changed

+11
-8
lines changed

4 files changed

+11
-8
lines changed

pkgs/dart_mcp_server/CHANGELOG.md

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,4 @@
1-
# 0.1.1-wip
2-
3-
* Instruct clients to prefer MCP tools over running tools in the shell.
4-
5-
# 0.1.0 (Dart SDK 3.8.0)
1+
# Dart SDK 3.8.0 - WP
62

73
* Handle relative paths under roots without trailing slashes.
84
* Fix executable paths for dart/flutter on windows.
@@ -40,3 +36,5 @@
4036
* Add a hot reload tool.
4137
* Add basic analysis support.
4238
* Add the beginnings of a Dart tooling MCP server.
39+
* Instruct clients to prefer MCP tools over running tools in the shell.
40+
* Reduce output size of `run_tests` tool to save on input tokens.

pkgs/dart_mcp_server/lib/src/mixins/dash_cli.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ base mixin DashCliSupport on ToolsSupport, LoggingSupport, RootsTrackingSupport
6868
Future<CallToolResult> _runTests(CallToolRequest request) async {
6969
return runCommandInRoots(
7070
request,
71-
arguments: ['test'],
71+
arguments: ['test', '--reporter=failures-only'],
7272
commandDescription: 'dart|flutter test',
7373
processManager: processManager,
7474
knownRoots: await roots,

pkgs/dart_mcp_server/pubspec.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
name: dart_mcp_server
2-
version: 0.1.1-wip
32
description: >-
43
An MCP server for Dart projects, exposing various developer tools to AI
54
models.

pkgs/dart_mcp_server/test/tools/dart_cli_test.dart

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,13 +166,19 @@ dependencies:
166166
command: [
167167
endsWith(flutterExecutableName),
168168
'test',
169+
'--reporter=failures-only',
169170
'foo_test.dart',
170171
'bar_test.dart',
171172
],
172173
workingDirectory: exampleFlutterAppRoot.path,
173174
)),
174175
equalsCommand((
175-
command: [endsWith(dartExecutableName), 'test', 'zip_test.dart'],
176+
command: [
177+
endsWith(dartExecutableName),
178+
'test',
179+
'--reporter=failures-only',
180+
'zip_test.dart',
181+
],
176182
workingDirectory: dartCliAppRoot.path,
177183
)),
178184
]);

0 commit comments

Comments
 (0)