Skip to content

fix: Merging dev to main for the demo env tool string length issue#412

Merged
Roopan-Microsoft merged 4 commits into
mainfrom
dev
Aug 28, 2025
Merged

fix: Merging dev to main for the demo env tool string length issue#412
Roopan-Microsoft merged 4 commits into
mainfrom
dev

Conversation

@Dhruvkumar-Microsoft

Copy link
Copy Markdown
Contributor

Purpose

This pull request primarily removes or comments out a large number of HR and marketing tool functions from the backend, likely to reduce available functionality or for refactoring purposes. Additionally, it standardizes the JSON output format in tool documentation by removing indentation. There are also minor changes to how tool lists are handled in the planner agent and template.

The most important changes are:

Removal/commenting out of tool functions

  • Most HR-related functions in hr_tools.py have been commented out, including processing leave requests, updating policies, conducting exit interviews, scheduling reviews, approving expenses, managing transfers, tracking attendance, and more. This significantly reduces the available HR automation capabilities. [1] [2] [3] [4]
  • Several marketing tool functions in marketing_tools.py have been commented out, such as conducting customer surveys, scheduling marketing events, tracking campaign performance, developing brand strategies, creating content calendars, and updating website content. [1] [2]

JSON output formatting

  • The JSON output for tool documentation in both generic_tools.py and hr_tools.py is now generated without indentation, resulting in a more compact format. [1] [2]

Planner agent changes

  • In planner_agent.py, the tools_list is now passed directly (as a list) instead of being stringified, which may affect template rendering or downstream processing.
  • A line in the planner agent's template that instructed the agent to ask the user for additional information in the first step has been removed, potentially altering the initial planning behavior.

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

Dhruvkumar-Microsoft and others added 4 commits August 20, 2025 23:46
fix: Removed the str conversion of the kernel function tools and renoved kernel functions for consistent response
@Roopan-Microsoft Roopan-Microsoft merged commit 79aaf3e into main Aug 28, 2025
17 checks passed
@github-actions

Copy link
Copy Markdown

🎉 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: Merging dev to main for the demo env tool string length issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants