feat: show errors with --show-errors regardless of increase (#49)

* feat: show errors with --show-errors regardless of increase

Previously, --check-files and --stage-record-file with --show-errors
would only display detailed errors when errors had increased. Now
errors are shown for all checked files that have errors, matching
the behavior of check-all.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* docs: update README and rename "Detailed errors" to "Errors"

- Clarify that --show-errors displays errors regardless of increase
- Add example showing errors when no increase
- Rename "Detailed errors:" heading to "Errors:" for brevity

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: frankieyan

README.md

@@ -100,20 +100,28 @@ npx @doist/react-compiler-tracker
 
 ### `--show-errors`
 
-Shows error information from the compiler including file path, line number, and error reason. Can be combined with any command.
+Shows error information from the compiler including file path, line number, and error reason. Can be combined with any command. Errors are displayed for all checked files regardless of whether they increased.
 
 ```bash
 npx @doist/react-compiler-tracker --check-files --show-errors src/components/Button.tsx
 ```
 
-Example output:
+Example output when errors have increased:
 ```
-React Compiler errors have increased in:
+Errors:
+    - src/components/Button.tsx: Line 15: Cannot access refs during render (x2)
+    - src/components/Button.tsx: Line 28: Invalid hook usage
+❌ React Compiler errors have increased in:
   • src/components/Button.tsx: +3
 
-Detailed errors:
+Please fix the errors and run the command again.
+```
+
+When no new errors are introduced, you still see the existing errors:
+```
+Errors:
     - src/components/Button.tsx: Line 15: Cannot access refs during render (x2)
-    - src/components/Button.tsx: Line 28: Invalid hook usage
+✅ No new React Compiler errors in checked files
 ```
 
 ## Integration Examples

src/index.integration.test.mts

@@ -46,7 +46,7 @@ describe('CLI', () => {
             const output = runCLI(['--show-errors'])
 
             expect(output).toContain('Found 4 React Compiler issues')
-            expect(output).toContain('Detailed errors:')
+            expect(output).toContain('Errors:')
             expect(output).toMatch(/Line \d+:/)
         })
     })
@@ -83,11 +83,23 @@ describe('CLI', () => {
             const output = runCLI(['--check-files', '--show-errors', 'src/bad-hook.ts'])
 
             expect(output).toContain('React Compiler errors have increased')
-            expect(output).toContain('Detailed errors:')
+            expect(output).toContain('Errors:')
             expect(output).toContain(
                 'src/bad-hook.ts: Line 6: Cannot access refs during render (x3)',
             )
         })
+
+        it('--show-errors shows errors even when no increase', () => {
+            runCLI(['--overwrite'])
+            const output = runCLI(['--check-files', '--show-errors', 'src/bad-hook.ts'])
+
+            expect(output).toContain('Errors:')
+            expect(output).toContain(
+                'src/bad-hook.ts: Line 6: Cannot access refs during render (x3)',
+            )
+            expect(output).not.toContain('React Compiler errors have increased')
+            expect(output).toContain('✅ No new React Compiler errors')
+        })
     })
 
     describe('--overwrite', () => {
@@ -118,7 +130,7 @@ describe('CLI', () => {
             const output = runCLI(['--overwrite', '--show-errors'])
 
             expect(output).toContain('Records saved to')
-            expect(output).toContain('Detailed errors:')
+            expect(output).toContain('Errors:')
             expect(output).toMatch(/Line \d+:/)
         })
     })
@@ -149,10 +161,21 @@ describe('CLI', () => {
             const output = runCLI(['--stage-record-file', '--show-errors', 'src/bad-hook.ts'])
 
             expect(output).toContain('React Compiler errors have increased')
-            expect(output).toContain('Detailed errors:')
+            expect(output).toContain('Errors:')
             expect(output).toMatch(/Line \d+:/)
         })
 
+        it('--stage-record-file --show-errors shows errors even when no increase', () => {
+            runCLI(['--overwrite'])
+            const output = runCLI(['--stage-record-file', '--show-errors', 'src/bad-hook.ts'])
+
+            expect(output).toContain('Errors:')
+            expect(output).toContain(
+                'src/bad-hook.ts: Line 6: Cannot access refs during render (x3)',
+            )
+            expect(output).not.toContain('React Compiler errors have increased')
+        })
+
         it('checks provided files with existing records', () => {
             // First create records
             runCLI(['--overwrite'])

src/index.mts

@@ -166,7 +166,7 @@ async function runOverwriteRecords({
         let message = `✅ Records saved to ${recordsFilePath}. Found ${totalErrors} total React Compiler issues across ${compilerErrors.size} files`
 
         if (showErrors) {
-            message += '\n\nDetailed errors:'
+            message += '\n\nErrors:'
             message += formatErrorDetails()
         }
 
@@ -333,7 +333,7 @@ async function runCheckAllFiles({
         let message = `Found ${totalErrors} React Compiler issues across ${compilerErrors.size} files`
 
         if (showErrors) {
-            message += '\n\nDetailed errors:'
+            message += '\n\nErrors:'
             message += formatErrorDetails()
         }
 
@@ -401,17 +401,20 @@ function checkErrorChanges({
         }
     }
 
+    // Show detailed errors for all checked files if requested
+    if (showErrors) {
+        const errorDetails = formatErrorDetails(filePaths)
+        if (errorDetails) {
+            console.log('Errors:' + errorDetails)
+        }
+    }
+
     // Report increases (exit with error)
     if (increaseEntries.length) {
         const errorList = increaseEntries.map(([filePath, count]) => `  • ${filePath}: +${count}`)
 
         let errorMessage = `React Compiler errors have increased in:\n${errorList.join('\n')}`
 
-        if (showErrors) {
-            errorMessage += '\n\nDetailed errors:'
-            errorMessage += formatErrorDetails(increaseEntries.map(([filePath]) => filePath))
-        }
-
         errorMessage += '\n\nPlease fix the errors and run the command again.'
 
         exitWithError(errorMessage)