Repository navigation
fix(platform): repair failures found in Cloudflare acceptance - #569
Open
Brooooooklyn wants to merge 11 commits into
Open
Brooooooklyn wants to merge 11 commits into
Brooooooklyn wants to merge 11 commits into
Conversation
🚀 Deploying Preview to Cloudflare 🚀Preview URL: https://codex-platform-e2e-20261023-void-cloud.voidzero-docs.workers.dev (commit 21bb87e)This URL reflects your latest Preview deploymentPreview Deployments by commit
|
Brooooooklyn
marked this pull request as ready for review
October 23, 2026 11:29
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 723240e248
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Live Cloudflare acceptance exposed failures in interrupted platform updates, slow application requests, usage reporting, page revalidation and account prerequisite guidance. This change fixes those paths and preserves the provider resources used by repeatable live tests.
Changes
Validation
Final local verification on
21bb87ef0:vp test runpassed 736 files and 14,547 tests (31 tests skipped, zero unhandled errors), and all 33 package typechecks passed. The prerequisite-message tests pass 184 focused cases; the legacy-KV follow-up passes 39 focused cases. Both path-specific and full-purge endpoint regressions failed before that fix and pass afterward. Affected verification passed 10,974 tests for the CLI changes and 4,907 for the cache follow-up. Core runtime/package and documentation builds, formatting and lint passed. Independent reviews found no remaining actionable issues in these changes.A rebuilt CLI was tested against the unpaid account: the real installation plan reports R2 code 10042 and Workers for Platforms code 10121 together, with account-specific links and retry instructions. It exits before creating installation files, and the existing platform registry is unchanged. The captured real Analytics Engine code 10089 failure was replayed through the updated formatter and now directs the operator to Enable and the appropriate recovery command.
The free-account comparison also verified a native static app and stock Void AI Gateway creation. Analytics Engine binding upload failed with 10089 before activation and passed after zero-price dashboard activation while Workers Free remained selected. R2 was not activated and WfP was not purchased; a fresh managed platform cannot complete installation on that account until those prerequisites are satisfied. Airwallex’s generic AI Gateway 500/code 7000 was not reproduced.
Both paid-account Cloudflare installations were deployed and independently verified on
852341fc6with packaged manifest34f1cea97be8251055b5002ec9eb2cdaa83393be73b81cd6f23532daafb00a25. Provider readback verified all five active Worker versions at 100% traffic, all 19/15 retained resource IDs, settings and migration history. Three 600-second page-action purges and one all-project purge returned fresh first-load MISS content in the same Cloudflare location, with independent R2 before/deleted/replacement readback. Project usage returns HTTP 200; AI usage matches the provider aggregate after whole-neuron rounding. All 24 managed apps passed final reachability; the kitchen smoke passed 14/14.That live acceptance covered an upgraded and a fresh installation, 24 managed apps (239 functional checks; 81 browser checks across 16 apps), four native app classes, two PostgreSQL variants, two genuine users, teams and ownership, admin/user settings, quota/suspension controls, protected apps, connected-app publishing/private preview, inbound/outbound email, queues, cron, AI, Live SSE, WebSockets and Sandbox. Earlier local acceptance also covered 243 real PostgreSQL checks, playground builds and packed npm/pnpm consumers for four adapters.
Remote CI, Platform CI, Prepublish Test, and Security Analysis completed successfully on
21bb87ef0: 19 successful checks, one skipped, none failing or pending. The skipped credentialedlive-e2ejob is separate from the provider-backed acceptance above. Automated review completed on this head with no findings; the legacy-KV review thread is resolved.Known limitations
The legacy-KV cleanup follow-up has local endpoint regression coverage and has not been redeployed to the acceptance installations; their runtime remains
852341fc6. No existing MySQL service was available, so MySQL has local wire-protocol coverage only. Verified-email invitation acceptance, direct Google/GitHub account linking, optional GitHub build services and third-party AI BYOK were not configured. Successful project custom-domain attachment remains unverified: the test installation disables that capability, and its denial path preserves DNS and the existing app. Destructive remote teardown is excluded by the retained-resource test policy. An ambiguous pre-fix email receipt remains preserved; recovery and a duplicate send have not been attempted. No additional paid plan was purchased.