-
Notifications
You must be signed in to change notification settings - Fork 44
remove apps dangeroues tools #135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
gregra81
merged 2 commits into
feat/ori/add-monday-apps-basic-tools
from
greg/remove_dangeroues_tools
Nov 6, 2025
Merged
remove apps dangeroues tools #135
gregra81
merged 2 commits into
feat/ori/add-monday-apps-basic-tools
from
greg/remove_dangeroues_tools
Nov 6, 2025
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
oricho123
approved these changes
Nov 6, 2025
|
|
||
| const response = await this.executeApiRequest<EnvVarResponse>( | ||
| HttpMethod.DELETE, | ||
| API_ENDPOINTS.CODE.MANAGE_ENV(appId, key), |
Collaborator
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this as well
oricho123
added a commit
that referenced
this pull request
Nov 9, 2025
* Enhance agent toolkit with new Monday.com tools and update structure - Added new tools for managing Monday.com resources, including creating, deleting, and updating items and columns. - Introduced tools for fetching board schemas and user details. - Updated the toolkit to support dynamic API tools configuration. - Refactored the initialization process for better organization and error handling. - Enhanced the overall structure of the toolkit for improved maintainability. * Change index.js error * Fix type errors * Update type definition for allMondayAppsTools to accept constructor functions * Remove redundant comments * Enhance agent toolkit with new Monday.com app management tools - Introduced tools for creating, retrieving, and promoting apps and app features. - Added functionality for managing environment variables and storage data. - Updated the structure to support new app version and feature management capabilities. - Improved error handling and response formatting across new tools. * Enhance app feature and version management in agent toolkit - Improved response formatting for app feature creation and retrieval, providing detailed information about features and their states. - Updated schemas to include detailed app feature and version structures. - Enhanced error handling and response messages for app version retrieval and deployment status checks, offering clearer insights into the process. - Added detailed summaries for app versions and deployment statuses to improve user experience. * Implement base class for Monday.com app tools and improve app creation response - Introduced a new base class for Monday.com app tools to streamline API interactions and enhance code organization. - Updated the app creation response to include app ID and version for better clarity and tracking. * Bump version for agent-toolkit and monday-api-mcp to 1.1.0 - Updated version numbers in package.json for both agent-toolkit and monday-api-mcp to reflect the latest release. * Refactor tool types and introduce ToolSubType enumeration - Updated ToolType enumeration to simplify types: replaced QUERY, MUTATION, and ALL_API with API. - Introduced ToolSubType enumeration to categorize tools as READ, WRITE, or ALL_API. - Modified existing tools to utilize the new ToolSubType for better clarity and organization. - Enhanced filtering functions to accommodate the new structure, ensuring backward compatibility with existing configurations. * Update version numbers for agent-toolkit to 2.0.2 and monday-api-mcp to 1.0.3 in package.json files * Refactor Monday Apps tools to incorporate ToolSubType enumeration - Updated various Monday Apps tools to include the ToolSubType property, categorizing them as READ or WRITE. - Modified the filter function to accept MondayAppsToolType for better type safety. - Removed the deprecated base class for Monday Apps tools, streamlining the codebase. * Remove duplicate import * Add Monday Apps tools option to README.md - Introduced a new command line option `--enable-monday-apps-tools` for enabling Monday Apps tools in the documentation. - Updated the README to reflect this addition, enhancing clarity for users on available features. * Refactor tool types to remove redundant "type" field from API tools - Removed the "type" field from various tool classes, standardizing them to use the ToolType.API enumeration. - Cleaned up import statements across multiple files for better organization and clarity. - Ensured consistency in tool definitions by aligning with the new ToolSubType structure. * Refactor tool filtering location and imports - Introduced a new utility file for filtering API and Monday Apps tools, consolidating the filtering logic. - Removed the deprecated utils file for Monday Apps tools. - Updated import statements across various files to utilize the new filtering utilities. - Added BaseMondayApiTool to the exports for better accessibility. * Refactor tool initialization and filtering logic - Consolidated tool initialization by introducing a new utility function to create tool instances based on their type. - Updated the filtering logic to utilize the new `getFilteredTools` function, streamlining the process of obtaining applicable tools. - Removed deprecated filtering functions and cleaned up import statements for better organization. - Added a new types file to define `ToolsConfiguration` and `MondayAgentToolkitConfig` for improved type management. * Refactor tool types to standardize usage of ToolType enumeration - Removed the ToolSubType enumeration from various tool classes, replacing it with the ToolType enumeration for consistency. - Updated import statements to reflect the changes in tool type definitions. - Ensured all tools now utilize the ToolType for categorization, enhancing clarity and organization across the toolkit. * Refactor tool instance creation logic - Removed the handling for BaseMondayAppsTool in the createToolInstance function, simplifying the tool initialization process. - Updated the function to only create instances of BaseMondayApiTool and the default tool type, enhancing clarity in tool instantiation. * Add support for BaseMondayAppsTool in tool instance creation - Updated the createToolInstance function to handle instances of BaseMondayAppsTool, allowing for initialization with an apiToken. - Enhanced the tool instantiation logic to support additional tool types, improving flexibility in the toolkit. * Refactor tool type definitions to use ToolType enumeration - Replaced ToolSubType with ToolType in various Monday Apps tools for consistency. - Updated import statements and tool definitions across multiple files to reflect this change. - Enhanced clarity and organization in the toolkit by standardizing tool categorization. * Refactor tool instance creation and filtering logic - Updated the createToolInstance function to accept a generic Tool type, enhancing type safety and clarity. - Modified the filtering logic in getFilteredTools to use the ToolType enumeration consistently, improving the accuracy of tool filtering based on type. - Cleaned up conditional checks for filtering tools, streamlining the overall logic. * Remove 'enableMondayAppsTools' configuration and related argument from the tools configuration and args config files. * Remove unused input tool registration and related error handling methods from the MondayAgentToolkit class. * Remove unused types and configuration related to tools from the agent toolkit. * Add annotations to Monday Apps tools for improved metadata * Update dependencies in yarn.lock to latest versions, including @modelcontextprotocol/sdk, @mondaydotcomorg/api, axios, and lodash packages. Add new types for jsonwebtoken and ms, and include buffer-equal-constant-time and ecdsa-sig-formatter packages. * Refactor AppFeature type handling by removing AppFeatureType enum and changing type fields to string in app-feature-schemas. This simplifies the schema definitions and enhances flexibility in feature type representation. * Change base-monday-apps-tool file name * Update README and args.config to change Monday Apps tools argument to a mode selection. The new mode allows users to specify "api" for API tools only or "apps" for Monday Apps tools, enhancing clarity and usability. * Update README to clarify the usage of the Dynamic API Tools argument, ensuring consistency in documentation. * Refactor execution methods in Monday Apps tools to use a protected executeInternal method for improved structure and tracking. This change enhances the base tool's execution flow and allows for consistent error handling and performance tracking across all tools. * Revert "Refactor AppFeature type handling by removing AppFeatureType enum and changing type fields to string in app-feature-schemas. This simplifies the schema definitions and enhances flexibility in feature type representation." This reverts commit 749541e. * remove apps dangeroues tools (#135) * remove apps dangeroues tools * update app feature types + remove redundant apps routes (#137) * Update package versions for agent-toolkit to 2.27.0 and monday-api-mcp to 1.9.0 --------- Co-authored-by: Greg Rashkevitch <[email protected]>
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.
No description provided.