feat: add refined-github-projects ECC bundle - #62
ecc-tools[bot] wants to merge 12 commits into
Conversation
…github-projects/SKILL.md)
…github-projects/SKILL.md)
…github-projects/agents/openai.yaml)
…incts/inherited/refined-github-projects-instincts.yaml)
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor auto review
Found 6 actionable issue(s) on changed lines.
Generated ECC guidance has several high-confidence factual errors (monorepo, ~/ aliases, fictional import paths, wrong type folders) that would steer agents into incorrect layout and imports. Committed Codex MCP defaults also pin @latest and enable a remote third-party server.
Generated automatically when this PR was submitted using Cursor CLI with --model auto.
|
|
||
| ## Architecture | ||
|
|
||
| ### Project Structure: Monorepo |
There was a problem hiding this comment.
This is a single-package WXT browser extension, not a monorepo—pnpm-workspace.yaml has no packages: entries and there is no packages/ tree. Correct this (and the identical .agents/skills/.../SKILL.md copy) so agents do not invent a multi-package layout.
|
|
||
| ### Guidelines | ||
|
|
||
| - Group code by type (components, services, utils) |
There was a problem hiding this comment.
components/, services/, and utils/ do not exist here. Point agents at the real type folders already listed in Source Layout (features/, lib/, ui/, background/, entries/, assets/). Mirror the fix in the .agents skill copy.
| | Classes | PascalCase | | ||
| | Constants | SCREAMING_SNAKE_CASE | | ||
|
|
||
| ### Import Style: Path Aliases (@/, ~/) |
There was a problem hiding this comment.
Only @ → src is configured (vitest.config.ts); the repo has zero ~/ imports. Drop ~/ from this guidance (and the .agents copy) or agents will emit broken import paths.
|
|
||
| ```typescript | ||
| // Use path aliases for imports | ||
| import { Button } from '@/components/Button' |
There was a problem hiding this comment.
These examples reference modules that do not exist (@/components/Button, @/hooks/useAuth, @/lib/api). Replace with real paths such as @/ui/..., @/lib/..., @/features/... so agents follow the actual codebase.
|
|
||
| ## Action | ||
|
|
||
| Place code in the appropriate type folder (components/, services/, utils/, etc.) |
There was a problem hiding this comment.
Action instructs agents to place code in components/, services/, utils/, which are not present. Align the Action with the Evidence folders (assets, background, entries, features, lib, and ui) before this instinct is imported.
|
|
||
| [mcp_servers.context7] | ||
| command = "npx" | ||
| args = ["-y", "@upstash/context7-mcp@latest"] |
There was a problem hiding this comment.
Repo-local Codex defaults should not use unpinned @latest MCP packages (also on Playwright below) or silently enable the remote Exa MCP at https://mcp.exa.ai/mcp. Pin versions and keep third-party/remote MCPs opt-in via ~/.codex/config.toml to avoid supply-chain and data-exfiltration risk for anyone loading this repo config.
1 participant
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/refined-github-projects/SKILL.md.agents/skills/refined-github-projects/SKILL.md.agents/skills/refined-github-projects/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/refined-github-projects-instincts.yaml.claude/commands/refactoring.mdOptional: Continuous Learning (12 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 to standardize Claude Code and Codex workflows for refined-github-projects. Previously no ECC artifacts existed; now the repo provides skills, a Codex baseline, multi‑agent roles, and a refactoring command to align tooling with project conventions.
.claude/skills/...and.agents/skills/...match the repo; adjust as needed..codex/config.tomlenables MCP servers@modelcontextprotocol/server-github,@upstash/context7-mcp,@modelcontextprotocol/server-memory,@playwright/mcp, and@modelcontextprotocol/server-sequential-thinking; approval policy is on-request, sandbox is workspace-write, and multi-agent is enabled.~/.codex/config.toml; do not commit secrets./instinct-import .claude/homunculus/instincts/inherited/refined-github-projects-instincts.yaml.Written for commit c73cd7d. Summary will update on new commits.