Skip to content

Move next-sitemap to devDependencies with postbuild script#69

Merged
jaseel0 merged 2 commits intogooglefrom
copilot/sub-pr-68
Mar 5, 2026
Merged

Move next-sitemap to devDependencies with postbuild script#69
jaseel0 merged 2 commits intogooglefrom
copilot/sub-pr-68

Conversation

Copy link
Contributor

Copilot AI commented Mar 5, 2026

next-sitemap was incorrectly listed as a production dependency and invoked via npx inline in the build script, causing unnecessary inclusion in the production bundle and unreliable CI execution.

Changes

  • package.json
    • Moved next-sitemap from dependenciesdevDependencies
    • Simplified build script to next build only
    • Added postbuild script to invoke next-sitemap via the locally installed binary
"scripts": {
  "build": "next build",
  "postbuild": "next-sitemap"
},
"devDependencies": {
  "next-sitemap": "^4.2.3"
}

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Mar 5, 2026
10 tasks
@vercel
Copy link
Contributor

vercel bot commented Mar 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
readme-gen-ai Ready Ready Preview, Comment Mar 5, 2026 6:53pm

Co-authored-by: jaseel0 <225665919+jaseel0@users.noreply.github.com>
Copilot AI changed the title [WIP] Add next-sitemap configuration Move next-sitemap to devDependencies with postbuild script Mar 5, 2026
@naheel0 naheel0 marked this pull request as ready for review March 5, 2026 18:55
@naheel0 naheel0 requested a review from adithyanmkd as a code owner March 5, 2026 18:55
@jaseel0 jaseel0 merged commit d0a21c2 into google Mar 5, 2026
2 of 3 checks passed
@jaseel0 jaseel0 deleted the copilot/sub-pr-68 branch March 5, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants