fix: Removed the str conversion of the kernel function tools and renoved kernel functions for consistent response#389
Merged
Merged
Conversation
…ernel functions for consistent response
Fr4nc3
approved these changes
Aug 21, 2025
Roopan-Microsoft
approved these changes
Aug 27, 2025
|
🎉 This PR is included in version 2.2.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
deepammi
pushed a commit
to deepammi/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator
that referenced
this pull request
Dec 7, 2025
fix: Removed the str conversion of the kernel function tools and renoved kernel functions for consistent response
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.
Purpose
This pull request primarily removes (comments out) a large number of HR and marketing tool kernel functions, reducing the available functionality in those modules. Additionally, it changes the JSON output formatting for tool documentation in both HR and generic tools modules, and slightly modifies how tool arguments are generated in the planner agent.
The most important changes are:
Removal (Commenting Out) of Kernel Functions
HR Tools:
hr_tools.py, including those for processing leave requests, updating policies, conducting exit interviews, scheduling performance reviews, approving expense claims, sending company announcements, managing employee transfers, tracking attendance, organizing wellness programs, managing retirement plans, handling overtime requests, issuing bonuses, scheduling wellness checks, handling employee suggestions, and updating employee privileges. This significantly reduces the HR automation capabilities. [1] [2] [3] [4]Marketing Tools:
marketing_tools.py, such as conducting customer surveys, scheduling marketing events, tracking campaign performance, developing brand strategy, creating content calendars, updating website content, conducting market research, handling customer feedback, and managing social media accounts. This reduces the available marketing automation features. [1] [2] [3]Changes to Tool Documentation Output
hr_tools.pyandgeneric_tools.pyto remove pretty-printing (indentation), resulting in a more compact JSON string. [1] [2]Planner Agent Argument Generation
_generate_argsmethod inplanner_agent.pyto use the rawtools_listinstead of its string representation, potentially affecting how tool arguments are passed or logged.Does this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information