feat: improve skill scores across all five skills#1
Open
rohan-tessl wants to merge 1 commit intoRewriteToday:mainfrom
Open
feat: improve skill scores across all five skills#1rohan-tessl wants to merge 1 commit intoRewriteToday:mainfrom
rohan-tessl wants to merge 1 commit intoRewriteToday:mainfrom
Conversation
Hey @RewriteToday 👋 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 | |-------|--------|-------|--------| | agent-sms-inbox | 67% | 87% | +20% | | sms-best-practices | 86% | 97% | +11% | | rewrite | 89% | 95% | +6% | | send-sms | 94% | 100% | +6% | | rewrite-inbound | 88% | 89% | +1% | All five skills in the repo were reviewed and improved — this PR is intentionally scoped to keep it reviewable. Changes: - agent-sms-inbox: Added natural trigger terms, validation checkpoints, inline policy gate example, merged redundant sections - sms-best-practices: Added webhook signature verification example, explicit retry workflow - rewrite: Added Quick Start curl example, tightened Platform Reality Check, added validation and error guidance - send-sms: Added Quick Example curl command, concrete JSON output contract - rewrite-inbound: Added inline signature verification example, concrete JSON output contract
|
@rohan-tessl your agent messed up the PR. It created two PR's. One at the right repo and one in a foreign repo. 😅 |
Member
|
@rohan-tessl thanks for the PR!! A few fixes are needed before this is ready to merge:
The direction is good overall, especially the added quick-starts and the tighter guidance around retries/policy gates. After those fixes, this should be in much better shape to merge. |
almeidazs
requested changes
Apr 6, 2026
Member
almeidazs
left a comment
There was a problem hiding this comment.
Thanks for the PR. Before this can be merged, a few changes need to be addressed:
- Fix the new REST example in
send-sms/SKILL.md. It currently useshttps://api.rewrite.com/messages, but the rest of the repo documentshttps://api.rewritetoday.com/v1/messages. - Align the normalized output example in
rewrite-inbound/SKILL.mdwithrewrite-inbound/references/payload-contract.md. Right now the skill introduces a different shape (webhookEventId,rewriteMessageId, nestedcontact) than the reference contract (eventId,messageId,contact,contactId,analysis,raw). - Minor consistency issue:
agent-sms-inbox/SKILL.mdpoints toreferences/installation.mdfor SDK types, but that file does not actually define the payload/schema types.
The overall direction is useful, especially the added quick starts and more concrete retry/policy guidance. After those are fixed, this should be ready for another pass.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hey @cpoepke 👋
I ran your skills through
tessl skill reviewat work and found some targeted improvements.Here's the full before/after:
This PR is intentionally scoped to 5 skills to keep it reviewable. The remaining 8 skills (louder, mute, personality, quieter, stop, teammate, unmute, volume) already score well (71–86%) and can be improved in follow-ups or via automated review on future PRs.
Changes summary
All 5 skills:
config (55% → 99%):
claude-talk config KEY)say -v '?'withclaude-talk tts voicesfor consistencyhelp (66% → 95%):
lsofcommand to troubleshootingchat (70% → 94%):
start (68% → 90%):
install (68% → 81%):
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 🙏