add more constants for map keys

pkgs/dart_tooling_mcp_server/lib/src/mixins/analyzer.dart

@@ -13,6 +13,7 @@ import 'package:language_server_protocol/protocol_generated.dart' as lsp;
 import 'package:meta/meta.dart';
 
 import '../lsp/wire_format.dart';
+import '../utils/constants.dart';
 
 /// Mix this in to any MCPServer to add support for analyzing Dart projects.
 ///
@@ -245,7 +246,7 @@ base mixin DartAnalyzerSupport
     for (var entry in diagnostics.entries) {
       for (var diagnostic in entry.value) {
         final diagnosticJson = diagnostic.toJson();
-        diagnosticJson['uri'] = entry.key.toString();
+        diagnosticJson[ParameterNames.uri] = entry.key.toString();
         messages.add(TextContent(text: jsonEncode(diagnosticJson)));
       }
     }
@@ -263,7 +264,7 @@ base mixin DartAnalyzerSupport
     final errorResult = await _ensurePrerequisites(request);
     if (errorResult != null) return errorResult;
 
-    final query = request.arguments!['query'] as String;
+    final query = request.arguments![ParameterNames.query] as String;
     final result = await _lspConnection.sendRequest(
       lsp.Method.workspace_symbol.toString(),
       lsp.WorkspaceSymbolParams(query: query).toJson(),
@@ -306,7 +307,7 @@ base mixin DartAnalyzerSupport
     );
     diagnostics[diagnosticParams.uri] = diagnosticParams.diagnostics;
     log(LoggingLevel.debug, {
-      'uri': diagnosticParams.uri,
+      ParameterNames.uri: diagnosticParams.uri,
       'diagnostics':
           diagnosticParams.diagnostics.map((d) => d.toJson()).toList(),
     });
@@ -369,14 +370,14 @@ base mixin DartAnalyzerSupport
     description: 'Look up a symbol or symbols in all workspaces by name.',
     inputSchema: Schema.object(
       properties: {
-        'query': Schema.string(
+        ParameterNames.query: Schema.string(
           description:
               'Queries are matched based on a case-insensitive partial name '
               'match, and do not support complex pattern matching, regexes, '
               'or scoped lookups.',
         ),
       },
-      required: ['query'],
+      required: [ParameterNames.query],
     ),
     annotations: ToolAnnotations(title: 'Project search', readOnlyHint: true),
   );

pkgs/dart_tooling_mcp_server/lib/src/mixins/dtd.dart

@@ -13,6 +13,8 @@ import 'package:meta/meta.dart';
 import 'package:vm_service/vm_service.dart';
 import 'package:vm_service/vm_service_io.dart';
 
+import '../utils/constants.dart';
+
 /// Mix this in to any MCPServer to add support for connecting to the Dart
 /// Tooling Daemon and all of its associated functionality (see
 /// https://pub.dev/packages/dtd).
@@ -149,7 +151,7 @@ base mixin DartToolingDaemonSupport
       return _dtdAlreadyConnected;
     }
 
-    if (request.arguments?['uri'] == null) {
+    if (request.arguments?[ParameterNames.uri] == null) {
       return CallToolResult(
         isError: true,
         content: [
@@ -160,7 +162,7 @@ base mixin DartToolingDaemonSupport
 
     try {
       _dtd = await DartToolingDaemon.connect(
-        Uri.parse(request.arguments!['uri'] as String),
+        Uri.parse(request.arguments![ParameterNames.uri] as String),
       );
       unawaited(_dtd!.done.then((_) async => await _resetDtd()));
 
@@ -502,8 +504,8 @@ base mixin DartToolingDaemonSupport
         'command. Do not just make up a random URI to pass.',
     annotations: ToolAnnotations(title: 'Connect to DTD', readOnlyHint: true),
     inputSchema: Schema.object(
-      properties: {'uri': Schema.string()},
-      required: const ['uri'],
+      properties: {ParameterNames.uri: Schema.string()},
+      required: const [ParameterNames.uri],
     ),
   );
 

pkgs/dart_tooling_mcp_server/lib/src/utils/constants.dart

@@ -7,6 +7,8 @@ extension ParameterNames on Never {
   static const command = 'command';
   static const packageName = 'packageName';
   static const paths = 'paths';
+  static const query = 'query';
   static const root = 'root';
   static const roots = 'roots';
+  static const uri = 'uri';
 }

pkgs/dart_tooling_mcp_server/test/test_harness.dart

@@ -10,6 +10,7 @@ import 'package:async/async.dart';
 import 'package:dart_mcp/client.dart';
 import 'package:dart_tooling_mcp_server/src/mixins/dtd.dart';
 import 'package:dart_tooling_mcp_server/src/server.dart';
+import 'package:dart_tooling_mcp_server/src/utils/constants.dart';
 import 'package:dtd/dtd.dart';
 import 'package:path/path.dart' as p;
 import 'package:process/process.dart';
@@ -116,7 +117,7 @@ class TestHarness {
     final result = await callToolWithRetry(
       CallToolRequest(
         name: connectTool.name,
-        arguments: {'uri': fakeEditorExtension.dtdUri},
+        arguments: {ParameterNames.uri: fakeEditorExtension.dtdUri},
       ),
     );
 

pkgs/dart_tooling_mcp_server/test/tools/analyzer_test.dart

@@ -6,6 +6,7 @@ import 'dart:io';
 
 import 'package:dart_mcp/server.dart';
 import 'package:dart_tooling_mcp_server/src/mixins/analyzer.dart';
+import 'package:dart_tooling_mcp_server/src/utils/constants.dart';
 import 'package:test/test.dart';
 import 'package:test_descriptor/test_descriptor.dart' as d;
 
@@ -96,7 +97,7 @@ void main() {
       final result = await testHarness.callToolWithRetry(
         CallToolRequest(
           name: DartAnalyzerSupport.resolveWorkspaceSymbolTool.name,
-          arguments: {'query': 'DartAnalyzerSupport'},
+          arguments: {ParameterNames.query: 'DartAnalyzerSupport'},
         ),
       );
       expect(result.isError, isNot(true));
@@ -130,7 +131,7 @@ void main() {
       final result = await testHarness.callToolWithRetry(
         CallToolRequest(
           name: DartAnalyzerSupport.resolveWorkspaceSymbolTool.name,
-          arguments: {'query': 'DartAnalyzerSupport'},
+          arguments: {ParameterNames.query: 'DartAnalyzerSupport'},
         ),
         expectError: true,
       );

pkgs/dart_tooling_mcp_server/test/tools/dtd_test.dart

@@ -7,6 +7,7 @@ import 'dart:convert';
 import 'package:async/async.dart';
 import 'package:dart_mcp/server.dart';
 import 'package:dart_tooling_mcp_server/src/mixins/dtd.dart';
+import 'package:dart_tooling_mcp_server/src/utils/constants.dart';
 import 'package:test/test.dart';
 import 'package:vm_service/vm_service.dart';
 
@@ -347,7 +348,7 @@ void main() {
             await resourceUpdatedQueue.next,
             isA<ResourceUpdatedNotification>().having(
               (n) => n.uri,
-              'uri',
+              ParameterNames.uri,
               resource.uri,
             ),
           );
@@ -371,7 +372,7 @@ void main() {
             await resourceUpdatedQueue.next,
             isA<ResourceUpdatedNotification>().having(
               (n) => n.uri,
-              'uri',
+              ParameterNames.uri,
               resource.uri,
             ),
           );