Lint

Fixed challenge expectation

server.py

@@ -602,12 +602,10 @@ async def handle_list_tools() -> list[Tool]:
             # Log to activity file as well
             try:
                 mcp_activity_logger = logging.getLogger("mcp_activity")
-                friendly_name = client_info.get('friendly_name', 'Claude')
-                raw_name = client_info.get('name', 'Unknown')
-                version = client_info.get('version', 'Unknown')
-                mcp_activity_logger.info(
-                    f"MCP_CLIENT_INFO: {friendly_name} (raw={raw_name} v{version})"
-                )
+                friendly_name = client_info.get("friendly_name", "Claude")
+                raw_name = client_info.get("name", "Unknown")
+                version = client_info.get("version", "Unknown")
+                mcp_activity_logger.info(f"MCP_CLIENT_INFO: {friendly_name} (raw={raw_name} v{version})")
             except Exception:
                 pass
     except Exception as e:

tests/test_challenge.py

@@ -24,9 +24,9 @@ class TestChallengeTool:
     def test_tool_metadata(self):
         """Test that tool metadata matches requirements"""
         assert self.tool.get_name() == "challenge"
-        assert "CRITICAL CHALLENGE PROMPT" in self.tool.get_description()
-        assert "critical examination" in self.tool.get_description()
-        assert "default agreement" in self.tool.get_description()
+        assert "prevent reflexive agreement" in self.tool.get_description()
+        assert "think critically" in self.tool.get_description()
+        assert "thoughtful evaluation" in self.tool.get_description()
         assert self.tool.get_default_temperature() == 0.2  # TEMPERATURE_ANALYTICAL
 
     def test_requires_model(self):
@@ -154,8 +154,8 @@ class TestChallengeTool:
 
         assert "prompt" in fields
         assert fields["prompt"]["type"] == "string"
-        assert "statement" in fields["prompt"]["description"]
-        assert "critically evaluate" in fields["prompt"]["description"]
+        assert "message or statement" in fields["prompt"]["description"]
+        assert "analyze critically" in fields["prompt"]["description"]
 
     def test_required_fields_list(self):
         """Test required fields list"""

utils/client_info.py

@@ -25,19 +25,16 @@ CLIENT_NAME_MAPPINGS = {
     "claude-desktop": "Claude",
     "claude-code": "Claude",
     "anthropic": "Claude",
-
     # Gemini variants
     "gemini-cli-mcp-client": "Gemini",
     "gemini-cli": "Gemini",
     "gemini": "Gemini",
     "google": "Gemini",
-
     # Other known clients
     "cursor": "Cursor",
     "vscode": "VS Code",
     "codeium": "Codeium",
     "copilot": "GitHub Copilot",
-
     # Generic MCP clients
     "mcp-client": "MCP Client",
     "test-client": "Test Client",