Open
Conversation
- 新增 setup.sh:curl 一行命令自动检测工具、转换、安装 - 新增 guide.html:安装后 Web 引导页(深色/浅色自适应) - 新增 docs/QUICKSTART.md:3 条路径快速上手 - 新增 docs/TOOLS-GUIDE.md:各工具详细安装说明 - 新增 scripts/search-agents.sh:按关键词搜索 180 个智能体 - 新增 examples/:代码审查工作流示例 - 精简 README.md(696→288 行),折叠智能体总览表格 - 修复 lint-agents.sh 缺少 6 个部门目录的 bug - 修改 install.sh:找不到 integrations/ 时自动调用 convert.sh - 修改 .gitignore:允许 integrations/ 目录提交 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- 新增 workflows/:产品上线、代码审查、营销活动 3 个多智能体协作模板 - 新增 scripts/sync-upstream.sh:自动同步上游仓库新增智能体 - 新增 scripts/create-agent.sh:交互式智能体创建向导 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- 检测已安装工具后,只对需要 integrations/ 的工具运行 convert.sh - Claude Code 和 Copilot 直接复制源 .md,无需转换 - EXIT trap 先 cd 出临时目录再删除,避免 Windows/Git Bash 文件锁 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Git Bash 在 Windows 上偶尔无法立即释放文件句柄, 导致 rm -rf 临时目录失败。加 2>/dev/null || true 避免 无害的清理错误影响安装脚本的退出码。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- 标题改为 agency-agents-yhz,去掉 upstream 的语言切换链接 - badges 指向 Yhazrin/agency-agents-yhz - 新增"这个 fork 做了什么"区块,突出一键安装、工作流、搜索脚本等 - 实战工作流区块展示 5 个预定义协作流程 - 实用脚本区块列出全部 7 个脚本及其用途 - 智能体列表精简为按部门折叠,非工程部用一行概括 - 控制在 ~200 行,新用户 2 分钟抓住重点 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Owner
|
感谢你花时间做了这么多工作!上手体验的优化方向很好,但这个 PR 有几个问题需要解决: 需要修改的:
有价值的部分:
建议拆分成小 PR:
这样每个 PR 都小而聚焦,更容易 review 和合并。期待你的拆分版本! |
Owner
|
感谢这个 PR!提升新用户上手体验的方向非常对,能看出花了不少心思。不过有几个问题需要先解决: 需要修复的问题1. 仓库名被改了(阻塞合并)PR 里把项目名从 2. 有合并冲突当前和 main 有冲突,需要 rebase 到最新的 main 分支解决一下。 3. 建议拆分 PR这个 PR 一次做了太多事情(一键安装、文档拆分、工作流模板、搜索脚本、guide.html 等),review 起来比较吃力。建议拆成 2-3 个独立的 PR,例如:
这样每个 PR 职责单一,review 更快,也更容易合并。 4.
|
Author
|
感谢!
…On Thu, Mar 26, 2026 at 10:44 PM jnMetaCode ***@***.***> wrote:
*jnMetaCode* left a comment (jnMetaCode/agency-agents-zh#27)
<#27?email_source=notifications&email_token=BQFO4RMUQ2UDEFX5D3TJ3F34SU643A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJTGU2TGNRTGQZKM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2LK4DSL5RW63LNMVXHIX3POBSW4X3DNRUWG2Y#issuecomment-4135536342>
感谢这个 PR!提升新用户上手体验的方向非常对,能看出花了不少心思。不过有几个问题需要先解决:
需要修复的问题 1. 仓库名被改了(阻塞合并)
PR 里把项目名从 agency-agents-zh 全局替换成了 agency-agents-yhz,链接也指向了
Yhazrin/agency-agents-yhz。这个需要全部改回来——README 标题、badge 链接、curl
安装地址、guide.html 里的引用等。
2. 有合并冲突
当前和 main 有冲突,需要 rebase 到最新的 main 分支解决一下。
3. 建议拆分 PR
这个 PR 一次做了太多事情(一键安装、文档拆分、工作流模板、搜索脚本、guide.html 等),review 起来比较吃力。建议拆成 2-3
个独立的 PR,例如:
- *PR A*:文档重组(README 精简 + docs/ 拆分)
- *PR B*:安装体验优化(setup.sh + install.sh 改进)
- *PR C*:工作流模板 + 实用脚本
这样每个 PR 职责单一,review 更快,也更容易合并。
4. curl | bash 安全性
一键 curl 安装是个好想法,但建议在文档里同时提供 clone 后本地执行的方式作为替代,让用户可以先审查脚本内容。
------------------------------
以上问题里,*第 1 点(仓库名)是必须修复的*,其余是建议。修复后我们尽快 review 合并 👍
—
Reply to this email directly, view it on GitHub
<#27?email_source=notifications&email_token=BQFO4RMUQ2UDEFX5D3TJ3F34SU643A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTIMJTGU2TGNRTGQZKM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2LK4DSL5RW63LNMVXHIX3POBSW4X3DNRUWG2Y#issuecomment-4135536342>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BQFO4RK46XBM6CBUQHMAO7L4SU643AVCNFSM6AAAAACW2W4OF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DCMZVGUZTMMZUGI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
本 PR 聚焦 新用户的上手体验,不改动任何智能体内容本身。核心思路:一行命令装好,3 分钟内能用。
新增功能
修复
精简
Details
折叠影响范围
测试