Skip to content

Add modulemapper skill#110

Open
KrishnaAgarwal7531 wants to merge 12 commits intotinyfish-io:mainfrom
KrishnaAgarwal7531:add-modulemapper-skill
Open

Add modulemapper skill#110
KrishnaAgarwal7531 wants to merge 12 commits intotinyfish-io:mainfrom
KrishnaAgarwal7531:add-modulemapper-skill

Conversation

@KrishnaAgarwal7531
Copy link
Copy Markdown
Contributor

No description provided.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 4, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 315260e0-a246-4cdf-a24b-faade7b4f69c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@simantak-dabhade simantak-dabhade left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/sdk
import { 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!

@KrishnaAgarwal7531
Copy link
Copy Markdown
Contributor Author

@simantak-dabhade made the changes

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.

3 participants