Skip to main content
← Back to list
01Issue
FeatureShippedSwamp CLI
Assigneesstack72

Consolidate MethodReportContext construction — manual and workflow report paths build contexts divergently

Opened by keeb · 4/11/2026· Shipped 4/14/2026

Sibling to #35 (which fixes MethodContext divergence). The same structural pattern exists for MethodReportContext: it is hand-built inline at two production sites — src/libswamp/models/run.ts:471 for method-scope reports (in the error-path failed-execution branch) and src/libswamp/workflows/run.ts:677 for workflow-scope reports. Future MethodReportContext fields added to one path will silently miss the other, same class of recurring regression as #35, different type.

Proposed fix: apply the same factory pattern #35 introduces for MethodContext. Extract buildMethodReportContext(common, invocation) in src/domain/reports/report_context.ts (or nearby), route both production sites through it, add an architecture guard test mirroring method_context_arch_test.ts.

Out of scope for #35 per keep-scope-small principle. Filing as its own issue so the fix has its own blast radius and its own PR.

02Bog Flow
OPENTRIAGEDIN PROGRESSSHIPPED+ 1 MOREASSIGNED+ 2 MOREREVIEW+ 3 MOREPR_MERGEDSHIPPED

Shipped

4/14/2026, 8:03:56 PM

Click a lifecycle step above to view its details.

03Sludge Pulse
stack72 assigned stack724/14/2026, 7:23:59 PM

Sign in to post a ripple.