chore: standardize error handling, add .editorconfig, enhance ESLint rules, and expand test coverage

- Fix 13 untyped catch blocks to use catch (error: unknown) pattern
- Add .editorconfig for cross-IDE consistency
- Add prefer-const and no-floating-promises ESLint rules
- Fix 3 ESLint violations (let->const in agent.ts/formatter.ts, void in watch.ts)
- Add 174 new tests across 9 test files:
  utils, api, lib, analysis, analysis-orchestration,
  config-resolution, server, templates, plugin-api

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: Arthur742Ramos

.editorconfig

@@ -0,0 +1,15 @@
+root = true
+
+[*]
+indent_style = space
+indent_size = 2
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+
+[*.md]
+trim_trailing_whitespace = false
+
+[Makefile]
+indent_style = tab

eslint.config.js

@@ -15,6 +15,7 @@ export default [
       parserOptions: {
         ecmaVersion: "latest",
         sourceType: "module",
+        project: "./tsconfig.json",
       },
     },
     plugins: {
@@ -29,6 +30,8 @@ export default [
       // Allow flexibility for small CLIs; keep TS strictness handled by tsc.
       "@typescript-eslint/no-explicit-any": "warn",
       "@typescript-eslint/no-unused-vars": "error",
+      "@typescript-eslint/no-floating-promises": "error",
+      "prefer-const": "error",
     },
   },
 

src/agent.ts

@@ -1079,7 +1079,7 @@ No markdown, no explanations, just the JSON object starting with { and ending wi
     }
 
     // Cast to RepoFacts (ValidatedRepoFacts is compatible)
-    let facts = result.facts as unknown as RepoFacts;
+    const facts = result.facts as unknown as RepoFacts;
 
     // Merge with detected stack info (trust deterministic detection)
     facts.stack = {

src/commands/docs-command.ts

@@ -22,8 +22,8 @@ export async function runDocsCommand(
     }
     repoSource = await resolveRepo(repoUrl, process.cwd(), opts.branch || undefined);
     console.log(chalk.dim(`Analyzing: ${repoSource.repoInfo.fullName}`));
-  } catch (error) {
-    console.error(chalk.red(`❌ Failed to resolve repository: ${error}`));
+  } catch (error: unknown) {
+    console.error(chalk.red(`❌ Failed to resolve repository: ${error instanceof Error ? error.message : String(error)}`));
     process.exit(1);
   }
 

src/deps.ts

@@ -115,7 +115,7 @@ async function extractNpmDependencies(repoPath: string): Promise<DependencyAnaly
       peer,
       categories,
     };
-  } catch (err) {
+  } catch (err: unknown) {
     console.debug?.(`npm dep extraction skipped: ${err instanceof Error ? err.message : err}`);
     return null;
   }
@@ -172,7 +172,7 @@ async function extractCargoDependencies(repoPath: string): Promise<DependencyAna
       peer: [],
       categories: [],
     };
-  } catch (err) {
+  } catch (err: unknown) {
     console.debug?.(`Cargo dep extraction skipped: ${err instanceof Error ? err.message : err}`);
     return null;
   }

src/diff.ts

@@ -41,7 +41,7 @@ async function getPackageJsonAtRef(
       { cwd: repoPath }
     );
     return JSON.parse(stdout);
-  } catch (err) {
+  } catch (err: unknown) {
     console.debug?.(`package.json not available at ref ${ref}: ${err instanceof Error ? err.message : err}`);
     return {};
   }
@@ -108,7 +108,7 @@ async function getFileDiff(
       { cwd: repoPath, maxBuffer: FILE_DIFF_MAX_BUFFER }
     );
     return stdout;
-  } catch (err) {
+  } catch (err: unknown) {
     console.debug?.(`git diff failed for ${filePath}: ${err instanceof Error ? err.message : err}`);
     return "";
   }
@@ -145,7 +145,7 @@ function extractDependencyChanges(
     for (const dep of baseDeps) {
       if (!headDeps.has(dep)) removed.push(dep);
     }
-  } catch (err) {
+  } catch (err: unknown) {
     console.debug?.(`Dependency comparison failed: ${err instanceof Error ? err.message : err}`);
   }
 
@@ -173,7 +173,7 @@ async function extractEnvVarChanges(
     envFiles.map(async (file) => {
       try {
         return await getFileDiff(repoPath, baseRef, headRef, file);
-      } catch (err) {
+      } catch (err: unknown) {
         console.debug?.(`Env scan diff failed for ${file}: ${err instanceof Error ? err.message : err}`);
         return null;
       }
@@ -204,7 +204,7 @@ async function extractEnvVarChanges(
     codeFiles.slice(0, MAX_CODE_FILES_FOR_ENV_SCAN).map(async (file) => {
       try {
         return await getFileDiff(repoPath, baseRef, headRef, file);
-      } catch (err) {
+      } catch (err: unknown) {
         console.debug?.(`Code env scan failed for ${file}: ${err instanceof Error ? err.message : err}`);
         return null;
       }
@@ -250,7 +250,7 @@ function extractCommandChanges(
         newCommands.push(`npm run ${name}`);
       }
     }
-  } catch (err) {
+  } catch (err: unknown) {
     console.debug?.(`Command extraction failed: ${err instanceof Error ? err.message : err}`);
   }
 
@@ -290,7 +290,7 @@ async function detectBreakingChanges(
         breakingChanges.push(`Major version bump: ${baseVersion} → ${headVersion}`);
       }
     }
-  } catch (err) {
+  } catch (err: unknown) {
     console.debug?.(`Version comparison failed: ${err instanceof Error ? err.message : err}`);
   }
 
@@ -314,7 +314,7 @@ async function detectBreakingChanges(
           breakingChanges.push(`Removed export: ${match[1]} in ${file}`);
         }
       }
-    } catch (err) {
+    } catch (err: unknown) {
       console.debug?.(`Export removal check failed for ${file}: ${err instanceof Error ? err.message : err}`);
     }
   }

src/formatter.ts

@@ -69,7 +69,7 @@ function convertInlineFormatting(line: string): string {
 export function markdownToHtml(md: string): string {
   // Pull out code blocks so they aren't processed line-by-line
   const codeBlockPlaceholders: string[] = [];
-  let processed = md.replace(/```(\w*)\n([\s\S]*?)```/g, (match) => {
+  const processed = md.replace(/```(\w*)\n([\s\S]*?)```/g, (match) => {
     const idx = codeBlockPlaceholders.length;
     codeBlockPlaceholders.push(convertCodeBlocks(match));
     return `\x00CODEBLOCK_${idx}\x00`;

src/interactive.ts

@@ -391,7 +391,7 @@ export async function quickAsk(
     const answer = await session.ask(question);
     await session.stop();
     return answer;
-  } catch (error) {
+  } catch (error: unknown) {
     await session.stop();
     throw error;
   }

src/watch.ts

@@ -164,7 +164,7 @@ export function startWatch(
   const triggerCheck = () => {
     if (!running && !stopped) {
       log("Local git ref change detected, checking...");
-      check();
+      void check();
     }
   };
 

src/web/templates.ts

@@ -223,7 +223,7 @@ export function getIndexHtml(): string {
         currentJobId = jobId;
         streamProgress(jobId);
       } catch (err) {
-        addProgressItem(err.message, 'error');
+        addProgressItem(err instanceof Error ? err.message : String(err), 'error');
         btn.disabled = false;
         btn.textContent = 'Analyze';
       }