Skip to content

fix: Removed the str conversion of the kernel function tools and renoved kernel functions for consistent response #389

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

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Dhruvkumar-Microsoft
Copy link
Contributor

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:

  • Commented out many HR-related kernel functions in 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:

  • Commented out several marketing-related kernel functions in 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

  • Changed the JSON output for tool documentation in both hr_tools.py and generic_tools.py to remove pretty-printing (indentation), resulting in a more compact JSON string. [1] [2]

Planner Agent Argument Generation

  • Modified the _generate_args method in planner_agent.py to use the raw tools_list instead of its string representation, potentially affecting how tool arguments are passed or logged.

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

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.

2 participants