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

fix(repositories): replace timezone-naive datetime.now() with UTC-aware datetime.now(tz=UTC) #10960

Open
HAL9000 wants to merge 4 commits from feature/1915-timezone-aware-datetime into master
pull from: feature/1915-timezone-aware-datetime
merge into: cleveragents:master
Conversation 9 Commits 4 Files Changed 4
+565 -11

4 Commits

Author SHA1 Message Date
CleverAgents Bot c8d72aa029 ci: stop master workflow on PR updates
CI / lint (pull_request) Has been cancelled
Details
CI / typecheck (pull_request) Has been cancelled
Details
CI / security (pull_request) Has been cancelled
Details
CI / quality (pull_request) Has been cancelled
Details
CI / unit_tests (pull_request) Has been cancelled
Details
CI / integration_tests (pull_request) Has been cancelled
Details
CI / e2e_tests (pull_request) Has been cancelled
Details
CI / coverage (pull_request) Has been cancelled
Details
CI / build (pull_request) Has been cancelled
Details
CI / docker (pull_request) Has been cancelled
Details
CI / helm (pull_request) Has been cancelled
Details
CI / push-validation (pull_request) Has been cancelled
Details
CI / status-check (pull_request) Has been cancelled
Details 
Remove the stale pull_request trigger from master.yml so PR branch commits do not launch the master workflow.

Maintenance patch for PR #10960.
 2026-06-10 20:21:24 -04:00
HAL9000 60f28db3d5 fix(tests): fix duplicate @then step definitions causing AmbiguousStep in tdd_1915
CI / lint (pull_request) Successful in 1m4s
Details
CI / typecheck (pull_request) Successful in 1m25s
Details
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / quality (pull_request) Successful in 1m12s
Details
CI / security (pull_request) Successful in 1m56s
Details
CI / push-validation (pull_request) Successful in 31s
Details
CI / helm (pull_request) Successful in 37s
Details
CI / build (pull_request) Successful in 55s
Details
CI / benchmark-regression (pull_request) Failing after 58s
Details
CI / integration_tests (pull_request) Successful in 4m27s
Details
CI / e2e_tests (pull_request) Successful in 4m28s
Details
CI / unit_tests (pull_request) Failing after 6m7s
Details
CI / docker (pull_request) Has been skipped
Details
CI / coverage (pull_request) Has been skipped
Details
CI / status-check (pull_request) Failing after 4s
Details
2026-05-05 05:41:29 +00:00
HAL9000 b2142adf0e fix(repositories): remove @tdd_expected_fail tags and fix step definitions for bug #1915
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / helm (pull_request) Successful in 53s
Details
CI / push-validation (pull_request) Successful in 46s
Details
CI / build (pull_request) Successful in 1m31s
Details
CI / quality (pull_request) Successful in 1m43s
Details
CI / lint (pull_request) Failing after 1m49s
Details
CI / benchmark-regression (pull_request) Failing after 1m21s
Details
CI / typecheck (pull_request) Successful in 2m19s
Details
CI / security (pull_request) Successful in 2m29s
Details
CI / e2e_tests (pull_request) Failing after 4m57s
Details
CI / integration_tests (pull_request) Successful in 5m42s
Details
CI / unit_tests (pull_request) Failing after 5m45s
Details
CI / coverage (pull_request) Has been skipped
Details
CI / docker (pull_request) Has been skipped
Details
CI / status-check (pull_request) Failing after 4s
Details 
The bug is fixed (all datetime.now() replaced with datetime.now(tz=UTC)).
Remove @tdd_expected_fail tags from all 6 TDD scenarios so they are now
verified as passing. Rewrite step definitions to use correct domain models
and repository interfaces (ProjectRepository, PlanRepository, ActionRepository,
ActorRepository, SessionRepository, DebugAttemptRepository). Fix unused
import of datetime.datetime in step file.
 2026-05-04 21:01:59 +00:00
HAL9000 40f0e1ca6e fix(repositories): replace timezone-naive datetime.now() with UTC-aware datetime.now(tz=UTC)
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / lint (pull_request) Failing after 54s
Details
CI / build (pull_request) Successful in 1m2s
Details
CI / push-validation (pull_request) Successful in 31s
Details
CI / helm (pull_request) Successful in 36s
Details
CI / quality (pull_request) Successful in 1m24s
Details
CI / typecheck (pull_request) Successful in 1m26s
Details
CI / security (pull_request) Successful in 1m28s
Details
CI / unit_tests (pull_request) Failing after 4m5s
Details
CI / coverage (pull_request) Has been skipped
Details
CI / docker (pull_request) Has been skipped
Details
CI / integration_tests (pull_request) Successful in 4m44s
Details
CI / benchmark-regression (pull_request) Failing after 43s
Details
CI / e2e_tests (pull_request) Successful in 5m7s
Details
CI / status-check (pull_request) Failing after 3s
Details 
Multiple repository update() methods were using timezone-naive datetime.now() instead of UTC-aware datetime.now(tz=UTC), creating inconsistency with the UTC-aware domain model timestamps.

Affected repositories:
- NamespacedProjectRepository.update()
- LifecyclePlanRepository.update()
- DebugAttemptRepository.update()
- ActionRepository.update()
- ActorRepository.update()
- ToolRepository.update()
- SkillRepository.update()
- SessionRepository.update()

All 22 occurrences of timezone-naive datetime.now() in repositories.py have been replaced with UTC-aware datetime.now(tz=UTC).

Added TDD test scenarios to verify UTC-aware timestamps are set correctly.

ISSUES CLOSED: #1915
 2026-05-03 01:04:37 +00:00