feat(website): add VCS landing pages and sitemap alias - #970
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Greptile SummaryThis PR publishes Astro’s generated URL set at
Confidence Score: 4/5The PR appears safe to merge, with the non-blocking concern that the new production-build behavior lacks required automated coverage. The alias generation is compatible with the current sitemap output and build environments, but filename or artifact regressions would currently be detected only through manual verification. Files Needing Attention: website/package.json, website/scripts/publish-sitemap-alias.ts Important Files Changed
Prompt To Fix All With AI### Issue 1
website/package.json:7
**Sitemap alias lacks test coverage**
The production build now relies on fixed sitemap output paths and filenames without an automated test covering the generated alias. A regression in the Astro output or copy behavior can fail deployment or publish an incorrect sitemap without being caught by the test suite.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Reviews (1): Last reviewed commit: "fix(website): publish sitemap.xml" | Re-trigger Greptile |
| "scripts": { | ||
| "dev": "astro dev", | ||
| "build": "astro build", | ||
| "build": "astro build && bun scripts/publish-sitemap-alias.ts", |
There was a problem hiding this comment.
Sitemap alias lacks test coverage
The production build now relies on fixed sitemap output paths and filenames without an automated test covering the generated alias. A regression in the Astro output or copy behavior can fail deployment or publish an incorrect sitemap without being caught by the test suite.
Context Used: guidelines.mdc Cursor rule (source)
Prompt To Fix With AI
This is a comment left during a code review.
Path: website/package.json
Line: 7
Comment:
**Sitemap alias lacks test coverage**
The production build now relies on fixed sitemap output paths and filenames without an automated test covering the generated alias. A regression in the Astro output or copy behavior can fail deployment or publish an incorrect sitemap without being caught by the test suite.
**Context Used:** guidelines.mdc Cursor rule ([source](https://github.com/modem-dev/modem/blob/main/.cursor/rules/guidelines.mdc))
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Summary
/git/,/jujutsu/, and/sapling/workflow landing pages.mdvariants for coding agents and answer engines/sitemap.xmland pointrobots.txtat itVerification
bun run --cwd website checkbun run --cwd website buildwebsite/distcmp website/dist/sitemap.xml website/dist/sitemap-0.xmlPlatform: Linux
This PR description was generated by Pi using GPT-5.6 Sol