Repository navigation
Python: feat: Add ChatKit integration with a sample application - #1273
Merged
Merged
Conversation
- Created a new frontend application using React and Vite for the ChatKit integration. - Added essential files including package.json, vite.config.ts, and Tailwind CSS configuration. - Implemented core components: App, Home, ChatKitPanel, ThemeToggle, and hooks for color scheme management. - Established SQLite-based store implementation for ChatKit data persistence in store.py. - Integrated theme toggling functionality for light and dark modes. - Set up ESLint and TypeScript configurations for better development experience.
Eduard van Valkenburg (eavanvalkenburg)
left a comment
Member
There was a problem hiding this comment.
Love it! But I'm a bit concerned about how much code there is in package vs sample, is there a way we can make that easier? and how will we maintain that front-end code?
Contributor
Author
|
Waiting for fix to address 3.10 compatibility: openai/chatkit-python#12 |
Eric Zhu (ekzhu)
marked this pull request as ready for review
October 17, 2025 17:42
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new agent-framework-chatkit subpackage that enables integration between Microsoft Agent Framework and OpenAI ChatKit for frontend UI. The integration provides helper utilities to convert data types between the two frameworks and includes a complete end-to-end weather assistant sample application.
Key changes:
- New Python package for ChatKit integration with utilities for streaming responses and type conversion
- Complete sample application with FastAPI backend and React frontend
- Updated pre-commit configuration to handle frontend JSON files
Reviewed Changes
Copilot reviewed 24 out of 28 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| python/packages/chatkit/agent_framework_chatkit/ | Core integration package with converter and streaming utilities |
| python/packages/chatkit/tests/ | Unit tests for the ChatKit integration functionality |
| python/samples/chatkit-integration/ | Complete weather assistant sample with backend and frontend |
| python/packages/core/agent_framework/chatkit/ | Lazy loading module for ChatKit integration |
| python/.pre-commit-config.yaml | Updated to exclude frontend JSON files from checks |
Files not reviewed (1)
- python/samples/chatkit-integration/frontend/package-lock.json: Language not supported
Contributor
Python Test Coverage Report •
Python Unit Test Overview
|
|||||||||||||||||||||||||||||||||||
Tao Chen (TaoChenOSU)
approved these changes
Nov 4, 2025
Co-authored-by: Evan Mattson <35585003+moonbox3@users.noreply.github.com>
Evan Mattson (moonbox3)
approved these changes
Nov 5, 2025
Aris Nguyen (arisng)
pushed a commit
to arisng/agent-framework
that referenced
this pull request
Feb 2, 2026
…osoft#1273) * feat: Add ChatKit integration with a new frontend application - Created a new frontend application using React and Vite for the ChatKit integration. - Added essential files including package.json, vite.config.ts, and Tailwind CSS configuration. - Implemented core components: App, Home, ChatKitPanel, ThemeToggle, and hooks for color scheme management. - Established SQLite-based store implementation for ChatKit data persistence in store.py. - Integrated theme toggling functionality for light and dark modes. - Set up ESLint and TypeScript configurations for better development experience. * git ignore * fix mypy * add mising file * minimal frontend for chatkit sample * update ignore files * version * set python version lowerbound on chatkit * update project settings for chatkit * update setup * update setup * update setup * update setup * weather widget * add select city widget sample * remove widget helper * update chatkit to include file attachments and cover more thread item types * update readme with mermaid diagram * update diagram * update instructions * update chatkit dependency * fix converter imports * move to demos/ * move to demos/ -- rename references * support multiple session instead of using global variable in sample * support chunk streaming * fix tests * Update python/samples/demos/chatkit-integration/store.py Co-authored-by: Evan Mattson <35585003+moonbox3@users.noreply.github.com> * use local host --------- Co-authored-by: Evan Mattson <35585003+moonbox3@users.noreply.github.com>
4 of 7 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation and Context
Introduce
agent-framework-chatkitsubpackage for integration with ChatKit for frontend UI.Description
stream_agent_response,ThreadItemConverterandsimple_to_agent_inputto convert data types between ChatKit and Agent FrameworkContribution Checklist