Repository navigation
fix: repair release validation and desktop restoration - #694
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
⛔ Files ignored due to path filters (5)
📒 Files selected for processing (3)
Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 6 remain after this review. WalkthroughIntegration request helpers now cancel request contexts on errors and response-body closure. Node-lane cancellation returns session IDs from cancellation preparation and enqueues cleanup sessions. The desktop projection waits for window-manager configuration, and the related E2E and hook tests were updated. ChangesIntegration request cleanup
Node-lane cancellation
Electron DevTools E2E test
Desktop projection readiness
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The reviewed changes show no actionable issue that should prevent merging. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to The change repairs cleanup precedence while preserving transactional cancellation. No introduced security weakness was established, but incomplete verification of downstream stopping authority and older-session behavior prevents a minimal-risk assessment. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
Resilience and Maintainability Implications
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
React Doctor found no new issues. 🎉 Reviewed by React Doctor for commit |
|
What & why
Corrige as falhas de validação na main, na release #688 e neste PR após a atualização de dependências em #692:
Cancelar uma lane de
run-agentinseria uma limpeza genérica antes do encerramento terminal do vínculo. A inserção idempotente preservava a causa errada. A transação agora captura a sessão endereçada, encerra o vínculo e só então preenche as limpezas restantes, preservando sessão e época.Os leitores SSE dos testes HTTP/UDS podiam ficar presos no encerramento do corpo com o transporte do Go 1.27. Os helpers agora cancelam o contexto da requisição antes de fechar o corpo. A reprodução falhou 10/10 vezes antes e passou 10/10 depois, com prazos e assertivas originais.
O cenário de sino desconectado interrompia um endpoint antigo do catálogo, enquanto o sino consumia o ledger de notificações normalmente. O teste existente agora interrompe o endpoint realmente consumido e observa a falha antes de conferir o aviso.
A restauração de janelas montava um desktop vazio e seu composer quando o snapshot chegava antes da configuração do layout. A projeção agora espera essa configuração antes de expor os desktops; o limite de 500 ms e as assertivas de geometria permanecem iguais.
O teste de segurança do Electron enviava o segundo atalho sem recuperar o foco nativo após abrir DevTools em uma janela separada. O cenário agora observa o foco da janela do produto antes de injetar o atalho, como exige a API do Electron.
Implementação e verificação executadas com Codex.
How you verified it
61a44e350: 83 testes passaram (14,4 min). A restauração das 12 janelas levou 72,9 ms, mantendo o limite de 500 ms; nenhuma tarefa de arraste excedeu 50 ms.make gate: passou; lint Go sem problemas, testes API/store com race e lane Web (lint/typecheck/test/codegen) verdes.112135716565.61a44e350, os 30 testes desktop passaram, incluindo E2E-034 (12,9 s) e a estabilidade de uma hora. Job112135716585.Evidência e cenários atualizados em
docs/qa/reports/2026-10-05-dependency-upgrades.md,RT-web-attention-bell-jump.mdeLP-per-lane-node-control.md.CI final do head
61a44e35093589865c44dc20e9625bf751ff9bc2: 33 checks aprovados, 17 skips esperados, nenhuma falha ou pendência. CodeRabbit aprovou esse head e não há threads abertas.Impact
A correção interna afeta a limpeza de sessões ao cancelar uma lane pelo Web, CLI, HTTP/UDS ou
compozy__loop_node_cancel. Formatos públicos, isolamento por workspace, época do vínculo, hooks, configuração, esquema SQLite e skills permanecem compatíveis. A prontidão da projeção Web é interna e o ajuste de foco afeta somente a preparação do teste Electron. Auditoria registrada emdocs/_memory/change-impact.md; sem alteração necessária no site. Nota de release incluída.make gatepasses locally; this PR is delivered only after its required CI checks are green