Add tutor-finder-skill to tinyfish-cookbook #121
Add tutor-finder-skill to tinyfish-cookbook #121KrishnaAgarwal7531 wants to merge 4 commits intotinyfish-io:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
simantak-dabhade
left a comment
There was a problem hiding this comment.
Hey @KrishnaAgarwal7531! Same two fixes as the other PRs:
1. Remove tutor-finder.skill
The binary .skill file isn't needed — just a folder with SKILL.md is the correct format. Please delete skills/tutor-finder/tutor-finder.skill.
2. Fix the folder nesting
Current: skills/tutor-finder/tutor-finder/SKILL.md
Should be: skills/tutor-finder/SKILL.md
Solid skill overall — good platform selection logic and dedup across platforms. Quick fixes and we're good!
|
@simantak-dabhade made the changes |
simantak-dabhade
left a comment
There was a problem hiding this comment.
Hey! The SKILL.md itself is great — well-structured, correct parallel execution pattern, good edge case handling.
One thing still needs to be fixed:
- Remove the binary
.skillfile (skills/tutor-finder/tutor-finder.skill) — this shouldn't be in the repo. Only theSKILL.mdis needed.
Also minor:
2. The skill writes to /tmp/tf_*.json but never cleans up. Add a rm /tmp/tf_*.json 2>/dev/null at the end.
Once that binary file is removed, this is good to go!
No description provided.