docs: send retired train-path URLs to flow - #179
Merged
Conversation
The homepage already teaches record, compile, replay. Search still lands on /getting-started/quickstart/ and old package-mirror URLs that taught capture-then-train. Keep those routes as Flow-first redirects and fail the docs contract if that copy returns.
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.
Why
A person who starts at https://docs.openadapt.ai already sees
openadapt quickstartandopenadapt flow record | compile | replay. A person who arrives from search often doesn't. Google still quotes/getting-started/quickstart/asopenadapt capture startthenopenadapt train start --model qwen3vl-2b, and/packages/openadapt-desktop/as AI training-data collection. Those pages were already stubbed, but the quickstart stub still said "legacy model-training path", and the other old URLs 404'd, so the January snippet stayed.What changed
Retired routes now serve a Flow-first redirect. The theme already emits
noindexand a meta refresh fromredirect_to. The bodies show the current commands so a recrawl has something true to index./getting-started/(including quickstart, installation, permissions) goes to Get started or Desktop install./packages/ml/,/packages/retrieval/,/packages/openadapt-desktop/, and the rest of that tree) go to the product catalog. Research packages are labeled Research./LEGACY_FREEZE/and/legacy/freeze/are marked historical and send the reader to Get started./architecture/goes to Concepts./cli/goes to theopenadapt flowreference.scripts/validate_docs.pynow fails the build if current-product pages teach the retired capture-then-train CLI, or if a retired route loses its redirect. README sync will not overwrite a redirect stub.What a reviewer should check
/getting-started/quickstart/showsopenadapt quickstartandopenadapt flow record | compile | replay, not a model-training CLI.concepts/vlm-appliance.mdis the current optional on-prem grounder, not the retired train path; this change leaves it alone.Opened by an agent session, not the founder.