@korajs/core 1.0.0-beta.1 → 1.0.0-beta.2

package/dist/bindings/index.d.cts

@@ -1,5 +1,5 @@
-import { p as KoraEventEmitter } from '../events-CqDkr1LN.cjs';
-import { S as ScopeMap } from '../build-scope-map-DtTr9C7A.cjs';
+import { p as KoraEventEmitter } from '../events-DhblJN4r.cjs';
+import { S as ScopeMap } from '../build-scope-map-K25AyQXb.cjs';
 
 /** Sync status shape consumed by framework binding hooks (matches {@link SyncStatusInfo} in `@korajs/sync`). */
 interface KoraBindingSyncStatus {

package/dist/bindings/index.d.ts

@@ -1,5 +1,5 @@
-import { p as KoraEventEmitter } from '../events-CqDkr1LN.js';
-import { S as ScopeMap } from '../build-scope-map-DvOHbYjn.js';
+import { p as KoraEventEmitter } from '../events-DhblJN4r.js';
+import { S as ScopeMap } from '../build-scope-map-BvdTzmm4.js';
 
 /** Sync status shape consumed by framework binding hooks (matches {@link SyncStatusInfo} in `@korajs/sync`). */
 interface KoraBindingSyncStatus {

package/dist/{build-scope-map-DvOHbYjn.d.ts → build-scope-map-BvdTzmm4.d.ts}

@@ -1,4 +1,4 @@
-import { S as SchemaDefinition } from './events-CqDkr1LN.js';
+import { S as SchemaDefinition } from './events-DhblJN4r.js';
 
 /**
  * Per-collection scope map: `{ collectionName: { field: value, ... } }`

package/dist/{build-scope-map-DtTr9C7A.d.cts → build-scope-map-K25AyQXb.d.cts}

@@ -1,4 +1,4 @@
-import { S as SchemaDefinition } from './events-CqDkr1LN.cjs';
+import { S as SchemaDefinition } from './events-DhblJN4r.cjs';
 
 /**
  * Per-collection scope map: `{ collectionName: { field: value, ... } }`

package/dist/{events-CqDkr1LN.d.cts → events-DhblJN4r.d.cts}

@@ -826,6 +826,21 @@ type KoraEvent = {
     code: string;
     message: string;
     retriable: boolean;
+} | {
+    /**
+     * The server rejected one of THIS client's outbound operations before it
+     * became authoritative. The op has been diverted out of the pending sync
+     * queue into the durable rejected store (kept, not retried); the app can
+     * surface the reason and decide whether to roll back the optimistic local
+     * write or let the user edit and resubmit.
+     */
+    type: 'sync:operation-rejected';
+    operationId: string;
+    collection: string;
+    recordId: string;
+    code: string;
+    message: string;
+    retriable: boolean;
 } | {
     type: 'query:subscribed';
     queryId: string;
@@ -880,6 +895,34 @@ type KoraEvent = {
     type: 'store:quota-exceeded';
     dbName: string;
     message: string;
+} | {
+    /**
+     * OPFS persistence was unavailable, so the store fell back to a
+     * NON-PERSISTENT in-memory database. Anything written this session is lost
+     * on reload. This is emitted instead of failing silently so the condition
+     * is observable rather than a data-loss trap.
+     */
+    type: 'store:opfs-unavailable';
+    dbName: string;
+    /**
+     * Why OPFS could not be used. `lock-conflict` means another runtime on this
+     * origin already holds the OPFS pool for this database; `timeout` means the
+     * VFS install did not complete in time (common in headless CI); `unsupported`
+     * means the runtime has no usable OPFS.
+     */
+    reason: 'lock-conflict' | 'timeout' | 'unsupported';
+    message: string;
+} | {
+    /**
+     * Another runtime on this origin was already using this database name, so
+     * this runtime attached to it as a follower and now SHARES that one
+     * database. That is intended for multiple tabs of the SAME app; it is a bug
+     * if these are logically separate apps, which should each use a distinct
+     * store name (`store: { name: '...' }`) to stay isolated.
+     */
+    type: 'store:db-name-collision';
+    dbName: string;
+    message: string;
 } | {
     type: 'replay:completed';
     targetOperationId: string;

package/dist/{events-CqDkr1LN.d.ts → events-DhblJN4r.d.ts}

@@ -826,6 +826,21 @@ type KoraEvent = {
     code: string;
     message: string;
     retriable: boolean;
+} | {
+    /**
+     * The server rejected one of THIS client's outbound operations before it
+     * became authoritative. The op has been diverted out of the pending sync
+     * queue into the durable rejected store (kept, not retried); the app can
+     * surface the reason and decide whether to roll back the optimistic local
+     * write or let the user edit and resubmit.
+     */
+    type: 'sync:operation-rejected';
+    operationId: string;
+    collection: string;
+    recordId: string;
+    code: string;
+    message: string;
+    retriable: boolean;
 } | {
     type: 'query:subscribed';
     queryId: string;
@@ -880,6 +895,34 @@ type KoraEvent = {
     type: 'store:quota-exceeded';
     dbName: string;
     message: string;
+} | {
+    /**
+     * OPFS persistence was unavailable, so the store fell back to a
+     * NON-PERSISTENT in-memory database. Anything written this session is lost
+     * on reload. This is emitted instead of failing silently so the condition
+     * is observable rather than a data-loss trap.
+     */
+    type: 'store:opfs-unavailable';
+    dbName: string;
+    /**
+     * Why OPFS could not be used. `lock-conflict` means another runtime on this
+     * origin already holds the OPFS pool for this database; `timeout` means the
+     * VFS install did not complete in time (common in headless CI); `unsupported`
+     * means the runtime has no usable OPFS.
+     */
+    reason: 'lock-conflict' | 'timeout' | 'unsupported';
+    message: string;
+} | {
+    /**
+     * Another runtime on this origin was already using this database name, so
+     * this runtime attached to it as a follower and now SHARES that one
+     * database. That is intended for multiple tabs of the SAME app; it is a bug
+     * if these are logically separate apps, which should each use a distinct
+     * store name (`store: { name: '...' }`) to stay isolated.
+     */
+    type: 'store:db-name-collision';
+    dbName: string;
+    message: string;
 } | {
     type: 'replay:completed';
     targetOperationId: string;

package/dist/index.cjs

@@ -94,6 +94,7 @@ __export(index_exports, {
   migrate: () => migrate,
   migrationStepsToSQL: () => migrationStepsToSQL,
   op: () => op,
+  quoteIdent: () => quoteIdent,
   resolveAtomicOp: () => resolveAtomicOp,
   revealSecret: () => revealSecret,
   rollbackStepsToSQL: () => rollbackStepsToSQL,
@@ -1310,7 +1311,7 @@ function validateStateMachineDefinition(collectionName, sm, fields) {
 }
 
 // src/schema/define.ts
-var COLLECTION_NAME_RE = /^[a-z][a-z0-9_]*$/;
+var COLLECTION_NAME_RE = /^[A-Za-z][A-Za-z0-9_]*$/;
 var FIELD_NAME_RE = /^[a-z][a-zA-Z0-9_]*$/;
 var RESERVED_FIELDS = /* @__PURE__ */ new Set(["id", "_created_at", "_updated_at", "_version", "_deleted"]);
 function defineSchema(input) {
@@ -1375,7 +1376,7 @@ function validateVersion(version) {
 function validateCollectionName(name) {
   if (!COLLECTION_NAME_RE.test(name)) {
     throw new SchemaValidationError(
-      `Collection name "${name}" is invalid. Must be lowercase, start with a letter, and contain only letters, numbers, and underscores.`,
+      `Collection name "${name}" is invalid. Must start with a letter and contain only letters, numbers, and underscores.`,
       { collection: name }
     );
   }
@@ -1567,6 +1568,11 @@ function validateRelation(name, relation, collections) {
   }
 }
 
+// src/schema/quote-ident.ts
+function quoteIdent(name) {
+  return `"${name.replace(/"/g, '""')}"`;
+}
+
 // src/schema/sql-gen.ts
 function generateSQL(collectionName, collection, relations) {
   const statements = [];
@@ -1578,7 +1584,7 @@ function generateSQL(collectionName, collection, relations) {
     if (relations) {
       for (const rel of Object.values(relations)) {
         if (rel.from === collectionName && rel.field === fieldName) {
-          colDef += ` REFERENCES ${rel.to}(id)`;
+          colDef += ` REFERENCES ${quoteIdent(rel.to)}(id)`;
           fkFields.push(fieldName);
           break;
         }
@@ -1591,36 +1597,40 @@ function generateSQL(collectionName, collection, relations) {
   columns.push("_version TEXT NOT NULL DEFAULT ''");
   columns.push("_field_versions TEXT NOT NULL DEFAULT '{}'");
   columns.push("_deleted INTEGER NOT NULL DEFAULT 0");
-  statements.push(`CREATE TABLE IF NOT EXISTS ${collectionName} (
+  statements.push(
+    `CREATE TABLE IF NOT EXISTS ${quoteIdent(collectionName)} (
   ${columns.join(",\n  ")}
-)`);
+)`
+  );
   for (const [fieldName, descriptor] of Object.entries(collection.fields)) {
     const colDef = columnDefinition(fieldName, descriptor);
-    statements.push(`--kora:safe-alter
-ALTER TABLE ${collectionName} ADD COLUMN ${colDef}`);
+    statements.push(
+      `--kora:safe-alter
+ALTER TABLE ${quoteIdent(collectionName)} ADD COLUMN ${colDef}`
+    );
   }
   statements.push(
     `--kora:safe-alter
-ALTER TABLE ${collectionName} ADD COLUMN _version TEXT NOT NULL DEFAULT ''`
+ALTER TABLE ${quoteIdent(collectionName)} ADD COLUMN _version TEXT NOT NULL DEFAULT ''`
   );
   statements.push(
     `--kora:safe-alter
-ALTER TABLE ${collectionName} ADD COLUMN _field_versions TEXT NOT NULL DEFAULT '{}'`
+ALTER TABLE ${quoteIdent(collectionName)} ADD COLUMN _field_versions TEXT NOT NULL DEFAULT '{}'`
   );
   for (const indexField of collection.indexes) {
     statements.push(
-      `CREATE INDEX IF NOT EXISTS idx_${collectionName}_${indexField} ON ${collectionName} (${indexField})`
+      `CREATE INDEX IF NOT EXISTS idx_${collectionName}_${indexField} ON ${quoteIdent(collectionName)} (${quoteIdent(indexField)})`
     );
   }
   for (const fkField of fkFields) {
     if (!indexedFields.has(fkField)) {
       statements.push(
-        `CREATE INDEX IF NOT EXISTS idx_${collectionName}_${fkField} ON ${collectionName} (${fkField})`
+        `CREATE INDEX IF NOT EXISTS idx_${collectionName}_${fkField} ON ${quoteIdent(collectionName)} (${quoteIdent(fkField)})`
       );
     }
   }
   statements.push(
-    `CREATE TABLE IF NOT EXISTS _kora_ops_${collectionName} (
+    `CREATE TABLE IF NOT EXISTS ${quoteIdent(`_kora_ops_${collectionName}`)} (
   id TEXT PRIMARY KEY NOT NULL,
   node_id TEXT NOT NULL,
   type TEXT NOT NULL,
@@ -1634,7 +1644,7 @@ ALTER TABLE ${collectionName} ADD COLUMN _field_versions TEXT NOT NULL DEFAULT '
 )`
   );
   statements.push(
-    `CREATE INDEX IF NOT EXISTS idx_kora_ops_${collectionName}_record_id ON _kora_ops_${collectionName} (record_id)`
+    `CREATE INDEX IF NOT EXISTS idx_kora_ops_${collectionName}_record_id ON ${quoteIdent(`_kora_ops_${collectionName}`)} (record_id)`
   );
   return statements;
 }
@@ -1652,6 +1662,9 @@ function generateFullDDL(schema) {
   statements.push(
     "CREATE TABLE IF NOT EXISTS _kora_sync_queue (\n  id TEXT PRIMARY KEY NOT NULL,\n  payload TEXT NOT NULL\n)"
   );
+  statements.push(
+    "CREATE TABLE IF NOT EXISTS _kora_sync_rejected (\n  operation_id TEXT PRIMARY KEY NOT NULL,\n  collection TEXT NOT NULL,\n  record_id TEXT NOT NULL,\n  code TEXT NOT NULL,\n  message TEXT NOT NULL,\n  retriable INTEGER NOT NULL,\n  rejected_at INTEGER NOT NULL\n)"
+  );
   statements.push(
     "CREATE TABLE IF NOT EXISTS _kora_audit_traces (\n  id TEXT PRIMARY KEY NOT NULL,\n  recorded_at INTEGER NOT NULL,\n  event_type TEXT NOT NULL,\n  collection TEXT NOT NULL,\n  record_id TEXT NOT NULL,\n  field TEXT NOT NULL,\n  strategy TEXT NOT NULL,\n  tier INTEGER NOT NULL,\n  constraint_name TEXT,\n  trace_json TEXT NOT NULL\n)"
   );
@@ -1668,7 +1681,7 @@ function generateFullDDL(schema) {
 }
 function columnDefinition(fieldName, descriptor) {
   const sqlType = mapFieldType(descriptor);
-  const parts = [fieldName, sqlType];
+  const parts = [quoteIdent(fieldName), sqlType];
   if (descriptor.required && descriptor.defaultValue === void 0 && !descriptor.auto) {
     parts.push("NOT NULL");
   }
@@ -1677,7 +1690,7 @@ function columnDefinition(fieldName, descriptor) {
   }
   if (descriptor.kind === "enum" && descriptor.enumValues) {
     const values = descriptor.enumValues.map((v) => `'${v}'`).join(", ");
-    parts.push(`CHECK (${fieldName} IN (${values}))`);
+    parts.push(`CHECK (${quoteIdent(fieldName)} IN (${values}))`);
   }
   return parts.join(" ");
 }
@@ -2932,14 +2945,18 @@ function migrationStepsToSQL(steps) {
         statements.push(addFieldSQL(step.collection, step.field, step.descriptor));
         break;
       case "removeField":
-        statements.push(`ALTER TABLE ${step.collection} DROP COLUMN ${step.field}`);
+        statements.push(
+          `ALTER TABLE ${quoteIdent(step.collection)} DROP COLUMN ${quoteIdent(step.field)}`
+        );
         break;
       case "renameField":
-        statements.push(`ALTER TABLE ${step.collection} RENAME COLUMN ${step.from} TO ${step.to}`);
+        statements.push(
+          `ALTER TABLE ${quoteIdent(step.collection)} RENAME COLUMN ${quoteIdent(step.from)} TO ${quoteIdent(step.to)}`
+        );
         break;
       case "addIndex":
         statements.push(
-          `CREATE INDEX IF NOT EXISTS idx_${step.collection}_${step.field} ON ${step.collection} (${step.field})`
+          `CREATE INDEX IF NOT EXISTS idx_${step.collection}_${step.field} ON ${quoteIdent(step.collection)} (${quoteIdent(step.field)})`
         );
         break;
       case "removeIndex":
@@ -2957,13 +2974,13 @@ function rollbackStepsToSQL(migration) {
 }
 function addFieldSQL(collection, field, descriptor) {
   const sqlType = mapFieldType2(descriptor);
-  const parts = [`ALTER TABLE ${collection} ADD COLUMN ${field}`, sqlType];
+  const parts = [`ALTER TABLE ${quoteIdent(collection)} ADD COLUMN ${quoteIdent(field)}`, sqlType];
   if (descriptor.defaultValue !== void 0) {
     parts.push(`DEFAULT ${sqlDefault2(descriptor.defaultValue)}`);
   }
   if (descriptor.kind === "enum" && descriptor.enumValues) {
     const values = descriptor.enumValues.map((v) => `'${v}'`).join(", ");
-    parts.push(`CHECK (${field} IN (${values}))`);
+    parts.push(`CHECK (${quoteIdent(field)} IN (${values}))`);
   }
   return parts.join(" ");
 }
@@ -3314,6 +3331,7 @@ function generateProtoDefinitions(schema) {
   migrate,
   migrationStepsToSQL,
   op,
+  quoteIdent,
   resolveAtomicOp,
   revealSecret,
   rollbackStepsToSQL,