-
Notifications
You must be signed in to change notification settings - Fork 29
feat: infer attachments #400
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
6a7a486 to
658aa1f
Compare
658aa1f to
b6b7e37
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds support for inferring attachments in runtime schema generation by integrating attachment transformation utilities from the uipath.runtime.schema module.
Changes:
- Replaced local schema transformation functions with imported utilities from
uipath.runtime.schema - Added attachment transformation to the input schema processing pipeline
- Updated dependency versions to support the new attachment transformation features
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| tests/runtime/test_schema.py | Updated test imports to use transform_references from uipath.runtime.schema and added comprehensive parametrized tests for attachment schema generation across different model types (Pydantic, dataclass, TypedDict) |
| src/uipath_langchain/runtime/schema.py | Replaced local _resolve_refs and _process_nullable_types functions with imported transform_references, transform_nullable_types, and transform_attachments utilities; integrated transform_attachments into input schema processing |
| pyproject.toml | Bumped minimum versions of uipath (2.4.0 → 2.4.14) and uipath-runtime (0.4.0 → 0.4.1) dependencies to support new transformation features |
| InputModel = exec_globals["InputModel"] | ||
|
|
||
| def generate_simple_langgraph_graph(): | ||
| # note: type ignore is needed here since mypy can t validate a dynamically created object's type |
Copilot
AI
Jan 13, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected spelling of 'can t' to 'can't'.
| # note: type ignore is needed here since mypy can t validate a dynamically created object's type | |
| # note: type ignore is needed here since mypy can't validate a dynamically created object's type |
| dependencies = [ | ||
| "uipath>=2.4.0, <2.5.0", | ||
| "uipath-runtime>=0.4.0, <0.5.0", | ||
| "uipath>=2.4.14,<2.5.0", |
Copilot
AI
Jan 13, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space after comma in version constraint. Should be >=2.4.14, <2.5.0 to match the formatting style used in line 9.
| "uipath>=2.4.14,<2.5.0", | |
| "uipath>=2.4.14, <2.5.0", |
b6b7e37 to
81c7f1f
Compare
Development Package