Skip to content

fix: metadata and documentation accuracy — 5 quick wins#146

Merged
dean0x merged 1 commit intomainfrom
fix/quick-wins-metadata-batch
Mar 15, 2026
Merged

fix: metadata and documentation accuracy — 5 quick wins#146
dean0x merged 1 commit intomainfrom
fix/quick-wins-metadata-batch

Conversation

@dean0x
Copy link
Owner

@dean0x dean0x commented Mar 15, 2026

Summary

Five mechanical fixes from code review — metadata, frontmatter, and documentation accuracy only. No behavioral changes.

  • H3: Add missing YAML frontmatter to audit-claude.md command
  • H4: Fix subagent_type from "general-purpose" to "Synthesizer" in debug.md
  • H6: Correct plugin count from "9 core + 8 optional" to "8 core + 9 optional" in CLAUDE.md (2 locations)
  • M2: Add skills: frontmatter to Skimmer (knowledge-persistence) and Simplifier (core-patterns) agents
  • M8: Catalog 3 missing skills (ambient-router, search-first, test-driven-development) in skills-architecture.md

Verification

  • npm run build passes (45 skill copies, 20 agent copies, 17 plugins)
  • npm test passes (250/250 tests)
  • grep -r "general-purpose" plugins/ → 0 hits
  • grep "9 core" CLAUDE.md → 0 hits

Test plan

  • Build passes
  • All 250 tests pass
  • No behavioral changes — metadata/docs only

- Add missing YAML frontmatter to audit-claude command (H3)
- Fix subagent_type from "general-purpose" to "Synthesizer" in debug.md (H4)
- Correct plugin count from "9 core + 8 optional" to "8 core + 9 optional" in CLAUDE.md (H6)
- Add skills frontmatter to Skimmer (knowledge-persistence) and Simplifier (core-patterns) agents (M2)
- Catalog ambient-router, search-first, and test-driven-development in skills-architecture.md (M8)
@dean0x dean0x merged commit 3c169c0 into main Mar 15, 2026
4 checks passed
@dean0x dean0x deleted the fix/quick-wins-metadata-batch branch March 15, 2026 16:20
dean0x added a commit that referenced this pull request Mar 15, 2026
## Summary

Follow-up to quick wins batch (#146). Fixes 4 non-behavioral
metadata/documentation inconsistencies:

- **Remove `name=` from debug.md Task() calls** — non-standard parameter
not used by any other command
- **Add `synthesizer` to debug plugin agent declarations** — debug.md
spawns Synthesizer in Phase 4 but didn't declare it in plugin.json or
plugins.ts
- **Fix README skill count** — 31 → 32 (9 core + 8 optional + 15
specialized = 32)
- **Align plugins.ts descriptions with marketplace.json** — 7 plugin
descriptions now match their marketplace counterparts

## Test plan

- [x] `npm run build` passes (synthesizer now distributed to debug
plugin: 2 agents)
- [x] `npm test` — 250/250 tests pass, no regressions
- [x] `grep 'name=' plugins/devflow-debug/commands/debug.md` returns no
matches

Co-authored-by: Dean Sharon <deanshrn@gmain.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.

1 participant