Cleaner fix for RPC issue

[tovrstra]

Improves fix from PR #13

Summary by Sourcery

This pull request addresses a bug in the RPC handling mechanism that caused lost connection errors in Python 3.11. It simplifies the connection handling by removing the need for a list of stop events and includes a workaround for a server closure issue fixed in Python 3.12.1.

• Bug Fixes:
  • Resolved an issue where the server context handler exits before all open requests are handled, preventing lost connection errors in Python 3.11.
• Enhancements:
  • Simplified the RPC handling by removing the need for a list of stop events, streamlining the connection handling process.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request refines the previous fix for the RPC issue by removing the stop_events list and simplifying the connection handling logic. Additionally, it introduces a workaround for a known issue with server.wait_closed() in Python versions below 3.12.1.

File-Level Changes

Files	Changes
stepup/core/rpc.py	Refactored RPC handling to remove unnecessary stop_events list and added a workaround for a known issue in Python versions below 3.12.1.

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[deepsource-io]

Here's the code health analysis summary for commits 2abd30a..618bb2a. View details on DeepSource ↗.

Analysis Summary

Analyzer	Status	Summary	Link
Shell	✅ Success		View Check ↗
Python	❌ Failure	❗ 2 occurences introduced	View Check ↗

---

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

[sourcery-ai]

Hey @tovrstra - I've reviewed your changes and they look great!

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟡 Testing: 1 issue found
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[tovrstra]

See #14 for more details.