Skip to content

fix: ignore unknown activity params instead of throwing - #1483

Merged
andreiancuta-uipath merged 2 commits into
mainfrom
fix/ignore-unknown-activity-params
Mar 24, 2026
Merged

andreiancuta-uipath merged 2 commits into
mainfrom
fix/ignore-unknown-activity-params

Conversation

@andreiancuta-uipath

Copy link
Copy Markdown
Contributor

This can occur when the model hallucinates an extra parameter.
Matches the behaviour of the previous runtime, which ensures existing agents won't start failing when we switch them to URT.

@github-actions github-actions Bot added the test:uipath-langchain Triggers tests in the uipath-langchain-python repository label Mar 24, 2026
@andreiancuta-uipath
andreiancuta-uipath force-pushed the fix/ignore-unknown-activity-params branch 3 times, most recently from e2f39cc to b6b05a5 Compare March 24, 2026 11:02
requests = httpx_mock.get_requests()
sent_request = requests[1]
request_json = json.loads(sent_request.content.decode())
assert "unknown_param" not in request_json

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: technically not a complete assertion, because parameters can be in headers, paths or queries

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now checking all places

@andreiancuta-uipath
andreiancuta-uipath force-pushed the fix/ignore-unknown-activity-params branch from b6b05a5 to 4fd699f Compare March 24, 2026 12:02
@andreiancuta-uipath
andreiancuta-uipath force-pushed the fix/ignore-unknown-activity-params branch from 4fd699f to 6458bb3 Compare March 24, 2026 13:57
@andreiancuta-uipath
andreiancuta-uipath force-pushed the fix/ignore-unknown-activity-params branch from 6458bb3 to 471f861 Compare March 24, 2026 14:04
@andreiancuta-uipath
andreiancuta-uipath merged commit 5b4d584 into main Mar 24, 2026
117 checks passed
@andreiancuta-uipath
andreiancuta-uipath deleted the fix/ignore-unknown-activity-params branch March 24, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:uipath-langchain Triggers tests in the uipath-langchain-python repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants