Skip to content

🚀 Migrate Project Creation from npm to pnpm for Faster Installs#97

Merged
triprjt merged 3 commits intomainfrom
feat/npm-pnpm
Mar 6, 2025
Merged

🚀 Migrate Project Creation from npm to pnpm for Faster Installs#97
triprjt merged 3 commits intomainfrom
feat/npm-pnpm

Conversation

@yaswanthBonumaddi
Copy link
Collaborator

🚀 Migrate Project Creation from npm to pnpm for Faster Installs

🔹 Summary

This PR replaces npm with pnpm in the Godspeed project creation flow to significantly improve dependency installation speed and efficiency.

🔹 Key Changes
• Replaced npm install with pnpm install for better performance.
• Auto-installs pnpm if missing, ensuring a smooth setup for all users.
• Improved installation speed:
• 🚀 2× faster cold installs
• ⚡ 3-4× faster cached installs
• 📂 Saves 60-80% disk space

🔹 Why This Change?

pnpm installs dependencies faster and more efficiently by using a global store and linking dependencies, reducing redundant downloads.

✅ Testing
• Verified successful project creation with pnpm installation.
• Tested behavior when pnpm is missing (auto-installs correctly).
• Ensured dependencies install in the correct project directory.

This improves developer experience by reducing setup time and ensuring consistency across environments. Let me know if any adjustments are needed! 🚀

Copy link
Contributor

@triprjt triprjt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are using npm at:

  1. /src/commands/plugin/index.ts -> install plugins
  2. src/index.ts -> spawnSync("npm", ["run", "gen-crud-api"] | prisma prepare | spawnSync("npm", ["run", "serve"],

Please make changes here

@yaswanthBonumaddi
Copy link
Collaborator Author

yaswanthBonumaddi commented Mar 5, 2025

Can you make those changes in cli (npm to pnpm) because I am in the middle of ts-context-loading. It will take more time to complete.

@yaswanthBonumaddi
Copy link
Collaborator Author

@triprjt mostly i updated pnpm i the all places where we are using npm. check this PR

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.

2 participants