docs(factories): lengthen automations title for SEO and match sidebar label - #652
docs(factories): lengthen automations title for SEO and match sidebar label#652warp-factories[bot] wants to merge 3 commits into
Conversation
Give the factories automations page a unique SEO title while keeping the short sidebar label. Other short-title warnings are intentional allowlisted exceptions. Co-Authored-By: Oz <oz-agent@warp.dev>
|
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 updates the factory automations page frontmatter title from Automations to Factory automations while preserving the short sidebar label. The change is scoped to SEO title clarity and aligns with nearby factory page title patterns.
Concerns
- No blocking concerns found.
- No approved spec context was provided, so there is no implementation spec drift to assess.
- No security-relevant code or configuration changes were introduced.
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
Update the sidebar label for the automations page from 'Automations' to 'Factory automations', matching the 'Factory agents' and 'Factory skills' pages that precede it in the Factory configuration group. Co-Authored-By: Warp <agent@warp.dev>
|
@rachaelrenk Agreed, and Hongyi confirmed as well — updated. The "Factory configuration" group's two preceding entries are Factory agents and Factory skills, so I renamed this page's sidebar label from "Automations" to "Factory automations" to match. I updated it in both places that control it: the page's own I left the two pages after this one in the group — "Definitions as code" and "Infrastructure & security" — unprefixed, since neither your comment nor Hongyi's reply asked for a broader rename across the section; happy to revisit if you'd like those touched too. Verified with |
There was a problem hiding this comment.
Overview
The rework correctly makes the page title and both sidebar sources read "Factory automations," matching the adjacent Factory configuration pages. CI, the docs build, and the internal-link audit pass, and the generated navigation and llms index carry the new label.
Findings
Found: 0 critical, 1 important, 0 suggestions, 0 nits
Verdict
Request changes — add current visual proof of the rendered navigation change to the PR before merge.
| and filters which events start a run. | ||
| sidebar: | ||
| label: Automations | ||
| label: Factory automations |
There was a problem hiding this comment.



Summary
Automated SEO audit of docs.warp.dev found 7
title_too_shortwarnings across 380 pages. Six match intentional allowlisted short titles. This PR fixes the remaining page by giving factory automations a descriptive frontmatter title, and (per review feedback) a matching sidebar label.Changes
src/content/docs/factories/automations.mdxtitle: Automations→title: Factory automationssidebar.label: Automations→sidebar.label: Factory automationssrc/sidebar.tsfactories/automationssidebar override tolabel: 'Factory automations'to match the rendered label above.Audit notes
Allowlisted short titles left unchanged:
Rework changes
Addressed review feedback from @rachaelrenk (requester @hongyi-chen agreed):
sidebar.label) and thesrc/sidebar.tsoverride that renders it, so the two never desync. Left the two following siblings ("Definitions as code", "Infrastructure & security") unchanged since neither the comment nor the requester's agreement asked for a broader group rename.npm run build(build succeeds, rendereddist/client/factories/automations/index.htmlshows "Factory automations" in the sidebar nav, page title, and H1) and.agents/skills/check_for_broken_links/check_links.py --internal-only(0 broken links across 380 files / 3967 internal links).Originating thread: https://warpdev.slack.com/archives/C09BVK0PL3Y/p1788311846296959
Co-Authored-By: Warp agent@warp.dev