Skip to content

ai: 添加编码指南skill#1602

Merged
MistEO merged 2 commits intov2from
skill/coding
Mar 25, 2026
Merged

ai: 添加编码指南skill#1602
MistEO merged 2 commits intov2from
skill/coding

Conversation

@MistEO
Copy link
Copy Markdown
Contributor

@MistEO MistEO commented Mar 25, 2026

看看,主要是 go 的,每次 cursor 写出来的都不符合 Copilot 的规范,干脆单独整理一份

Summary by Sourcery

为 MaaEnd pipeline JSON 编写和 go-service Go 代码规范新增专用的 Claude 技能,并更新现有的 Go 服务说明,指向新的共享指南。

增强:

  • 将内联的 go-service 说明替换为对新的共享 Go 服务指南技能的引用,以集中化并简化代码规范的维护。

文档:

  • 引入一份全面的 MaaEnd Pipeline JSON 编写指南作为 Claude 技能,涵盖命名、识别/动作、流程控制、可复用节点以及审阅清单。
  • 添加一份详细的 Go 服务编码指南作为 Claude 技能,定义架构边界、包布局、注册模式、日志记录和注释规范。
  • 提供一份 Pipeline 字段参考文档,总结 MaaFramework Pipeline 协议字段、默认值和用法,用于快速查阅。
Original summary in English

Summary by Sourcery

Add dedicated Claude skills for MaaEnd pipeline JSON authoring and go-service Go code conventions, and update existing Go service instructions to point to the new shared guide.

Enhancements:

  • Replace the inline go-service instructions with a reference to the new shared Go service guide skill to centralize and simplify maintenance of coding conventions.

Documentation:

  • Introduce a comprehensive MaaEnd Pipeline JSON authoring guide as a Claude skill, covering naming, recognition/actions, flow control, reusable nodes, and review checklist.
  • Add a detailed Go service coding guide as a Claude skill, defining architecture boundaries, package layout, registration patterns, logging, and commenting standards.
  • Provide a Pipeline field reference document summarizing MaaFramework Pipeline protocol fields, defaults, and usage for quick lookup.

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.

嗨,我已经审阅了你的修改,看起来很棒!


Sourcery 对开源项目是免费的——如果你觉得我们的评审有帮助,欢迎分享 ✨
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进后续的评审。
Original comment in English

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 ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.


- 上下文(组件名、步骤、场景)用链式字段,禁止拼进 `Msg`。
- 错误、参数、识别结果一律用链式字段(`.Err(err)`、`.Int("x", x)`)。

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.

日志里再加一条统一用英文吧,我今天看好像有个模块的作者喜欢一会中文日志,一会英文日志的

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

倒是无所谓,反正都是开发者自己看,自己能看懂就行了

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.

我觉得可能是ai的锅,这种写法不像人类.jpg

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

本 PR 旨在把 MaaEnd 的 Go(go-service)与 Pipeline JSON 编写规范整理为独立的 “skill” 文档,便于后续 AI/协作者统一遵循同一份指南。

Changes:

  • 新增 .claude/skills/go-service-guide:沉淀 go-service 的结构/注册/日志/注释等规范与模板
  • 新增 .claude/skills/pipeline-guide:提供 Pipeline 编写指南与字段速查表
  • 精简 go-service 路径专属指令文件为对 skill 文档的引用,并调整 .gitignore 中与 .claude 相关规则

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
.gitignore 移除对 .claude/ 的忽略规则
.github/instructions/go-service.instructions.md go-service 路径指令改为引用 go-service skill 文档
.claude/skills/pipeline-guide/field-reference.md 新增 Pipeline 字段速查表
.claude/skills/pipeline-guide/SKILL.md 新增 Pipeline 编写规范与模式说明
.claude/skills/go-service-guide/SKILL.md 新增 go-service 编写规范与模板

@MistEO MistEO merged commit 3f989f8 into v2 Mar 25, 2026
1 check passed
@MistEO MistEO deleted the skill/coding branch March 25, 2026 15:50
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.

3 participants