Skip to content

refactor(agents): remove duplicate agent that already exists as skill#234

Open
mvanhorn wants to merge 3 commits intoEveryInc:mainfrom
mvanhorn:refactor/move-skill-only-agents-to-skills
Open

refactor(agents): remove duplicate agent that already exists as skill#234
mvanhorn wants to merge 3 commits intoEveryInc:mainfrom
mvanhorn:refactor/move-skill-only-agents-to-skills

Conversation

@mvanhorn
Copy link

@mvanhorn mvanhorn commented Mar 8, 2026

Summary

Fixes #156 - Agent files for skill-only components waste tokens and error on invocation.

  • Removed agents/workflow/every-style-editor.md - this agent is a duplicate of the existing skills/every-style-editor/ skill. The skill version is more comprehensive (structured 4-phase review process with reference files) while the agent version is a simpler copy
  • Updated plugin.json agent counts from 29 to 28 (both .claude-plugin and .cursor-plugin)
  • Updated README.md workflow agent table (count 5 to 4, removed duplicate row)

Why conservative scope

After reading each agent file and cross-referencing against existing skills:

  • every-style-editor is the only clear duplicate where both an agent file and a skill directory exist with the same name and overlapping purpose
  • ankane-readme-writer (agent) vs andrew-kane-gem-writer (skill) are related but cover different scopes (README writing vs gem code patterns) - not duplicates
  • dhh-rails-reviewer (agent) vs dhh-rails-style (skill) serve different purposes (code review persona vs code writing style guide) - not duplicates
  • All research and review agents are legitimately used via the Task/Agent tool and should remain as agents

Token savings

~100-200 tokens per session saved by removing 1 eagerly-loaded agent description from the Task tool definition. The every-style-editor agent description was 64 lines.

Test plan

  • Verify every-style-editor skill still works via /every-style-editor or Skill tool invocation
  • Verify no workflow commands break (checked compound.md - it lists the editor as a recommendation, not a direct Task invocation)
  • Verify remaining 28 agents load correctly

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

mvanhorn and others added 3 commits March 8, 2026 12:43
Replace all hardcoded localhost:3000 references with dynamic port
detection. The command now checks (in priority order): explicit
--port argument, CLAUDE.md config, package.json scripts, .env files,
then falls back to 3000.

Closes EveryInc#164

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The every-style-editor agent file was a duplicate of the existing
every-style-editor skill. Agent files are eagerly loaded into the
Task tool definition on every API call (~100-200 tokens each),
while skills are lazy-loaded only when invoked. Removing the
duplicate saves tokens and eliminates potential runtime errors
when the agent is invoked via Task tool instead of Skill tool.

Changes:
- Delete agents/workflow/every-style-editor.md (skill version
  in skills/every-style-editor/ already exists)
- Update README.md workflow agent count from 5 to 4
- Update plugin.json agent counts from 29 to 28

Fixes EveryInc#156

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add Phase 0 context budget check that warns users when running /compound
near context limits, and offers a compact-safe single-pass alternative
that avoids launching 5 parallel subagents.

Closes EveryInc#198

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

Agent files for skill-only components waste tokens and error on invocation

1 participant