Fix LocalFileDestination timestamp formatting concurrency #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

henribredt wants to merge 1 commit into main from codex/review-swift-concurrency-implementation

Owner

henribredt commented Oct 29, 2025

Summary

move timestamp formatting for file logs into the FileOperationActor to ensure serialization
build log lines inside the actor to avoid sharing the non-sendable DateFormatter across tasks

Testing

swift test (fails: package requires Swift tools 6.2.0 but environment has 6.1.0)

https://chatgpt.com/codex/tasks/task_e_6902909cdfac8323bc12ee9856726a8c


          Move file timestamp formatting into actor

4124cb6

henribredt added the codex label

— with

ChatGPT Codex Connector

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels