fix: keep weixin_oc polling after inbound timeouts

[1zzxy1]

Summary

• treat weixin_oc inbound long-poll timeouts as recoverable instead of exiting the adapter loop
• keep the adapter polling after TimeoutError from getupdates
• add a regression test covering the timeout recovery path

Closes #6901

Testing

• uv run --group dev pytest tests/test_weixin_oc_adapter.py -q

Summary by Sourcery

Handle Weixin OC inbound long-poll timeouts as recoverable errors so the adapter continues polling instead of exiting.

Bug Fixes:

• Prevent the Weixin OC adapter run loop from terminating when inbound long-polling getupdates calls raise asyncio.TimeoutError.

Tests:

• Add a regression test verifying the Weixin OC adapter continues polling, logs the timeout, and shuts down cleanly after an inbound long-poll timeout.

[gemini-code-assist]

Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request addresses an issue where the weixin_oc adapter would cease operation if an inbound long-poll request timed out. The changes introduce robust error handling for asyncio.TimeoutError within the polling mechanism, allowing the adapter to log the timeout and continue its polling cycle without interruption. This ensures the weixin_oc integration remains resilient and continuously processes updates even under transient network conditions or slow responses.

Highlights

• Timeout Handling: The weixin_oc adapter now gracefully handles asyncio.TimeoutError during inbound long-polling, treating these timeouts as recoverable events rather than fatal errors that stop the adapter.
• Continuous Polling: The adapter's main loop will continue to poll for updates even after encountering an asyncio.TimeoutError from the _poll_inbound_updates method, ensuring service continuity.
• Regression Test: A new regression test has been added to specifically cover the scenario where an inbound long-poll timeout occurs, verifying that the adapter recovers and continues its operation as expected.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[sourcery-ai]

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

---

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 and I'll use the feedback to improve your reviews.}

[gemini-code-assist]

Code Review

This pull request modifies the weixin_oc_adapter to gracefully handle asyncio.TimeoutError during inbound long-polling, logging the timeout and ensuring the polling loop continues. A new test case has been added to verify this behavior. The review suggests an improvement in the new test to use patch.object for mocking, which would enhance robustness and clarity.

[gemini-code-assist]

To improve test robustness and clarity, consider using patch.object as a context manager for monkey-patching. This practice ensures that the original method is restored automatically after the test, even in case of errors, and it eliminates the need for a type: ignore comment.

    with patch.object(adapter, '_poll_inbound_updates', new=fake_poll_inbound_updates):
        with patch('astrbot.core.platform.sources.weixin_oc.weixin_oc_adapter.logger') as mock_logger:
            await adapter.run()