Skip to content

feat: improve writing-autests skill score (38% → 92%)#383

Open
yogesh-tessl wants to merge 1 commit into
yahoo:masterfrom
yogesh-tessl:improve/skill-review-optimization
Open

feat: improve writing-autests skill score (38% → 92%)#383
yogesh-tessl wants to merge 1 commit into
yahoo:masterfrom
yogesh-tessl:improve/skill-review-optimization

Conversation

@yogesh-tessl
Copy link
Copy Markdown

@yogesh-tessl yogesh-tessl commented May 15, 2026

Hey @bneradt 👋

truly good work on this. 9 skills covering the full dev loop from planning issues to writing AuTests to building and creating PRs, that's a really complete agent setup. The AGENTS.md is well-structured too, especially the emphasis on backward compatibility and matching existing style.

ran your skills through tessl skill review at work and found some targeted improvements for the writing-autests skill. Here's the before/after:

Skill Before After Change
writing-autests 38% 92% +54%

picked writing-autests because it had the most room for improvement and it's a core skill for the project. AuTests are the backbone of your e2e testing.

Changes made to writing-autests
  • Expanded description with a "Use when..." clause listing concrete trigger scenarios (writing e2e tests, adding replay YAML, updating .test.py files, configuring gold files)
  • Added directory structure reference showing the standard <test-name>/ layout with .test.py, replay_files/, and gold/ subdirectories
  • Added a numbered workflow (6 steps: create dir → write .test.py → create replay YAML → add gold files → run → verify)
  • Added a minimal test example based on real patterns from the repo (using AddClientProcess, AddServerProcess, AddProxyProcess)
  • Added replay YAML structure example showing the meta/sessions/transactions format with esc_json encoding
  • Documented key extension APIs (AddClientProcess, AddServerProcess, AddProxyProcess, Testers.ContainsExpression, Testers.ExcludesExpression)
  • Fixed path typo (test/auteststests/autests)
  • Cross-referenced the run-autests sibling skill for running tests

quick honest disclosure. I work at https://github.com/tesslio where we build tooling around skills like these. Not a pitch, just saw room for improvement and wanted to contribute.

if you want to self-improve your skills, or define your own scenarios to pressure test, just ask your agent (Claude Code, Codex, etc.) to evaluate and optimize your skill with Tessl. Ping me @yogesh-tessl, if you hit any snags.

Hey @bneradt 👋

I ran your skills through `tessl skill review` at work and found some targeted improvements for the `writing-autests` skill. Here's the full before/after:

| Skill | Before | After | Change |
|-------|--------|-------|--------|
| writing-autests | 38% | 92% | **+54%** |
| build-pv | 52% | — | — |
| plan-issue | 53% | — | — |
| commit | 65% | — | — |
| run-autests | 65% | — | — |
| review-pr | 67% | — | — |
| apply-pr-comments | 68% | — | — |
| create-pr | 76% | — | — |
| run-unit-tests | 77% | — | — |

I picked `writing-autests` because it had the most room for improvement and it's a core skill for the project — AuTests are the backbone of your e2e testing.

<details>
<summary>Changes made to <code>writing-autests</code></summary>

- **Expanded description** with a "Use when..." clause listing concrete trigger scenarios (writing e2e tests, adding replay YAML, updating .test.py files, configuring gold files)
- **Added directory structure reference** showing the standard `<test-name>/` layout with `.test.py`, `replay_files/`, and `gold/` subdirectories
- **Added a numbered workflow** (6 steps: create dir → write .test.py → create replay YAML → add gold files → run → verify)
- **Added a minimal test example** based on real patterns from the repo (using `AddClientProcess`, `AddServerProcess`, `AddProxyProcess`)
- **Added replay YAML structure example** showing the `meta`/`sessions`/`transactions` format with `esc_json` encoding
- **Documented key extension APIs** (`AddClientProcess`, `AddServerProcess`, `AddProxyProcess`, `Testers.ContainsExpression`, `Testers.ExcludesExpression`)
- **Fixed path typo** (`test/autests` → `tests/autests`)
- **Cross-referenced** the `run-autests` sibling skill for running tests

</details>

I also stress-tested your `run-autests` skill against a few real-world task evals and it held up really well on parallel test execution with `autest.sh -j` and selective test filtering with `-f`. Kudos for that.

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](https://docs.tessl.io/evaluate/optimize-a-skill-using-best-practices) and ask it to optimize your skill. Ping me — [@yogesh-tessl](https://github.com/yogesh-tessl) — if you hit any snags.

Thanks in advance 🙏
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