Apply suggestions from code review

PederHP · halter73 · web-flow · commit e2ba19d903f8 · 2025-08-26T20:36:30.000+02:00
Co-authored-by: Stephen Halter &lt;halter73@gmail.com&gt;
diff --git a/samples/AspNetCoreMcpPerSessionTools/Program.cs b/samples/AspNetCoreMcpPerSessionTools/Program.cs
@@ -20,9 +20,6 @@
             var toolCollection = mcpOptions.Capabilities?.Tools?.ToolCollection;
             if (toolCollection != null)
             {
-                // Clear all tools first
-                toolCollection.Clear();
-
                 // Add tools based on the requested category
                 switch (toolCategory?.ToLower())
                 {
@@ -51,10 +48,7 @@
                 }
             }
         };
-    })
-    .WithTools<ClockTool>()
-    .WithTools<CalculatorTool>()
-    .WithTools<UserInfoTool>();
+    });
 
 // Add OpenTelemetry for observability
 builder.Services.AddOpenTelemetry()

Original file line number	Diff line number	Diff line change
`@@ -20,9 +20,6 @@`
`20`	`20`	`var toolCollection = mcpOptions.Capabilities?.Tools?.ToolCollection;`
`21`	`21`	`if (toolCollection != null)`
`22`	`22`	`{`
`23`		`- // Clear all tools first`
`24`		`- toolCollection.Clear();`
`25`		`-`
`26`	`23`	`// Add tools based on the requested category`
`27`	`24`	`switch (toolCategory?.ToLower())`
`28`	`25`	`{`
`@@ -51,10 +48,7 @@`
`51`	`48`	`}`
`52`	`49`	`}`
`53`	`50`	`};`
`54`		`- })`
`55`		`- .WithTools<ClockTool>()`
`56`		`- .WithTools<CalculatorTool>()`
`57`		`- .WithTools<UserInfoTool>();`
	`51`	`+ });`
`58`	`52`
`59`	`53`	`// Add OpenTelemetry for observability`
`60`	`54`	`builder.Services.AddOpenTelemetry()`