dirsql 0.4.69__tar.gz → 0.4.70__tar.gz

{dirsql-0.4.69 → dirsql-0.4.70}/Cargo.lock

@@ -552,7 +552,7 @@ checksum = "6184e33543162437515c2e2b48714794e37845ec9851711914eec9d308f6ebe8"
 
 [[package]]
 name = "dirsql"
-version = "0.4.69"
+version = "0.4.70"
 dependencies = [
  "assert_cmd",
  "axum",
@@ -601,7 +601,7 @@ dependencies = [
 
 [[package]]
 name = "dirsql-py-ext"
-version = "0.4.69"
+version = "0.4.70"
 dependencies = [
  "dirsql",
  "pyo3",

{dirsql-0.4.69 → dirsql-0.4.70}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dirsql
-Version: 0.4.69
+Version: 0.4.70
 Requires-Dist: bin-shim>=0.1
 Summary: Ephemeral SQL index over a local directory
 Keywords: sql,filesystem,directory,sqlite,index

{dirsql-0.4.69/packages/rust → dirsql-0.4.70}/docs/reference/path-tables.md

@@ -295,6 +295,15 @@ SELECT path FROM './dist';                 -- scans dist/ even when gitignored
 The `.gitignore` files in force there, including those above it, still
 filter what lies beneath it.
 
+A `node_modules` component after a glob names it too, at any depth:
+
+```sql
+SELECT path FROM './**/node_modules/*/package.json';  -- every node_modules
+SELECT path FROM './*/node_modules/**';               -- one level down
+```
+
+Only the literal name counts; `'./**/*.js'` still skips every `node_modules`.
+
 ### Hidden files
 
 A path component starting with `.` is skipped unless the path spells it, the

{dirsql-0.4.69 → dirsql-0.4.70}/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.69"
+version = "0.4.70"
 # 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.70/packages/python/changelog.d/2026-10-05-path-table-node-modules-named.md

@@ -0,0 +1,6 @@
+**Fixed** A path-table pattern that names `node_modules` after a glob
+component now scans it: `'./**/node_modules/**'` lists every `node_modules`
+in the tree and `'./*/node_modules/*/*'` the ones one level down. Only a
+`node_modules` in the literal prefix used to count, so these returned nothing.
+A pattern that does not name `node_modules`, such as `'./**'` or
+`'./**/*.js'`, still skips it.

{dirsql-0.4.69 → dirsql-0.4.70}/packages/python/docs/reference/path-tables.md

@@ -295,6 +295,15 @@ SELECT path FROM './dist';                 -- scans dist/ even when gitignored
 The `.gitignore` files in force there, including those above it, still
 filter what lies beneath it.
 
+A `node_modules` component after a glob names it too, at any depth:
+
+```sql
+SELECT path FROM './**/node_modules/*/package.json';  -- every node_modules
+SELECT path FROM './*/node_modules/**';               -- one level down
+```
+
+Only the literal name counts; `'./**/*.js'` still skips every `node_modules`.
+
 ### Hidden files
 
 A path component starting with `.` is skipped unless the path spells it, the

dirsql-0.4.70/packages/python/e2e-attestations/claude-1271-node-modules-named.json

@@ -0,0 +1,7 @@
+{
+  "command": "just test-e2e",
+  "ran_at": 1791224899,
+  "exit_code": 0,
+  "commit": "e48a79c9fc67e5c2d69ec6c77116e5d2730684bb",
+  "branch": "claude/1271-node-modules-named"
+}

dirsql-0.4.70/packages/python/migrations.d/2026-10-05-path-table-node-modules-named.md

@@ -0,0 +1,37 @@
+### Core: `node_modules` named after a glob is scanned
+
+#### Summary
+
+A path-table pattern with a literal `node_modules` component after a glob,
+such as `'./**/node_modules/**'`, now scans the `node_modules` directories it
+names; it used to return nothing. Every surface that resolves a path-table is
+affected: the CLI, the REPL, the HTTP server, the SDKs, and the `--on-file`
+form. Nothing breaks; such a query can return more rows. A pattern that does
+not name `node_modules` still skips it, and declared `[[table]]` globs are
+unchanged.
+
+#### Required changes
+
+_None._
+
+#### Deprecations removed
+
+_None._
+
+#### Behavior changes without code changes
+
+- `'./**/node_modules/**'` lists files under every `node_modules`.
+- `'./*/node_modules/*/*'` lists files two levels into each `node_modules`
+  one level down.
+
+#### Verification
+
+```bash
+cd "$(mktemp -d)"
+mkdir -p node_modules/pkg pkg/node_modules/dep
+touch node_modules/pkg/index.js pkg/node_modules/dep/i.js
+dirsql query "SELECT path FROM './**/node_modules/*/*' ORDER BY path"
+# expected: [{"path":"node_modules/pkg/index.js"},{"path":"pkg/node_modules/dep/i.js"}]
+dirsql query "SELECT path FROM './**/*.js'"
+# expected: []
+```

{dirsql-0.4.69 → dirsql-0.4.70}/packages/rust/Cargo.toml

@@ -6,7 +6,7 @@ description = "Ephemeral SQL index over a local directory"
 # version stays for the binding crates (publish = false). The literal
 # intentionally lags the published version: putitoutthere rewrites it to
 # the planned release version at build time and never commits it back.
-version = "0.4.69"
+version = "0.4.70"
 edition.workspace = true
 # Literal `license` (not `license.workspace = true`) because
 # putitoutthere's preflight check reads `[package].license` directly

{dirsql-0.4.69 → dirsql-0.4.70/packages/rust}/docs/reference/path-tables.md

@@ -295,6 +295,15 @@ SELECT path FROM './dist';                 -- scans dist/ even when gitignored
 The `.gitignore` files in force there, including those above it, still
 filter what lies beneath it.
 
+A `node_modules` component after a glob names it too, at any depth:
+
+```sql
+SELECT path FROM './**/node_modules/*/package.json';  -- every node_modules
+SELECT path FROM './*/node_modules/**';               -- one level down
+```
+
+Only the literal name counts; `'./**/*.js'` still skips every `node_modules`.
+
 ### Hidden files
 
 A path component starting with `.` is skipped unless the path spells it, the

{dirsql-0.4.69 → dirsql-0.4.70}/packages/rust/src/db.rs

@@ -315,7 +315,8 @@ pub struct Db {
     /// exactly where `files` used to exist implicitly. A user who declared
     /// tables and forgot `files` gets the plain SQLite error.
     hint_legacy_files_table: bool,
-    /// Skip rules a path-table scan applies, seeded with the built-in defaults.
+    /// Configured skip rules a path-table scan applies on top of the table's
+    /// [`PathTable::default_ignores`].
     path_table_ignore: Vec<String>,
     /// Whether a path-table scan respects `.gitignore` files. On by default;
     /// the CLI's `--no-ignore` turns it off.
@@ -351,14 +352,6 @@ fn worker_error(message: String) -> DbError {
     DbError::Sqlite(rusqlite::Error::UserFunctionError(message.into()))
 }
 
-/// The skip rules a fresh `Db` starts with.
-fn default_path_table_ignore() -> Vec<String> {
-    path_table::DEFAULT_IGNORES
-        .iter()
-        .map(|s| (*s).to_string())
-        .collect()
-}
-
 impl Db {
     /// Open the default, ephemeral `Db`: an **anonymous disk-backed temp
     /// database** (`Connection::open("")`), not `:memory:`.
@@ -382,7 +375,7 @@ impl Db {
             scope,
             path_table_root: None,
             hint_legacy_files_table: false,
-            path_table_ignore: default_path_table_ignore(),
+            path_table_ignore: Vec::new(),
             path_table_gitignore: true,
             path_table_parser: None,
             calls: CallReporter::new(),
@@ -407,7 +400,7 @@ impl Db {
             scope,
             path_table_root: None,
             hint_legacy_files_table: false,
-            path_table_ignore: default_path_table_ignore(),
+            path_table_ignore: Vec::new(),
             path_table_gitignore: true,
             path_table_parser: None,
             calls: CallReporter::new(),
@@ -1170,10 +1163,15 @@ impl Db {
             .path_table_root
             .as_deref()
             .expect("a path-table resolves only under an index root");
+        let ignore: Vec<String> = table
+            .default_ignores()
+            .map(str::to_string)
+            .chain(self.path_table_ignore.iter().cloned())
+            .collect();
         self.conn.execute_batch(&path_table_ddl(
             name,
             table,
-            &self.path_table_ignore,
+            &ignore,
             self.path_table_gitignore,
             self.path_table_parser.as_deref(),
             index_root,
@@ -3083,14 +3081,6 @@ mod tests {
         );
     }
 
-    #[test]
-    fn default_path_table_ignore_carries_the_documented_defaults() {
-        assert_eq!(
-            default_path_table_ignore(),
-            vec!["**/node_modules/**".to_string(), "**/.git/**".to_string()]
-        );
-    }
-
     fn docs_path_table() -> PathTable {
         PathTable {
             root: PathBuf::from("/root"),
@@ -3304,17 +3294,12 @@ mod tests {
     }
 
     #[test]
-    fn add_path_table_ignore_extends_the_defaults() {
+    fn add_path_table_ignore_accumulates_configured_rules() {
         let mut db = Db::new().unwrap();
         db.add_path_table_ignore(vec!["*.tmp".to_string()]);
+        db.add_path_table_ignore(vec!["*.bak".to_string()]);
 
-        assert_eq!(db.path_table_ignore.last(), Some(&"*.tmp".to_string()));
-        assert!(
-            db.path_table_ignore
-                .contains(&"**/node_modules/**".to_string()),
-            "the built-in defaults must survive: {:?}",
-            db.path_table_ignore
-        );
+        assert_eq!(db.path_table_ignore, ["*.tmp", "*.bak"]);
     }
 
     #[test]

{dirsql-0.4.69 → dirsql-0.4.70}/packages/rust/src/path_table.rs

@@ -16,8 +16,11 @@ use typed_path::{
 
 /// Directories a path-table scan skips, at any depth. They are skipped only
 /// *beneath* the literal part of the path you write, so naming one explicitly
-/// still scans it.
-pub const DEFAULT_IGNORES: [&str; 2] = ["**/node_modules/**", "**/.git/**"];
+/// still scans it. See [`PathTable::default_ignores`] for `node_modules`
+/// named after a glob.
+pub const DEFAULT_IGNORES: [&str; 2] = [NODE_MODULES_IGNORE, "**/.git/**"];
+
+const NODE_MODULES_IGNORE: &str = "**/node_modules/**";
 
 /// The glob a directory expands to: one level, like `ls`. Any depth is
 /// spelled explicitly as `**`.
@@ -36,6 +39,18 @@ pub struct PathTable {
     pub path_prefix: String,
 }
 
+impl PathTable {
+    /// The [`DEFAULT_IGNORES`] this table applies. A glob with a
+    /// `node_modules` component names that directory wherever the component
+    /// sits, so its skip rule is dropped.
+    pub fn default_ignores(&self) -> impl Iterator<Item = &'static str> {
+        let names_node_modules = self.glob.split('/').any(|c| c == "node_modules");
+        DEFAULT_IGNORES
+            .into_iter()
+            .filter(move |rule| !(names_node_modules && *rule == NODE_MODULES_IGNORE))
+    }
+}
+
 /// What a name SQLite could not find turns out to be.
 #[derive(Debug, PartialEq)]
 pub enum Resolution {
@@ -315,6 +330,32 @@ mod tests {
         assert_eq!(DEFAULT_IGNORES, ["**/node_modules/**", "**/.git/**"]);
     }
 
+    fn table_globbing(glob: &str) -> PathTable {
+        PathTable {
+            root: PathBuf::from("/r"),
+            glob: glob.to_string(),
+            path_prefix: String::new(),
+        }
+    }
+
+    fn default_ignores_of(glob: &str) -> Vec<&'static str> {
+        table_globbing(glob).default_ignores().collect()
+    }
+
+    #[test]
+    fn a_glob_not_naming_node_modules_keeps_every_default_ignore() {
+        assert_eq!(default_ignores_of("**"), DEFAULT_IGNORES);
+        assert_eq!(default_ignores_of("**/*.js"), DEFAULT_IGNORES);
+        assert_eq!(default_ignores_of("node_modules*/*"), DEFAULT_IGNORES);
+    }
+
+    #[test]
+    fn a_glob_naming_node_modules_anywhere_drops_its_ignore() {
+        assert_eq!(default_ignores_of("**/node_modules/**"), ["**/.git/**"]);
+        assert_eq!(default_ignores_of("*/node_modules/*/*"), ["**/.git/**"]);
+        assert_eq!(default_ignores_of("node_modules"), ["**/.git/**"]);
+    }
+
     #[test]
     fn has_glob_metacharacter_spots_each_metacharacter() {
         assert!(has_glob_metacharacter("a*"));

{dirsql-0.4.69 → dirsql-0.4.70}/packages/rust/src/scanner.rs

@@ -1031,6 +1031,7 @@ mod tests {
             ignore: &ignore,
             glob: Some(&glob),
             gitignore: false,
+            in_repo: false,
             frames: Vec::new(),
         };
         assert!(walk.follows(false, &[1], Kind::Dir));
@@ -1046,6 +1047,7 @@ mod tests {
             ignore: &ignore,
             glob: Some(&glob),
             gitignore: false,
+            in_repo: false,
             frames: Vec::new(),
         };
         assert!(walk.follows(false, &[], Kind::Dir));