fix(qqbot): accept is_reconnect kwarg in connect() to match base adapter interface

[ulnit]

Problem

The QQ bot adapter's connect() method does not accept the is_reconnect keyword argument that gateway/run.py passes during reconnection:

QQAdapter.connect() got an unexpected keyword argument 'is_reconnect'

This causes an infinite crash-restart loop — the gateway catches the TypeError, waits 60s, retries, crashes again.

Root Cause

gateway/run.py line 3132 calls:

return await adapter.connect(is_reconnect=is_reconnect)

The base class (base.py) defines:

async def connect(self, *, is_reconnect: bool = False) -> bool:

All other platform adapters already implement this signature:

• signal.py ✅
• weixin.py ✅
• whatsapp_cloud.py ✅
• bluebubbles.py ✅
• yuanbao.py ✅
• api_server.py ✅
• qqbot/adapter.py ❌ — missing the parameter

Fix

One-line change to match the base class interface:

- async def connect(self) -> bool:
+ async def connect(self, *, is_reconnect: bool = False) -> bool:

Verification

• All 161 tests in tests/gateway/test_qqbot.py pass
• Change is backward compatible (keyword-only with default value)
• QQ bot successfully connects after applying the fix

[alt-glitch]

This was generated by AI during triage.

Duplicate of #52922 (earliest open canonical fix for #52914). Identical one-line connect(self, *, is_reconnect: bool = False) signature fix in gateway/platforms/qqbot/adapter.py. This is a saturated cluster — #52966 and #53540 are also tracking against #52922. Cross-linking so a maintainer can pick the canonical one.

[tonydwb]

Code Review Summary

Verdict: Approved

Adds is_reconnect keyword argument to QQAdapter.connect() to match the base adapter interface. Minimal one-line change.

Note

• Competing fix: PR #53540 by a different author implements the identical change. Either PR can be merged; the other should be closed.

---

Reviewed by Hermes Agent

[teknium1]

Thanks for the focused QQBot reconnect fix. This is an automated hermes-sweeper review; current main already provides the exact requested behavior.

• gateway/platforms/qqbot/adapter.py:281 already defines async def connect(self, *, is_reconnect: bool = False) -> bool.
• gateway/run.py:3463 and gateway/run.py:3466 forward is_reconnect to every adapter, matching the base contract at gateway/platforms/base.py:2887.
• Commit 276542c729c10ff9d093760897f4c2d1256a79ce implemented this QQBot fix and added coverage in tests/gateway/test_qqbot.py:193; current main also has a repository-wide adapter contract regression test.
• The duplicate discussion correctly identified this as part of the same saturated QQBot-fix cluster.