dirsql 0.3.90__tar.gz → 0.3.91__tar.gz

{dirsql-0.3.90 → dirsql-0.3.91}/Cargo.lock

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

{dirsql-0.3.90 → dirsql-0.3.91}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dirsql
-Version: 0.3.90
+Version: 0.3.91
 Summary: Ephemeral SQL index over a local directory
 Keywords: sql,filesystem,directory,sqlite,index
 Author: Kevin Scott

{dirsql-0.3.90/packages/rust → dirsql-0.3.91}/docs/reference/sdk.md

@@ -81,8 +81,8 @@ DirSQL::builder()
     .tables(tables)                 // optional; append one with .table(t)
     .ignore(patterns)               // optional
     .config(config_toml_path)       // optional
-    .persist(true)                  // optional; default false
-    .persist_path(path)             // optional
+    .persist(cache_path)            // optional; Some(path), or None for the
+                                    //   default <root>/.dirsql/cache.db
     .extensions(extensions)         // optional; append one with .extension(e)
     .poll_interval(duration)        // optional; watch-loop cadence, default 200ms
     .build()                        // -> Result<DirSQL>  (synchronous scan)
@@ -114,7 +114,10 @@ directory" error.
   changed are re-parsed. (The CLI exposes the same switch as the
   [`--persist [PATH]`](./cli.md#server-mode) flag; it is not a config key.)
 - `persist_path` / `persistPath` — Override the cache location. Ignored when
-  persistence is off. Constructor values are used as given.
+  persistence is off. Constructor values are used as given. In Rust these two
+  parameters collapse into a single builder method: `.persist(None)` enables
+  persistence at the default location, `.persist(Some(path))` enables it at
+  `path`.
 - `extensions` — SQLite extensions to load at startup, before any table
   DDL (enable → load → disable, so SQL `load_extension()` is never
   exposed). Each entry pairs a shared-library `path` with an optional

{dirsql-0.3.90 → dirsql-0.3.91}/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.3.90"
+version = "0.3.91"
 # abi3 note: the `abi3-py311` pyo3 feature (below) builds ONE stable-ABI
 # `cp311-abi3` wheel per platform that loads on every CPython >= 3.11
 # (matching `pyproject.toml`'s `requires-python = ">=3.11"`), instead of

{dirsql-0.3.90 → dirsql-0.3.91/packages/python}/docs/reference/sdk.md

@@ -81,8 +81,8 @@ DirSQL::builder()
     .tables(tables)                 // optional; append one with .table(t)
     .ignore(patterns)               // optional
     .config(config_toml_path)       // optional
-    .persist(true)                  // optional; default false
-    .persist_path(path)             // optional
+    .persist(cache_path)            // optional; Some(path), or None for the
+                                    //   default <root>/.dirsql/cache.db
     .extensions(extensions)         // optional; append one with .extension(e)
     .poll_interval(duration)        // optional; watch-loop cadence, default 200ms
     .build()                        // -> Result<DirSQL>  (synchronous scan)
@@ -114,7 +114,10 @@ directory" error.
   changed are re-parsed. (The CLI exposes the same switch as the
   [`--persist [PATH]`](./cli.md#server-mode) flag; it is not a config key.)
 - `persist_path` / `persistPath` — Override the cache location. Ignored when
-  persistence is off. Constructor values are used as given.
+  persistence is off. Constructor values are used as given. In Rust these two
+  parameters collapse into a single builder method: `.persist(None)` enables
+  persistence at the default location, `.persist(Some(path))` enables it at
+  `path`.
 - `extensions` — SQLite extensions to load at startup, before any table
   DDL (enable → load → disable, so SQL `load_extension()` is never
   exposed). Each entry pairs a shared-library `path` with an optional

{dirsql-0.3.90 → dirsql-0.3.91}/packages/python/e2e-attestation.json

@@ -1,6 +1,6 @@
 {
   "command": "uv run python -m pytest tests/e2e/ -x -q",
-  "ran_at": 1783637710,
+  "ran_at": 1783639859,
   "exit_code": 0,
-  "commit": "c7c626ca03beb1b23e26ab96febf3c9df5d4a67f"
+  "commit": "8b7d30b7097ed56f7f1cca359b7a67ffaf8c4e65"
 }

{dirsql-0.3.90 → dirsql-0.3.91}/packages/python/src/lib.rs

@@ -154,10 +154,7 @@ mod python {
                         builder = builder.config(c);
                     }
                     if persist {
-                        builder = builder.persist(true);
-                    }
-                    if let Some(p) = persist_path {
-                        builder = builder.persist_path(p);
+                        builder = builder.persist(persist_path);
                     }
                     if !rust_extensions.is_empty() {
                         builder = builder.extensions(rust_extensions);

{dirsql-0.3.90/packages/python → dirsql-0.3.91/packages/rust}/docs/reference/sdk.md

@@ -81,8 +81,8 @@ DirSQL::builder()
     .tables(tables)                 // optional; append one with .table(t)
     .ignore(patterns)               // optional
     .config(config_toml_path)       // optional
-    .persist(true)                  // optional; default false
-    .persist_path(path)             // optional
+    .persist(cache_path)            // optional; Some(path), or None for the
+                                    //   default <root>/.dirsql/cache.db
     .extensions(extensions)         // optional; append one with .extension(e)
     .poll_interval(duration)        // optional; watch-loop cadence, default 200ms
     .build()                        // -> Result<DirSQL>  (synchronous scan)
@@ -114,7 +114,10 @@ directory" error.
   changed are re-parsed. (The CLI exposes the same switch as the
   [`--persist [PATH]`](./cli.md#server-mode) flag; it is not a config key.)
 - `persist_path` / `persistPath` — Override the cache location. Ignored when
-  persistence is off. Constructor values are used as given.
+  persistence is off. Constructor values are used as given. In Rust these two
+  parameters collapse into a single builder method: `.persist(None)` enables
+  persistence at the default location, `.persist(Some(path))` enables it at
+  `path`.
 - `extensions` — SQLite extensions to load at startup, before any table
   DDL (enable → load → disable, so SQL `load_extension()` is never
   exposed). Each entry pairs a shared-library `path` with an optional

{dirsql-0.3.90 → dirsql-0.3.91}/packages/rust/src/bin/dirsql.rs

@@ -76,10 +76,7 @@ impl Cli {
     /// → persist at `<path>`.
     fn apply_persist(&self, mut builder: dirsql::DirSQLBuilder) -> dirsql::DirSQLBuilder {
         if let Some(path) = &self.persist {
-            builder = builder.persist(true);
-            if let Some(path) = path {
-                builder = builder.persist_path(path);
-            }
+            builder = builder.persist(path.as_ref());
         }
         builder
     }

{dirsql-0.3.90 → dirsql-0.3.91}/packages/rust/src/lib.rs

@@ -932,20 +932,15 @@ impl DirSQLBuilder {
         self
     }
 
-    /// Enable persistent on-disk storage. When `true`, the SQLite database is
-    /// written to `<root>/.dirsql/cache.db` (override via
-    /// [`persist_path`](Self::persist_path)) so subsequent startups only
-    /// re-parse files that have actually changed. See
-    /// `docs/howto/persist.md` for the reconcile contract.
-    pub fn persist(mut self, persist: bool) -> Self {
-        self.persist = persist;
-        self
-    }
-
-    /// Override the location of the persistent cache file. Ignored when
-    /// [`persist`](Self::persist) is `false`.
-    pub fn persist_path(mut self, path: impl AsRef<Path>) -> Self {
-        self.persist_path = Some(path.as_ref().to_path_buf());
+    /// Enable persistent on-disk storage. `None` writes the SQLite database to
+    /// the default `<root>/.dirsql/cache.db`; `Some(path)` writes it to `path`.
+    /// Either way, subsequent startups only re-parse files that have actually
+    /// changed. See `docs/howto/persist.md` for the reconcile contract.
+    pub fn persist(mut self, path: Option<impl AsRef<Path>>) -> Self {
+        self.persist = true;
+        if let Some(path) = path {
+            self.persist_path = Some(path.as_ref().to_path_buf());
+        }
         self
     }
 
@@ -2651,8 +2646,7 @@ mod internal_tests {
             .ignore(["skip/**"])
             .extensions(Vec::<Extension>::new())
             .suppress_config_extensions(true)
-            .persist(true)
-            .persist_path(&cache)
+            .persist(Some(&cache))
             .poll_interval(Duration::from_millis(50))
             .build()
             .unwrap();
@@ -2661,6 +2655,38 @@ mod internal_tests {
         assert_eq!(db.inner.poll_interval, Duration::from_millis(50));
     }
 
+    #[test]
+    fn persist_none_enables_default_path() {
+        let resolved = DirSQL::builder()
+            .root("/tmp/x")
+            .persist(None::<&Path>)
+            .resolve()
+            .unwrap();
+        assert!(resolved.persist);
+        assert!(resolved.persist_path.is_none());
+    }
+
+    #[test]
+    fn persist_some_enables_explicit_path() {
+        let resolved = DirSQL::builder()
+            .root("/tmp/x")
+            .persist(Some("/tmp/x/custom.db"))
+            .resolve()
+            .unwrap();
+        assert!(resolved.persist);
+        assert_eq!(
+            resolved.persist_path,
+            Some(PathBuf::from("/tmp/x/custom.db"))
+        );
+    }
+
+    #[test]
+    fn persist_unset_leaves_persistence_off() {
+        let resolved = DirSQL::builder().root("/tmp/x").resolve().unwrap();
+        assert!(!resolved.persist);
+        assert!(resolved.persist_path.is_none());
+    }
+
     /// A second persist build over the same root+cache finds a compatible
     /// meta block and reuses the cache instead of a cold rebuild.
     #[test]
@@ -2674,8 +2700,7 @@ mod internal_tests {
                 "*.txt",
                 |_| vec![],
             )])
-            .persist(true)
-            .persist_path(&cache)
+            .persist(Some(&cache))
             .build()
             .unwrap();
         drop(first);
@@ -2686,8 +2711,7 @@ mod internal_tests {
                 "*.txt",
                 |_| vec![],
             )])
-            .persist(true)
-            .persist_path(&cache)
+            .persist(Some(&cache))
             .build()
             .unwrap();
         assert!(second.query("SELECT * FROM t").is_ok());