Ensure arguments forwarded to code-pushup binary

Author: matejchalk

__tests__/monorepo.test.ts

@@ -66,8 +66,8 @@ describe('monorepo projects detection', () => {
     await expect(listMonorepoProjects(DEFAULT_INPUTS)).resolves.toEqual<
       ProjectConfig[]
     >([
-      { name: 'backend', bin: 'npx nx run backend:code-pushup' },
-      { name: 'frontend', bin: 'npx nx run frontend:code-pushup' }
+      { name: 'backend', bin: 'npx nx run backend:code-pushup --' },
+      { name: 'frontend', bin: 'npx nx run frontend:code-pushup --' }
     ])
   })
 
@@ -111,10 +111,10 @@ describe('monorepo projects detection', () => {
     await expect(listMonorepoProjects(DEFAULT_INPUTS)).resolves.toEqual<
       ProjectConfig[]
     >([
-      { name: 'backoffice', bin: 'npx turbo run code-pushup -F backoffice' },
-      { name: 'website', bin: 'npx turbo run code-pushup -F website' },
-      { name: 'api', bin: 'npx turbo run code-pushup -F api' },
-      { name: 'auth', bin: 'npx turbo run code-pushup -F auth' }
+      { name: 'backoffice', bin: 'npx turbo run code-pushup -F backoffice --' },
+      { name: 'website', bin: 'npx turbo run code-pushup -F website --' },
+      { name: 'api', bin: 'npx turbo run code-pushup -F api --' },
+      { name: 'auth', bin: 'npx turbo run code-pushup -F auth --' }
     ])
   })
 
@@ -145,9 +145,9 @@ describe('monorepo projects detection', () => {
     await expect(listMonorepoProjects(DEFAULT_INPUTS)).resolves.toEqual<
       ProjectConfig[]
     >([
-      { name: 'backend', bin: 'pnpm run code-pushup -F backend' },
-      { name: 'frontend', bin: 'pnpm run code-pushup -F frontend' },
-      { name: '@repo/utils', bin: 'pnpm run code-pushup -F @repo/utils' }
+      { name: 'backend', bin: 'pnpm -F backend run code-pushup' },
+      { name: 'frontend', bin: 'pnpm -F frontend run code-pushup' },
+      { name: '@repo/utils', bin: 'pnpm -F @repo/utils run code-pushup' }
     ])
   })
 
@@ -204,8 +204,8 @@ describe('monorepo projects detection', () => {
     await expect(listMonorepoProjects(DEFAULT_INPUTS)).resolves.toEqual<
       ProjectConfig[]
     >([
-      { name: 'backend', bin: 'npm -w backend exec code-pushup' },
-      { name: 'frontend', bin: 'npm -w frontend exec code-pushup' }
+      { name: 'backend', bin: 'npm -w backend exec code-pushup --' },
+      { name: 'frontend', bin: 'npm -w frontend exec code-pushup --' }
     ])
   })
 

dist/index.js

@@ -28,8 +28,8 @@ export const npmHandler: MonorepoToolHandler = {
       .map(({ name, packageJson }) => ({
         name,
         bin: hasScript(packageJson, options.task)
-          ? `npm -w ${name} run ${options.task}`
-          : `npm -w ${name} exec ${options.task}`
+          ? `npm -w ${name} run ${options.task} --`
+          : `npm -w ${name} exec ${options.task} --`
       }))
   }
 }

dist/index.js.map

@@ -22,7 +22,7 @@ export const nxHandler: MonorepoToolHandler = {
     const projects = parseProjects(stdout)
     return projects.map(project => ({
       name: project,
-      bin: `npx nx run ${project}:${options.task}`
+      bin: `npx nx run ${project}:${options.task} --`
     }))
   }
 }

src/monorepo/handlers/npm.ts

@@ -34,7 +34,7 @@ export const pnpmHandler: MonorepoToolHandler = {
       .map(({ name, packageJson }) => ({
         name,
         bin: hasScript(packageJson, options.task)
-          ? `pnpm run ${options.task} -F ${name}`
+          ? `pnpm -F ${name} run ${options.task}`
           : `pnpm -F ${name} exec ${options.task}`
       }))
   }

src/monorepo/handlers/nx.ts

@@ -26,7 +26,7 @@ export const turboHandler: MonorepoToolHandler = {
         const projects = await handler.listProjects(options)
         return projects.map(({ name }) => ({
           name,
-          bin: `npx turbo run ${options.task} -F ${name}`
+          bin: `npx turbo run ${options.task} -F ${name} --`
         }))
       }
     }