fix(db): harden workspace file size cutover

[waleedlatif1]

Summary

• Treat database-ordered text cursors as opaque so locale collation cannot abort the bounded backfill
• Remove implicit legacy size projections and fail closed when canonical size metadata is missing

Type of Change

• Bug fix

Testing

• 96 DB tests
• App and DB typechecks
• Full lint and strict API validation
• Migration safety check and locale-aware PostgreSQL reproduction

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Aug 26, 2026 8:32pm

[greptile-apps]

Greptile Summary

The PR completes the workspace-file canonical size cutover and hardens its backfill behavior.

• Replaces legacy size projections with canonical sizeBytes reads that fail closed when metadata is missing.
• Makes the bounded backfill treat database-ordered text IDs as opaque cursors.
• Updates file operations, storage accounting, mocks, and tests for the canonical size contract.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
packages/db/script-migrations/0008_backfill_workspace_file_size_bytes.ts	Treats database-ordered text IDs as opaque cursors while retaining bounded, idempotent keyset backfill behavior.
packages/db/script-migrations/0003_backfill_workspace_storage_usage.ts	Ensures canonical file sizes are populated before workspace storage usage is reconciled.
packages/db/schema.ts	Establishes the canonical workspace-file projection around sizeBytes while retaining the legacy field for migration compatibility.
apps/sim/lib/billing/storage/payer-transfer.ts	Uses canonical size metadata for storage transfers and rejects incomplete size data rather than silently misaccounting it.
apps/sim/lib/uploads/contexts/workspace/workspace-file-manager.ts	Converts workspace-file lifecycle and accounting paths to the canonical size accessor.
apps/sim/app/api/files/export/[id]/route.ts	Uses canonical metadata for export accounting and omits embedded assets whose canonical size cannot be established.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[SQL migration adds size_bytes] --> B[Compatibility trigger synchronizes size columns]
  B --> C[Script migration backfills null size_bytes in bounded batches]
  C --> D[Application queries project canonical workspace-file columns]
  D --> E[getWorkspaceFileSize validates sizeBytes]
  E -->|Valid| F[File operations and storage accounting continue]
  E -->|Missing or invalid| G[Operation fails closed]

Loading

_{Reviews (2): Last reviewed commit: "fix(files): skip malformed export assets" | Re-trigger Greptile}

[cubic-dev-ai]

All reported issues were addressed across 28 files

<sub>Reply with feedback, questions, or to request a fix.

Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic review

[cubic-dev-ai]

@cubic review

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 28 files

Confidence score: 5/5

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

_{Re-trigger cubic}