[affected][bugfix]: maxBuffer for execSync set to 50MB for large git …

…repos

apps/affected/src/changedFiles.ts

@@ -1,5 +1,6 @@
 import { execSync } from 'child_process';
 import * as github from '@actions/github';
+import { EXEC_SYNC_MAX_BUFFER } from './constants';
 
 export enum ChangeStatus {
   Added = 'added',
@@ -57,7 +58,7 @@ export const getChangedFiles = async (): Promise<ChangedFile[]> => {
     baseDiffCommand = 'git diff --name-status HEAD~1 HEAD';
   }
 
-  const output = execSync(baseDiffCommand, { encoding: 'utf-8' }).trim();
+  const output = execSync(baseDiffCommand, { encoding: 'utf-8', maxBuffer: EXEC_SYNC_MAX_BUFFER }).trim();
   if (output) {
     // Each line of output is formatted like: "<STATUS>\t<FILE_PATH>"
     changedFiles = output

apps/affected/src/constants.ts

@@ -0,0 +1 @@
+export const EXEC_SYNC_MAX_BUFFER = 50 * 1024 * 1024; // 50 MB buffer

apps/affected/src/evaluateStatementsForHashes.ts

@@ -2,14 +2,15 @@ import { execSync } from 'child_process';
 import picomatch from 'picomatch';
 import crypto from 'crypto';
 import { AST, Expression } from './parser.types';
+import { EXEC_SYNC_MAX_BUFFER } from './constants';
 
 interface EvaluationResult {
   matchedFiles: string[];
   excludedFiles: string[];
 }
 
 export function allGitFiles() {
-  return execSync('git ls-files', { encoding: 'utf-8' }).split('\n').filter(Boolean);
+  return execSync('git ls-files', { encoding: 'utf-8', maxBuffer: EXEC_SYNC_MAX_BUFFER }).split('\n').filter(Boolean);
 };
 
 export async function evaluateStatementsForHashes(statements: AST, allFiles: string[]): Promise<Record<string, string>> {

apps/e2e/src/affected/changed-files.spec.ts

@@ -1,6 +1,7 @@
 import { execSync } from 'child_process';
 import * as github from '@actions/github';
 import { ChangeStatus, mapGitStatusCode, getChangedFiles } from  "../../../affected/src/changedFiles";
+import { EXEC_SYNC_MAX_BUFFER } from '../../../affected/src/constants';
 
 jest.mock('child_process');
 jest.mock('@actions/github', () => {
@@ -69,7 +70,7 @@ describe('changed-files.spec', () => {
       mockExecSync.mockReturnValueOnce('');
       const files = await getChangedFiles();
       expect(files).toEqual([]);
-      expect(mockExecSync).toHaveBeenCalledWith('git diff --name-status base-sha head-sha', { encoding: 'utf-8' });
+      expect(mockExecSync).toHaveBeenCalledWith('git diff --name-status base-sha head-sha', { encoding: 'utf-8', maxBuffer: EXEC_SYNC_MAX_BUFFER });
     });
 
     it('parses changed files for a pull_request event', async () => {
@@ -99,23 +100,23 @@ describe('changed-files.spec', () => {
       mockExecSync.mockReturnValueOnce('');
 
       await getChangedFiles();
-      expect(mockExecSync).toHaveBeenCalledWith('git diff --name-status env-base-sha env-head-sha', { encoding: 'utf-8' });
+      expect(mockExecSync).toHaveBeenCalledWith('git diff --name-status env-base-sha env-head-sha', { encoding: 'utf-8', maxBuffer: EXEC_SYNC_MAX_BUFFER  });
     });
 
     it('handles push event by comparing HEAD~1 and HEAD', async () => {
       (github.context as any).eventName = 'push';
       mockExecSync.mockReturnValueOnce('');
 
       await getChangedFiles();
-      expect(mockExecSync).toHaveBeenCalledWith('git diff --name-status HEAD~1 HEAD', { encoding: 'utf-8' });
+      expect(mockExecSync).toHaveBeenCalledWith('git diff --name-status HEAD~1 HEAD', { encoding: 'utf-8', maxBuffer: EXEC_SYNC_MAX_BUFFER  });
     });
 
     it('handles unknown events by falling back to HEAD~1 and HEAD', async () => {
       (github.context as any).eventName = 'unknown_event';
       mockExecSync.mockReturnValueOnce('');
 
       await getChangedFiles();
-      expect(mockExecSync).toHaveBeenCalledWith('git diff --name-status HEAD~1 HEAD', { encoding: 'utf-8' });
+      expect(mockExecSync).toHaveBeenCalledWith('git diff --name-status HEAD~1 HEAD', { encoding: 'utf-8', maxBuffer: EXEC_SYNC_MAX_BUFFER  });
     });
 
     it('handles filenames with tabs (unlikely, but safe)', async () => {