chore(containers): ship the LiteParse page-ceiling image - #745
Conversation
Staging and production pin the extractor image by digest, so the container changes merged in #744 — the explicit 5,000-page ceiling, the 422 refusal for documents no tier can read, and the concurrency gate that sheds load with a retryable 503 — did not ship with the Worker. Production still truncated long PDFs at 1,000 pages silently. Built from the merge commit and verified against the pushed artifact rather than a local rebuild: cross-compiled linux/amd64 on an arm64 host, parses a real PDF end to end, and three concurrent parses yield one 503 and two successes.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
Capy auto-review is paused for this organization because the usage-cycle auto-review limit has been reached. Increase the limit or turn it off in billing settings to resume automatic reviews. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Wrangler configuration updates the pinned ChangesWorkspaceFileProcessor image update
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryUpdates the staging and production Confidence Score: 5/5Safe to merge: staging and production use the intended immutable LiteParse image, and the configuration parses successfully. No publishable findings remain after exercising the changed image bindings and Wrangler configuration generation. Files Needing Attention: No files need additional attention;
What T-Rex did
Reviews (1): Last reviewed commit: "chore(containers): ship the LiteParse pa..." | Re-trigger Greptile |
#744 merged the container-side fixes but could not ship them: staging and production pin the extractor image by digest, and Workers Builds deploys the Worker without rebuilding a pinned container. Production is still silently truncating long PDFs at 1,000 pages.
This bumps the pin to an image built from the merge commit.
c48cadd2→6f5ce124What the new image contains
Verified against the pushed artifact, not a local rebuild
Arch=amd64 OS=linux— wrangler cross-compiled correctly from an arm64 host, which is the failure mode worth catching herestatus 200, 8 pages)pnpm checkclean; both staging and production pins updated together, other images untouchedNote
Only the extractor image moves. The dev config builds from the Dockerfile and is unaffected.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.Summary by CodeRabbit