dirsql 0.3.86__tar.gz → 0.3.87__tar.gz

{dirsql-0.3.86 → dirsql-0.3.87}/Cargo.lock

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

{dirsql-0.3.86 → dirsql-0.3.87}/PKG-INFO

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

{dirsql-0.3.86/packages/rust → dirsql-0.3.87}/docs/reference/hooks.md

@@ -102,7 +102,6 @@ row-mapping rules.
 | Placeholder | Value |
 |---|---|
 | `{path}` | The matched file's path **relative to the index root**. Appended automatically as a final argument when the command omits it, so `extract.py` and `extract.py {path}` behave identically. |
-| `{abspath}` | The matched file's absolute path. |
 | `{root}` | The index root directory. |
 
 ### `pre-query`

{dirsql-0.3.86 → dirsql-0.3.87}/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.86"
+version = "0.3.87"
 # 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.86 → dirsql-0.3.87}/packages/python/docs/reference/hooks.md

@@ -102,7 +102,6 @@ row-mapping rules.
 | Placeholder | Value |
 |---|---|
 | `{path}` | The matched file's path **relative to the index root**. Appended automatically as a final argument when the command omits it, so `extract.py` and `extract.py {path}` behave identically. |
-| `{abspath}` | The matched file's absolute path. |
 | `{root}` | The index root directory. |
 
 ### `pre-query`

{dirsql-0.3.86 → dirsql-0.3.87}/packages/python/e2e-attestation.json

@@ -1,6 +1,6 @@
 {
   "command": "uv run python -m pytest tests/e2e/ -x -q",
-  "ran_at": 1783631000,
+  "ran_at": 1783632150,
   "exit_code": 0,
-  "commit": "10123e5e2561015cbf7cceb07e838145f32318b9"
+  "commit": "89d9e986dee0ee7abaab16b6d43e9b17543d8f01"
 }

{dirsql-0.3.86 → dirsql-0.3.87/packages/rust}/docs/reference/hooks.md

@@ -102,7 +102,6 @@ row-mapping rules.
 | Placeholder | Value |
 |---|---|
 | `{path}` | The matched file's path **relative to the index root**. Appended automatically as a final argument when the command omits it, so `extract.py` and `extract.py {path}` behave identically. |
-| `{abspath}` | The matched file's absolute path. |
 | `{root}` | The index root directory. |
 
 ### `pre-query`

{dirsql-0.3.86 → dirsql-0.3.87}/packages/rust/src/command.rs

@@ -9,7 +9,7 @@
 //!   single argument — but no shell is ever invoked: there is no globbing,
 //!   piping, or variable expansion. To get a real shell, ask for one
 //!   explicitly (`sh -c '…'`).
-//! - **Placeholders.** `{path}`, `{args}`, `{abspath}`, `{root}` (and any
+//! - **Placeholders.** `{path}`, `{args}`, `{root}` (and any
 //!   others a caller supplies) are substituted into whole argv tokens, every
 //!   occurrence. Substitution is single-pass and left-to-right per token, so a
 //!   value that itself contains `{…}` is never re-scanned — a substituted value
@@ -437,10 +437,7 @@ mod tests {
 
     #[test]
     fn substitute_only_placeholder_is_dropped_when_absent() {
-        assert_eq!(
-            argv("run", &[Placeholder::new("abspath", "/tmp/x")]),
-            ["run"]
-        );
+        assert_eq!(argv("run", &[Placeholder::new("extra", "/tmp/x")]), ["run"]);
     }
 
     #[test]

{dirsql-0.3.86 → dirsql-0.3.87}/packages/rust/src/lib.rs

@@ -1428,9 +1428,8 @@ fn build_tables_from_config(
 /// into rows.
 ///
 /// Placeholders: `{path}` (the file relative to `root`, append-if-absent so
-/// `cmd` and `cmd {path}` behave identically), `{abspath}` (the absolute path),
-/// and `{root}` (the index root). `{path}` falls back to the absolute path
-/// when the file is not under `root`.
+/// `cmd` and `cmd {path}` behave identically) and `{root}` (the index root).
+/// `{path}` falls back to the absolute path when the file is not under `root`.
 ///
 /// Per-file isolation: any failure — a spawn/exit/timeout error from
 /// [`command::run_command`], or output that is not a JSON array of objects —
@@ -1451,7 +1450,6 @@ fn run_on_file(
         .unwrap_or_else(|_| abs_path.to_string());
     let placeholders = [
         Placeholder::append("path", rel),
-        Placeholder::new("abspath", abs_path),
         Placeholder::new("root", root.to_string_lossy().into_owned()),
     ];
 
@@ -2890,6 +2888,25 @@ mod internal_tests {
         assert_eq!(rows[0]["n"], Value::Integer(1));
     }
 
+    /// `{abspath}` is not in the substitution table: it is left literal like any
+    /// unknown `{…}`, so `printf` receives the string `{abspath}` verbatim. The
+    /// template references `{path}` so that arg is the real path (and no path is
+    /// appended), isolating the `{abspath}` behavior in the `q` column.
+    #[test]
+    fn run_on_file_does_not_substitute_abspath() {
+        let dir = TempDir::new().unwrap();
+        let abs = dir.path().join("f.txt");
+        let rows = run_on_file(
+            r#"printf '[{"p":"%s","q":"%s"}]' {path} {abspath}"#,
+            &abs.to_string_lossy(),
+            dir.path(),
+            dir.path(),
+            command::DEFAULT_COMMAND_TIMEOUT,
+        );
+        assert_eq!(rows.len(), 1);
+        assert_eq!(rows[0]["q"], Value::Text("{abspath}".into()));
+    }
+
     /// A command that cannot be spawned yields no rows (per-file isolation).
     #[test]
     fn run_on_file_returns_no_rows_on_spawn_failure() {