Repository navigation
fix(sidebar): Agentes voltam para dentro de cada PC - #20
Merged
Merged
Conversation
A v0.4.1 (upstream devswha#521) trocou a seção Agentes de cada PC por uma lista única no fim da barra lateral, com agentes de todos os PCs misturados. Agora cada PC desenha a própria lista, entre Projetos e Histórico, só com os agentes daquele PC e sem repetir o nome do PC na linha. - Recolher o PC recolhe os agentes dele; "Precisa de você" continua visível. - A dobra de Agentes é guardada por PC (herdr-web-ui:sidebar-section:<pc>:agents); no celular começa recolhida, com a contagem visível. - aria-label "Agentes em {PC}" para distinguir as seções no leitor de tela. - Sem gutter duplicado dentro do nav; design-system espelhado. - Testes: needs-input-regression cobre posição, dobra independente por PC e persistência após remontar (reprova com chave global); o trecho do ui-regression que exigia Agentes visíveis com o PC recolhido foi invertido. - CHANGELOG registra a divergência do upstream. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…teste no celular - Agents usa a mesma cabeça (.sidebar-section-header + FoldChevron): chevron sempre à vista e "Agents · n"; CSS próprio da cabeça removido (src e bundle.css) - DESIGN.md, RosterRow README/preview descrevem a lista por PC entre Projects e History - ui-regression: no celular Agents começa recolhida, mostra a contagem e abre com um toque - AgentSidebar recebe o PC (machine) em vez de [machine], para o useMemo não invalidar a cada render Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
O que muda
A v0.4.1 do upstream (devswha#521, trazida no #19) trocou a seção Agentes de cada PC por uma lista única no fim da barra lateral, com os agentes de todos os PCs misturados abaixo do Histórico. Este PR volta a seção para dentro de cada PC, como no mockup aprovado (
mockup-agentes-pc, opção A):herdr-web-ui:sidebar-section:<pc>:agents); no celular começa recolhida, com a contagem.Agentes · N), reaproveitando.sidebar-section-header/FoldChevron.aria-label"Agentes em {PC}" para distinguir as seções no leitor de tela (4 idiomas).DESIGN.md,src/AGENTS.mdedesign-system/(RosterRow,bundle.css) atualizados. CHANGELOG registra a divergência do upstream.Testes
bun run typecheck,bun run test:unit(2169 pass, 0 fail, incluii18n.test),bun run build.scripts/ui-regression.ts(incluineeds-input-regression) escripts/sidebar-activity-demo-regression.ts: passam.main,needs-input-regressionfalha (.agents-sidebarcom o PC recolhido:1 !== 0) e o trecho invertido doui-regressiontambém (PC folding folds its Agents with it);needs-input-regression.ts:146);ui-regression.ts:1560).ui-regressionque exigia Agentes visíveis com o PC recolhido foi invertido, porque verificava o comportamento que este PR muda.Revisão: code review e security review em agentes novos, sem contexto (sem achados de segurança; avisos tratados no segundo commit).
Fora do escopo
UI_EVIDENCE_DIRdefinido, oui-regressionfalha num clique depois do print do celular (o viewport não volta). Reproduz namain; vai num PR à parte.🤖 Generated with Claude Code