Skip to content

Conversation

@Roopan-Microsoft
Copy link
Collaborator

Purpose

  • This pull request primarily removes or comments out a large number of HR and marketing-related kernel functions, effectively disabling many previously available operations in hr_tools.py and marketing_tools.py. Additionally, it changes the JSON output formatting for tool documentation generation to be more compact (removing indentation). There are also minor adjustments to how tool lists are handled in the planner agent.

Major removals and changes:

HR and Marketing Kernel Functions

  • Commented out a significant number of @kernel_function-decorated methods in hr_tools.py, disabling features such as processing leave requests, updating policies, scheduling reviews, managing transfers, handling overtime, issuing bonuses, and more. This reduces the set of available HR automation tools. [1] [2] [3] [4]
  • Commented out several marketing-related kernel functions in marketing_tools.py, such as conducting customer surveys, scheduling marketing events, tracking campaign performance, developing brand strategies, creating content calendars, and updating website content. [1] [2]

Tool Documentation Output

  • Modified the generate_tools_json_doc methods in both generic_tools.py and hr_tools.py to output compact JSON (no indentation), making the output less human-readable but more space-efficient. [1] [2]

Planner Agent Adjustments

  • Changed the _generate_args method in planner_agent.py to use the tools list directly instead of its string representation, likely to improve downstream processing.
  • Removed a line from the _get_template function in planner_agent.py that instructed the agent to ask the user for additional information as the first step, possibly altering initial agent 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants