Fix CI and CodeRabbit feedback on PR #13974
completedSlava requested remediation for PR #13974 / BOLT-1707 on branch BOLT-1707/domain-device. Inspect all failed CI logs and unresolved CodeRabbit review threads; make valid scope-contained fixes directly on the existing PR branch. Use a dedicated worktree, set required git identity, run domains/device lint before each push, commit with Co-authored-by: Slava <[email protected]>, push, monitor CI and CodeRabbit until green. Do not merge.
Event Timeline
created
progress
Fixed CI formatter violations and corrected the cleanup query into valid PostgreSQL DELETE syntax; pushed ea21be9. Local domains/device biome lint passes. Targeted Jest is blocked locally because the worktree dependency state lacks @quramy/jest-prisma-node; CI is now running.
progress
Addressed all valid CodeRabbit findings in 8c6282b: PostgreSQL DELETE USING (with structural SQL coverage), 10,000-chunk safety cap plus coverage, and removed deprecated batchSize parsing/forwarding. Local domains/device biome lint passes. CI and re-review running.
progress
CI found one test-only assertion mismatch: generated SQL begins with formatter indentation. Updated the structural assertion to trim leading whitespace while retaining DELETE USING/no-JOIN coverage; pushed the follow-up. Local domains/device biome lint passes; CI restarted.
status_change
in_progress → completed