fix(billing): cast outbox payloads for JSONB operators - #6928
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Enterprise follow-up rollups now cast Tests assert the generated SQL includes Reviewed by Cursor Bugbot for commit 7ac3614. Configure here. |
Greptile SummaryThis PR fixes outbox follow-up correlation against the PostgreSQL
Confidence Score: 5/5The PR appears safe to merge, with no actionable correctness or security issues identified. The outbox payload column is non-null PostgreSQL JSON, and the explicit casts make the existing JSONB operators type-compatible while preserving matching behavior for the string operation identifiers written by current callers.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/billing/enterprise-provisioning.ts | Adds the required JSON-to-JSONB casts to Enterprise follow-up rollup expressions without changing their correlation semantics. |
| apps/sim/lib/billing/enterprise-provisioning.test.ts | Adds focused assertions that the generated join and filter expressions contain the required JSONB casts and operators. |
| apps/sim/lib/core/outbox/service.ts | Corrects the shared source-operation predicate so JSONB containment receives compatible operands. |
| apps/sim/lib/core/outbox/service.test.ts | Adds a regression assertion for the JSONB cast and containment expression generated by the shared predicate. |
Reviews (1): Last reviewed commit: "fix(billing): cast outbox payloads for J..." | Re-trigger Greptile
Summary
Type of Change
Testing
bun run lintbun run type-checkbun run check:auditsbun run check:api-validationbun run apps/sim/scripts/check-block-registry.ts origin/stagingbunx vitest run lib/billing/enterprise-provisioning.test.ts lib/core/outbox/service.test.ts(78 tests)Checklist