feat: Implement discriminated union pattern for ModuleResult<T>#1895

Merged

thomhurst merged 17 commits intomainfrom

feature/1869-module-result-discriminated-union

Jan 8, 2026

Commits on Jan 7, 2026

docs: Add design for ModuleResult discriminated union pattern

thomhurst
and
claude
committed
docs: add implementation plan for ModuleResult discriminated union
thomhurst
committed
refactor(IModuleResult): add safe accessor properties for discriminated union pattern
thomhurst
committed
refactor(ModuleResult): implement discriminated union with Success/Failure/Skipped variants

thomhurst
and
claude
committed
refactor: remove SkippedModuleResult and TimedOutModuleResult (replaced by variants)
thomhurst
committed
refactor(ModuleResultFactory): simplify to use ModuleResult static factory methods
thomhurst
committed
refactor(ModuleExecutionPipeline): use discriminated union factory methods

thomhurst
and
claude
committed
deprecate: mark ModuleFailedException and ModuleSkippedException as obsolete
thomhurst
committed
test: update test helpers for discriminated union pattern

thomhurst
and
claude
committed
test: update unit tests for discriminated union pattern

thomhurst
and
claude
committed
fix: update Build project modules for discriminated union pattern

thomhurst
and
claude
committed
fix: add JSON serialization support and fix history repository handling

thomhurst
and
claude
committed
fix: address code review issues for discriminated union PR

thomhurst
and
claude
committed
fix: address PR #1895 review comments

thomhurst
and
claude
committed
Merge main into feature/1869-module-result-discriminated-union

thomhurst
and
claude
committed
fix: update DirectModuleHooksTests to use new discriminated union API

thomhurst
and
claude
committed
fix: update Azure example modules to use ValueOrDefault

thomhurst
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: Implement discriminated union pattern for ModuleResult<T>#1895

feat: Implement discriminated union pattern for ModuleResult<T>#1895
thomhurst merged 17 commits intomainfrom
feature/1869-module-result-discriminated-union

Commits on Jan 7, 2026

docs: Add design for ModuleResult discriminated union pattern

docs: add implementation plan for ModuleResult discriminated union

refactor(IModuleResult): add safe accessor properties for discriminated union pattern

refactor(ModuleResult): implement discriminated union with Success/Failure/Skipped variants

refactor: remove SkippedModuleResult and TimedOutModuleResult (replaced by variants)

refactor(ModuleResultFactory): simplify to use ModuleResult static factory methods

refactor(ModuleExecutionPipeline): use discriminated union factory methods

deprecate: mark ModuleFailedException and ModuleSkippedException as obsolete

test: update test helpers for discriminated union pattern

test: update unit tests for discriminated union pattern

fix: update Build project modules for discriminated union pattern

fix: add JSON serialization support and fix history repository handling

fix: address code review issues for discriminated union PR

fix: address PR #1895 review comments

Merge main into feature/1869-module-result-discriminated-union

fix: update DirectModuleHooksTests to use new discriminated union API

fix: update Azure example modules to use ValueOrDefault