coderadar-rs 0.9.0__tar.gz → 0.9.2__tar.gz

{coderadar_rs-0.9.0 → coderadar_rs-0.9.2}/Cargo.lock

@@ -445,7 +445,7 @@ checksum = "773648b94d0e5d620f64f280777445740e61fe701025087ec8b57f45c791888b"
 
 [[package]]
 name = "core_indexer"
-version = "0.9.0"
+version = "0.9.2"
 dependencies = [
  "crossbeam-channel",
  "dashmap",

{coderadar_rs-0.9.0 → coderadar_rs-0.9.2}/Cargo.toml

@@ -3,6 +3,6 @@ members = ["core_indexer"]
 resolver = "2"
 
 [workspace.package]
-version = "0.9.0"
+version = "0.9.2"
 edition = "2021"
 license = "MIT"

{coderadar_rs-0.9.0 → coderadar_rs-0.9.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: coderadar-rs
-Version: 0.9.0
+Version: 0.9.2
 Classifier: Development Status :: 4 - Beta
 Classifier: Intended Audience :: Developers
 Classifier: License :: OSI Approved :: MIT License
@@ -21,7 +21,7 @@ Requires-Dist: pytest>=7.0 ; extra == 'dev'
 Requires-Dist: pytest-asyncio>=0.21 ; extra == 'dev'
 Requires-Dist: hypothesis>=6.0 ; extra == 'dev'
 Requires-Dist: mypy>=1.0 ; extra == 'dev'
-Requires-Dist: ruff>=0.1 ; extra == 'dev'
+Requires-Dist: ruff==0.16.5 ; extra == 'dev'
 Requires-Dist: graphviz>=0.20 ; extra == 'visualizers'
 Provides-Extra: dev
 Provides-Extra: visualizers
@@ -32,7 +32,7 @@ License: MIT
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown; charset=UTF-8; variant=GFM
 
-# CodeRadar v0.9.0
+# CodeRadar v0.9.2
 
 [![CI](https://github.com/opticsWolf/coderadar/actions/workflows/ci.yml/badge.svg)](https://github.com/opticsWolf/coderadar/actions/workflows/ci.yml)
 [![PyPI](https://img.shields.io/pypi/v/coderadar-rs?label=pypi)](https://pypi.org/project/coderadar-rs/)
@@ -89,7 +89,7 @@ Rust Core (ProjectedGraph, Tree-sitter 41-lang, Parallel Extraction,
 | Metric | Value |
 |--------|-------|
 | **Languages indexed** | 41 (12 Tier 1, 29 Tier 2, 330+ Tier 3) |
-| **Tests** | 1119 (368 Rust + 751 Python) |
+| **Tests** | 1128 (369 Rust + 759 Python) |
 | **MCP Tools** | 22 — 17 `codegraph_*` (explore, search, node, affected, query, search_similar, compute_embeddings, module_children, as_of, traverse, get_smells, dead_code, find_clones, find_scaffolding, reindex, update_file, set_project) + 5 `coderadar_*` (resolve, replace_body, update_signature, rename, create_entity) |
 | **Query surface** | Pest structural + Macrame agent traversals + vector search |
 | **Frameworks** | Django, Flask, FastAPI, Go, Actix, Express, Spring Boot, Laravel, ASP.NET, Rails, NestJS, Vue Router, React Router |
@@ -264,7 +264,7 @@ not assume the cwd is the project:
 |------|-----------|------------|----------|
 | **Tier 1** | Python, TypeScript, JavaScript, Rust, Go, Java, C, C++, Ruby, PHP, C#, Kotlin | Import → Signature → Framework | Full tool suite |
 | **Tier 2** | Swift, Scala, Lua, Elixir, Zig, R, Bash, Dart, Protobuf, Dockerfile, SQL, HCL, CMake, GraphQL, Erlang, Haskell, **Nix, Shell, Groovy, Perl, SystemVerilog, OCaml, Clojure, F#, Verilog, Julia, PowerShell, Emacs Lisp, Objective-C** | Import → Signature | replace_body, create_entity |
-| **Tier 3** | Shell, SQL, HTML, CSS, YAML, TOML, JSON, Markdown + 280 more | Signature Match only | replace_body, create_entity |
+| **Tier 3** | HTML, CSS, YAML, TOML, JSON, Markdown + 280 more | Signature Match only | replace_body, create_entity |
 
 ## Resolution Cascade
 
@@ -402,6 +402,24 @@ tracked finding with a battery anchor (see
   (`#[pyfunction]` never reached the graph before) with `#[pymodule]`
   added as a production root.
 
+## v0.9.1–v0.9.2 Highlights — CI reds closed + docs refresh
+
+- **Temporal traversal on aliased roots (v0.9.1).** Windows CI failed
+  `test_as_of_temporal_traversal` 3/3 runs (`got []`) while local +
+  Ubuntu passed: GitHub runners set `TEMP` with an 8.3 short component
+  (`C:\Users\RUNNER~1\…`), the walk spells paths as passed while
+  `INDEXED_ROOT` is filesystem-canonicalized, the lexical strip missed,
+  ids fell back to absolute form, and analyze retired all 3 fixture
+  concepts as orphans in the same run. `canonical_file_form` now
+  FS-canonicalizes once on a strip miss and retries (mismatch path only),
+  with verbatim-alias (Windows) and symlink-alias (unix) regression tests.
+- **Lint fully green.** The 951-error ruff backlog is cleaned to zero
+  against pinned `ruff@0.16.5` (fixture excludes, mechanical fixes,
+  per-site `noqa` where deliberate) — `cargo fmt` + clippy + ruff all pass.
+- **This refresh (v0.9.2).** Test counts 1119 → 1128 (369 Rust + 759
+  Python), query files 18 → 41 (one per Tier-1/2 language), Shell/SQL
+  de-duplicated out of the Tier-3 row (both ship `.scm` files in Tier 2).
+
 ## v0.7.18 Feature Highlights — the fossil-mcp port
 
 v0.7.3 through v0.7.18 ported the best of fossil-mcp's detector suite onto CodeRadar's graph
@@ -639,7 +657,7 @@ truth, scaling, dead-code retirement, configuration, and the MCP layer.
 
 ```
 core_indexer/              # Rust core
-  queries/                 # 18 .scm query files (one per language)
+  queries/                 # 41 .scm query files (one per Tier-1/2 language)
   src/
     extract/               # Tree-sitter: tagger (query cursor) + walker (hierarchy) + docstring + decorators
     update/                # Incremental diff + patch + WAL
@@ -676,7 +694,7 @@ py_agent/src/coderadar/    # Python layer
     visualizers/           # Mermaid + Graphviz (SCC cycle highlighting)
 
 docs/                      # Specifications + code review + performance roadmap
-tests/                     # 748 Python tests (E2E incl. dead-code/clones/scaffold/CFG/
+tests/                     # 759 Python tests (E2E incl. dead-code/clones/scaffold/CFG/
                            #   centrality/dead-branch/RTA goldens, mutation E2E, MCP,
                            #   framework resolvers, ingest parity, benchmarks)
   mcp/                     # Root resolution, background init, lifecycle, project_path

{coderadar_rs-0.9.0 → coderadar_rs-0.9.2}/README.md

@@ -1,4 +1,4 @@
-# CodeRadar v0.9.0
+# CodeRadar v0.9.2
 
 [![CI](https://github.com/opticsWolf/coderadar/actions/workflows/ci.yml/badge.svg)](https://github.com/opticsWolf/coderadar/actions/workflows/ci.yml)
 [![PyPI](https://img.shields.io/pypi/v/coderadar-rs?label=pypi)](https://pypi.org/project/coderadar-rs/)
@@ -55,7 +55,7 @@ Rust Core (ProjectedGraph, Tree-sitter 41-lang, Parallel Extraction,
 | Metric | Value |
 |--------|-------|
 | **Languages indexed** | 41 (12 Tier 1, 29 Tier 2, 330+ Tier 3) |
-| **Tests** | 1119 (368 Rust + 751 Python) |
+| **Tests** | 1128 (369 Rust + 759 Python) |
 | **MCP Tools** | 22 — 17 `codegraph_*` (explore, search, node, affected, query, search_similar, compute_embeddings, module_children, as_of, traverse, get_smells, dead_code, find_clones, find_scaffolding, reindex, update_file, set_project) + 5 `coderadar_*` (resolve, replace_body, update_signature, rename, create_entity) |
 | **Query surface** | Pest structural + Macrame agent traversals + vector search |
 | **Frameworks** | Django, Flask, FastAPI, Go, Actix, Express, Spring Boot, Laravel, ASP.NET, Rails, NestJS, Vue Router, React Router |
@@ -230,7 +230,7 @@ not assume the cwd is the project:
 |------|-----------|------------|----------|
 | **Tier 1** | Python, TypeScript, JavaScript, Rust, Go, Java, C, C++, Ruby, PHP, C#, Kotlin | Import → Signature → Framework | Full tool suite |
 | **Tier 2** | Swift, Scala, Lua, Elixir, Zig, R, Bash, Dart, Protobuf, Dockerfile, SQL, HCL, CMake, GraphQL, Erlang, Haskell, **Nix, Shell, Groovy, Perl, SystemVerilog, OCaml, Clojure, F#, Verilog, Julia, PowerShell, Emacs Lisp, Objective-C** | Import → Signature | replace_body, create_entity |
-| **Tier 3** | Shell, SQL, HTML, CSS, YAML, TOML, JSON, Markdown + 280 more | Signature Match only | replace_body, create_entity |
+| **Tier 3** | HTML, CSS, YAML, TOML, JSON, Markdown + 280 more | Signature Match only | replace_body, create_entity |
 
 ## Resolution Cascade
 
@@ -368,6 +368,24 @@ tracked finding with a battery anchor (see
   (`#[pyfunction]` never reached the graph before) with `#[pymodule]`
   added as a production root.
 
+## v0.9.1–v0.9.2 Highlights — CI reds closed + docs refresh
+
+- **Temporal traversal on aliased roots (v0.9.1).** Windows CI failed
+  `test_as_of_temporal_traversal` 3/3 runs (`got []`) while local +
+  Ubuntu passed: GitHub runners set `TEMP` with an 8.3 short component
+  (`C:\Users\RUNNER~1\…`), the walk spells paths as passed while
+  `INDEXED_ROOT` is filesystem-canonicalized, the lexical strip missed,
+  ids fell back to absolute form, and analyze retired all 3 fixture
+  concepts as orphans in the same run. `canonical_file_form` now
+  FS-canonicalizes once on a strip miss and retries (mismatch path only),
+  with verbatim-alias (Windows) and symlink-alias (unix) regression tests.
+- **Lint fully green.** The 951-error ruff backlog is cleaned to zero
+  against pinned `ruff@0.16.5` (fixture excludes, mechanical fixes,
+  per-site `noqa` where deliberate) — `cargo fmt` + clippy + ruff all pass.
+- **This refresh (v0.9.2).** Test counts 1119 → 1128 (369 Rust + 759
+  Python), query files 18 → 41 (one per Tier-1/2 language), Shell/SQL
+  de-duplicated out of the Tier-3 row (both ship `.scm` files in Tier 2).
+
 ## v0.7.18 Feature Highlights — the fossil-mcp port
 
 v0.7.3 through v0.7.18 ported the best of fossil-mcp's detector suite onto CodeRadar's graph
@@ -605,7 +623,7 @@ truth, scaling, dead-code retirement, configuration, and the MCP layer.
 
 ```
 core_indexer/              # Rust core
-  queries/                 # 18 .scm query files (one per language)
+  queries/                 # 41 .scm query files (one per Tier-1/2 language)
   src/
     extract/               # Tree-sitter: tagger (query cursor) + walker (hierarchy) + docstring + decorators
     update/                # Incremental diff + patch + WAL
@@ -642,7 +660,7 @@ py_agent/src/coderadar/    # Python layer
     visualizers/           # Mermaid + Graphviz (SCC cycle highlighting)
 
 docs/                      # Specifications + code review + performance roadmap
-tests/                     # 748 Python tests (E2E incl. dead-code/clones/scaffold/CFG/
+tests/                     # 759 Python tests (E2E incl. dead-code/clones/scaffold/CFG/
                            #   centrality/dead-branch/RTA goldens, mutation E2E, MCP,
                            #   framework resolvers, ingest parity, benchmarks)
   mcp/                     # Root resolution, background init, lifecycle, project_path

{coderadar_rs-0.9.0 → coderadar_rs-0.9.2}/core_indexer/src/fs/git.rs

@@ -105,9 +105,7 @@ pub fn is_worktree_clean(repo_path: &str) -> Result<bool, GitError> {
         .recurse_untracked_dirs(true)
         .include_ignored(false)
         .recurse_ignored_dirs(false);
-    let statuses = repo
-        .statuses(Some(&mut opts))
-        .map_err(GitError::Status)?;
+    let statuses = repo.statuses(Some(&mut opts)).map_err(GitError::Status)?;
     Ok(statuses.is_empty())
 }
 
@@ -135,12 +133,8 @@ mod tests {
         std::fs::write(dir.join(".gitignore"), ".coderadar/\n*.log\n").unwrap();
         let repo = git2::Repository::init(dir).unwrap();
         let mut index = repo.index().unwrap();
-        index
-            .add_path(std::path::Path::new("a.txt"))
-            .unwrap();
-        index
-            .add_path(std::path::Path::new(".gitignore"))
-            .unwrap();
+        index.add_path(std::path::Path::new("a.txt")).unwrap();
+        index.add_path(std::path::Path::new(".gitignore")).unwrap();
         index.write().unwrap();
         let tree_id = index.write_tree().unwrap();
         let tree = repo.find_tree(tree_id).unwrap();
@@ -156,7 +150,10 @@ mod tests {
         let dir = tempfile::tempdir().unwrap();
         init_commit_repo(dir.path());
         let root = dir.path().to_str().unwrap();
-        assert!(is_worktree_clean(root).unwrap(), "committed base must be clean");
+        assert!(
+            is_worktree_clean(root).unwrap(),
+            "committed base must be clean"
+        );
 
         std::fs::write(dir.path().join("top.log"), "x\n").unwrap();
         std::fs::create_dir_all(dir.path().join(".coderadar/store")).unwrap();
@@ -187,9 +184,7 @@ mod revision_tests {
         std::fs::write(dir.join(name), "v\n").unwrap();
         let repo = Repository::open(dir).unwrap();
         let mut index = repo.index().unwrap();
-        index
-            .add_path(std::path::Path::new(name))
-            .unwrap();
+        index.add_path(std::path::Path::new(name)).unwrap();
         index.write().unwrap();
         let tree_id = index.write_tree().unwrap();
         let tree = repo.find_tree(tree_id).unwrap();

{coderadar_rs-0.9.0 → coderadar_rs-0.9.2}/core_indexer/src/graph/module_resolution.rs

@@ -46,23 +46,48 @@ fn clean_lexical(p: &std::path::Path) -> std::path::PathBuf {
 /// yield the same ids. Paths outside the indexed root keep absolute form
 /// (no worse than today; they never matched anything anyway).
 pub(crate) fn canonical_file_form(path: &str) -> String {
-    let root = crate::indexed_root();
+    canonical_file_form_with_root(path, &crate::indexed_root())
+}
+
+/// Inner funnel with the root injectable (unit tests must not touch the
+/// process-global `INDEXED_ROOT`; parallel tests share it).
+fn canonical_file_form_with_root(path: &str, root: &std::path::Path) -> String {
     let cwd = std::env::current_dir().unwrap_or_else(|_| std::path::PathBuf::from("."));
     let p = std::path::Path::new(path);
     let abs = if p.is_absolute() {
         clean_lexical(p)
     } else {
         let via_cwd = clean_lexical(&cwd.join(p));
-        if via_cwd.starts_with(&root) {
+        if via_cwd.starts_with(root) {
             via_cwd
         } else {
             clean_lexical(&root.join(p))
         }
     };
-    let rel = match abs.strip_prefix(&root) {
+    let rel = match abs.strip_prefix(root) {
         Ok(r) if !r.as_os_str().is_empty() => r.to_path_buf(),
-        // Outside the root (or the root itself): absolute form, as before.
-        _ => return normalize_path_str(path),
+        // Aliased root: the walk spells the path as passed (8.3 short
+        // names like C:\Users\RUNNER~1\… on CI, symlinks, verbatim
+        // `\\?\` form, on-disk case) while INDEXED_ROOT is
+        // filesystem-canonicalized — a lexical strip can never match
+        // those. Resolve through the FS once (mismatch path only, never
+        // the hot path) and retry before falling back to absolute form,
+        // which `retire_noncanonical_concepts` would close as an orphan
+        // (Windows CI's `test_as_of_temporal_traversal`: 3 concepts
+        // retired, as_of reads []).
+        _ => match std::fs::canonicalize(&abs) {
+            Ok(canon) => {
+                let canon = crate::strip_verbatim_prefix(canon);
+                match canon.strip_prefix(root) {
+                    Ok(r) if !r.as_os_str().is_empty() => r.to_path_buf(),
+                    // Outside the root (or the root itself): absolute form.
+                    _ => return normalize_path_str(path),
+                }
+            }
+            // Unresolvable (update_file mints ids for files that may not
+            // exist yet): absolute form, as before.
+            _ => return normalize_path_str(path),
+        },
     };
     let sep = std::path::MAIN_SEPARATOR;
     let joined = rel
@@ -389,6 +414,50 @@ mod canonical_form_tests {
         assert_eq!(canonical_file_form("some/dir/x.py"), c);
     }
 
+    /// Aliased root (Windows): the walk may spell a path verbatim
+    /// (`\\\\?\\C:\\…`) while INDEXED_ROOT is the stripped form — same
+    /// file, lexical mismatch. Models CI's 8.3 TEMP (`RUNNER~1`), which
+    /// retired all 3 concepts of `test_as_of_temporal_traversal`'s fixture
+    /// as "non-canonical" and made as_of read [].
+    #[cfg(windows)]
+    #[test]
+    fn verbatim_aliased_path_still_mints_relative_id() {
+        let base = std::env::temp_dir().join(format!("cr_alias_{}", std::process::id()));
+        let real = base.join("real");
+        std::fs::create_dir_all(&real).unwrap();
+        std::fs::write(real.join("a.py"), "x = 1\n").unwrap();
+        // What INDEXED_ROOT holds: canonicalized, verbatim stripped.
+        let root = crate::strip_verbatim_prefix(std::fs::canonicalize(&real).unwrap());
+        // What the walk may yield for the same file: verbatim spelling.
+        let verbatim = std::path::PathBuf::from(format!(r"\\?\{}", real.join("a.py").display()));
+        let got = canonical_file_form_with_root(&verbatim.to_string_lossy(), &root);
+        assert!(
+            got.starts_with(r".\"),
+            "aliased path must mint a canonical relative id, got {got:?}"
+        );
+        std::fs::remove_dir_all(&base).ok();
+    }
+
+    /// Aliased root (POSIX): same mismatch class via symlink — lexical
+    /// strip of the unresolved spelling fails, the FS fallback resolves.
+    #[cfg(unix)]
+    #[test]
+    fn symlinked_path_still_mints_relative_id() {
+        let base = std::env::temp_dir().join(format!("cr_alias_{}", std::process::id()));
+        let real = base.join("real");
+        std::fs::create_dir_all(&real).unwrap();
+        std::fs::write(real.join("a.py"), "x = 1\n").unwrap();
+        std::os::unix::fs::symlink(&real, base.join("alias")).unwrap();
+        let root = std::fs::canonicalize(&real).unwrap();
+        let via_alias = base.join("alias").join("a.py");
+        let got = canonical_file_form_with_root(&via_alias.to_string_lossy(), &root);
+        assert!(
+            got.starts_with("./"),
+            "aliased path must mint a canonical relative id, got {got:?}"
+        );
+        std::fs::remove_dir_all(&base).ok();
+    }
+
     #[test]
     fn canonical_head_predicate_sorts_all_three_historical_forms() {
         assert!(is_canonical_file_head(r".\tests\x.py"));

{coderadar_rs-0.9.0 → coderadar_rs-0.9.2}/core_indexer/src/graph/projection_ops.rs

@@ -63,12 +63,9 @@ impl CodeGraph {
             // too, or this whole branch silently keeps everything. Collect
             // across ALL entity maps (functions/classes alone left
             // constants, aliases, imports and the module itself behind).
-            let canon =
-                super::module_resolution::canonical_file_form(file_path);
+            let canon = super::module_resolution::canonical_file_form(file_path);
             let prefixes = [format!("{}::", lookup), format!("{}::", canon)];
-            let matches = |id: &str| {
-                prefixes.iter().any(|p| id.starts_with(p.as_str()))
-            };
+            let matches = |id: &str| prefixes.iter().any(|p| id.starts_with(p.as_str()));
             for (func_id, _) in projection.functions.iter() {
                 if matches(func_id) && !removed.contains(func_id) {
                     removed.insert(func_id.clone());

{coderadar_rs-0.9.0 → coderadar_rs-0.9.2}/core_indexer/src/lib.rs

@@ -112,7 +112,7 @@ pub(crate) fn indexed_root() -> std::path::PathBuf {
 
 /// `std::fs::canonicalize` returns verbatim paths on Windows (`\\?\D:\…`),
 /// which defeat `starts_with`/`strip_prefix` against regular paths.
-fn strip_verbatim_prefix(p: std::path::PathBuf) -> std::path::PathBuf {
+pub(crate) fn strip_verbatim_prefix(p: std::path::PathBuf) -> std::path::PathBuf {
     let s = p.to_string_lossy();
     if let Some(rest) = s.strip_prefix(r"\\?\UNC\") {
         return std::path::PathBuf::from(format!(r"\\{rest}"));
@@ -2039,8 +2039,8 @@ fn search_entities(
     with_graph(|_graph, snap| {
         let tokens = search_tokens(query);
         let mut results: Vec<(usize, PyObject)> = Vec::new(); // (score, dict)
-        // Issue 5 (schema enums): a garbage kind used to match nothing and
-        // read as "no results". Refuse loudly like Strictness::parse.
+                                                              // Issue 5 (schema enums): a garbage kind used to match nothing and
+                                                              // read as "no results". Refuse loudly like Strictness::parse.
         const KNOWN_KINDS: &[&str] = &[
             "function",
             "class",
@@ -3529,10 +3529,7 @@ mod tests {
         assert_eq!(unresolved_ref_kind("external::len"), "builtin");
         assert_eq!(unresolved_ref_kind("external::isinstance"), "builtin");
         assert_eq!(unresolved_ref_kind("external::combine"), "external");
-        assert_eq!(
-            unresolved_ref_kind("external::requests.get"),
-            "external"
-        );
+        assert_eq!(unresolved_ref_kind("external::requests.get"), "external");
         assert_eq!(unresolved_ref_kind("Date::now"), "unresolved");
         assert_eq!(unresolved_ref_kind("C::run"), "unresolved");
     }