@cosmicdrift/kumiko-framework 0.270.0 → 0.273.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (104) hide show
  1. package/package.json +3 -3
  2. package/src/__tests__/fixtures/migrate-cross-tenant/manual-only.expected.ts +44 -0
  3. package/src/__tests__/fixtures/migrate-cross-tenant/manual-only.input.ts +44 -0
  4. package/src/__tests__/fixtures/migrate-cross-tenant/rewrite.expected.ts +41 -0
  5. package/src/__tests__/fixtures/migrate-cross-tenant/rewrite.input.ts +41 -0
  6. package/src/__tests__/migrate-cross-tenant-codemod.test.ts +106 -0
  7. package/src/__tests__/related-list-parent-filter.integration.test.ts +103 -0
  8. package/src/api/__tests__/nested-write-ownership.integration.test.ts +268 -0
  9. package/src/api/server.ts +7 -20
  10. package/src/bun-db/__tests__/query-guards.test.ts +60 -3
  11. package/src/bun-db/query.ts +24 -21
  12. package/src/changes.json +91 -0
  13. package/src/db/__tests__/event-store-executor-write-verbs.integration.test.ts +77 -0
  14. package/src/db/__tests__/event-store-executor.integration.test.ts +1 -1
  15. package/src/db/__tests__/replay-migration-sql.test.ts +17 -0
  16. package/src/db/__tests__/tenant-db-convention-cross-tenant.test.ts +72 -0
  17. package/src/db/__tests__/tenant-db-declared-unsafe-raw.test.ts +104 -0
  18. package/src/db/__tests__/tenant-db-global.test.ts +92 -0
  19. package/src/db/__tests__/tenant-db-no-raw.test.ts +75 -0
  20. package/src/db/__tests__/tenant-db.integration.test.ts +3 -1
  21. package/src/db/__tests__/unchecked-system-db.test.ts +76 -0
  22. package/src/db/apply-entity-event.ts +1 -1
  23. package/src/db/assert-exists-in.ts +1 -1
  24. package/src/db/event-store-executor-context.ts +5 -2
  25. package/src/db/event-store-executor-read.ts +6 -4
  26. package/src/db/event-store-executor-write.ts +24 -18
  27. package/src/db/queries/shadow-swap.ts +36 -0
  28. package/src/db/replay-migration-sql.ts +5 -5
  29. package/src/db/tenant-db-runner.ts +23 -0
  30. package/src/db/tenant-db.ts +97 -24
  31. package/src/derivatives/__tests__/derivatives-context.test.ts +0 -1
  32. package/src/engine/__tests__/boot-validator-nav-area.test.ts +116 -0
  33. package/src/engine/__tests__/boot-validator-projection-list.test.ts +91 -0
  34. package/src/engine/__tests__/boot-validator-query-output-schema.test.ts +143 -0
  35. package/src/engine/__tests__/boot-validator.test.ts +51 -0
  36. package/src/engine/__tests__/entity-create-caller-id.integration.test.ts +87 -0
  37. package/src/engine/__tests__/entity-handlers-escape-hatch.test.ts +91 -0
  38. package/src/engine/__tests__/entity-handlers.test.ts +28 -0
  39. package/src/engine/__tests__/entity-restore-tenant-isolation.integration.test.ts +30 -5
  40. package/src/engine/__tests__/entity-write-crosstenant.integration.test.ts +206 -47
  41. package/src/engine/__tests__/read-steps-tenant-filter.integration.test.ts +215 -0
  42. package/src/engine/__tests__/required-surface-keys.test.ts +16 -0
  43. package/src/engine/__tests__/screen.test.ts +150 -0
  44. package/src/engine/__tests__/soft-delete-cleanup.test.ts +0 -1
  45. package/src/engine/__tests__/steps-read.test.ts +204 -8
  46. package/src/engine/__tests__/steps-unsafe-projection-delete.test.ts +7 -4
  47. package/src/engine/__tests__/steps-unsafe-projection-upsert.test.ts +6 -3
  48. package/src/engine/boot-validator/__tests__/global-tenancy.test.ts +16 -1
  49. package/src/engine/boot-validator/__tests__/rate-limit-disabled-reason.test.ts +45 -0
  50. package/src/engine/boot-validator/entity-handler.ts +21 -0
  51. package/src/engine/boot-validator/global-tenancy.ts +13 -1
  52. package/src/engine/boot-validator/parent-ref.ts +1 -1
  53. package/src/engine/boot-validator/projection-list-screens.ts +67 -2
  54. package/src/engine/boot-validator/query-output-columns.ts +33 -0
  55. package/src/engine/boot-validator/screens.ts +154 -31
  56. package/src/engine/build-config-feature-schema.ts +7 -4
  57. package/src/engine/entity-handlers.ts +71 -19
  58. package/src/engine/feature-entity-handlers.ts +4 -4
  59. package/src/engine/index.ts +5 -0
  60. package/src/engine/registry-validate.ts +12 -4
  61. package/src/engine/schema-builder.ts +6 -0
  62. package/src/engine/screen-helpers.ts +52 -0
  63. package/src/engine/steps/_read-source.ts +12 -0
  64. package/src/engine/steps/read-find-many.ts +8 -2
  65. package/src/engine/steps/read-find-one.ts +11 -8
  66. package/src/engine/steps/unsafe-projection-delete.ts +2 -1
  67. package/src/engine/steps/unsafe-projection-upsert.ts +2 -1
  68. package/src/engine/types/index.ts +8 -1
  69. package/src/errors/i18n/de.yaml +14 -0
  70. package/src/errors/i18n/en.yaml +13 -0
  71. package/src/errors/reasons.ts +4 -0
  72. package/src/i18n/required-surface-keys.ts +6 -2
  73. package/src/jobs/__tests__/job-systemdb.integration.test.ts +5 -4
  74. package/src/jobs/__tests__/jobs.integration.test.ts +41 -0
  75. package/src/jobs/job-runner.ts +35 -5
  76. package/src/pipeline/__tests__/escape-hatch-audit.integration.test.ts +254 -0
  77. package/src/pipeline/__tests__/escape-hatch-report.test.ts +149 -0
  78. package/src/pipeline/__tests__/msp-rebuild.integration.test.ts +28 -0
  79. package/src/pipeline/__tests__/projection-rebuild.integration.test.ts +94 -0
  80. package/src/pipeline/__tests__/projections-runner-systemscope.integration.test.ts +101 -0
  81. package/src/pipeline/__tests__/query-projection.integration.test.ts +169 -14
  82. package/src/pipeline/__tests__/rate-limit-default.integration.test.ts +118 -0
  83. package/src/pipeline/__tests__/rate-limit-default.test.ts +117 -0
  84. package/src/pipeline/__tests__/search-consumer-named-events.integration.test.ts +5 -2
  85. package/src/pipeline/__tests__/system-identity-switch.test.ts +240 -1
  86. package/src/pipeline/dispatch-query.ts +27 -8
  87. package/src/pipeline/dispatch-shared.ts +141 -57
  88. package/src/pipeline/dispatch-stream.ts +1 -3
  89. package/src/pipeline/dispatch-write.ts +62 -14
  90. package/src/pipeline/dispatcher.ts +20 -0
  91. package/src/pipeline/escape-hatch-report.ts +127 -0
  92. package/src/pipeline/index.ts +8 -0
  93. package/src/pipeline/member-read-only-transaction.ts +75 -0
  94. package/src/pipeline/msp-rebuild.ts +2 -0
  95. package/src/pipeline/projection-rebuild.ts +2 -0
  96. package/src/pipeline/projections-runner.ts +15 -13
  97. package/src/pipeline/system-identity-switch.ts +121 -9
  98. package/src/scripts/codemod/migrate-cross-tenant.ts +335 -0
  99. package/src/scripts/codemod/migrate-db-raw.ts +19 -6
  100. package/src/scripts/codemod/migrate-open-to-all.ts +2 -1
  101. package/src/ui-types/index.ts +3 -0
  102. package/src/utils/__tests__/ids.test.ts +29 -1
  103. package/src/utils/ids.ts +10 -1
  104. package/src/utils/index.ts +1 -1
@@ -0,0 +1,254 @@
1
+ // fw#2861 — every escape-hatch use reports through AppContext._escapeHatchAuditSink
2
+ // (or warn-logs when no sink is wired). Real HTTP calls + setupTestStack — never
3
+ // createTestDispatcher. Modelled on unsafe-raw-escape-hatch.integration.test.ts and
4
+ // system-identity-switch.integration.test.ts.
5
+
6
+ import { afterAll, beforeAll, describe, expect, test } from "bun:test";
7
+ import { z } from "zod";
8
+ import { defineUnmanagedTable } from "../../db/entity-table-meta";
9
+ import { executeRawQuery } from "../../db/queries/raw-sql";
10
+ import { createSystemUser, defineFeature } from "../../engine";
11
+ import type { EscapeHatchUseEvent } from "../../engine/types";
12
+ import type { Logger } from "../../logging/types";
13
+ import { createTestUser, setupTestStack, type TestStack } from "../../stack";
14
+ import { ESCAPE_HATCH_USED_SIGNAL } from "../escape-hatch-report";
15
+
16
+ const globalStoreTable = defineUnmanagedTable({
17
+ tableName: "store_fw2861_escape_hatch_audit_items",
18
+ tenancy: "global",
19
+ columns: [
20
+ {
21
+ name: "id",
22
+ pgType: "uuid",
23
+ notNull: true,
24
+ primaryKey: true,
25
+ defaultSql: "gen_random_uuid()",
26
+ },
27
+ { name: "note", pgType: "text", notNull: true },
28
+ ],
29
+ });
30
+
31
+ const UNSAFE_RAW_REASON = "fw#2861 integration test — declared unsafeRaw write";
32
+ const GLOBAL_WRITE_REASON = "fw#2861 integration test — declared cross-tenant write";
33
+ const IDENTITY_SWITCH_REASON = "fw#2861 integration test — needs SYSTEM to look itself up";
34
+ const ACKNOWLEDGE_REASON = "fw#2861 integration test — system-wide scan";
35
+
36
+ const probeFeature = defineFeature("escape-hatch-audit-probe", (r) => {
37
+ r.storeTable(globalStoreTable, {
38
+ reason: "fw#2861 integration test — cross-tenant store table for db.global()",
39
+ });
40
+
41
+ r.queryHandler(
42
+ "whoami",
43
+ z.object({}),
44
+ async (query) => ({ id: query.user.id, tenantId: query.user.tenantId }),
45
+ { access: { roles: ["system", "User"] } },
46
+ );
47
+
48
+ r.writeHandler(
49
+ "unsafe-raw-write",
50
+ z.object({}),
51
+ async (_event, ctx) => {
52
+ const runner = ctx.db.unsafeRaw(UNSAFE_RAW_REASON);
53
+ const rows = await executeRawQuery<{ one: number }>(runner, "SELECT 1 AS one");
54
+ return { isSuccess: true as const, data: { one: rows[0]?.one } };
55
+ },
56
+ { access: { roles: ["User"] }, escapeHatch: { reason: UNSAFE_RAW_REASON } },
57
+ );
58
+
59
+ r.writeHandler(
60
+ "global-write",
61
+ z.object({}),
62
+ async (_event, ctx) => {
63
+ const row = await ctx.db.global(globalStoreTable).insertOne<{ id: string }>({
64
+ note: "hi",
65
+ });
66
+ return { isSuccess: true as const, data: row };
67
+ },
68
+ { access: { roles: ["User"] }, escapeHatch: { reason: GLOBAL_WRITE_REASON } },
69
+ );
70
+
71
+ r.writeHandler(
72
+ "identity-switch-write",
73
+ z.object({}),
74
+ async (event, ctx) =>
75
+ ctx
76
+ .queryAs(createSystemUser(event.user.tenantId), "escape-hatch-audit-probe:query:whoami", {})
77
+ .then((data) => ({ isSuccess: true as const, data })),
78
+ { access: { roles: ["User"] }, escapeHatch: { reason: IDENTITY_SWITCH_REASON } },
79
+ );
80
+
81
+ r.writeHandler(
82
+ "identity-switch-self-write",
83
+ z.object({}),
84
+ async (event, ctx) =>
85
+ ctx
86
+ .queryAs(event.user, "escape-hatch-audit-probe:query:whoami", {})
87
+ .then((data) => ({ isSuccess: true as const, data })),
88
+ { access: { roles: ["User"] }, escapeHatch: { reason: IDENTITY_SWITCH_REASON } },
89
+ );
90
+ });
91
+
92
+ const systemScopeProbeFeature = defineFeature("escape-hatch-audit-probe-system", (r) => {
93
+ r.systemScope();
94
+
95
+ r.queryHandler(
96
+ "acknowledge-cross-tenant-query",
97
+ z.object({}),
98
+ async (_query, ctx) => {
99
+ if (!ctx.systemDb) throw new Error("expected ctx.systemDb on a systemScope() handler");
100
+ const db = ctx.systemDb.acknowledgeCrossTenant(ACKNOWLEDGE_REASON);
101
+ return { tenantId: db.tenantId };
102
+ },
103
+ { access: { roles: ["User"] } },
104
+ );
105
+ });
106
+
107
+ const user = createTestUser({
108
+ id: 93,
109
+ tenantId: "11111111-0000-4000-8000-000000000093",
110
+ roles: ["User"],
111
+ });
112
+
113
+ function recordingSink(events: EscapeHatchUseEvent[]) {
114
+ return async (event: EscapeHatchUseEvent) => {
115
+ events.push(event);
116
+ };
117
+ }
118
+
119
+ describe("escape-hatch uses report through AppContext._escapeHatchAuditSink", () => {
120
+ let stack: TestStack;
121
+ const events: EscapeHatchUseEvent[] = [];
122
+
123
+ beforeAll(async () => {
124
+ stack = await setupTestStack({
125
+ features: [probeFeature, systemScopeProbeFeature],
126
+ extraContext: { _escapeHatchAuditSink: recordingSink(events) },
127
+ });
128
+ });
129
+
130
+ afterAll(async () => {
131
+ await stack.cleanup();
132
+ });
133
+
134
+ test("unsafeRaw reports exactly one unsafe-raw event with handler/kind/reason/tenantId/actor, deduped on a repeat call within the window", async () => {
135
+ events.length = 0;
136
+ await stack.http.writeOk("escape-hatch-audit-probe:write:unsafe-raw-write", {}, user);
137
+ // Identical (handler, kind, reason, tenantId, actor) — the 60s dedup window
138
+ // must suppress this second report.
139
+ await stack.http.writeOk("escape-hatch-audit-probe:write:unsafe-raw-write", {}, user);
140
+
141
+ const matches = events.filter((e) => e.kind === "unsafe-raw");
142
+ expect(matches).toEqual([
143
+ {
144
+ handler: "escape-hatch-audit-probe:write:unsafe-raw-write",
145
+ kind: "unsafe-raw",
146
+ reason: UNSAFE_RAW_REASON,
147
+ tenantId: user.tenantId,
148
+ actor: user.id,
149
+ },
150
+ ]);
151
+ });
152
+
153
+ test("db.global() write reports exactly one global-write event with the escapeHatch's reason", async () => {
154
+ events.length = 0;
155
+ await stack.http.writeOk("escape-hatch-audit-probe:write:global-write", {}, user);
156
+
157
+ const matches = events.filter((e) => e.kind === "global-write");
158
+ expect(matches).toEqual([
159
+ {
160
+ handler: "escape-hatch-audit-probe:write:global-write",
161
+ kind: "global-write",
162
+ reason: GLOBAL_WRITE_REASON,
163
+ tenantId: user.tenantId,
164
+ actor: user.id,
165
+ },
166
+ ]);
167
+ });
168
+
169
+ test("ctx.systemDb.acknowledgeCrossTenant reports exactly one acknowledge-cross-tenant event", async () => {
170
+ events.length = 0;
171
+ await stack.http.queryOk(
172
+ "escape-hatch-audit-probe-system:query:acknowledge-cross-tenant-query",
173
+ {},
174
+ user,
175
+ );
176
+
177
+ const matches = events.filter((e) => e.kind === "acknowledge-cross-tenant");
178
+ expect(matches).toEqual([
179
+ {
180
+ handler: "escape-hatch-audit-probe-system:query:acknowledge-cross-tenant-query",
181
+ kind: "acknowledge-cross-tenant",
182
+ reason: ACKNOWLEDGE_REASON,
183
+ tenantId: user.tenantId,
184
+ actor: user.id,
185
+ },
186
+ ]);
187
+ });
188
+
189
+ test("a granted SYSTEM identity switch reports exactly one identity-switch event with the target", async () => {
190
+ events.length = 0;
191
+ await stack.http.writeOk("escape-hatch-audit-probe:write:identity-switch-write", {}, user);
192
+
193
+ const matches = events.filter((e) => e.kind === "identity-switch");
194
+ expect(matches).toEqual([
195
+ {
196
+ handler: "escape-hatch-audit-probe:write:identity-switch-write",
197
+ kind: "identity-switch",
198
+ reason: IDENTITY_SWITCH_REASON,
199
+ tenantId: user.tenantId,
200
+ actor: user.id,
201
+ target: { id: createSystemUser(user.tenantId).id, tenantId: user.tenantId },
202
+ },
203
+ ]);
204
+ });
205
+
206
+ test("self-delegation with a grant reports no identity-switch event", async () => {
207
+ events.length = 0;
208
+ await stack.http.writeOk("escape-hatch-audit-probe:write:identity-switch-self-write", {}, user);
209
+
210
+ expect(events.filter((e) => e.kind === "identity-switch")).toEqual([]);
211
+ });
212
+ });
213
+
214
+ describe("escape-hatch uses without a sink fall back to a structured warn log", () => {
215
+ let stack: TestStack;
216
+ const warnCalls: Array<{ msg: string; data?: Record<string, unknown> }> = [];
217
+
218
+ beforeAll(async () => {
219
+ const log: Logger = {
220
+ info() {},
221
+ debug() {},
222
+ warn(msg, data) {
223
+ warnCalls.push({ msg, data });
224
+ },
225
+ error() {},
226
+ child() {
227
+ return log;
228
+ },
229
+ };
230
+ stack = await setupTestStack({
231
+ features: [probeFeature, systemScopeProbeFeature],
232
+ extraContext: { log },
233
+ });
234
+ });
235
+
236
+ afterAll(async () => {
237
+ await stack.cleanup();
238
+ });
239
+
240
+ test("unsafeRaw without a sink warn-logs security:escape-hatch-used", async () => {
241
+ warnCalls.length = 0;
242
+ await stack.http.writeOk("escape-hatch-audit-probe:write:unsafe-raw-write", {}, user);
243
+
244
+ const matches = warnCalls.filter((c) => c.msg === ESCAPE_HATCH_USED_SIGNAL);
245
+ expect(matches).toHaveLength(1);
246
+ expect(matches[0]?.data).toEqual({
247
+ handler: "escape-hatch-audit-probe:write:unsafe-raw-write",
248
+ kind: "unsafe-raw",
249
+ reason: UNSAFE_RAW_REASON,
250
+ tenantId: user.tenantId,
251
+ actor: user.id,
252
+ });
253
+ });
254
+ });
@@ -0,0 +1,149 @@
1
+ import { describe, expect, test } from "bun:test";
2
+ import type { EscapeHatchUseEvent } from "../../engine/types";
3
+ import type { Logger } from "../../logging/types";
4
+ import { testTenantId } from "../../stack";
5
+ import {
6
+ createEscapeHatchReportWindow,
7
+ ESCAPE_HATCH_USED_SIGNAL,
8
+ reportEscapeHatchUse,
9
+ } from "../escape-hatch-report";
10
+
11
+ const tenantId = testTenantId(1);
12
+
13
+ function recordingLogger(): Logger & {
14
+ readonly warnCalls: Array<{ msg: string; data?: Record<string, unknown> }>;
15
+ readonly errorCalls: Array<{ msg: string; data?: Record<string, unknown> }>;
16
+ } {
17
+ const warnCalls: Array<{ msg: string; data?: Record<string, unknown> }> = [];
18
+ const errorCalls: Array<{ msg: string; data?: Record<string, unknown> }> = [];
19
+ const logger: Logger & {
20
+ warnCalls: typeof warnCalls;
21
+ errorCalls: typeof errorCalls;
22
+ } = {
23
+ warnCalls,
24
+ errorCalls,
25
+ info() {},
26
+ debug() {},
27
+ warn(msg, data) {
28
+ warnCalls.push({ msg, data });
29
+ },
30
+ error(msg, data) {
31
+ errorCalls.push({ msg, data });
32
+ },
33
+ child() {
34
+ return logger;
35
+ },
36
+ };
37
+ return logger;
38
+ }
39
+
40
+ const baseEvent: EscapeHatchUseEvent = {
41
+ handler: 'write "widgets:write:create"',
42
+ kind: "unsafe-raw",
43
+ reason: "legacy migration read",
44
+ tenantId,
45
+ actor: "user-1",
46
+ };
47
+
48
+ describe("createEscapeHatchReportWindow", () => {
49
+ test("dedups an identical key within the window", () => {
50
+ const window = createEscapeHatchReportWindow({ windowMs: 1000 });
51
+ expect(window.shouldReport("k1", 0)).toBe(true);
52
+ expect(window.shouldReport("k1", 500)).toBe(false);
53
+ expect(window.shouldReport("k1", 999)).toBe(false);
54
+ });
55
+
56
+ test("reports again after the window expires", () => {
57
+ const window = createEscapeHatchReportWindow({ windowMs: 1000 });
58
+ expect(window.shouldReport("k1", 0)).toBe(true);
59
+ expect(window.shouldReport("k1", 1000)).toBe(true);
60
+ });
61
+
62
+ test("evicts the oldest entry beyond maxEntries", () => {
63
+ const window = createEscapeHatchReportWindow({ windowMs: 1000, maxEntries: 2 });
64
+ expect(window.shouldReport("k1", 0)).toBe(true);
65
+ expect(window.shouldReport("k2", 0)).toBe(true);
66
+ // k3 pushes the cache past its cap — k1 (oldest) is evicted to make room.
67
+ expect(window.shouldReport("k3", 0)).toBe(true);
68
+ // k1 was evicted, so it is treated as new and reports again.
69
+ expect(window.shouldReport("k1", 1)).toBe(true);
70
+ // k3 is still within its window and was not evicted.
71
+ expect(window.shouldReport("k3", 1)).toBe(false);
72
+ });
73
+ });
74
+
75
+ describe("reportEscapeHatchUse", () => {
76
+ test("sink present — calls the sink with the exact event and does not log.warn", () => {
77
+ const calls: EscapeHatchUseEvent[] = [];
78
+ const sink = async (event: EscapeHatchUseEvent) => {
79
+ calls.push(event);
80
+ };
81
+ const log = recordingLogger();
82
+
83
+ reportEscapeHatchUse(baseEvent, { sink, log });
84
+
85
+ expect(calls).toEqual([baseEvent]);
86
+ expect(log.warnCalls).toEqual([]);
87
+ });
88
+
89
+ test("no sink — log.warn called once with ESCAPE_HATCH_USED_SIGNAL and exact fields", () => {
90
+ const log = recordingLogger();
91
+
92
+ reportEscapeHatchUse(baseEvent, { log });
93
+
94
+ expect(log.warnCalls.length).toBe(1);
95
+ expect(log.warnCalls[0]?.msg).toBe(ESCAPE_HATCH_USED_SIGNAL);
96
+ expect(log.warnCalls[0]?.data).toEqual({
97
+ handler: baseEvent.handler,
98
+ kind: baseEvent.kind,
99
+ reason: baseEvent.reason,
100
+ tenantId: baseEvent.tenantId,
101
+ actor: baseEvent.actor,
102
+ });
103
+ });
104
+
105
+ test("no sink, event has a target — target fields are included", () => {
106
+ const log = recordingLogger();
107
+ const eventWithTarget: EscapeHatchUseEvent = {
108
+ ...baseEvent,
109
+ kind: "identity-switch",
110
+ target: { id: "user-2", tenantId },
111
+ };
112
+
113
+ reportEscapeHatchUse(eventWithTarget, { log });
114
+
115
+ expect(log.warnCalls[0]?.data).toEqual({
116
+ handler: eventWithTarget.handler,
117
+ kind: eventWithTarget.kind,
118
+ reason: eventWithTarget.reason,
119
+ tenantId: eventWithTarget.tenantId,
120
+ actor: eventWithTarget.actor,
121
+ targetId: "user-2",
122
+ targetTenantId: tenantId,
123
+ });
124
+ });
125
+
126
+ test("sink rejection — logs via log.error", async () => {
127
+ const log = recordingLogger();
128
+ const sink = async () => {
129
+ throw new Error("sink boom");
130
+ };
131
+
132
+ reportEscapeHatchUse(baseEvent, { sink, log });
133
+ // The sink runs async — let its rejection settle before asserting.
134
+ await Promise.resolve();
135
+ await Promise.resolve();
136
+
137
+ expect(log.errorCalls.length).toBe(1);
138
+ expect(log.errorCalls[0]?.msg).toBe(`${ESCAPE_HATCH_USED_SIGNAL}: audit sink failed`);
139
+ expect(log.errorCalls[0]?.data).toEqual({
140
+ handler: baseEvent.handler,
141
+ kind: baseEvent.kind,
142
+ reason: baseEvent.reason,
143
+ tenantId: baseEvent.tenantId,
144
+ actor: baseEvent.actor,
145
+ error: "sink boom",
146
+ });
147
+ expect(log.warnCalls).toEqual([]);
148
+ });
149
+ });
@@ -364,4 +364,32 @@ describe("rebuildMultiStreamProjection — guard rails", () => {
364
364
  expect(state?.status).toBe("dead");
365
365
  expect(state?.lastError).toMatch(/appendEvent/);
366
366
  });
367
+
368
+ test("aborts when the live table has RLS enabled — data survives (fw#2907)", async () => {
369
+ const erin = "00000000-0000-4000-8000-000000000e05";
370
+ await updateMany(
371
+ stack.db,
372
+ eventConsumerStateTable,
373
+ { status: "disabled", updatedAt: sql`now()` },
374
+ { name: SAGA_MSP },
375
+ );
376
+ await stack.http.writeOk("mspreb:write:invoice:bill", { customer: erin, cents: 20_00 }, admin);
377
+ await runFullDispatcher();
378
+
379
+ await asRawClient(stack.db).unsafe(
380
+ `ALTER TABLE "read_mspreb_balance" ENABLE ROW LEVEL SECURITY`,
381
+ );
382
+ try {
383
+ await expect(
384
+ rebuildMultiStreamProjection(BALANCE_MSP, { db: stack.db, registry: stack.registry }),
385
+ ).rejects.toThrow(/row level security/);
386
+
387
+ const [row] = await selectMany(stack.db, balanceTable, { customer: erin });
388
+ expect(row).toMatchObject({ invoicesCents: 20_00 });
389
+ } finally {
390
+ await asRawClient(stack.db).unsafe(
391
+ `ALTER TABLE "read_mspreb_balance" DISABLE ROW LEVEL SECURITY`,
392
+ );
393
+ }
394
+ });
367
395
  });
@@ -938,3 +938,97 @@ describe("rebuildProjection — stale-registry column fence (#835)", () => {
938
938
  expect(await getCount(group)).toBe(1);
939
939
  });
940
940
  });
941
+
942
+ describe("rebuildProjection — row level security guard (#2907)", () => {
943
+ test("aborts when the live table has RLS enabled — data and RLS survive", async () => {
944
+ const group = "00000000-0000-4000-8000-000000000051";
945
+ await appendCreatedEvent(group, "item1");
946
+ await rebuildProjection(qualifiedProjectionName, { db: testDb.db, registry });
947
+ expect(await getCount(group)).toBe(1);
948
+
949
+ const raw = asRawClient(testDb.db);
950
+ await raw.unsafe(`ALTER TABLE "read_rebuild_items_per_group" ENABLE ROW LEVEL SECURITY`);
951
+ try {
952
+ await expect(
953
+ rebuildProjection(qualifiedProjectionName, { db: testDb.db, registry }),
954
+ ).rejects.toThrow(/read_rebuild_items_per_group.*has row level security/);
955
+
956
+ // Live table untouched: row count AND the RLS flag itself survive.
957
+ expect(await getCount(group)).toBe(1);
958
+ const [pgClass] = await raw.unsafe<{ relrowsecurity: boolean }>(
959
+ `SELECT relrowsecurity FROM pg_class WHERE relname = 'read_rebuild_items_per_group'`,
960
+ );
961
+ expect(pgClass?.relrowsecurity).toBe(true);
962
+
963
+ const state = await getProjectionState(testDb.db, qualifiedProjectionName);
964
+ expect(state?.status).toBe("failed");
965
+ } finally {
966
+ await raw.unsafe(`ALTER TABLE "read_rebuild_items_per_group" DISABLE ROW LEVEL SECURITY`);
967
+ }
968
+
969
+ // RLS cleared again → rebuild works.
970
+ const result = await rebuildProjection(qualifiedProjectionName, { db: testDb.db, registry });
971
+ expect(result.eventsProcessed).toBe(1);
972
+ expect(await getCount(group)).toBe(1);
973
+ });
974
+
975
+ test("aborts on a policy alone, without RLS enabled — policy count reported and survives", async () => {
976
+ const group = "00000000-0000-4000-8000-000000000052";
977
+ await appendCreatedEvent(group, "item1");
978
+ await rebuildProjection(qualifiedProjectionName, { db: testDb.db, registry });
979
+
980
+ const raw = asRawClient(testDb.db);
981
+ await raw.unsafe(
982
+ `CREATE POLICY rls_guard_probe ON "read_rebuild_items_per_group" USING (true)`,
983
+ );
984
+ try {
985
+ await expect(
986
+ rebuildProjection(qualifiedProjectionName, { db: testDb.db, registry }),
987
+ ).rejects.toThrow(/policies: 1/);
988
+
989
+ const [policyCount] = await raw.unsafe<{ n: number }>(
990
+ `SELECT count(*)::int AS n FROM pg_policy p
991
+ JOIN pg_class c ON c.oid = p.polrelid
992
+ WHERE c.relname = 'read_rebuild_items_per_group'`,
993
+ );
994
+ expect(policyCount?.n).toBe(1);
995
+ } finally {
996
+ await raw.unsafe(`DROP POLICY IF EXISTS rls_guard_probe ON "read_rebuild_items_per_group"`);
997
+ }
998
+ });
999
+
1000
+ test("aborts on FORCE ROW LEVEL SECURITY alone, without ENABLE", async () => {
1001
+ const raw = asRawClient(testDb.db);
1002
+ await raw.unsafe(`ALTER TABLE "read_rebuild_items_per_group" FORCE ROW LEVEL SECURITY`);
1003
+ try {
1004
+ await expect(
1005
+ rebuildProjection(qualifiedProjectionName, { db: testDb.db, registry }),
1006
+ ).rejects.toThrow(/row level security/);
1007
+ } finally {
1008
+ await raw.unsafe(`ALTER TABLE "read_rebuild_items_per_group" NO FORCE ROW LEVEL SECURITY`);
1009
+ await raw.unsafe(`ALTER TABLE "read_rebuild_items_per_group" DISABLE ROW LEVEL SECURITY`);
1010
+ }
1011
+ });
1012
+
1013
+ test("swapShadowIntoLive rejects directly when RLS is enabled — live table survives", async () => {
1014
+ const group = "00000000-0000-4000-8000-000000000053";
1015
+ await appendCreatedEvent(group, "item1");
1016
+ await rebuildProjection(qualifiedProjectionName, { db: testDb.db, registry });
1017
+ expect(await getCount(group)).toBe(1);
1018
+
1019
+ const raw = asRawClient(testDb.db);
1020
+ await raw.unsafe(`ALTER TABLE "read_rebuild_items_per_group" ENABLE ROW LEVEL SECURITY`);
1021
+ try {
1022
+ const db = testDb.db as DbConnection; // @cast-boundary test-harness (TestDb.db is intentionally unknown)
1023
+ await expect(
1024
+ db.begin(async (tx: DbTx) => {
1025
+ await swapShadowIntoLive(tx, "read_rebuild_items_per_group");
1026
+ }),
1027
+ ).rejects.toThrow(/shadow swap:.*row level security/);
1028
+
1029
+ expect(await getCount(group)).toBe(1);
1030
+ } finally {
1031
+ await raw.unsafe(`ALTER TABLE "read_rebuild_items_per_group" DISABLE ROW LEVEL SECURITY`);
1032
+ }
1033
+ });
1034
+ });
@@ -0,0 +1,101 @@
1
+ // Pins the in-transaction inline-projection path over real HTTP for an
2
+ // r.systemScope() feature — the tenant-mode path is covered by domain-events-projections.integration.test.ts.
3
+
4
+ import { afterAll, afterEach, beforeAll, describe, expect, test } from "bun:test";
5
+ import {
6
+ integer as pgInteger,
7
+ table as pgTable,
8
+ text as pgText,
9
+ uuid as pgUuid,
10
+ } from "../../db/dialect";
11
+ import { insertOne, selectMany } from "../../db/query";
12
+ import {
13
+ createEntity,
14
+ createTextField,
15
+ defineEntityCreateHandler,
16
+ defineFeature,
17
+ } from "../../engine";
18
+ import {
19
+ resetEventStore,
20
+ setupTestStack,
21
+ type TestStack,
22
+ TestUsers,
23
+ unsafeCreateEntityTable,
24
+ } from "../../stack";
25
+
26
+ const sysItemEntity = createEntity({
27
+ table: "read_sysproj_items",
28
+ fields: { label: createTextField({ personal: false, reason: "test_fixture", required: true }) },
29
+ });
30
+
31
+ // No tenant_id column — mirrors the "system-scoped projection" table in
32
+ // query-projection.integration.test.ts, but fed by dispatch-write's
33
+ // in-transaction runProjections (r.projection) rather than a query handler.
34
+ const auditTable = pgTable("read_sysproj_item_audit", {
35
+ itemId: pgUuid("item_id").primaryKey(),
36
+ tenantId: pgUuid("tenant_id").notNull(),
37
+ label: pgText("label").notNull(),
38
+ hits: pgInteger("hits").notNull().default(1),
39
+ });
40
+
41
+ const sysProjFeature = defineFeature("sysproj", (r) => {
42
+ r.systemScope();
43
+ r.entity("sysproj-item", sysItemEntity);
44
+
45
+ r.projection({
46
+ name: "item-audit",
47
+ source: "sysproj-item",
48
+ table: auditTable,
49
+ apply: {
50
+ "sysproj-item.created": async (event, tx) => {
51
+ const p = event.payload as { label?: string };
52
+ await insertOne(tx, auditTable, {
53
+ itemId: event.aggregateId,
54
+ tenantId: event.tenantId,
55
+ label: p.label ?? "?",
56
+ });
57
+ },
58
+ },
59
+ });
60
+
61
+ r.writeHandler(
62
+ defineEntityCreateHandler("sysproj-item", sysItemEntity, {
63
+ access: { roles: ["SystemAdmin"] },
64
+ }),
65
+ );
66
+ });
67
+
68
+ let stack: TestStack;
69
+ const systemAdmin = TestUsers.systemAdmin;
70
+
71
+ beforeAll(async () => {
72
+ stack = await setupTestStack({ features: [sysProjFeature], systemHooks: [] });
73
+ await unsafeCreateEntityTable(stack.db, sysItemEntity, "sysproj-item");
74
+ });
75
+
76
+ afterAll(async () => {
77
+ await stack.cleanup();
78
+ });
79
+
80
+ afterEach(async () => {
81
+ await resetEventStore(stack, ["read_sysproj_items", "read_sysproj_item_audit"]);
82
+ });
83
+
84
+ describe("inline r.projection fires in-transaction for an r.systemScope() feature", () => {
85
+ test("HTTP write lands both the entity row and the projection row", async () => {
86
+ const created = await stack.http.writeOk<{ id: string }>(
87
+ "sysproj:write:sysproj-item:create",
88
+ { label: "widget-a" },
89
+ systemAdmin,
90
+ );
91
+
92
+ const auditRows = await selectMany<{ itemId: string; label: string; tenantId: string }>(
93
+ stack.db,
94
+ auditTable,
95
+ { itemId: created.id },
96
+ );
97
+ expect(auditRows).toHaveLength(1);
98
+ expect(auditRows[0]?.label).toBe("widget-a");
99
+ expect(auditRows[0]?.tenantId).toBe(systemAdmin.tenantId);
100
+ });
101
+ });