Add pnpm support

Author: sibiraj-s

README.md

@@ -124,6 +124,22 @@ steps:
       all-files: ${{ steps.filter.outputs.eslintrc == 'true' }}
 ```
 
+### Package Manager Support
+
+The action supports different package managers. You can specify which package manager to use with the `package-manager` input:
+
+```yml
+steps:
+  - uses: sibiraj-s/action-eslint@v3
+    with:
+      package-manager: 'npm' # or 'pnpm'
+```
+
+Supported package managers:
+
+- `npm` - Uses `npx eslint`
+- `pnpm` - Uses `pnpm exec eslint`
+
 ## Security
 
 For better security it is recommended to pin actions to a full length commit SHA.
@@ -132,7 +148,7 @@ Read more on [using third-party actions](https://docs.github.com/en/actions/lear
 
 ## Known Issues
 
-- Yarn 2+ is not supported
+- When using `package-manager: 'yarn'`, ensure you're using Yarn 1.x or Yarn 3+ with the modern `yarn exec` command. Yarn 2 (Berry) without proper exec setup may not work as expected.
 
 ## Debugging
 

action.yml

@@ -25,9 +25,9 @@ inputs:
   annotations:
     description: Enable or disable annotations
     required: false
-    default: true
+    default: 'true'
   working-directory:
-    description: Path to project containing npm/yarn installation
+    description: Path to project containing package manager installation
     required: false
   ignore-path:
     description: Path to ignore file.
@@ -38,8 +38,8 @@ inputs:
   all-files:
     description: Run eslint on all files.
     required: false
-    default: false
-  use-npx:
-    description: Enable or disable using npx instead of node node_modules/.bin/eslint
+    default: 'false'
+  package-manager:
+    description: Package manager to use (npm, pnpm). Defaults to npm if not specified
     required: false
-    default: false
+    default: npm

dist/main.js

@@ -6,6 +6,19 @@ import { disableAnnotations } from './annotations';
 import getFiles from './get-files';
 import getEslintArgs from './get-eslint-args';
 
+const getExecutionCommand = (): { command: string; args: string[] } => {
+  const packageManager = inputs.packageManager.toLowerCase();
+
+  switch (packageManager) {
+    case 'pnpm':
+      return { command: 'pnpm', args: ['dlx', 'eslint'] };
+    case 'npm':
+      return { command: 'npx', args: ['eslint'] };
+    default:
+      throw new Error(`Unsupported package manager: ${packageManager}. Supported: npm, pnpm`);
+  }
+};
+
 export const runEslint = async (): Promise<void> => {
   if (!inputs.annotations) {
     disableAnnotations();
@@ -19,13 +32,14 @@ export const runEslint = async (): Promise<void> => {
   }
 
   const eslintArgs = getEslintArgs();
+  const { command, args } = getExecutionCommand();
 
   const execArgs = [
-    inputs.useNpx ? 'eslint' : 'node_modules/.bin/eslint',
+    ...args,
     ...files,
     ...eslintArgs,
   ].filter(Boolean);
   const execOptions = { cwd: inputs.workingDirectory };
 
-  await exec(inputs.useNpx ? 'npx' : 'node', execArgs, execOptions);
+  await exec(command, execArgs, execOptions);
 };

dist/main.js.map

@@ -11,7 +11,7 @@ const inputs: Inputs = {
   ignorePath: getInput('ignore-path'),
   ignorePatterns: getMultilineInput('ignore-patterns'),
   allFiles: getBooleanInput('all-files'),
-  useNpx: getBooleanInput('use-npx'),
+  packageManager: getInput('package-manager'),
 };
 
 export default inputs;

src/eslint.ts

@@ -7,7 +7,7 @@ export interface Inputs {
   ignorePath: string;
   ignorePatterns: string[];
   allFiles: boolean;
-  useNpx: boolean;
+  packageManager: string;
 }
 
 export type FileNamesList = string[];