feat: sprint 79 — UX: padronização de header em ClustersView e UserManagementView

Programado em par com: Claude IA

Author: joseafilho

…-registry-menu-and-repository-listing.md …-registry-menu-and-repository-listing.md.scratch/sprint-68/issues/01-registry-menu-and-repository-listing.md renamed to .scratch/archive/sprint-68/issues/01-registry-menu-and-repository-listing.md .scratch/sprint-68/issues/01-registry-menu-and-repository-listing.md renamed to .scratch/archive/sprint-68/issues/01-registry-menu-and-repository-listing.md

@@ -0,0 +1,15 @@
+---
+title: ClustersView — padronizar header com buildSectionHeader
+status: done
+sprint: 79
+---
+
+## Problem
+`ClustersView` usa um `buildToolbar()` próprio com `H2` + botão "Add Cluster" com `LUMO_PRIMARY` sem `LUMO_SMALL`. Não tem botão Refresh nem Help. As ações de Test Connection e Delete ficam como colunas inline no grid, fora do padrão de `SelectionAction` das demais views.
+
+## Expected
+- Substituir `buildToolbar()` por `UiConstants.buildSectionHeader(...)` com H3
+- "Add Cluster" como `extraLeadingButton` com `LUMO_PRIMARY + LUMO_SMALL`
+- Test Connection e Delete como `SelectionAction` na barra (habilitados por seleção)
+- Refresh (recarrega a grid) e Help adicionados automaticamente pelo header
+- Coluna de ações inline removida do grid

…int-68/issues/02-repository-tags-view.md …int-68/issues/02-repository-tags-view.md.scratch/sprint-68/issues/02-repository-tags-view.md renamed to .scratch/archive/sprint-68/issues/02-repository-tags-view.md .scratch/sprint-68/issues/02-repository-tags-view.md renamed to .scratch/archive/sprint-68/issues/02-repository-tags-view.md

@@ -0,0 +1,16 @@
+---
+title: UserManagementView — padronizar header com buildSectionHeader
+status: done
+sprint: 79
+---
+
+## Problem
+`UserManagementView` usa um `buildToolbar()` próprio com `H2` + botão "Add User" com `LUMO_PRIMARY` sem `LUMO_SMALL`. Não tem botão Refresh nem Help. As ações de Deactivate e Edit Permissions ficam como colunas inline no grid.
+
+## Expected
+- Substituir `buildToolbar()` por `UiConstants.buildSectionHeader(...)` com H3
+- "Add User" como `extraLeadingButton` com `LUMO_PRIMARY + LUMO_SMALL`
+- Deactivate como `SelectionAction` destrutivo na barra
+- Edit Permissions como `SelectionAction` na barra
+- Refresh e Help adicionados automaticamente pelo header
+- Coluna de ações inline removida do grid

.scratch/sprint-79/issues/01-clusters-view-header-padrao.md

@@ -639,3 +639,15 @@
 - `PodsView`: novo `Checkbox` "Hide completed Job pods" (marcado por padrão); novo predicado `isCompletedJobPod` (`jobName` não vazio + `phase == "Succeeded"`) combinado ao filtro existente do `ListDataProvider`, junto com Name/Status/Node e o filtro de Job — Pods `Failed` de Jobs permanecem sempre visíveis, independente do toggle
 - Ao abrir via `?job=<nome>` (botão "View Pods" de `JobsView`/`CronJobsView`), o checkbox inicia desmarcado — evita grid vazia ao ver os pods de um Job já `Complete`; volta a marcado ao limpar o filtro de Job pelo `jobFilterBanner`
 - Issue: `.scratch/sprint-67/issues/01-hide-completed-job-pods.md`
+
+### Sprint 68 ✅ — Container Registry: menu Global, listagem de Repositories e Tags
+
+- `CONTEXT.md`: novos termos `Registry` (capacidade derivada do `Cluster`, alcançada via port-forward da API do Kubernetes para o `Service` `registry` no Namespace `kube-system` — sem entidade persistida, sem credenciais novas), `Repository` (coleção nomeada de versões de imagem) e `Tag` (referência nomeada a uma versão específica, com digest/size/created); `docs/adr/0006-registry-via-port-forward.md` documenta a decisão de alcançar o Registry via port-forward em vez de uma entidade/configuração própria
+- `RepositoryInfo`/`TagInfo` (novos DTOs): `RepositoryInfo(name, tagCount)`, `TagInfo(name, digest, size, createdAt)`
+- `RegistryService` (novo, `io.greencap.k8s.kubernetes`): `listRepositories(Cluster)` — port-forward para o `Service` `registry`/`kube-system` (porta `5000`, porta do container — Fabric8 `ServiceResource#portForward` encaminha direto para a porta do Pod, não resolve `targetPort`), `GET /v2/_catalog` + `/v2/<repo>/tags/list` via `java.net.http.HttpClient`; `listTags(Cluster, repository)` — para cada tag, `GET /v2/<repo>/manifests/<tag>` (digest via header `Docker-Content-Digest`, size = `config.size` + soma de `layers[].size`) e `GET /v2/<repo>/blobs/<configDigest>` (campo `created`, formatado via `NamespaceService.age(...)`); qualquer exceção (Service ausente, port-forward falha, catálogo vazio) → `log.warn` + `List.of()`, sem `KubernetesOperationException` — ausência do Registry é estado esperado, não falha de cluster
+- `Permission.GLOBAL_REGISTRY_VIEW` (novo, grupo Global): incluído em `operatorPermissions()`/`viewerPermissions()`; `V22__add_registry_permission.sql` concede a todos os usuários com `GLOBAL_INFRASTRUCTURE_VIEW`
+- `RegistryView` (nova): rota `registry`, item "Container Registry" no drawer GLOBAL (`MainLayout.buildRegistryNavItem()`, ícone `VaadinIcon.ARCHIVE`); grid de Repositories (Repository/Tags, filtro por nome), ação "View Tags" navega para `registry/<repository>`; estado vazio único ("No repositories found. Make sure the Service \"registry\" in the \"kube-system\" namespace is available on this Cluster.") sem distinguir Service ausente/port-forward falho/catálogo vazio
+- `RegistryTagsView` (nova): rota `registry/:repository*` (wildcard para repositories com `/` no nome, ex. `greencap-demo/backend`); cabeçalho com nome do repository + botão Back para `RegistryView`; grid de Tags (Tag/Digest/Size/Created) — coluna Digest com `overflow:hidden`/`text-overflow:ellipsis`/`title` (tooltip) em vez de truncamento fixo
+- `samples/greencap-demo/cluster-provision.sh`: addon `registry` habilitado junto de `metrics-server`/`ingress`; `create-demo.sh` refatorado — addons (antes espalhados entre os dois scripts) agora centralizados em `cluster-provision.sh`, `create-demo.sh` passa a só aplicar os manifests do demo; `README.md` atualizado
+- Validado ponta a ponta no `greencap-demo`: addon `registry` habilitado, imagens de teste (`greencap-demo/hello` com tags `v1`/`v2`/`latest`, `greencap-demo/backend` com tag `v1`) buildadas e enviadas via port-forward + `docker push`; menu "Container Registry" lista os repositories com contagem de tags e "View Tags" exibe nome/digest/size/created corretamente
+- Issues: `.scratch/archive/sprint-68/issues/01-registry-menu-and-repository-listing.md`, `02-repository-tags-view.md`

.scratch/sprint-79/issues/02-user-management-view-header-padrao.md

@@ -8,7 +8,6 @@
 
 | Sprint | Tema | Status |
 |--------|------|--------|
-| 68 | Container Registry — menu Global, listagem de Repositories e Tags | ✅ Concluído |
 | 69 | Fix — Container Registry: item ausente na treeview de permissões + View Tags na grid | ✅ Concluído |
 | 70 | Platform Settings — auto-refresh: nova opção "3 seconds" e novo default | ✅ Concluído |
 | 71 | Infraestrutura de Demo — PVC para persistir o Container Registry interno | ✅ Concluído |
@@ -18,6 +17,7 @@
 | 76 | Namespaces View — Global: listagem com contagens de recursos, Create e Delete Namespace | ✅ Concluído |
 | 77 | Topologia: nó Ingress + botão "Go to resource" + pré-filtro ?name= nas views | ✅ Concluído |
 | 78 | Topologia: correções de layout (randomize), tap em group nodes e botão Reset Positions | ✅ Concluído |
+| 79 | UX — Padronização de header: ClustersView e UserManagementView com buildSectionHeader | ✅ Concluído |
 
 ---
 
@@ -83,6 +83,12 @@
 
 > Mostra apenas as últimas 10 sprints. Histórico completo em `docs/sprints-archive.md` (ver `docs/agents/sprint-archiving.md`).
 
+### Sprint 79 ✅ — UX: padronização de header em ClustersView e UserManagementView
+
+- `ClustersView`: `buildToolbar()` com `H2` removido; substituído por `UiConstants.buildSectionHeader` com H3 + botão "Add Cluster" (`LUMO_PRIMARY + LUMO_SMALL`) como extra leading button; ações "Test Connection" (`VaadinIcon.CONNECT`) e "Remove" (`VaadinIcon.TRASH`, destrutivo) movidas para `SelectionAction` no header (habilitadas pela seleção de linha); coluna de ações inline removida do grid; `GridSelectionMemory` injetado com `configureSingleSelection`; `refreshGrid()` retorna `boolean` para uso como `BooleanSupplier`; imports `H2` e `HorizontalLayout` removidos
+- `UserManagementView`: mesmo padrão — botão "Add User" (`LUMO_PRIMARY + LUMO_SMALL`); ações "Edit Permissions" (`VaadinIcon.EDIT`) e "Deactivate" (`VaadinIcon.BAN`, destrutivo) como `SelectionAction`; proteções antes feitas via botão desabilitado por linha passaram para early-exit com toast em `openEditPermissionsDialog` (admin padrão bloqueado) e `confirmDeactivate` (auto-desativação e usuário já inativo bloqueados); imports `H2` e `H4` removidos
+- Issues: `.scratch/sprint-79/issues/01-clusters-view-header-padrao.md`, `02-user-management-view-header-padrao.md`
+
 ### Sprint 78 ✅ — Topologia: correções de layout (randomize), tap em group nodes e botão Reset Positions
 
 - `topology-graph.ts`: `randomize` dinâmico — `true` quando posições salvas ausentes (fix para nós empilhados na primeira renderização), `false` quando presentes (mantém layout salvo); guard `if (node.data('isGroup')) return` no tap handler (fix para painel lateral não abrir ao clicar em group nodes)
@@ -176,18 +182,6 @@
 - `RegistryView`: ação "View Tags" sai da barra de título (selection action) e passa para uma coluna de ações na própria grid (`UiConstants.addActionsColumn`, botão por linha), mesmo padrão de `JobsView` ("View Pods")
 - Issue: `.scratch/sprint-69/issues/01-fix-registry-permission-treeview-view-tags.md`
 
-### Sprint 68 ✅ — Container Registry: menu Global, listagem de Repositories e Tags
-
-- `CONTEXT.md`: novos termos `Registry` (capacidade derivada do `Cluster`, alcançada via port-forward da API do Kubernetes para o `Service` `registry` no Namespace `kube-system` — sem entidade persistida, sem credenciais novas), `Repository` (coleção nomeada de versões de imagem) e `Tag` (referência nomeada a uma versão específica, com digest/size/created); `docs/adr/0006-registry-via-port-forward.md` documenta a decisão de alcançar o Registry via port-forward em vez de uma entidade/configuração própria
-- `RepositoryInfo`/`TagInfo` (novos DTOs): `RepositoryInfo(name, tagCount)`, `TagInfo(name, digest, size, createdAt)`
-- `RegistryService` (novo, `io.greencap.k8s.kubernetes`): `listRepositories(Cluster)` — port-forward para o `Service` `registry`/`kube-system` (porta `5000`, porta do container — Fabric8 `ServiceResource#portForward` encaminha direto para a porta do Pod, não resolve `targetPort`), `GET /v2/_catalog` + `/v2/<repo>/tags/list` via `java.net.http.HttpClient`; `listTags(Cluster, repository)` — para cada tag, `GET /v2/<repo>/manifests/<tag>` (digest via header `Docker-Content-Digest`, size = `config.size` + soma de `layers[].size`) e `GET /v2/<repo>/blobs/<configDigest>` (campo `created`, formatado via `NamespaceService.age(...)`); qualquer exceção (Service ausente, port-forward falha, catálogo vazio) → `log.warn` + `List.of()`, sem `KubernetesOperationException` — ausência do Registry é estado esperado, não falha de cluster
-- `Permission.GLOBAL_REGISTRY_VIEW` (novo, grupo Global): incluído em `operatorPermissions()`/`viewerPermissions()`; `V22__add_registry_permission.sql` concede a todos os usuários com `GLOBAL_INFRASTRUCTURE_VIEW`
-- `RegistryView` (nova): rota `registry`, item "Container Registry" no drawer GLOBAL (`MainLayout.buildRegistryNavItem()`, ícone `VaadinIcon.ARCHIVE`); grid de Repositories (Repository/Tags, filtro por nome), ação "View Tags" navega para `registry/<repository>`; estado vazio único ("No repositories found. Make sure the Service \"registry\" in the \"kube-system\" namespace is available on this Cluster.") sem distinguir Service ausente/port-forward falho/catálogo vazio
-- `RegistryTagsView` (nova): rota `registry/:repository*` (wildcard para repositories com `/` no nome, ex. `greencap-demo/backend`); cabeçalho com nome do repository + botão Back para `RegistryView`; grid de Tags (Tag/Digest/Size/Created) — coluna Digest com `overflow:hidden`/`text-overflow:ellipsis`/`title` (tooltip) em vez de truncamento fixo
-- `samples/greencap-demo/cluster-provision.sh`: addon `registry` habilitado junto de `metrics-server`/`ingress`; `create-demo.sh` refatorado — addons (antes espalhados entre os dois scripts) agora centralizados em `cluster-provision.sh`, `create-demo.sh` passa a só aplicar os manifests do demo; `README.md` atualizado
-- Validado ponta a ponta no `greencap-demo`: addon `registry` habilitado, imagens de teste (`greencap-demo/hello` com tags `v1`/`v2`/`latest`, `greencap-demo/backend` com tag `v1`) buildadas e enviadas via port-forward + `docker push`; menu "Container Registry" lista os repositories com contagem de tags e "View Tags" exibe nome/digest/size/created corretamente
-- Issues: `.scratch/sprint-68/issues/01-registry-menu-and-repository-listing.md`, `02-repository-tags-view.md`
-
 
 ---
 

docs/sprints-archive.md

@@ -6,12 +6,10 @@
 import com.vaadin.flow.component.formlayout.FormLayout;
 import com.vaadin.flow.component.grid.Grid;
 import com.vaadin.flow.component.html.Div;
-import com.vaadin.flow.component.html.H2;
 import com.vaadin.flow.component.html.Span;
 import com.vaadin.flow.component.icon.VaadinIcon;
 import com.vaadin.flow.component.notification.Notification;
 import com.vaadin.flow.component.notification.NotificationVariant;
-import com.vaadin.flow.component.orderedlayout.HorizontalLayout;
 import com.vaadin.flow.component.orderedlayout.VerticalLayout;
 import com.vaadin.flow.component.select.Select;
 import com.vaadin.flow.component.textfield.TextArea;
@@ -38,27 +36,58 @@
 
 import java.io.IOException;
 import java.nio.charset.StandardCharsets;
+import java.util.ArrayList;
 import java.util.List;
 
 @Route(value = "clusters", layout = MainLayout.class)
 @PageTitle("Clusters — GreenCap K8s")
 @PermitAll
 public class ClustersView extends VerticalLayout implements BeforeEnterObserver {
 
+    private static final String HELP_TITLE = "Clusters";
+    private static final String HELP_TEXT = "A Cluster is a registered Kubernetes environment (kubeconfig). " +
+            "Select the active cluster using the radio button — all workload views will reflect its resources. " +
+            "Select a row and use \"Test Connection\" to verify reachability, or \"Remove\" to delete the registration " +
+            "(the cluster itself is not affected).";
+
     private final ClusterService clusterService;
     private final KubeconfigValidator kubeconfigValidator;
     private final ClusterContext clusterContext;
     private final UserService userService;
     private final Grid<Cluster> grid = new Grid<>(Cluster.class, false);
 
     public ClustersView(ClusterService clusterService, KubeconfigValidator kubeconfigValidator,
-                        ClusterContext clusterContext, UserService userService) {
+                        ClusterContext clusterContext, UserService userService,
+                        GridSelectionMemory selectionMemory) {
         this.clusterService = clusterService;
         this.kubeconfigValidator = kubeconfigValidator;
         this.clusterContext = clusterContext;
         this.userService = userService;
+
         setSizeFull();
-        add(buildToolbar(), buildGrid());
+        setPadding(true);
+
+        buildGrid();
+        UiConstants.configureSingleSelection(grid, selectionMemory, getClass().getSimpleName(), Cluster::getName);
+
+        boolean canWrite = SecurityUtils.hasPermission(Permission.GLOBAL_CLUSTERS_WRITE);
+
+        List<UiConstants.SelectionAction<Cluster>> selectionActions = List.of(
+                UiConstants.SelectionAction.of(VaadinIcon.CONNECT, "Test Connection", this::testConnection),
+                UiConstants.SelectionAction.destructive(VaadinIcon.TRASH, "Remove", canWrite, this::confirmDelete)
+        );
+
+        List<Button> extraButtons = new ArrayList<>();
+        if (canWrite) {
+            Button addBtn = new Button("Add Cluster", VaadinIcon.PLUS.create());
+            addBtn.addThemeVariants(ButtonVariant.LUMO_PRIMARY, ButtonVariant.LUMO_SMALL);
+            addBtn.addClickListener(e -> openAddDialog());
+            extraButtons.add(addBtn);
+        }
+
+        add(UiConstants.buildSectionHeader("Clusters", this::refreshGrid, HELP_TITLE, HELP_TEXT,
+                        grid, selectionActions, extraButtons),
+                grid);
     }
 
     @Override
@@ -70,28 +99,13 @@ public void beforeEnter(BeforeEnterEvent event) {
         refreshGrid();
     }
 
-    private HorizontalLayout buildToolbar() {
-        Button addBtn = new Button("Add Cluster", VaadinIcon.PLUS.create(),
-                e -> openAddDialog());
-        addBtn.addThemeVariants(ButtonVariant.LUMO_PRIMARY);
-        addBtn.setEnabled(SecurityUtils.hasPermission(Permission.GLOBAL_CLUSTERS_WRITE));
-
-        HorizontalLayout toolbar = new HorizontalLayout(new H2("Clusters"), addBtn);
-        toolbar.setDefaultVerticalComponentAlignment(Alignment.CENTER);
-        toolbar.expand(new H2("Clusters"));
-        toolbar.setWidthFull();
-        return toolbar;
-    }
-
-    private Grid<Cluster> buildGrid() {
+    private void buildGrid() {
         grid.addComponentColumn(this::buildRadioCell).setHeader("Active").setWidth("90px").setFlexGrow(0).setResizable(true);
         grid.addColumn(Cluster::getName).setHeader("Name").setSortable(true).setFlexGrow(1).setResizable(true);
         grid.addColumn(c -> c.getProvider().name()).setHeader("Provider").setWidth("120px").setResizable(true);
         grid.addComponentColumn(c -> statusBadge(c.getConnectionStatus()))
                 .setHeader("Status").setWidth("140px").setResizable(true);
-        UiConstants.addActionsColumn(grid, 2, this::buildActions);
         grid.setSizeFull();
-        return grid;
     }
 
     private Div buildRadioCell(Cluster cluster) {
@@ -141,23 +155,6 @@ private Span statusBadge(ConnectionStatus status) {
         return badge;
     }
 
-    private List<Button> buildActions(Cluster cluster) {
-        var testIcon = VaadinIcon.CONNECT.create();
-        testIcon.setSize(UiConstants.ICON_SIZE);
-        Button testBtn = new Button(testIcon, e -> testConnection(cluster));
-        testBtn.addThemeVariants(ButtonVariant.LUMO_TERTIARY, ButtonVariant.LUMO_ICON);
-        testBtn.getElement().setAttribute("title", "Test connection");
-
-        var deleteIcon = VaadinIcon.TRASH.create();
-        deleteIcon.setSize(UiConstants.ICON_SIZE);
-        Button deleteBtn = new Button(deleteIcon, e -> confirmDelete(cluster));
-        deleteBtn.addThemeVariants(ButtonVariant.LUMO_TERTIARY, ButtonVariant.LUMO_ICON, ButtonVariant.LUMO_ERROR);
-        deleteBtn.getElement().setAttribute("title", "Remove cluster");
-        deleteBtn.setEnabled(SecurityUtils.hasPermission(Permission.GLOBAL_CLUSTERS_WRITE));
-
-        return List.of(testBtn, deleteBtn);
-    }
-
     private void confirmDelete(Cluster cluster) {
         Dialog dialog = new Dialog();
         dialog.setHeaderTitle("Remove cluster");
@@ -303,8 +300,9 @@ private java.util.Optional<MainLayout> getMainLayout() {
                 .findFirst());
     }
 
-    private void refreshGrid() {
+    private boolean refreshGrid() {
         grid.setItems(clusterService.findAll());
+        return true;
     }
 
     private void notify(String message, NotificationVariant variant) {