Skip to content

feat(skills): add claude-bot and enhance github-action-diagnose#17

Closed
KadenZhang3321 wants to merge 34 commits into
opensourceways:mainfrom
KadenZhang3321:main
Closed

feat(skills): add claude-bot and enhance github-action-diagnose#17
KadenZhang3321 wants to merge 34 commits into
opensourceways:mainfrom
KadenZhang3321:main

Conversation

@KadenZhang3321

@KadenZhang3321 KadenZhang3321 commented Mar 31, 2026

Copy link
Copy Markdown

描述

新增 claude-bot skill,并同步更新 github-action-diagnose skill。

新增:skills/infrastructure/claude-bot/

集成于 GitHub Issues / Pull Requests 的 AI 助手,组织成员在 Issue 或 PR 评论中输入 @claude 即可触发 Claude 自动回复。底层通过可复用的 GitHub Actions Workflow 实现,支持 Anthropic API / OpenRouter / AWS Bedrock 多种接入方式。

核心文件:

  • .github/workflows/_claude-code.yml — 可复用核心工作流,供组织内各仓库通过 workflow_call 调用
  • .github/workflows/example-caller.yml — 调用方模板,复制到目标仓库即可接入
  • .github/workflows/README.md — 完整集成指南(前置条件、三步快速开始、安全说明、故障排查)
  • .github/scripts/setup-claude-environment.py — 自动创建/验证 GitHub Environment 的初始化脚本
  • skills/infrastructure/claude-bot/claude.md — Skill 文档及 Claude Bot 行为规范

安全机制(全部在 job 级别 if: 过滤):

  • 机器人账号自动跳过
  • @claude 关键词检测
  • 组织成员身份校验(可配置,org_name 留空时跳过)
  • 仓库写权限校验(write / admin)

更新:skills/infrastructure/github-action-diagnose/

  • 重构诊断流程(5 步 → 7 步),新增精度回归、代码 Bug 分类和 Action Plan 生成
  • 补全 CONTRIBUTING.md 要求的模板字段(使用场景、前置要求、相关 Skills、更新日志、作者)
  • 新增 references/ 参考资料目录(common-patterns、vllm-ascend、classification-guide、ascend-troubleshooting)
  • 恢复 references/ascend-troubleshooting.md

相关 Issue

(#18 (comment))

变更类型

  • Bug 修复
  • 新功能
  • 代码重构
  • 文档更新
  • 样式改进
  • 性能优化
  • 测试相关
  • 其他

审查清单

  • Skill 文件命名符合规范(小写连字符)
  • 使用了模板结构
  • 包含完整的文档(描述、使用场景、前置要求、示例、注意事项、更新日志、作者)
  • 提供了使用示例
  • 更新了 skills/infrastructure/README.md
  • PR 标题和描述清晰
  • 没有包含敏感信息(密码、密钥等)

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

Linking Issue Notice

@KadenZhang3321 , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .

Comment thread .github/workflows/_claude-code.yml
@tfhddd

tfhddd commented Mar 31, 2026

Copy link
Copy Markdown

commit需要合一

@tfhddd

tfhddd commented Mar 31, 2026

Copy link
Copy Markdown

@claude

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@tfhddd

tfhddd commented Apr 1, 2026

Copy link
Copy Markdown

/approve

@opensourceways-bot

Copy link
Copy Markdown

Review Code Feedback

Thanks for your review, @tfhddd, your opinion is very important to us. 👋
The maintainers will consider your advice carefully.

@KadenZhang3321 KadenZhang3321 changed the title Add skill: claude-bot; Update skill: github-action-diagnose feat:add claude-botskill & Update github-action-diagnose skill Apr 1, 2026
@KadenZhang3321 KadenZhang3321 changed the title feat:add claude-botskill & Update github-action-diagnose skill feat(skills): add claude-bot and enhance github-action-diagnose Apr 1, 2026
@pkking

pkking commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

/lgtm
/approve

@opensourceways-bot

Copy link
Copy Markdown

Review Code Feedback

  • The label lgtm-pkking was added to this pull request. It means that pkking reviewed the code changes. 👋
Tips
  • If this pull request is not merged while all conditions are met, comment /check-pr to try again. 😄

@opensourceways-bot

Copy link
Copy Markdown

Review Code Feedback

  • The label approved was added to this pull request. It means that pkking reviewed the code changes. 👋
Tips
  • If this pull request is not merged while all conditions are met, comment /check-pr to try again. 😄

Comment thread .github/workflows/README.md Outdated
Comment thread .github/workflows/README.md Outdated
Comment thread .github/workflows/README.md Outdated
@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@zkhzkhz

zkhzkhz commented Apr 13, 2026

Copy link
Copy Markdown

/check-pr

@opensourceways-bot

Copy link
Copy Markdown

Merge Verification Failed

@zkhzkhz, this pr is not mergeable and the reasons are below:

Label BlockList
  • the needs-issue label(s) caused the pull request to be unable to merge.
Not Enough Labels
  • the pull request needs 1 gate_check_pass labels, but now gets 0.
  • the pull request needs 2 lgtm labels, but now gets 1.
Label Usage Tips
  • gate_check_pass: The code compilation/build/testcase check job is passed.

@opensourceways-bot

Copy link
Copy Markdown
检查项 状态
敏感信息扫描
安全编码扫描
漏洞扫描
Check代码检查
开源license合规扫描
DT覆盖率
开发阶段设计文档检查
流水线链接 点击跳转查看日志

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

22 similar comments
@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

- Inline whitelist, permission and prompt logic into _claude-code.yml
  to remove dependency on external python scripts
- Fix agent-skills checkout to use correct repository reference
- Rename AGENT_PAT to DISPATCH_TOKEN for cross-repo access
- Move dependency installation before inline python checks
- Sanitize response_summary newlines for GITHUB_OUTPUT compatibility
- Install gh CLI for posting comments via gh api
- Simplify retry logic and remove obsolete template files
- Update caller templates to default allow_code_change=false
- Add apply-patch jobs that create draft PRs instead of direct push
- Update documentation to reflect current workflow_call-only design
@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

KadenZhang3321, thanks for your pull request. All authors of the commits have signed the CLA. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants