@catalyst-cloud/schema 0.1.19 → 0.1.21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@catalyst-cloud/schema",
-  "version": "0.1.19",
+  "version": "0.1.21",
   "type": "module",
   "description": "Typed Drizzle schema = single source of truth for the per-tenant Mirror DO SQLite store (CTC-13 / ADR-0002). Shared by the mirror Worker, the host-sync replica, and the browser OPFS replica.",
   "license": "MIT",

package/src/events/registry.ts

@@ -1600,6 +1600,13 @@ export const durableEventTypes = {
     wakes: false,
     entity: "ticket",
   },
+  "relay.phase.completed": {
+    rationale:
+      "CTC-985 — a relay-done.json receipt materialized a phase completion in the tenant DO's own relay_phase_state. ADR-0039 durable-fact example ('a phase started, completed, failed, or was preempted'), and the one fact that can create NEW eligible work for a ticket, so the dispatcher cohort must wake on it the same way fence.claimed and phase.dispatch.requested already do.",
+    ingestVerb: "artifacts.event",
+    wakes: { consumerClass: "dispatcher" },
+    entity: "ticket",
+  },
   "session.ended": {
     rationale: "An agent session ended — a real lifecycle boundary fact.",
     ingestVerb: "coordination.publish",
@@ -1937,10 +1944,6 @@ export const telemetryEventTypes = {
     reason:
       "Part of the recovery.skipped.* family: The bare fallback skip marker with no reason suffix (1 line) — same family.",
   },
-  "recovery.skipped.already-shipped-pr-3750-merged-head-matches-origin-main": {
-    reason:
-      "Part of the recovery.skipped.* family: Same free-text-reason-in-type defect as the pr-1071 sibling, for a different PR — flagged for phase-3.",
-  },
   "recovery.skipped.artifact-not-gate-visible": {
     reason:
       "Part of the recovery.skipped.* family: A recurring pre-condition check ('is the artifact visible to the gate yet') re-logged on every sweep pass until it resolves — 14,710 lines/mo.",
@@ -1965,10 +1968,6 @@ export const telemetryEventTypes = {
     reason:
       "Part of the recovery.skipped.* family: A recurring pre-condition check re-logged every sweep pass — 14,699 lines/mo.",
   },
-  "recovery.skipped.pr-1071-blocked-by-a-pre-existing-repo-wide-ci-failure-on-main-n": {
-    reason:
-      "Part of the recovery.skipped.* family: ⚠️ The clearest evidence of the family's defect: a full free-text incident explanation baked directly into the event NAME (confirmed against the raw log line) and re-logged 3,960 times for one PR. This is a producer bug flagged for the phase-3 inventory — the reason belongs in payload, never in `type`.",
-  },
   "recovery.skipped.prior-artifact-missing": {
     reason:
       "Part of the recovery.skipped.* family: A recurring pre-condition check re-logged every sweep pass — 35,141 lines/mo, the largest of the skip-reason family.",
@@ -1977,10 +1976,6 @@ export const telemetryEventTypes = {
     reason:
       "Part of the recovery.skipped.* family: A recurring pre-condition check re-logged every sweep pass — 28,906 lines/mo.",
   },
-  "recovery.skipped.source-conflict-ctl708-unavailable": {
-    reason:
-      "Part of the recovery.skipped.* family: Same free-text-reason-in-type defect, embedding a specific ticket's conflict detail in the type — flagged for phase-3.",
-  },
   "recovery.skipped.ticket-already-merged-pipeline-regressed": {
     reason:
       "Part of the recovery.skipped.* family: A recurring pre-condition check re-logged every sweep pass — 14,868 lines/mo.",

package/src/events/retention.ts

@@ -76,7 +76,19 @@ export function eventArchiveKey(coords: EventArchiveCoords): string {
   // is well-formed. Re-deriving the yyyy-mm-dd prefix from the parsed Date and requiring it to match
   // the input's own date prefix rejects that rollover instead of misfiling into a wrong partition.
   const roundTripDatePrefix = occurredAt.slice(0, 10);
-  const parsedDatePrefix = new Date(occurredAtMs).toISOString().slice(0, 10);
+  // Codex P2 (PR #1250): derive the parsed prefix in the TIMESTAMP'S OWN offset, not UTC — a
+  // valid instant like 2026-08-24T23:30:00-02:00 is already Aug 25 in UTC, so a UTC comparison
+  // rejects a well-formed date the regex deliberately accepts. The rollover check still fires:
+  // an out-of-range date (2026-02-30) rolls forward and lands on a different calendar date in
+  // ANY fixed offset.
+  const offsetMatch = /(?:Z|([+-])(\d{2}):(\d{2}))$/.exec(occurredAt);
+  const offsetMs =
+    offsetMatch && offsetMatch[1]
+      ? (offsetMatch[1] === "-" ? -1 : 1) *
+        (Number(offsetMatch[2]) * 3600 + Number(offsetMatch[3]) * 60) *
+        1000
+      : 0;
+  const parsedDatePrefix = new Date(occurredAtMs + offsetMs).toISOString().slice(0, 10);
   if (roundTripDatePrefix !== parsedDatePrefix) {
     throw new Error(
       `eventArchiveKey: occurredAt ${JSON.stringify(occurredAt)} is not a valid date`,

package/src/migrations.generated.ts

@@ -214,6 +214,13 @@ export const MIRROR_MIGRATIONS = {
         tag: "0028_burly_nemesis",
         breakpoints: true,
       },
+      {
+        idx: 29,
+        version: "6",
+        when: 1787615021252,
+        tag: "0029_square_silhouette",
+        breakpoints: true,
+      },
     ],
   },
   migrations: {
@@ -272,5 +279,7 @@ export const MIRROR_MIGRATIONS = {
     "0027_closed_miek":
       "CREATE TABLE `check_suites` (\n\t`repo_id` text NOT NULL,\n\t`check_suite_id` text PRIMARY KEY NOT NULL,\n\t`head_sha` text,\n\t`head_branch` text,\n\t`status` text,\n\t`conclusion` text,\n\t`app_slug` text,\n\t`latest_check_runs_count` integer,\n\t`updated_at` integer\n);\n--> statement-breakpoint\nCREATE INDEX `idx_check_suites_sha` ON `check_suites` (`head_sha`);--> statement-breakpoint\nCREATE TABLE `push_events` (\n\t`delivery_id` text PRIMARY KEY NOT NULL,\n\t`repo_id` text NOT NULL,\n\t`ref` text NOT NULL,\n\t`before` text,\n\t`after` text,\n\t`forced` integer,\n\t`created` integer,\n\t`deleted` integer,\n\t`base_ref` text,\n\t`pusher_id` text,\n\t`head_commit_sha` text,\n\t`updated_at` integer\n);\n--> statement-breakpoint\nCREATE INDEX `idx_push_events_repo_ref` ON `push_events` (`repo_id`,`ref`,`updated_at`);--> statement-breakpoint\nALTER TABLE `pull_requests` ADD `merge_commit_sha` text;",
     "0028_burly_nemesis": "ALTER TABLE `check_suites` ADD `pull_request_numbers` text;",
+    "0029_square_silhouette":
+      "CREATE INDEX `idx_relations_type_related` ON `relations` (`type`,`related_identifier`);",
   },
 } as const;

package/src/mirror.ts

@@ -148,7 +148,18 @@ export const relations = sqliteTable(
     related_identifier: text("related_identifier"),
     updated_at: integer("updated_at"),
   },
-  (t) => [index("idx_relations_issue").on(t.issue_identifier)],
+  (t) => [
+    index("idx_relations_issue").on(t.issue_identifier),
+    // CTC-990: serves evaluateBlockers' incoming-`blocks` lookup (work-eligibility.ts) —
+    // `WHERE type = 'blocks' AND related_identifier = ?`, run inside /work/acquire's
+    // transactionSync CAS burst, so it cannot afford a full scan. Composite (not single-column)
+    // because the predicate always constrains both columns with equality — measured to let SQLite
+    // seek on both. `relations` is a Drizzle-mirrored table (replicates to every host/browser
+    // replica), so its indexes belong here, in the SSOT, never as a DO-side
+    // `CREATE INDEX IF NOT EXISTS` — a DO-side index on a mirrored table would be invisible to the
+    // daemon/browser replicas and to this file's own fidelity tests (ADR-0002).
+    index("idx_relations_type_related").on(t.type, t.related_identifier),
+  ],
 );
 
 // Linear IssueHistory events (CTC-14) — the per-issue change timeline. Captured INLINE on the issues