docs(audit): fix missing_docs path resolution and document Vertical Tabs Summary mode#75
docs(audit): fix missing_docs path resolution and document Vertical Tabs Summary mode#75hongyi-chen wants to merge 2 commits into
Conversation
Repair the missing_docs audit script so it no longer produces false positives, refresh the feature surface map to match the live docs structure, and document the Vertical Tabs "Summary" tab item mode that shipped behind the VerticalTabsSummaryMode GA flag. Changes: - audit_docs.py: resolve mapped paths against the actual repo root (the prior code joined paths against docs_root.parent and double-counted src/content/docs/...). Accept .mdx as an alternate to .md and index.mdx as an alternate to README.md, matching the Astro Starlight layout. Fall back to a content search before flagging a mapped path as missing so renamed docs don't get flagged. - feature_surface_map.md: update mappings that pointed at the legacy agent-platform/warp-agents/ tree to the real agent-platform/capabilities/ and agent-platform/local-agents/ locations. Add VerticalTabsSummaryMode -> vertical-tabs.mdx. Move OpenWarpLaunchModal to the ignore list (one-time open-source launch modal; covered in the 2026 changelog). - vertical-tabs.mdx: document the new "Tab item" setting with "Focused session" and "Summary" options, including the table row and the key-features bullet. After the fix, the audit reports zero medium-or-higher severity gaps. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> 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 |
![oz-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 updates the missing-docs audit surface map to match the current Astro Starlight layout, fixes mapped-path resolution for .mdx and index.mdx pages in feature and CLI audits, and documents the Vertical Tabs Summary tab item mode. I reviewed the changed audit logic, mapped documentation paths, and documentation wording against the annotated diff.
Concerns
- No blocking concerns found.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
2 participants
Summary
missing_docsaudit script so it no longer produces false positives. The prior code joined surface-map paths againstdocs_root.parent(thesrc/content/directory), which doubledsrc/content/docs/...and made every mapped path appear missing. It also only checked for.mdeven though the live docs are.mdx. The script now resolves paths against the actual repo root, treats.mdxandindex.mdxas legitimate alternates of.mdandREADME.md, and falls back to a docs-wide content search before flagging a mapped path as missing.feature_surface_map.mdso it matches the live Astro Starlight layout: feature flags that previously pointed at the legacyagent-platform/warp-agents/tree now resolve toagent-platform/capabilities/andagent-platform/local-agents/.VerticalTabsSummaryModeGA flag) onterminal/windows/vertical-tabs.mdx, with a new "Tab item" row in the settings table and an entry in the key-features bullet list. Sources confirm the UI shows two segmented controls: View as (Panes/Tabs) and, when View as isTabs, Tab item (Focused session/Summary).OpenWarpLaunchModalflag as ignored. It gates a one-time open-source launch announcement already covered in the 2026 changelog (* Warp is now open source.) and has no recurring surface to document.After these changes,
python3 .agents/skills/missing_docs/scripts/audit_docs.py --severity mediumreports zero gaps.npm run buildsucceeds (318 pages built).Out of scope (follow-ups)
Test plan
python3 .agents/skills/missing_docs/scripts/audit_docs.py --warp-internal /workspace/warp-internal --warp-server /workspace/warp-server --severity mediumreports 0 gapsnpm run buildsucceeds (318 pages built)This PR was created by Oz (running Claude Code).