drivethrough-cli 0.1.11 → 0.1.12

package/dist/main.mjs

@@ -1071,25 +1071,41 @@ var init_universal = __esm({
   json       TEXT,
   created_at TEXT
 )`,
-      `CREATE INDEX IF NOT EXISTS idx_type   ON data(type)`,
-      // 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.
+      // ---- platform indexes ------------------------------------------------------
+      // Named `dt_*`, never `idx_*`. generatedColumnName() emits `idx_<type>_<field>`
+      // for a site's DECLARED indexes, so a site declaring type "type" with an index
+      // on "$.recent" would land on `idx_type_recent` — and applyDeclaredIndexes only
+      // checks for a colliding COLUMN, not a colliding INDEX, so its
+      // `CREATE INDEX IF NOT EXISTS` would silently no-op and the site's declared
+      // index would never exist. `dt_` is unreachable from that generator.
+      //
+      // Each statement pair DROPs the previous name before creating the current one:
+      // `CREATE ... IF NOT EXISTS` would otherwise see the old index and keep it, so
+      // databases provisioned earlier would never upgrade. Once upgraded both are
+      // no-ops, which keeps init() cheap to re-run.
+      // (type, created_at), not (type). Every query against this table leads with
+      // `type`, and the list queries all end `ORDER BY created_at DESC LIMIT n`.
+      // On the single column, SQLite filtered by type and then sorted EVERY row of
+      // that type to honour the ORDER BY — a 10k-row type returning 100 read 10k.
+      // The composite serves the filter exactly as before and satisfies the sort by
+      // walking the index backwards, stopping at the LIMIT. It is still one index,
+      // and D1 bills a written row per index touched rather than per column, so the
+      // read win costs nothing on writes.
+      `DROP INDEX IF EXISTS idx_type`,
+      `CREATE INDEX IF NOT EXISTS dt_type_recent ON data(type, created_at)`,
+      // PARTIAL. 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`
+      `DROP INDEX IF EXISTS idx_parent_present`,
+      `CREATE INDEX IF NOT EXISTS dt_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.11",
+  "version": "0.1.12",
   "description": "Agent-first hosting CLI. Say what you want. Stay in the car.",
   "type": "module",
   "license": "MIT",