Add prepare script to package.json #152
Conversation
This allows package installs from git refs to work properly, because it will build the dist dir.
There was a problem hiding this comment.
Pull Request Overview
This PR adds a prepare script to the package.json that runs the build process. This ensures that the package is built automatically when dependencies are installed, which is particularly useful for git dependencies or when the package is linked locally via npm link.
Key Changes
- Added a
preparenpm lifecycle script that executesnpm run build
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "build:cjs:generatePackageJson": "echo '{\\n \"type\": \"commonjs\"\\n}' > dist/commonjs/package.json", | ||
| "build:watch": "chokidar 'tsconfig*.json' 'src/**/*.{ts,tsx}' -c 'npm run build' --initial", | ||
| "typecheck": "tsc --noEmit", | ||
| "prepare": "npm run build", |
There was a problem hiding this comment.
The prepare script runs during npm install for all consumers, which could significantly slow down installation times. Consider using prepublishOnly (which is already present) for build automation before publishing, or use postinstall only if absolutely necessary. The prepare script is typically reserved for development dependencies setup, not full builds.
| "prepare": "npm run build", |
There was a problem hiding this comment.
prepare is not run when a package is installed via the npm registry, which is the most relevant case. This change will not increase the install time for developers using a tagged version.
1 participant
This allows package installs from git refs to work properly, because it will build the dist dir.
With this I am able to reference my fork / changes included in #151 in via
mkcode/better-auth#fixedand have it work seamlessly.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.