@gilbert_oliveira/commit-wizard 2.0.1 → 2.0.3

package/package.json

@@ -3,7 +3,7 @@
   "displayName": "Commit Wizard",
   "publisher": "gilbert-oliveira",
   "description": "CLI inteligente para gerar mensagens de commit usando OpenAI",
-  "version": "2.0.1",
+  "version": "2.0.3",
   "categories": [
     "Other",
     "SCM Providers"
@@ -21,7 +21,6 @@
     "test:watch": "bun test --watch",
     "test:coverage": "bun test && node scripts/generate-lcov.js",
     "test:coverage:report": "c8 --reporter=html --reporter=text --include='src/**/*.ts' --exclude='**/*.test.ts' bun test && open coverage/lcov-report/index.html",
-    "test:coverage:upload": "bun test && node scripts/generate-lcov.js && node scripts/upload-codecov.js",
     "prepublishOnly": "bun run build && bun test",
     "start": "bun run bin/commit-wizard.ts",
     "ci:test": "bun test --reporter=verbose",
@@ -32,6 +31,7 @@
     "release:patch": "./scripts/release.sh patch",
     "release:minor": "./scripts/release.sh minor",
     "release:major": "./scripts/release.sh major",
+    "check-changes": "node -e \"const { execSync } = require('child_process'); const changed = execSync('git diff --name-only HEAD~1 HEAD', { encoding: 'utf8' }).includes('package.json') || execSync('git diff --name-only HEAD~1 HEAD', { encoding: 'utf8' }).match(/(src|bin|scripts|dist)/\\.*/); console.log(changed ? '🚀 Mudanças detectadas - deploy recomendado!' : '⚠️ Nenhuma mudança significativa detectada'); process.exit(changed ? 0 : 1);\"",
     "type-check": "bun run tsc --noEmit",
     "lint": "bun run eslint --fix",
     "format": "bun run prettier --write ."

package/src/core/index.ts

@@ -278,7 +278,11 @@ async function handleSplitMode(gitStatus: any, config: any, args: CLIArgs) {
       log.info(`🤖 Gerando commit para: ${selectedFiles.join(', ')}`);
     }
 
-    const result = await generateWithRetry(fileDiffs, config, selectedFiles.filter((file): file is string => file !== undefined));
+    const result = await generateWithRetry(
+      fileDiffs,
+      config,
+      selectedFiles.filter((file): file is string => file !== undefined)
+    );
 
     if (!result.success) {
       log.error(`❌ Erro ao gerar commit: ${result.error}`);

package/src/core/openai.ts

@@ -37,14 +37,16 @@ export function buildPrompt(
 
   // Limitar tamanho do diff para economizar tokens
   const maxDiffLength = 6000;
-  const truncatedDiff = diff.length > maxDiffLength 
-    ? diff.substring(0, maxDiffLength) + '\n... (diff truncado)'
-    : diff;
+  const truncatedDiff =
+    diff.length > maxDiffLength
+      ? diff.substring(0, maxDiffLength) + '\n... (diff truncado)'
+      : diff;
 
   // Simplificar lista de arquivos se houver muitos
-  const fileList = filenames.length > 10 
-    ? `${filenames.length} arquivos: ${filenames.slice(0, 5).join(', ')}...`
-    : filenames.join(', ');
+  const fileList =
+    filenames.length > 10
+      ? `${filenames.length} arquivos: ${filenames.slice(0, 5).join(', ')}...`
+      : filenames.join(', ');
 
   const prompt = `Gere mensagem de commit em ${language} (${config.commitStyle}).
 
@@ -219,6 +221,25 @@ export function detectCommitType(
   return 'chore';
 }
 
+/**
+ * Processa a mensagem retornada pela OpenAI removendo formatação desnecessária
+ */
+export function processOpenAIMessage(message: string): string {
+  // Remover backticks de código APENAS se a mensagem inteira está envolvida em backticks
+  // Isso preserva nomes de funções, métodos e variáveis dentro da mensagem
+  if (message.match(/^```[\s\S]*```$/)) {
+    // Se a mensagem inteira está em um bloco de código, remover as marcações
+    message = message
+      .replace(/^```(?:plaintext|javascript|typescript|python|java|html|css|json|xml|yaml|yml|bash|shell|text)?\s*/, '')
+      .replace(/\s*```$/, '');
+  }
+
+  // Remover quebras de linha extras do início e fim
+  message = message.trim();
+
+  return message;
+}
+
 /**
  * Consome a API da OpenAI para gerar mensagem de commit
  */
@@ -275,11 +296,8 @@ export async function generateCommitMessage(
       };
     }
 
-    // Remover backticks se presentes
-    message = message.replace(/^```\s*/, '').replace(/\s*```$/, '');
-
-    // Remover quebras de linha extras
-    message = message.trim();
+    // Processar mensagem para remover formatação
+    message = processOpenAIMessage(message);
 
     // Extrair tipo da mensagem gerada pela OpenAI
     const extractedType = extractCommitTypeFromMessage(message);

package/src/core/smart-split.ts

@@ -28,17 +28,21 @@ function buildContextAnalysisPrompt(
 ): string {
   // Limitar o tamanho do diff para evitar exceder tokens
   const maxDiffLength = 8000; // Limite conservador
-  const truncatedDiff = overallDiff.length > maxDiffLength 
-    ? overallDiff.substring(0, maxDiffLength) + '\n... (diff truncado)'
-    : overallDiff;
+  const truncatedDiff =
+    overallDiff.length > maxDiffLength
+      ? overallDiff.substring(0, maxDiffLength) + '\n... (diff truncado)'
+      : overallDiff;
 
   // Calcular estatísticas básicas
   const totalFiles = files.length;
-  const fileTypes = files.reduce((acc, file) => {
-    const ext = file.split('.').pop() || 'sem-extensao';
-    acc[ext] = (acc[ext] || 0) + 1;
-    return acc;
-  }, {} as Record<string, number>);
+  const fileTypes = files.reduce(
+    (acc, file) => {
+      const ext = file.split('.').pop() || 'sem-extensao';
+      acc[ext] = (acc[ext] || 0) + 1;
+      return acc;
+    },
+    {} as Record<string, number>
+  );
 
   const fileStats = Object.entries(fileTypes)
     .map(([ext, count]) => `${ext}: ${count}`)
@@ -73,12 +77,15 @@ Agrupe arquivos relacionados. Máximo 5 grupos. Responda em JSON:
  */
 function buildFallbackPrompt(files: string[]): string {
   // Agrupar arquivos por diretório
-  const filesByDir = files.reduce((acc, file) => {
-    const dir = file.split('/').slice(0, -1).join('/') || 'root';
-    if (!acc[dir]) acc[dir] = [];
-    acc[dir].push(file);
-    return acc;
-  }, {} as Record<string, string[]>);
+  const filesByDir = files.reduce(
+    (acc, file) => {
+      const dir = file.split('/').slice(0, -1).join('/') || 'root';
+      if (!acc[dir]) acc[dir] = [];
+      acc[dir].push(file);
+      return acc;
+    },
+    {} as Record<string, string[]>
+  );
 
   const dirStats = Object.entries(filesByDir)
     .map(([dir, files]) => `${dir}: ${files.length} arquivo(s)`)
@@ -133,14 +140,16 @@ export async function analyzeFileContext(
     // Decidir qual prompt usar baseado no tamanho
     const maxDiffLength = 6000; // Limite mais conservador
     const useFallback = overallDiff.length > maxDiffLength;
-    
-    const prompt = useFallback 
+
+    const prompt = useFallback
       ? buildFallbackPrompt(files)
       : buildContextAnalysisPrompt(files, overallDiff);
 
     // Log opcional sobre o uso do fallback
     if (useFallback) {
-      console.warn(`⚠️  Diff muito grande (${overallDiff.length} chars), usando análise baseada em nomes de arquivos`);
+      console.warn(
+        `⚠️  Diff muito grande (${overallDiff.length} chars), usando análise baseada em nomes de arquivos`
+      );
     }
 
     const response = await fetch('https://api.openai.com/v1/chat/completions', {
@@ -250,7 +259,7 @@ export async function generateGroupDiff(group: FileGroup): Promise<string> {
         const diff = getFileDiff(file);
         // Limitar tamanho do diff individual
         const maxDiffLength = 4000;
-        return diff.length > maxDiffLength 
+        return diff.length > maxDiffLength
           ? diff.substring(0, maxDiffLength) + '\n... (diff truncado)'
           : diff;
       } catch {
@@ -294,10 +303,12 @@ export async function generateGroupDiff(group: FileGroup): Promise<string> {
             });
             // Limitar conteúdo do arquivo novo
             const maxContentLength = 2000;
-            const truncatedContent = content.length > maxContentLength 
-              ? content.substring(0, maxContentLength) + '\n... (conteúdo truncado)'
-              : content;
-            
+            const truncatedContent =
+              content.length > maxContentLength
+                ? content.substring(0, maxContentLength) +
+                  '\n... (conteúdo truncado)'
+                : content;
+
             return `diff --git a/${file} b/${file}\nnew file mode 100644\nindex 0000000..${Math.random().toString(36).substr(2, 7)}\n--- /dev/null\n+++ b/${file}\n@@ -0,0 +1,${truncatedContent.split('\n').length} @@\n${truncatedContent
               .split('\n')
               .map((line) => `+${line}`)
@@ -341,10 +352,12 @@ export async function generateGroupDiff(group: FileGroup): Promise<string> {
             });
             // Limitar conteúdo do arquivo recriado
             const maxContentLength = 2000;
-            const truncatedContent = content.length > maxContentLength 
-              ? content.substring(0, maxContentLength) + '\n... (conteúdo truncado)'
-              : content;
-            
+            const truncatedContent =
+              content.length > maxContentLength
+                ? content.substring(0, maxContentLength) +
+                  '\n... (conteúdo truncado)'
+                : content;
+
             return `diff --git a/${file} b/${file}\nindex 0000000..${Math.random().toString(36).substr(2, 7)} 100644\n--- a/${file}\n+++ b/${file}\n@@ -1 +1,${truncatedContent.split('\n').length} @@\n${truncatedContent
               .split('\n')
               .map((line) => `+${line}`)
@@ -361,8 +374,8 @@ export async function generateGroupDiff(group: FileGroup): Promise<string> {
   // Limitar tamanho total do diff combinado
   const combinedDiff = diffs.join('\n');
   const maxTotalLength = 8000;
-  
-  return combinedDiff.length > maxTotalLength 
+
+  return combinedDiff.length > maxTotalLength
     ? combinedDiff.substring(0, maxTotalLength) + '\n... (diff total truncado)'
     : combinedDiff;
 }
@@ -478,7 +491,9 @@ export async function handleSmartSplitMode(
 
     if (!result.success) {
       if (!args.silent) {
-        log.error(`❌ Erro ao gerar commit para ${group.name}: ${result.error}`);
+        log.error(
+          `❌ Erro ao gerar commit para ${group.name}: ${result.error}`
+        );
       }
       continue;
     }
@@ -515,11 +530,14 @@ export async function handleSmartSplitMode(
       } else {
         // Para múltiplos arquivos, usar commit normal mas com apenas os arquivos do grupo
         const { execSync } = await import('child_process');
+        // Importar função de escape do módulo git
+        const { escapeShellArg } = await import('../git/index.ts');
         try {
           // Fazer commit apenas dos arquivos do grupo
-          const filesArg = group.files.map((f) => `"${f}"`).join(' ');
+          const filesArg = group.files.map((f) => escapeShellArg(f)).join(' ');
+          const escapedMessage = escapeShellArg(result.suggestion.message || '');
           execSync(
-            `git commit ${filesArg} -m "${(result.suggestion.message || '').replace(/"/g, '\\"')}"`,
+            `git commit ${filesArg} -m ${escapedMessage}`,
             {
               stdio: 'pipe',
             }
@@ -575,11 +593,14 @@ export async function handleSmartSplitMode(
           } else {
             // Para múltiplos arquivos, usar commit normal mas com apenas os arquivos do grupo
             const { execSync } = await import('child_process');
+            // Importar função de escape do módulo git
+            const { escapeShellArg } = await import('../git/index.ts');
             try {
               // Fazer commit apenas dos arquivos do grupo
-              const filesArg = group.files.map((f) => `"${f}"`).join(' ');
+              const filesArg = group.files.map((f) => escapeShellArg(f)).join(' ');
+              const escapedMessage = escapeShellArg(commitMessage);
               execSync(
-                `git commit ${filesArg} -m "${commitMessage.replace(/"/g, '"')}"`,
+                `git commit ${filesArg} -m ${escapedMessage}`,
                 {
                   stdio: 'pipe',
                 }
@@ -624,11 +645,14 @@ export async function handleSmartSplitMode(
             } else {
               // Para múltiplos arquivos, usar commit normal mas com apenas os arquivos do grupo
               const { execSync } = await import('child_process');
+              // Importar função de escape do módulo git
+              const { escapeShellArg } = await import('../git/index.ts');
               try {
                 // Fazer commit apenas dos arquivos do grupo
-                const filesArg = group.files.map((f) => `"${f}"`).join(' ');
+                const filesArg = group.files.map((f) => escapeShellArg(f)).join(' ');
+                const escapedMessage = escapeShellArg(editAction.message || '');
                 execSync(
-                  `git commit ${filesArg} -m "${(editAction.message || '').replace(/"/g, '"')}"`,
+                  `git commit ${filesArg} -m ${escapedMessage}`,
                   {
                     stdio: 'pipe',
                   }

package/src/git/index.ts

@@ -13,6 +13,15 @@ export interface GitCommitResult {
   error?: string;
 }
 
+/**
+ * Escapa caracteres especiais para uso seguro em comandos shell
+ */
+export function escapeShellArg(arg: string): string {
+  // Para mensagens de commit, usamos aspas simples que são mais seguras
+  // Escapamos apenas aspas simples dentro da string
+  return `'${arg.replace(/'/g, "'\"'\"'")}'`;
+}
+
 /**
  * Verifica se estamos em um repositório Git
  */
@@ -80,8 +89,9 @@ export function getFileDiff(filename: string): string {
  */
 export function executeCommit(message: string): GitCommitResult {
   try {
-    // Executar commit
-    execSync(`git commit -m "${message.replace(/"/g, '\\"')}"`, {
+    // Usar aspas simples para evitar problemas com caracteres especiais
+    const escapedMessage = escapeShellArg(message);
+    execSync(`git commit -m ${escapedMessage}`, {
       stdio: 'pipe',
     });
 
@@ -115,8 +125,10 @@ export function executeFileCommit(
   message: string
 ): GitCommitResult {
   try {
-    // Commit apenas do arquivo específico
-    execSync(`git commit "${filename}" -m "${message.replace(/"/g, '\\"')}"`, {
+    // Usar aspas simples para evitar problemas com caracteres especiais
+    const escapedMessage = escapeShellArg(message);
+    const escapedFilename = escapeShellArg(filename);
+    execSync(`git commit ${escapedFilename} -m ${escapedMessage}`, {
       stdio: 'pipe',
     });
 

package/src/utils/polyfill.ts

@@ -1,6 +1,6 @@
 /**
  * Polyfill para stripVTControlCharacters para compatibilidade com Bun/Node.js
- * 
+ *
  * Este polyfill resolve problemas de compatibilidade onde o Bun não mapeia
  * corretamente a função stripVTControlCharacters do módulo util do Node.js.
  * A função foi adicionada ao Node.js v16.14.0+ mas pode não estar disponível
@@ -10,7 +10,7 @@
 /**
  * Remove caracteres de controle VT de uma string
  * Implementação baseada na função nativa do Node.js
- * 
+ *
  * @param str - String da qual remover os caracteres de controle
  * @returns String limpa sem caracteres de controle VT/ANSI
  */
@@ -23,19 +23,22 @@ function stripVTControlCharacters(str: string): string {
   // Baseada na implementação oficial do Node.js
   const esc = String.fromCharCode(27); // ESC character (\u001B)
   const csi = String.fromCharCode(155); // CSI character (\u009B)
-  const ansiRegex = new RegExp(`[${esc}${csi}][[()#;?]*(?:[0-9]{1,4}(?:;[0-9]{0,4})*)?[0-9A-ORZcf-nqry=><]`, 'g');
-  
+  const ansiRegex = new RegExp(
+    `[${esc}${csi}][[()#;?]*(?:[0-9]{1,4}(?:;[0-9]{0,4})*)?[0-9A-ORZcf-nqry=><]`,
+    'g'
+  );
+
   return str.replace(ansiRegex, '');
 }
 
 // Interceptar require/import do módulo util antes de qualquer outra coisa
- 
+
 const Module = require('module');
 const originalRequire = Module.prototype.require;
 
-Module.prototype.require = function(id: string) {
+Module.prototype.require = function (id: string) {
   const result = originalRequire.apply(this, arguments);
-  
+
   // Se estiver importando o módulo util e stripVTControlCharacters não existe, adicionar
   if (id === 'util' && result && !result.stripVTControlCharacters) {
     result.stripVTControlCharacters = stripVTControlCharacters;
@@ -47,11 +50,10 @@ Module.prototype.require = function(id: string) {
       configurable: false,
     });
   }
-  
+
   return result;
 };
 
- 
 declare global {
   // eslint-disable-next-line @typescript-eslint/no-namespace
   namespace NodeJS {
@@ -62,15 +64,17 @@ declare global {
 }
 
 // Disponibilizar globalmente também como fallback
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-if (typeof globalThis !== 'undefined' && !(globalThis as any).stripVTControlCharacters) {
+ 
+if (
+  typeof globalThis !== 'undefined' &&
+  !(globalThis as any).stripVTControlCharacters
+) {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   (globalThis as any).stripVTControlCharacters = stripVTControlCharacters;
 }
 
 // Tentar aplicar diretamente ao módulo util se possível
 try {
-   
   const util = require('util');
   if (!util.stripVTControlCharacters) {
     util.stripVTControlCharacters = stripVTControlCharacters;