Skip to content

Fix ACPClient weakref accumulation#152

Open
miky-rola wants to merge 2 commits intoaiming-lab:mainfrom
miky-rola:acp-weakref-accumulation
Open

Fix ACPClient weakref accumulation#152
miky-rola wants to merge 2 commits intoaiming-lab:mainfrom
miky-rola:acp-weakref-accumulation

Conversation

@miky-rola
Copy link
Contributor

Summary

  • Prune dead weakrefs from _live_instances on each new ACPClient init
  • Guard atexit.register() with a class flag so it only registers once

Test plan

  • test_dead_weakrefs_pruned — dead refs removed on new instance creation
  • test_atexit_registered_once — atexit registers exactly once
  • test_atexit_cleanup_closes_live_and_clears — cleanup calls close and clears list
  • Existing ACP tests pass (8/8)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant