@uipath/data-fabric-tool 1.198.0 → 1.199.0-preview.104

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@uipath/data-fabric-tool",
   "license": "MIT",
-  "version": "1.198.0",
+  "version": "1.199.0-preview.104",
   "description": "Manage Data Fabric entities and records.",
   "type": "module",
   "main": "./dist/tool.js",
@@ -14,5 +14,5 @@
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"
   },
-  "gitHead": "1fadf03d7a8dd102742571dff569fdac11808afb"
+  "gitHead": "829a8ab8a25bce5b62c284bd1ddc674d2947f647"
 }

package/src/commands/choice-sets.spec.ts

@@ -438,7 +438,7 @@ describe("choice-sets create", () => {
             "test",
             "choice-sets",
             "create",
-            "X",
+            "ExpenseTypes",
         ]);
         expect(OutputFormatter.error).toHaveBeenCalledWith(
             expect.objectContaining({
@@ -448,6 +448,28 @@ describe("choice-sets create", () => {
         );
         expect(process.exitCode).toBe(1);
     });
+
+    it("should reject a choice set name that fails validation", async () => {
+        const sdk = mockSdk();
+        const program = buildProgram();
+        await program.parseAsync([
+            "node",
+            "test",
+            "choice-sets",
+            "create",
+            "1BadName",
+        ]);
+        expect(sdk.entities.choicesets.create).not.toHaveBeenCalled();
+        expect(OutputFormatter.error).toHaveBeenCalledWith(
+            expect.objectContaining({
+                Result: "Failure",
+                Message: expect.stringContaining(
+                    "Invalid choice set name '1BadName'",
+                ),
+            }),
+        );
+        expect(process.exitCode).toBe(1);
+    });
 });
 
 describe("choice-sets update", () => {
@@ -456,7 +478,7 @@ describe("choice-sets update", () => {
         process.exitCode = undefined;
     });
 
-    it("should update a choice set's metadata", async () => {
+    it("should update a choice set's metadata when both flags are passed", async () => {
         const sdk = mockSdk();
         const program = buildProgram();
         await program.parseAsync([
@@ -467,11 +489,14 @@ describe("choice-sets update", () => {
             "cs-1",
             "--display-name",
             "New Name",
+            "--description",
+            "New description",
         ]);
         expect(sdk.entities.choicesets.updateById).toHaveBeenCalledWith(
             "cs-1",
             {
                 displayName: "New Name",
+                description: "New description",
             },
         );
         expect(OutputFormatter.success).toHaveBeenCalledWith(
@@ -483,7 +508,7 @@ describe("choice-sets update", () => {
         );
     });
 
-    it("should error when no update fields are provided", async () => {
+    it("should reject a partial update (only --display-name)", async () => {
         const sdk = mockSdk();
         const program = buildProgram();
         await program.parseAsync([
@@ -492,12 +517,62 @@ describe("choice-sets update", () => {
             "choice-sets",
             "update",
             "cs-1",
+            "--display-name",
+            "New Name",
         ]);
         expect(sdk.entities.choicesets.updateById).not.toHaveBeenCalled();
         expect(OutputFormatter.error).toHaveBeenCalledWith(
             expect.objectContaining({
                 Result: "Failure",
-                Message: "No update fields provided",
+                Message: expect.stringContaining(
+                    "requires both --display-name and --description",
+                ),
+            }),
+        );
+        expect(process.exitCode).toBe(1);
+    });
+
+    it("should reject a partial update (only --description)", async () => {
+        const sdk = mockSdk();
+        const program = buildProgram();
+        await program.parseAsync([
+            "node",
+            "test",
+            "choice-sets",
+            "update",
+            "cs-1",
+            "--description",
+            "New description",
+        ]);
+        expect(sdk.entities.choicesets.updateById).not.toHaveBeenCalled();
+        expect(OutputFormatter.error).toHaveBeenCalledWith(
+            expect.objectContaining({
+                Result: "Failure",
+                Message: expect.stringContaining(
+                    "requires both --display-name and --description",
+                ),
+            }),
+        );
+        expect(process.exitCode).toBe(1);
+    });
+
+    it("should error when no update flags are provided", async () => {
+        const sdk = mockSdk();
+        const program = buildProgram();
+        await program.parseAsync([
+            "node",
+            "test",
+            "choice-sets",
+            "update",
+            "cs-1",
+        ]);
+        expect(sdk.entities.choicesets.updateById).not.toHaveBeenCalled();
+        expect(OutputFormatter.error).toHaveBeenCalledWith(
+            expect.objectContaining({
+                Result: "Failure",
+                Message: expect.stringContaining(
+                    "requires both --display-name and --description",
+                ),
             }),
         );
         expect(process.exitCode).toBe(1);
@@ -515,6 +590,8 @@ describe("choice-sets update", () => {
             "choice-sets",
             "update",
             "missing",
+            "--display-name",
+            "y",
             "--description",
             "x",
         ]);
@@ -715,6 +792,29 @@ describe("choice-set-values", () => {
         expect(process.exitCode).toBe(1);
     });
 
+    it("should reject a choice-set value name that hits the keyword blocklist", async () => {
+        const sdk = mockSdk();
+        const program = buildValuesProgram();
+        await program.parseAsync([
+            "node",
+            "test",
+            "choice-set-values",
+            "create",
+            "cs-1",
+            "internal",
+        ]);
+        expect(sdk.entities.choicesets.insertValueById).not.toHaveBeenCalled();
+        expect(OutputFormatter.error).toHaveBeenCalledWith(
+            expect.objectContaining({
+                Result: "Failure",
+                Message: expect.stringContaining(
+                    "Choice-set value name 'internal' is a reserved C# keyword",
+                ),
+            }),
+        );
+        expect(process.exitCode).toBe(1);
+    });
+
     it("should update a value's display name", async () => {
         const sdk = mockSdk();
         const program = buildValuesProgram();
@@ -985,6 +1085,8 @@ describe("choice-sets --folder-key forwarding", () => {
             "cs-1",
             "--display-name",
             "New",
+            "--description",
+            "Updated description",
             "--folder-key",
             "folder-2",
         ]);
@@ -992,6 +1094,7 @@ describe("choice-sets --folder-key forwarding", () => {
             "cs-1",
             {
                 displayName: "New",
+                description: "Updated description",
                 folderKey: "folder-2",
             },
         );

package/src/commands/choice-sets.ts

@@ -21,6 +21,7 @@ import type {
 import { type Command, Option } from "commander";
 import { fail, requireDestructiveConfirmation } from "../utils/output";
 import { connectOrFail } from "../utils/sdk-client";
+import { validateChoiceValueName, validateName } from "../utils/validate";
 
 const OUTPUT_CODES = {
     ChoiceSetList: "ChoiceSetList",
@@ -365,6 +366,9 @@ export const registerChoiceSetsCommand = (program: Command) => {
         .trackedAction(
             processContext,
             async (name: string, options: CreateOptions) => {
+                const nameErr = validateName(name, "choice set");
+                if (nameErr) return fail(nameErr.message, nameErr.instructions);
+
                 const sdk = await connectOrFail(options.tenant);
                 if (!sdk) return;
 
@@ -423,13 +427,18 @@ export const registerChoiceSetsCommand = (program: Command) => {
         .trackedAction(
             processContext,
             async (choiceSetId: string, options: UpdateOptions) => {
+                // Server-side quirk: sending only --description returns
+                // "DisplayName is required."; sending only --display-name
+                // returns Internal Server Error. Both are required together.
+                // To change one, re-send the other's current value from
+                // `choice-sets list`.
                 if (
-                    options.displayName === undefined &&
+                    options.displayName === undefined ||
                     options.description === undefined
                 ) {
                     return fail(
-                        "No update fields provided",
-                        "Provide at least one of --display-name or --description.",
+                        "choice-sets update requires both --display-name and --description",
+                        "The server rejects a partial update with 'DisplayName is required.' or Internal Server Error. To change one field, run 'uip df choice-sets list' first to read the current value of the other, then re-send both with --display-name and --description.",
                     );
                 }
 
@@ -623,6 +632,9 @@ export const registerChoiceSetValuesCommand = (program: Command) => {
                 name: string,
                 options: ValueCreateOptions,
             ) => {
+                const nameErr = validateChoiceValueName(name);
+                if (nameErr) return fail(nameErr.message, nameErr.instructions);
+
                 const sdk = await connectOrFail(options.tenant);
                 if (!sdk) return;