Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: thomhurst/ModularPipelines
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v3.0.125
Choose a base ref
...
head repository: thomhurst/ModularPipelines
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v3.1.0
Choose a head ref
  • 11 commits
  • 6 files changed
  • 1 contributor

Commits on Jan 18, 2026

  1. feat(console): add deferred flush methods to IOutputCoordinator inter… 

    …face
    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    d889070 View commit details
    Browse the repository at this point in the history

  2. feat(console): implement deferred output storage and flush in OutputC… 

    …oordinator
    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    5ee2800 View commit details
    Browse the repository at this point in the history

  3. feat(logging): use OnModuleCompletedAsync instead of EnqueueAndFlushA… 

    …sync
    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    9af5e21 View commit details
    Browse the repository at this point in the history

  4. feat(console): wire up progress state to OutputCoordinator

    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    2e02066 View commit details
    Browse the repository at this point in the history

  5. feat(executors): call FlushDeferredAsync after progress ends

    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    d2ab0a7 View commit details
    Browse the repository at this point in the history

  6. feat(console): add IAsyncDisposable safety net to OutputCoordinator

    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    2c25be4 View commit details
    Browse the repository at this point in the history

  7. test(console): add unit tests for deferred output flush

    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    c523f62 View commit details
    Browse the repository at this point in the history

  8. fix(console): address PR review feedback for deferred output 

    - Fix race condition: move SetProgressActive(true) inside _phaseLock
  to prevent modules from completing before OutputCoordinator knows
  progress is active
- Add explicit SetProgressActive(false) when creating NoOpProgressSession
  for state consistency
- Restore "Some output may be lost" warning in timeout message
    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    2cff4d6 View commit details
    Browse the repository at this point in the history

  9. fix(console): address second round of PR review feedback 

    - Add separate _deferredLock for deferred operations to reduce lock
  contention with immediate flush operations
- Clear stale deferred outputs when SetProgressActive(true) is called
  to prevent memory leaks from previous crashed pipeline runs
- Improve DisposeAsync documentation to clarify it's a safety net and
  the primary flush happens in PipelineOutputScope.DisposeAsync
    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    4dddd71 View commit details
    Browse the repository at this point in the history

  10. fix(console): address third round of PR review feedback 

    - Remove IAsyncDisposable implementation (dead code - DI container
  doesn't automatically dispose async disposables for singletons)
- Clear deferred outputs when SetProgressActive(false) is called to
  handle cancellation scenarios where FlushDeferredAsync wasn't called
- This ensures memory cleanup happens whenever progress ends, regardless
  of whether it was cancelled or completed normally
    @thomhurst
    thomhurst committed Jan 18, 2026
    Configuration menu
    Copy the full SHA
    c9be15a View commit details
    Browse the repository at this point in the history

  11. +semver:minor - Merge pull request #2227 from thomhurst/feature/defer… 

    …red-output-flush

feat: defer module output until pipeline end when live progress enabled
    @thomhurst
    thomhurst authored Jan 18, 2026
    Configuration menu
    Copy the full SHA
    3d69c4c View commit details
    Browse the repository at this point in the history
Loading