Walkthrough

Updates MemoryTarget’s internal ThreadSafeList: modifies Add logic to purge excess items in a loop before appending, and restores a locking GetEnumerator that yields items safely during iteration.

Changes

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Caller
  participant ThreadSafeList as ThreadSafeList<T>
  participant InnerList as Inner List

  Caller->>ThreadSafeList: Add(item, maxListCount)
  activate ThreadSafeList
  ThreadSafeList->>ThreadSafeList: lock()
  loop while Count ≥ maxListCount
    ThreadSafeList->>InnerList: RemoveAt(0)
  end
  ThreadSafeList->>InnerList: Add(item)
  ThreadSafeList-->>ThreadSafeList: unlock()
  deactivate ThreadSafeList

  Caller->>ThreadSafeList: GetEnumerator()
  activate ThreadSafeList
  ThreadSafeList->>ThreadSafeList: lock()
  Note over ThreadSafeList: Yield items under lock
  ThreadSafeList-->>Caller: IEnumerator<T>
  ThreadSafeList-->>ThreadSafeList: unlock() on completion
  deactivate ThreadSafeList

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

I tidied lists with a rhythmic hop,
Old heads roll while new ones pop.
I guard the loop with gentle lock,
Each nibble counted, tick-tock, tick-tock.
Memory burrow neat and tight—
Add, enumerate, all done right.
Carrot commits approved tonight! 🥕🐇

Pre-merge checks and finishing touches

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud