Refine MCP skills: align search-assets/sdk-reference with assets.list overload; drop webhooks#3
Open
imagekitio wants to merge 11 commits into
Open
Refine MCP skills: align search-assets/sdk-reference with assets.list overload; drop webhooks#3imagekitio wants to merge 11 commits into
imagekitio wants to merge 11 commits into
Conversation
… 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
…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
…ebhooks from MCP skill
…atch only to branch + re-throw)
…archQuery), not 'always searchQuery'
Collaborator
manu4543
reviewed
Jun 26, 2026
Collaborator
shivamik
reviewed
Jun 29, 2026
…and disambiguation" This reverts commit 1ffc9ce.
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.



Summary
Builds on #2 and refines the two skills that interact most with TypeScript typing, so the agent gets correct results even when a skill isn't loaded. Pairs with companion SDK changes in
imagekit-nodejs(an honestassets.listoverload + an expandedexecutetool prompt).search-assetsassets.listoverload behavior:searchQuery→ pass{ type: 'file' }/{ type: 'folder' }for a typedFile[]/Folder[](no narrowing).searchQuery→ the API ignores thetype/tags/nameparams, so the result stays(File | Folder)[]; puttype = "file"inside the query and narrow withfor...of+if.client.customMetadataFields.list()to get valid field names/types before building"customMetadata.<field>"queries (operators depend on the field type).searchQueryoverrides the top-leveltype/tags/nameparams.imagekit-sdk-referenceassets.listnarrowing guidance to reflect the overload (narrowing only required for the union /searchQuerycases).executecode; when branching on failure, duck-type the error instead ofinstanceof ImageKit.APIError(a value import of the SDK crashes at runtime in the Deno sandbox).executesandbox.Companion SDK changes (separate
imagekit-nodejsPR)Companion PR - imagekit-developer/imagekit-nodejs#163
assets.listoverloads gated onsearchQuery?: undefinedso the typedFile[]/Folder[]return is only produced when there's nosearchQuery(no more type lie).executetool prompt expanded with list/search + pagination + nullable-field + custom-metadata-discovery guidance.