Skip to content

feat: add new skills for writing typescript code for imagekit_api mcp server#2

Open
shivamik wants to merge 5 commits into
mainfrom
imagekit-api-mcp-service
Open

feat: add new skills for writing typescript code for imagekit_api mcp server#2
shivamik wants to merge 5 commits into
mainfrom
imagekit-api-mcp-service

Conversation

@shivamik

Copy link
Copy Markdown
Collaborator
  • Bump up the version
  • Add new skill for writing typescript code for imagekit_api mcp server

shivamik added 2 commits June 18, 2026 18:40
… and imagekit-plugin

chore(mcp): fix API URL in mcp configuration

docs(README): add imagekit-sdk-reference skill to documentation

chore(skills): include imagekit-sdk-reference in skills list
Comment thread skills/mcp-preflight/SKILL.md Outdated
@imagekitio

imagekitio commented Jun 21, 2026

Copy link
Copy Markdown

We need to add skill that covers advance search query syntax. Search is common use-case and without skill, it would not be able to do it right in first few attempts. Ultimately it uses dev tool search_docs and does find correct syntax but a skill could have prevented it. This skill should pick examples from docs, especially around custom metadata and their CRUD too.

On similar lines, we also need a skill for integration. We have SDK for all languages but maybe a simple skill to just cover what we have so that if anyone is integrating imagekit, agent knows what all is possible and how to build further using search_doc.

shivamik added 3 commits June 24, 2026 19:28
…e type gotchas

- Add search-assets skill: searchQuery filter syntax, operators, and field reference for client.assets.list()
- Add imagekit-integrations skill: index of ImageKit SDKs, plugins, and integrations
- Register both new skills under the General grouping in skills.sh.json
- imagekit-sdk-reference: add TypeScript Gotchas section explaining why .filter() with `item is File` fails in MCP/Deno (global File shadows SDK File); recommend for...of + if narrowing; document nullable props, .find() undefined, and webhook discrimination
- mcp-preflight: clarify execute tool description and upload routing wording
file.AITags?.map(...); // ✅
```

### Webhook event discrimination

@manu4543 manu4543 Jun 25, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Not a blocker, just curious, why would mcp ever need webhook related stuff?

|------|----------|
| `search_doc` | Local search that only searches TypeScript SDK code — use to find method signatures and types |
| `execute` | LLM writes TypeScript code and the server executes it. **All operations go through this tool.** |
| `execute` | Executes TypeScript code against the ImageKit SDK to perform CRUD operations |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

In most of the gotach, it is due to incorrect types, right? Can't LLM just provide javascript code and can't this execute run that. What is the benefit of TS code here?

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