Skip to content

Conversation

@Roopan-Microsoft
Copy link
Collaborator

Purpose

  • This pull request includes several updates to the initialization logic across multiple agent classes and renames various kernel tool methods to improve clarity and consistency. Additionally, there are updates to the get_all_kernel_functions method to return a dictionary instead of a list, enhancing its usability.

Initialization Logic Updates:

  • Updated the __init__ method in multiple agent classes (generic_agent.py, hr_agent.py, human_agent.py, marketing_agent.py, procurement_agent.py, product_agent.py, tech_support_agent.py) to check for tools using if not tools instead of if tools is None. This ensures compatibility with more cases where tools might be empty. [1] [2] [3] [4] [5] [6] [7]

Kernel Tool Method Renaming:

  • Renamed kernel tool methods across various files to improve clarity and align with naming conventions. Examples include:
    • organize_health_and_wellness_programorganize_wellness_program in hr_tools.py.
    • generate_social_media_postsgenerate_social_posts in marketing_tools.py.
    • handle_procurement_risk_assessmentassess_procurement_risk in procurement_tools.py.
    • develop_product_training_materialcreate_training_material in product_tools.py.
    • assist_procurement_with_tech_equipmentsupport_procurement_tech in tech_support_tools.py.

Enhancements to get_all_kernel_functions:

  • Modified the get_all_kernel_functions method in marketing_tools.py to return a dictionary (dict[str, Callable]) instead of a list. This change improves the accessibility of kernel functions by providing named keys. [1] [2]

Deprecated or Removed Methods:

  • Removed the optimize_seo_strategy method from marketing_tools.py. This function is no longer included in the kernel tools.

Additional Method Updates:

  • Shortened and standardized method names for consistency, such as:
    • manage_customer_retention_programmanage_retention_program in marketing_tools.py.
    • run_influencer_marketing_campaignrun_influencer_campaign in marketing_tools.py.
    • provide_tech_support_for_marketingassist_marketing_tech in tech_support_tools.py.

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

@Fr4nc3 Fr4nc3 merged commit 4f60775 into feature/azd-semantickernel Apr 30, 2025
1 of 2 checks passed
@github-actions
Copy link

github-actions bot commented May 2, 2025

🎉 This PR is included in version 1.3.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

blessing-msft pushed a commit that referenced this pull request Aug 1, 2025
fix: initialization logic across multiple agent classes and renames various kernel tool methods
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