feat(phase0): rename /agent-platform/ → /agents/ (IA restructure Phase 0) - #420
feat(phase0): rename /agent-platform/ → /agents/ (IA restructure Phase 0)#420rachaelrenk wants to merge 7 commits into
Conversation
|
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 moves the Agents docs from /agent-platform/ to /agents/, updates sidebar/cross-link references, and adds permanent redirects for the old URLs. I found no security issues, and the provided spec context contains no approved repository spec to compare against.
Concerns
- The llms custom set still exposes the old "Agent Platform" naming even though the section URL and sidebar now use Agents.
- The PR includes an unrelated
yamldependency bump and resulting lockfile churn that should be reverted unless it is intentionally required.
Verdict
Found: 0 critical, 0 important, 2 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
- Move src/content/docs/agent-platform/ → src/content/docs/agents/ - Update all sidebar slug references in src/sidebar.ts - Update astro.config.mjs llms.txt paths - Update 'agents' display-name mapping in CustomSidebar.astro - Fix all /agent-platform/ internal cross-links across 105+ content files - Add 50 permanent (308) redirects in vercel.json Rationale: the Agents tab is labeled 'Agents' but linked to /agent-platform/, creating a confusing pairing with /platform/. The /agents/ vs /platform/ URL distinction is immediately legible. This is standalone technical debt cleanup with no dependency on other IA restructure phases. Co-Authored-By: Oz <oz-agent@warp.dev>
…by sed) The bulk sed pass changed assets/agent-platform/ → assets/agents/ in 15 files, but the assets directory itself was not renamed. This restores the correct paths. Co-Authored-By: Oz <oz-agent@warp.dev>
a430962 to
7041852
Compare
…aml dep - astro.config.mjs: update llms.txt custom set label/description from 'Agent Platform' → 'Agents' to match the renamed section URL/path - package.json + package-lock.json: remove accidental yaml dep added during local dev (unrelated to this rename, not needed at build time) - agents/capabilities/slash-commands.mdx: fix remaining /agent-platform/ href links that were missed by the initial sed pass 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 performs a broad mechanical rename of the Agents docs URL prefix from /agent-platform/ to /agents/, with sidebar/config updates and matching redirects for moved pages. The migration itself looks consistent, and no security findings or approved spec-context drift were identified.
Concerns
package.jsonremoves the directyamldependency even though build-time code imports it directly inscripts/generate-openapi-json.mjsandsrc/pages/api.astro, making the build rely on a transitive dependency.
Verdict
Found: 0 critical, 1 important, 0 suggestions
Request changes
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
hongyi-chen
left a comment
There was a problem hiding this comment.
approving to unblock, leaving high level thoughts in slack thread
The frontmatter title still said "Agent platform FAQs" while the sidebar label already said "Agent FAQs". Aligning them removes the pre-rename copy that was flagged during QA review of the preview. Co-Authored-By: Oz <oz-agent@warp.dev>
Summary
Renames the Agents documentation URL prefix from
/agent-platform/to/agents/across the docs site. This standalone Phase 0 change removes ambiguity between the Agents section and the Automation Platform section before the broader information architecture restructure.Why now
The navigation label is Agents, but its URL prefix was
/agent-platform/, which was easy to confuse with/platform/. Moving the section now prevents additional pages from accumulating under the legacy prefix and reduces future redirect work.Changes
src/content/docs/agent-platform/tosrc/content/docs/agents/while preserving file history./agents/.308redirects from the legacy/agent-platform/URLs to their/agents/equivalents.assets/agent-platform/; only documentation routes changed.Notes
scripts/generate_rename_redirects.pyis included as the utility used to generate the redirect set.main.Conversation: https://staging.warp.dev/conversation/42fe184c-9601-4da8-b158-200b89b4fd9c
Co-Authored-By: Oz oz-agent@warp.dev