fix: CI test + mypy failures on main

- mypy: rename inner http handler in `cli.py` visualize-serve path
  from `handler` to `http_handler`. The daemon dispatcher at the top
  of `main()` already binds `handler: Callable[[Namespace], None]`,
  and reusing the name for `functools.partial(SimpleHTTPRequestHandler, ...)`
  made mypy unify the two scopes and reject both the assignment and the
  HTTPServer call site.

- test: align `test_install_qoder_config` with the canonical
  `_detect_serve_command()` pattern used by sibling tests in the same
  class (lines 528, 594). The hard-coded `"code-review-graph"` fallback
  predates PR #287, which switched the no-uvx fallback to
  `sys.executable -m code_review_graph serve` for venv robustness.
  On CI runners without `uvx` the test was asserting against a stale
  contract.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: tirth8205

code_review_graph/cli.py

@@ -1034,13 +1034,13 @@ def main() -> None:
 
                     serve_dir = html_path.parent
                     port = 8765
-                    handler = functools.partial(
+                    http_handler = functools.partial(
                         http.server.SimpleHTTPRequestHandler,
                         directory=str(serve_dir),
                     )
                     print(f"Serving at http://localhost:{port}/graph.html")
                     print("Press Ctrl+C to stop.")
-                    with http.server.HTTPServer(("localhost", port), handler) as httpd:
+                    with http.server.HTTPServer(("localhost", port), http_handler) as httpd:
                         try:
                             httpd.serve_forever()
                         except KeyboardInterrupt:

tests/test_skills.py

@@ -793,8 +793,7 @@ def test_install_qoder_config(self, tmp_path):
         assert "mcpServers" in data
         assert "code-review-graph" in data["mcpServers"]
         assert data["mcpServers"]["code-review-graph"]["type"] == "stdio"
-        import shutil
-        expected_cmd = "uvx" if shutil.which("uvx") else "code-review-graph"
+        expected_cmd, _ = _detect_serve_command()
         assert data["mcpServers"]["code-review-graph"]["command"] == expected_cmd