Skip to content

feat: Manejo PRs grandes, permisos y git add#35

Merged
thomas-vilte merged 6 commits into
masterfrom
dev
Dec 9, 2025
Merged

feat: Manejo PRs grandes, permisos y git add#35
thomas-vilte merged 6 commits into
masterfrom
dev

Conversation

@thomas-vilte

@thomas-vilte thomas-vilte commented Dec 4, 2025

Copy link
Copy Markdown
Owner
  1. Implementación de fallback para diffs grandes de PRs de GitHub

    • Propósito: Solucionar errores cuando un PR es demasiado grande (más de 20,000 líneas) y GitHub no puede generar un único diff, lo que antes provocaba un fallo.
    • Impacto técnico: Ahora, si la API de GitHub devuelve un error 406 (Not Acceptable) por un diff grande, la aplicación obtiene y concatena los diffs de cada commit individualmente. Esto mejora la robustez y permite procesar PRs de gran tamaño. Se añadieron mensajes i18n para informar al usuario.
  2. Manejo mejorado de errores de permisos al actualizar PRs en GitHub

    • Propósito: Ofrecer una experiencia más clara y una solución directa al usuario cuando su token de GitHub carece de los permisos (scopes) necesarios para actualizar un PR.
    • Impacto técnico: Se detecta el error HTTP 403 Forbidden al intentar actualizar un PR. En lugar de un error genérico, se muestra un mensaje i18n específico que detalla los scopes necesarios (por ejemplo, repo o public_repo) y los pasos para actualizar el token, reduciendo el tiempo de depuración.
  3. Refinamiento de la precisión y contexto de ejecución de git add

    • Propósito: Asegurar que el comando git add actúe de manera más precisa sobre el archivo especificado y se ejecute siempre desde la raíz del repositorio, evitando comportamientos inesperados.
    • Impacto técnico: Se cambió la llamada de git add --all -- --file a git add -- --file y se implementó la obtención de la raíz del repositorio (getRepoRoot) para establecer correctamente el directorio de trabajo (cmd.Dir) del comando git. Esto mejora la fiabilidad de las operaciones con git.

@thomas-vilte thomas-vilte changed the title feat: Implementa fallback para diffs grandes de PRs de GitHub feat: Manejo PRs grandes, permisos y git add Dec 9, 2025
@thomas-vilte thomas-vilte added feature New features fix Bug fixes refactor Code restructuring or cleaning without functional changes test Testing, trials, and coverage labels Dec 9, 2025
@thomas-vilte thomas-vilte merged commit 270edd6 into master Dec 9, 2025
1 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New features fix Bug fixes refactor Code restructuring or cleaning without functional changes test Testing, trials, and coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant