Skip to content

feat!: trim to 7 unique skills, remove 18 bundled duplicates - #12

Open
griffinwork40 wants to merge 3 commits into
mainfrom
refactor/trim-duplicates
Open

griffinwork40 wants to merge 3 commits into
mainfrom
refactor/trim-duplicates

Conversation

@griffinwork40

Copy link
Copy Markdown
Owner

What

Remove 18 skills that duplicated agent-afk's bundled plugin (awa-bundled). Keep the 7 skills unique to this plugin.

Why

These 18 skills were manually synced copies of what ships with npm i agent-afk. The duplication caused:

  • Sync tax: every bundled skill update required a parallel PR here
  • Silent shadowing: plugin skills won via first-wins resolution in skill-bridge.ts, so stale plugin copies overrode newer bundled versions

What stays (7 unique skills)

Skill Purpose
agentify Bootstrap sub-agent skills for local coding agent CLIs
appmap Map a web UI, generate browser automation skill
forge-friction Surface friction patterns from session telemetry
integrate One-prompt API integration pipeline
provideme Provider bridge for coding agent CLIs
resolve Resolve PR review feedback in parallel
web Parallel browser automation across tabs

What was removed (18 duplicates)

automate, contract, devils-advocate, diagnose, false-completion-gate, fix-pr, gather, ground-claim, ground-state, parallelize, polish, refactor, research, review, shadow-verify, ship, simplify, spec

All available in agent-afk v5.x+ bundled plugin.

Also updated

  • README.md, AFK.md, CLAUDE.md: reflect 7-skill reality
  • plugin.json: version 2.2.0 -> 3.0.0 (breaking change)
  • .release-please-manifest.json: synced to 3.0.0

BREAKING CHANGE: 18 skills that duplicated agent-afk's bundled plugin
(awa-bundled) have been removed. These skills ship with agent-afk v5.x+
and were causing a manual sync tax and silent first-wins shadowing.

Removed: automate, contract, devils-advocate, diagnose,
false-completion-gate, fix-pr, gather, ground-claim, ground-state,
parallelize, polish, refactor, research, review, shadow-verify, ship,
simplify, spec.

Kept (unique to this plugin): agentify, appmap, forge-friction,
integrate, provideme, resolve, web.
F1: Restore skills/contract/SKILL.md — shared I/O schema primitive
loaded by all 6 orchestrator skills via /contract directive and both
agents via skills: [contract] frontmatter.

F2: Replace deleted ground-state calibration exemplar in qualify agent
with surviving integrate skill.
@griffinwork40

Copy link
Copy Markdown
Owner Author

pr-triage: 2 blocking findings @ b963019

1. medium · blocking · api-compat · .claude-plugin/marketplace.json:17

marketplace.json description not synced with plugin.json. Line 17 still reads "description": "multi-agent orchestration, simplified." while plugin.json was updated to "7 high-leverage skills for agent-afk that aren't bundled in core.". The marketplace listing is the primary public install surface — visitors see stale framing.

Suggestion: Sync marketplace.json:17 description with the new plugin.json description.

2. medium · blocking · spec-compliance · AFK.md:7,42 + skills/ directory

Skill count mismatch: 8 skill directories exist (agentify, appmap, contract, forge-friction, integrate, provideme, resolve, web) but all docs say "7 skills" and omit contract. The PR body claims 18 removals but net removals are 17 (contract was restored as a structural dependency). AFK.md:7 says "7 high-leverage skills"; AFK.md:42 lists 7 skills without contract.

Suggestion: Either (a) update all docs to "8 skills" and add contract to every skill table, or (b) document contract as a shared primitive not counted as user-facing, and correct the PR body removal count from 18 to 17.

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