Skip to content

docs: add Factory code review recipe - #897

Draft
warp-agent-staging[bot] wants to merge 3 commits into
factory/factory-use-cases-structurefrom
factory/factory-use-case-code-review
Draft

warp-agent-staging[bot] wants to merge 3 commits into
factory/factory-use-cases-structurefrom
factory/factory-use-case-code-review

Conversation

@warp-agent-staging

@warp-agent-staging warp-agent-staging Bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

What this feature does

A code-review factory independently inspects ready pull requests, identifies actionable defects, and reports findings without editing, approving, or merging the code. Shipped in v0.2026.10.07.08.29.stable_00 (2026-10-07).

Summary

Replaces the code-review scaffold with a review-only recipe based on current pull request events, the default review role, and the public review-only factory example.

Changes

src/content/docs/factories/use-cases/code-review.mdx

  • Adds prerequisites, provider-specific initial and update event mappings, agent settings, and definition-as-code examples.
  • Adds a six-step setup and test procedure.
  • Documents re-review, independent model or harness selection, duplicate-review prevention, and human merge authority.

Content design plan

Audience and JTBD: An engineering team that wants consistent independent review on ready pull requests without giving an agent merge authority.

Problem: Existing docs describe agents and automations but do not show a focused review-only configuration or its safety boundaries.

Goals:

  • The reader can route ready pull requests to a review agent.
  • The reader can constrain findings to actionable defects and unmet requirements.
  • The reader can test that review does not mutate the branch.

Purpose and value: The recipe adapts a validated factory pattern and clarifies how automated review complements repository protection.

Content type: Feature documentation (combined) — the reader needs both decision guidance and setup steps.

Skill and template: draft_feature_doc / .agents/templates/feature-doc.md.

High-impact scenarios:

  • Covers: initial review on ready pull requests and optional review after new commits.
  • Excludes: automatic approval, branch mutation, and merging.

Unverified claims

  • GitLab and Azure DevOps event mappings were checked against current Factory automation references but not exercised in live provider integrations.

Documentation risk

Risk: engineering-review-required
Rationale: Adds new public Factory workflow guidance and configuration examples that require product review.
Docs override: none

PR stack

Dependency: #895 is the shared base and targets main. PRs #896–#901 each independently target factory/factory-use-cases-structure; no guide PR depends on another guide PR.

Human review focus

  • Confirm the recommended initial and re-review events.
  • Confirm that the output and permission boundaries match the intended review-only product path.

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

@cla-bot cla-bot Bot added the cla-signed label Oct 10, 2026
@vercel

vercel Bot commented Oct 10, 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 10, 2026 1:04am UTC

Request Review

@warp-agent-staging warp-agent-staging Bot added the warpy-factory Opened by the Warp factory agents label Oct 10, 2026
@warp-agent-staging

Copy link
Copy Markdown
Contributor Author

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

This branch was successfully deployed

1 active deployment
Preview — 4204e5c5 Deployed Oct 10, 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 warpy-factory Opened by the Warp factory agents

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants