Skip to content

fix(people): address sixth round of cubic review findings#2894

Merged
Marfuen merged 1 commit into
mainfrom
mariano/cs-312-cubic-review-fixes-6
May 21, 2026
Merged

fix(people): address sixth round of cubic review findings#2894
Marfuen merged 1 commit into
mainfrom
mariano/cs-312-cubic-review-fixes-6

Conversation

@Marfuen
Copy link
Copy Markdown
Contributor

@Marfuen Marfuen commented May 21, 2026

Summary

Fixes 5 valid Cubic findings, replied won't-fix on 2 others:

Fixed:

  • Propagate archiver errors to output stream (prevents truncated ZIP as apparent success)
  • Clamp "Days since" to 0 for future offboard dates (negative value UX bug)
  • Disable evidence dropzone while upload is processing (prevents silent ignore)
  • Batch vendor attachment queries in getAccessRevocations (N+1 → 1 query)
  • Catch P2002 unique constraint race on revocation create (500 → 400)

Won't fix (replied on PR):

Test plan

  • Trigger archiver error during export — response should terminate, not return partial ZIP
  • Set future offboard date — "Days since" should show 0
  • Click dropzone while upload in progress — should be visually disabled
  • View vendor revocations for org with many vendors — should load fast (batched)
  • Send duplicate revocation request — should get 400, not 500

🤖 Generated with Claude Code


Summary by cubic

Fixes offboarding checklist issues to prevent truncated exports, improve evidence upload UX, harden revocation creation, and speed up vendor revocation loading. Supports Linear CS-312 by making employment event evidence collection and review more reliable.

  • Bug Fixes
    • Propagate archive errors to the response stream to prevent partial ZIPs.
    • Clamp “Days since” to 0 for future offboard dates.
    • Disable evidence dropzone while an upload is processing.
    • Batch vendor attachment lookups in getAccessRevocations (N+1 → 1 query).
    • Catch Prisma P2002 on revocation create and return 400 instead of 500.

Written for commit 83b0d05. Summary will update on new commits. Review in cubic

- propagate archiver errors to output stream to prevent truncated ZIPs
- clamp "days since" to 0 for future offboard dates
- disable evidence dropzone while upload is processing
- batch vendor attachment queries in getAccessRevocations (N+1 fix)
- catch P2002 race on revocation create, return 400 instead of 500

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@linear
Copy link
Copy Markdown

linear Bot commented May 21, 2026

CS-312

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Building Building Preview, Comment May 21, 2026 3:43pm
comp-framework-editor Ready Ready Preview, Comment May 21, 2026 3:43pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
portal Skipped Skipped May 21, 2026 3:43pm

Request Review

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@Marfuen Marfuen merged commit 080b1f9 into main May 21, 2026
9 of 10 checks passed
@Marfuen Marfuen deleted the mariano/cs-312-cubic-review-fixes-6 branch May 21, 2026 15:44
@claudfuen
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 3.60.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants