Conversation
public/raw/, public/search-index.json, public/sitemap.xml, and lib/lastUpdated.json are all regenerated by the lib/generate-*.js scripts that run before `next build` in both the dev and build npm scripts. Tracking them in git caused noisy diffs on every PR that touched markdown (the entire public/raw/ tree is a byte-for-byte copy of pages/) and created unnecessary merge conflict risk. Since Vercel runs `npm run build` which chains the generate scripts before `next build --webpack`, these files will still be present in the static export — they just won't be in the repo.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 15 minutes and 19 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (51)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
The previous buildCommand ran next build directly, bypassing the generate-*.js scripts that produce public/raw/, search-index.json, sitemap.xml, and lastUpdated.json. This was fine when those files were tracked in git, but now that they're gitignored the build needs to generate them first. npm run build already chains the generate scripts before next build.
|
@s-adamantine Oops sorry meant to merge #118 not this one. But hopefully you are OK with it anyway! If not we can revert and discuss. |
|
Oh. I merged #118 an hour ago and somehow forgot already?! This is why I shouldn't work on Sundays 😢 |
1 participant
Summary
public/raw/,public/search-index.json,public/sitemap.xml, andlib/lastUpdated.jsonThese files are regenerated by
lib/generate-*.jsscripts that run beforenext buildin bothnpm run devandnpm run build. Since Vercel runsnpm run build, they'll still be present in every deployment.Why:
public/raw/is a byte-for-byte copy ofpages/— every markdown PR had doubled diffs and unnecessary merge conflict risk. The other three files are similarly regenerated from source on every build.Test plan
/raw/*.mdfiles correctly/sitemap.xmlpnpm devlocally still generates all files before starting the dev server