Add rpa common pitfall with pkgs that end up in proj folder#77
Open
AdiNicuPopa wants to merge 4 commits intomainfrom
Open
Add rpa common pitfall with pkgs that end up in proj folder#77AdiNicuPopa wants to merge 4 commits intomainfrom
AdiNicuPopa wants to merge 4 commits intomainfrom
Conversation
AdiNicuPopa
requested review from
DragosUnguru and
RaduAna-Maria
as code owners
| - Use LINQ with explicit conversion: `dtData.AsEnumerable().Where(Function(row) CDbl(row("Amount")) > 1000).CopyToDataTable()` | ||
| - Convert the column type after reading: loop through rows and convert values, or clone the DataTable with the correct column types | ||
|
|
||
| ### `uip rpa restore` dumps .nupkg files into the project directory |
Collaborator
There was a problem hiding this comment.
@aoltean16 shouldn't we rather fail without a destination path? And instruct it like this in there.
I feel like adding this would pollute the skill.
DragosUnguru
requested changes
Mar 31, 2026
Comment on lines
+63
to
+73
| ### Stale Validation Cache | ||
|
|
||
| Studio may cache validation results internally. If `get-errors` reports persistent errors after multiple edits to the same file — especially OverloadGroup errors that should be resolved by your changes — the cache may be stale. To clear it: | ||
|
|
||
| ```bash | ||
| uip rpa close-project --project-dir "<PROJECT_DIR>" --format json | ||
| uip rpa open-project --project-dir "<PROJECT_DIR>" --format json | ||
| ``` | ||
|
|
||
| Then re-run `get-errors`. This forces Studio to reload all files from disk and revalidate from scratch. | ||
|
|
Contributor
There was a problem hiding this comment.
We shouldn't do this: closing and reopening studio for every "get errors" isn't a solution.
Also, this has been fixed at the get-errors-command-level. It should now return the freshest errors if you fetch the latest Studio version.
So we should be okay to revert this section.
Comment on lines
+502
to
+519
| ### `uip rpa restore` dumps .nupkg files into the project directory | ||
|
|
||
| When calling `uip rpa restore` without `--destination-path`, the command restores NuGet packages directly into the project root directory, polluting it with hundreds of `.nupkg` files. These are cached package files, not project artifacts, and should not be there. | ||
|
|
||
| **Always specify `--destination-path`** pointing to a location outside the project directory: | ||
|
|
||
| ```bash | ||
| # Correct — restore to a temp/cache directory | ||
| uip rpa restore --project-path "{projectRoot}" --destination-path "{projectRoot}/.local/packages" --format json | ||
|
|
||
| # Wrong — packages end up in project root | ||
| uip rpa restore --project-path "{projectRoot}" --format json | ||
| ``` | ||
|
|
||
| If you already have `.nupkg` files littering the project root, clean them up: | ||
| ```bash | ||
| rm "{projectRoot}"/*.nupkg | ||
| ``` |
Contributor
There was a problem hiding this comment.
Same for this -- this looks more like something that should be addressed through the command's business logic itself rather than have the model deal with these.
Let's revert this section and open a PR or a Jira to track this change. It should default the package restore dump path to where Studio usually does too.
Comment on lines
+58
to
+59
| | PDF activities (ReadPDFText, GetPDFPageCount, ExtractPDFPageRange, ManagePDFPassword, ExportPDFPageAsImage, ExtractImagesFromPDF, ReadXPSText) | `FileName` (string) | `ResourceFile` (IResource) | — | | ||
| | PDF convert activities (ConvertHtmlToPDF, ConvertTextToPDF) | `FileName` (string, when InputMode=File) | `ResourceFile` (IResource, when InputMode=File) | `Html`/`Text` (string, when InputMode=Content) | |
Contributor
There was a problem hiding this comment.
There is another table below that handles conditional properties ## Conditional Property Requirements
Author
There was a problem hiding this comment.
moved the conditional properties
skills/uipath-rpa-workflows/SKILL.md
Outdated
Comment on lines
+327
to
+328
| **IMPORTANT — `.Item` child elements:** The default XAML returned by this command may contain `.Item` child elements with `ItemArgument` nodes (e.g., `<activity:SomeActivity.Item><upap:ItemArgument .../></activity:SomeActivity.Item>`). These are internal scaffolding for OverloadGroup (FileName/ResourceFile) switching. **Do NOT copy `.Item` children into your generated XAML.** Instead, set the desired property (e.g., `FileName`) directly on the activity element and omit the `.Item` child. Studio auto-generates the internal structure when it loads the workflow. Including a misconfigured `.Item` child causes `"None of the overload groups have all their required/optional activity arguments configured"` errors. See [common-pitfalls.md](./references/common-pitfalls.md) for details. | ||
|
|
Contributor
There was a problem hiding this comment.
This is almost the same as the common-pitfalls.md text. Keep just a reference here
AdiNicuPopa
force-pushed
the
fix/rpa-common-pitfalls-project-deps-in-project-folder
branch
from
801e4de to
de8cc6a
Compare
DragosUnguru
approved these changes
Apr 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
We noticed this while testing the PDF activities.
Problem: uip rpa restore without --destination-path dumps all NuGet packages into the project root
Fix: Always pass --destination-path (e.g., {projectRoot}/.local/packages) to keep packages out of the project directory
UPDATE: fixing more issues
Issue 1: Studio caches validation results and get-errors might miss some errors.
Fix: updated
skills/uipath-rpa-workflows/references/validation-and-fixing.mdIssue 2: presence of .Item elements may cause "None of the overload groups have all their required/optional activity arguments configured" errors.
Fix: updated
skills/uipath-rpa-workflows/references/common-pitfalls.mdandskills/uipath-rpa-workflows/SKILL.mdIssue 3: agent created workflows in subfolders and named them
x:Class="Subfolder.WorkflowName"; this caused a runtime error because of the dot. Correct version isx:Class="Subfolder_WorkflowName"Fix: added a note in xaml-basics-and-rules.md, under the XAML File Anatomy section