Merge pull request #148 from codacy/add-language

feat: run cli discover on file creation CF-1720

Author: nedaKaighobadi

src/cli/CodacyCli.ts

@@ -36,6 +36,8 @@ export abstract class CodacyCli {
 
   public abstract analyze(options: { file?: string; tool?: string }): Promise<ProcessedSarifResult[] | null>
 
+  public abstract configDiscover(filePath: string): Promise<void>
+
   public getCliCommand(): string {
     return this._cliCommand
   }

src/cli/MacCodacyCli.ts

@@ -220,4 +220,36 @@ export class MacCodacyCli extends CodacyCli {
       }
     }
   }
+
+  public async configDiscover(filePath: string): Promise<void> {
+    await this.preflightCodacyCli(true)
+
+    if (!this.getCliCommand()) {
+      throw new Error('CLI command not found. Please install the CLI first.')
+    }
+
+    Logger.debug(`Running Codacy CLI config discover for ${filePath}`)
+
+    try {
+      const { stdout, stderr } = await this.execAsync(
+        `${this.getCliCommand()} config discover ${this.preparePathForExec(filePath)}`
+      )
+
+      if (stderr) {
+        Logger.warn(`Codacy CLI config discover warnings: ${stderr}`)
+      }
+
+      if (stdout) {
+        Logger.debug(`Codacy CLI config discover output: ${stdout}`)
+      }
+
+      Logger.debug(`Codacy CLI config discover completed for ${filePath}`)
+    } catch (error: unknown) {
+      if (error instanceof CodacyError) {
+        throw error
+      } else {
+        throw new CodacyError('Failed to run Codacy config discover', error as Error, 'CLI')
+      }
+    }
+  }
 }

src/cli/WinCodacyCli.ts

@@ -31,4 +31,8 @@ export class WinCodacyCli extends CodacyCli {
   public analyze(_options: { file?: string; tool?: string }): Promise<ProcessedSarifResult[] | null> {
     throw new Error(NOT_SUPPORTED)
   }
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars
+  public configDiscover(_filePath: string): Promise<void> {
+    throw new Error(NOT_SUPPORTED)
+  }
 }

src/extension.ts

@@ -22,6 +22,40 @@ import { APIState, Repository as GitRepository } from './git/git'
 import { configureGuardrails, configureMCP, updateMCPConfig, updateMCPState } from './commands/configureMCP'
 import { addRepository, joinOrganization } from './onboarding'
 
+/**
+ * Handle file creation events by running Codacy CLI config discover
+ */
+const handleFileCreation = async (filePath: string, codacyCloud: CodacyCloud) => {
+  const ignorePaths = [
+    '.git/',
+    'node_modules/',
+    '.codacy/',
+    '.vscode/',
+    '.windsurf/',
+    '.cursor/',
+    '.github/',
+    'readme/',
+  ]
+  try {
+    if (ignorePaths.some((path) => filePath.includes(path))) {
+      return
+    }
+
+    // Skip if it's not a regular file
+    const stat = await vscode.workspace.fs.stat(vscode.Uri.file(filePath))
+    if (stat.type !== vscode.FileType.File) {
+      return
+    }
+
+    Logger.debug(`File created: ${filePath}, running config discover...`)
+
+    // Run config discover for the new file
+    await codacyCloud.cli?.configDiscover(filePath)
+  } catch (error) {
+    Logger.warn(`Failed to run config discover for ${filePath}: ${error}`)
+  }
+}
+
 /**
  * Helper function to register all extension commands
  * @param context
@@ -268,6 +302,15 @@ export async function activate(context: vscode.ExtensionContext) {
       item.onClick()
     })
 
+    // Listen for file creation and run config discover
+    context.subscriptions.push(
+      vscode.workspace.onDidCreateFiles(async (event) => {
+        for (const file of event.files) {
+          await handleFileCreation(file.fsPath, codacyCloud)
+        }
+      })
+    )
+
     const analysisMode = vscode.workspace.getConfiguration().get('codacy.cli.analysisMode')
     const cliVersion = vscode.workspace.getConfiguration().get('codacy.cli.cliVersion')
     // When the user doesn't have a specific version, update the CLI to the latest version

src/views/ProblemsDiagnosticCollection.ts

@@ -63,8 +63,8 @@ export class CliIssueDiagnostic extends vscode.Diagnostic {
       result.level === 'error'
         ? vscode.DiagnosticSeverity.Error
         : result.level === 'warning'
-        ? vscode.DiagnosticSeverity.Warning
-        : vscode.DiagnosticSeverity.Information
+          ? vscode.DiagnosticSeverity.Warning
+          : vscode.DiagnosticSeverity.Information
 
     const range = new vscode.Range(
       (result.region?.startLine || 1) - 1,