review feedback

Signed-off-by: Huamin Chen <[email protected]>

Author: rootfs

examples/mcp-classifier-server/server.py

@@ -530,9 +530,7 @@ async def health_check(request):
     parser.add_argument(
         "--http", action="store_true", help="Run in HTTP mode instead of stdio"
     )
-    parser.add_argument(
-        "--port", type=int, default=8090, help="HTTP port (default: 8090)"
-    )
+    parser.add_argument("--port", type=int, default=8090, help="HTTP port")
     args = parser.parse_args()
 
     if args.http:

src/semantic-router/pkg/connectivity/mcp/stdio_client.go

@@ -76,7 +76,7 @@ func (c *StdioClient) Connect() error {
 		}()
 
 		c.log(LoggingLevelDebug, fmt.Sprintf("Starting MCP client..."))
-		// Note: In mcp-go v0.42.0-beta.1, connection starts automatically; Start() method is not used
+		// Note: Connection starts automatically; Start() method is not required
 
 		c.log(LoggingLevelDebug, fmt.Sprintf("Initializing MCP client..."))
 		// Initialize the client with a simple request (no params needed for basic initialization)

src/semantic-router/pkg/utils/classification/classifier.go

@@ -296,7 +296,7 @@ func newClassifierWithOptions(cfg *config.RouterConfig, options ...option) (*Cla
 // NewClassifier creates a new classifier with model selection and jailbreak/PII detection capabilities.
 // Both in-tree and MCP classifiers can be configured simultaneously for category classification.
 // At runtime, in-tree classifier will be tried first, with MCP as a fallback,
-// allowing flexible deployment scenarios such as gradual migration or A/B testing.
+// allowing flexible deployment scenarios such as gradual migration.
 func NewClassifier(cfg *config.RouterConfig, categoryMapping *CategoryMapping, piiMapping *PIIMapping, jailbreakMapping *JailbreakMapping) (*Classifier, error) {
 	options := []option{
 		withJailbreak(jailbreakMapping, createJailbreakInitializer(cfg.PromptGuard.UseModernBERT), createJailbreakInference(cfg.PromptGuard.UseModernBERT)),

src/semantic-router/pkg/utils/classification/mcp_classifier.go

@@ -18,7 +18,7 @@ import (
 
 const (
 	// DefaultMCPThreshold is the default confidence threshold for MCP classification.
-	// For multi-class classification, a value of 0.5 means that a predicted class must have more than 50% confidence
+	// For multi-class classification, a value of 0.5 means that a predicted class must have at least 50% confidence
 	// to be selected. Adjust this threshold as needed for your use case.
 	DefaultMCPThreshold = 0.5
 )