docs(factories): launch polish pass - #565
Merged
Merged
Conversation
Follow-up sweep for the launch, from the original audit: - The llms.txt site description still said "the Oz platform" and never mentioned Factories. It is the first thing an AI engine reads about Warp and the most-consumed AI-facing artifact we publish, so the rename skipping it was the costliest miss of the set. - The /platform/software-factory redirect only matched the trailing-slash form. Every internal link that used to point there had no slash, and 1,231 of the 1,292 existing entries use the bare or `(/?)` form. Now matches both. - CopyPageButton stripped `import ... from ...` lines from anywhere in the page, not just the leading MDX import block, so "Copy page" silently removed `import OpenAI from "openai";` from the Node sample in the Ollama guide. Now walks the leading block only. - The API reference page's screen-reader h1 still said "Warp & Oz". - Platform credits never mentioned factories, though the quickstart sends readers there to learn what a factory costs. Factory runs are cloud agent runs, so the existing rule already covers them; now it says so. - factory-as-code listed `gemini` as an accepted harness and linked to a page documenting three. The list is right (the schema enum derives from AllAgentHarnesses), so the sentence now separates what the schema validates from what a team can run. Co-Authored-By: Warp <agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
I'm starting a first review of this pull request. You can follow along in the session on Warp. I cancelled the in-progress review run because this pull request was closed. Powered by Oz |
Collaborator
Author
|
This PR was generated with Warp. Comment   |
|
Your GitHub account is not connected to Warp. Connect it here. |
Contributor
|
Your Warp account is not a member of any team with access to this repository. |
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
2 participants
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.
Follow-up to #563, targeting
hyc/factory-launch. Six small fixes from the original audit of #508 that weren't part of the review-feedback batch. Ready to merge — no open questions.astro.config.mjs— llms.txt site descriptionvercel.json— software-factory redirect/platform/software-factory/with a trailing slash. Every internal link that used to point there had no slash, and 1,231 of 1,292 existing entries use the bare or(/?)form. Now(/?), so both resolve.CopyPageButton.astroimport ... from ...from anywhere in the page rather than just the leading MDX import block, so "Copy page" silently deletedimport OpenAI from "openai";from the Node sample in the Ollama guide. Now walks the leading block only; verified against a page with a code-sample import, a page with MDX imports, and one with both.api.astroh1still read "Warp & Oz HTTP API reference".platform-credits.mdxfactory-as-code.mdxgeminias an accepted harness while linking to a page documenting three. The list is correct — the schema enum derives fromAllAgentHarnessesin warp-server — so the sentence now separates what the schema validates from what a team can actually run.One thing I could not fix
The hero image on
/platform/overview/(most-flexible-platform-for-building-with-agents.png) carries what looks like a presenter annotation from an internal deck — a starred callout reading "Use to program against agents" — plus deck-style title framing. It's a launch-visible page and needs a re-export rather than a docs change, so I left it alone. The second diagram on that page (platform-architecture.png) is clean.Validation
npm run buildclean · 0 broken internal links (3,695 checked) ·factory-proper-noun,platform-determiner, andhardcoded-varall 0 on changed files · both style_lint suites pass · confirmed "Warp Factories" now appears in the builtllms.txt.Plans: