Repository navigation
docs: rewrite the Warp Factories overview copy - #850
Conversation
Reshape the factories overview from a launch-post structure into a technical overview: a concrete definition up front, one short section per part of a factory, sizing guidance, and related pages. Removes the "Who benefits", "What you get", product-table, and "Key terms" sections and the undefined "delivery policy" coinage. The #sizing-a-factory anchor that three pages link to is preserved. Aligns the opening definition on how-factories-work and the docs homepage with the overview, trims the diagram legend, cuts the salesy self-improvement framing, and adds the missing Related pages section. Co-Authored-By: Warp <agent@warp.dev> Co-Authored-By: Oz <oz-agent@warp.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I completed the review and no human review was requested for this pull request. Comment Powered by Oz |
There was a problem hiding this comment.
Overview
This PR rewrites the Warp Factories overview and aligns related entry-point copy across the docs homepage and how-it-works page. The copy is clearer overall, and I did not find security-specific issues or spec drift because no approved spec context was provided.
Concerns
- The PR body declares
Documentation riskaslow, but this diff adds or changes public technical claims about supported work sources, stages/checkpoints, execution, self-hosting, metrics, and harness/model behavior. The docs quality policy's low-risk allowlist excludes changes to technical claim categories even when they are sourced from existing canonical pages, so this should be reclassified asengineering-review-requiredor carry an explicit docs verification/waiver.
Verdict
Found: 0 critical, 1 important, 1 suggestions
Request changes
Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
| * **Factory definition** - Version-controlled agents, automations, runners, scorers, skills, and webhooks, either Warp-managed or in a GitHub repository your team owns. See [definitions as code](/factories/factory-as-code/). | ||
| * **Execution** - Every stage is a cloud agent run on Warp-hosted or [managed self-hosted](/factories/self-hosting/) compute, using the factory's repositories and each stage's configured model and [harness](/platform/harnesses/). | ||
| * **Factory dashboard** - Metrics, work items by stage, runs, and costs. See the [factory dashboard](/factories/factory-dashboard/). | ||
| * **Measure and improve** - [Scorers](/factories/measure-and-improve/) classify completed runs, benchmarks compare configurations, and Self-improvement turns repeat failures into pull requests for your review. |
There was a problem hiding this comment.
💡 [SUGGESTION] Use “repeated failures” here; “repeat failures” reads like a typo.
| * **Measure and improve** - [Scorers](/factories/measure-and-improve/) classify completed runs, benchmarks compare configurations, and Self-improvement turns repeat failures into pull requests for your review. | |
| * **Measure and improve** - [Scorers](/factories/measure-and-improve/) classify completed runs, benchmarks compare configurations, and Self-improvement turns repeated failures into pull requests for your review. |
…w-copy # Conflicts: # src/content/docs/factories/how-factories-work.mdx # src/content/docs/factories/index.mdx # src/content/docs/index.mdx
Tighten the opening (drop "tools your team already uses"), make the component headings parallel, say "repeated failures" consistently, and note that benchmarks now compare harnesses as well as models and runners per #841. Co-Authored-By: Warp <agent@warp.dev> Co-Authored-By: Oz <oz-agent@warp.dev>
There was a problem hiding this comment.
Review summary
The independent agent completed its review for this commit.
Findings
src/content/docs/factories/how-factories-work.mdx:18— [SUGGESTION] The Work sources legend item omits Microsoft Teams and Azure DevOps, whichsrc/content/docs/factories/index.mdx:59lists, so the two pages disagree. Requested change: add both sources, linked to /factories/integrations/teams/ and /factories/integrations/azure-devops/.src/content/docs/factories/index.mdx:63— [SUGGESTION] The clause "so the same runners, models, APIs, and security controls apply" restates the consequence of the previous clause. Requested change: cut it, or replace it with the one control that matters to the reader and link to infrastructure and security.
Verdict
Approve
Add Microsoft Teams and Azure DevOps to the how-factories-work work sources legend so it matches the overview, and drop the restated consequence clause from the overview's Execution section in favor of naming what the infrastructure page covers. Co-Authored-By: Warp <agent@warp.dev> Co-Authored-By: Oz <oz-agent@warp.dev>
Same treatment as the factories overview: a concrete opener, three short lists of what Enterprise adds (administration, security and compliance, shared team configuration), role-based entry points, and related pages. Removes the "Who Warp Enterprise is for", "What this section covers", "Why enterprises choose Warp", and "Support and resources" sections and the marketing adjectives. Every retained claim was checked against the enterprise page it links to. Co-Authored-By: Warp <agent@warp.dev> Co-Authored-By: Oz <oz-agent@warp.dev>
This reverts commit ff03ffb.
There was a problem hiding this comment.
Review summary
The independent agent completed its review for this commit.
Findings
src/content/docs/factories/index.mdx:51— [IMPORTANT] "Merging is enforced by your repository's branch protection" implies branch protection is always in place, buthow-factories-work.mdx:71only recommends it for human-only merges andfactory-agents.mdx:153says repository permissions decide who merges. Requested change: say the factory never merges and that repository permissions and branch protection decide who can.src/content/docs/index.mdx:34— [SUGGESTION] "returns a pull request where the work started" is inaccurate for Slack, Teams, and Jira sources, where the pull request is opened in the repository and linked back. Requested change: match the overview wording, for example "returns the result, usually a pull request, where the work started."src/content/docs/factories/index.mdx:59— [SUGGESTION] The Work sources paragraph has ten links and repeats the source lists inhow-factories-work.mdx:18andconnect-your-factory.mdx. Requested change: shorten it to a sentence naming a few sources and link to connect-your-factory for the full list.src/content/docs/factories/index.mdx:67— [SUGGESTION] "shows work items by stage" is not documented onfactory-dashboard.mdx, and the same phrase is onhow-factories-work.mdx:24. Requested change: describe what the Dashboard page shows (autonomy, cycle time, runs, cost per pull request) or link to a page that documents a stage view, and fix both pages.src/content/docs/factories/index.mdx:4— [SUGGESTION] The PR title and body still describe anenterprise/index.mdxrewrite that head commit9de16d4reverted, including the risk rationale andSource files consultedlist. Requested change: update the PR title and body to cover only the three changed pages, or restore the Enterprise change.
Verdict
Request changes
Shorten the lead to two sentences, keep the CTA cards directly under it, and open a "What a factory does" section for the foreman, example, and human-checkpoint detail so the body is not an unbroken block of prose before the first H2. Rewrite the description so it stops repeating the lead paragraph it renders next to. Co-Authored-By: Warp <agent@warp.dev> Co-Authored-By: Oz <oz-agent@warp.dev>
There was a problem hiding this comment.
Two factual claims need correction before approval. Current-head CI passes the build, link, editorial, technical-reference, CodeQL, and Vercel checks; local style, link, risk-contract, and compression checks also pass.
Found: 0 critical, 2 important, 1 suggestion, 0 nits.
Responding as Docs Factory (V2): Open session · View in factory
Co-Authored-By: Oz <oz-agent@warp.dev>
|
This PR was generated with Warp. Comment |
There was a problem hiding this comment.
Review summary
The independent agent completed its review for this commit.
Findings
src/content/docs/factories/index.mdx:13— [SUGGESTION] The lead says a factory "returns a pull request where the work started", but a pull request isn't always where work starts (Slack, Linear, Jira), and line 30 says "usually as a pull request". The docs homepage uses "returns the result, usually a pull request, where the work started", so the definition sentence differs across the overview,how-factories-work.mdx, andindex.mdxdespite the PR's alignment claim. Requested change: use the homepage wording "returns the result, usually a pull request, where the work started", or reuse one identical sentence on all three pages.src/content/docs/factories/index.mdx:30— [SUGGESTION] The foreman dispatching the triage, spec, implement, and review agents is stated on lines 13, 30, and 49, and the human checkpoints on lines 32 and 55, which breaks the "Say it once" rule in AGENTS.md. Requested change: cut the first sentence of this paragraph or the matching text under "Foreman and factory agents", and fold the line 32 checkpoint sentence into the "Stages and checkpoints" section.
Verdict
Approve
Co-Authored-By: Oz <oz-agent@warp.dev>
There was a problem hiding this comment.
All requested fixes are addressed on the current head, and the final independent re-review found no remaining findings. Local build, style, link, risk-contract, and compression checks pass; current CI has no failures.
Responding as Docs Factory (V2): Open session · View in factory
There was a problem hiding this comment.
Review summary
The independent agent completed its review for this commit.
Findings
src/content/docs/factories/index.mdx:43— [SUGGESTION] The work item definition ("an issue, a support thread, a pull request, or a scheduled job") differs fromhow-factories-work.mdx:12("issue, support request, pull request, or Factory MCP task"), so two pages define the same term differently. Requested change: define it once and link to it from the other page, or use the same examples in both.src/content/docs/factories/index.mdx:28— [SUGGESTION] The "What a factory does" section is a single sentence before the video and diagram. Requested change: fold the sentence into the opening paragraph at line 13 and place the video and diagram under "The parts of a factory", or add concrete detail to the section.src/content/docs/factories/index.mdx:61— [SUGGESTION] The Work sources paragraph lists the same eleven sources ashow-factories-work.mdx:18andconnect-your-factory.mdx, giving three copies to keep in sync. Requested change: name the main sources and link to connect your factory for the full list.src/content/docs/factories/index.mdx:69— [SUGGESTION] The metric names "pull request cycle time" and "cost per pull request" don't match the dashboard labels PR cycle time and Cost per PR infactory-dashboard.mdx, and that page isn't listed underSource files consulted. Requested change: use the dashboard's labels or drop the metric names and link to the dashboard, and addfactory-dashboard.mdxtoSource files consultedin the PR body.src/content/docs/factories/index.mdx:65— [NIT] "ordinary" in "an ordinary cloud agent run" adds nothing. Requested change: remove it (suggestion block provided in the inline comment).
Verdict
Approve with nits



Summary
The Warp Factories overview (
docs.warp.dev/factories) read like a launch post rather than the entry page of technical docs: persona and feature-pitch sections ("Who benefits", "What you get"), a product-positioning table, heavy bullet-paragraphs, and an undefined "delivery policy" coinage, with no single place that said what a factory is. Its siblinghow-factories-workopened with a different definition and repeated much of the same material.This PR reshapes the overview into the register of the warp.dev/factories page and the GitHub Actions overview that
AGENTS.mdpoints to: a concrete definition with an example up front, one short section per part of a factory, sizing guidance, and related pages. The same one-sentence definition now appears on the overview,how-factories-work, and the docs homepage.Changes
src/content/docs/factories/index.mdx
#sizing-a-factoryanchor linked fromhow-factories-work,connect-your-factory, andquickstartstill resolves.src/content/docs/factories/how-factories-work.mdx
src/content/docs/index.mdx
Content design plan
/factoriesfrom the marketing site or search and wants to know, in about a minute, what a factory is and how it is put together before deciding whether to read the quickstart.how-factories-workone click away.how-factories-work.Unverified claims
None new. The rewrite adds no UI labels, paths, flags, defaults, or eligibility claims that were not already published on a canonical factories page. Each retained claim was checked against the page it links to (
factory-agentsfor the harness list and Foreman name,how-factories-workfor the default checkpoints,connect-your-factoryfor the source list,factory-dashboardfor dashboard metrics, andmeasure-and-improve/self-improvementfor proposals arriving as pull requests) rather than againstwarp-internalorwarp-server. Review feedback confirmed that the factory never merges and repository permissions plus branch protection determine who can approve and merge.Documentation risk
Risk: engineering-review-required
Rationale: Copy rewrite that restates existing claims (work sources, stages and checkpoints, execution and self-hosting, metrics, harness choice) on the overview, where some did not previously appear; classified conservatively per the policy. No source files in warp-internal or warp-server back the change, so no engineering owner resolves and none is requested; Docs validates each claim against the canonical factories page it links to. The pages consulted are all under src/content/docs/factories/ at the merged main SHA.
Source files consulted: factory-agents.mdx@b43e00b7, how-factories-work.mdx@b43e00b7, connect-your-factory.mdx@b43e00b7, factory-dashboard.mdx@b43e00b7, benchmarks.mdx@b43e00b7, measure-and-improve/self-improvement.mdx@b43e00b7, infrastructure-and-security.mdx@b43e00b7
Engineering review status: not-applicable
Docs override: none
Merge with #841
Merged
mainafter #841 (Factories launch readiness) landed on the same three files. Kept its changes: the Early Access callouts are gone, the two-card "Get started / Open Warp Factories"CardGridsits under the overview's intro, and the homepage sentence no longer mentions Early Access. Updated the overview's benchmarks sentence to "models, harnesses, and runners" to match the revised benchmarks page. Also applied the automated reviewer's suggestion ("repeated failures") and reclassified the risk level as it recommended.The independent review's two suggestions are addressed in
892cc2d3: Microsoft Teams and Azure DevOps added to thehow-factories-worklegend, and the restated-consequence clause in the overview's Execution section replaced with what the infrastructure page covers. The sidebar's Early Access badge is left as is: #841 removed it and then deliberately restored it ("retain Factories Early Access badge").Follow-up review findings are addressed in
f66db114: the overview now returns the result, usually a pull request, to the source, and its examples no longer repeat the dispatch and checkpoint details covered below.Additional context
style_lintreports 0 errors on the three files (the remaining warnings are "bolded term not in glossary" notes on bold list lead terms);npm run buildpasses; the internal link checker finds 0 broken links;id="sizing-a-factory"and both LinkCards are present in the built/factories/page.trunk checkwas not run locally (Trunk CLI not installed); relying on CI.Co-Authored-By: Warp agent@warp.dev
Co-Authored-By: Oz oz-agent@warp.dev