refactor: move conversational utils from cli to eval module#1402
Merged
cristipufu merged 1 commit intomainfrom Mar 3, 2026
Merged
refactor: move conversational utils from cli to eval module#1402cristipufu merged 1 commit intomainfrom
cristipufu merged 1 commit intomainfrom
Conversation
Move LegacyEvalChatMessagesMapper and related models from uipath._cli._evals._conversational_utils to uipath.eval.models._conversational_utils so the eval module has no dependency on the cli namespace. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
github-actions
bot
added
test:uipath-langchain
There was a problem hiding this comment.
Pull request overview
Refactors legacy conversational-eval utilities by relocating them from the CLI namespace into the uipath.eval models layer, removing uipath.eval’s dependency on uipath._cli and updating import sites accordingly.
Changes:
- Added
uipath.eval.models._conversational_utilscontaining legacy conversational eval Pydantic models and mapping helpers. - Updated
uipath.evalcode to import legacy conversational utilities from the new module. - Updated the conversational utils test to import from the new location.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| tests/cli/eval/test_conversational_utils.py | Updates imports to reference the new eval module location. |
| src/uipath/eval/models/evaluation_set.py | Switches legacy conversational model imports to the new local module. |
| src/uipath/eval/models/_conversational_utils.py | New module defining legacy conversational schemas and message mappers. |
| src/uipath/eval/helpers.py | Switches mapper import to the new eval models module. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
radu-mocanu
approved these changes
Mar 3, 2026
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.
Description
We're planning on separating the
uipath.evalmodule into another package, we shouldn't reference CLI modules in our eval runtime.Summary
UiPathLegacyEvalChatMessagesMapperand all legacy conversational eval models fromuipath._cli._evals._conversational_utilstouipath.eval.models._conversational_utilsuipath.evalmodule's dependency on theuipath._clinamespacehelpers.py,evaluation_set.py, and test fileTest plan
🤖 Generated with Claude Code