dirsql 0.4.86__tar.gz → 0.4.87__tar.gz

{dirsql-0.4.86 → dirsql-0.4.87}/Cargo.lock

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

{dirsql-0.4.86 → dirsql-0.4.87}/PKG-INFO

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

{dirsql-0.4.86/packages/rust → dirsql-0.4.87}/docs/reference/path-tables.md

@@ -59,7 +59,9 @@ alternation: `'./docs/{a,b}.md'` is `docs/a.md` and `docs/b.md`, and groups
 nest. A group holding a `..` range counts: `'./log/{1..3}.txt'`,
 `'./{a..c}.md'`, `'./{01..10}.md'` (zero-padded) and `'./{1..10..2}.md'`
 (every second value). Any other brace is literal text, so `'./br/{q}.md'`
-names the file `br/{q}.md`, and `\{` is always a literal brace.
+names the file `br/{q}.md`, and `\{` is always a literal brace. A `{`, `}`
+or `[` with no partner is literal too, in a path-table and in a config
+`[[table]] glob`.
 
 The scan starts at the last directory named outright before the first glob
 component, and descends only as deep as the pattern can match --

{dirsql-0.4.86 → dirsql-0.4.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.4.86"
+version = "0.4.87"
 # 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.87/packages/python/changelog.d/2026-10-05-unbalanced-brackets-literal.md

@@ -0,0 +1 @@
+**Fixed** An unmatched `{`, `}` or `[` in a path-table or a config `[[table]] glob` is a literal character, as in bash, instead of a glob compile error.

{dirsql-0.4.86 → dirsql-0.4.87/packages/python}/docs/reference/path-tables.md

@@ -59,7 +59,9 @@ alternation: `'./docs/{a,b}.md'` is `docs/a.md` and `docs/b.md`, and groups
 nest. A group holding a `..` range counts: `'./log/{1..3}.txt'`,
 `'./{a..c}.md'`, `'./{01..10}.md'` (zero-padded) and `'./{1..10..2}.md'`
 (every second value). Any other brace is literal text, so `'./br/{q}.md'`
-names the file `br/{q}.md`, and `\{` is always a literal brace.
+names the file `br/{q}.md`, and `\{` is always a literal brace. A `{`, `}`
+or `[` with no partner is literal too, in a path-table and in a config
+`[[table]] glob`.
 
 The scan starts at the last directory named outright before the first glob
 component, and descends only as deep as the pattern can match --

dirsql-0.4.87/packages/python/e2e-attestations/claude-1268-unbalanced-brackets.json

@@ -0,0 +1,7 @@
+{
+  "command": "uv run python -m pytest tests/e2e/glob_speed_depth_bound_test.py tests/e2e/glob_speed_recursive_test.py tests/e2e/glob_speed_one_level_test.py tests/e2e/glob_speed_matchers_test.py tests/e2e/glob_speed_dotfiles_test.py tests/e2e/query_contract_test.py tests/e2e/fanout_test.py tests/e2e/capture_collision_test.py -x -q",
+  "ran_at": 1791325871,
+  "exit_code": 0,
+  "commit": "45535f123cfc3ea9cf9460c228196e2630d377db",
+  "branch": "claude/1268-unbalanced-brackets"
+}

dirsql-0.4.87/packages/python/migrations.d/2026-10-05-unbalanced-brackets-literal.md

@@ -0,0 +1,29 @@
+### Core: unbalanced braces and brackets in a glob are literal
+
+#### Summary
+
+A `{`, `}` or `[` with no partner in a glob is now a literal character, as in
+bash, on path-tables and on config `[[table]] glob`. It used to fail with a
+glob compile error. Nothing that compiled before changes meaning.
+
+#### Required changes
+
+_None._
+
+#### Deprecations removed
+
+_None._
+
+#### Behavior changes without code changes
+
+- `'./{a.md'`, `'./a}.md'`, `'./[a.md'` and `'./a[b.md'` match files named
+  exactly that instead of failing to compile.
+
+#### Verification
+
+```bash
+cd "$(mktemp -d)"
+touch '[a.md'
+dirsql query "SELECT path FROM './[a.md'"
+# expected: [{"path":"[a.md"}]
+```

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

{dirsql-0.4.86/packages/python → dirsql-0.4.87/packages/rust}/docs/reference/path-tables.md

@@ -59,7 +59,9 @@ alternation: `'./docs/{a,b}.md'` is `docs/a.md` and `docs/b.md`, and groups
 nest. A group holding a `..` range counts: `'./log/{1..3}.txt'`,
 `'./{a..c}.md'`, `'./{01..10}.md'` (zero-padded) and `'./{1..10..2}.md'`
 (every second value). Any other brace is literal text, so `'./br/{q}.md'`
-names the file `br/{q}.md`, and `\{` is always a literal brace.
+names the file `br/{q}.md`, and `\{` is always a literal brace. A `{`, `}`
+or `[` with no partner is literal too, in a path-table and in a config
+`[[table]] glob`.
 
 The scan starts at the last directory named outright before the first glob
 component, and descends only as deep as the pattern can match --

{dirsql-0.4.86 → dirsql-0.4.87}/packages/rust/src/brace.rs

@@ -198,7 +198,7 @@ fn steps(lo: i64, hi: i64, step: i64) -> Option<impl Iterator<Item = i128>> {
 
 /// Escape the braces bash left as literal text, so globset matches them
 /// rather than reading an alternation. A complete `[...]` class is copied as
-/// written; an unclosed one ends escaping so globset reports the error.
+/// written; an unclosed `[` is a literal character.
 fn escape_braces(word: &str) -> String {
     let chars: Vec<char> = word.chars().collect();
     let mut out = String::with_capacity(word.len());
@@ -222,10 +222,7 @@ fn escape_braces(word: &str) -> String {
                     out.extend(&chars[i..=end]);
                     resume = end + 1;
                 }
-                None => {
-                    out.extend(&chars[i..]);
-                    return out;
-                }
+                None => out.push_str("[[]"),
             },
             ('{', _) => out.push_str("[{]"),
             ('}', _) => out.push_str("[}]"),
@@ -411,9 +408,18 @@ mod tests {
     }
 
     #[test]
-    fn an_unclosed_class_ends_escaping() {
-        assert_eq!(escape_braces("x[{q}"), "x[{q}");
-        assert_eq!(escape_braces("x[]{"), "x[]{");
+    fn an_unclosed_bracket_is_literal() {
+        assert_eq!(escape_braces("x[{q}"), "x[[][{]q[}]");
+        assert_eq!(escape_braces("x[]{"), "x[[]][{]");
+        assert_eq!(escape_braces("[!a"), "[[]!a");
+        assert_eq!(escape_braces("a].md"), "a].md");
+    }
+
+    #[test]
+    fn expand_leaves_an_unpartnered_brace_literal() {
+        assert_eq!(expand("{a.md"), vec!["[{]a.md"]);
+        assert_eq!(expand("a}.md"), vec!["a[}].md"]);
+        assert_eq!(expand("{a,b}}.md"), vec!["a[}].md", "b[}].md"]);
     }
 
     #[test]

{dirsql-0.4.86 → dirsql-0.4.87}/packages/rust/src/matcher.rs

@@ -323,7 +323,7 @@ mod tests {
 
     #[test]
     fn invalid_glob_returns_error() {
-        let result = TableMatcher::new(&[("[invalid", "t")], &[]);
+        let result = TableMatcher::new(&[("[z-a]", "t")], &[]);
         assert!(result.is_err());
     }
 

{dirsql-0.4.86 → dirsql-0.4.87}/packages/rust/src/parsed_vtab.rs

@@ -291,7 +291,7 @@ mod tests {
 
     #[test]
     fn parse_module_args_rejects_an_invalid_glob() {
-        let args = args_with(&[b"'/tmp'", b"'['", b"'cat'", b"'gitignore'", b"'/tmp'"]);
+        let args = args_with(&[b"'/tmp'", b"'[z-a]'", b"'cat'", b"'gitignore'", b"'/tmp'"]);
         assert!(parse_module_args(&args).is_err());
     }
 

{dirsql-0.4.86 → dirsql-0.4.87}/packages/rust/src/scanner.rs

@@ -1110,7 +1110,7 @@ mod tests {
 
     #[test]
     fn compile_glob_rejects_an_invalid_pattern() {
-        assert!(compile_glob("[").is_err());
+        assert!(compile_glob("[z-a]").is_err());
     }
 
     #[test]

{dirsql-0.4.86 → dirsql-0.4.87}/packages/rust/src/vtab.rs

@@ -620,7 +620,7 @@ mod tests {
 
     #[test]
     fn parse_module_args_rejects_an_invalid_glob() {
-        let args = args_with(&[b"'/tmp'", b"'['", b"''", b"'gitignore'"]);
+        let args = args_with(&[b"'/tmp'", b"'[z-a]'", b"''", b"'gitignore'"]);
         assert!(parse_module_args(&args).is_err());
     }
 

{dirsql-0.4.86 → dirsql-0.4.87}/packages/rust/src/vtab_scaffold.rs

@@ -669,7 +669,7 @@ mod tests {
 
     #[test]
     fn compile_glob_rejects_an_invalid_pattern() {
-        let err = compile_glob("[").unwrap_err();
+        let err = compile_glob("[z-a]").unwrap_err();
         assert!(
             matches!(err, Error::ModuleError(_)),
             "invalid globs surface as module errors, got {err:?}"