Automated SEO fixes: improve docs link anchors#169
Conversation
Replace redirecting URLs, improve generic link anchors, add VideoEmbed titles, and add contextual internal links for under-linked docs pages. Co-Authored-By: Oz <oz-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 |
There was a problem hiding this comment.
Overview
This PR applies broad documentation SEO cleanup by replacing redirecting or raw URLs, adding descriptive anchors, and filling in missing VideoEmbed titles. I did not find broken internal links in the changed additions, security concerns, or spec drift; spec_context.md is empty for this run.
Concerns
- Several automated rewrites still leave generic or awkward link/video labels that do not fully achieve the SEO/accessibility goal.
- No security findings.
Verdict
Found: 0 critical, 0 important, 3 suggestions
Approve with nits
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
| ::: | ||
|
|
||
| <VideoEmbed url="https://youtu.be/11rz9OYQ8Hg" /> | ||
| <VideoEmbed url="https://youtu.be/11rz9OYQ8Hg" title="Codebase Context video 1" /> |
There was a problem hiding this comment.
💡 [SUGGESTION] Codebase Context video 1 is still a generic numbered title, so it gives screen readers and search snippets little context about this embed.
| <VideoEmbed url="https://youtu.be/11rz9OYQ8Hg" title="Codebase Context video 1" /> | |
| <VideoEmbed url="https://youtu.be/11rz9OYQ8Hg" title="Managing Codebase Context indexing in Warp" /> |
Remove redundant vendor prefixes from web search documentation links so the bullets jump directly to descriptive anchors. Co-Authored-By: Oz <oz-agent@warp.dev>
Make VideoEmbed title attributes more specific and clean up Oz web app link copy in integration quickstarts. Validation: npm run build passed locally. Co-Authored-By: Oz <oz-agent@warp.dev>
Co-authored-by: oz-for-oss[bot] <277970191+oz-for-oss[bot]@users.noreply.github.com>
Document link treatment standards in the docs style guide and extend style_lint to flag generic anchors, raw URL link text, redundant link prefixes, named-page article issues, and missing or generic VideoEmbed titles. Co-Authored-By: Oz <oz-agent@warp.dev>
Clean up awkward link phrasing from SEO anchor updates, including missing articles, duplicated Oz web app wording, and pricing/login destination language. Co-Authored-By: Oz <oz-agent@warp.dev>
|
/oz-review |
|
I'm re-reviewing this pull request in response to a review 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 applies broad SEO-oriented documentation link cleanup, adds VideoEmbed titles, and extends the docs style lint skill with link and embed-title checks. The changes are functional overall, and I did not find security issues or spec-drift concerns; spec_context.md was empty.
Concerns
- A few changed link rewrites still violate the new link guidance by omitting required articles or repeating list prefixes that the new linter flags.
- One changed file still contains a raw URL anchor in the same resource list being cleaned up; the inline suggestion fixes that hunk, and similar pre-existing raw URL anchors outside commentable lines can be handled in a follow-up pass.
Verdict
Found: 0 critical, 0 important, 4 suggestions
Approve with nits
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
| The simplest way to set up the integration is **using the Oz web app**: | ||
|
|
||
| 1. Navigate to the Oz web app at [oz.warp.dev/integrations](https://oz.warp.dev/integrations). | ||
| 1. Navigate to the Oz web app at [Integrations page in the Oz web app](https://oz.warp.dev/integrations). |
There was a problem hiding this comment.
💡 [SUGGESTION] This link text needs the article required by the new style guide, and removing the extra "Oz web app at" avoids repeating the destination.
| 1. Navigate to the Oz web app at [Integrations page in the Oz web app](https://oz.warp.dev/integrations). | |
| 1. Navigate to the [Integrations page in the Oz web app](https://oz.warp.dev/integrations). |
| * **Privacy policy** - [Warp privacy policy](https://www.warp.dev/legal/privacy-policy) | ||
| * **Trust center** - [trust.warp.dev](https://trust.warp.dev) — security documentation and compliance reports. See also [Trust Center](/enterprise/security-and-compliance/trust-center/) for details on requesting SOC 2 reports, subprocessors, and vendor security assessments. | ||
| * **Subprocessors** - [warp.dev/legal/subprocessors](https://www.warp.dev/legal/subprocessors) | ||
| * **Subprocessors** - [Warp subprocessors list](https://www.warp.dev/legal/subprocessors) |
There was a problem hiding this comment.
💡 [SUGGESTION] These resource links still trip the new link-context/raw-URL checks; make the links themselves the list items and name the Trust Center destination.
| * **Privacy policy** - [Warp privacy policy](https://www.warp.dev/legal/privacy-policy) | |
| * **Trust center** - [trust.warp.dev](https://trust.warp.dev) — security documentation and compliance reports. See also [Trust Center](/enterprise/security-and-compliance/trust-center/) for details on requesting SOC 2 reports, subprocessors, and vendor security assessments. | |
| * **Subprocessors** - [warp.dev/legal/subprocessors](https://www.warp.dev/legal/subprocessors) | |
| * **Subprocessors** - [Warp subprocessors list](https://www.warp.dev/legal/subprocessors) | |
| * [Warp privacy policy](https://www.warp.dev/legal/privacy-policy) | |
| * [Warp Trust Center](https://trust.warp.dev) — security documentation and compliance reports. See also [Trust Center](/enterprise/security-and-compliance/trust-center/) for details on requesting SOC 2 reports, subprocessors, and vendor security assessments. | |
| * [Warp subprocessors list](https://www.warp.dev/legal/subprocessors) |
| ## 1. Install Warp | ||
|
|
||
| Download Warp from [warp.dev](https://www.warp.dev/download) and follow the installer for your platform. | ||
| Download Warp from [Warp download page](https://www.warp.dev/download) and follow the installer for your platform. |
There was a problem hiding this comment.
💡 [SUGGESTION] Add the article before this named destination page to match the new guidance.
| Download Warp from [Warp download page](https://www.warp.dev/download) and follow the installer for your platform. | |
| Download Warp from the [Warp download page](https://www.warp.dev/download) and follow the installer for your platform. |
| </TabItem> | ||
| <TabItem label="Linux"> | ||
| Download the package for your distribution from [warp.dev/download](https://www.warp.dev/download). For Debian/Ubuntu: | ||
| Download the package for your distribution from [Warp download page](https://www.warp.dev/download). For Debian/Ubuntu: |
There was a problem hiding this comment.
💡 [SUGGESTION] Add the article before this named destination page to match the new guidance.
| Download the package for your distribution from [Warp download page](https://www.warp.dev/download). For Debian/Ubuntu: | |
| Download the package for your distribution from the [Warp download page](https://www.warp.dev/download). For Debian/Ubuntu: |
Co-Authored-By: Oz <oz-agent@warp.dev>
Summary
titleprops toVideoEmbedcomponents so embeds have meaningful metadata for accessibility and SEO.style_lintchecks so future docs updates catch link and embed issues before review.What changed
SEO link cleanup
Video embeds
VideoEmbedtitleprops across the docs.Review follow-ups and wording polish
Style guide and linting
VideoEmbedtitles.style_lintchecks for link anchor quality, link context quality, andVideoEmbedtitle coverage/specificity.Validation
TOTAL_ISSUES=0npm run build: passedpython3 -m py_compile .agents/skills/style_lint/style_lint.py: passedpython3 .agents/skills/style_lint/style_lint.py --changed --output /tmp/style-lint-link-quality.json: passedgit --no-pager diff --check: passedtrunk check: skipped because Trunk is not installed locallyArtifacts
Co-Authored-By: Oz oz-agent@warp.dev