Skip to content

修正了QQ websoket连接的文档错误#7279

Merged
RC-CHN merged 1 commit intoAstrBotDevs:masterfrom
Thelia-Lzr:patch-1
Apr 2, 2026
Merged

修正了QQ websoket连接的文档错误#7279
RC-CHN merged 1 commit intoAstrBotDevs:masterfrom
Thelia-Lzr:patch-1

Conversation

@Thelia-Lzr
Copy link
Copy Markdown
Contributor

@Thelia-Lzr Thelia-Lzr commented Apr 1, 2026

Modifications / 改动点

  • 修正了QQ websoket连接的文档错误(为什么这里会突然出现NapCatQQ)
  • This is NOT a breaking change. / 这不是一个破坏性变更。

Checklist / 检查清单

  • 😊 If there are new features added in the PR, I have discussed it with the authors through issues/emails, etc.
    / 如果 PR 中有新加入的功能,已经通过 Issue / 邮件等方式和作者讨论过。

  • 👀 My changes have been well-tested, and "Verification Steps" and "Screenshots" have been provided above.
    / 我的更改经过了良好的测试,并已在上方提供了“验证步骤”和“运行截图”

  • 🤓 I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations in requirements.txt and pyproject.toml.
    / 我确保没有引入新依赖库,或者引入了新依赖库的同时将其添加到 requirements.txtpyproject.toml 文件相应位置。

  • 😮 My changes do not introduce malicious code.
    / 我的更改没有引入恶意代码。

Summary by Sourcery

Documentation:

  • Update the QQ official WebSocket integration guide to describe a successful AstrBot connection to the QQ official interface without referencing NapCatQQ.

Copilot AI review requested due to automatic review settings April 1, 2026 14:08
@auto-assign auto-assign bot requested review from LIghtJUNction and Soulter April 1, 2026 14:08
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 1, 2026
@Thelia-Lzr Thelia-Lzr changed the title Update connection success message for AstrBot 修正了QQ websoket连接的文档错误 Apr 1, 2026
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've found 1 issue

Prompt for AI Agents
Please address the comments from this code review:

## Individual Comments

### Comment 1
<location path="docs/zh/platform/qqofficial/websockets.md" line_range="90" />
<code_context>
 ## 🎉 大功告成

-此时,你的 AstrBot 和 NapCatQQ 应该已经连接成功。使用 `私聊` 的方式在 QQ 对机器人发送 `/help` 以检查是否连接成功。
+此时,你的 AstrBot 应该已经成功连接 QQ 官方接口。使用 `私聊` 的方式在 QQ 对机器人发送 `/help` 以检查是否连接成功。
</code_context>
<issue_to_address>
**suggestion (typo):** Consider adjusting the phrase “在 QQ 对机器人发送” to a more natural form such as “在 QQ 中对机器人发送”或“在 QQ 上向机器人发送”。

“在 QQ 对机器人发送”在语法和习惯用法上略显不自然,建议改为“在 QQ 中对机器人发送 `/help`”或“在 QQ 上向机器人发送 `/help`”,可读性会更好。
</issue_to_address>

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

## 🎉 大功告成

此时,你的 AstrBot 和 NapCatQQ 应该已经连接成功。使用 `私聊` 的方式在 QQ 对机器人发送 `/help` 以检查是否连接成功。
此时,你的 AstrBot 应该已经成功连接 QQ 官方接口。使用 `私聊` 的方式在 QQ 对机器人发送 `/help` 以检查是否连接成功。
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion (typo): Consider adjusting the phrase “在 QQ 对机器人发送” to a more natural form such as “在 QQ 中对机器人发送”或“在 QQ 上向机器人发送”。

“在 QQ 对机器人发送”在语法和习惯用法上略显不自然,建议改为“在 QQ 中对机器人发送 /help”或“在 QQ 上向机器人发送 /help”,可读性会更好。

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Chinese QQ Official WebSocket documentation to correct the “connection success” wording by removing an incorrect reference to NapCatQQ, clarifying that AstrBot connects to the QQ Official interface.

Changes:

  • Fix the “🎉 大功告成” section text to describe a successful connection to the QQ 官方接口 (WebSocket) rather than mentioning NapCatQQ.

@dosubot dosubot bot added the area:platform The bug / feature is about IM platform adapter, such as QQ, Lark, Telegram, WebChat and so on. label Apr 1, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the documentation for the QQ official interface WebSocket connection, clarifying that AstrBot connects directly to the official interface rather than NapCatQQ. A review comment suggests refining the wording to be more definitive about the successful connection status to improve user confidence.

## 🎉 大功告成

此时,你的 AstrBot 和 NapCatQQ 应该已经连接成功。使用 `私聊` 的方式在 QQ 对机器人发送 `/help` 以检查是否连接成功。
此时,你的 AstrBot 应该已经成功连接 QQ 官方接口。使用 `私聊` 的方式在 QQ 对机器人发送 `/help` 以检查是否连接成功。
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

为了让文档更清晰、更具确定性,建议修改措辞。当用户按照文档步骤操作完成后,应当是确定的成功状态,使用“已成功连接”比“应该已经成功连接”能给予用户更强的信心。

Suggested change
此时,你的 AstrBot 应该已经成功连接 QQ 官方接口。使用 `私聊` 的方式在 QQ 对机器人发送 `/help` 以检查是否连接成功。
此时,你的 AstrBot 已成功连接至 QQ 官方接口。使用 `私聊` 的方式在 QQ 对机器人发送 `/help` 以检查是否连接成功。

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 2, 2026
@RC-CHN RC-CHN merged commit 2de2318 into AstrBotDevs:master Apr 2, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:platform The bug / feature is about IM platform adapter, such as QQ, Lark, Telegram, WebChat and so on. lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants