feat: add everything-claude-code ECC bundle#685
Closed
ecc-tools[bot] wants to merge 15 commits into
Closed
Conversation
…g-claude-code/SKILL.md)
…g-claude-code/SKILL.md)
…g-claude-code/agents/openai.yaml)
…-claude-code-guardrails.md)
…ing-claude-code-research-playbook.md)
…claude-code-team-config.json)
3 tasks
Contributor
There was a problem hiding this comment.
ecc-tools[bot] has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.
Contributor
Author
|
3 tasks
Contributor
Author
Analysis CompleteGenerated ECC bundle from 500 commits | Confidence: 100% View Pull Request #685Repository Profile
Detected Workflows (8)
Generated Instincts (16)
After merging, import with: Files
|
This was referenced Mar 20, 2026
Contributor
There was a problem hiding this comment.
4 issues found across 15 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name=".claude/rules/everything-claude-code-guardrails.md">
<violation number="1" location=".claude/rules/everything-claude-code-guardrails.md:17">
P2: The guardrail’s file-naming rule is inconsistent with the repo convention (`camelCase` vs lowercase-with-hyphens). Update it to match documented standards.</violation>
</file>
<file name=".agents/skills/everything-claude-code/SKILL.md">
<violation number="1" location=".agents/skills/everything-claude-code/SKILL.md:34">
P3: The commit prefix list is incomplete compared with the examples in the same document; include all observed prefixes (or explicitly mark the list as partial) to avoid conflicting guidance.</violation>
<violation number="2" location=".agents/skills/everything-claude-code/SKILL.md:130">
P2: The file naming convention is incorrect: the repo examples shown here use kebab-case paths (and uppercase `SKILL.md`), not camelCase.</violation>
</file>
<file name=".claude/commands/feature-development.md">
<violation number="1" location=".claude/commands/feature-development.md:23">
P2: The suggested sequence contradicts the repo’s mandatory test-first workflow by telling contributors to implement before writing tests.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
|
|
||
| ## Code Style | ||
|
|
||
| - Use `camelCase` file naming. |
Contributor
There was a problem hiding this comment.
P2: The guardrail’s file-naming rule is inconsistent with the repo convention (camelCase vs lowercase-with-hyphens). Update it to match documented standards.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .claude/rules/everything-claude-code-guardrails.md, line 17:
<comment>The guardrail’s file-naming rule is inconsistent with the repo convention (`camelCase` vs lowercase-with-hyphens). Update it to match documented standards.</comment>
<file context>
@@ -0,0 +1,34 @@
+
+## Code Style
+
+- Use `camelCase` file naming.
+- Prefer `relative` imports and `mixed` exports.
+
</file context>
|
|
||
| | Element | Convention | | ||
| |---------|------------| | ||
| | Files | camelCase | |
Contributor
There was a problem hiding this comment.
P2: The file naming convention is incorrect: the repo examples shown here use kebab-case paths (and uppercase SKILL.md), not camelCase.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .agents/skills/everything-claude-code/SKILL.md, line 130:
<comment>The file naming convention is incorrect: the repo examples shown here use kebab-case paths (and uppercase `SKILL.md`), not camelCase.</comment>
<file context>
@@ -0,0 +1,429 @@
+
+| Element | Convention |
+|---------|------------|
+| Files | camelCase |
+| Functions | camelCase |
+| Classes | PascalCase |
</file context>
Suggested change
| | Files | camelCase | | |
| | Files | kebab-case paths; preserve existing special-case filenames (for example `SKILL.md`) | |
| ## Suggested Sequence | ||
|
|
||
| 1. Understand the current state and failure mode before editing. | ||
| 2. Make the smallest coherent change that satisfies the workflow goal. |
Contributor
There was a problem hiding this comment.
P2: The suggested sequence contradicts the repo’s mandatory test-first workflow by telling contributors to implement before writing tests.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .claude/commands/feature-development.md, line 23:
<comment>The suggested sequence contradicts the repo’s mandatory test-first workflow by telling contributors to implement before writing tests.</comment>
<file context>
@@ -0,0 +1,36 @@
+## Suggested Sequence
+
+1. Understand the current state and failure mode before editing.
+2. Make the smallest coherent change that satisfies the workflow goal.
+3. Run the most relevant verification for touched files.
+4. Summarize what changed and what still needs review.
</file context>
|
|
||
| ### Commit Style: Conventional Commits | ||
|
|
||
| ### Prefixes Used |
Contributor
There was a problem hiding this comment.
P3: The commit prefix list is incomplete compared with the examples in the same document; include all observed prefixes (or explicitly mark the list as partial) to avoid conflicting guidance.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .agents/skills/everything-claude-code/SKILL.md, line 34:
<comment>The commit prefix list is incomplete compared with the examples in the same document; include all observed prefixes (or explicitly mark the list as partial) to avoid conflicting guidance.</comment>
<file context>
@@ -0,0 +1,429 @@
+
+### Commit Style: Conventional Commits
+
+### Prefixes Used
+
+- `fix`
</file context>
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.
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/everything-claude-code/SKILL.md.agents/skills/everything-claude-code/SKILL.md.agents/skills/everything-claude-code/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/everything-claude-code-instincts.yaml.claude/rules/everything-claude-code-guardrails.md.claude/research/everything-claude-code-research-playbook.md.claude/team/everything-claude-code-team-config.json.claude/enterprise/controls.md.claude/commands/feature-development.md.claude/commands/add-new-skill.md.claude/commands/add-new-agent.mdOptional: Continuous Learning (17 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
Summary by cubic
Adds a repo-local ECC bundle for Claude Code and Codex to provide project-specific skills, guardrails, and workflow commands. This standardizes conventions, research flows, and team setup across tools.
New Features
Migration
Written for commit eb06b29. Summary will update on new commits.