fix(core): ensure --yolo does not force headless mode (#18976)

This commit is contained in:
N. Taylor Mullen
2026-02-13 07:43:50 -08:00
committed by GitHub
parent d5dfae6bbf
commit d0c6a56c65
2 changed files with 5 additions and 8 deletions

View File

@@ -120,18 +120,18 @@ describe('isHeadlessMode', () => {
}
});
it('should return true if -y or --yolo is in process.argv as a fallback', () => {
it('should return false if -y or --yolo is in process.argv as a fallback', () => {
const originalArgv = process.argv;
process.argv = ['node', 'index.js', '-y'];
try {
expect(isHeadlessMode()).toBe(true);
expect(isHeadlessMode()).toBe(false);
} finally {
process.argv = originalArgv;
}
process.argv = ['node', 'index.js', '--yolo'];
try {
expect(isHeadlessMode()).toBe(true);
expect(isHeadlessMode()).toBe(false);
} finally {
process.argv = originalArgv;
}

View File

@@ -44,9 +44,6 @@ export function isHeadlessMode(options?: HeadlessModeOptions): boolean {
return true;
}
// Fallback: check process.argv for flags that imply headless or auto-approve mode.
return process.argv.some(
(arg) =>
arg === '-p' || arg === '--prompt' || arg === '-y' || arg === '--yolo',
);
// Fallback: check process.argv for flags that imply headless mode.
return process.argv.some((arg) => arg === '-p' || arg === '--prompt');
}