feat(dispatch): eigen antwoordadressen opvragen (M41 G8d) #177

Merged

janpeter merged 1 commit from feat/m41-reply-addresses-list into main

2026-10-02 20:22:13 +02:00

janpeter commented

2026-10-02 20:01:47 +02:00

Owner

Waarom

M41 increment 4, gat G8d. De dialoog "Automatisch uitvoeren" in scrum4me-workers las queue_dispatch_reply_addresses rechtstreeks uit de database. De runtimerol van workers (scrum4me_web_runtime) heeft daar geen recht op, dus de dialoog kon nooit antwoordadressen tonen. De service moet ze daarom zelf kunnen leveren.

Wat

GET /dispatch/v1/reply-addresses → {"reply_addresses":[{"address":"mac:jp"}]}.
- Alleen de eigen ingeschakelde adressen van de geauthenticeerde actor, gesorteerd.
- Er is geen user-parameter; ?user_id= wordt genegeerd.
- Anoniem geeft 401 voordat de database wordt geraakt. Demogebruikers worden geweigerd door de refresh van de actor, en workers-aanroepers moeten admin zijn, zoals overal.
- Operatie list_reply_addresses.
Clientmethode listReplyAddresses().
De DB-rol scrum4me_dispatch had al SELECT op de tabel; er is geen schema- of policywijziging.

Bewijs

__tests__/dispatch/http.test.ts: eerst rood (2 failures, methode ontbrak), daarna 10/10.
Nieuwe integratiegevallen in end-to-end.integration.test.ts:
- twee ingeschakelde adressen en één uitgeschakeld geeft precies de twee, gesorteerd;
- de adressen van een andere gebruiker zijn nooit zichtbaar;
- ?user_id= verandert niets;
- de route staat in de 401-matrix.
Dispatch-integratiesuite: 280 groen. Unitsuite: 2021 groen. Typecheck schoon.
Kanttekening: de integratiegevallen zijn alleen tegen de afgemaakte code gedraaid. Rood eerst is alleen op unitniveau gezien.

Hoort bij scrum4me-workers feat/m41-g8-screen-links (G8d). Story ST-1617, taak T-1954.

🤖 Generated with Claude Code

## Waarom M41 increment 4, gat G8d. De dialoog "Automatisch uitvoeren" in scrum4me-workers las `queue_dispatch_reply_addresses` rechtstreeks uit de database. De runtimerol van workers (`scrum4me_web_runtime`) heeft daar geen recht op, dus de dialoog kon nooit antwoordadressen tonen. De service moet ze daarom zelf kunnen leveren. ## Wat - `GET /dispatch/v1/reply-addresses` → `{"reply_addresses":[{"address":"mac:jp"}]}`. - Alleen de eigen ingeschakelde adressen van de geauthenticeerde actor, gesorteerd. - Er is geen user-parameter; `?user_id=` wordt genegeerd. - Anoniem geeft 401 voordat de database wordt geraakt. Demogebruikers worden geweigerd door de refresh van de actor, en workers-aanroepers moeten admin zijn, zoals overal. - Operatie `list_reply_addresses`. - Clientmethode `listReplyAddresses()`. - De DB-rol `scrum4me_dispatch` had al SELECT op de tabel; er is geen schema- of policywijziging. ## Bewijs - `__tests__/dispatch/http.test.ts`: eerst rood (2 failures, methode ontbrak), daarna 10/10. - Nieuwe integratiegevallen in `end-to-end.integration.test.ts`: - twee ingeschakelde adressen en één uitgeschakeld geeft precies de twee, gesorteerd; - de adressen van een andere gebruiker zijn nooit zichtbaar; - `?user_id=` verandert niets; - de route staat in de 401-matrix. - Dispatch-integratiesuite: 280 groen. Unitsuite: 2021 groen. Typecheck schoon. - Kanttekening: de integratiegevallen zijn alleen tegen de afgemaakte code gedraaid. Rood eerst is alleen op unitniveau gezien. Hoort bij scrum4me-workers `feat/m41-g8-screen-links` (G8d). Story ST-1617, taak T-1954. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit

2026-10-02 20:01:47 +02:00

feat(dispatch): eigen antwoordadressen opvragen (M41 G8d)

CI / Final merge attestation and immutable publication (pull_request) Has been skipped

Details

CI / PR candidate (never published) (pull_request) Successful in 8m14s

Details

e91da75829

GET /dispatch/v1/reply-addresses geeft de ingeschakelde antwoordadressen van
de geauthenticeerde actor terug, gesorteerd op adres. De gebruiker is nooit
een parameter. Plus client.listReplyAddresses().

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

s4m-codex-reviewer approved these changes

2026-10-02 20:08:54 +02:00

s4m-codex-reviewer left a comment

APPROVED

Findings

Geen blokkerende bevindingen.

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

De GET-route leidt de gebruiker uitsluitend uit de geauthenticeerde actor af, filtert op ingeschakelde eigen adressen en sorteert deterministisch. Client-, route- en integratietests dekken het nieuwe contract, inclusief het negeren van een meegegeven user_id.

## APPROVED ### Findings - Geen blokkerende bevindingen. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. De GET-route leidt de gebruiker uitsluitend uit de geauthenticeerde actor af, filtert op ingeschakelde eigen adressen en sorteert deterministisch. Client-, route- en integratietests dekken het nieuwe contract, inclusief het negeren van een meegegeven `user_id`.

janpeter merged commit 4f0bdb1935 into main

2026-10-02 20:22:13 +02:00

janpeter referenced this pull request from a commit

2026-10-02 20:22:15 +02:00

Merge pull request 'feat(dispatch): eigen antwoordadressen opvragen (M41 G8d)' (#177)

janpeter referenced this pull request from a commit

2026-10-03 07:35:24 +02:00

docs(audit): document GET /reply-addresses (PR #177)