fix(get_changed_files): return empty list when not a PR event (#128)

Author: ReenigneArcher

actions/get_changed_files/README.md

@@ -39,4 +39,5 @@ steps:
 - The action uses pagination to retrieve all changed files, so it works correctly for PRs with many files.
 - Changed files are returned as a newline-separated string in the `changed_files` output, which safely handles
   filenames containing spaces.
-- If no `pr_number` is provided and the action is not running in a `pull_request` event context, the action will fail.
+- If no `pr_number` is provided and the action is not running in a `pull_request` event context,
+  an empty string will be returned.

actions/get_changed_files/get_changed_files.js

@@ -31,7 +31,8 @@ async function getChangedFilesAction({ github, context, core }) {
   const prNumber = process.env.INPUT_PR_NUMBER || context.payload?.pull_request?.number;
 
   if (!prNumber) {
-    core.setFailed('No pull request number provided. Set the pr_number input or run this action in a pull_request event.');
+    console.log('Not a pull request event. Returning empty list of changed files.');
+    core.setOutput('CHANGED_FILES', '');
     return;
   }
 

tests/get_changed_files/get_changed_files.test.js

@@ -160,7 +160,7 @@ describe('getChangedFilesAction', () => {
     expect(mockCore.setOutput).toHaveBeenCalledWith('CHANGED_FILES', 'changed.js');
   });
 
-  test('should fail when no PR number is available', async () => {
+  test('should return empty list when no PR number is available (non-PR event)', async () => {
     const mockGithub = createMockGithub();
     const mockContext = {
       ...createMockContext(),
@@ -170,9 +170,9 @@ describe('getChangedFilesAction', () => {
 
     await getChangedFilesAction({ github: mockGithub, context: mockContext, core: mockCore });
 
-    expect(mockCore.setFailed).toHaveBeenCalledWith(
-      expect.stringContaining('No pull request number provided')
-    );
+    expect(mockCore.setFailed).not.toHaveBeenCalled();
+    expect(mockCore.setOutput).toHaveBeenCalledWith('CHANGED_FILES', '');
+    expect(consoleOutput.some(line => line.includes('Not a pull request event'))).toBe(true);
   });
 
   test('should fail when API throws an error', async () => {