Repository navigation
feat(uipath-maestro-flow): promote the builder-SDK Flow skill; Studio Web pins v1 - #3624
Conversation
…each package Review follow-up on #3624. The default package now ships the builder-SDK Flow skill, but uipath-human-in-the-loop, uipath-agents and uipath-platform still taught hand-editing `.flow` JSON, and the 11 link rewrites pointed at "guidance" v2 does not have (0 hits for agentInputVariables, variableUpdates, contentTokens in v2). A hand edit to the compiled `.flow` is lost on the next compile (verified: an edited node label and an added key were gone after `compile -o`). Each affected passage is now a flavor block: - 11 pointer blocks: the canonical text points at the v2 file that covers the topic (hitl.md, inline-agent.md, return.md, operate.md, connector-params.md#structured-filters-ceql, #schema-dynamic-..., event-trigger.md#filter-operators, SKILL.md#builder-frame). The Studio Web override restores the original v1 line verbatim. - CEQL: the platform doc's "never hand-write the CEQL string" now names the builder-SDK exception, where v2 says to write the string and the SDK derives the tree. - Generated (non-catalog) activity: v2 has no node for one (rawNode refuses uipath.connector.*), so the default text says so and offers managed HTTP in connector mode instead of pointing at nothing. - 6 gates (HITL surface row, apptask, coded action app, inline-in-flow, coded quickstart step 7, embedding-in-flows) route builder-SDK work to the v2 file; their Studio Web overrides are empty. The Studio Web build is byte-identical to main again (diff -r of the whole composed tree is empty). Link check passes, including the new block-aware rule. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KSLJwM2pMwZa7ygePiY6mQ
…lag only as a retry Review follow-up (from #3624): forcing `--@UiPath:registry=https://registry.npmjs.org` breaks customers behind a corporate mirror with public egress blocked. Run plain `npm install` first; retry once with the flag only on E401/E404 when `npm config get @UiPath:registry` names GitHub Packages. Verified: that setup fails with E401, and the flagged retry installs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KSLJwM2pMwZa7ygePiY6mQ
|
@rockymadden thanks for the review. Here's where each point landed. Everything is pushed; head is 🟠 Sibling skills contradict the shipped Flow skill: fixed (a6c5ce5)I confirmed your reasoning: after a hand edit to the compiled The fix uses flavor blocks, so each package gets the guidance for the Flow generation it ships:
"Look at all the other skills that hardlink to Flow"The link checker walks every relative link in Losing capabilitiesOperate, diagnose and evaluate were ported earlier (#3386, #3421, #3423). Author is now #3642: Do the smoke tests work with v2 in place of v1?Yes. The earlier CI run of this PR, with v2 Flow in the default catalog, passed the smoke gate at 95.1% (117/123).
"Ideally no legacy skill; audit the flavor for v2 instead"I'd prefer that too, but Studio Web can't run v2 at all right now:
No flavor override can fix that, so the pin is the interim measure. The real fix is filed as UiPath/cli#4629. The composer refuses a classic tree that no flavor pins, so the tree has to be deleted when that issue closes. I renamed it from 🟡 Studio Web's Flow loses eval coverageIt keeps two measurements:
Not verified:
|
… token (#3626) * fix(uipath-maestro-flow/preview): install the SDK from public npm, no token The install note said `@uipath/maestro-builder-sdk` resolves through GitHub Packages and needs a `read:packages` token. It is published to public npmjs.com too (flow-builder-sdk release.yml; `latest` is 6.11.6 on both feeds), so a customer without a GitHub token was told the install could not work. Now the command carries `--@UiPath:registry=https://registry.npmjs.org`, the same flag uipath-coded-apps uses, so an `.npmrc` that routes `@uipath` to GitHub Packages cannot turn it into a 401. Verified: with such an `.npmrc` and no token, the command installs 6.11.6. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KSLJwM2pMwZa7ygePiY6mQ * fix(uipath-maestro-flow/preview): plain npm install first; registry flag only as a retry Review follow-up (from #3624): forcing `--@UiPath:registry=https://registry.npmjs.org` breaks customers behind a corporate mirror with public egress blocked. Run plain `npm install` first; retry once with the flag only on E401/E404 when `npm config get @UiPath:registry` names GitHub Packages. Verified: that setup fails with E401, and the flagged retry installs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KSLJwM2pMwZa7ygePiY6mQ --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Default-package guidance still contains several v1-only instructions and two promoted Flow examples or validation rules are demonstrably incorrect.
Review effort: Balanced
Findings: 4
Open (4)
What changed in this PR
Promotes the builder-SDK Maestro Flow skill to the default catalog while retaining classic JSON authoring for Studio Web through flavor pinning.
Changes:
- Moves Flow v2 into the default skill tree and retains v1 under
classic/. - Adds flavor composition, validation, link checking, and Studio Web overrides.
- Updates dependent skills, documentation, ownership, and tests for generation-aware routing.
Six unresolved issues were identified, including instructions that still direct builder-SDK users to edit generated .flow JSON, incomplete dual-tree anchor enforcement, and incorrect Flow examples.
| File | Description |
|---|---|
.claude/skills/manage-skill-flavors/SKILL.md |
Documents canonical-generation pins. |
.github/workflows/verb-gate.yml |
Extends verb checks to classic skills. |
CODEOWNERS |
Updates ownership for moved Flow files. |
CONTRIBUTING.md |
Documents the classic tree and flavor pins. |
classic/skills/uipath-maestro-flow/.maintenance/README.md |
Retains v1 maintenance documentation. |
classic/skills/uipath-maestro-flow/.maintenance/check-all.sh |
Retains the v1 aggregate checker. |
classic/skills/uipath-maestro-flow/.maintenance/check-anchors.sh |
Retains v1 anchor checks. |
classic/skills/uipath-maestro-flow/.maintenance/check-depth.sh |
Retains v1 depth checks. |
classic/skills/uipath-maestro-flow/.maintenance/check-link-text.sh |
Retains v1 link-text checks. |
classic/skills/uipath-maestro-flow/.maintenance/check-links.sh |
Retains v1 link checks. |
classic/skills/uipath-maestro-flow/.maintenance/check-orphans.sh |
Retains v1 orphan checks. |
classic/skills/uipath-maestro-flow/.maintenance/check-plugin-pairs.sh |
Retains v1 plugin-pair checks. |
classic/skills/uipath-maestro-flow/.maintenance/check-template.sh |
Retains v1 template checks. |
classic/skills/uipath-maestro-flow/.maintenance/check-uip-commands.sh |
Retains v1 CLI checks. |
classic/skills/uipath-maestro-flow/.maintenance/check-versions.sh |
Retains v1 version checks. |
classic/skills/uipath-maestro-flow/SKILL.md |
Moves the v1 Flow contract into classic/. |
classic/skills/uipath-maestro-flow/references/author/CAPABILITY.md |
Retains v1 authoring guidance. |
classic/skills/uipath-maestro-flow/references/author/brownfield.md |
Retains v1 brownfield guidance. |
classic/skills/uipath-maestro-flow/references/author/editing-operations-cli.md |
Retains v1 CLI editing guidance. |
classic/skills/uipath-maestro-flow/references/author/editing-operations-json.md |
Retains v1 JSON editing guidance. |
classic/skills/uipath-maestro-flow/references/author/editing-operations.md |
Retains v1 editing guidance. |
classic/skills/uipath-maestro-flow/references/author/greenfield.md |
Retains v1 greenfield guidance. |
classic/skills/uipath-maestro-flow/references/author/planning-arch.md |
Retains v1 architecture planning. |
classic/skills/uipath-maestro-flow/references/author/planning-impl.md |
Retains v1 implementation planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/agent/impl.md |
Retains v1 agent implementation guidance. |
classic/skills/uipath-maestro-flow/references/author/plugins/agent/planning.md |
Retains v1 agent planning guidance. |
classic/skills/uipath-maestro-flow/references/author/plugins/agentic-process/impl.md |
Retains v1 agentic-process implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/agentic-process/planning.md |
Retains v1 agentic-process planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/api-workflow/impl.md |
Retains v1 API-workflow implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/api-workflow/planning.md |
Retains v1 API-workflow planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/batch-transform/impl.md |
Retains v1 batch-transform implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/batch-transform/planning.md |
Retains v1 batch-transform planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/connector-trigger/impl.md |
Retains v1 connector-trigger implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/connector-trigger/planning.md |
Retains v1 connector-trigger planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/connector/impl-inline.md |
Retains v1 inline connector guidance. |
classic/skills/uipath-maestro-flow/references/author/plugins/connector/impl.md |
Retains v1 connector implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/connector/planning.md |
Retains v1 connector planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/conversational-agent/impl.md |
Retains v1 conversational-agent implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/conversational-agent/planning.md |
Retains v1 conversational-agent planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/data-fabric/impl.md |
Retains v1 Data Fabric implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/data-fabric/planning.md |
Retains v1 Data Fabric planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/decision/impl.md |
Retains v1 decision implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/decision/planning.md |
Retains v1 decision planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/delay/impl.md |
Retains v1 delay implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/delay/planning.md |
Retains v1 delay planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/end/impl.md |
Retains v1 end-node implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/end/planning.md |
Retains v1 end-node planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/flow/impl.md |
Retains v1 subflow implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/flow/planning.md |
Retains v1 subflow planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/hitl/impl.md |
Retains v1 HITL implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/hitl/planning.md |
Retains v1 HITL planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/http/impl-connector.md |
Retains v1 connector HTTP guidance. |
classic/skills/uipath-maestro-flow/references/author/plugins/http/impl-manual.md |
Retains v1 manual HTTP guidance. |
classic/skills/uipath-maestro-flow/references/author/plugins/http/impl.md |
Retains v1 HTTP implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/http/planning.md |
Retains v1 HTTP planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/inline-agent/impl.md |
Retains v1 inline-agent implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/inline-agent/planning.md |
Retains v1 inline-agent planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/inline-voice-agent/impl.md |
Retains v1 voice-agent implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/inline-voice-agent/planning.md |
Retains v1 voice-agent planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/ixp/impl.md |
Retains v1 IXP implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/ixp/planning.md |
Retains v1 IXP planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/loop/impl.md |
Retains v1 loop implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/loop/planning.md |
Retains v1 loop planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/merge/impl.md |
Retains v1 merge implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/merge/planning.md |
Retains v1 merge planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/queue/impl.md |
Retains v1 queue implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/queue/planning.md |
Retains v1 queue planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/rpa/impl.md |
Retains v1 RPA implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/rpa/planning.md |
Retains v1 RPA planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/scheduled-trigger/impl.md |
Retains v1 scheduled-trigger implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/scheduled-trigger/planning.md |
Retains v1 scheduled-trigger planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/script/impl.md |
Retains v1 script implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/script/planning.md |
Retains v1 script planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/subflow/impl.md |
Retains v1 subflow implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/subflow/planning.md |
Retains v1 subflow planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/summarize/impl.md |
Retains v1 summarize implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/summarize/planning.md |
Retains v1 summarize planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/switch/impl.md |
Retains v1 switch implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/switch/planning.md |
Retains v1 switch planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/terminate/impl.md |
Retains v1 terminate implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/terminate/planning.md |
Retains v1 terminate planning. |
classic/skills/uipath-maestro-flow/references/author/plugins/transform/impl.md |
Retains v1 transform implementation. |
classic/skills/uipath-maestro-flow/references/author/plugins/transform/planning.md |
Retains v1 transform planning. |
classic/skills/uipath-maestro-flow/references/diagnose/CAPABILITY.md |
Retains v1 diagnostic capability. |
classic/skills/uipath-maestro-flow/references/diagnose/failure-modes.md |
Retains v1 failure-mode guidance. |
classic/skills/uipath-maestro-flow/references/diagnose/troubleshooting-guide.md |
Retains v1 troubleshooting guidance. |
classic/skills/uipath-maestro-flow/references/evaluate/CAPABILITY.md |
Retains v1 evaluation capability. |
classic/skills/uipath-maestro-flow/references/evaluate/commands-reference.md |
Retains v1 evaluation commands. |
classic/skills/uipath-maestro-flow/references/evaluate/eval-sets-guide.md |
Retains v1 eval-set guidance. |
classic/skills/uipath-maestro-flow/references/evaluate/evaluators-guide.md |
Retains v1 evaluator guidance. |
classic/skills/uipath-maestro-flow/references/evaluate/running-guide.md |
Retains v1 evaluation-run guidance. |
classic/skills/uipath-maestro-flow/references/evaluate/upload-safety.md |
Retains v1 upload-safety guidance. |
classic/skills/uipath-maestro-flow/references/operate/CAPABILITY.md |
Retains v1 operations capability. |
classic/skills/uipath-maestro-flow/references/operate/manage.md |
Retains v1 management guidance. |
classic/skills/uipath-maestro-flow/references/operate/run.md |
Retains v1 runtime guidance. |
classic/skills/uipath-maestro-flow/references/operate/ship.md |
Retains v1 deployment guidance. |
classic/skills/uipath-maestro-flow/references/shared/action-nodes.md |
Retains v1 action-node guidance. |
classic/skills/uipath-maestro-flow/references/shared/cli-commands.md |
Retains v1 CLI command reference. |
classic/skills/uipath-maestro-flow/references/shared/cli-conventions.md |
Retains v1 CLI conventions. |
classic/skills/uipath-maestro-flow/references/shared/file-format.md |
Retains v1 file-format guidance. |
classic/skills/uipath-maestro-flow/references/shared/node-output-wiring.md |
Retains v1 output-wiring guidance. |
classic/skills/uipath-maestro-flow/references/shared/ux-narration-and-todos.md |
Retains v1 UX guidance. |
classic/skills/uipath-maestro-flow/references/shared/variables-and-expressions.md |
Retains v1 variable guidance. |
preview/.claude-plugin/plugin.json |
Updates preview plugin contents. |
preview/skills/uipath-maestro-flow |
Adds a compatibility symlink to promoted Flow. |
preview/skills/uipath-maestro-flow/SKILL.md |
Removes the former standalone preview copy. |
scripts/audit-capability-port.mjs |
Points capability auditing at the promoted tree. |
scripts/check-skill-links.mjs |
Adds classic and pinned-tree link resolution. |
scripts/compose-skill-flavor.mjs |
Implements classic canonical pins. |
skill-flavors/studioweb/uipath-agents/references/coded/embedding-in-flows.md |
Restores v1 coded-agent Flow guidance. |
skill-flavors/studioweb/uipath-agents/references/coded/quickstart.md |
Restores v1 coded-agent quickstart guidance. |
skill-flavors/studioweb/uipath-agents/references/lowcode/capabilities/inline-in-flow/inline-in-flow.md |
Restores v1 inline-agent guidance. |
skill-flavors/studioweb/uipath-agents/references/lowcode/prompting/autonomous-agent-prompting-guide.md |
Restores v1 prompt-wiring guidance. |
skill-flavors/studioweb/uipath-human-in-the-loop/SKILL.md |
Restores v1 Flow HITL routing. |
skill-flavors/studioweb/uipath-human-in-the-loop/references/hitl-node-apptask.md |
Restores v1 AppTask instructions. |
skill-flavors/studioweb/uipath-human-in-the-loop/references/hitl-node-coded-action-app.md |
Restores v1 coded-app instructions. |
skill-flavors/studioweb/uipath-human-in-the-loop/references/hitl-node-quickform.md |
Restores v1 QuickForm instructions. |
skill-flavors/studioweb/uipath-maestro-flow/.canonical |
Pins Studio Web Flow to classic. |
skill-flavors/studioweb/uipath-platform/references/integration-service/activities.md |
Restores v1 activity guidance. |
skill-flavors/studioweb/uipath-platform/references/integration-service/activity-generation.md |
Restores v1 activity-generation guidance. |
skill-flavors/studioweb/uipath-platform/references/integration-service/triggers.md |
Restores v1 trigger operator guidance. |
skills/uipath-agents/references/coded/embedding-in-flows.md |
Routes embedded agents through Flow v2. |
skills/uipath-agents/references/coded/quickstart.md |
Adds builder-SDK coded-agent wiring. |
skills/uipath-agents/references/lowcode/capabilities/inline-in-flow/inline-in-flow.md |
Adds builder-SDK inline-agent routing. |
skills/uipath-agents/references/lowcode/prompting/autonomous-agent-prompting-guide.md |
Updates builder-SDK prompt wiring. |
skills/uipath-human-in-the-loop/SKILL.md |
Routes Flow HITL through the SDK skill. |
skills/uipath-human-in-the-loop/references/hitl-node-apptask.md |
Adds builder-SDK AppTask guidance. |
skills/uipath-human-in-the-loop/references/hitl-node-coded-action-app.md |
Adds builder-SDK coded-app guidance. |
skills/uipath-human-in-the-loop/references/hitl-node-quickform.md |
Adds builder-SDK QuickForm guidance. |
skills/uipath-maestro-flow/SKILL.md |
Promotes the builder-SDK Flow contract. |
skills/uipath-maestro-flow/examples/AquariumSafetyStop.flow.ts |
Promotes the safety-stop example. |
skills/uipath-maestro-flow/examples/ArchiveCardExtract.flow.ts |
Promotes the document extraction example. |
skills/uipath-maestro-flow/examples/BakeOffEntryForm.flow.ts |
Promotes the form example. |
skills/uipath-maestro-flow/examples/BeeHiveLedger.flow.ts |
Promotes the ledger example. |
skills/uipath-maestro-flow/examples/BeltProgression.flow.ts |
Promotes the progression example. |
skills/uipath-maestro-flow/examples/BirdCountLookup.flow.ts |
Promotes the lookup example. |
skills/uipath-maestro-flow/examples/ClubDirectory.flow.ts |
Promotes the directory example. |
skills/uipath-maestro-flow/examples/ConcertSoundcheck.flow.ts |
Promotes the soundcheck example. |
skills/uipath-maestro-flow/examples/DoorbellLog.flow.ts |
Promotes the event-log example. |
skills/uipath-maestro-flow/examples/FestivalMapScaffold.flow.ts |
Promotes the scaffold example. |
skills/uipath-maestro-flow/examples/FieldTripQuickForm.flow.ts |
Promotes the QuickForm example. |
skills/uipath-maestro-flow/examples/FossilCatalogEnrich.flow.ts |
Promotes the enrichment example. |
skills/uipath-maestro-flow/examples/GallerySubmission.flow.ts |
Promotes the submission example. |
skills/uipath-maestro-flow/examples/GreenhouseWatering.flow.ts |
Promotes the scheduling example. |
skills/uipath-maestro-flow/examples/HarbourRadioLine.flow.ts |
Promotes the queue example. |
skills/uipath-maestro-flow/examples/HerbariumDispatch.flow.ts |
Promotes the dispatch example. |
skills/uipath-maestro-flow/examples/KilnReview.flow.ts |
Promotes the review example. |
skills/uipath-maestro-flow/examples/LibraryDeskChat.flow.ts |
Promotes the conversational example. |
skills/uipath-maestro-flow/examples/LighthouseSignal.flow.ts |
Promotes the branching example. |
skills/uipath-maestro-flow/examples/MeteorShowerPages.flow.ts |
Promotes the pagination example. |
skills/uipath-maestro-flow/examples/NeighborhoodWalkPlanner.flow.ts |
Promotes the planning example. |
skills/uipath-maestro-flow/examples/ObservatorySeeing.flow.ts |
Promotes the observation example. |
skills/uipath-maestro-flow/examples/OralHistoryDigest.flow.ts |
Promotes the summarization example. |
skills/uipath-maestro-flow/examples/PlanetariumConfirmation.flow.ts |
Promotes the confirmation example. |
skills/uipath-maestro-flow/examples/PlantNameAdvisor.flow.ts |
Promotes the agent example. |
skills/uipath-maestro-flow/examples/PostcardCaption.flow.ts |
Promotes the caption example. |
skills/uipath-maestro-flow/examples/PotteryStudioCallback.flow.ts |
Promotes the callback example. |
skills/uipath-maestro-flow/examples/RecipeScaler.flow.ts |
Promotes the subflow example. |
skills/uipath-maestro-flow/examples/SeedPacketReader.flow.ts |
Promotes the seed extraction example. |
skills/uipath-maestro-flow/examples/TideTableConverter.flow.ts |
Promotes the transform example. |
skills/uipath-maestro-flow/examples/TrailLogSummary.flow.ts |
Promotes the summary example. |
skills/uipath-maestro-flow/examples/WorkshopInventory.flow.ts |
Promotes the inventory example. |
skills/uipath-maestro-flow/examples/bindings.json |
Promotes example bindings. |
skills/uipath-maestro-flow/references/CLI-LOOP.md |
Promotes the compile/check workflow. |
skills/uipath-maestro-flow/references/agent-resources.md |
Promotes agent-resource guidance. |
skills/uipath-maestro-flow/references/agent.md |
Promotes agent-node guidance. |
skills/uipath-maestro-flow/references/agentic-process.md |
Promotes agentic-process guidance. |
skills/uipath-maestro-flow/references/api-workflow.md |
Promotes API-workflow guidance. |
skills/uipath-maestro-flow/references/batch-transform.md |
Promotes batch-transform guidance. |
skills/uipath-maestro-flow/references/bindings.md |
Promotes binding guidance. |
skills/uipath-maestro-flow/references/branch.md |
Promotes branch guidance. |
skills/uipath-maestro-flow/references/brownfield.md |
Promotes brownfield guidance. |
skills/uipath-maestro-flow/references/connector-params.md |
Promotes connector parameter guidance. |
skills/uipath-maestro-flow/references/conversational.md |
Promotes conversational-flow guidance. |
skills/uipath-maestro-flow/references/data-fabric.md |
Promotes Data Fabric guidance. |
skills/uipath-maestro-flow/references/delay.md |
Promotes delay guidance. |
skills/uipath-maestro-flow/references/diagnose.md |
Promotes diagnostic guidance. |
skills/uipath-maestro-flow/references/document-pipeline.md |
Promotes document-pipeline guidance. |
skills/uipath-maestro-flow/references/error-handling.md |
Promotes error-handling guidance. |
skills/uipath-maestro-flow/references/evaluate.md |
Promotes evaluation guidance. |
skills/uipath-maestro-flow/references/event-trigger.md |
Promotes event-trigger guidance. |
skills/uipath-maestro-flow/references/form-trigger.md |
Promotes form-trigger guidance. |
skills/uipath-maestro-flow/references/hitl.md |
Promotes HITL builder guidance. |
skills/uipath-maestro-flow/references/http.md |
Promotes HTTP guidance. |
skills/uipath-maestro-flow/references/inline-agent.md |
Promotes inline-agent guidance. |
skills/uipath-maestro-flow/references/ixp.md |
Promotes IXP guidance. |
skills/uipath-maestro-flow/references/loops.md |
Promotes loop guidance. |
skills/uipath-maestro-flow/references/manual-trigger.md |
Promotes manual-trigger guidance. |
skills/uipath-maestro-flow/references/operate.md |
Promotes operational guidance. |
skills/uipath-maestro-flow/references/or-processes.md |
Promotes Orchestrator-process guidance. |
skills/uipath-maestro-flow/references/parallel-merge.md |
Promotes parallel-merge guidance. |
skills/uipath-maestro-flow/references/placeholder.md |
Promotes placeholder guidance. |
skills/uipath-maestro-flow/references/published-function.md |
Promotes published-function guidance. |
skills/uipath-maestro-flow/references/queue.md |
Promotes queue guidance. |
skills/uipath-maestro-flow/references/return.md |
Promotes return-value guidance. |
skills/uipath-maestro-flow/references/rpa-workflow.md |
Promotes RPA workflow guidance. |
skills/uipath-maestro-flow/references/scheduled-trigger.md |
Promotes scheduled-trigger guidance. |
skills/uipath-maestro-flow/references/script.md |
Promotes script guidance. |
skills/uipath-maestro-flow/references/subflow.md |
Promotes subflow guidance. |
skills/uipath-maestro-flow/references/summarize.md |
Promotes summarize guidance. |
skills/uipath-maestro-flow/references/switch.md |
Promotes switch guidance. |
skills/uipath-maestro-flow/references/terminate.md |
Promotes termination guidance. |
skills/uipath-maestro-flow/references/transform.md |
Promotes transformation guidance. |
skills/uipath-maestro-flow/references/voice.md |
Promotes voice-agent guidance. |
skills/uipath-platform/references/integration-service/activities.md |
Routes Flow activities to builder guidance. |
skills/uipath-platform/references/integration-service/activity-generation.md |
Updates Flow activity generation. |
skills/uipath-platform/references/integration-service/triggers.md |
Updates Flow trigger operator routing. |
tests/README.md |
Documents classic Flow test setup. |
tests/experiments/flow-v2-preview.yaml |
Keeps the preview experiment pointed at v2. |
tests/scripts/check-skill-links.test.mjs |
Adds pinned-tree link tests. |
tests/scripts/compose-skill-flavor.test.mjs |
Adds flavor-pin composition tests. |
tests/tasks/uipath-maestro-flow/_shared/test_flow_inline_agent_prompts.py |
Points prompt tests at promoted Flow. |
tests/tasks/uipath-maestro-flow/_shared/test_preview_skill_read_window.py |
Points read-window tests at promoted Flow. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
… Web pins v1 Nightly coder-eval shows the builder-SDK Flow skill (v2) at success-rate parity with the JSON-authoring skill (v1), and UiPath/cli#4575 (deb7802) ships `flow compile/check/decompile/merge` and `uip maestro registry` on stable CLI builds. So v2 becomes the shipped uipath-maestro-flow. - preview/skills/uipath-maestro-flow -> skills/uipath-maestro-flow (v2, with all four capabilities: author #3642, operate, diagnose, evaluate). - skills/uipath-maestro-flow -> classic/skills/uipath-maestro-flow (v1, byte-identical), pinned by the Studio Web flavor via skill-flavors/studioweb/uipath-maestro-flow/.canonical (#3625). Studio Web's browser bundle cannot start the SDK process `flow compile` needs; the real fix is UiPath/cli#4629. - preview/skills/uipath-maestro-flow is a relative symlink to the promoted tree, so flow-v2-preview keeps loading all three SDK skills. Case and BPMN stay preview-only. - Description keeps v1's routing terms ("Maestro Automate", `.flow`, IXP document extraction, inline agents, HITL, chat/voice) and the uipath-ixp / uipath-agents negatives; BPMN routes on `.bpmn` and `.bpmn.ts`. 911/1024 chars. - Sibling skills (human-in-the-loop, agents, platform) follow the Flow generation of each package through flavor blocks: the default package points at the v2 file that covers the topic (11 pointers, 6 gates), the Studio Web override restores the v1 text. The composed Studio Web tree is byte-identical to main. - CODEOWNERS, verb gate (classic/skills), read-window and inline-agent prompt tests, CONTRIBUTING (classic/, symlinks), tests/README, flow-v2-preview.yaml, preview plugin.json follow the new layout. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KSLJwM2pMwZa7ygePiY6mQ
995daaf to
246e4b6
Compare
… fatal cross-generation anchors Copilot review follow-ups on #3624: - uipath-agents inline-in-flow: the ordering note and the Flow Wiring body still sent builder-SDK readers to direct `.flow` authoring. Both are flavor blocks now; the default says `inlineAgent({...})` is the node and `compile` emits it and its edges. Studio Web keeps its text. - uipath-human-in-the-loop SKILL: a builder-SDK Flow request stops after surface detection and goes to uipath-maestro-flow (hitl.md); Steps 2-6 are for Case, Low-Code Agent, Maestro and JSON-authored `.flow`. - uipath-platform triggers: builder-SDK flows stop after step 2 and use `filters: [{ field, <operator>: value }]`; steps 3-6 are the classic structured tree. - check-skill-links: a dead anchor in the other generation is always fatal (cross-generation anchors are new, so they start clean). Studio Web composed tree still byte-identical to main. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KSLJwM2pMwZa7ygePiY6mQ
|
@rockymadden @dmetzgar I rebased onto What changed since your approvals:
Checks on this head, run locally:
Copilot's overview also mentions two findings about "incorrect Flow examples" that were never posted inline; if anyone sees them, point me at them. |
rockymadden
left a comment
There was a problem hiding this comment.
🟢 Approved. The rebase is a clean re-move, and ab589ca09 closes Copilot's four points. Two minors below; neither blocks.
What
- Same change I approved at
995daaf47: v2 Flow ships asskills/uipath-maestro-flow, v1 moves toclassic/, and Studio Web pins it through.canonical. - The rebase picks up #3610, #3616–#3620, #3626 and #3642 in the promoted tree. #3625's commits drop out.
ab589ca09adds flavor blocks to agentsinline-in-flow.md, the HITLSKILL.mdand platformtriggers.md. It also makes dead cross-generation anchors fatal.
Checked since my approval
- ✅
classic/skills/uipath-maestro-flowis byte-identical tomain's v1 tree at the base72eb7e37d. - ✅
skills/uipath-maestro-flowequals the tree I approved plus exactly the merged preview fixes. The only extra difference is the description (inline). - ✅ Outside the moved trees,
995daaf47→ head differs only byab589ca09. - ✅ No
maincommit after the base touches the moved paths. #3165 edits two agents files this PR also flavors, but the edited lines don't overlap and GitHub reports the PR mergeable. - ✅ The SDK shapes the new blocks cite exist:
filters: [{ field, <op>: value }]and#filter-operatorsinevent-trigger.md,hitl(...)inhitl.md, and theinlineAgentsidecar ininline-agent.md. - ✅ CI: 70 pass, 2 skipped.
- ❓ Not re-verified by me: the Studio Web composed tree is still byte-identical to
main. I read the new overrides, which restore the original text or leave the block empty, but I did not run the composer.
tl;dr
The rebase is faithful and the Copilot follow-ups are sound. Two small fixes to consider: the description drops #3618's "list IxP models" wording, and the new HITL stop gate disagrees with the surface row above it about JSON-only .flow.
…n the Flow surface Review follow-ups on #3624: - Description: "IXP document extraction (add or list IXP models)" restores the listing route #3618 added (routing_listing prompts ask to list, not author; uipath-ixp sends listing here). 936/1024 chars. - uipath-human-in-the-loop: the stop gate now keys on the Flow surface (a `.flow.ts`, a `.flow`, or a new flow), matching the surface row, and hands the request to uipath-maestro-flow, which decompiles an existing `.flow` first. Steps 2-6 are Case, Low-Code Agent and Maestro only in the default package; Studio Web's override is unchanged. Studio Web composed tree still byte-identical to the merge base. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KSLJwM2pMwZa7ygePiY6mQ
…maestro-skills # Conflicts: # CONTRIBUTING.md # scripts/check-skill-links.mjs
main promoted the builder-SDK Flow skill (#3624) and pinned the old one for Studio Web as classic. Resolution: - skills/uipath-maestro-flow/SKILL.md: take main's; rule 9 no longer exists - the hand-written-JSON guidance (node configure, debug note) follows its file into classic/skills/.../plugins/data-fabric/impl.md, which Studio Web still ships - references/data-fabric.md (new SDK skill) gains "An entity the solution authors itself": take folderKey (the 99999999-... placeholder) from `df entities get --local` and resourceKey from the Source: Local row of `solution resources list --kind Entity`, not the tenant; its "omit both" fallback is exactly the Entity-does-not-exist failure for a local entity. Also the INLINE_ENTITY_UNBOUND fix for a .flow.ts, and the debug note - local-entities.md: both authoring paths; the SDK paragraph sits in a flavor block that the studioweb flavor overrides to nothing Co-authored-by: Claude <noreply@anthropic.com>
…1.203 (#3841) * fix(uipath-maestro-bpmn): resolve merge markers shipped in two references (#3491) The #2387 rebase left four unresolved conflict blocks in local-metadata-regeneration-guide.md and operate/CAPABILITY.md. Where the sides disagree, keep the one SKILL.md already states: validate after refresh, never update-metadata. Where they do not overlap, keep both. Add the validate step the kept text pointed at but the list lacked. Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com> (cherry picked from commit 83bfc47) * fix(uipath-maestro-flow): drop alpha.uipath.com from customer-facing examples The #3792 cherry-pick (#3838) covers every other skill, but these four maestro-flow references were rewritten on main by #3624 and so fall outside it. Use the same <AUTHORITY_URL> login placeholder and a cloud.uipath.com example host. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Rocky Madden <github@rockymadden.com> Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>

Summary
Short version: the builder-SDK Flow skill (v2) becomes the shipped
uipath-maestro-flow. The JSON-authoring skill (v1) moves toclassic/, and only the Studio Web flavor keeps using it, because Studio Web cannot run the SDK yet. Case and BPMN stay inpreview/.Why now:
deb7802) shipsuip maestro flow compile/check/decompile/mergeanduip maestro registryon stable CLI builds. Before it, stableuipdid not have the commands v2 uses.Related PRs and merge order
cp -RL); feat(flow-skill): Add capability discovery to Phase 1 planning #151 keeps the v1 arm on v1 Flow by building it fromclassic/. Without feat(flow-skill): Add capability discovery to Phase 1 planning #151 both arms load v2 Flow after this merges. Original item: UiPath/coder_eval_uipath#147 (cp -RLin the v2-arm staging). Merge first; without it the ADO v2 arm silently loses Flow once this lands.references/author.md), the fourth capability port after operate/diagnose/evaluate. Lands at the preview path; this PR moves it.preview/path. The last commit here carries the identical line at the promoted path, so either merge order works.Open PRs that edit the moving paths. Each will conflict after this PR merges.
skill-flow-solution-select-ask, one of the two Flow smoke failures below.classic/: feat(uipath-maestro-case): author folder-scoped Data Fabric entity pickers [ENGCE-60825] #3595, docs(skills): steer agents to Integration Service connections, not direct API keys #3594, fix(hitl): forbid shape on HITL node instances, add failure mode #3482, docs(platform): local entity authoring for coding agents #3453, docs(maestro-flow): correct flow debug fault guidance #3428, fix(uipath-maestro-flow, uipath-human-in-the-loop): address post-merge review comments on #3180 #3267, docs(uipath-maestro-flow): HITL nodes pause the run; never use one to show a value #3089. Their authors need to retarget: port the change to the v2 tree, and toclassic/only if Studio Web needs it.What moved
preview/skills/uipath-maestro-flow(v2)skills/uipath-maestro-flowskills/uipath-maestro-flow(v1)classic/skills/uipath-maestro-flowpreview/skills/uipath-maestro-flow→ symlink to../../skills/uipath-maestro-flowflow-v2-preview.yamlarm (still loads all three SDK skills)Case and BPMN are not promoted. Their preview trees cover authoring only: no operate/diagnose, and for Case no SDD design or troubleshooting. They follow the promotion plan's phases 4–6 separately.
Naming. The retained tree is
classic/, notlegacy/: it stays shipped and supported for as long as Studio Web needs it. "Classic" is UiPath's term for an older generation that is still supported.Why Studio Web keeps v1: flavor pins (commit 1)
uip maestro flow compilehands off to the SDK by starting a separate process (uipath-cli/packages/builder-sdk/src/delegate.ts). In Studio Web's browser bundle every process start throws (packages/cli-browser/src/shims/child-process.browser.ts). So a Studio Web agent cannot author.flow.ts. Sparse marker overrides cannot fix that, and the composer had no way to keep the working generation for one host.New rule in
scripts/compose-skill-flavor.mjs:With that pin, the Studio Web flavor composes
uipath-maestro-flowfromclassic/skills/uipath-maestro-flow/, and its existing overrides apply to that tree.skills:validaterejects:skills/does not ship;classic/skills/<skill>/SKILL.mdis missing;classic/skills/<skill>that no flavor pins, so the classic tree must be deleted once Studio Web moves off it.Studio Web output is byte-identical to
main:diff -rof the whole composedbuild/skills/studiowebtree fromorigin/mainand from this branch is empty.check-skill-links.mjsnow also scansclassic/skills/, resolves pinned-skill flavor and classic-tree links against the classic tree, and requires any link from another skill into a pinned skill to resolve in both trees.Other changes (commits 2–3)
@uipath/maestro-builder-sdkis also public on npmjs.com (latest= 6.11.6 on both feeds). The skill now runs plainnpm install, which respects corporate mirrors, and retries with--@uipath:registry=https://registry.npmjs.orgonly on E401/E404 when@uipathroutes to GitHub Packages..flowfiles, IXP document extraction, inline agents, HITL, chat/voice) plus the uipath-ixp / uipath-agents negatives. 902/1024 chars.uipath-agents,uipath-human-in-the-loopanduipath-platformtaught hand-editing.flowJSON, and 11 links pointed into v1-only files.main.…/author/plugins/ixp/→…/references/ixp.md); owners added forclassic/.verb-gate.ymlalso scansclassic/skills; the 220-line router-window test and the inline-agent prompt test read the promoted tree.flow-v2-preview.yaml, previewplugin.json.Verification
npm run skills:validate: OK (default 28 skills / 1778 files; studioweb 28 / 1802 / 288 replacements).npm run skills:check-links: all links resolve (7135 links acrossskills,skill-flavors,classic/skills).npm run skills:test: 103 pass, 1 fail. The failing test (pack builds complete…, annpm publish --dry-runoutput assertion) fails the same way onorigin/mainon the dev box when run with the other suites, and passes alone 3/3. Environment, not this change.npm run skills:pack,check-skills-sh.py,check-skill-status.py(+--check-readme),validate-skill-descriptions.sh: OK.Verb gate (the CI blocking rule): 0 blocking findings in
skills/uipath-maestro-flowandclassic/skills/uipath-maestro-flow.pytest:
tests/tasks/uipath-maestro-flow1572 passed;uipath-maestro-bpmn957;uipath-maestro-case224;uipath-agents+uipath-planner111;tests/scripts283.Symlink loading: a Claude Code plugin whose
skills/<name>is a relative symlink loads that skill (probe skill returned its secret word). coder-eval's Codex and Antigravity staging follow symlinks (is_dir()+symlink_toby absolute path).Activation (local approximation of the gate). I ran all 61
tests/tasks/activation/uipath-maestro-flow.jsonlprompts throughclaude -p(claude-opus-5-5, 1 turn, only this repo's plugin loaded, user settings off).main: 43/61.mainis a gain.PR smoke gate (v2 Flow in the default catalog): passed, 95.1% (117/123). Two Flow tasks failed, both known v2 gaps:
skill-flow-solution-select-ask(0.58): the v2 skill tells the agent not to ask which solution to use. Fixed by open fix(uipath-maestro-flow/preview): look for an existing solution before scaffolding #3616.skill-flow-datafabric-smoke-update-existing-flow(0.57):flow validateof the edited seed failed. The 09-02 RCA tied this task to v2 decompile/compile handling of registry-era seeds (flow-sdk Improve generate-task skill, consolidate tag taxonomy #672 fixed; skills(activation): close coverage gaps + tighten descriptions #673 is Tao's call).CI activation gate for
uipath-maestro-flow: passed.Claude Code Review bot: errored at startup on this PR and on feat(flavors): let a flavor pin a skill to its previous generation #3625 and fix(uipath-maestro-flow/preview): install the SDK from public npm, no token #3626 (
is_error: true, modelclaude-opus-4-6, $0 cost). That's a problem in the review workflow; it produced no review.Things reviewers should know
deb7802).main→devchannel is fine today, because dev CLI builds already have the commands.MAESTRO_FLOW_SDK_SETUP=1).nightly.yamlandsmoke.yamldo not set it, so Flow tasks in this repo's CI now take the skill's ownnpm installpath from public npm. That is the customer path, and it adds install time per task.classic/. This repo's ownnightly.yamlloads the repo root, so it measures v2 Flow from here on.preview/skills/uipath-maestro-flow. Two copy it without following symlinks. Both fixes work before and after this PR, so they can merge first:coder_eval_uipatheval_runner/scripts/flow_v2/ci/ado-run.shusescp -R, so the v2 arm would silently lose Flow. Fix: UiPath/coder_eval_uipath#147 (cp -RL). Merge that first.flow-builder-sdkdocker/Dockerfileextracts onlypreview/skills/*at a pinned commit. It fails loudly on the next pin bump. Fix: UiPath/flow-builder-sdk#859 (extract both subtrees,cp -RL; also updates the promotion plan).classic/beyond the validators. v1 Flow is measured only if a Studio Web nightly exercises the composed flavor.classic/skills/uipath-maestro-flowis readable there, the same waytests/tasks/uipath-maestro-bpmn/README.mddocuments for v1 BPMN. Prompts must not name v1 Flow reference files.Follow-ups (not in this PR)
classic/can be deleted..claude/commands/test-coverage.mdstill describes v1'sreferences/plugins/layout.@astridstaicu — please review the flavor pin (commit 1). It is what keeps Studio Web on the working Flow skill.
🤖 Generated with Claude Code
https://claude.ai/code/session_01KSLJwM2pMwZa7ygePiY6mQ