Skip to content

Commit 901e2ce

Browse files
authored
Mark "root" as required. (#285)
This marks the "root" parameter for the create_project command as required, which it is, but wasn't previously marked as such.
1 parent 078abd2 commit 901e2ce

File tree

3 files changed

+7
-2
lines changed

3 files changed

+7
-2
lines changed

pkgs/dart_mcp/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
- Update the tool calling example to include progress notifications.
44
- Remove a reference to "screenshot" for a generic error that occurs for more
55
than just screenshots.
6+
- Mark the "root" parameter for create_project required.
67

78
## 0.3.3
89

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

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,11 @@ base mixin DashCliSupport on ToolsSupport, LoggingSupport, RootsTrackingSupport
253253
'boilerplate and example code. Defaults to true.',
254254
),
255255
},
256-
required: [ParameterNames.directory, ParameterNames.projectType],
256+
required: [
257+
ParameterNames.directory,
258+
ParameterNames.projectType,
259+
ParameterNames.root,
260+
],
257261
),
258262
);
259263

pkgs/dart_mcp_server/test/tools/dart_cli_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -434,7 +434,7 @@ dependencies:
434434
expect(result.isError, true);
435435
expect(
436436
(result.content.first as TextContent).text,
437-
contains('missing `root` key'),
437+
contains('Required property "root" is missing'),
438438
);
439439
expect(testProcessManager.commandsRan, isEmpty);
440440
});

0 commit comments

Comments
 (0)