Repository navigation
Remove legacy documentation dispatch after cutover - #5391
kojiwakayama wants to merge 5 commits into
Conversation
📦 Client bundle boundary
A server module in a client graph aborts hydration in the browser. New leaks fail CI; known leaks are tracked in |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
…ocumentation-sync
…ocumentation-sync
|
@codex review |
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. |
|
Codex Review: Didn't find any major issues. Already looking forward to the next diff. Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
|



After the replacement documentation is live, Code releases no longer need a cross-repository dispatch to the legacy docs repository. Remove
sync-docs.yml, its dedicated GitHub App entry from the security checklist, and the obsolete hardening-test expectations. Framework documentation continues to ship in theveryfrontnpm package through the existing release process.The final diff is three files: 5 additions and 54 deletions. Scoped release and Homebrew publishing checks remain enforced. No package version changes.
Validation on refreshed branch
366d97963b94c4ac44e5e6f34979ed06d4b88be6:deno task test:file src/security/repository-hardening.test.tspassed all 13 steps with exit code 0.git diff --checkpass. Searches find no remaining references to the deleted workflow or its dedicated App credentials in source, scripts or workflow configuration.codex review --base origin/mainfound the workflow, checklist and tests consistent with no code findings. The reviewer's separate test attempt hit a cache-permission error; the direct focused run above completed successfully.Hold as draft with auto-merge disabled. Do not merge until Studio production documentation is live, public checks pass, and routing rollback/reapply has been exercised. The user's production hold remains in force. Dedicated credential removal is a later retirement step; no workflow or credential has been disabled or removed from the default branch by this PR.