@uipath/skills 1.199.0 → 1.200.0

package/skills/uipath-maestro-bpmn/validator/validate-bpmn.mjs

@@ -1,164 +0,0 @@
-#!/usr/bin/env node
-// Offline semantic validator for UiPath Maestro BPMN XML.
-//
-// Pipeline:
-//   1. Parse the BPMN with bpmn-moddle using the UiPath extension descriptor
-//      (same getModdle() + fromXML() pattern as PO.Frontend).
-//   2. Reconstruct the PO.Frontend Node[]/Edge[]/CanvasState model (model.mjs).
-//   3. Run ALL 19 PO.Frontend validation rules (rules.mjs) per diagram, plus a
-//      whole-model variable-existence check.
-//   4. (Optional) Run the Maestro-original connection-liveness ping via uip CLI.
-//
-// Prints `VALID` and exits 0 when there are no ERROR-severity findings.
-// Otherwise lists every finding (rule code + message) and exits non-zero.
-// WARNING-severity findings are printed but do not gate.
-//
-// Usage: node validate-bpmn.mjs <bpmn-file> [resources]
-//   resources: comma-separated release names, or a numeric folder ID, for the
-//              optional solution-resource liveness ping (requires uip CLI auth).
-import BpmnModdle from "bpmn-moddle";
-import sax from "sax";
-import descriptor from "./uipath-moddle.v1.json" with { type: "json" };
-import { readFileSync, existsSync } from "fs";
-import { execFileSync } from "child_process";
-import { buildModel, collectKnownVariableIds, allNodes, allEdges } from "./model.mjs";
-import { validateDiagram, validateVariableExistence, validateVariableNotSet, SEVERITY } from "./rules.mjs";
-
-// Strict XML well-formedness check via sax's strict mode. Returns the first
-// error message (with a line number) or null when the document is well-formed.
-function strictXmlError(xml) {
-  const parser = sax.parser(true, {});
-  let error = null;
-  parser.onerror = (e) => {
-    if (!error) {
-      const line = (parser.line ?? 0) + 1;
-      error = `${e.message.split("\n")[0]} (near line ${line})`;
-    }
-  };
-  try {
-    parser.write(xml).close();
-  } catch (e) {
-    if (!error) error = String(e.message ?? e).split("\n")[0];
-  }
-  return error;
-}
-
-// --- uip CLI resolution (cached) — used only for the liveness-ping extra. ---
-let _uipBin;
-function resolveUipBin() {
-  if (_uipBin !== undefined) return _uipBin;
-  const home = process.env.HOME || process.env.USERPROFILE || "";
-  const candidates = ["uip", `${home}/.bun/bin/uip`, `${home}/.local/bin/uip`, "/usr/local/bin/uip"];
-  for (const p of candidates) {
-    try {
-      if ((p.startsWith("/") || p.startsWith(home)) && !existsSync(p)) continue;
-      execFileSync(p, ["--version"], { encoding: "utf8", timeout: 5000, stdio: "pipe" });
-      _uipBin = p;
-      return p;
-    } catch {
-      /* keep trying */
-    }
-  }
-  _uipBin = null;
-  return null;
-}
-
-function pingConnections(resourceArg) {
-  // Maestro-original extra: liveness-check bound IS connections. Best-effort;
-  // a missing/unauthenticated CLI is reported as a NOTE, never a hard failure.
-  const bin = resolveUipBin();
-  if (!bin) return { note: "uip CLI not found; skipped connection liveness ping." };
-  try {
-    const args = ["is", "connections", "ping", "--output", "json"];
-    if (/^\d+$/.test(resourceArg)) args.push("--folder", resourceArg);
-    const out = execFileSync(bin, args, { encoding: "utf8", timeout: 30000, stdio: "pipe" });
-    return { ok: true, raw: out.trim() };
-  } catch (e) {
-    return { note: `connection liveness ping unavailable: ${String(e.message || e).split("\n")[0]}` };
-  }
-}
-
-const file = process.argv[2];
-const resourceArg = process.argv[3];
-if (!file) {
-  console.error("Usage: node validate-bpmn.mjs <bpmn-file> [resources]");
-  process.exit(1);
-}
-
-const xml = readFileSync(file, "utf8");
-
-// Strict XML well-formedness gate. bpmn-moddle's SAX parser (saxen) is lenient:
-// it accepts unescaped `&`/`<`, `--` inside comments, and other malformed input
-// that strict parsers — and the Studio Web canvas import — reject. A validator
-// that is more permissive than the real import gives false confidence, so parse
-// strictly first and refuse anything the import would refuse.
-{
-  const err = strictXmlError(xml);
-  if (err) {
-    console.error(`WELL-FORMEDNESS ERROR: ${err}`);
-    process.exit(2);
-  }
-}
-
-const moddle = new BpmnModdle({ uipath: descriptor });
-
-let definitions;
-try {
-  const result = await moddle.fromXML(xml);
-  definitions = result.rootElement;
-  if (result.warnings?.length) {
-    for (const w of result.warnings) console.error(`PARSE WARNING: ${w.message ?? w}`);
-  }
-} catch (e) {
-  console.error(`SCHEMA ERROR: failed to parse BPMN XML: ${e.message ?? e}`);
-  process.exit(2);
-}
-
-// Build the model and run all rules.
-const canvasState = buildModel(definitions);
-const knownIds = collectKnownVariableIds(canvasState);
-
-const findings = [];
-for (const diagram of Object.values(canvasState.diagramsById)) {
-  // Skip the trivial root-only "single blank start, no edges" diagram, mirroring
-  // ValidateBpmnFlowUtils.validateBpmnFlow.
-  const realNodes = diagram.nodes.filter((n) => n.type !== "bpmn:Participant");
-  if (
-    diagram.diagramId === canvasState.rootDiagramId &&
-    realNodes.length === 1 &&
-    realNodes[0].type === "bpmn:StartEvent" &&
-    diagram.edges.length === 0
-  ) {
-    continue;
-  }
-  findings.push(...validateDiagram(diagram, canvasState, { enableMissingRootVariableValidation: true }));
-}
-findings.push(...validateVariableExistence(allNodes(canvasState), allEdges(canvasState), knownIds));
-findings.push(...validateVariableNotSet(allNodes(canvasState), allEdges(canvasState), knownIds, canvasState));
-
-// Optional connection liveness ping (Maestro-original extra).
-if (resourceArg) {
-  const ping = pingConnections(resourceArg);
-  if (ping.note) console.error(`NOTE: ${ping.note}`);
-  else if (ping.ok) console.error(`connection liveness ping ok`);
-}
-
-const errors = findings.filter((f) => f.severity === SEVERITY.ERROR);
-const warnings = findings.filter((f) => f.severity === SEVERITY.WARNING);
-
-const fmt = (f) => `  [${f.severity}] ${f.code}: ${f.message}${f.elementId ? ` (element: ${f.elementId})` : ""}`;
-
-if (warnings.length) {
-  console.error("WARNINGS:");
-  for (const w of warnings) console.error(fmt(w));
-}
-
-if (errors.length === 0) {
-  console.log("VALID");
-  process.exit(0);
-}
-
-console.error("ERRORS:");
-for (const e of errors) console.error(fmt(e));
-console.error(`\n${errors.length} blocking error(s), ${warnings.length} warning(s).`);
-process.exit(1);

package/skills/uipath-review/references/agents/agents-common-rules.md

@@ -1,32 +0,0 @@
-# Agents — Common Rules (both formats)
-
-Judgment rules that apply to **both** coded and low-code agents. Read [`../rule-format.md`](../rule-format.md) and [`../rule-catalog-workflow.md`](../rule-catalog-workflow.md) first.
-
-> **This catalog is judgment-only.** Deterministic agent checks (eval counts, tool counts, schema-property presence, set-membership, regex, run-artifact analysis) run in the `uip agent review` / `uip codedagent review` CLI, which the reviewer invokes **first** (SKILL.md Step 2.5). The CLI returns those findings in the same rule format. This file holds only rules that require an LLM to read source and reason — what a regex or count cannot decide reliably.
-
-Companion files:
-
-- [`agents-lowcode-rules.md`](agents-lowcode-rules.md) — low-code judgment rules
-- [`agents-coded-rules.md`](agents-coded-rules.md) — coded judgment rules
-
----
-
-## SchemaChecker
-
-| rule_id | severity | category | trigger | detection_method | suggested_fix |
-|---|---|---|---|---|---|
-| `SCHEMA_NO_DESCRIPTIONS` | judgment | schema | Input/output schema properties lack **informative** descriptions | Locate the schema (see [Schema location](#schema-location)). Read each property's `description`. Assess — not mere presence: is each description informative enough for the LLM to populate / consume the field correctly and for Studio Web to render a useful hint? A field named `id` described as `"the id"` adds no signal; a missing description on a non-obvious field is worse. Emit when a meaningful share of properties are undescribed or described with content that restates the field name. file = schema source. (The deterministic `>50%-missing` count runs in the review CLI; this rule judges informativeness.) | Add a `description` to each property that states what it holds, valid values, and format — Studio Web shows these as field hints and the LLM uses them as semantic signals. |
-
-### Schema location
-
-- **Coded** — Read `entry-points.json` if present; else `uipath.json` → `.entryPoints[0].input` / `.entryPoints[0].output`. The finding's `file` points to whichever source was read.
-- **Low-code normalized** — Read the normalized JSON. Use `.input_schema` and `.output_schema`.
-- **Low-code agent-builder** — Read `entry-points.json`. Use `.entryPoints[0].input` and `.entryPoints[0].output`.
-
-Assess input and output schema independently — emit a separate finding per schema when each is weak.
-
----
-
-## What this catalog cannot do
-
-The agent applies these rules from the project source as it is *checked into the repo*. It cannot verify runtime behaviour (whether the LLM follows the prompt, whether tool selection is correct, whether guardrails fire) or validate live IDs (connection IDs, index names, recipients) — those require running the agent or calling APIs.

package/skills/uipath-rpa/assets/before-after-hooks-template.md

@@ -1,115 +0,0 @@
-# Before/After Hooks Template
-
-## IBeforeAfterRun on Individual Workflow/Test Case
-
-**File: `TestLoginFlow.cs`**
-
-```csharp
-using UiPath.CodedWorkflows;
-
-namespace {{PROJECT_NAME}}
-{
-    public class TestLoginFlow : CodedWorkflow, IBeforeAfterRun
-    {
-        public void Before(BeforeRunContext context)
-        {
-            Log($"[BEFORE] Starting {context.RelativeFilePath}");
-            // Open browser, navigate to login page
-        }
-
-        public void After(AfterRunContext context)
-        {
-            Log($"[AFTER] Finished {context.RelativeFilePath}");
-            // Close browser, clean up
-        }
-
-        [TestCase]
-        public void Execute()
-        {
-            // Before() has already run
-
-            // Arrange
-            string username = "testuser";
-
-            // Act
-            var result = workflows.Login(username: username, password: "pass123");
-
-            // Assert
-            testing.VerifyExpression(result.success, "Login should succeed");
-
-            // After() will run automatically
-        }
-    }
-}
-```
-
-## Partial Class CodedWorkflow — Hooks for ALL Files
-
-**File: `CodedWorkflowHooks.cs`** (Coded Source File — NOT a workflow, no entry point)
-
-```csharp
-using UiPath.CodedWorkflows;
-
-namespace {{PROJECT_NAME}}
-{
-    public partial class CodedWorkflow : IBeforeAfterRun
-    {
-        public void Before(BeforeRunContext context)
-        {
-            Log($"[BEFORE] Execution started for {context.RelativeFilePath}");
-
-            // Example: Open application
-            // var app = uiAutomation.Open("myApp");
-
-            // Example: Log in
-            // Login("testuser", "password");
-        }
-
-        public void After(AfterRunContext context)
-        {
-            Log($"[AFTER] Execution finished for {context.RelativeFilePath}");
-
-            // Example: Close application
-            // uiAutomation.Close(app);
-
-            // Example: Clean up test data
-            // DeleteTestData();
-        }
-    }
-}
-```
-
-## Partial Class CodedWorkflow — Shared Logic (Without Hooks)
-
-**File: `CodedWorkflowExtensions.cs`** (Coded Source File)
-
-```csharp
-using UiPath.CodedWorkflows;
-
-namespace {{PROJECT_NAME}}
-{
-    public partial class CodedWorkflow
-    {
-        // Shared helper available in all workflows and test cases
-        protected string GetEnvironmentUrl()
-        {
-            var env = system.GetAsset("Environment").ToString();
-            return env == "prod" ? "https://app.example.com" : "https://staging.example.com";
-        }
-
-        // Shared constant
-        protected const int MaxRetries = 3;
-    }
-}
-```
-
-## Usage in Any Workflow
-
-```csharp
-[Workflow]
-public void Execute()
-{
-    string url = GetEnvironmentUrl();  // available via partial class
-    Log($"Using environment: {url}");
-}
-```

package/skills/uipath-rpa/assets/helper-utility-template.md

@@ -1,22 +0,0 @@
-# UiPath Coded Helper/Utility Classes Templates
-
-Ready-to-use templates for UiPath coded class files. Replace placeholders in `{{PLACEHOLDER}}` format.
-
----
-
-## Helper/Utility Class (.cs) — No Attribute
-
-```csharp
-using System;
-using System.Collections.Generic;
-
-namespace {{PROJECT_NAME}}
-{
-    public class {{CLASS_NAME}}
-    {
-        {{IMPLEMENTATION}}
-    }
-}
-```
-
-IMPORTANT!: Helper classes do NOT inherit from `CodedWorkflow`, do NOT have `[Workflow]` or `[TestCase]` attributes. They are NOT entry points and do not appear in the fileInfoCollection attribute from `project.json`.

package/skills/uipath-rpa/assets/testcase-template.md

@@ -1,92 +0,0 @@
-# UiPath Coded Test Cases Templates
-
-Ready-to-use templates for UiPath coded test cases files. Replace placeholders in `{{PLACEHOLDER}}` format.
-
-> **Using statements:** These templates include only the minimal required usings. Add service-specific usings based on actual usage — see [coding-guidelines.md](../references/coded/coding-guidelines.md) for the full mapping.
-
----
-
-## Coded Test Case (.cs) — Basic
-
-```csharp
-using System;
-using System.Collections.Generic;
-using UiPath.CodedWorkflows;
-// Add service-specific usings as needed — see references/coding-guidelines.md
-
-namespace {{PROJECT_NAME}}
-{
-    public class {{CLASS_NAME}} : CodedWorkflow
-    {
-        [TestCase]
-        public void Execute()
-        {
-            // Arrange
-            {{ARRANGE}}
-
-            // Act
-            {{ACT}}
-
-            // Assert
-            testing.VerifyExpression({{ASSERTION}});
-        }
-    }
-}
-```
-
-## Coded Test Case (.cs) — Data-Driven with Default Parameters
-
-```csharp
-using System;
-using System.Collections.Generic;
-using UiPath.CodedWorkflows;
-// Add service-specific usings as needed — see references/coding-guidelines.md
-
-namespace {{PROJECT_NAME}}
-{
-    public class {{CLASS_NAME}} : CodedWorkflow
-    {
-        [TestCase]
-        public void Execute(System.String {{paramName}} = "{{defaultValue}}")
-        {
-            // Arrange
-            {{ARRANGE}}
-
-            // Act
-            {{ACT}}
-
-            // Assert
-            testing.VerifyExpression({{ASSERTION}});
-        }
-    }
-}
-```
-
-## Coded Test Case (.cs) — Data-Driven with Test Data Queue
-
-```csharp
-using System;
-using System.Collections.Generic;
-using UiPath.CodedWorkflows;
-// Add service-specific usings as needed — see references/coding-guidelines.md
-
-namespace {{PROJECT_NAME}}
-{
-    public class {{CLASS_NAME}} : CodedWorkflow
-    {
-        [TestCase]
-        public void Execute()
-        {
-            // Arrange — get test data from queue
-            var item = testing.GetTestDataQueueItem("{{QUEUE_NAME}}");
-            {{EXTRACT_FIELDS}}
-
-            // Act
-            {{ACT}}
-
-            // Assert
-            testing.VerifyExpression({{ASSERTION}});
-        }
-    }
-}
-```