feat: generate package-manager-aware CI scripts - #593
Conversation
| # Gather current perf measurements & compare results | ||
| git switch --detach - | ||
|
|
||
| bun install |
There was a problem hiding this comment.
could we make this a template file, with some simple templating syntax for npm/yarn/bun install & npm/yarn/bun run .
It makes more sense then having separate copies of the file
There was a problem hiding this comment.
Implemented in 44fda7d. packages/cli/src/templates/reassure-tests is now the single template, and setUpCiScript() renders its install/run placeholders for npm, Yarn, or Bun after package-manager detection. The focused init suite covers all three outputs plus placeholder removal and executable mode.
|
As far as I can tell this basically changes the template file to use bun instead of yarn. The run script is created from template but intended to be owned by the user, so they can already manually change the yarn installs => bun installs. If you want to continue working on it, then let's change scope so that changes in this PR would detect package manager, say npm/yarn/bun (pnpm might be more difficult due to need for hoisting which is not the default) and make |
|
Thanks for clarifying the intended scope. I updated the PR in 44fda7d to detect npm, Yarn, and Bun from packageManager or lockfiles, then render a single reassure-tests.sh template with the matching commands. Explicit packageManager values take precedence and Yarn remains the fallback. I also added coverage for all three declarations, npm/Yarn/Bun lockfiles, stale-lockfile precedence, executable mode, and complete placeholder rendering. yarn validate, yarn format:check, yarn build, and all 10 focused CLI tests pass. |
Summary
Verification
Part of #452