dirsql 0.4.91__tar.gz → 0.4.92__tar.gz

{dirsql-0.4.91 → dirsql-0.4.92}/Cargo.lock

@@ -552,7 +552,7 @@ checksum = "6184e33543162437515c2e2b48714794e37845ec9851711914eec9d308f6ebe8"
 
 [[package]]
 name = "dirsql"
-version = "0.4.91"
+version = "0.4.92"
 dependencies = [
  "assert_cmd",
  "axum",
@@ -603,7 +603,7 @@ dependencies = [
 
 [[package]]
 name = "dirsql-py-ext"
-version = "0.4.91"
+version = "0.4.92"
 dependencies = [
  "dirsql",
  "pyo3",

{dirsql-0.4.91 → dirsql-0.4.92}/PKG-INFO

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

{dirsql-0.4.91 → dirsql-0.4.92}/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.91"
+version = "0.4.92"
 # 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.92/packages/python/changelog.d/2026-10-06-config-dotnames.md

@@ -0,0 +1,3 @@
+**Changed** A declared `[[table]] glob` no longer matches a dot-named file or
+directory unless the glob spells it: `**/*.md` skips `.env.md` and
+`.hid/z.md`, while `.hid/*.md` lists `.hid/z.md`.

dirsql-0.4.92/packages/python/e2e-attestations/claude-1258-config-dotnames.json

@@ -0,0 +1,7 @@
+{
+  "command": "uv run python -m pytest tests/e2e/glob_speed_dotfiles_test.py tests/e2e/glob_speed_recursive_test.py tests/e2e/glob_speed_matchers_test.py tests/e2e/query_contract_test.py -x -q",
+  "ran_at": 1791335904,
+  "exit_code": 0,
+  "commit": "dd146cf1e0f60982d186b8393e96ca484fb4b0b8",
+  "branch": "claude/1258-config-dotnames"
+}

dirsql-0.4.92/packages/python/migrations.d/2026-10-06-config-dotnames.md

@@ -0,0 +1,41 @@
+### A config glob hides dot-named entries unless it spells them
+
+#### Summary
+
+`glob = "**/*.md"` used to match `.env.md` and everything under `.hid/`. It
+now matches neither, as a path-table and bash (with `dotglob` off) do. A config
+that read dot-named files without naming them loses those rows.
+
+#### Required changes
+
+| Before | After |
+| --- | --- |
+| `glob = "**/*.md"` (included `.hid/z.md`) | `glob = ".hid/*.md"` or `glob = "{*,.hid}/*.md"` |
+
+#### Deprecations removed
+
+_None._
+
+#### Behavior changes without code changes
+
+- A dot-named component matches only where the glob spells a dot (`.hid`,
+  `.*`).
+- A glob that names the dot directory (`.hid/*.md`) is unchanged.
+
+#### Verification
+
+```bash
+cd "$(mktemp -d)"
+mkdir .hid
+echo '[{"p":"a"}]' > a.json
+echo '[{"p":"h"}]' > .hid/h.json
+cat > .dirsql.toml <<'TOML'
+[[table]]
+name = "t"
+glob = "**/*.json"
+ddl = "CREATE TABLE t (p TEXT)"
+on-file = "jq -sc add"
+TOML
+dirsql query "SELECT p FROM t" -c .dirsql.toml
+# expected: [{"p":"a"}]
+```

{dirsql-0.4.91 → dirsql-0.4.92}/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.91"
+version = "0.4.92"
 edition.workspace = true
 # Literal `license` (not `license.workspace = true`) because
 # putitoutthere's preflight check reads `[package].license` directly

{dirsql-0.4.91 → dirsql-0.4.92}/packages/rust/src/matcher.rs

@@ -2,6 +2,7 @@ use crate::posix_class::expand_posix_classes;
 use crate::scanner::{PathGlob, compile_glob};
 use globset::GlobBuilder;
 use regex::{Regex, RegexBuilder};
+use std::ffi::OsStr;
 use std::path::Path;
 
 #[derive(Debug, thiserror::Error)]
@@ -142,13 +143,20 @@ impl TableMatcher {
     pub fn match_all(&self, path: &Path) -> Vec<MatchResult> {
         self.entries
             .iter()
-            .filter(|entry| entry.pattern.is_match(path))
+            .filter(|entry| entry.pattern.is_match_unhidden(path))
             .map(|entry| MatchResult {
                 table_name: entry.table_name.clone(),
             })
             .collect()
     }
 
+    /// Whether any table's glob spells the dot-named `name`.
+    pub(crate) fn spells_dot_name(&self, name: &OsStr) -> bool {
+        self.entries
+            .iter()
+            .any(|entry| entry.pattern.spells_dot_name(name))
+    }
+
     /// Returns true if the path matches any ignore pattern.
     pub fn is_ignored(&self, path: &Path) -> bool {
         self.ignore_set.iter().any(|p| p.is_match(path))

{dirsql-0.4.91 → dirsql-0.4.92}/packages/rust/src/scanner.rs

@@ -236,7 +236,7 @@ enum Kind {
 }
 
 impl PathGlob {
-    fn spells_dot_name(&self, name: &OsStr) -> bool {
+    pub(crate) fn spells_dot_name(&self, name: &OsStr) -> bool {
         self.spelled_dot_names
             .iter()
             .any(|p| p.is_match(Path::new(name)))
@@ -246,6 +246,17 @@ impl PathGlob {
         self.files.iter().any(|p| p.is_match(rel_path))
     }
 
+    /// [`is_match`](Self::is_match), and every dot-named component of the path
+    /// is one the glob spells.
+    pub(crate) fn is_match_unhidden(&self, rel_path: &Path) -> bool {
+        self.is_match(rel_path)
+            && rel_path
+                .components()
+                .map(|c| c.as_os_str())
+                .filter(|name| is_dot_named(name))
+                .all(|name| self.spells_dot_name(name))
+    }
+
     fn has_components(&self) -> bool {
         !self.components.is_empty()
     }
@@ -584,10 +595,11 @@ impl Walk<'_> {
     }
 
     fn admits_name(&self, name: &OsStr) -> bool {
-        match self.glob {
-            Some(glob) => !is_dot_named(name) || glob.spells_dot_name(name),
-            None => true,
-        }
+        !is_dot_named(name)
+            || match self.glob {
+                Some(glob) => glob.spells_dot_name(name),
+                None => self.ignore.spells_dot_name(name),
+            }
     }
 
     fn next_states(&self, states: &[usize], name: &OsStr, kind: Kind) -> Vec<usize> {
@@ -1018,7 +1030,7 @@ mod tests {
         fs::create_dir_all(root.path().join("a").join("node_modules").join("pkg")).unwrap();
         fs::create_dir_all(root.path().join("a").join(RESERVED_DIR)).unwrap();
         fs::write(root.path().join("a").join("f.md"), "").unwrap();
-        let ignore = TableMatcher::new(&[], &["**/node_modules/**"]).unwrap();
+        let ignore = TableMatcher::new(&[(".dirsql/*", "t")], &["**/node_modules/**"]).unwrap();
         let a = root.path().join("a");
         let mut dirs = scan_dirs(root.path(), &a, &ignore);
         dirs.sort();
@@ -1040,7 +1052,7 @@ mod tests {
     fn explore_enters_a_dirsql_directory_below_the_top_level() {
         let root = tempfile::tempdir().unwrap();
         fs::create_dir_all(root.path().join("a").join(RESERVED_DIR)).unwrap();
-        let ignore = TableMatcher::new(&[], &[]).unwrap();
+        let ignore = TableMatcher::new(&[("**/.dirsql/*", "t")], &[]).unwrap();
         let top = Place {
             walk: walk_with(&ignore, None, Vec::new()),
             dir: root.path().to_path_buf(),
@@ -1393,8 +1405,8 @@ mod tests {
     }
 
     #[test]
-    fn admits_every_dot_named_entry_when_no_glob_governs_the_walk() {
-        let ignore = TableMatcher::new(&[], &[]).unwrap();
+    fn a_table_walk_admits_a_dot_name_only_a_table_glob_spells() {
+        let ignore = TableMatcher::new(&[(".env", "t")], &[]).unwrap();
         let walk = walk_with(&ignore, None, Vec::new());
         assert!(walk.admits(
             1,
@@ -1403,6 +1415,23 @@ mod tests {
             Path::new("/r/.env"),
             Path::new(".env")
         ));
+        assert!(!walk.admits(
+            1,
+            false,
+            OsStr::new(".other"),
+            Path::new("/r/.other"),
+            Path::new(".other")
+        ));
+    }
+
+    #[test]
+    fn a_match_with_an_unspelled_dot_component_is_hidden() {
+        let glob = compile_glob("**/*.md").unwrap();
+        assert!(glob.is_match_unhidden(Path::new("sub/b.md")));
+        assert!(!glob.is_match_unhidden(Path::new(".env.md")));
+        assert!(!glob.is_match_unhidden(Path::new(".hid/z.md")));
+        let spelled = compile_glob(".hid/*.md").unwrap();
+        assert!(spelled.is_match_unhidden(Path::new(".hid/z.md")));
     }
 
     #[test]