Summary

• Remove dead locals (id, eventType, retry) in generated SSE client processEvent; fields are written directly on the event struct.
• Fix struct event decoding: decode directly into the struct pointer to avoid pointer-to-pointer issues; keep primitive parsing via partial_sse_parse.
• Leave retry header unassigned client-side pending typed conversion (server still emits retry as per SSE spec).

Repro

• In a downstream repo (e.g., aura), design defines SSE endpoints with SSEEventID/SSEEventType mapped to fields on the StreamingResult/Result.
• goa gen then go test ./gen/... failed with compiler errors in generated http/front/client/sse.go: unused id/eventType locals.

Root cause

• client_sse.go.tpl declared local variables for id/eventType/retry but set event fields directly; locals were unused.
• Additionally, when the event type is a struct and no DataField is specified, the template initialized event as a pointer but used a parse partial that decodes into &event, resulting in a pointer-to-pointer decode target.

Fix

• Remove unused local declarations.
• For struct events without DataField, decode JSON into the struct pointer (event) directly; primitives keep using the parse partial.

Validation

• goa unit tests pass locally.
• Regenerating the downstream repo with replace goa.design/goa/v3 => /Users/raphael/src/goa eliminates the compile errors in generated SSE HTTP client code.

Notes

• Retry parsing remains untyped on the client path; if you want typed retry propagation into user event structs we can add a field-type-aware parse in a follow-up. This change avoids the previous incorrect string assignment into numeric fields while keeping server-side retry output unchanged.