Repository navigation
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: true
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 |
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.
Frozen candidate and resulting behavior
Consolidates the complete existing founding-builder draft tree onto current main in one CMS review target. Final source:
af0f4aa73cb55e3565b53792bab91916d99a41b5, CMS v3.3.2-rc.8. Unchanged exact package inputs: CLI #67, 0.3.0-rc.2 atd851dd980ad43ce3c44e5f03bf68048cd443f3bc; MCP #30, 0.1.11-rc.2 at8c96de27908373983bfff831ee99e4231e0873c8.The initial consolidated tree was verified identical to the full prior draft input. All 27 historical PR heads remain unchanged/open/draft; do not merge overlapping candidate chains in sequence.
release/rc8/INPUTS.jsonandSCOPE.mdrecord the frozen scope.New runtime changes are limited to the accepted
frame-ancestors 'self'policy (retaining static-preview script blocking/cache behavior) and lazy compiler-worker startup. Previously authentication/dashboard redirects started idle compiler workers; now workers start independently on actual compilation and retain reuse/caches and error handling. This addresses the reproduced Firefox navigation integration failure while preserving editor compilation.Existing synchronization/nested data/identity safety, content-only source permissions/repeater ordering, compiler/hydration/cache, dependency security, production validation, type/Svelte-warning and browser/draft fixes are retained. References: #1239, #1241, #1259, #1235, #1309, #705, #804, #1314. Original #1235/#705 fixture uncertainty remains; no issue is closed. Trusted editor compiled-output publication remains; an untrusted-code sandbox is not claimed.
Final validation
Recovery compares ten collections, all record identities/values, nested data, image bytes, active publication and client authentication/source denial. It upgrades one copy, restores a different pristine-backup copy using matching RC.7 bytes, verifies the backup unchanged and preserves newer candidate writes separately. No old executable opens upgraded data.
Review, rollout and remaining gates
release/rc8/REVIEW.mdrecords agent integration review. Maintainer approval remains outstanding; draft-skipped automated review is not approval. The local RC.8 kit report/manifest/checksums retain final source-bound evidence outside the source commit so recording results cannot replace the tested binary identity.See
release/rc8/ROLLOUT.mdandRECOVERY.md. Later authorized order: approved MCP first, refresh references/locks and requalify any changed bytes, compatible CMS before public CLI, CLI last with a fresh registry handoff. Main/tag/release automation can publish containers/change consumers, including prereleases; control those paths before later authorization.Pilot accounts/sites/client assignments are operator-provisioned. Real SMTP invitation/setup/reset delivery, selected pilot host/support ownership and explicit release/deployment/invitation authorization gate activation. Physical phones, applicable real domain/TLS/CDN and operational backup/support gate first-client launch. Representative legacy fixtures gate actual migrations. General cross-origin embeds/marketplace previews need a separately approved/tested exception before upgrading a host that relies on them. RC.7 hosted/Safari/soak evidence remains historical; RC.8 was not deployed and physical devices/real mail are not claimed.
Keep this and every historical PR draft. No merge, tag/release, registry/container/site publication, deployment, live-mail configuration or outbound invitation was performed. Local/CI tests use generated data and loopback publication only. Recommendation: GO for maintainer review; NO-GO for activation until the stated gates and separate authorization.