Reject an empty --report= even when --no-report is also given

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TrbmayRraf25aTwCCj1iv2

Author: claude

ai-slop-detector.ts

@@ -279,10 +279,10 @@ export function unknownFlags(flagArgs: readonly string[]): string[] {
 
 /** Where to write the JSON report: `--report=<path>`, nowhere with `--no-report`, else ./ai-slop-report.json. */
 export function reportPathFrom(flagArgs: readonly string[], rootDir: string): string | undefined {
-  if (flagArgs.includes('--no-report')) return undefined;
   const flag = flagArgs.filter(arg => arg.startsWith('--report=')).at(-1);
   const value = flag?.slice('--report='.length);
   if (value === '') throw new UsageError('--report needs a file path, e.g. --report=reports/karpeslop.json');
+  if (flagArgs.includes('--no-report')) return undefined;
   return path.resolve(rootDir, value ?? 'ai-slop-report.json');
 }
 

tests/ai-slop-detector.behavior.test.ts

@@ -643,6 +643,7 @@ test('the report path comes from --report=<path>, and --no-report writes none',
   assert.equal(reportPathFrom(['--report=out/k.json'], '/repo'), path.resolve('/repo', 'out/k.json'));
   assert.equal(reportPathFrom(['--report=a.json', '--no-report'], '/repo'), undefined);
   assert.throws(() => reportPathFrom(['--report='], '/repo'), /--report needs a file path/);
+  assert.throws(() => reportPathFrom(['--report=', '--no-report'], '/repo'), /--report needs a file path/);
   const projectDir = fs.mkdtempSync(path.join(os.tmpdir(), 'karpeslop-report-'));
   fs.writeFileSync(path.join(projectDir, 'index.ts'), 'export const a = 1;\n', 'utf-8');
   const run = (...flags: string[]) => spawnSync(