Files
graph_explorer/issues/0036b-kind-discriminator-and-group-loader.md

84 lines
2.5 KiB
Markdown

---
id: 0036b
title: NodeGroups window con kind (Table | Group) y loader para Groups
status: done
priority: high
created: 2026-05-04
parent: 0036
depends_on: [0036a]
---
## Objetivo
Que la NodeGroups window admita dos kinds: `Table` (DuckDB-backed,
comportamiento actual) y `Group` (entidades hijas con `group_id` set).
La window se elige por el `type_ref` del contenedor; el loader y las
columnas mostradas se ramifican por kind.
## Cambios
### `NodeGroupsWindowState` extendido
Anyadir campo:
```cpp
enum class NodeGroupsKind { Table, Group };
NodeGroupsKind kind = NodeGroupsKind::Table;
```
### Loaders por kind (en `node_groups.cpp`)
- **kind = Table**: comportamiento actual (`node_groups_load_metadata`
+ `node_groups_page_rows` sobre DuckDB).
- **kind = Group**: nuevo loader que hace
`SELECT id, name, type_ref, status, updated_at FROM entities
WHERE group_id = ? ORDER BY updated_at DESC LIMIT ? OFFSET ?`
+ `SELECT count(*)` para `total_rows`.
Las columnas que se renderizan son fijas:
`id, name, type_ref, status, updated_at`.
Convertir el dispatch en un metodo o switch dentro de
`node_groups_load_metadata` y `node_groups_page_rows` que mire `kind`.
### Columnas dinamicas en el render
Hoy `views.cpp` (en la pintura de la window) asume las columnas
DuckDB. Adaptar para que cuando `kind == Group` use las columnas
fijas listadas arriba.
### Apertura programatica
Para que 0036c pueda abrir una window de Group, exponer una API
limpia tipo:
```cpp
NodeGroupsWindowState* node_groups_open(AppState& app,
const std::string& container_id,
NodeGroupsKind kind);
```
Que crea la entrada en `app.node_groups_windows[container_id]` si no
existe, le pone el kind, y retorna puntero. El caller puede setear
`focus_request = true` antes de devolver el control al render.
## Acceptance criteria
- Tests pytest siguen verdes.
- Manual: insertar via SQL un nodo `Group` con 5 entidades hijas
(`group_id = <ese group>`), luego en la app llamar a
`node_groups_open(app, "<group_id>", Group)` (o disparar via test
unitario en C++ si se incluye), recargar render → la window
muestra las 5 entidades con columnas id/name/type_ref/status/
updated_at correctas.
- Apertura de un Table existente (kind=Table) sigue funcionando
identico (regresion).
## TBD
Branch `issue/0036b-kind-and-group-loader`, merge `--no-ff` a master.
## Out of scope
- Disparar drill-in desde doble click sobre Group (es 0036c).
- Promote / row click (0036d-e).