Skip to content

AutoGen Studio: Serve responding complete messages #6520

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 6 commits into
base: main
Choose a base branch
from

Conversation

mariohmol
Copy link

@mariohmol mariohmol commented May 13, 2025

Why are these changes needed?

Run the Studio with serve:
autogenstudio serve --team notebooks/team.json --port 8084

Open the doc endpoint (http://127.0.0.1:8084/docs) and make a request, you will notice the response does not show content for messages.

With this fix you will see full payload:
Screenshot from 2025-05-13 02-35-54

This error might be something related to FastAPI vs pydantic, as we are using some "complex" model structuring .
TaskResult -> BaseChatMessage(BaseMessage, ABC) -> BaseTextChatMessage(BaseChatMessage, ABC) -> TextMessage(BaseTextChatMessage)

Probably the serializer use its parent to serialize, and then can't access extension attributes declared by children classes.

PS: We have /docs page with the available endpoints, but it is not mentioned on our documentation or we could show on the Studio Deploy view. Seems that the documentation is out of date as many times it says --workflow but now is --team

Related issue number

Opened a discussion on discord.

Checks

@mariohmol
Copy link
Author

@microsoft-github-policy-service agree

@victordibia victordibia self-assigned this May 30, 2025
Copy link

codecov bot commented Jun 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.69%. Comparing base (9065c6f) to head (469de84).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6520   +/-   ##
=======================================
  Coverage   79.69%   79.69%           
=======================================
  Files         228      228           
  Lines       17025    17025           
=======================================
  Hits        13568    13568           
  Misses       3457     3457           
Flag Coverage Δ
unittests 79.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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