docs(skills): expand aeo_crosslink_audit scope to include Factories - #779
Conversation
The run log shows Factories/software factory pages surfacing as link targets in 4 of the last 5 runs even though Factories was never a pilot topic (Agents, Cloud agents, Orchestration only). Warp Factories is now the primary product focus, so this promotes Factories to a fourth pilot topic: - Add Factories (src/content/docs/factories/) to ## Scope, audited as both source pages and link targets - Extend Peec and GSC signal collection to include factory-related prompts/queries - Update the PR title format and AEO brief goal to name factories - Update the no-change report's Topic area line Co-Authored-By: Warp <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 |
![warp-for-oss[bot]](https://avatars.githubusercontent.com/in/3450139?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview
This PR broadens the internal aeo_crosslink_audit skill so its recurring cross-link audit includes the existing Warp Factories docs alongside agents, cloud agents, and orchestration. The change is limited to skill instructions and PR/no-change report templates; no public documentation pages, commands, code examples, or product behavior claims are changed.
Concerns
- No blocking concerns found.
- The supplemental security pass found no new secret-handling or unsafe external-call issue in the changed lines.
- No approved or repository spec context was provided, so there was no implementation/spec drift to assess.
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
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review summary
The independent agent completed its review for this commit.
Findings
- PR description, "Why" section — [SUGGESTION] Claims the skill "could only link into Factories docs from in-scope pages, never audit Factories pages themselves as sources," but the cited run log contradicts this: PR #612 edited
src/content/docs/factories/index.mdxandfactories/factory-agents.mdx, and PR #691 editedfactories/factory-dashboard.mdx, both adding outbound links from Factories pages (i.e. auditing them as sources). Requested resolution: revise the rationale to say the gap being closed is systematic, equal-footing scanning of Factories as a topic, not a literal prior inability to edit Factories pages as sources.
Verdict
Approve with nits
There was a problem hiding this comment.
Review summary
The independent agent completed its review for this commit.
Findings
.agents/skills/aeo_crosslink_audit/SKILL.md:25— [SUGGESTION] The new Factories bullet in the Scope list ("Factories (src/content/docs/factories/) — audited as both source pages ... on the same footing as the other three topics") is a full descriptive clause while sibling bullets (Agents, Cloud agents, Orchestration) are bare nouns. Requested change: trim the bullet toFactories (src/content/docs/factories/)and move the source/target-page nuance to a short sentence after the list for consistency.
Verdict
Approve
warp-agent-staging
Bot
added
the
factory:docs-factory
|
This PR was generated with Warp. Comment    |
![warp-agent-staging[bot]](https://avatars.githubusercontent.com/in/2196476?s=60&v=4){kind=small}
There was a problem hiding this comment.
Reviewed the full diff at a01a8890. The companion outer-loop scope is now aligned with the audit's Factories expansion; no remaining findings.
Responding as Docs Factory (V2): Open session
There was a problem hiding this comment.
Review summary
The independent agent completed its review for this commit.
Findings
- PR body
## Changessection — [SUGGESTION] Lists edits only to.agents/skills/aeo_crosslink_audit/SKILL.md, omitting the accompanying edit to.agents/skills/improve-aeo-crosslink-skill/SKILL.md:85. Requested resolution: add a bullet describing the improve-aeo-crosslink-skill edit so the description matches the full 2-file diff. .agents/skills/improve-aeo-crosslink-skill/SKILL.md:85— [NIT] "Scope (agents, cloud agents, orchestration, Warp Factories) is too narrow..." mixes the branded product name into an otherwise lowercase topic list, unlike the sibling aeo_crosslink_audit/SKILL.md, which consistently uses lowercase 'factories'. Requested resolution: change to "orchestration, factories" for consistency.
Verdict
Approve with nits
|
Your GitHub account is not connected to Warp. Connect it here. |
This branch was successfully deployed
2 participants
Summary
Promotes Factories to a fourth pilot topic for the
aeo_crosslink_auditscheduled agent, alongside Agents, Cloud agents, and Orchestration.Why
The run log (
.agents/logs/aeo_crosslink_audit_runs.md) shows Factories and software factory pages appearing in four of the last five runs (2026-08-24 PR #612, 2026-09-07 PR #691, 2026-09-14 PR #741, 2026-09-21 PR #775), including edits to Factories pages themselves such asfactories/index.mdx,factories/factory-agents.mdx, andfactories/factory-dashboard.mdx, as a side effect of auditing in-scope pages. Factories was never a scoped topic in its own right, so those edits happened opportunistically rather than from a systematic scan of Factories content on equal footing with the other three topics. Warp Factories is now the primary product focus, so this closes that gap by making Factories a first-class pilot topic instead of an incidental byproduct.Changes
.agents/skills/aeo_crosslink_audit/SKILL.mdsrc/content/docs/factories/to the Scope section, audited as both source pages and valid link targets, on the same footing as the other three topicsDocumentation risk
Risk: low
Rationale: Skill-infra-only change to aeo_crosslink_audit/SKILL.md; no src/content/docs/ pages touched, no new product claims.
Docs override: none
Unverified claims
Co-Authored-By: Warp agent@warp.dev