use pnpm exec for tool commands in scripts

Author: jdalton

scripts/build.mjs

@@ -32,7 +32,7 @@ async function main() {
       logger.log('Building TypeScript declarations only...')
       const exitCode = await runSequence([
         { args: ['run', 'clean:dist:types'], command: 'pnpm' },
-        { args: ['--project', 'tsconfig.dts.json'], command: 'tsgo' },
+        { args: ['exec', 'tsgo', '--project', 'tsconfig.dts.json'], command: 'pnpm' },
       ])
       process.exitCode = exitCode
       return
@@ -43,8 +43,8 @@ async function main() {
       const exitCode = await runSequence([
         { args: ['run', 'clean:dist'], command: 'pnpm' },
         {
-          args: ['-c', '.config/rollup.dist.config.mjs'],
-          command: 'rollup',
+          args: ['exec', 'rollup', '-c', '.config/rollup.dist.config.mjs'],
+          command: 'pnpm',
         },
       ])
       process.exitCode = exitCode
@@ -58,8 +58,8 @@ async function main() {
     const srcExitCode = await runSequence([
       { args: ['run', 'clean:dist'], command: 'pnpm' },
       {
-        args: ['-c', '.config/rollup.dist.config.mjs'],
-        command: 'rollup',
+        args: ['exec', 'rollup', '-c', '.config/rollup.dist.config.mjs'],
+        command: 'pnpm',
       },
     ])
 
@@ -71,7 +71,7 @@ async function main() {
     // Build types
     const typesExitCode = await runSequence([
       { args: ['run', 'clean:dist:types'], command: 'pnpm' },
-      { args: ['--project', 'tsconfig.dts.json'], command: 'tsgo' },
+      { args: ['exec', 'tsgo', '--project', 'tsconfig.dts.json'], command: 'pnpm' },
     ])
 
     process.exitCode = typesExitCode

scripts/coverage.mjs

@@ -35,7 +35,7 @@ async function main() {
     if (values['type-only']) {
       logger.log('Collecting type coverage...')
       const exitCode = await runSequence([
-        { args: [], command: 'type-coverage' },
+        { args: ['exec', 'type-coverage'], command: 'pnpm' },
       ])
       process.exitCode = exitCode
       return
@@ -71,7 +71,7 @@ async function main() {
     }
 
     const typeExitCode = await runSequence([
-      { args: [], command: 'type-coverage' },
+      { args: ['exec', 'type-coverage'], command: 'pnpm' },
     ])
 
     process.exitCode = typeExitCode

scripts/lint-fix.mjs

@@ -20,6 +20,7 @@ async function main() {
     const linters = [
       {
         args: [
+          'exec',
           'oxlint',
           '-c=.config/.oxlintrc.json',
           '--ignore-path=.oxlintignore',
@@ -31,11 +32,12 @@ async function main() {
         name: 'oxlint',
       },
       {
-        args: ['biome', 'format', '--log-level=none', '--fix', '.'],
+        args: ['exec', 'biome', 'format', '--log-level=none', '--fix', '.'],
         name: 'biome',
       },
       {
         args: [
+          'exec',
           'eslint',
           '--config',
           '.config/eslint.config.mjs',
@@ -52,7 +54,7 @@ async function main() {
     for (const { args, name } of linters) {
       logger.log(`  - Running ${name}...`)
       // eslint-disable-next-line no-await-in-loop
-      const result = await runCommandQuiet(args[0], args.slice(1))
+      const result = await runCommandQuiet('pnpm', args)
 
       // These linters can exit with non-zero when they make fixes
       // So we don't treat that as an error