feat(nav): link naar de agent-messages queue in de navigatie #72

Merged

janpeter merged 1 commit from feat/queue-messages-nav into main

2026-07-25 23:53:41 +02:00

janpeter commented

2026-07-25 23:38:31 +02:00

Owner

Eén regel plus een test.

Het probleem

/queue/messages is een complete pagina — push, claim, reply, cancel, requeue, delete, clear, purge, live SSE — maar stond niet in NAV_LINKS. Hij was dus alleen bereikbaar door de URL met de hand te typen.

Dat viel niet op zolang het Ops-dashboard nog een Messages-scherm had. Sinds dat in PR #113 is verwijderd (de queue verhuisde naar deze app) is er helemaal geen ingang meer, terwijl dit nu het enige dashboard voor de agent-queue is.

De wijziging

   { href: '/worker-insights', label: 'Insights' },
+  { href: '/queue/messages', label: 'Messages' },
   { href: '/context', label: 'Context' },

Geplaatst na Insights: dat sluit het blok af waarmee je de vloot observeert, vóórdat de tools en instellingen beginnen. Label in het Engels, conform de omliggende items en de <h1> van de pagina zelf ("Agent messages").

De test

Pint zowel de href als het aria-current-gedrag vast. Bewezen dat hij iets vangt: met de nav-regel verwijderd valt precies die ene test om, de andere negen blijven groen.

zonder de regel : 1 failed | 9 passed
met de regel    : 10 passed

npm run typecheck schoon.

Context

De queue draait sinds vanavond op de scrum4me-database in plaats van ops_dashboard; de cutover is afgerond en op alle drie de hosts geverifieerd. Deze PR heeft daar verder niets mee te maken — de link ontbrak al langer.

🤖 Generated with Claude Code

Eén regel plus een test. ## Het probleem `/queue/messages` is een complete pagina — push, claim, reply, cancel, requeue, delete, clear, purge, live SSE — maar stond niet in `NAV_LINKS`. Hij was dus alleen bereikbaar door de URL met de hand te typen. Dat viel niet op zolang het Ops-dashboard nog een Messages-scherm had. Sinds dat in PR #113 is verwijderd (de queue verhuisde naar deze app) is er helemaal geen ingang meer, terwijl dit nu het enige dashboard voor de agent-queue is. ## De wijziging ```diff { href: '/worker-insights', label: 'Insights' }, + { href: '/queue/messages', label: 'Messages' }, { href: '/context', label: 'Context' }, ``` Geplaatst na Insights: dat sluit het blok af waarmee je de vloot observeert, vóórdat de tools en instellingen beginnen. Label in het Engels, conform de omliggende items en de `<h1>` van de pagina zelf ("Agent messages"). ## De test Pint zowel de `href` als het `aria-current`-gedrag vast. Bewezen dat hij iets vangt: met de nav-regel verwijderd valt precies die ene test om, de andere negen blijven groen. ``` zonder de regel : 1 failed | 9 passed met de regel : 10 passed ``` `npm run typecheck` schoon. ## Context De queue draait sinds vanavond op de `scrum4me`-database in plaats van `ops_dashboard`; de cutover is afgerond en op alle drie de hosts geverifieerd. Deze PR heeft daar verder niets mee te maken — de link ontbrak al langer. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit

2026-07-25 23:38:31 +02:00

feat(nav): link naar de agent-messages queue in de navigatie

CI / Verify (pull_request) Successful in 1m51s

Details

2aafacdece

De pagina /queue/messages is compleet (push, claim, reply, cancel, requeue,
delete, clear, purge, live SSE) maar stond niet in NAV_LINKS. Hij was dus
alleen bereikbaar door de URL met de hand te typen, en na het verwijderen van
de Messages-feature uit het Ops-dashboard was er helemaal geen ingang meer.

Geplaatst na Insights: dat sluit het blok af waarmee je de vloot observeert,
voordat de tools en instellingen beginnen.

De test pint de link en het aria-current-gedrag vast; zonder de nav-regel valt
hij om -- nagemeten met een mutatie.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

s4m-codex-reviewer approved these changes

2026-07-25 23:40:56 +02:00

s4m-codex-reviewer left a comment

APPROVED

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

Findings

Geen findings. De link naar components/shared/nav-bar.tsx:18 wijst naar de bestaande en gedocumenteerde admin-route /queue/messages, en de nieuwe test in __tests__/components/shared/nav-bar.test.tsx:56 borgt zowel de href als de actieve navigatiestatus.

# APPROVED geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen findings. De link naar `components/shared/nav-bar.tsx:18` wijst naar de bestaande en gedocumenteerde admin-route `/queue/messages`, en de nieuwe test in `__tests__/components/shared/nav-bar.test.tsx:56` borgt zowel de href als de actieve navigatiestatus.

janpeter merged commit 1baaf659cb into main

2026-07-25 23:53:41 +02:00

janpeter referenced this pull request from a commit

2026-07-25 23:53:42 +02:00

Merge pull request 'feat(nav): link naar de agent-messages queue in de navigatie' (#72) from feat/queue-messages-nav into main

janpeter referenced this pull request from a commit

2026-07-26 07:48:32 +02:00

docs(audit): NavBar-lijst compleet incl. Messages (PR #72)