test(stack): re-enable null round-trip — guarded by #493 fix

Author: coderdan

packages/stack/__tests__/searchable-json-pg.test.ts

@@ -116,11 +116,34 @@ describe('searchableJson postgres integration', () => {
       await verifyRow(rows[0], plaintext)
     }, 30000)
 
-    // stack intentionally drops null handling from its Encrypted type
-    // (see "feat(stack): remove null from Encrypted type"), so unlike
-    // protect, encrypt(null) returns a real SteVec rather than null.
-    // The null pass-through scenario therefore doesn't apply to stack.
-    it.skip('round-trips null values', async () => {})
+    it('round-trips null values', async () => {
+      // stack's encrypt() public signature still excludes null from its
+      // JsPlaintext input; the runtime guard restored in #493 handles
+      // null as defense in depth, so the cast bypasses only the type
+      // narrowing — the runtime behavior under test is the same as
+      // protect's.
+      const encrypted = await protectClient.encrypt(null as any, {
+        column: table.metadata,
+        table: table,
+      })
+
+      if (encrypted.failure) throw new Error(encrypted.failure.message)
+      expect(encrypted.data).toBeNull()
+
+      const [inserted] = await sql`
+        INSERT INTO "protect-ci-jsonb-stack" (metadata, test_run_id)
+        VALUES (NULL, ${TEST_RUN_ID})
+        RETURNING id
+      `
+
+      const rows = await sql`
+        SELECT id, (metadata).data as metadata FROM "protect-ci-jsonb-stack"
+        WHERE id = ${inserted.id}
+      `
+
+      expect(rows).toHaveLength(1)
+      expect(rows[0].metadata).toBeNull()
+    }, 30000)
   })
 
   // ─── jsonb_path_query: path-based selector queries ─────────────────