Skip to content

fix: 修复setGgroupHighlights的命名为setGroupHighlights#562

Merged
yusheng929 merged 2 commits intomainfrom
copilot/fix-set-ggroup-highlights
Oct 13, 2025
Merged

fix: 修复setGgroupHighlights的命名为setGroupHighlights#562
yusheng929 merged 2 commits intomainfrom
copilot/fix-set-ggroup-highlights

Conversation

Copy link
Contributor

Copilot AI commented Oct 12, 2025

问题描述

AdapterOneBot 类中,方法名 setGgroupHighlights 存在拼写错误,多了一个小写的 'g',导致方法名不符合命名规范,且与基类 AdapterBase 和接口定义 AdapterType 不一致。

修复内容

packages/core/src/adapter/onebot/core/core.ts 文件第 992 行的方法名从 setGgroupHighlights 修正为 setGroupHighlights

修改前:

async setGgroupHighlights (_: string, messageId: string, create: boolean) {

修改后:

async setGroupHighlights (_: string, messageId: string, create: boolean) {

验证

  • ✅ 方法签名现在与基类 AdapterBase 中的定义一致
  • ✅ 方法签名现在与接口 AdapterType 中的定义一致
  • ✅ 使用 pnpm run build:core 成功构建,无编译错误
  • ✅ 构建产物中确认方法名已正确修复

此修复确保了代码的一致性和正确性,使群精华消息的设置/取消功能的接口命名符合规范。

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • registry.npmmirror.com
    • Triggering command: node /home/REDACTED/.npm/_npx/4876f32048baf8ac/node_modules/.bin/prebuild-install -r napi --pkg_version=5.1.7 --pkg_name=sqlite3 (dns block)
    • Triggering command: node /home/REDACTED/.npm/_npx/c65dbefa27d408e0/node_modules/.bin/prebuild-install -r node --pkg_version=0.12.0 --pkg_name=node-pty-prebuilt-multiarch (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>fix: 修复setGgroupHighlights的命名</issue_title>
<issue_description>将错误的接口命名setGgroupHighlights修复为setGroupHighlights,多了个小写g</issue_description>

<agent_instructions>需要使用pnpm 9进行安装依赖,修复完成之后,需要使用pnpm run build:core进行验证是否存在编译错误</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #561


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: sj817 <74231782+sj817@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix naming issue in setGgroupHighlights method fix: 修复setGgroupHighlights的命名为setGroupHighlights Oct 12, 2025
Copilot AI requested a review from sj817 October 12, 2025 13:35
@github-actions
Copy link
Contributor

你可以通过以下命令安装该版本:

pnpm add https://pkg.pr.new/node-karin@151f0bd -w

1 similar comment
@github-actions
Copy link
Contributor

你可以通过以下命令安装该版本:

pnpm add https://pkg.pr.new/node-karin@151f0bd -w

@yusheng929 yusheng929 marked this pull request as ready for review October 13, 2025 00:05
@yusheng929 yusheng929 merged commit 8f149a6 into main Oct 13, 2025
5 checks passed
@yusheng929 yusheng929 deleted the copilot/fix-set-ggroup-highlights branch October 13, 2025 00:05
@github-actions github-actions bot mentioned this pull request Oct 13, 2025
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.

fix: 修复setGgroupHighlights的命名

3 participants