Hotfix/uipath langchain 0.16.7.post1 - #1065
Conversation
…1048) Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
5a16c99 to
3b69fd2
Compare
There was a problem hiding this comment.
🟡 Changes recommended
CI triggering/PR scope are inconsistent with the stated intent (checks for release branches can still be skipped on version-bump pushes, and the PR includes a functional PII behavior change not reflected in the PR description/title).
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This hotfix branch updates the package release metadata to 0.16.7.post1, adjusts GitHub Actions triggers to include release/** branches, and (via the cherry-pick) changes how PII entity thresholds are derived from policy rows so custom policy entries aren’t silently dropped.
Changes:
- Bump project version from
0.16.7to0.16.7.post1(and updateuv.lockaccordingly). - Update CI/CD workflows to run on
release/**branches. - Change
PiiMasker._entity_thresholds_from_policyto request thresholds for every row inpii-entity-table(ignoringpii-entity-is-enabled) and add/adjust regression tests.
File summaries
| File | Description |
|---|---|
uv.lock |
Updates lock metadata and reflects the local package version bump. |
pyproject.toml |
Bumps the package version to 0.16.7.post1. |
.github/workflows/ci.yml |
Extends CI triggers to release/** branches (but still ignores pyproject.toml on push). |
.github/workflows/cd.yml |
Extends CD triggers to release/** branches for pyproject.toml changes. |
src/uipath_langchain/agent/tools/internal_tools/pii_masker.py |
Changes policy parsing so all table rows become entityThresholds entries. |
tests/agent/tools/internal_tools/test_pii_masker.py |
Updates/adds tests to validate the new threshold extraction behavior. |
Review details
- Files reviewed: 5/6 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| branches: | ||
| - main | ||
| - 'release/**' | ||
| paths-ignore: | ||
| - pyproject.toml |
| Every row in the table is requested. ``pii-entity-is-enabled`` is not | ||
| consulted: built-in categories carry it as ``true`` while custom rows | ||
| (e.g. ``FINationalID`` at 0.8) omit it entirely, so keying off it dropped | ||
| those rows from ``entityThresholds``. The service treats that list as an | ||
| allowlist, meaning a dropped row is never detected and its PII reaches |
| [project] | ||
| name = "uipath-langchain" | ||
| version = "0.16.7" | ||
| version = "0.16.7.post1" | ||
| description = "Python SDK that enables developers to build and deploy LangGraph agents to the UiPath Cloud Platform" |
|
| push: | ||
| branches: | ||
| - main | ||
| - 'release/**' |
There was a problem hiding this comment.
should this just be always on?
Cherry pick #1048 bump the current release version from 0.16.7 -> 0.16.7.post1
CI changes so the the checks run against the branch