dirsql 0.4.13__tar.gz → 0.4.14__tar.gz

{dirsql-0.4.13 → dirsql-0.4.14}/Cargo.lock

@@ -500,7 +500,7 @@ dependencies = [
 
 [[package]]
 name = "dirsql-py-ext"
-version = "0.4.13"
+version = "0.4.14"
 dependencies = [
  "dirsql",
  "pyo3",

{dirsql-0.4.13 → dirsql-0.4.14}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dirsql
-Version: 0.4.13
+Version: 0.4.14
 Requires-Dist: tomli>=2 ; python_full_version < '3.11'
 Summary: Ephemeral SQL index over a local directory
 Keywords: sql,filesystem,directory,sqlite,index

{dirsql-0.4.13 → dirsql-0.4.14}/packages/python/Cargo.toml

@@ -4,7 +4,7 @@ name = "dirsql-py-ext"
 # pypi/maturin handler can rewrite it via `write-version` before
 # `maturin build`. `pyproject.toml` declares `dynamic = ["version"]`
 # and maturin reads this field. Mirrors `packages/rust/Cargo.toml`.
-version = "0.4.13"
+version = "0.4.14"
 # abi3 note: the `abi3-py310` pyo3 feature (below) builds ONE stable-ABI
 # `cp310-abi3` wheel per platform that loads on every CPython >= 3.10
 # (matching `pyproject.toml`'s `requires-python = ">=3.10"`), instead of

dirsql-0.4.14/packages/python/e2e-attestations/claude-772-extension-without-config.json

@@ -0,0 +1,7 @@
+{
+  "command": "uv run python -m pytest tests/e2e/ -x -q",
+  "ran_at": 1785877241,
+  "exit_code": 0,
+  "commit": "83f767a7d90cefe1aa6ab136e36a739cfb5dc425",
+  "branch": "claude/772-extension-without-config"
+}

dirsql-0.4.14/packages/rust/changelog.d/2026-08-04-configless-extension.md

@@ -0,0 +1 @@
+**Fixed** — `--extension` is now honored when no `-c`/`--config` is given. The configless path (no config, no `--include-default`) short-circuited before extensions were applied, so `dirsql query "SELECT vec_version()" --extension <path>::<entry>` reported `no such function: vec_version` as though the flag had not been passed — and, worse, `--extension /nonexistent.so` exited **0**, silently succeeding on a load that never happened. Both now behave: the extension loads and a bad path fails with `failed to build the index: failed to load extension '<path>'` and exit 1, matching the with-config path. This mattered because loading an extension to query a path-table (`SELECT * FROM './'`) needs no config at all, and because the npm/pip launchers pass resolved `--extension` flags — a launcher-resolved extension was dropped whenever the user had no config. #772.

{dirsql-0.4.13 → dirsql-0.4.14}/packages/rust/src/bin/dirsql.rs

@@ -581,6 +581,15 @@ fn load_configless_state(cfg: &ConfigArgs, path_table_parser: Option<String>) ->
     };
 
     let mut builder = cfg.apply_index_flags(DirSQL::builder().root(root));
+    // `--extension` applies here too (#772). Without this the flag was
+    // silently ignored whenever no `-c` was given: `dirsql query "SELECT
+    // vec_version()" --extension <path>` reported `no such function` rather
+    // than loading anything, and `--extension /nonexistent.so` exited 0. A
+    // path-table query over an extension-provided function is a legitimate
+    // configless use, and a bad path must still fail loudly.
+    if !cfg.extension.is_empty() {
+        builder = builder.extensions(parse_extension_specs(&cfg.extension));
+    }
     if let Some(command) = path_table_parser {
         builder = builder.path_table_parser(command);
     }
@@ -932,6 +941,31 @@ mod tests {
         );
     }
 
+    #[test]
+    fn configless_state_still_carries_extension_specs() {
+        // `--extension` with no `-c` took the configless path, which never
+        // applied the specs: the flag was silently ignored and a bad path
+        // exited 0 (#772). Parsing is the seam a unit test can reach; the
+        // load itself needs a real SQLite handle and is covered at the
+        // integration tier.
+        let cfg = ConfigArgs {
+            config: Vec::new(),
+            include_default: false,
+            extension: vec!["/ext/vec0.so::sqlite3_vec_init".to_string()],
+            persist: None,
+            no_ignore: false,
+        };
+        assert!(cfg.config.is_empty(), "this is the configless shape");
+        assert!(
+            !cfg.extension.is_empty(),
+            "and it still carries an extension"
+        );
+        let specs = parse_extension_specs(&cfg.extension);
+        assert_eq!(specs.len(), 1);
+        assert_eq!(specs[0].path, PathBuf::from("/ext/vec0.so"));
+        assert_eq!(specs[0].entrypoint.as_deref(), Some("sqlite3_vec_init"));
+    }
+
     #[test]
     fn parse_extension_specs_handles_bare_path_and_entrypoint() {
         let specs = vec![