René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Bug: `get_module_summary` returns files=0/functions=0/lines=0 on Windows (path_matches only checks `/` separator)","articleBody":"## Bug Report\n\n### Summary\n\n`get_module_summary` returns `files=0`, `functions=0`, `classes=0`, `lines=0` on Windows for any directory. `external_deps` incorrectly returns all workspace-wide external dependencies instead of module-scoped ones.\n\n### Environment\n- **OS**: Windows 10/11\n- **Package**: `@astudioplus/codegraph-mcp` 0.18.2–0.18.4\n- **Graph size**: 65,556 nodes, 2,615 CodeFiles\n\n### Root Cause\n\nFile: `crates/codegraph-server/src/domain/module_summary.rs`, function `path_matches()` (line ~217):\n\n```rust\nfn path_matches(path: \u0026str, prefix: \u0026str) -\u003e bool {\n    if prefix.is_empty() { return true; }\n    if path == prefix { return true; }\n    path.starts_with(prefix) \u0026\u0026 path[prefix.len()..].starts_with('/')  // \u003c-- BUG\n}\n```\n\nOn Windows, the graph stores paths with backslash separators (via `file_path.display().to_string()`):\n- `path` = `\"C:\\Users\\...\\agent\\prompt_builder.py\"`\n- `prefix` = `\"C:\\Users\\...\\agent\"`\n\n`path.starts_with(prefix)` is `true`, but `path[prefix.len()..]` starts with `\\` not `/`, so the function returns `false`. Every CodeFile/Function/Class node is skipped, resulting in all-zero counts.\n\nI verified this with a local Rust reproduction:\n\n```\npath_matches_original(\"C:\\Users\\...\\agent\\prompt_builder.py\", \"C:\\Users\\...\\agent\") =\u003e false\npath_matches_fixed(\"C:\\Users\\...\\agent\\prompt_builder.py\", \"C:\\Users\\...\\agent\")    =\u003e true\n```\n\n### Proposed Fix\n\nNormalise both sides to forward slashes before comparing:\n\n```rust\n#[inline]\nfn path_matches(path: \u0026str, prefix: \u0026str) -\u003e bool {\n    if prefix.is_empty() {\n        return true;\n    }\n    let norm_path: String = path.replace('\\\\', \"/\");\n    let norm_prefix: String = prefix.replace('\\\\', \"/\");\n    if norm_path == norm_prefix {\n        return true;\n    }\n    norm_path.starts_with(norm_prefix.as_str())\n        \u0026\u0026 norm_path[norm_prefix.len()..].starts_with('/')\n}\n```\n\nWith corresponding Windows test cases:\n\n```rust\n#[test]\nfn test_path_matches_windows_backslash() {\n    assert!(path_matches(\n        \"C:\\\\Users\\\\me\\\\project\\\\src\\\\main.rs\",\n        \"C:/Users/me/project/src\"\n    ));\n}\n\n#[test]\nfn test_path_matches_windows_backslash_prefix() {\n    assert!(path_matches(\n        \"C:\\\\Users\\\\me\\\\project\\\\src\\\\main.rs\",\n        \"C:\\\\Users\\\\me\\\\project\\\\src\"\n    ));\n}\n\n#[test]\nfn test_path_matches_windows_no_partial_dir() {\n    assert!(!path_matches(\n        \"C:\\\\Users\\\\me\\\\project\\\\src_foo\\\\main.rs\",\n        \"C:/Users/me/project/src\"\n    ));\n}\n```\n\n### Additional Issue: `trim_end_matches('/')` doesn't trim `\\`\n\nLine 64: `let prefix = directory.trim_end_matches('/');`\n\nThis also needs to trim backslash on Windows. Suggested fix:\n\n```rust\nlet prefix = directory.trim_end_matches(|c| c == '/' || c == '\\\\');\n```\n\n### Steps to Reproduce\n\n1. Install codegraph-mcp on Windows\n2. Index a workspace\n3. Call `get_module_summary` with any valid directory path:\n   ```\n   get_module_summary(path=\"C:\\\\Users\\\\me\\\\project\\\\src\")\n   ```\n4. Observe: `files=0`, `total_functions=0`, `total_classes=0`, `total_lines=0`\n\n### Expected Result\n\nNon-zero counts matching the actual files/functions/classes in that directory.\n\n### Workaround\n\nNone currently available — the bug is in the compiled Rust binary.\n","author":{"url":"https://github.com/MuBeiGe","@type":"Person","name":"MuBeiGe"},"datePublished":"2026-06-05T15:58:47.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/5/CodeGraph/issues/5"}