Skip to content

docs(factories): lengthen automations title for SEO and match sidebar label - #652

Open
warp-factories[bot] wants to merge 3 commits into
mainfrom
automated-seo-fixes/factory-automations-title
Open

docs(factories): lengthen automations title for SEO and match sidebar label#652
warp-factories[bot] wants to merge 3 commits into
mainfrom
automated-seo-fixes/factory-automations-title

Conversation

@warp-factories

@warp-factories warp-factories Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Automated SEO audit of docs.warp.dev found 7 title_too_short warnings 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.mdx

  • title: Automationstitle: Factory automations
  • sidebar.label: Automationssidebar.label: Factory automations

src/sidebar.ts

  • Updated the factories/automations sidebar override to label: 'Factory automations' to match the rendered label above.

Audit notes

Allowlisted short titles left unchanged:

  • Guides
  • Changelog
  • Artifacts
  • Tab Configs
  • Split panes
  • Tabs

Rework changes

Addressed review feedback from @rachaelrenk (requester @hongyi-chen agreed):

  • Sidebar label consistency (implemented, thread has no inline GraphQL thread — it's a top-level PR conversation comment, replied to directly): the "Factory configuration" sidebar group's two preceding entries are "Factory agents" and "Factory skills". Renamed this page's sidebar label from "Automations" to "Factory automations" to match, in both the page frontmatter (sidebar.label) and the src/sidebar.ts override 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.
  • Verified via npm run build (build succeeds, rendered dist/client/factories/automations/index.html shows "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

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>
@warp-factories warp-factories Bot added the seo SEO improvements label Aug 28, 2026
@cla-bot cla-bot Bot added the cla-signed label Aug 28, 2026
@vercel

vercel Bot commented Aug 28, 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 Sep 2, 2026 1:35am UTC

Request Review

@warp-factories
warp-factories Bot requested a review from dannyneira August 28, 2026 07:02
@warp-factories

Copy link
Copy Markdown
Contributor Author

This PR was generated with Warp.

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

View run View conversation

@rachaelrenk
rachaelrenk marked this pull request as ready for review August 28, 2026 13:16
@warp-for-oss

warp-for-oss Bot commented Aug 28, 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

Copy link
Copy Markdown
Contributor

If we're going to rename the page title for SEO (which I've wondered if we should add "Factories"...), we should consider updating the sidebar label as well. The preceding two pages both have start with "Factory", so "Factory automations" would match the pattern. Of course, all of these are in the "Factory configuration" group so "Factory" is kind of implied. Thoughts?
image

@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 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>
@warp-agent-staging warp-agent-staging Bot changed the title Automated SEO fixes: lengthen Factory automations title docs(factories): lengthen automations title for SEO and match sidebar label Sep 2, 2026
@warp-agent-staging warp-agent-staging Bot added the warpy-factory Opened by the Warp factory agents label Sep 2, 2026
@warp-agent-staging

Copy link
Copy Markdown
Contributor

@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 sidebar.label frontmatter and the src/sidebar.ts override that actually renders the nav entry (the override takes precedence, so both had to change together to avoid a desync).

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 npm run build (the rendered sidebar nav, page title, and H1 all show "Factory automations") and the repo's internal broken-link checker (0 broken links).

@warp-agent-staging warp-agent-staging 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

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

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.

⚠️ [IMPORTANT] This user-facing sidebar change has no visual proof attached to the PR. Add a current screenshot of the rendered docs page showing the active Factory automations sidebar item and the matching page H1, then include it in the PR's artifacts block. The static render is sufficient here; no interaction needs to be recorded.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed seo SEO improvements warpy-factory Opened by the Warp factory agents

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants