fix(driver-sql): materialize declared object-level indexes (#1459)

[os-zhuang]

Summary

Fixes #1459. The SQL driver synced columns and field-level unique, but silently dropped object-level declared indexes (ObjectSchema.indexes: [{ fields, unique }]). As a result several documented multi-column UNIQUE / dedup guarantees were never enforced at the DB level — a fresh dev --fresh sqlite DB showed only PK autoindexes.

Affected guarantees that depended on an unenforced index:

• sys_notification_delivery UNIQUE(notification_id, recipient_id, channel) — outbox enqueue() convergence-on-conflict
• sys_notification_receipt UNIQUE(notification_id, user_id, channel) — bell mark-read / inbox delivered-receipt upserts
• sys_notification.dedup_key UNIQUE (feat(messaging): race-safe dedup + opt-in retention (ADR-0030 hardening) #1458) — emit() dedup race-safety
• sys_api_key.key, sys_organization.slug, and other platform objects' declared unique indexes

Changes

Wired declared-index sync into initObjects (syncDeclaredIndexes):

• Materializes single- and multi-column indexes after the table is created/altered (so every referenced column physically exists), including UNIQUE.
• NULL-distinct semantics are the cross-dialect default (SQLite/Postgres/MySQL), so multiple NULL rows stay insertable while non-NULL duplicates are rejected — matching the convergence-on-conflict pattern the messaging pipeline relies on.
• Idempotent: deterministic, length-bounded index names (hash-suffixed past the 60-char mark for PG/MySQL limits) + per-dialect existing-index introspection (PRAGMA index_list / pg_indexes / information_schema.STATISTICS). An "already exists" race is absorbed.
• Indexes referencing a non-materialized (virtual formula) column are skipped with a warning instead of failing sync.

Flipped the indexes capability flag false → true.

Tests

Added 7 tests to sql-driver-schema.test.ts, all asserting via PRAGMA index_list on a fresh in-memory sqlite DB:

• single-column unique index materialized + rejects duplicates
• multi-column (compound) unique index — same tuple rejected, differing tuple allowed
• multiple NULLs allowed under a unique index (NULL-distinct)
• non-unique index materialized
• idempotent across repeated initObjects runs
• declared index over a virtual column is skipped with a warning (no throw)
• supports.indexes === true

Full driver-sql suite: 132 passed. Package builds clean (ESM/CJS/DTS).

Notes

Postgres/MySQL paths use the same knex alterTable + introspection helpers but were not exercised in CI here (no live PG/MySQL in this environment). The implementation is dialect-portable; a Postgres assertion can be added if a fixture is available.

https://claude.ai/code/session_01AgJ1guiaybKRbMfqAPCDeN

---

Generated by Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
spec	Canceled		Jun 1, 2026 9:04am