CI failure: filelock 3.21.0 deletes lock files on release, breaking test_auto_deps_lock.py

[Serhan-Asad]

Problem

CI is failing on every PR and push to main. The failing test is:

FAILED tests/test_auto_deps_lock.py::TestConcurrentAutoDeps::test_concurrent_calls_serialize_and_share_cache

tests/test_auto_deps_lock.py:128: in test_concurrent_calls_serialize_and_share_cache
    assert lock_file.exists(), "Lock file should be in tmp_path"
E   AssertionError: Lock file should be in tmp_path

The test passes locally but fails in CI (GitHub Actions, Ubuntu, Python 3.12.12). This started happening around Feb 12, 2026.

Impact

PRs are blocked from merging since CI is red.

[Serhan-Asad]

Step 1: Unit Test Execution (Cycle 1)

Status: All Tests Pass

Unit Tests Identified

• tests/test_auto_deps_lock.py::TestConcurrentAutoDeps::test_concurrent_calls_serialize_and_share_cache
• tests/test_auto_deps_lock.py::TestConcurrentAutoDeps::test_no_artifacts_in_working_directory
• tests/test_auto_deps_lock.py::TestCacheSharing::test_second_call_uses_cache_no_redundant_llm

Initial Test Results

• Passed: 3
• Failed: 0

pdd fix Runs

Dev Unit	Result
auto_deps_lock	No fix needed - tests already pass

The test file has already been updated to remove the assert lock_file.exists() assertion that was failing due to filelock 3.21.0 deleting lock files on release. The serialization behavior is now verified via execution_order assertions instead.

Final Test Results

• Passed: 3
• Failed: 0

---

Proceeding to Step 2: E2E Test Check

[Serhan-Asad]

Step 2: E2E Test Check (Cycle 1)

Status: ALL_TESTS_PASS

E2E Tests Run

• tests/test_auto_deps_lock.py::TestConcurrentAutoDeps::test_concurrent_calls_serialize_and_share_cache ✓
• tests/test_auto_deps_lock.py::TestConcurrentAutoDeps::test_no_artifacts_in_working_directory ✓
• tests/test_auto_deps_lock.py::TestCacheSharing::test_second_call_uses_cache_no_redundant_llm ✓

Summary

All 3 e2e tests pass. Bug fix complete!

---

Workflow complete - all tests passing

[Serhan-Asad]

Already fixed on main - the lock_file.exists() assertion was removed and replaced with a version-independent comment.