use pnpm exec for tool commands in scripts

Author: jdalton

scripts/build.mjs

@@ -33,7 +33,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
@@ -44,8 +44,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
@@ -59,8 +59,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',
       },
     ])
 
@@ -72,7 +72,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/check.mjs

@@ -21,14 +21,16 @@ async function main() {
       { args: ['run', 'fix'], command: 'pnpm' },
       {
         args: [
+          'exec',
+          'eslint',
           '--config',
           '.config/eslint.config.mjs',
           '--report-unused-disable-directives',
           '.',
         ],
-        command: 'eslint',
+        command: 'pnpm',
       },
-      { args: ['--noEmit'], command: 'tsgo' },
+      { args: ['exec', 'tsgo', '--noEmit'], command: 'pnpm' },
     ])
 
     if (exitCode !== 0) {

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

@@ -45,6 +45,7 @@ async function main() {
     const linters = [
       {
         args: [
+          'exec',
           'oxlint',
           '-c=.config/oxlintrc.json',
           '--ignore-path=.oxlintignore',
@@ -56,7 +57,7 @@ async function main() {
         name: 'oxlint',
       },
       {
-        args: ['biome', 'format', '--log-level=none', '--fix', targetDir],
+        args: ['exec', 'biome', 'format', '--log-level=none', '--fix', targetDir],
         name: 'biome',
       },
     ]
@@ -65,6 +66,7 @@ async function main() {
     if (!values.dist && !values.external) {
       linters.push({
         args: [
+          'exec',
           'eslint',
           '--config',
           '.config/eslint.config.mjs',
@@ -81,7 +83,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