Repository navigation
fix(connections): send boolean activity params as true/false [PC-5201] - #1936
Conversation
There was a problem hiding this comment.
🟢 Approval recommended
The implementation matches the stated behavior, preserves non-boolean values, and includes focused regression coverage.
0 open findings
What changed in this PR
Corrects Integration Service activity serialization so boolean parameters use lowercase wire values.
Changes:
- Adds shared boolean parameter serialization.
- Covers query, header, path, and multipart parameters with tests.
- Bumps
uipath-platformto 0.2.35.
| File | Description |
|---|---|
packages/uipath-platform/src/uipath/platform/connections/_connections_service.py |
Serializes boolean activity parameters as true/false. |
packages/uipath-platform/tests/services/test_connections_service.py |
Tests boolean serialization across parameter locations. |
packages/uipath-platform/pyproject.toml |
Bumps package version. |
packages/uipath-platform/uv.lock |
Updates locked package version. |
packages/uipath/uv.lock |
Updates the parent package’s local dependency lock. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Epixa-Dogfood ReviewReview unavailable: Not reviewed: What this PR doesThe pull request author describes the intended change as follows: fix(connections): send boolean activity params as true/false PC-5201 This review did not reuse a local review of this change. Set up Epixa local review. Full review: 1d947f2 |



Summary
true/falseinstead of Python'sTrue/False.uipath-platformto 0.2.35.Why
_build_activity_request_specconverted every value withstr(value), so a boolean tool argument went out as?saveAsDraft=True. The IS connector hooks compare these values to the lowercase string, for examplerequestParams?.saveAsDraft == 'true'in the GmailEmailReply/ForwardMailV2and Outlooksend-mail-v2/ReplyToEmailV2hooks. The check fails, so "Save as draft" is ignored and the email is sent. The same check is used forreplyToAll,markAsRead,includeSubfoldersand other flags.Reported by a customer in UV-16521 (Gmail "Reply to Email" with "Save as draft" set as a static value). The .NET agent executor already sends lowercase values. Maestro hit the same issue in ENGCE-56530.
Testing
saveAsDraft=Trueand saved as a draft withsaveAsDraft=true.'True' == 'true'.