fix: Correct comment extraction logic and remove console.log from tests

- Fix comment extraction to only check lines BETWEEN import declarations, not WITHIN multi-line imports
- Remove all console.log statements from test files (tests should use assertions, not console output)
- Prevents inline comments (e.g., /* mid */ B) from being incorrectly extracted as standalone comments
- All 251 main tests + 162 comparison tests passing

Author: JohannesHoppe

comparison-test-harness/test-cases/09-demo-for-video.test.ts

@@ -182,8 +182,6 @@ export class DemoComponent implements OnInit {
       2,
       'Should have exactly 2 blank lines after imports in legacy mode'
     );
-
-    console.log('✅ Demo file test PASSED - both extensions produce identical, expected output');
   });
 
   test('129. Demo file - NEW extension with modern defaults (not legacy)', async () => {
@@ -309,8 +307,6 @@ export class DemoComponent implements OnInit {
       1,
       'Should have exactly 1 blank line after imports with modern defaults'
     );
-
-    console.log('✅ Demo file with modern defaults PASSED - 1 blank line as expected');
   });
 
 });

comparison-test-harness/test-cases/999-manual-proof.test.ts

@@ -97,7 +97,6 @@ const x = A + B + Z;
 `;
 
     const oldResult = await organizeImportsOld(input, { disableImportRemovalOnOrganize: true });
-    console.log('PROOF 1 - Old extension (no comments):', JSON.stringify(oldResult));
 
     assert.strictEqual(oldResult, expected, 'Old extension sorts imports without comments');
   });
@@ -115,7 +114,6 @@ const x = A + B + Z;
     // Testing what old extension ACTUALLY does with trailing line comments
     // Disable removal to focus on comment handling
     const oldResult = await organizeImportsOld(input, { disableImportRemovalOnOrganize: true });
-    console.log('PROOF 2 - Old extension (trailing comments):', JSON.stringify(oldResult));
 
     // The old extension STRIPS trailing line comments
     const expected = `import { A, B, Z } from 'lib';
@@ -140,7 +138,6 @@ const x = A + B + Z;
     // Testing what old extension ACTUALLY does with leading block comments
     // Disable removal to focus on comment handling
     const oldResult = await organizeImportsOld(input, { disableImportRemovalOnOrganize: true });
-    console.log('PROOF 3 - Old extension (leading block comment):', JSON.stringify(oldResult));
 
     // The old extension STRIPS leading block comments
     const expected = `import { A, B, Z } from 'lib';

src/imports/import-manager.ts

@@ -733,15 +733,29 @@ export class ImportManager {
     let importSectionEndLine = lastImport.getEndLineNumber() - 1;
 
     // Extract comments between imports (old TypeScript Hero moves them after imports)
+    // Only extract standalone comment lines that are BETWEEN import declarations,
+    // not lines that are WITHIN a multi-line import declaration
     const commentsBetweenImports: string[] = [];
     for (let i = importSectionStartLine; i <= importSectionEndLine; i++) {
-      const lineText = this.document.lineAt(i).text;
-      const trimmedText = lineText.trim();
-      // Check if line is a comment (and not an import statement)
-      // Check the trimmed version but preserve the original with indentation
-      if ((trimmedText.startsWith('//') || trimmedText.startsWith('/*') || trimmedText.startsWith('*')) &&
-          !trimmedText.includes('import ')) {
-        commentsBetweenImports.push(lineText);
+      const lineNumber = i + 1; // Convert to 1-indexed for comparison with ts-morph
+
+      // Check if this line is within any import declaration
+      const isWithinImport = allImports.some(imp => {
+        const start = imp.getStartLineNumber();
+        const end = imp.getEndLineNumber();
+        return lineNumber >= start && lineNumber <= end;
+      });
+
+      // Only check for standalone comments if the line is NOT within an import
+      if (!isWithinImport) {
+        const lineText = this.document.lineAt(i).text;
+        const trimmedText = lineText.trim();
+        const isStandaloneComment = (trimmedText.startsWith('//') || trimmedText.startsWith('/*') || trimmedText.startsWith('*')) &&
+                                     !trimmedText.includes('import ');
+
+        if (isStandaloneComment) {
+          commentsBetweenImports.push(lineText);
+        }
       }
     }