Propagate changes from PR #124 to all templates#131
Merged
Conversation
bbqiu
force-pushed
the
bbqiu/propagate-changes
branch
4 times, most recently
from
d944136 to
aacfa56
Compare
bbqiu
force-pushed
the
bbqiu/propagate-changes
branch
from
aacfa56 to
067f881
Compare
Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com> other files Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com> . Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com> . Signed-off-by: Bryan Qiu <bryan.qiu@databricks.com>
bbqiu
force-pushed
the
bbqiu/propagate-changes
branch
from
067f881 to
b303506
Compare
dhruv0811
approved these changes
Feb 27, 2026
Contributor
dhruv0811
left a comment
dhruv0811
left a comment
There was a problem hiding this comment.
Left a minor comment, otherwise looks good! New versions working for me smoothly in local runs.
|
|
||
| @invoke() | ||
| async def invoke_handler(request: ResponsesAgentRequest) -> ResponsesAgentResponse: | ||
| workspace_client = WorkspaceClient() |
Contributor
There was a problem hiding this comment.
Do we want to wire up the new session_id field here too? Like we did for streaming?
Contributor
Author
There was a problem hiding this comment.
will include the changes in a followup pr
4 tasks
jennsun
added a commit
that referenced
this pull request
May 7, 2026
## Summary - Removes `import litellm` and `litellm.suppress_debug_info = True` from 4 agent templates that never declared `litellm` in `pyproject.toml` - Fixes `ModuleNotFoundError: No module named 'litellm'` crash on Databricks Apps deployment - Affected templates: `agent-langgraph`, `agent-openai-agents-sdk`, `agent-openai-agents-sdk-multiagent`, `agent-migration-from-model-serving` <img width="1353" height="834" alt="image" src="https://github.com/user-attachments/assets/8efc5d19-312c-4864-8a49-e899c9326653" /> due to #208 ## Context https://databricks.slack.com/archives/C065NC65Q9F/p1778114094978419 The `litellm` import was added in #131 to suppress verbose debug logging. It worked locally because `databricks-agents` pulls it in transitively, but deployed Apps environments can resolve dependencies differently, causing the import to fail on startup. The advanced templates (`agent-langgraph-advanced`, `agent-openai-advanced`) never used this import and work fine without it. ## Test plan - [x] Verified no remaining `litellm` references in any template - [x] Confirmed Python syntax validity for all 4 changed files - [x] Confirmed `agent-langgraph-advanced` and `agent-openai-advanced` (which never had litellm) work correctly as evidence this is safe to remove - [ ] Deploy affected templates to Databricks Apps and verify startup succeeds 🤖 Generated with [Claude Code](https://claude.com/claude-code)
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
2 participants
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.
changes that should be identical across templates:
a. slack threads: https://databricks.slack.com/archives/C022KUWGVQS/p1771469388988059, https://databricks.slack.com/archives/C022KUWGVQS/p1771842499541559
changes that are not identical: