cleveragents/cleveragents-core
4
0
Fork 3
Code Issues 6.0k Pull Requests 115 Actions Packages Projects Releases Wiki Activity

fix(tui): fix thread-safety race in reference_parser catalog cache #8269

Open
HAL9000 wants to merge 9 commits from fix/concurrency-catalog-cache-lock-7590 into master
pull from: fix/concurrency-catalog-cache-lock-7590
merge into: cleveragents:master
Conversation 55 Commits 9 Files Changed 5
+325 -51

9 Commits

Author SHA1 Message Date
controller-ci-rerun 494da85edd chore: re-trigger CI [controller]
CI / load-versions (pull_request) Successful in 17s
Details
CI / push-validation (pull_request) Successful in 27s
Details
CI / lint (pull_request) Successful in 51s
Details
CI / quality (pull_request) Successful in 51s
Details
CI / build (pull_request) Successful in 53s
Details
CI / helm (pull_request) Successful in 54s
Details
CI / security (pull_request) Successful in 1m11s
Details
CI / unit_tests (pull_request) Successful in 5m23s
Details
CI / docker (pull_request) Successful in 2m6s
Details
CI / integration_tests (pull_request) Successful in 8m40s
Details
CI / typecheck (pull_request) Failing after 13m34s
Details
CI / coverage (pull_request) Has been cancelled
Details
CI / status-check (pull_request) Has been cancelled
Details
2026-06-18 10:19:40 -04:00
controller-ci-rerun 9c3fd674d0 chore: re-trigger CI [controller]
CI / load-versions (pull_request) Successful in 16s
Details
CI / push-validation (pull_request) Successful in 27s
Details
CI / lint (pull_request) Successful in 43s
Details
CI / build (pull_request) Successful in 41s
Details
CI / quality (pull_request) Successful in 54s
Details
CI / typecheck (pull_request) Successful in 1m10s
Details
CI / security (pull_request) Successful in 1m26s
Details
CI / helm (pull_request) Successful in 40s
Details
CI / unit_tests (pull_request) Successful in 5m14s
Details
CI / docker (pull_request) Successful in 2m41s
Details
CI / integration_tests (pull_request) Failing after 18m48s
Details
CI / coverage (pull_request) Failing after 22m23s
Details
CI / status-check (pull_request) Has been cancelled
Details
2026-06-18 09:43:10 -04:00
controller-ci-rerun 1737921628 chore: re-trigger CI [controller]
CI / load-versions (pull_request) Successful in 14s
Details
CI / push-validation (pull_request) Successful in 24s
Details
CI / lint (pull_request) Successful in 45s
Details
CI / build (pull_request) Successful in 42s
Details
CI / unit_tests (pull_request) Failing after 53s
Details
CI / quality (pull_request) Successful in 55s
Details
CI / docker (pull_request) Has been skipped
Details
CI / typecheck (pull_request) Successful in 1m17s
Details
CI / security (pull_request) Successful in 1m18s
Details
CI / coverage (pull_request) Has been skipped
Details
CI / helm (pull_request) Successful in 40s
Details
CI / integration_tests (pull_request) Successful in 8m45s
Details
CI / status-check (pull_request) Failing after 3s
Details
2026-06-18 09:32:52 -04:00
controller-ci-rerun 6d603ee33d chore: re-trigger CI [controller]
CI / load-versions (pull_request) Successful in 15s
Details
CI / push-validation (pull_request) Successful in 31s
Details
CI / lint (pull_request) Successful in 56s
Details
CI / typecheck (pull_request) Successful in 1m21s
Details
CI / build (pull_request) Successful in 49s
Details
CI / security (pull_request) Successful in 1m14s
Details
CI / helm (pull_request) Successful in 45s
Details
CI / unit_tests (pull_request) Failing after 1m16s
Details
CI / docker (pull_request) Has been skipped
Details
CI / coverage (pull_request) Has been skipped
Details
CI / quality (pull_request) Successful in 1m18s
Details
CI / integration_tests (pull_request) Successful in 8m56s
Details
CI / status-check (pull_request) Failing after 12s
Details
2026-06-18 09:19:03 -04:00
controller-ci-rerun 703a5aa725 chore: re-trigger CI [controller] 2026-06-18 09:19:03 -04:00
controller-ci-rerun fe52983aef chore: re-trigger CI [controller] 2026-06-18 09:19:03 -04:00
HAL9000 c7c7817fcd fix(tui): apply ruff format to tdd_reference_parser_catalog_lock_steps.py 2026-06-18 09:19:03 -04:00
controller-ci-rerun d02aa69924 chore: re-trigger CI [controller] 2026-06-18 09:19:03 -04:00
HAL9000 e54f28d0a1 fix(tui): fix thread-safety race in reference_parser catalog cache 
Added threading.Lock to prevent race conditions in _catalog_cache dict.
Two concurrent threads calling _catalog() while cache was stale could
both walk the filesystem and write results simultaneously, causing partial
dict updates or RuntimeError on concurrent iteration.

Changes:
- Added import threading and from threading import Lock
- Added _catalog_lock: Lock = threading.Lock() at module level
- Wrapped entire _catalog() body in with _catalog_lock: for atomic access
- Added 4 BDD scenarios in tdd_reference_parser_catalog_lock.feature
- Updated CHANGELOG.md with fix entry under [Unreleased] > Fixed
- Updated CONTRIBUTORS.md with contribution entry

ISSUES CLOSED: #7590
 2026-06-18 09:19:03 -04:00