Skip to content

docs: prepare Warp Factories docs for launch - #748

Merged
hongyi-chen merged 71 commits into
mainfrom
rrenk/grow-6127-factories-ga
Oct 6, 2026
Merged

hongyi-chen merged 71 commits into
mainfrom
rrenk/grow-6127-factories-ga

Conversation

@rachaelrenk

@rachaelrenk rachaelrenk commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Prepares the Warp Factories documentation for launch by reconciling the existing restructure with current product behavior in warp and warp-server. The update resolves conflicts with current main, moves current self-hosting guidance into the Factory information architecture, adds native Azure DevOps guidance, expands the factory-definition reference, restores missing navigation, repairs cross-links, and flattens redirect chains.

Changes

  • Audits the complete Warp Factories documentation set against current source behavior, including setup, automations, integrations, runners, self-hosting, API and SDK surfaces, and measurement workflows.
  • Rewrites the docs homepage as a concise map of Warp, the Warp Agent, the Automation Platform, and Warp Factories. Removes duplicated capability, multi-model, open-source, and privacy sections in favor of links to their canonical pages.
  • Documents native Azure DevOps Services setup, identities, administrator requirements, repository access, supported events and filters, default mention-and-assignment routing, and troubleshooting. Distinguishes Azure DevOps Services from Azure DevOps Server and standalone PAT-based cloud-agent access.
  • Expands the factory-definition reference from generated schemas, including Azure DevOps, deferred repositories, routers, benchmark substitutions and metadata, scorer evidence, self-improvement, spawning, timeouts, required repositories, Windows runners, and failed-run retention.
  • Moves current deployment-pattern and managed self-hosting guidance into the Factory information architecture, including external orchestrators, Docker and Kubernetes task-image requirements, Direct and command backends, monorepo worktrees, and agent-key versus self-hosted-worker-key usage.
  • Documents current launch entitlements: GitHub-backed factory definitions, creator credential strategy, and third-party harnesses require a paid plan; team-managed keys and endpoints are available on Business and Enterprise plans.
  • Documents authorized custom routers in model catalogs and benchmark pickers, terminal-but-scoreable blocked benchmark trials, and the current scorer evidence model. Removes obsolete benchmark baseline and candidate roles and the screenshot that exposed the retired control.
  • Preserves the production Microsoft Teams disconnected-channel message and recovery flow while keeping staging-only setup delegation out of public availability claims.
  • Preserves current Kubernetes operations and the skill-action API catalog while resolving content conflicts.
  • Restores launch-relevant sidebar entries for ChatGPT subscriptions, Enterprise Workspaces, Microsoft Teams, self-hosted data storage, the API quickstart, and the Sentry SDK demo, and removes obsolete Universal and Classic input entries.
  • Repairs affected inbound and outbound links, updates canonical destinations, and changes the redirect checker to recognize Vercel named parameters.
  • Flattens redirect chains without changing published redirect sources.

Content design plan

  • Target audience and their JTBD - Developers and platform administrators evaluating or configuring Warp Factories for launch who need accurate setup, integration, operation, and reference documentation.
  • Problem statement - The staged restructure conflicted with current docs and omitted or contradicted product behavior added since the branch was created.
  • Goals - Give readers a complete path from creating a factory through connecting work sources, operating agents, defining resources as code, and measuring outcomes; preserve published links and discoverability.
  • Purpose and value added - This audit reconciles the launch documentation with the current product implementation and generated schemas rather than preserving stale staged assumptions.
  • Content types and models - Conceptual, procedural, quickstart, reference, and troubleshooting content, following each existing page's job.
  • High-impact scenarios - Covers first factory creation, native GitHub, GitLab, Azure DevOps, Slack, Microsoft Teams, Linear, and Jira routing, definition-as-code lookup, managed and self-hosted execution, and API-driven workflows.

Authoritative sources

  • warpdotdev/warp@88afc88fe2251e77c66a80e80518affeafcbf7e8 (master)
  • warpdotdev/warp-server@3b4638b56ae2b9e926fc1f7d59b2062deb4e2110 (develop)
  • warpdotdev/docs@46935ea5b192a8145f8eff46243726fea97ec2f0 (main), merged without force-pushing.
  • Proposed docs head: 3826de732815d9cf967f6563ec155d4837efd298.
  • Plan eligibility was verified from billing/config/tiers/build_business.yaml and billing/config/tiers/_base_enterprise.yaml, where Business and Enterprise grant MANAGED_BYOK_BYOE.
  • Azure DevOps rollout and configuration were verified from config/prod.yaml, config/features/features.go, and client/packages/factory/.env.production; trigger and filter behavior was verified from model/types/triggers/triggers.go and logic/factoryfile/schema/v1alpha1/automation.schema.json; the editor limitation was verified from client/packages/factory/src/pages/FactoryAutomations/azure-devops.ts.
  • Factory-definition claims were verified from the generated files under logic/factoryfile/schema/v1alpha1/.

Validation

  • git diff --check - passed.
  • python3 scripts/test_check_redirects.py - passed, 3 tests.
  • python3 scripts/check_redirects.py - passed, 1,336 redirects loaded, 536 statically checked, 0 broken.
  • Redirect audit - 408 changed destinations, 0 added or removed sources, 0 duplicate sources, 0 remaining exact or optional-slash chains, and 0 fragment mismatches.
  • python3 .agents/skills/check_for_broken_links/check_links.py --internal-only - passed, 394 files and 4,291 internal links checked, 0 broken.
  • Full link check - passed, 394 files, 4,291 internal links, and 1,727 external links checked, with 0 broken; 29 bot-blocked responses across 13 URLs were reported separately.
  • Live missing-docs map audit against the pinned warp and warp-server checkouts - passed with 0 gaps.
  • Targeted restoration and launch-information-architecture assertions - passed, 18 checks proving current-main self-hosting, federation, and Admin Panel content was preserved with only intentional launch path and terminology changes.
  • Benchmark-role assertions - passed, 6 checks confirming the removed baseline and candidate controls no longer appear in the public docs.
  • python3 .agents/skills/validate_ui_refs/validate_ui_refs.py --changed --require-provenance - passed across 131 changed files with 0 Settings-path, Command Palette, or formatting issues.
  • npm run typecheck - passed with 0 errors and 7 existing hints.
  • npm run build - passed; generated public/openapi.json at 197.1 KB and produced 395 HTML files and 393 Markdown documents.
  • npm run test:homepage-json-ld - passed.
  • Compression contract - passed for Factory deployment patterns, infrastructure and security, the Factory overview, and the cloud-agent documentation map.
  • python3 -m unittest discover -s .agents/skills/doc_quality_policy -p 'test_*.py' - passed, 110 tests.
  • python3 .agents/skills/missing_docs/scripts/test_audit_docs.py - passed, 29 tests.
  • python3 .agents/skills/docs_consistency_audit/scripts/test_audit_consistency.py - passed, 41 tests.
  • python3 .agents/skills/sync-openapi-spec/scripts/sync_openapi.py --mode self-test - passed.
  • Independent review follow-up - resolved the important findings by correcting active tooling paths, including the final guide-drafting references, and Business-plan eligibility, then applied the concrete compression and canonical-URL suggestions.

Validation caveats

  • python3 .agents/skills/style_lint/style_lint.py --changed reports 332 branch-wide warnings across 132 changed files. None intersect lines added by the review-finding revisions.
  • trunk check was unavailable because Trunk is not installed in this environment.
  • The authoritative OpenAPI comparison still reports unrelated public-spec drift: seven source-only paths, one target-only scores path, schema differences, and an unclassified networking tag and path. The stale generated error-documentation URL is fixed; this PR does not silently fold the remaining separate OpenAPI sync into the launch audit.
  • The 2,071-word managed Kubernetes page and its four callouts are inherited canonical content moved by this PR, not new expansion.

Rollout caveats

  • factory_reply_no_reply_ack is enabled in staging and disabled in production, so the public docs do not claim the reply acknowledgment before rollout.
  • integration_setup_delegation is enabled in staging and disabled in production, so the Microsoft Teams docs cover the production disconnected state and recovery rather than the delegated setup flow.
  • The Warp CLI already documents the shipped 25-file limit for oz agent run-cloud --attach; no content change was needed.

Unverified claims

None. In particular, the pinned production server and client configuration enables Azure DevOps environments, factory identities, and automations, and the pinned canonical OpenAPI spec names the API "Warp Platform API."

Documentation risk

Risk: engineering-review-required
Rationale: This PR restructures launch documentation, changes navigation and redirects, and documents current integration defaults and infrastructure behavior across multiple product surfaces.
Source files consulted: the pinned warp-server revision's billing tier configs; production and staging feature configs; Factory production environment; public OpenAPI; Azure DevOps trigger, generated schema, and automation editor sources; router catalog; incremental benchmark scoring; and Teams handler. The current docs source was consulted for deployment patterns, self-hosting, cloud-provider federation, and Admin Panel workspace behavior. Exact paths are listed in Authoritative sources.
Requested engineering reviewers: hongyi-chen
Engineering review status: pending
Docs override: none

Co-Authored-By: Warp agent@warp.dev

Plans:

@cla-bot cla-bot Bot added the cla-signed label Sep 15, 2026
@vercel

vercel Bot commented Sep 15, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Oct 6, 2026 4:57am UTC

Request Review

@warp-for-oss

warp-for-oss Bot commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

@rachaelrenk

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 /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@rachaelrenk rachaelrenk added the warpy-factory Opened by the Warp factory agents label Sep 15, 2026

@warp-for-oss warp-for-oss Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview

This PR reorganizes Factory, Automation Platform, Agent API/SDK, and legacy Oz CLI documentation, with sidebar updates and redirects for moved pages. The overall structure is directionally clear, but several changed lines introduce broken or corrupted public documentation.

Concerns

  • src/content/docs/factories/developer-tools.mdx contains duplicated/corrupted prose in the new API and SDK section.
  • Two external documentation URLs were accidentally rewritten to non-existent paths.
  • The redirect list omits /platform/deployment-patterns even though that page moved to /factories/deployment-patterns/.
  • Comment audit: no new explanatory code comments were added; existing comments were removed or retained in sidebar context only.
  • Test audit: no tests were added or changed.
  • Spec/security audit: no approved spec context was attached; the docs/security pass found no secret exposure or unsafe examples beyond the broken documentation links called out inline.

Verdict

Found: 3 critical, 1 important, 0 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

Comment thread src/content/docs/factories/developer-tools.mdx Outdated
Comment thread src/content/docs/agents/cli/oz-cli/federate.mdx Outdated
Comment thread vercel.json

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • PR body ## Documentation risk section — [IMPORTANT] Risk is correctly declared 'engineering-review-required' for this 114-file restructure, but the block omits 'Source files consulted' and 'Requested engineering reviewers', even though new/rewritten pages (e.g. src/content/docs/factories/self-hosting/index.mdx, src/content/docs/factories/deployment-patterns.mdx) restate Enterprise-plan gating, ZDR/network, and self-hosting behavior claims. Without cited sources the independent review pass in .agents/references/doc-quality-policy.md step 2 cannot verify claims against source, and source-owner review-request resolution has nothing to key off. Requested change: add a 'Source files consulted' line (or an explicit 'N/A — reorganization of previously published content, no new technical claims' note naming the pages it was cross-checked against) so the engineering-review-required declaration is actionable.
  • src/content/docs/platform/skills-as-agents.mdx:15 — [SUGGESTION] Newly added sentence 'This page describes skills that run independently of a factory.' is a meta-opener that narrates the page instead of stating the distinction directly (AGENTS.md → Voice & tone → No meta-openers). Requested change: rewrite to state the scope directly, e.g. 'Skills that run independently of a factory are covered here; for skills scoped to a factory or its agents, see factory skills.'
  • src/content/docs/platform/integrations/github-actions.mdx:51 — [NIT] The touched line (link target updated to /agents/cli/oz-cli/api-keys/) still bolds 'Warp API Key' with a capital K, the only such casing in the repo; every other instance (e.g. src/content/docs/platform/integrations/quickstart-github-actions.mdx:17) uses 'Warp API key'. Requested change: lowercase to 'Warp API key' for consistency.

Verdict

Request changes

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • PR body:39 — [CRITICAL] The ## Documentation risk section has Engineering review status: pending. (with a trailing period). check_pr_contract.py's field parser captures the raw remainder of the line as the value, so it reads pending. instead of the valid enum value pending, and the required 'Docs technical references' CI check fails with invalid engineering review status 'pending.'; must be one of ('not-applicable', 'pending', 'approved'). Verified by running check_pr_contract.py --body against this PR body and the 108 changed docs files at head 9162794: it failed with exactly this error, and passed once the trailing period was removed. Requested resolution: edit the PR body so the line reads exactly Engineering review status: pending (no trailing period).

Verdict

Request changes

@rachaelrenk

Copy link
Copy Markdown
Contributor Author

/warp-agent-review

@warp-for-oss

warp-for-oss Bot commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

@rachaelrenk

I'm re-reviewing this pull request in response to a review request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@warp-for-oss warp-for-oss Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview

This PR restructures the Factory, API/SDK, legacy CLI, deployment, self-hosting, sidebar, and redirect documentation. I reviewed the attached PR description, annotated diff, empty spec context, docs guidance, and supplemental security guidance.

Concerns

  • The new managed self-hosting overview drops the cross-surface routing examples that the migrated backend pages still link to for CLI, schedule, integration, API, and web UI routing guidance.
  • No security findings were identified in the docs and workflow changes. The spec context was empty, so there was no implementation-spec drift to check.

Verdict

Found: 0 critical, 1 important, 0 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

Comment thread src/content/docs/factories/self-hosting/index.mdx Outdated

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • src/pages/api.astro:39-265 — [SUGGESTION] The page hardcodes the literal string "Warp Platform API" nine times (the <title>, OG/Twitter meta tags, the WarpTopbar crumb, the visually-hidden h1, the endpoint-index aria-label, and the Scalar metaData block) instead of importing { VARS } from '@data/vars' and using {VARS.WARP_PLATFORM_API} the way the .mdx content correctly does, even though the PR's own '## Unverified claims' section marks this exact product name as pending confirmation from the requested engineering reviewer (hongyi-chen). Requested resolution: import VARS in api.astro (or reuse the already-computed specInfo.title / specInfo.description, which the page already derives from the same synced spec) and replace the hardcoded occurrences, so a reviewer-driven rename or reversion stays a one-line vars.ts change instead of a manual multi-spot fix.
  • src/content/docs/platform/quickstart.mdx — [SUGGESTION] check_compression_contract.py reports the page at 1,174 words with 3 callouts, exceeding the quickstart contract's 800-word/2-callout ceiling. Verified this is not a new violation from this PR — the diff on this file only updates internal link paths (e.g. /reference/cli/quickstart/ → /agents/cli/oz-cli/quickstart/), matching the PR body's claim that this is a pre-existing, pre-restructure exception. Requested resolution: file (or link) a tracking issue for the deferred callout-consolidation/cut pass called out in the PR body, so the exception doesn't get silently re-deferred through the next navigation restructure.

Verdict

Approve with nits

Co-Authored-By: Warp <agent@warp.dev>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • src/content/docs/platform/unmanaged-execution.mdx:110-116 — [SUGGESTION] Two callouts (:::caution pinning the Docker image digest, immediately followed by :::note on pod sandboxing) sit back-to-back; this predates the PR (carried over unchanged from platform/self-hosting/unmanaged.mdx) but the file is already being touched for this rename. AGENTS.md → Callouts and hints disallows consecutive callouts. Requested resolution: merge the two callouts or move one into body prose in this PR or a fast-follow, since the file is already open.

Verdict

Approve

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • src/content/docs/platform/unmanaged-execution.mdx:138 — [IMPORTANT] The Related pages entry says 'The Standalone CLI-only agents section describes the unmanaged model conceptually,' but the restructured src/content/docs/factories/deployment-patterns.mdx no longer has a 'Standalone CLI-only agents' (or any CLI-only) section — it now only covers 'Warp-hosted execution' and 'Managed self-hosting', with unmanaged/CLI execution referenced solely via one row in the model-choice comparison table. Update this line to accurately describe the restructured target page (e.g. point to the comparison table or link directly to /platform/unmanaged-execution/ context) instead of a section that no longer exists.
  • src/content/docs/guides/agent-workflows/run-a-software-factory-in-the-cloud.mdx:34 — [SUGGESTION] States that Deployment patterns gives 'a full comparison of {WARP_AUTOMATION_PLATFORM}-hosted, CLI-based, and self-hosted execution patterns,' but the restructured factories/deployment-patterns.mdx no longer treats CLI-based/unmanaged execution as a first-class pattern with its own section — it appears only as one row in the model-choice table linking out to /platform/unmanaged-execution/. Reword to reflect the current page structure, e.g. 'a comparison of Warp-hosted and managed self-hosted execution, plus where to find unmanaged/CLI-based execution.'
  • src/content/docs/platform/quickstart.mdx — [SUGGESTION] Independently confirmed via check_compression_contract.py: this page is 1,174 words with 3 callouts, over the quickstart maximum (800 words / 2 callouts). The diff here only updates redirected link paths and the {VARS.WARP_PLATFORM_API} rename — no prose was added — so this PR does not expand the pre-existing violation, matching the PR body's justification (tracked by #782). No action required here; keep #782 open for the dedicated cut pass.

Verdict

Request changes

Co-Authored-By: Warp <agent@warp.dev>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • src/sidebar.ts:412-413 — [NIT] In the new 'Factory configuration' group, label: 'Factory configuration', and items: [ are indented one level shallower (4 tabs) than the surrounding object's opening brace and its leading comments (5 tabs), unlike every sibling group in the file. Requested resolution: re-indent lines 412-413 to 5 tabs (or run the project formatter) to match the rest of the file.

Verdict

Approve with nits

@warp-agent-staging

Copy link
Copy Markdown
Contributor

This PR was generated with Warp.

Comment @warp-staging-factory on this PR to send it follow-up work.

View run View conversation View on Slack

warp-agent-staging Bot and others added 3 commits October 6, 2026 00:23
Resolve the launch information architecture against current platform behavior, including self-hosted worker authentication and nondeprecated run endpoints.

Co-Authored-By: Warp <agent@warp.dev>
Co-Authored-By: Warp <agent@warp.dev>
Co-Authored-By: Warp <agent@warp.dev>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • src/content/docs/factories/integrations/azure-devops.mdx:13 — [CRITICAL] The page, which this PR changes, contains the marker {/* VERIFY: Confirm first-class Azure DevOps Services support is enabled in production before merging this documentation. */}. It is not listed under ## Unverified claims, so check_pr_contract.py --body fails with an unlisted VERIFY marker. Requested change: add one bullet for this marker to ## Unverified claims, or resolve the claim and remove the marker. The declared engineering-review-required risk is correct and stays.
  • src/content/docs/platform/transitioning-from-oz.mdx:14-19 — [IMPORTANT] The Future direction column makes unconfirmed forward-looking product claims: oz functionality moves toward the Warp CLI, the warp CLI is the preferred direction, SDK packages may be renamed or repackaged under Warp, and Warp Factories remains the primary product experience. The PR body and content design plan say the page excludes unconfirmed migration dates, destinations, and replacement surfaces, and no cited source supports these statements. Requested change: remove the Future direction column or limit it to claims backed by a cited source, such as Existing workflows continue to work. Otherwise add the sources to Source files consulted and list each claim under ## Unverified claims.
  • src/content/docs/factories/integrations/azure-devops.mdx:74 — [IMPORTANT] The new text says Azure DevOps doesn't add a default automation, which reverses the previous statement that new Azure DevOps factories include a default automation. Lines 102 and 104 add that definition files accept the Azure DevOps push event and source_repos for pull request events. The Source files consulted list covers only worker_host routing and self-hosted dispatch, so no cited source supports these behavior and schema claims. Requested change: cite the warp-server or client source files that define the default automations, the supported Azure DevOps events, and source_repos, or revert these additions. If they stay unverified, list each under ## Unverified claims.
  • src/content/docs/platform/quickstart.mdx:105 — [SUGGESTION] The PR edits this quickstart, which is 1,174 words with 3 callouts. The compression contract allows at most 800 words and 2 callouts, and check_compression_contract.py exits 1. The PR body justifies the overage as pre-existing and tracks the cut pass in #782, so this does not block. Requested change: keep the #782 reference and schedule the cut pass, or reclassify the page as a tutorial.
  • developers/agent-api-openapi.yaml:4 — [SUGGESTION] info.title is changed to Warp Platform API and the description now says factories are supported. The PR body says the public name is still pending engineering confirmation, and this file powers the published API reference. Requested change: hold the title and description override until the reviewer confirms the name, or record the confirmation in ## Documentation risk before merge.

Verdict

Request changes

warp-agent-staging Bot and others added 2 commits October 6, 2026 01:06
Co-Authored-By: Warp <agent@warp.dev>
Co-Authored-By: Warp <agent@warp.dev>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • src/content/docs/platform/transitioning-from-oz.mdx:17 — [IMPORTANT] The table says the Oz web app's "Existing workflows remain supported" and line 21 says "no endpoint migration is required", but the PR's own cited source (GROW-6127) records the Oz web-app retirement in the weeks after the October 6 GA. These are public support and compatibility commitments that the ## Unverified claims section does not list. Requested change: have the engineering reviewer confirm each commitment (lines 17, 18, 20, 21, 36), then either soften the wording to what is confirmed or list the commitments under ## Unverified claims.
  • vercel.json:6566 — [SUGGESTION] The new /platform/* to /factories/* redirects sit behind the existing /agent-platform/cloud-agents/:path* rule (lines 6526 and 6531), which sends moved pages such as deployment-patterns, warp-hosting, runners, and self-hosting/* to /platform/... first. Those URLs now take two hops. Requested change: add direct /agent-platform/cloud-agents/... redirects to the final /factories/... destinations, placed before the wildcard rules.
  • PR body ## Documentation risk, Source files consulted — [SUGGESTION] The entry cites a ticket (GROW-6127) and a prose description of earlier sources. The warp-server files have no @<sha> pin, so a reviewer cannot verify the claims against a fixed revision. Requested change: replace the ticket and prose with path@<sha> entries for each source file, and add the head SHA of the former Platform pages that were moved.
  • .agents/references/terminology.md:275 — [SUGGESTION] The glossary entry "Agent API" is replaced by "Warp Platform API", but the AGENTS.md list of proper-noun feature names still includes Agent API. Requested change: update AGENTS.md to match, or note in the glossary that "Agent API keys" remains the name of the credential type.

Verdict

Request changes

warp-agent-staging Bot and others added 2 commits October 6, 2026 01:16
Co-Authored-By: Warp <agent@warp.dev>
Co-Authored-By: Warp <agent@warp.dev>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • AGENTS.md:591 — [IMPORTANT] The PR body and its ## Documentation risk rationale never mention that this PR also raises the quickstart limit from 600 to 800 words (AGENTS.md, .agents/references/doc-quality-policy.md, check_compression_contract.py, the quickstart and guide skills and templates) and replaces warpdotdev/oz-agent-action with an apt-installed oz in .github/workflows/agent-docs-review.yml (about line 40). The loosened limit is the same exception this PR relies on for the 723-word factory quickstart, so the gate is changed in the PR it is applied to. Requested change: move the policy, review-workflow, and verify_review_signal changes to a separate PR, or list them in Summary and Changes and extend the risk Rationale to cover them. If the policy change stays in this PR, add a note that the quickstart exception does not depend on it.
  • src/content/docs/factories/connect-your-factory.mdx:78 — [SUGGESTION] The heading ## factory endpoints is lowercase, and line 80 starts a sentence with [factory endpoints] (also the table cell at line 32). Headings use sentence case, and a sentence-initial word is capitalized. This looks like a find-and-replace side effect. Requested change: write the heading as ## Factory endpoints and begin the sentence on line 80 with Factory endpoints.
  • src/content/docs/factories/quickstart.mdx:15 — [SUGGESTION] The opening paragraph still explains the concept and uses marketing phrasing (a fleet of agents wired to your software development lifecycle, while people stay in control of key decisions). Line 17 then opens with the meta-opener In this quickstart, you will.... A quickstart should skip explanation and state the outcome. Requested change: cut the concept paragraph to one or two plain sentences, link to how-factories-work for the model, and replace line 17 with the outcome and time budget.
  • src/content/docs/factories/quickstart.mdx:40 — [SUGGESTION] The figcaption Click + next to Factories to open the setup wizard. is procedural, and the caption on line 70 (Toggle which default agents the foreman can dispatch.) is too. Captions should orient the reader, not instruct, and the step text already gives the action. Requested change: rewrite as orienting sentences, for example The Factories section of the sidebar. and The Pick your factory agents screen.
  • src/content/docs/factories/quickstart.mdx:5 — [SUGGESTION] The description promises in less than 10 minutes, but the quickstart convention is about five minutes. The page labels two sections at about 5 minutes each and runs 723 words. Requested change: either cut the page toward the 600-word, five-minute target or state the 10-minute budget consistently in the opening paragraph and record that in the PR rationale.
  • src/content/docs/factories/index.mdx:16 — [SUGGESTION] The new definition A factory is a cloud automation loop around your software development lifecycle is abstract and is repeated in index.mdx:72 and quickstart.mdx:15. It replaces a plainer description (requests become pull requests). Requested change: restore a concrete definition (what comes in, what the agents do, what comes out) and keep it in one place, linking from the other pages.
  • src/content/docs/index.mdx:2 — [SUGGESTION] The homepage rewrite is not described in the PR body. It renames the page to Warp products and getting started and removes the Privacy and security section (SOC 2, Zero Data Retention, disabling AI features), the Open source section, the multi-model section, and the video. Requested change: add the homepage change to the PR summary, confirm that dropping the privacy and open-source content is intended, and check whether the title change affects the page's search entry.
  • src/content/docs/platform/transitioning-from-oz.mdx:2 — [SUGGESTION] The title and sidebar label About the transition from the {{WEB_APP}} promise transition information about the Oz web app. The body is only a table of current documentation and never says what is changing or when. Requested change: either add the actual transition information (what moves, what stays, timing) or retitle the page as a documentation map. Also decide whether it belongs in both the Factories and Automation Platform tabs (src/sidebar.ts:526 and src/sidebar.ts:554).
  • src/sidebar.ts:496 — [SUGGESTION] On main, the API quickstart, the Sentry monitoring example, and each individual error page were sidebar entries. Now only the Overview, endpoint pages, and Errors index are listed. factories/api-and-sdk/quickstart and factories/api-and-sdk/demo-sentry-monitoring-with-sdk are reachable only through links on developer-tools.mdx. Requested change: add the quickstart and Sentry example back under API & SDKs, or note in the PR that dropping them from the sidebar is intentional.
  • src/data/vars.ts:10 — [SUGGESTION] The comment now calls the oz binary and the Oz web app supported legacy surfaces with no end date. AGENTS.md (Automation Platform terminology) and .agents/references/terminology.md still say these names hold only until 2026-10-06. Requested change: update those two files to match the new comment, or restore the date in vars.ts, so contributors get one consistent rule.
  • src/content/docs/factories/self-hosting/managed-kubernetes.mdx:1 — [SUGGESTION] check_compression_contract.py reports 2071 words (maximum 1500) and 4 callouts (budget 2). The PR body justifies this as inherited content, so it is non-blocking. Requested change: record a follow-up issue or cut-pass owner for this page, as the contract asks for an over-maximum exception, and avoid moving more content into it.
  • .agents/references/doc-quality-policy.md:41 — [SUGGESTION] The ## Documentation risk section lists Source files consulted as default branches, generated schemas, and published docs. The file paths and SHAs are in a separate Authoritative sources section, not in the field the policy defines (path@sha). Security-sensitive Azure DevOps claims, such as the Manager's Project Collection Administrators membership and the Microsoft Graph Application.ReadWrite.OwnedBy permission in src/content/docs/factories/integrations/azure-devops.mdx:26 and :45, could not be checked against source from this review checkout. Requested change: list the specific warp-server files at the pinned SHA in Source files consulted, and leave the engineering review for hongyi-chen as pending until that owner confirms these claims.

Verdict

Request changes

warp-agent-staging Bot and others added 2 commits October 6, 2026 01:35
Remove unrelated documentation-policy and review-workflow changes, tighten the Factory quickstart to the existing compression contract, and restore moved API pages to navigation.\n\nCo-Authored-By: Warp <agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

The independent agent completed its review for this commit.

Findings

  • src/content/docs/factories/self-hosting/managed-kubernetes.mdx:13 — [SUGGESTION] Page is about 2,071 words with 4 callouts, over the 1500-word and 2-callout compression budget (the PR body justifies this as inherited content). Requested change: run a deletion-only "Cut again" pass, fold the top-of-page cross-link :::note into prose or Related pages, and move the later :::caution into the step it applies to.
  • src/content/docs/platform/transitioning-from-oz.mdx:2 — [SUGGESTION] The title is "Cloud agent documentation map" but the new URL slug is /platform/transitioning-from-oz, which describes a past rename and does not match the title or sidebar label. Requested change: rename the file to something like platform/documentation-map.mdx, update the two src/sidebar.ts references, and add a vercel.json redirect for the old slug if it has been published.
  • AGENTS.md:820 — [SUGGESTION] Removing Agent API from the proper-noun list is part of the rename to Warp Platform API, but AGENTS.md:1135 still says "Warp Agent API", src/pages/api.astro:5 still says "upstream Agent API source", and .agents/references/docs-worthiness-criteria.md:44 still says "public Agent API". Requested change: update those references so the terminology source of truth, glossary, and comments agree, leaving the separate "Agent API keys" term alone.

Verdict

Approve

warp-agent-staging Bot and others added 5 commits October 6, 2026 03:52
Preserve the Factory information architecture while incorporating current self-hosting, entitlement, model router, benchmark, and Teams behavior.\n\nCo-Authored-By: Oz <oz-agent@warp.dev>
Sync benchmark schema and UI guidance with warp-server develop at 3b4638b5.\n\nCo-Authored-By: Oz <oz-agent@warp.dev>
Restore current self-hosting, federation, and workspace guidance while preserving the Factory information architecture. Align plan eligibility, harness support, canonical links, and redirects with the launch sources.

Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>

This branch was successfully deployed

1 active deployment
Preview — 3826de73 Deployed Oct 6, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed factory:docs-factory Label associated to the "docs-factory" factory warpy-factory Opened by the Warp factory agents

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants