Fluxo end-to-end de IA para commits e PRs com GitContext Layer e integração com GitHub App#100
Merged
Conversation
…ext API' body='- Backend: Added asyncio import and switched the GenerateCommitMessageAgent invocation to asyncio.run(...). This enables non-blocking, asynchronous generation of the commit message within the existing sync API path.\n- Backend-Frontend API contract: Frontend now calls the diff_context endpoint at /api/prai/diff_context instead of /api/prai/gen, aligning with the new backend route.\n- Frontend: Updated response handling to use result.commit_message as the source for the commit message, instead of assembling from subject/body.\n- Frontend: Small code cleanup across several UI files (ConfigPanel.tsx, RepositorySelector.tsx, useGitOperations.ts, Index.tsx) involving minor comment/whitespace removals for clarity.\n- Rationale: Improve non-blocking behavior for AI-driven commit message generation and synchronize frontend with the updated backend API and response shape. This reduces coupling and ensures the UI renders the generated commit message correctly.\n- Notes: Ensure the Python environment supports asyncio.run (Python 3.7+). No user-facing changes beyond the updated API and message handling.'
What I changed - Back-End/api.py: Adapted to use a structured AI commit output. GenerateCommitMessageAgent now returns an object with subject and body; the code builds the final commit_message from commit_output.subject and commit_output.body. This enables richer, well-formed commit messages instead of a flat string. - Front-End/electron/main.cjs: Reworked Git command handling for robustness and UX: - Added enhanced logging and error handling for git subprocesses. - Introduced ensureGitUserConfigured to automatically set a local Git user.name/user.email if missing, using env vars or sensible defaults. - Improved executeCommit workflow: verify there are changes, add all, attempt commit, and if identity issues arise, configure locally and retry once. Supports optional push. - Minor adjustments to subprocess close handling to surface meaningful errors. - Front-End/electron/preload.cjs: Minor cleanup of a header comment to reflect current state (no functional change). - Front-End/git-genius-commit/src/pages/Index.tsx: Added auto_commit to the default config, enabling automatic commits by default. - Front-End/git-genius-commit/src/types/git.ts: Extended GitConfig with auto_commit: boolean to reflect new capability. Why - Structured AI commit messages improve readability and tooling support by separating subject and body. - Robust Git workflow reduces failures due to missing user configuration or no-change states, and provides a smoother UX with automatic config and optional push. - Default behavior now includes auto-commit, with a corresponding type flag, making the feature easily controllable from the UI. Impact - No breaking API changes for existing clients; the backend now returns a structured object but preserves the overall API surface for commit_message. Front-end changes mainly add resilience and new configuration options. - The new local Git user configuration prevents common commit failures when the environment lacks global user settings. - The UI will now reflect an auto_commit option and can be toggled via the existing configuration flow.
What I changed
- Front-End: Electron main process (
Front-End/git-genius-commit/electron/main.cjs
)
- Introduced a robust in-flight guard (monitoringState.inFlight) to prevent concurrent analyzeAndCommit invocations.
- Added payload deduplication and throttling:
- computeHash function to generate a stable hash for payloads (diff + modified files).
- lastPayloadHash and lastAnalysisTimestamp tracking, plus throttle_ms (60 seconds) to avoid re-sending identical payloads too frequently.
- Skip duplicate payloads within the throttle window.
- Enhanced analyzeAndCommit with concurrency control, deduplication, and improved error handling.
- Implemented a multi-URL-safe hash approach: Node crypto -> Web Crypto -> FNV-1a fallback to ensure a hash in diverse environments.
- Updated startMonitoring and periodic checks to honor inFlight and dedupe logic.
- Ensured finally block resets inFlight to false regardless of success or failure.
- Front-End: Config panel default change (
Front-End/git-genius-commit/src/components/ConfigPanel.tsx
)
- Set default auto_commit to true in the payload built for the config, aligning with new auto-commit behavior.
- Front-End: Typings clean-up (
Front-End/git-genius-commit/src/types/git.ts
)
- Minor trailing whitespace cleanup to keep typings clean.
- Back-End: Minor formatting fix in API file (
Back-End/api.py
)
- Removed an extra blank line near diff_context input parsing for consistency.
Why
- Prevent race conditions and excessive API calls when the monitor detects rapid changes or quick successive diffs.
- Ensure that identical payloads (diff + modified files) aren’t re-sent within a configurable window, reducing unnecessary API usage and improving stability.
- Provide a resilient hashing mechanism that works across environments (Node.js crypto, browser Web Crypto, or a deterministic fallback).
- Align UI defaults with new automatic commit behavior to improve user experience by reducing manual steps.
Notes
- The computeHash function is asynchronous and used to deduplicate payloads before making API requests.
- In-flight state is reset in a finally block to avoid leaving the monitor in a stuck state after errors.
- No breaking changes to the API on the back end; the changes are primarily on the client-side orchestration to optimize requests and concurrency.
What: Repositioned the ConfigPanel from the left column to the right column in Front-End/git-genius-commit/src/pages/Index.tsx. The component is now rendered alongside the diff/commit area and the manual analyze control, instead of in the left pane. Why: To place configuration controls closer to the commit workflow, improving accessibility while reviewing diffs and preparing commits. The ConfigPanel still receives the same config prop and onSave handler; no behavioral changes to the component were introduced. Impact: Only the UI layout changed for the Index page; no logic changes or new dependencies were added. The ConfigPanel remains intact and functional with the existing config and save workflow.
Este commit consolida uma série de melhorias no pipeline de geração de commits, abrangendo suporte multilíngua, persistência de commits, APIs expandidas, fluxo de PR automático, integração frontend/backend, e normalização de configuração. Abaixo os pontos-chave e o objetivo de cada mudança:
O que foi feito
- Backend e persistência de commits
- Criação do modelo CommitMessage no Back-End/Models/postgreSQL.py com campos como id, user_id, title, author, commit_hash, message, ai_generated_message, original_diff, total_tokens, status, pr_linked_id, created_at, processed_at, processed_by, error_message, além de relacionamento com PullRequest.
- PullRequest enriquecido para associar commits; novos campos de settings refletidos no modelo SystemSettings.
- Nova camada de persistência que suporta armazenar mensagens geradas por IA, diffs originais, tokens consumidos e estado de processamento, preparando o caminho para auditoria e reuso de mensagens.
- Motivo: habilitar rastreabilidade completa das mensagens de commit geradas por IA e facilitar fluxos de integração com PRs e histórico de alterações.
- API expandidas e configuração centralizada
- API expõe throttle_ms, lines_threshold, files_threshold, time_threshold, auto_push, auto_create_pr, commit_language, auto_process_prs. Endpoints GET /api/settings (get_settings) e PUT/PATCH para atualizar settings.
- Novos endpoints para commits: GET /api/commit-messages (lista paginada com busca) e GET /api/commit-messages/<commit_id> (detalhes do commit).
- Novo fluxo de diff_context/pré-processamento que utiliza diff_context via process_git_context_layer para gerar backlog de commits com base no idioma e hash do commit; payload inclui language e hash; remoção da dependência direta de GenerateCommitMessageAgent nesse caminho; suporte a criação automática de PR no fluxo de análise.
- Motivo: oferecer visibilidade, controle de configuração e integração entre componentes sem acoplamentos rígidos, além de suportar operações de IA de forma mais previsível.
- Fluxo de diff/contexto e PRs
- Diff context reestruturado para gerar backlog de commits com base no idioma (commit_language) e hash, com envio de payload adequado ao backend. Implementação orientada a configuráveis endpoints e fluxo de PR automático.
- Validação de merge expandida no backend para aceitar strings booleanas ou booleanos reais, assegurando comportamento consistente no fluxo de PR.
- Motivo: aumentar robustez e automação do ciclo de vida de PRs, reduzindo intervenções manuais.
- Frontend e Electron / fluxo de configuração
- Atualização massiva de front-end (Electron) para suportar fetch seguro, autenticação por token, carregamento dinâmico de configurações via /api/settings e propagação de commitLanguage/auto_create_pr no fluxo de diff_context.
- Implementação de createPullRequest no frontend com uso da API do GitHub (token), incluindo configuração de título/corpo do PR e detecção de upstream para push automático quando necessário.
- Monitoramento e estado de configuração: monitoramentoState.config passou a armazenar configurações normalizadas; startMonitoring persiste config normalizada; shouldTrigger/analyze/commit passam monitoringState.config para garantir consistência.
- Adição da página CommitMessages, rota /commits e navegação no sidebar;
integração com a nova API de commits e com o fluxo de PR automatizado.
- Novas interfaces de credenciais via Electron IPC: set-token, set-email, set-password no processo principal; preload expõe sendToken, sendEmail, sendPassword paraRenderer.
- Normalização de configuração com fallback backward-compatible: normalizeConfig centraliza lines_threshold, files_threshold, time_threshold, auto_push, auto_create_pr, api_endpoint, commitLanguage, GITHUB_TOKEN e força auto_commit para true.
- Motivo: assegurar uma experiência de usuário coesa, segura e estável, com configurações consistentes entre frontend e backend, além de permitir fluxos de PR automatizados com idioma específico.
- Geração de PDFs de faturas
- generate_invoice_pdf.py teve layout simplificado (removido cabeçalho informativo, itens e linha de total no topo), mantendo informações essenciais da fatura.
- Motivo: simplificar o layout de PDFs mantendo as informações críticas.
- Workflows, testes e build/dev
- Workflow de PR (pr.yml) atualizado para enviar merge: "False" ao gerar/relacionar PR via API, mantendo consistência com novos parâmetros de controle de mescla.
- TestDiscovery: URL de registro alterada para localhost (http://localhost:5910/api/register) para ambiente de desenvolvimento.
- Build/dev: build.py agora usa docker-compose up --build -d (sem especificar serviço), abrindo para todos os serviços.
- Docker Compose: volumes de bind mounts para Front-End/webpr:/app e Back-End:/app comentados/removidos; passagem para dependência de imagens, reduzindo dependência de código local durante build/deploy.
- Dependências, tipagem e UI
- Atualizações abrangentes de dependências no lockfile, incluindo React types (de 18.3.24 para 18.3.25), axios, Electron builder, e diversas bibliotecas de build/dev; suporte a novas ferramentas de packaging (Electron), fetch, and utilities de rede e validação.
- Front-end: adicionadas dependências node-fetch e simple-git nos pacotes Front-End/git-genius-commit; App.tsx reestruturado com AppRoutes e AuthProvider; ConfigPanel, StatusMonitor, useGitOperations, types/git.ts, index.css e outras partes do frontend atualizadas para refletir o novo fluxo de configuração e autenticação.
- Nova página CommitMessages no Front-End/webpr, com rota dedicada; atualização do sidebar para incluir Commits.
- Observabilidade UI: StatusMonitor agora exibe tokens de uso e consulta backend via /api/dashboard-data com X-API-TOKEN, atualizando a cada 30s.
- Observação sobre qualidade: melhorias de logs, renomeação de módulos (git_contex_layer_process) para refletir novos fluxos de processamento de contexto de git, e correção de possíveis falhas de runtime indicadas (ex.: typo na confi).
- Nota sobre migração e impacto
- Este é um refator grande que introduz modelos de dados, endpoints e fluxos de processamento novos. Pode exigir migrações de banco de dados para criar CommitMessage e updates em PullRequest. A normalização de configuração facilita futuras evoluções sem quebrar clientes mais antigos, com defaults conservadores.
- Caso haja ambiente com armazenamento de mensagens de IA, há agora suporte para auditoria, buscas e detalhamento de commits com linguagem específica e hash de commit.
Resumo de benefício
- Fluxo de commits mais inteligente e acessível: suporte multilíngua, geração de mensagens de commit coerentes, e integração automática com PRs.
- Maior visibilidade e controle via API de settings e commit messages, com histórico e detalhes dos commits gerados.
- Frontend/backend integrados com autenticação tokenizada, configuração centralizada, e uma experiência de usuário mais fluida (incluindo nova página de Commits).
- Melhoria de qualidade de código e infraestrutura com limpeza de fluxo, logs mais descritivos, e ajustes de build/deploy para ambientes modernos.
ualers2
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Descrição
Este PR introduz um fluxo completo que utiliza IA para gerar mensagens de commit a partir de diffs, com suporte a chunks para diffs grandes, multilíngua (pt/en) e saída estruturada, integrado a uma nova Camada de Contexto do Git (Git Context Layer) para análise, soma de chunks e persistência de resultados. Inclui autenticação via GitHub App e tokens de instalação para ações automáticas (PRs/commits) via API, exposição de dados de planos, verificação de signatures de webhooks, e melhorias no fluxo de PRs com suporte a merge via API. Também centraliza mudanças envolvendo IA, tokenização, planos e integrações com o GitHub, além de aprimorar a infraestrutura e as configurações do sistema. Observações sobre prompts de outros agentes são consideradas para evitar quebras no PR.
Este PR consolida mudanças significativas para IA em commits/PRs, integração com GitHub App e a introdução de uma camada local de contexto de Git para automação inteligente, com suporte a testes e configuração avançada.
Mudanças Principais
Notas: este conjunto de mudanças centraliza várias melhorias associadas ao GitContext Layer, IA, tokenização, planos e integrações com o GitHub, incluindo um front-end Electron para uso diário e automação de commits/PRs.
Por que esta mudança?
Impactos esperados:
Como testar
Observações Adicionais (Opcional)
Se houver necessidade de alinhamento com políticas internas, sugiro discutir ajustes na documentação de endpoints e nos contratos de payloads antes do merge.