Add modulemapper skill#110
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! The skill concept is solid and the SKILL.md content/description looks great. A few things to fix before we can merge:
1. Remove modulemapper.skill
The binary .skill file isn't how skills work — no compiled artifact needed. The correct format is just a folder with a SKILL.md inside. Please delete skills/modulemapper-skill/modulemapper.skill.
2. Move SKILL.md to the right location
Currently nested at skills/modulemapper-skill/modulemapper/SKILL.md — should be flat at skills/modulemapper-skill/SKILL.md.
3. Fix the README install instructions
The README tells users to download modulemapper.skill and upload it to Claude.ai → Settings → Skills. That's not how this works. Please update the install instructions to reflect the correct folder-based setup.
4. Migrate the app from raw fetch() to the TinyFish SDK
modulemapper/src/app/api/scrape/route.ts is calling the TinyFish endpoint directly via fetch(). All cookbook submissions need to use the official SDK instead:
npm install @tiny-fish/sdkimport { TinyFish } from "@tiny-fish/sdk";
const client = new TinyFish();
const stream = await client.agent.stream({ url, goal });Quick fixes and we're good to go!
|
@simantak-dabhade made the changes |
No description provided.