4 Commits

Author SHA1 Message Date
HAL9000 29a3e70c36 fix(concurrency): add @tdd_issue tag and fix ruff formatting
CI / lint (pull_request) Successful in 38s
CI / helm (pull_request) Successful in 27s
CI / build (pull_request) Successful in 29s
CI / quality (pull_request) Successful in 53s
CI / typecheck (pull_request) Successful in 1m0s
CI / push-validation (pull_request) Successful in 33s
CI / security (pull_request) Successful in 1m5s
CI / integration_tests (pull_request) Successful in 8m13s
CI / unit_tests (pull_request) Successful in 8m16s
CI / docker (pull_request) Successful in 1m31s
CI / coverage (pull_request) Successful in 7m44s
CI / status-check (pull_request) Successful in 4s
- Add missing @tdd_issue companion tag to @tdd_issue_7623 scenario;
  before_scenario hook raised ValueError causing hook_error on every run
- Apply ruff format to validation_pipeline.py and
  validation_pipeline_concurrency_steps.py (2 files reformatted)
- Add use_step_matcher("parse") reset at end of concurrency steps
  to match convention in validation_pipeline_steps.py

ISSUES CLOSED: #7623
2026-06-02 04:29:20 -04:00
HAL9000 bd955696f9 fix(pr-7811 review): resolve remaining CI and code style blockers
Address all 4 remaining blocking issues from PR #7811 review:

- Delete duplicate _validation_pipeline_mock.py in features/steps/
  (the inline MockValidationExecutor is kept where it belongs)
- Replace # type: ignore[return-value] with cast(dict[str, Any], ...)
  to satisfy the zero-type-suppressions policy for test additions
- Add @tdd_issue_7623 regression tag on concurrency scenario
- Remove unreachable dead-code RuntimeError guard in
  _install_thread_local_streams() (constructors never return None)

All CI lint checks now pass. Source files remain within line limits.
2026-06-02 04:29:20 -04:00
HAL9000 2609787d35 fix(concurrency): fix ValidationPipeline.run() sys.stdout replacement #7623
Introduce a reference-counted shared stream wrapper manager so concurrent
ValidationPipeline.run() calls correctly restore the true sys.stdout/stderr
after all pipelines finish.

- Add _install_thread_local_streams() / _release_thread_local_streams()
  protected by threading.Lock; first caller saves originals and installs
  wrappers, last caller restores them
- Remove io.TextIOBase inheritance from _ThreadLocalStream to avoid Python
  3.13 read-only slot conflict; use cast(TextIO, ...) at assignment sites
- encoding property returns str | None matching io.TextIOBase signature
- Replace assert guards with explicit RuntimeError fail-fast checks
- Add Behave scenario: Concurrent pipelines restore global streams after
  execution
- Split validation_pipeline_steps.py (547→428 lines) by extracting
  MockValidationExecutor to _validation_pipeline_mock.py and edge-case
  steps to validation_pipeline_edge_steps.py
- Update CHANGELOG.md and CONTRIBUTORS.md

ISSUES CLOSED: #7623
2026-06-02 04:29:20 -04:00
CoreRasurae 4874f2ad6f feat(validation): add validation pipeline and results model
ISSUES CLOSED: #175
2026-02-25 10:03:57 +00:00