Data tool consolidation #65
Merged
Conversation
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
docs: Add Designer MCP Installation Instructions
docs: update setup instructions for Webflow MCP Bridge App
revise Cursor config for official remote MCP support
Duplication instruction removed
Local Installation Documentation
Added TypeScript types for text and image content, along with a unified ToolResponse type. Introduced utility functions for creating text and image content, and for formatting tool responses. This improves type safety and clarity in handling different response formats.
zmcnellis
approved these changes
Oct 16, 2025
| ); | ||
| return formatResponse(response); | ||
| for (const action of actions) { | ||
| if (action.list_components) { |
There was a problem hiding this comment.
might be cleaner as a switch statement?
There was a problem hiding this comment.
Thanks for pointing that out. A switch-case isn’t ideal for this pattern since it can’t handle multiple operations per action. However, I’ve removed the else if chain that was preventing other properties from being checked. I fixed it by switching to independent if statements instead, this way, all properties are properly evaluated.
src/tools/components.ts
Outdated
| ComponentPropertyUpdateSchema | ||
| } from "../schemas"; | ||
| import { formatErrorResponse, formatResponse } from "../utils"; | ||
| import { Content, formatErrorResponse, textContent, toolResponse } from "../utils"; |
There was a problem hiding this comment.
Suggested change
| import { Content, formatErrorResponse, textContent, toolResponse } from "../utils"; | |
| import { type Content, formatErrorResponse, textContent, toolResponse } from "../utils"; |
Updated the CMS tool to enhance TypeScript type definitions for function arguments and responses, ensuring better clarity and maintainability. Consolidated import statements and improved formatting for better readability. This refactor aligns with recent updates across other tools for a unified codebase.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
Similar to DE Tools, this PR introduces updates to Data Tools. Instead of maintaining separate tools for each Data API, we’ve consolidated them by type. For example, the site tool can now be used to list sites, publish, or get site details.