Repository navigation
improvement(docs): reduce navigation payload and build work - #9285
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
 [Medium impact] The PR appears safe to merge; no actionable new issues were found. SummaryThis PR reduces docs navigation and page payloads.
The latest changes load sitemap dates from async page data and guard browsers without Reviews (4) · Last reviewed commit: "fix(docs): preserve sitemap dates with l..." · Reviewed by Greptile |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
All reported issues were addressed across 9 files
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
|
@greptile |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
No issues found across 11 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
waleedlatif1
changed the title
|
@greptile |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 11 files
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
|
@greptile |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
1 issue found across 12 files
Confidence score: 5/5
apps/docs/app/sitemap.tsloads each page sequentially, which can slow sitemap generation. UsePromise.allto load pages concurrently.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="apps/docs/app/sitemap.ts">
<violation number="1" location="apps/docs/app/sitemap.ts:12">
P3: The loop awaits each page's `load()` in series, so sitemap generation waits on every MDX body one after another. Collect the promises and await them together with `Promise.all` so the loads run concurrently.</violation>
</file>
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
This branch was successfully deployed
1 participant
Summary
Type of Change
Testing
All CI checks passed, including the docs production build, HTTP/content negotiation checks, and all 10 desktop/mobile browser cases. Browser checks cover navigation, every API request language, and Markdown clipboard success, failure, retry, and client navigation; CI uploads screenshots and a JSON report. The Vercel preview deployment succeeded and was checked in the browser.
Scoped Biome, diff checks, and Fumadocs source generation also passed. No heavy local build or test suite was run.
Checklist
test-auditauthoring gate)