drivethrough-cli 0.1.10 → 0.1.11

package/dist/main.mjs

@@ -1072,7 +1072,24 @@ var init_universal = __esm({
   created_at TEXT
 )`,
       `CREATE INDEX IF NOT EXISTS idx_type   ON data(type)`,
-      `CREATE INDEX IF NOT EXISTS idx_parent ON data(parent_id)`
+      // idx_parent is PARTIAL. D1 bills a written row per index touched, so a full
+      // index on parent_id charged every insert for an entry the common case never
+      // reads: most rows are top-level (a form submission with no parent), and
+      // `WHERE parent_id = ?` can only match a non-null value. Excluding NULLs
+      // keeps every query that used it — `type = ?1 AND parent_id = ?2` — on an
+      // index, while top-level inserts stop paying for it.
+      //
+      // The one reader of NULL parents (selectSessionRow's `parent_id IS NULL`
+      // branch) cannot use a partial index, and did not want this one anyway: it
+      // filters on type first, and for the unique_per types it actually runs on,
+      // uniq_session_<type> covers it.
+      //
+      // DROP first, under the old name: databases created before this change
+      // already hold the full index, and `CREATE ... IF NOT EXISTS` would see it
+      // and silently keep it. Renaming makes both statements true no-ops once the
+      // upgrade has happened, so init() stays cheap to re-run.
+      `DROP INDEX IF EXISTS idx_parent`,
+      `CREATE INDEX IF NOT EXISTS idx_parent_present ON data(parent_id) WHERE parent_id IS NOT NULL`
     ];
     UNIVERSAL_SCHEMA_SQL = UNIVERSAL_SCHEMA_STATEMENTS.join(";\n") + ";";
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "drivethrough-cli",
-  "version": "0.1.10",
+  "version": "0.1.11",
   "description": "Agent-first hosting CLI. Say what you want. Stay in the car.",
   "type": "module",
   "license": "MIT",