Repository navigation
AE-2513: show double-brace placeholders in trajectory prompt error - #1968
Draft
al3xanndru wants to merge 1 commit into
Draft
al3xanndru wants to merge 1 commit into
al3xanndru wants to merge 1 commit into
Conversation
…jectory prompt error
The validator requires {{ExpectedAgentBehavior}} but its message asked for
{ExpectedAgentBehavior}, which misled users fixing their prompt.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
This branch has not been deployed
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.



Problem
AE-2513: when a legacy trajectory evaluator's prompt lacks a required
placeholder, loading the evaluator fails with:
The check (
LegacyTrajectoryEvaluator.validate_prompt_placeholder) looks for{{ExpectedAgentBehavior}}withdouble braces. A user who follows the message literally and writes
{ExpectedAgentBehavior}still fails validation.The DRI triage of SRE-679533 flagged this. In that incident,
uip agent eval evaluator addhad written a prompt with{{ExpectedBehavior}}; the root-cause fix is in UiPath/cli.Change
Prompt must contain {{ExpectedAgentBehavior}} and {{AgentRunHistory}} placeholders.The validation logic is unchanged.
uipathversion bumped from 2.14.44 to 2.14.45 (pyproject.tomlanduv.lock), per repo convention. 2.14.45 isnot on PyPI yet.
Tests
test_legacy_trajectory_prompt_error_names_double_brace_placeholdersintests/evaluators/test_legacy_trajectory_evaluator.py. It is parametrized over a prompt that uses{{ExpectedBehavior}}(the CLI's old default) and a prompt missing
{{AgentRunHistory}}, and asserts that theValidationErrornames bothplaceholders with double braces. Both cases fail on the old message and pass with the fix.
pytest tests/evaluators/: 554 passed.ruff check,ruff format --checkandmypyon the changed files: clean.Risks / open questions
this repo.
AE-2513: …). If commitlint requires conventional commits, retitle it tofix(eval): … [AE-2513].Drafted by the wabbit-dri draft-fixes loop for AE-2513 (draft #1), for a human to review: not ready to merge as is.
🤖 Generated with Claude Code