Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

🔄 Implement specific error handling with retry logic

Abierto
  • ¿Algo no está bien?
#66 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
35/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
git, javascript
Área
api, cli

Línea de trabajo

Comienza leyendo el manejador de errores en index.js:111-114 y, después, sigue la llamada a OpenAI y el código de carga de configuración referenciados en el issue. Revisa la sección 4 de claudedocs/quality-analysis-report.md e identifica las pruebas existentes antes de cambiar el comportamiento. Se considera terminado cuando estén cubiertos el manejo específico de errores, el comportamiento de reintento, la recuperación de la configuración, los mensajes accionables, las pruebas y la documentación.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

enhancement error-handling high-priority

Problem

Current error handling is too generic (index.js:111-114). All errors result in immediate process termination without distinguishing between recoverable and fatal errors.

} catch (error) {
  console.error('Error while summarizing Git changes:', error)
  process.exit(1) // ⚠️ All errors treated the same
}

Issues:

  • No retry logic for transient failures (network issues, rate limits)
  • All errors exit with same code (1)
  • No helpful recovery suggestions
  • Poor user experience for common issues

Current Behavior

  • ❌ Network timeout → immediate exit
  • ❌ OpenAI rate limit (429) → immediate exit
  • ❌ Invalid API key → immediate exit
  • ❌ Git command error → immediate exit

All errors show generic message without actionable guidance.

Proposed Solution

1. Specific Error Type Handling
} catch (error) {
  // Git not found
  if (error.code === 'ENOENT') {
    console.error('❌ Git not found. Please install git.')
    console.error('   Visit: https://git-scm.com/downloads')
    process.exit(1)
  }
  
  // OpenAI rate limit
  if (error.status === 429) {
    console.error('⏱️  OpenAI rate limit reached.')
    console.error('   Please wait a moment and try again.')
    process.exit(1)
  }
  
  // Network error
  if (error.code === 'ENOTFOUND' || error.code === 'ETIMEDOUT') {
    console.error('🌐 Network error. Check your internet connection.')
    process.exit(1)
  }
  
  // Invalid API key
  if (error.status === 401) {
    console.error('🔑 Invalid OpenAI API key.')
    console.error('   Run: git gpt open-api-key add')
    process.exit(1)
  }
  
  // Generic error with context
  console.error('❌ Unexpected error:', error.message)
  console.error('   Please report at: https://github.com/laststance/git-gpt-commit/issues')
  if (error.stack) {
    console.error('\nStack trace:', error.stack)
  }
  process.exit(1)
}
2. Retry Logic with Exponential Backoff
/**
 * Call OpenAI API with retry logic for transient failures
 * @param {Object} parameters - OpenAI API parameters
 * @param {number} maxRetries - Maximum retry attempts
 * @returns {Promise<Object>} API response
 */
async function callOpenAIWithRetry(parameters, maxRetries = 3) {
  for (let i = 0; i < maxRetries; i++) {
    try {
      return await openai.chat.completions.create(parameters)
    } catch (error) {
      const isRateLimit = error.status === 429
      const isNetworkError = ['ETIMEDOUT', 'ECONNRESET'].includes(error.code)
      const shouldRetry = (isRateLimit || isNetworkError) && i < maxRetries - 1
      
      if (shouldRetry) {
        const delay = Math.pow(2, i) * 1000 // Exponential backoff: 1s, 2s, 4s
        console.log(`⏳ ${isRateLimit ? 'Rate limited' : 'Network error'}. Retrying in ${delay}ms... (${i + 1}/${maxRetries})`)
        await new Promise(resolve => setTimeout(resolve, delay))
      } else {
        throw error
      }
    }
  }
}
3. Config File Error Recovery
function loadConfig() {
  try {
    if (fs.existsSync(CONFIG_FILE)) {
      const configContent = fs.readFileSync(CONFIG_FILE, 'utf8')
      const config = JSON.parse(configContent)
      
      // Validate config structure
      if (typeof config !== 'object') {
        throw new Error('Config file is not a valid object')
      }
      
      // Apply config with validation
      if (config.model && typeof config.model === 'string') {
        model = config.model
      }
      // ... rest of config loading
    }
  } catch (error) {
    if (error instanceof SyntaxError) {
      console.warn('⚠️  Config file is corrupted. Using defaults.')
      console.warn(`   Config location: ${CONFIG_FILE}`)
      console.warn('   Consider deleting and reconfiguring.')
    } else {
      console.error('Error loading configuration:', error.message)
    }
    // Continue with defaults
  }
}

Benefits

  • ✅ Better user experience with actionable error messages
  • ✅ Automatic recovery from transient failures
  • ✅ Reduced frustration from rate limits
  • ✅ Clearer guidance for fixing issues
  • ✅ More robust error handling

Acceptance Criteria

  • Implement specific error type handling for common scenarios
  • Add retry logic with exponential backoff for API calls
  • Improve config file error recovery
  • Add helpful error messages with resolution steps
  • Add tests for error scenarios
  • Document error handling behavior

Priority

High - Significantly impacts user experience

Related

Quality analysis report: claudedocs/quality-analysis-report.md section 4

Lenguaje dominante
JavaScript
Estrellas
36
Forks
3
Sin PR fusionados en 30 d

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de laststance/git-gpt-commit

Todos los issues de laststance/git-gpt-commit

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.