Skip to content

fix: update agent-framework import and constructor to match latest SD…#250

Open
biswapm wants to merge 1 commit intomainfrom
feature/agent-framework-latest-version-fix
Open

fix: update agent-framework import and constructor to match latest SD…#250
biswapm wants to merge 1 commit intomainfrom
feature/agent-framework-latest-version-fix

Conversation

@biswapm
Copy link
Copy Markdown
Contributor

@biswapm biswapm commented Mar 20, 2026

Summary

  • Replace deprecated ChatAgent with Agent from agent_framework
  • Rename constructor parameter chat_client to client to match latest SDK

Test plan

  • Verify agent starts successfully with updated SDK
  • Confirm message processing works end-to-end

…K API

Replace deprecated ChatAgent with Agent and rename chat_client param to client.
Copilot AI review requested due to automatic review settings March 20, 2026 14:45
@biswapm biswapm requested a review from a team as a code owner March 20, 2026 14:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the sample agent to align with the latest agent_framework SDK by switching to the new Agent API and its updated constructor signature.

Changes:

  • Replaced deprecated ChatAgent import/usage with Agent
  • Updated constructor argument from chat_client= to client=

@github-actions
Copy link
Copy Markdown

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@biswapm biswapm enabled auto-merge (squash) March 26, 2026 13:40
@biswapm biswapm disabled auto-merge March 26, 2026 13:40
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.

4 participants