dirsql 0.3.79__tar.gz → 0.3.80__tar.gz

{dirsql-0.3.79 → dirsql-0.3.80}/Cargo.lock

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

{dirsql-0.3.79 → dirsql-0.3.80}/PKG-INFO

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

{dirsql-0.3.79 → dirsql-0.3.80}/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.79"
+version = "0.3.80"
 edition.workspace = true
 publish = false
 readme = "README.md"

{dirsql-0.3.79 → dirsql-0.3.80}/packages/python/e2e-attestation.json

@@ -1,6 +1,6 @@
 {
   "command": "uv run python -m pytest tests/e2e/ -x -q",
-  "ran_at": 1783534394,
+  "ran_at": 1783542175,
   "exit_code": 0,
-  "commit": "b66bf19ce2c568fc2aa08773caf8c708bad462e9"
+  "commit": "327255640a688bbd372efc17ab3f64cc59e8f651"
 }

{dirsql-0.3.79 → dirsql-0.3.80}/packages/rust/src/db.rs

@@ -280,14 +280,14 @@ impl Db {
         // `_dirsql_internal_rows`, keyed on the row's rowid. A column-less row
         // (SQLite requires ≥1 declared column, so this is only reachable
         // defensively) uses `DEFAULT VALUES`.
-        let columns: Vec<String> = row.keys().cloned().collect();
+        let columns: Vec<String> = row.keys().map(|c| format!("\"{c}\"")).collect();
         let sql = if columns.is_empty() {
-            format!("INSERT INTO {} DEFAULT VALUES", table)
+            format!("INSERT INTO \"{table}\" DEFAULT VALUES")
         } else {
             let placeholders: Vec<String> =
                 (1..=columns.len()).map(|i| format!("?{}", i)).collect();
             format!(
-                "INSERT INTO {} ({}) VALUES ({})",
+                "INSERT INTO \"{}\" ({}) VALUES ({})",
                 table,
                 columns.join(", "),
                 placeholders.join(", "),
@@ -1323,6 +1323,22 @@ mod tests {
         assert!(matches!(err, DbError::InvalidIdentifier(_)), "got: {err:?}");
     }
 
+    #[test]
+    fn insert_row_round_trips_reserved_word_column() {
+        let db = Db::new().unwrap();
+        db.create_table("CREATE TABLE t (path TEXT, \"order\" INTEGER)")
+            .unwrap();
+        let row = HashMap::from([
+            ("path".into(), Value::Text("a".into())),
+            ("order".into(), Value::Integer(7)),
+        ]);
+        db.insert_row("t", &row, "f.json", 0).unwrap();
+
+        let rows = db.get_rows_by_file("t", "f.json").unwrap();
+        assert_eq!(rows.len(), 1);
+        assert_eq!(rows[0]["order"], Value::Integer(7));
+    }
+
     #[test]
     fn load_extension_missing_file_errors() {
         let db = Db::new().unwrap();