Repository navigation
feat(site): ship the restyled docs to opencode.ai - #62466
Merged
Merged
Conversation
There was a problem hiding this comment.
No blocking problems. With legacy/ and the restyled switch gone, every docs build renders the restyle, and nothing still points at the removed code.
I ran the docs build after bun run generate twice, once with --mode dev and once with the production default. Both pass the budgets, link validation and the asset-prefix check, and the two outputs are byte-identical, with the same hashed files dev.opencode.ai serves (client-Cjc98lXL.js, docs-D6fIs2-I.css). typecheck:tsc and bun test (192) pass.
The public API is unchanged: this is internal to the site build, and --mode on the docs build is still accepted and ignored, so check-docs.yml keeps working.
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.
What does this PR do?
Ships the restyled docs (#62424, #62425) to opencode.ai/docs. Readers there now get the same docs dev.opencode.ai has served since #62425: the site's shell with David's docs components, the lockup menu, the bracketed section nav, the Contents rail, the site's search palette and the phone menus. URLs, Markdown twins (
Accept: text/markdownand/docs/<page>.md),llms.txt, the sitemap (opencode.ai URLs), Pagefind search, the V1 fallbacks (/docs/ja/→/v1/docs/ja/) and the funnel attributes stay as they are. dev.opencode.ai keeps itsnoindexfrom #62430.The build-time switch is gone, so every deploy builds the same docs:
src/docs/legacy/(the old document, layout, components, rehype plugins,client.tsandglobal.css)restyled(mode)fromsrc/docs/config.ts, therestyleoption invite.ts,script/build.tsandrender.tsx, andrestyleinvirtual:docs-pagessrc/docs/AGENTS.mddescribes the one designcheck-docs.ymlstill passes--mode devto the docs build. The build accepts and ignores it so CI keeps passing without touching.github/**. A follow-up can drop--mode devfrom the workflow and the option fromsrc/docs/script/build.tstogether. #62428 (build both modes in CI) no longer applies.How did you verify your code works?
services/site:bun typecheck,bun run typecheck:tsc,bun test(192 pass); rootbun run checkbun script/build.ts --mode productionand--mode devon an exe.dev VM: both pass the budgets (client 5.7 kB, docs.css 26.0 kB gzipped), link validation, the asset-prefix check and the Worker start check (production withoutX-Robots-Tag, dev with it). 70 pages, 69 Markdown twins/docsoutput of the production build is byte-identical to the dev build's and to origin/dev's--mode devoutput (diff -r, same hashed files as dev.opencode.ai serves today:docs-D6fIs2-I.css,client-Cjc98lXL.js). The only difference between the two deploys is dev'snoindexrule in_headers/docs/config/renders with the sidebar and Contents, search for "keybinds" returns 5 results and closes on Escape, the lockup menu opens and closes (outside click, keyboard), the phone menu opens and closes at 390 px, no console errorsVisual