Skip to content

Improve/skill review optimization - #5

Open
yogesh-tessl wants to merge 2 commits into
1jehuang:mainfrom
yogesh-tessl:improve/skill-review-optimization
Open

Improve/skill review optimization#5
yogesh-tessl wants to merge 2 commits into
1jehuang:mainfrom
yogesh-tessl:improve/skill-review-optimization

Conversation

@yogesh-tessl

Copy link
Copy Markdown

Hey @1jehuang 👋

A Rust native host at 830KB bridging WebSocket to Firefox's native messaging, with 12ms per-command execution and E2E benchmarks against Claude Opus. This is serious agent infrastructure. The fact that you've documented the SKILL.md sync problem in CLAUDE.md (three copies that need to stay aligned) tells me you care about skill quality as much as runtime performance. I'd like to contribute some improvements to the skill files.

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

score_card

Changes summary:

Conciseness improvements (2/3 → 3/3)

  • Trimmed all three SKILL.md files significantly (root: 552→181 lines, skill/: 423→118 lines, rust-cli/resources/: 452→121 lines)
  • Moved detailed examples and setup docs into companion REFERENCE.md files alongside each SKILL.md
  • Condensed Rich Text Editors explanation to inline notes
  • Kept all action reference tables compact and complete

Progressive disclosure improvements (2/3 → 3/3)

  • Created REFERENCE.md files for each skill location containing detailed examples for: fork, parallel, tryUntil, authentication setup, evaluate/pageWorld, scroll, form state, isolated sessions, and iframe handling
  • Main SKILL.md files now serve as concise, self-contained references with a single link to REFERENCE.md for advanced details

Workflow clarity improvements

  • Added validation checkpoints to the Recommended Workflow (e.g., re-run getContent to verify state after interactions and confirm login succeeded)

Frontmatter cleanup

  • Converted description fields to standard quoted string format

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.

I also added a lightweight 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).

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 🙏

Hey @1jehuang 👋

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 |
|-------|--------|-------|--------|
| firefox-browser (root SKILL.md) | 89% | 99% | +10% |
| firefox-browser (skill/SKILL.md) | 90% | 99% | +9% |
| firefox-browser (rust-cli/resources/SKILL.md) | 90% | 99% | +9% |

Changes summary:

Conciseness improvements (2/3 → 3/3)
- Trimmed all three SKILL.md files significantly (root: 552→181 lines, skill/: 423→118 lines, rust-cli/resources/: 452→121 lines)
- Moved detailed examples and setup docs into companion REFERENCE.md files alongside each SKILL.md
- Condensed Rich Text Editors explanation to inline notes
- Kept all action reference tables compact and complete

Progressive disclosure improvements (2/3 → 3/3)
- Created REFERENCE.md files for each skill location containing detailed examples for: fork, parallel, tryUntil, authentication setup, evaluate/pageWorld, scroll, form state, isolated sessions, and iframe handling
- Main SKILL.md files now serve as concise, self-contained references with a single link to REFERENCE.md for advanced details

Workflow clarity improvements
- Added validation checkpoints to the Recommended Workflow (e.g., re-run getContent to verify state after interactions and confirm login succeeded)

Frontmatter cleanup
- Converted description fields to standard quoted string format

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 https://docs.tessl.io/evaluate/optimize-a-skill-using-best-practices and ask it to optimize your skill. Ping me - @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