Skip to content

fix(connections): send boolean activity params as true/false - #1936

Merged
dianagrecu-uipath merged 1 commit into
mainfrom
fix/is-activity-bool-params
Oct 9, 2026
Merged

dianagrecu-uipath merged 1 commit into
mainfrom
fix/is-activity-bool-params

Conversation

@dianagrecu-uipath

Copy link
Copy Markdown
Contributor

Summary

  • Integration Service activity parameters sent in the query string, the path, headers or as multipart form fields now serialize booleans as true/false instead of Python's True/False.
  • Strings, numbers and request bodies are unchanged.
  • Bumps uipath-platform to 0.2.35.

Why

_build_activity_request_spec converted every value with str(value), so a boolean tool argument went out as ?saveAsDraft=True. The IS connector hooks compare these values to the lowercase string, for example requestParams?.saveAsDraft == 'true' in the Gmail EmailReply / ForwardMailV2 and Outlook send-mail-v2 / ReplyToEmailV2 hooks. The check fails, so "Save as draft" is ignored and the email is sent. The same check is used for replyToAll, markAsRead, includeSubfolders and 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

  • Reproduced on alpha with the Outlook "Send Email" tool: the same request was delivered with saveAsDraft=True and saved as a draft with saveAsDraft=true.
  • New unit tests cover boolean query, header, path and multipart values; they fail on main with 'True' == 'true'.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 10:50
@github-actions github-actions Bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-integrations labels Oct 9, 2026

Copilot AI left a comment

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.

🟢 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-platform to 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.

@sonarqubecloud

sonarqubecloud Bot commented Oct 9, 2026

Copy link
Copy Markdown

@dianagrecu-uipath
dianagrecu-uipath merged commit de4423f into main Oct 9, 2026
133 of 147 checks passed
@dianagrecu-uipath
dianagrecu-uipath deleted the fix/is-activity-bool-params branch October 9, 2026 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants