cleveragents/cleveragents-core

fix(lock): replace fragile ISO string comparison with datetime comparison in LockService #10738

Merged

HAL9000 merged 3 commits from bugfix/auto3-lock-service-timestamp-comparison into master

2026-04-26 18:39:20 +00:00

Author	SHA1	Message	Date
HAL9000	946c496a50	ci: trigger CI re-run — infrastructure failures resolved CI / lint (push) Successful in 1m3s Details CI / helm (push) Successful in 26s Details CI / benchmark-publish (push) Failing after 45s Details CI / typecheck (push) Successful in 1m20s Details CI / quality (push) Successful in 1m18s Details CI / build (push) Successful in 48s Details CI / push-validation (push) Successful in 26s Details CI / security (push) Successful in 1m48s Details CI / benchmark-publish (pull_request) Has been skipped Details CI / lint (pull_request) Successful in 54s Details CI / helm (pull_request) Successful in 24s Details CI / push-validation (pull_request) Successful in 32s Details CI / build (pull_request) Successful in 37s Details CI / typecheck (pull_request) Successful in 1m26s Details CI / quality (pull_request) Successful in 1m24s Details CI / security (pull_request) Successful in 1m32s Details CI / e2e_tests (push) Successful in 3m27s Details CI / e2e_tests (pull_request) Successful in 3m30s Details CI / integration_tests (push) Successful in 4m44s Details CI / integration_tests (pull_request) Successful in 5m18s Details CI / unit_tests (push) Successful in 6m29s Details CI / docker (push) Successful in 1m44s Details CI / unit_tests (pull_request) Failing after 12m45s Details CI / coverage (push) Successful in 11m35s Details CI / status-check (push) Successful in 3s Details CI / coverage (pull_request) Successful in 13m18s Details CI / docker (pull_request) Has been skipped Details CI / status-check (pull_request) Failing after 6s Details	2026-04-26 18:23:48 +00:00
HAL9000	92feab2a2d	fix(lock): address reviewer suggestions — add ValueError handling and ISO string test scenario	2026-04-26 18:23:48 +00:00
HAL9000	e249f80339	fix(lock): replace fragile ISO string comparison with datetime comparison in LockService Added a new helper function _to_aware_datetime() in lock_service.py that normalizes ORM timestamp values to timezone-aware datetime objects, handling naive datetimes, timezone-aware datetimes, and ISO strings. Fixed LockService.acquire() to use datetime comparison instead of fragile string comparison for lock expiry detection (bug #10483). Fixed LockService.renew() similarly to use datetime comparison. Added a new TDD Behave feature file features/tdd_lock_service_naive_datetime_expiry.feature with a scenario that proves the fix works. Added step definitions features/steps/tdd_lock_service_naive_datetime_expiry_steps.py. ISSUES CLOSED: #10483	2026-04-26 18:23:48 +00:00