Skip to content

improve tdd skill structure - #12

Closed
fernandezbaptiste wants to merge 1 commit into
smartfrog:mainfrom
fernandezbaptiste:improve-tdd-skill
Closed

improve tdd skill structure#12
fernandezbaptiste wants to merge 1 commit into
smartfrog:mainfrom
fernandezbaptiste:improve-tdd-skill

Conversation

@fernandezbaptiste

@fernandezbaptiste fernandezbaptiste commented Mar 27, 2026

Copy link
Copy Markdown

hey @smartfrog, thanks for building opencode-froggy. really like the breadth of agent types covering the full dev workflow, from rubber-duck debugging to architecture review. Kudos on soon reaching 60 stars! I've just starred it.

ran your tdd skill through agent evals and spotted a few quick wins that took it from ~67% to ~94% performance:

  • expanded description with trigger terms like write tests, fix a bug, red-green-refactor, test coverage

  • restructured protocol into numbered process with verify checkpoints + converted test level guidance to decision tables

  • added error recovery table for 5 common failure scenarios

also added a GitHub Action that auto-reviews any skill.md changed in a PR (includes min permissions, uses a pinned action version, only posts a review comment).

this means that it gives you and your contributors an instant quality signal before you have to review yourself (no signup, no tokens needed).

optionally, you can enable optimize mode by adding a token as TESSL_API_TOKEN in your repo secrets - when enabled, the action suggests improvements you can accept by commenting /apply-optimize.

these were easy changes to bring the skill in line with what performs well against Anthropic's best practices. honest disclosure, I work at tessl.io where we build tooling around this. not a pitch, just fixes that were straightforward to make! happy to answer any questions on the changes.

- expand frontmatter description with trigger terms
- restructure protocol into numbered process with verify checkpoints
- convert test level guidance and quality rules to decision tables
- add error recovery table for common failure scenarios
- add .github/workflows/skill-review.yml for automated skill review + optimize on PRs
- add .github/workflows/skill-apply-optimize.yml for /apply-optimize command
@fdematos

fdematos commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Thanks for the contribution @fernandezbaptiste . I’m not comfortable merging the workflow additions.
They introduce a third-party action with write permissions, a comment-triggered apply flow, and fork-PR assumptions that do not look safe or reliable for this repository.

Please split the PR and keep only the skill/tdd/SKILL.md changes if you want the content update reviewed independently.

@fdematos fdematos closed this Apr 8, 2026
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.

2 participants