Skip to content

add: shared-a2a-skill — A2A protocol skill for cross-agent communication - #56

Closed
lilozhao wants to merge 2 commits into
Qoder-AI:mainfrom
lilozhao:add-a2a-skill
Closed

add: shared-a2a-skill — A2A protocol skill for cross-agent communication#56
lilozhao wants to merge 2 commits into
Qoder-AI:mainfrom
lilozhao:add-a2a-skill

Conversation

@lilozhao

Copy link
Copy Markdown
Contributor

碳硅契(Carbon-Silicon Bond)系列技能包,提交到 Qoder 社区 marketplace。

仓库: https://github.com/lilozhao/carbon-silicon-bond-protocol
社区: https://csbc.lilozkzy.top/

@cyandata

Copy link
Copy Markdown
Collaborator

感谢提交。我们审核发现,shared-a2a-skill 当前版本存在敏感信息硬编码问题,暂不建议合入。

主要问题是:Skill 或相关配置中疑似包含硬编码的 token、密钥、服务地址、身份凭证或其他敏感配置。这类信息一旦进入公开仓库或 Marketplace 包,即使后续删除,也可能已经通过提交历史、缓存或分发包泄露。

请先完成以下处理:

  1. 移除所有硬编码敏感信息,改为由用户通过环境变量、本地配置文件或显式输入提供。
  2. 将示例配置替换为占位符,例如 YOUR_API_KEYYOUR_ENDPOINT,不要放真实值。
  3. 如果已提交过真实密钥、token 或身份凭证,请立即吊销并重新生成,不能只在代码里删除。
  4. 如果敏感信息已经进入 git 历史,建议基于干净分支重新提交,避免 Marketplace 继续分发包含泄露历史的内容。
  5. 补充说明该 Skill 会连接哪些服务、需要哪些权限、数据会发送到哪里。

完成上述修改后可以重新提交 review。在敏感信息硬编码问题解决前,我们建议关闭当前 PR。

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.

2 participants