feat: add skills ECC bundle - #5
Open
ecc-tools[bot] wants to merge 11 commits into
Open
Conversation
…kills-instincts.yaml)
WardLu
requested changes
Aug 3, 2026
WardLu
left a comment
WardLu
left a comment
Owner
There was a problem hiding this comment.
请求修改后再合并。
主要阻塞项：
- 自动生成的 repo skill 与实际仓库不符：把 Markdown/Skill 集合描述成 Python 项目，并生成了 TypeScript 示例、PascalCase/camelCase 规则。
.codex/config.toml使用多个npx ... @latest或未锁定的远程 MCP，存在不可复现和供应链风险。- repo-local 配置隐式启用
workspace-write、联网搜索和多代理，扩大所有贡献者会话的默认行为，需要明确的安全边界和最小权限说明。
在这些问题修复并重新检查前，不建议合并。
|
|
||
| ## Overview | ||
|
|
||
| This skill teaches Claude the development patterns and conventions used in skills. |
Owner
There was a problem hiding this comment.
该生成 skill 将仓库描述为 Python 项目，并给出 TypeScript import/export、PascalCase 文件名和 camelCase 函数名等规则。当前仓库实际是 Markdown/Skill 集合，这些结论不准确，会误导后续代理。请基于真实仓库结构重生成，或删除这份自动生成 skill。
| sandbox_mode = "workspace-write" | ||
| web_search = "live" | ||
|
|
||
| [mcp_servers.github] |
Owner
There was a problem hiding this comment.
该 repo-local Codex 配置会在打开仓库时加载多个外部 MCP，并使用 npx 的 @latest 版本。版本不可复现且会执行未经锁定的远程包，存在供应链和行为漂移风险。请锁定明确版本，并补充为什么这些 MCP 是本仓库必需的；否则不要把该配置放入仓库。
|
|
||
| # ECC Tools generated Codex baseline | ||
| approval_policy = "on-request" | ||
| sandbox_mode = "workspace-write" |
Owner
There was a problem hiding this comment.
这里启用了 sandbox_mode = "workspace-write"、联网搜索和多代理，这会改变所有在本仓库中运行的 Codex 会话行为。请先说明这是有意的仓库级策略，并补充最小权限与安全边界；否则应移除 repo-local 配置，避免隐式改变使用者的本地 Codex 行为。
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
1 participant
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.
Summary
Auto-generated ECC bundle from repository analysis.
What This Does
Merging this PR adds repo-local ECC artifacts for both Claude Code and Codex. The generated bundle captures repository patterns, Codex baseline config, and reusable workflow scaffolds derived from git history analysis.
Analysis Scope
Files
.claude/ecc-tools.json.claude/skills/skills/SKILL.md.agents/skills/skills/SKILL.md.agents/skills/skills/agents/openai.yaml.claude/identity.json.codex/config.toml.codex/AGENTS.md.codex/agents/explorer.toml.codex/agents/reviewer.toml.codex/agents/docs-researcher.toml.claude/homunculus/instincts/inherited/skills-instincts.yamlOptional: Continuous Learning (5 instincts)
This PR also includes instincts for the continuous-learning-v2 skill. These are optional and only useful if you use that skill.
Import after merging:
Review Checklist
ECC Tools | Everything Claude Code