codingest 0.2.23__tar.gz → 0.2.25__tar.gz

{codingest-0.2.23 → codingest-0.2.25}/Cargo.lock

@@ -302,7 +302,7 @@ dependencies = [
 
 [[package]]
 name = "codingest"
-version = "0.2.23"
+version = "0.2.25"
 dependencies = [
  "aho-corasick",
  "base64 0.22.1",
@@ -338,7 +338,7 @@ dependencies = [
 
 [[package]]
 name = "codingest-cli"
-version = "0.2.23"
+version = "0.2.25"
 dependencies = [
  "anyhow",
  "clap",
@@ -350,7 +350,7 @@ dependencies = [
 
 [[package]]
 name = "codingest-mcp"
-version = "0.2.23"
+version = "0.2.25"
 dependencies = [
  "anyhow",
  "codingest",
@@ -361,7 +361,7 @@ dependencies = [
 
 [[package]]
 name = "codingest-py"
-version = "0.2.23"
+version = "0.2.25"
 dependencies = [
  "codingest",
  "codingest-cli",
@@ -1368,9 +1368,9 @@ dependencies = [
 
 [[package]]
 name = "kglite"
-version = "0.17.7"
+version = "0.17.10"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e59bdace8eb48e7db04cdc5c066956f494bbcc6fdddb661b4f09764a49428c4e"
+checksum = "e14ff39495529815519e61ed78e7cac6aaaa37b5641b79e78693bf7ae0befa98"
 dependencies = [
  "bzip2",
  "chrono",
@@ -1386,6 +1386,7 @@ dependencies = [
  "memmap2",
  "petgraph",
  "postcard",
+ "pulldown-cmark",
  "rayon",
  "regex",
  "rstar 0.13.0",
@@ -1404,11 +1405,12 @@ dependencies = [
 
 [[package]]
 name = "kglite-mcp-server"
-version = "0.17.7"
+version = "0.17.10"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "57a37838e2d50d16363c4dc28d7d9a791d9f7edd374660ddbece6578f5c7601b"
+checksum = "4fb67310c835576a085fb0f35c6a7e70bfbe2bde663809d9b1d1bbcc55986b00"
 dependencies = [
  "anyhow",
+ "base64 0.22.1",
  "bytes",
  "chrono",
  "clap",
@@ -1510,9 +1512,9 @@ dependencies = [
 
 [[package]]
 name = "mcp-methods"
-version = "0.4.11"
+version = "0.4.12"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "31c5a17334897b67053e843235be8c66610d0c6684c845bea30e70ecff9dd204"
+checksum = "5bcc5fdac2ffd82893fbc640b569785bdc63ead7f3c6c3d695685b692ddf2d1a"
 dependencies = [
  "anyhow",
  "clap",
@@ -1742,6 +1744,17 @@ dependencies = [
  "cc",
 ]
 
+[[package]]
+name = "pulldown-cmark"
+version = "0.13.4"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "e9f068eba8e7071c5f9511831b44f32c740d5adf574e990f946ddb53db2f314e"
+dependencies = [
+ "bitflags",
+ "memchr",
+ "unicase",
+]
+
 [[package]]
 name = "pyo3"
 version = "0.29.0"
@@ -2792,6 +2805,12 @@ version = "1.20.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "b6f5e870be6c3b371b77fe0ee0bafb859fa4964b4404c27de1d380043c4dda20"
 
+[[package]]
+name = "unicase"
+version = "2.9.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "dbc4bc3a9f746d862c45cb89d705aa10f187bb96c76001afab07a0d35ce60142"
+
 [[package]]
 name = "unicode-ident"
 version = "1.0.24"

{codingest-0.2.23 → codingest-0.2.25}/Cargo.toml

@@ -3,7 +3,7 @@ members = ["crates/codingest", "crates/codingest-cli", "crates/codingest-mcp", "
 resolver = "2"
 
 [workspace.package]
-version = "0.2.23"
+version = "0.2.25"
 edition = "2021"
 rust-version = "1.88"
 license = "MIT"
@@ -12,7 +12,83 @@ repository = "https://github.com/kkollsga/codingest"
 [workspace.dependencies]
 base64 = "0.22"
 sha2 = "0.11"
-# kglite engine + MCP server — imported as crates.io libraries. The 0.17.7
+# kglite engine + MCP server — imported as crates.io libraries. The 0.17.10
+# floor is a pure Cypher-executor fix release: `cargo semver-checks` reports no
+# API change across 196 checks, and codingest needs no source edit for it. It
+# moves anyway because both fixes were SILENT wrong answers on the query
+# surface `codingest query` and the embedded `codingest-mcp` server expose —
+# the class of defect a floor exists to keep users off:
+#   * a non-aggregating `WITH` is a scope barrier again. It projected the row's
+#     values but left the node / edge / path *identity* bindings in place, so a
+#     name the projection dropped stayed silently bound: a later `MATCH` on
+#     that name anchored the stale node instead of scanning, `count(*)` behind
+#     the barrier counted 1 instead of the label, and `CREATE` / `SET` /
+#     `MERGE` / `FOREACH` after such a `WITH` acted on nothing with no error.
+#     `WITH n AS m` now frees `n`; `WITH *` and a projected `WITH n` keep their
+#     bindings as before. An *aggregating* `WITH` rebuilds its rows from the
+#     projection and was correct throughout, which is why this survived.
+#   * `*` written beside another projection item now expands. Alone it always
+#     did; alongside anything else it was projected as an ordinary expression —
+#     a column literally named `*` holding `1` — and every value-carrying name
+#     in scope was dropped, so `UNWIND [1] AS a WITH *, a + 1 AS b RETURN a, b`
+#     answered `{a: null, b: 2}`. Two spellings lost whole rows: `WITH *,
+#     count(*) AS c` grouped by that constant, and `WITH DISTINCT *, 1 AS k`
+#     deduplicated every row down to one. `RETURN *` now also lists a path
+#     variable, and where `*` and an explicit item would project the same name
+#     the explicit item wins.
+#   Neither reaches codingest's own Cypher. The builder writes graphs and never
+#   queries them, and the shipped code-review recipe catalogue and skill
+#   examples contain no `WITH` clause and no `*` projection at all
+#   (grep-verified across `assets/code_review/recipes.json`, both `skills/`
+#   trees, `query.rs` and the CLI/MCP tests). Optimized and unoptimized plans
+#   agreed on every old wrong answer, so these were invisible to kglite's
+#   optimizer-differential corpus and are now pinned there by absolute goldens.
+#   Zero source changes here; every frozen golden byte-identical.
+# The preceding 0.17.9
+# floor covers two same-day releases, 0.17.8 (the Obsidian vault format) and
+# 0.17.9 (the vault structure profile). Both land almost entirely on
+# `dialect="obsidian"`, which codingest never selects — but they rebuilt the
+# `kglite::okf` *type* surface underneath our docs pass, so this is the first
+# kglite bump since 0.16.x that needs a source change:
+#   * `okf::BuildOptions` gained `profile` (a `Profile` the parser reads for
+#     its conventions) and lost `embed` (a field the loader never read — it
+#     embedded nothing in any dialect, so removing it changes no behaviour),
+#     and `okf::walk::DiscoveredFile` gained `size` / `mtime`. Our docs pass
+#     built both as exhaustive struct literals. `BuildOptions` is now
+#     constructed through `BuildOptions::for_dialect(Dialect::Okf)` with only
+#     the two fields we deliberately differ on overridden — a `Profile` cannot
+#     be a struct literal outside kglite any more, and `for_dialect` is the
+#     constructor that keeps a future field at its dialect default instead of
+#     failing the build a third time. `size`/`mtime` are `0`/`None`: they feed
+#     only `okf::fingerprint` and the vault's attachment nodes, and
+#     `parse_concepts` — the one entry point we call — reads neither.
+#   * The rest of the breaking set does not reach us: `okf::build` now returns
+#     `BuildOutput { graph, report }`, `links::extract_links` and
+#     `walk::discover` take the options/profile instead of a `Dialect` /
+#     `skip_dirs` slice, `Dialect` gained `Obsidian`, `Link` lost
+#     `is_wikilink`, and `ExportOptions` / `ExportReport` gained
+#     `edge_tables` / `warnings`. codingest calls none of them (grep: zero
+#     hits outside `docs/mod.rs`) — we use `okf::parse_concepts` and nothing
+#     else, and we read only `concept_id` / `file_path` / `title` / `body` /
+#     `props` off each `ConceptDoc`, never its links.
+#   * Behaviour under `Dialect::Okf` is the old behaviour by construction:
+#     `Profile::default()` is the OKF-bundle profile with every vault feature
+#     (wikilinks, embeds, inline tags, frontmatter edges, aliases, folder
+#     notes, attachments, `structure:`) off, which is what the parser hardcoded
+#     before. The one 0.17.8 parser fix that is dialect-independent and lands
+#     on a value we store is the heading rule: a line starting with `#` is a
+#     heading only when one to six `#` are followed by a space, a tab, or the
+#     line end, so an inline-tag line like `#project see …` is no longer read
+#     as an `# H1`. That feeds our `:Doc` title fallback. The 31 frozen
+#     goldens are the gate on it, and they are byte-identical across this move.
+#   * kglite-mcp-server: `--vault` mode and the vault-authoring skill are new
+#     surface we do not register, and the bundled skill set is now gated by
+#     graph shape — the four code-graph skills bundle only where the graph
+#     carries `Function` / `Class`, which is exactly a codingest graph, so the
+#     skills an agent reaches through `codingest-mcp` are unchanged while a
+#     document deployment stops paying for ours. Our producer layer
+#     (`ServerExtensions::with_skills` / `with_recipes`) is untouched.
+# The preceding 0.17.7
 # floor adds the producer-level methodology hook codingest asked for on
 # 2026-09-16: `ServerExtensions::with_skills` / `with_recipes` let the binary
 # embedding the server register its own skill layer and Cypher recipe
@@ -546,8 +622,8 @@ sha2 = "0.11"
 # and no disk storage. Neither 0.15.7 nor 0.15.8 changes the `kglite::api` we
 # call, graph output, property encoding, or `.kgl` serialization.
 # Keep both libraries aligned so the graph and server share one engine release.
-kglite = { version = "0.17.7", default-features = false }
-kglite-mcp-server = { version = "0.17.7", default-features = false }
+kglite = { version = "0.17.10", default-features = false }
+kglite-mcp-server = { version = "0.17.10", default-features = false }
 
 # Dependency debuginfo capped at line tables (kglite coordination, 2026-08-31,
 # after unbounded debug trees filled the build-cache disk to zero): a

{codingest-0.2.23 → codingest-0.2.25}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codingest
-Version: 0.2.23
+Version: 0.2.25
 Classifier: Development Status :: 3 - Alpha
 Classifier: Intended Audience :: Developers
 Classifier: Programming Language :: Python :: 3
@@ -12,7 +12,7 @@ Classifier: Programming Language :: Python :: 3.14
 Classifier: Programming Language :: Rust
 Classifier: Topic :: Software Development
 Classifier: Topic :: Software Development :: Libraries
-Requires-Dist: kglite>=0.17.7,<0.18
+Requires-Dist: kglite>=0.17.10,<0.18
 License-File: LICENSE
 Summary: Build fast, queryable code graphs for AI agents, MCP code review, and local or GitHub repository analysis.
 Keywords: ai-agents,mcp,code-review,knowledge-graph,code-analysis,github,tree-sitter,cypher
@@ -194,7 +194,7 @@ codingest skill install        # code-review Agent Skill
 ```
 
 The wheel bundles every grammar plus the `codingest` and `codingest-mcp`
-commands. KGLite ≥0.17.7 is installed automatically as the query/storage
+commands. KGLite ≥0.17.10 is installed automatically as the query/storage
 engine; its MCP server and the transitive `mcp-methods` framework power the
 builder-aware Codingest server. Nothing else needs to be installed.
 
@@ -207,7 +207,7 @@ Python prerequisites.
 ```toml
 [dependencies]
 codingest = "0.2"
-kglite = "0.17.7"
+kglite = "0.17.10"
 ```
 
 ```rust
@@ -279,7 +279,7 @@ diagnostic.
 ## Dependency policy
 
 `kglite` and `kglite-mcp-server` use matching crates.io requirements with a
-0.17.7 minimum and a shared lockfile. This keeps the builder, persistence
+0.17.10 minimum and a shared lockfile. This keeps the builder, persistence
 handoff, and embedded MCP server on one compatible engine patch line.
 
 ## Parity with the (now-removed) in-tree component

{codingest-0.2.23 → codingest-0.2.25}/README.md

@@ -167,7 +167,7 @@ codingest skill install        # code-review Agent Skill
 ```
 
 The wheel bundles every grammar plus the `codingest` and `codingest-mcp`
-commands. KGLite ≥0.17.7 is installed automatically as the query/storage
+commands. KGLite ≥0.17.10 is installed automatically as the query/storage
 engine; its MCP server and the transitive `mcp-methods` framework power the
 builder-aware Codingest server. Nothing else needs to be installed.
 
@@ -180,7 +180,7 @@ Python prerequisites.
 ```toml
 [dependencies]
 codingest = "0.2"
-kglite = "0.17.7"
+kglite = "0.17.10"
 ```
 
 ```rust
@@ -252,7 +252,7 @@ diagnostic.
 ## Dependency policy
 
 `kglite` and `kglite-mcp-server` use matching crates.io requirements with a
-0.17.7 minimum and a shared lockfile. This keeps the builder, persistence
+0.17.10 minimum and a shared lockfile. This keeps the builder, persistence
 handoff, and embedded MCP server on one compatible engine patch line.
 
 ## Parity with the (now-removed) in-tree component

{codingest-0.2.23 → codingest-0.2.25}/crates/codingest/Cargo.toml

@@ -10,7 +10,7 @@ keywords = ["knowledge-graph", "code-analysis", "tree-sitter", "cypher", "kglite
 categories = ["development-tools", "parser-implementations"]
 
 # Requires the kglite 0.15 engine API (`kglite::api::code_entities`); the
-# workspace pins the current engine floor (0.17.7).
+# workspace pins the current engine floor (0.17.10).
 
 [lib]
 name = "codingest"

{codingest-0.2.23 → codingest-0.2.25}/crates/codingest/src/docs/mod.rs

@@ -249,20 +249,29 @@ fn discover_and_parse(root: &Path) -> Result<Vec<DocEntry>, String> {
     // computes titles from frontmatter / first heading, honors `kg_skip`,
     // flattens frontmatter). We bypass `okf::walk` so `.rst` shares one traversal and so
     // `index.md` is kept (the docs pass builds no Folder hierarchy).
+    // `for_dialect` rather than a struct literal: it carries the dialect's
+    // `Profile` (the conventions the parser reads — id scheme, reserved keys,
+    // wikilinks, collection encoding), which a literal cannot construct from
+    // outside kglite and which a bare `dialect:` assignment would leave at the
+    // wrong dialect's defaults. Only the two fields the docs pass deliberately
+    // differs on are overridden: every `.md` is admitted (a repo's docs carry
+    // no frontmatter) and the body is stored (the `:Doc` node's prose).
     let md_opts = okf::BuildOptions {
-        dialect: okf::Dialect::Okf,
         require_frontmatter: false,
-        respect_skip: true,
-        skip_dirs: Vec::new(),
         with_body: true,
-        embed: false,
+        ..okf::BuildOptions::for_dialect(okf::Dialect::Okf)
     };
+    // `size` / `mtime` feed only `okf::fingerprint` and the attachment nodes,
+    // neither of which the docs pass reaches — `parse_concepts` reads nothing
+    // but the two paths — so the walk does not stat a second time to fill them.
     let md_files: Vec<okf::walk::DiscoveredFile> = found
         .iter()
         .filter(|d| d.format == DocFormat::Markdown)
         .map(|d| okf::walk::DiscoveredFile {
             rel_path: d.rel_path.clone(),
             abs_path: d.abs_path.clone(),
+            size: 0,
+            mtime: None,
         })
         .collect();
     let mut docs: Vec<DocEntry> = okf::parse_concepts(&md_files, &md_opts)

{codingest-0.2.23 → codingest-0.2.25}/crates/codingest-cli/Cargo.toml

@@ -14,7 +14,7 @@ name = "codingest"
 path = "src/main.rs"
 
 [dependencies]
-codingest = { version = "0.2.23", path = "../codingest" }
+codingest = { version = "0.2.25", path = "../codingest" }
 kglite = { workspace = true }
 anyhow = "1.0.104"
 clap = { version = "4.6.4", features = ["derive"] }

{codingest-0.2.23 → codingest-0.2.25}/crates/codingest-cli/skills/codingest-code-review/SKILL.md

@@ -14,7 +14,7 @@ git diff, source reading, and literal-text search; it does not replace them.
 > codingest` also provides the builder-aware `codingest-mcp` server and a
 > compatible graph engine. Rust-only environments can alternatively use Cargo.
 > Examples that invoke the external `kglite` CLI require a compatible
-> `kglite>=0.17.7,<0.18`; verify `kglite --version` before using its
+> `kglite>=0.17.10,<0.18`; verify `kglite --version` before using its
 > agent-response features.
 
 Inspect the diff and repository guidance first. Identify changed symbols and

{codingest-0.2.23 → codingest-0.2.25}/crates/codingest-cli/skills/codingest-code-review/references/mcp-upgrade.md

@@ -54,7 +54,7 @@ preconditions are still current.
 ## External CLI agent output is optional
 
 The examples below require an external `kglite` CLI satisfying
-`>=0.17.7,<0.18`:
+`>=0.17.10,<0.18`:
 
 ```console
 kglite --version

{codingest-0.2.23 → codingest-0.2.25}/crates/codingest-mcp/Cargo.toml

@@ -14,7 +14,7 @@ name = "codingest-mcp"
 path = "src/main.rs"
 
 [dependencies]
-codingest = { version = "0.2.23", path = "../codingest" }
+codingest = { version = "0.2.25", path = "../codingest" }
 kglite-mcp-server = { workspace = true }
 anyhow = "1.0.104"
 

{codingest-0.2.23 → codingest-0.2.25}/crates/codingest-mcp/tests/response_contract.rs

@@ -802,7 +802,7 @@ fn graph_overview_schema_refreshes_after_root_switch() {
 // ── Producer methodology (kglite 0.17.7 `with_skills` / `with_recipes`) ────
 //
 // The fixture manifest never mentions `skills:`, which is the shape every
-// codingest-mcp deployment ships in. Under kglite 0.17.7 that means the
+// codingest-mcp deployment ships in. Under kglite 0.17.10 that means the
 // server serves kglite's bundled methodology plus codingest's own layer —
 // the `code_review` skill and the `code_review/*` recipe catalogue — in every
 // mode, including this manifest-driven workspace boot.

{codingest-0.2.23 → codingest-0.2.25}/crates/codingest-py/Cargo.toml

@@ -42,7 +42,7 @@ doctest = false
 pyo3 = { version = "0.29" }
 # The engine-free builder (parsers, call/type/route edges, rev merge, repo
 # clone, manifest reader). Docs pass gated by this crate's `docs` feature.
-codingest = { version = "0.2.23", path = "../codingest", default-features = false }
+codingest = { version = "0.2.25", path = "../codingest", default-features = false }
 # `.kgl` persistence (`kglite::api::io::save_graph`) for the serialize half of
 # the build-then-load handoff, plus the `DirGraph` type the builder returns.
 kglite = { workspace = true }
@@ -53,9 +53,9 @@ tempfile = "3.27.0"
 # through `_run_cli`, so `pip install codingest` also installs the `codingest`
 # command without a second wheel or a duplicated builder build. Mirrors
 # kglite-py's `kglite-cli` bundling.
-codingest-cli = { version = "0.2.23", path = "../codingest-cli" }
+codingest-cli = { version = "0.2.25", path = "../codingest-cli" }
 # Source-building MCP composition. The graph server and generic MCP lifecycle
 # remain owned by its existing kglite-mcp-server -> mcp-methods Rust dependency
 # chain; linking this thin library into the extension gives pip users the same
 # builder-aware server as the standalone `codingest-mcp` binary.
-codingest-mcp = { version = "0.2.23", path = "../codingest-mcp" }
+codingest-mcp = { version = "0.2.25", path = "../codingest-mcp" }

{codingest-0.2.23 → codingest-0.2.25}/crates/codingest-py/src/lib.rs

@@ -86,7 +86,7 @@ fn load_via_kglite(py: Python<'_>, path: &Path) -> PyResult<Py<PyAny>> {
         PyRuntimeError::new_err(format!(
             "codingest.build() returns a kglite.KnowledgeGraph, but importing \
              `kglite` failed ({e}). Install it: `pip install \
-             'kglite>=0.17.7,<0.18'`."
+             'kglite>=0.17.10,<0.18'`."
         ))
     })?;
     let graph = kglite.call_method1("load", (path.to_string_lossy().as_ref(),))?;

{codingest-0.2.23 → codingest-0.2.25}/pyproject.toml

@@ -42,9 +42,9 @@ dynamic = ["version"]
 # minor releases (e.g. the v3->v4 break, which refuses old files outright), so
 # pip must never be free to pair our writer with a reader from a different
 # minor. This range therefore mirrors the Cargo semver range for the pin above
-# (`version = "0.17.7"` == `>=0.17.7,<0.18.0`) exactly — bump both together.
+# (`version = "0.17.10"` == `>=0.17.10,<0.18.0`) exactly — bump both together.
 dependencies = [
-    "kglite>=0.17.7,<0.18",
+    "kglite>=0.17.10,<0.18",
 ]
 
 [project.urls]