Skip to content

feat: add refined-github-projects ECC bundle - #61

Closed
ecc-tools[bot] wants to merge 12 commits into
mainfrom
ecc-tools/refined-github-projects-1781444461944
Closed

feat: add refined-github-projects ECC bundle#61
ecc-tools[bot] wants to merge 12 commits into
mainfrom
ecc-tools/refined-github-projects-1781444461944

Conversation

@ecc-tools

@ecc-tools ecc-tools Bot commented Aug 8, 2026

Copy link
Copy Markdown

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

  • Commit history patterns and conventions
  • Code architecture and structure
  • Testing patterns and coverage
  • Recurring workflows

Files

Path Description
.claude/ecc-tools.json ECC install manifest used for upgrades, repair, and uninstall.
.claude/skills/refined-github-projects/SKILL.md Repository-specific Claude Code skill generated from git history.
.agents/skills/refined-github-projects/SKILL.md Codex-facing copy of the generated repository skill.
.agents/skills/refined-github-projects/agents/openai.yaml Codex skill metadata so the repo skill appears cleanly in the skill interface.
.claude/identity.json Suggested identity.json baseline derived from repository conventions.
.codex/config.toml Repo-local Codex MCP and multi-agent baseline aligned with ECC defaults.
.codex/AGENTS.md Codex usage guide that points at the generated repo skill and workflow bundle.
.codex/agents/explorer.toml Read-only explorer role config for Codex multi-agent work.
.codex/agents/reviewer.toml Read-only reviewer role config focused on correctness and security.
.codex/agents/docs-researcher.toml Read-only docs researcher role config for API verification.
.claude/homunculus/instincts/inherited/refined-github-projects-instincts.yaml Continuous-learning instincts derived from repository patterns.
.claude/commands/refactoring.md Workflow command scaffold for refactoring.
Optional: 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:

/instinct-import .claude/homunculus/instincts/inherited/refined-github-projects-instincts.yaml

Review Checklist

  • Verify detected patterns are accurate
  • Confirm generated config, commands, and skill metadata match the repo’s real workflow
  • Check best practices align with team standards before merging

ECC Tools | Everything Claude Code


Summary by cubic

Adds a repo-local ECC bundle with Claude Code and Codex skills, a Codex baseline, multi‑agent roles, and a refactoring workflow to enable repo-aware development without touching runtime code.

  • New Features

    • Repo skills: .claude/skills/refined-github-projects/SKILL.md and .agents/skills/refined-github-projects/SKILL.md
    • Codex baseline: .codex/config.toml with MCP servers (GitHub, Context7, Exa, Memory, Playwright, Sequential Thinking) and multi‑agent roles (Explorer, Reviewer, Docs Researcher)
    • Refactoring command: .claude/commands/refactoring.md for repeatable cleanup workflow
    • Optional instincts: .claude/homunculus/instincts/inherited/refined-github-projects-instincts.yaml
  • Migration

    • If using continuous-learning v2, import instincts after merge: /instinct-import .claude/homunculus/instincts/inherited/refined-github-projects-instincts.yaml

Written for commit f6d8dd5. Summary will update on new commits.

Review in cubic

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor auto review

Found 6 actionable issue(s) on changed lines.

Generated ECC skill/instincts misrepresent this single-package WXT extension: they label it a monorepo, invent ~/ and @/components/@/hooks imports, and steer new code into non-existent components//services//utils/ folders instead of features/lib/ui/background/entries. Same SKILL.md content is duplicated under .claude and .agents—both need correcting.

Generated automatically when this PR was submitted using Cursor CLI with --model auto.


## Architecture

### Project Structure: Monorepo

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a single-package WXT browser extension, not a monorepo (pnpm-workspace.yaml only holds pnpm settings; there are no workspace packages). Relabel as a single-package app so agents do not invent package boundaries.


### Guidelines

- Group code by type (components, services, utils)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These guidelines conflict with the real layout listed above. Direct new code into features/, lib/, ui/, background/, or entries/—not generic components/, services/, or utils/ folders that do not exist here. Apply the same fix in .agents/skills/refined-github-projects/SKILL.md.

| Classes | PascalCase |
| Constants | SCREAMING_SNAKE_CASE |

### Import Style: Path Aliases (@/, ~/)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only @/ is used (see vitest.config.ts and src/**); there are no ~/ imports. Drop ~/ from the documented alias set so agents do not emit broken import paths.


```typescript
// Use path aliases for imports
import { Button } from '@/components/Button'

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These examples invent modules (@/components/Button, @/hooks/useAuth) that are not in this repo. Replace with real paths such as @/ui/..., @/lib/..., and @/features/... (and mirror the change in the identical .agents skill copy).


## Action

Use path aliases (@/, ~/) instead of relative imports

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same inaccuracy as the skill: teach @/ only. Recommending ~/ will cause invalid imports in this codebase.


## Action

Place code in the appropriate type folder (components/, services/, utils/, etc.)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Evidence lists background, entries, features, lib, but the action tells agents to use components/, services/, utils/. Update the action to those real top-level folders so continuous-learning instincts do not fight the source layout.

@fathiraz fathiraz closed this Aug 10, 2026
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.

1 participant