Repository navigation
docs: add AEO cross-links for agents, orchestration, and factories - #930
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
This PR was generated with Warp. Comment |
|
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 adds targeted internal cross-links across existing agent, orchestration, and factory documentation pages so readers can continue related workflows from pages that already mention those concepts. The changes are documentation-only, the introduced internal links resolve to existing docs pages, and the declared low documentation risk matches the cross-link-only scope.
Concerns
- No blocking concerns found.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
There was a problem hiding this comment.
Review summary
The independent agent completed its review for this commit.
Findings
- src/content/docs/agents/capabilities/skills.mdx:451 (PR-wide metadata) — [CRITICAL] The Documentation risk section contains
Engineering review status: not-required, which fails structural validation; the descriptive cross-link diff itself qualifies as low risk. Requested resolution: replace the field withEngineering review status: not-applicableor omit it, then rerun check_pr_contract.py against all six changed public pages without --enforce-engineering-gate. - src/content/docs/agents/capabilities/skills.mdx:451 — [IMPORTANT] The changed feature page has 2,165 prose words and four callouts, exceeding the 1,500-word and two-callout budgets without PR-body justification. The merge-base already has 2,136 words and four callouts. Requested resolution: document an exception rationale for both inherited overages and the instructional context retained in this narrow cross-link change, or reduce padding to meet the budgets without losing necessary choices, consequences, or success signals; do not mechanically split the page.
- src/content/docs/agents/capabilities/skills.mdx:468 — [SUGGESTION] The new Factory skills bullet repeats the same destination and scope distinction added at line 451. Requested resolution: keep one contextual pointer and remove the other; this preserves the route to factory-specific procedures without repeating the same choice within 17 lines.
Verdict
Request changes
Retain the contextual pointer beside skills-based agents and remove the repeated closing bullet from PR #930. Co-Authored-By: Oz <oz-agent@warp.dev>


Summary
Adds small internal cross-links so readers can continue agent, cloud-agent, orchestration, and factories workflows from pages that already mentioned those concepts without linking them.
AEO brief
Goal: Identify small internal cross-link improvements for agents, cloud agents, orchestration, and factories docs.
Recommendation: Update existing pages only (no new pages).
Source signals:
sc-domain:warp.dev, ~28 days ending 2026-10-16): strong traffic to factories marketing and docs (/factories/, branded "warp factories" / "warp factory" / "warp software factory" queries), plus agent CLI and multi-agent surfaces.Vocabulary map:
Docs should say: Warp Factories, factory automations, multi-agent orchestration, scheduled agents, factory skills
Content scope:
Existing docs to touch or link:
guides/agent-workflows/how-to-run-unattended-agents.mdx- trigger chooser lacked the factory automation path.agents/capabilities/skills.mdx- skills-as-agents path was covered; factory-scoped skills were not.platform/agents.mdx- Warp Factories GitHub App authorship was unlinked.factories/index.mdxand multi-agent guide - standing factory vs ad-hoc orchestration next steps.factories/automations.mdx- no pointer to standalone scheduled agents outside a factory.Links added
Reader next step
Changes
src/content/docs/guides/agent-workflows/how-to-run-unattended-agents.mdx
src/content/docs/agents/capabilities/skills.mdx
src/content/docs/platform/agents.mdx
src/content/docs/factories/index.mdx
src/content/docs/factories/automations.mdx
src/content/docs/guides/agent-workflows/how-to-run-multiple-ai-coding-agents.mdx
Open questions for human review
Documentation risk
Risk: low
Rationale: Cross-link-only edits to existing pages; no new product claims, commands, UI paths, or availability changes.
Source files consulted: none (link targets already exist in docs)
Requested engineering reviewers: none
Engineering review status: not-applicable
Docs override: none
Skills page compression exceptions
agents/capabilities/skills.mdxhas 2,150 prose words against the 1,500-word feature-page budget; the merge-base has 2,136. The retained teaching explains discovery and name conflicts, file format, argument substitution, project versus global locations, creation, and supporting files. These examples and explanations let readers choose a scope, write a usable skill, and recognize what happens when arguments are missing. This cross-link change adds only a 14-word contextual pointer to factory-specific procedures, whose details remain on Factory skills. The deletion-only pass removed the duplicate pointer rather than cutting instructional context or mechanically splitting the canonical skills-authoring page.Validation
python3 .agents/skills/style_lint/style_lint.py --changed— zero errors; the same 19 warnings as before this revision.python3 .agents/skills/check_for_broken_links/check_links.py --internal-only— zero broken links.python3 .agents/skills/validate_ui_refs/validate_ui_refs.py --changed --require-provenance— zero issues across all six changed pages.check_pr_contract.pyagainst all six changed public pages, without--enforce-engineering-gate, andcheck_pr_body.py— passed; zero VERIFY markers.npm run typecheck,npm run test:heading-vars,npm run build,npm run test:heading-tocs, andnpm run test:homepage-json-ld— passed.check_compression_contract.pyforskills.mdx— reports the two budget overages justified above, not a mechanical pass.Unverified claims