feat: v9-tabellen en 80%-desktopbreedte Workers (S-2026-10-02-v9-desktop) #117

Merged
janpeter merged 14 commits from codex/v9-desktop-rest-workers into main 2026-10-02 17:29:25 +02:00
Owner

Sprint

S-2026-10-02-v9-desktop: resterende desktoproutes en recordtabellen in scrum4me-workers. Bijbehorende Scrum4Me-PR: #283.

Wijzigingen

  • Models en Job-kinds zijn 80% breed op desktop. Models heeft vijf v9-sortkolommen met bewaarde lokale voorkeur en stabiele model-ID voor de Actief-actie; Job-kinds houdt zijn vaste volgorde zonder sortering.
  • Worker Logs is 80% breed en sorteert negen kolommen op de geladen 10/25/50/100 runs plus eventuele pin. De geselecteerde detailrij blijft gekoppeld aan de run-ID; ontbrekende waarden staan in beide richtingen achteraan.
  • Worker Insights is 80% breed en behoudt zijn zes diagnostische bronranglijsten zonder sorteerknoppen.
  • API Tokens gebruikt de v9-renderer met dezelfde localStorage-sleutel, sorteerregels en tokenacties.

Verificatie

  • npm run verify -- -- --maxWorkers=2 --minWorkers=1: 164 testbestanden, 1356 tests geslaagd, 18 overgeslagen; lint 0 fouten met 1 bestaande waarschuwing; typecheck geslaagd.
  • Docker builder geslaagd; manifest sha256:166f18e3f86e2921c7ab04aacfa78858a3cc82f2bb6f2dd49371216d9c845d91.
  • Lokale browserproeven op 1728×906, 1280×800, 1024×768 en 768×800: A, B inclusief gevulde Worker Insights.
  • Onafhankelijke lokale code-review: geen open bevindingen. Eindrapport.

Acceptatiegrens

Alle browserproeven en sorteervoorkeurwrites gebruikten synthetische gegevens in een wegwerpomgeving. Geen productie-modeltoggle, productie-tokenactie, merge of deployment in deze opdracht. T-1921 blijft open tot gedeployd runtimegedrag en JP's praktijkcontrole zijn bevestigd.

## Sprint S-2026-10-02-v9-desktop: resterende desktoproutes en recordtabellen in scrum4me-workers. Bijbehorende Scrum4Me-PR: [#283](https://git.jp-visser.nl/janpeter/Scrum4Me/pulls/283). ## Wijzigingen - Models en Job-kinds zijn 80% breed op desktop. Models heeft vijf v9-sortkolommen met bewaarde lokale voorkeur en stabiele model-ID voor de Actief-actie; Job-kinds houdt zijn vaste volgorde zonder sortering. - Worker Logs is 80% breed en sorteert negen kolommen op de geladen 10/25/50/100 runs plus eventuele pin. De geselecteerde detailrij blijft gekoppeld aan de run-ID; ontbrekende waarden staan in beide richtingen achteraan. - Worker Insights is 80% breed en behoudt zijn zes diagnostische bronranglijsten zonder sorteerknoppen. - API Tokens gebruikt de v9-renderer met dezelfde localStorage-sleutel, sorteerregels en tokenacties. ## Verificatie - `npm run verify -- -- --maxWorkers=2 --minWorkers=1`: 164 testbestanden, 1356 tests geslaagd, 18 overgeslagen; lint 0 fouten met 1 bestaande waarschuwing; typecheck geslaagd. - Docker builder geslaagd; manifest `sha256:166f18e3f86e2921c7ab04aacfa78858a3cc82f2bb6f2dd49371216d9c845d91`. - Lokale browserproeven op 1728×906, 1280×800, 1024×768 en 768×800: [A](https://git.jp-visser.nl/janpeter/scrum4me-workers/src/branch/codex/v9-desktop-rest-workers/docs/superpowers/reports/2026-10-02-v9-a-workers-proof.md), [B inclusief gevulde Worker Insights](https://git.jp-visser.nl/janpeter/scrum4me-workers/src/branch/codex/v9-desktop-rest-workers/docs/superpowers/reports/2026-10-02-v9-b-workers-proof.md). - Onafhankelijke lokale code-review: geen open bevindingen. [Eindrapport](https://git.jp-visser.nl/janpeter/scrum4me-workers/src/branch/codex/v9-desktop-rest-workers/docs/superpowers/reports/2026-10-02-v9-final-workers-proof.md). ## Acceptatiegrens Alle browserproeven en sorteervoorkeurwrites gebruikten synthetische gegevens in een wegwerpomgeving. Geen productie-modeltoggle, productie-tokenactie, merge of deployment in deze opdracht. T-1921 blijft open tot gedeployd runtimegedrag en JP's praktijkcontrole zijn bevestigd.
s4m-codex-reviewer approved these changes 2026-10-02 17:12:10 +02:00
Dismissed
s4m-codex-reviewer left a comment

Verdict: APPROVED

Findings

  • Geen blocking/error-severity findings.

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

De PR volgt de relevante Workers-v9-standaard: 80%-containers zonder globale shellwijziging, lokale versieerde sorteerkeuzes, stabiele rij-id's, niet-sorteerbare acties en behoud van de betekenisvolle Insights/Job-kinds-volgorde. De adapteruitbreiding houdt een geselecteerde logdetailrij gekoppeld aan dezelfde record-id; de regressietests dekken sortering, null/0-waarden, persistence en selectie over refresh.

## Verdict: APPROVED ### Findings - Geen blocking/error-severity findings. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. De PR volgt de relevante Workers-v9-standaard: 80%-containers zonder globale shellwijziging, lokale versieerde sorteerkeuzes, stabiele rij-id's, niet-sorteerbare acties en behoud van de betekenisvolle Insights/Job-kinds-volgorde. De adapteruitbreiding houdt een geselecteerde logdetailrij gekoppeld aan dezelfde record-id; de regressietests dekken sortering, null/0-waarden, persistence en selectie over refresh.
ci(ST-1613): begrens Vitest-concurrency op gedeelde runner
All checks were successful
CI / Verify (pull_request) Successful in 3m35s
b922dd3ee9
s4m-codex-reviewer left a comment

Verdict: REQUEST_CHANGES

  • ERROR — .forgejo/workflows/ci.yml:32: de extra -- in npm run verify -- -- --maxWorkers=2 --minWorkers=1 wordt doorgegeven aan de scriptketen. Daardoor start de laatste stap als npm test -- -- --maxWorkers=2 --minWorkers=1, dus Vitest ontvangt -- vóór de workeropties en behandelt die niet als CLI-opties (of als testfilters). De CI kan hierdoor falen of draait niet met de bedoelde workerlimiet. Gebruik één separator: npm run verify -- --maxWorkers=2 --minWorkers=1.

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

## Verdict: REQUEST_CHANGES - **ERROR** — `.forgejo/workflows/ci.yml:32`: de extra `--` in `npm run verify -- -- --maxWorkers=2 --minWorkers=1` wordt doorgegeven aan de scriptketen. Daardoor start de laatste stap als `npm test -- -- --maxWorkers=2 --minWorkers=1`, dus Vitest ontvangt `--` vóór de workeropties en behandelt die niet als CLI-opties (of als testfilters). De CI kan hierdoor falen of draait niet met de bedoelde workerlimiet. Gebruik één separator: `npm run verify -- --maxWorkers=2 --minWorkers=1`. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
Sign in to join this conversation.
No reviewers
No labels
severity/s4
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/scrum4me-workers!117
No description provided.