Skip to content

Commit b488f1d

Browse files
committed
Document race condition fix policy
Add guidelines prohibiting race condition fixes via timeouts, delays, or reduced parallelism. Such approaches mask problems rather than fixing root causes.
1 parent aca0a95 commit b488f1d

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

.claude/CLAUDE.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -188,6 +188,15 @@ Client (ssh exercise@host -p 2222)
188188

189189
Pending tasks in the codebase are marked with `FIXME(claude)` and `TODO(claude)`. When the user requests to process todos or fixmes, search for these markers and address them.
190190

191+
## Fixing Race Conditions
192+
193+
**Never fix race conditions by:**
194+
- Adding timeouts or delays (e.g., `time.sleep()`)
195+
- Reducing the number of threads or parallel processes
196+
- Reducing test parallelism (e.g., changing `-n 10` to `-n 4`)
197+
198+
These approaches hide the underlying problem rather than fixing it. Race conditions must be fixed by addressing the root cause: proper synchronization, locking, atomic operations, or architectural changes.
199+
191200
## Commit Messages
192201

193202
- Do not include Claude as author or co-author in commit messages.

0 commit comments

Comments
 (0)