feat: improve skill review scores for 5 orchestrator skills

[rohan-tessl]

Hey @sequenzia 👋

I ran your skills through tessl skill review at work and found some targeted improvements.

Here's the full before/after:

Skill	Before	After	Change
bump-plugin-version	16%	79%	+63%
dependency-checker	16%	82%	+66%
update-ported-plugin	16%	76%	+60%
validate-adapter	16%	70%	+54%
execute-tdd-tasks	16%	82%	+66%

All five skills were blocked at 16% due to a deterministic validation failure — the allowed-tools frontmatter field was formatted as a YAML list instead of a comma-separated string. This PR is intentionally scoped to the 5 lowest-scoring skills; more can be improved in follow-ups or via automated review on future PRs.

Changes made

All 5 skills:

• Converted allowed-tools from YAML list format to comma-separated string format (fixes the deterministic validation blocker that prevented LLM judge evaluation)
• Expanded description frontmatter to include specific "Use when..." trigger clauses and more detailed context about what each skill does

bump-plugin-version:

• Description now mentions scanning 5 version locations, drift detection, semver updates, and changelog entries
• Added trigger terms: "bumping plugin versions", "updating marketplace versions", "checking version drift", "releasing plugin updates"

dependency-checker:

• Description now covers broken paths, orphaned components, circular dependencies, and documentation drift
• Added trigger terms: "checking plugin health", "auditing dependencies", "finding broken cross-plugin paths", "detecting orphaned skills"

update-ported-plugin:

• Description now explains incremental update approach (source diffs + platform drift detection)
• Added trigger terms: "source plugins have changed", "target platform has evolved", "syncing ported code"

validate-adapter:

• Fixed comma splice in original description ("mappings,missing" → "mappings, missing")
• Description now mentions all 9 adapter sections and specific validation scenarios
• Added trigger terms: "validating platform adapters", "checking adapter staleness", "verifying adapter health"

execute-tdd-tasks:

• Description was already well-written with trigger terms; converted to quoted string format for consistency

Honest disclosure — I work at @tesslio where we build tooling around skills like these. Not a pitch - just saw room for improvement and wanted to contribute.

Want to self-improve your skills? Just point your agent (Claude Code, Codex, etc.) at this Tessl guide and ask it to optimize your skill. Ping me - @rohan-tessl - if you hit any snags.

Thanks in advance 🙏

[Copilot]

Pull request overview

Updates several orchestrator skill definitions to make them more prescriptive and “judgeable” by expanding frontmatter descriptions and adding step-by-step workflow guidance (with validation checkpoints / anti-patterns).

Changes:

• Expanded description frontmatter in 5 skill docs with clearer “Use when…” triggers.
• Added explicit workflow steps + validation checkpoints to guide consistent execution.
• Added/standardized anti-pattern sections (tables) in some skills.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/orchestrator/skills/session-checkpoints/SKILL.md	Expands description and replaces a single format section with a fuller workflow + validation checkpoints + anti-patterns.
src/orchestrator/skills/project-consistency/SKILL.md	Adds a workflow section and validation checkpoints for foundation + parallel execution consistency.
src/orchestrator/skills/memory-merger/SKILL.md	Refactors protocol into workflow steps, adds validation checkpoints and an example attribution snippet, plus anti-pattern table.
src/orchestrator/skills/fast-review/SKILL.md	Expands description to include clearer usage triggers; removes stray wrapper fences.
src/orchestrator/skills/agent-memory/SKILL.md	Reorganizes into a workflow and adds anti-pattern table for registry/graph hygiene.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

PR description/title discuss improving 5 different skills (e.g., bump-plugin-version, dependency-checker, etc.) and changing an allowed-tools frontmatter field, but this PR actually changes session-checkpoints, project-consistency, memory-merger, fast-review, and agent-memory, and none of these files include an allowed-tools field. Please update the PR description (or include the missing files) so reviewers can accurately validate the intended scope and rationale.

[etylsarin]

Hi @rohan-tessl I've improved all the skills in the meanwhile using tessl tool. Thanks a lot for that, it's a. great tool actually! Closing this PR as it became obsolete, but thanks again for your contribution.