mirror of
https://github.com/affaan-m/ECC.git
synced 2026-09-08 07:37:48 +02:00
ac30ff3ea249bf5f94dbc5e9b18ab681cc9af91d
4
Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
|
cfb3370df8 |
docs: add Antigravity setup and usage guide (#552)
* docs: add Antigravity setup and usage guide Addresses #462 — users were confused about Antigravity skills setup. Adds a comprehensive guide covering: - Install mapping (ECC → .agent/ directory) - Directory structure after install - openai.yaml agent config format - Managing installs (list, doctor, uninstall) - Cross-target comparison table - Troubleshooting common issues - How to contribute skills with Antigravity support Also links the guide from the README FAQ section. * fix: address review feedback on Antigravity guide - Remove spurious skills/ row from install mapping table, add note clarifying .agents/skills/ is static repo layout not installer-mapped - Fix repair section: doctor.js diagnoses, repair.js restores - Fix .agents/ → .agent/ path typo in custom skills section - Clarify 3-step workflow for adding Antigravity skills - Fix antigravity-project → antigravity in comparison table - Fix "flatten" → "flattened" grammar in README - Clarify openai.yaml full nested path structure * fix: clarify .agents/ vs .agent/ naming and fix Cursor comparison - Explain that .agents/ (with 's') is ECC source, .agent/ (no 's') is Antigravity runtime — installer copies between them - Fix Cursor Agents/Skills column: Cursor has no explicit agents/skills mapping (only rules), changed from 'skills/' to 'N/A' * fix: correct installer behavior claims and command style - Fix .agents/ vs .agent/ note: clarify that only rules, commands, and agents (no dot) are explicitly mapped by the installer. The dot-prefixed .agents/ directory falls through to default scaffold, not a direct copy. - Fix contributor workflow: remove false auto-deploy claim for openai.yaml. Clarify .agents/ is static repo layout, not installer-deployed. - Fix uninstall command: use direct script call (node scripts/uninstall.js) for consistency with doctor.js, repair.js, list-installed.js. * fix: add missing agents/ step to contributor workflow Contributors must add an agent definition at agents/ (no dot) for the installer to deploy it to .agent/skills/ at runtime. Without this step, skills only exist in the static .agents/ layout and are never deployed. --------- Co-authored-by: vazidmansuri005 <vazidmansuri005@users.noreply.github.com> |
||
|
|
d697f2ebac |
feat(skills): add architecture-decision-records skill (#555)
* feat(skills): add architecture-decision-records skill
Adds a skill that captures architectural decisions made during coding
sessions as structured ADR documents (Michael Nygard format).
Features:
- Auto-detects decision moments from conversation signals
- Records context, alternatives considered with pros/cons, and consequences
- Maintains numbered ADR files in docs/adr/ with an index
- Supports ADR lifecycle (proposed → accepted → deprecated/superseded)
- Categorizes decisions worth recording vs trivial ones to skip
- Integrates with planner, code-reviewer, and codebase-onboarding skills
Includes Antigravity support via .agents/skills/ and openai.yaml.
* fix: address review feedback on ADR skill
- Add missing "why did we choose X?" read-ADR trigger to .agents/ copy
- Add canonical-reference link to .agents/ SKILL.md pointing to full version
- Remove integration reference to non-existent codebase-onboarding skill
* fix: add initialization step and sync .agents/ trigger
- Add Step 1 to workflow: initialize docs/adr/ directory, README.md
index, and template.md on first use when directory doesn't exist
- Add "API design" to .agents/ alternatives trigger to match canonical
version
* fix: address ADR workflow gaps and implicit signal safety
- Init step: seed README.md with index table header so Step 8 can
append rows correctly on first ADR
- Add read-path workflow: graceful handling when docs/adr/ is empty
or absent ("No ADRs found, would you like to start?")
- Implicit signals: add "do not auto-create without user confirmation"
guard, tighten triggers to require conclusion/rationale not just
discussion, remove overly broad "testing strategy" trigger
* fix: require user confirmation before creating files
- Canonical SKILL.md: init step now asks user before creating docs/adr/
- .agents/ condensed version: add confirmation gate for implicit signals
and explicit consent step before any file writes
* fix: require user approval before writing ADR file, add refusal path
* fix: remove .agents/ duplicate, keep canonical in skills/
---------
Co-authored-by: vazidmansuri005 <vazidmansuri005@users.noreply.github.com>
|
||
|
|
0efd6ed914 |
feat(commands): add /context-budget optimizer command (#554)
* feat(commands): add /context-budget optimizer command Adds a command that audits context window token consumption across agents, skills, rules, MCP servers, and CLAUDE.md files. Detects bloated agent descriptions, redundant components, MCP over-subscription, and CLAUDE.md bloat. Produces a prioritized report with specific token savings per optimization. Directly relevant to #434 (agent descriptions too verbose, ~26k tokens causing performance warnings). * fix: address review feedback on context-budget command - Add $ARGUMENTS to enable --verbose flag passthrough - Fix MCP token estimate: 45 tools × ~500 tokens = ~22,500 (was ~2,200) - Fix heavy agents example: all 3 now exceed 200-line threshold - Fix description threshold: warning at >30 words, fail at >50 words - Add Step 4 instructions (was empty) - Fix audit cadence: "quarterly" → "regularly" + "monthly" consistently - Fix Output Format heading level under Step 4 - Replace "Antigravity" with generic "harness versions" - Recalculate total overhead to match corrected MCP numbers * fix: correct MCP tool count and savings percentage in sample output - Fix MCP tool count: table now shows 87 tools matching the issues section (was 45 in table vs 87 in issues) - Fix savings percentage: 5,100 / 66,400 = 7.7% (was 20.6%) - Recalculate total overhead and effective context to match * fix: correct sample output arithmetic - Fix total overhead: 66,400 → 66,100 to match component table sum (12,400 + 6,200 + 2,800 + 43,500 + 1,200 = 66,100) - Fix MCP savings: ~1,500 → ~27,500 tokens (55 tools × 500 tokens/tool) to match the per-tool formula defined in Step 1 - Reorder optimizations by savings (MCP removal is now #1) - Fix total savings and percentage (31,100 / 66,100 = 47.0%) * fix: distinguish always-on vs on-demand agent overhead Agent descriptions are always loaded into Task tool routing context, but the full agent body is only loaded when invoked. The audit now measures both: description-only tokens as always-on overhead and full-file tokens as worst-case overhead. This resolves the contradiction between Step 1 (counting full files) and Tip 1 (saying only descriptions are loaded per session). * fix: simplify agent accounting and resolve inconsistencies - Revert to single agent overhead metric (full file tokens) — simpler and matches what the report actually displays - Add back 200-line threshold for heavy agents in Step 1 - Fix heavy agents action to match issue type (split/trim, not description-only) - Remove .agents/skills/ scan path (doesn't exist in ECC repo) - Consolidate description threshold to single 30-word check * fix: add model assumption and verbose mode activation - Step 4: assume 200K context window by default (Claude has no way to introspect its model at runtime) - Step 4: add explicit instruction to check $ARGUMENTS for --verbose flag and include additional output when present * fix: handle .agents/skills/ duplicates in skill scan Skills scan now checks .agents/skills/ for Codex harness copies and skips identical duplicates to avoid double-counting overhead. * fix: add savings estimate to heavy agents action for consistency * feat(skills): add context-budget backing skill, slim command to delegator * fix: use structurally detectable classification criteria instead of session frequency --------- Co-authored-by: vazidmansuri005 <vazidmansuri005@users.noreply.github.com> |
||
|
|
72c013d212 |
feat(skills): add codebase-onboarding skill (#553)
* feat(skills): add codebase-onboarding skill Adds a skill that systematically analyzes an unfamiliar codebase and produces two artifacts: a structured onboarding guide and a starter CLAUDE.md tailored to the project's conventions. Four-phase workflow: 1. Reconnaissance — parallel detection of manifests, frameworks, entry points, directory structure, tooling, and test setup 2. Architecture mapping — tech stack, patterns, key directories, request lifecycle tracing 3. Convention detection — naming, error handling, async patterns, git workflow from recent history 4. Artifact generation — scannable onboarding guide + project-specific CLAUDE.md Includes Antigravity support via .agents/skills/ and openai.yaml. * fix: address review feedback on codebase-onboarding skill - Rename headings to match skill format: When to Activate → When to Use, Onboarding Workflow → How It Works - Add Examples section with 3 usage scenarios - Mark Phase 4 Next.js paths as example with HTML comments - Fix CLAUDE.md generation to read/enhance existing file first - Replace abbreviated .agents/ SKILL.md with full copy per repo convention * fix: add example marker to Common Tasks template section Adds <!-- Example for a Node.js project --> comment to Common Tasks, matching the markers already on Key Entry Points and Where to Look. Syncs .agents/ copy. * fix: add missing example markers and shorten default_prompt - Add example comment to Tech Stack table in Phase 4 template - Add example comment to Key Directories block in Phase 2 - Shorten openai.yaml default_prompt to match repo convention (~60 chars) - Sync .agents/ SKILL.md copy * fix: add empty-repo fallback and remove hardcoded output path - Phase 3: add fallback for repos with no git history - Example 1: remove hardcoded docs/ path assumption, output to conversation or project root instead - Sync .agents/ copy * fix: remove .agents/ duplicate, keep canonical in skills/ * fix: clarify Example 1 output destination * fix: add shallow-clone fallback to git conventions detection --------- Co-authored-by: vazidmansuri005 <vazidmansuri005@users.noreply.github.com> |