@contrail/cli 3.6.0-alpha-8-alpha.de819d3 → 3.6.0-alpha-8-alpha.afa500a

package/README.md

@@ -17,7 +17,7 @@ $ npm install -g @contrail/cli
 $ contrail COMMAND
 running command...
 $ contrail (--version)
-@contrail/cli/3.6.0-alpha-8-alpha.de819d3 linux-x64 node-v24.21.0
+@contrail/cli/3.6.0-alpha-8-alpha.afa500a linux-x64 node-v24.21.0
 $ contrail --help [COMMAND]
 USAGE
   $ contrail COMMAND
@@ -686,7 +686,7 @@ Update an entity
 
 ```
 USAGE
-  $ contrail entities update ENTITYNAME [IDS] [-l error|warn|info|debug] [-e true|false] [-S] [-d] [-i <value>]
+  $ contrail entities update ENTITYNAME [IDS] [-l error|warn|info|debug] [-e true|false] [-S] [-d] [-i <value>] [-F]
     [--suffix <value>] [--apiVersion 1|2]
 
 ARGUMENTS
@@ -694,11 +694,12 @@ ARGUMENTS
   IDS         One or more entity ids to update
 
 FLAGS
+  -F, --force                      Force update without validation. Internal use only
   -S, --skipHydrateRelationships   Skip hydrating relationships when updating the entity
-  -d, --dryRun                     If true, does not update the entities but simulates the update
+  -d, --dryRun                     If true, does not create the entity but simulates the creation
   -e, --alwaysConfirmEnv=<option>  [default: true] The Environment variables
                                    <options: true|false>
-  -i, --inputFile=<value>          A file with a JSON array of {id, ...} objects; each object is one entity to update.
+  -i, --inputFile=<value>          File containing properties to update the entity (optional)
   -l, --logLevel=<option>          The Log Level
                                    <options: error|warn|info|debug>
   --apiVersion=<option>            Api-Version header to send; selects the V1 or V2 variant of the endpoint

package/dist/commands/entities/create.js

@@ -290,6 +290,8 @@ class CreateEntity extends basecommand_1.BaseCommand {
     }
 }
 exports.default = CreateEntity;
+// Every row is attempted; a failed row never stops the run. The summary pairs
+// each failed input with its error so the failed rows can be retried as-is.
 function summarizeSettledCreates(inputs, settled) {
     const created = [];
     const failed = [];
@@ -536,7 +538,9 @@ function castValueToPropertyType(value, propertyType) {
             if (Array.isArray(value)) {
                 return value;
             }
-            // the old comma syntax parses as one literal value; warn instead of guessing at a split
+            // Migration hazard: the old comma syntax parses as one string with a
+            // comma in it. Warn with the repeated-key equivalent instead of
+            // guessing at a split.
             if (typeof value === 'string' && value.includes(',')) {
                 console.error(`Value "${value}" for a multi-value property contains a comma and is treated as one literal value. To pass several values, repeat the key: slug=${value
                     .split(',')

package/dist/commands/entities/create.spec.js

@@ -333,6 +333,8 @@ describe('Entity Creation E2E Happy Path', () => {
                 const propertySpecs = (0, propertySpecifications_1.parsePropertySpecificationsIntoMap)([
                     'roles=option',
                 ]);
+                // a single roles value stays scalar at the parse layer and is
+                // normalized to an array where it is consumed
                 const roles = [].concat(propertySpecs.get('roles'));
                 // Test that property level matching works with specified roles
                 expect((0, create_1.doesPropertyLevelMatchRole)(types_1.PropertyLevel.OPTION, roles[0])).toBe(true);

package/dist/commands/entities/delete.js

@@ -91,8 +91,8 @@ class DeleteEntity extends basecommand_1.BaseCommand {
                 console.error('deleted entities:');
             }
             this.log(JSON.stringify(response, null, 2));
-            if (response.failed.length > 0) {
-                console.error(`${response.failed.length} of ${ids.length} deletes failed.`);
+            if (response.idsErrored.length > 0) {
+                console.error(`${response.idsErrored.length} of ${ids.length} deletes failed.`);
                 process.exitCode = 1;
             }
         }
@@ -103,26 +103,32 @@ class DeleteEntity extends basecommand_1.BaseCommand {
 }
 exports.default = DeleteEntity;
 async function deleteEntityByIds(entityName, ids, routing = {}) {
-    const deletedIds = [];
-    const notFoundIds = [];
-    const failed = [];
+    const idsDeleted = new Set();
+    const idsNotFound = new Set();
+    const idsErrored = new Set();
     const limit = (0, p_limit_1.default)(10);
     const deletePromises = ids.map((id) => limit(() => deleteEntityById(entityName, id, routing).then((entity) => {
         if (!entity) {
-            notFoundIds.push(id);
-            return;
+            idsNotFound.add(id);
+            return null;
         }
-        deletedIds.push(id);
+        idsDeleted.add(id);
+        return entity;
     }, (error) => {
         if (error.message.includes('not found')) {
-            notFoundIds.push(id);
+            idsNotFound.add(id);
         }
         else {
-            failed.push({ id, error: (0, common_1.sdkErrorMessage)(error) });
+            idsErrored.add(id);
         }
+        return null;
     })));
     await Promise.all(deletePromises);
-    return { deletedIds, notFoundIds, failed };
+    return {
+        idsDeleted: Array.from(idsDeleted),
+        idsNotFound: Array.from(idsNotFound),
+        idsErrored: Array.from(idsErrored),
+    };
 }
 async function deleteEntityById(entityName, id, routing = {}) {
     const client = new sdk_1.Entities();

package/dist/commands/entities/get.js

@@ -157,9 +157,6 @@ class GetEntity extends basecommand_1.BaseCommand {
                     apiVersion: apiVersion,
                     take,
                 });
-                if (response === null) {
-                    this.error(`Entity not found: ${propertySpecBySlug.get('id')}`, { exit: 1 });
-                }
                 if (outputFile) {
                     fs_1.default.writeFileSync(outputFile, JSON.stringify(response, null, 2));
                     console.error(`Output saved to ${outputFile}`);
@@ -170,9 +167,6 @@ class GetEntity extends basecommand_1.BaseCommand {
             }
         }
         catch (e) {
-            if (e.message?.startsWith('Entity not found:')) {
-                throw e;
-            }
             this.error(`Error fetching entity: ${e.message}`);
         }
     }
@@ -188,8 +182,6 @@ async function getEntityByTypePath(typePath, propertySpec, options = {}) {
     }
     return find(entityName, {
         id,
-        // a missing id returns null (no retries); the command turns it into a clean not-found error
-        ignore404: id !== undefined,
         skipHydrateRelationships,
         relations,
         propertySlugs,
@@ -209,10 +201,9 @@ function convertPropertySpecIntoCriteria(propertySpec) {
 }
 async function find(entityName, options = {}) {
     const client = new sdk_1.Entities();
-    const { criteria, id, ignore404, propertySlugs, skipHydrateRelationships, relations, skip, suffix, apiVersion, take, } = options;
+    const { criteria, id, propertySlugs, skipHydrateRelationships, relations, skip, suffix, apiVersion, take, } = options;
     const queryOptions = {
         entityName,
-        ignore404,
         skipHydrateRelationships,
         relations,
         skip,

package/dist/commands/entities/update.d.ts

@@ -6,6 +6,7 @@ export default class UpdateEntity extends BaseCommand {
         skipHydrateRelationships: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         dryRun: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         inputFile: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces/parser").CustomOptions>;
+        force: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
         suffix: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces/parser").CustomOptions>;
         apiVersion: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces/parser").CustomOptions>;
         alwaysConfirmEnv: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces/parser").CustomOptions>;

package/dist/commands/entities/update.js

@@ -27,13 +27,18 @@ class UpdateEntity extends basecommand_1.BaseCommand {
         }),
         dryRun: core_1.Flags.boolean({
             char: 'd',
-            description: 'If true, does not update the entities but simulates the update',
+            description: 'If true, does not create the entity but simulates the creation',
             required: false,
             default: false,
         }),
         inputFile: core_1.Flags.string({
             char: 'i',
-            description: 'A file with a JSON array of {id, ...} objects; each object is one entity to update.',
+            description: 'File containing properties to update the entity (optional)',
+            required: false,
+        }),
+        force: core_1.Flags.boolean({
+            char: 'F',
+            description: 'Force update without validation. Internal use only',
             required: false,
         }),
         suffix: core_1.Flags.string({
@@ -62,7 +67,7 @@ class UpdateEntity extends basecommand_1.BaseCommand {
         await (0, login_1.authUser)();
         const { args, argv } = await this.parse(UpdateEntity);
         const { entityName } = args;
-        const { dryRun: isDryRun, inputFile: inputFilename, skipHydrateRelationships, suffix, apiVersion, } = this.flags;
+        const { dryRun: isDryRun, inputFile: inputFilename, skipHydrateRelationships, suffix, apiVersion, force, } = this.flags;
         const { ids: positionalIds, specs } = (0, propertySpecifications_1.partitionIdsAndSpecs)(argv.slice(1));
         const propertySpec = (0, propertySpecifications_1.parsePropertySpecificationsIntoMap)(specs);
         if (propertySpec.has('id') || propertySpec.has('ids')) {
@@ -96,6 +101,7 @@ class UpdateEntity extends basecommand_1.BaseCommand {
                     skipHydrateRelationships,
                     suffix,
                     apiVersion: apiVersion,
+                    force,
                 });
         }
         try {
@@ -117,6 +123,8 @@ class UpdateEntity extends basecommand_1.BaseCommand {
     }
 }
 exports.default = UpdateEntity;
+// Every row is attempted; a failed row never stops the run. The summary pairs
+// each failed id with its error so the failed rows can be retried.
 function summarizeSettledUpdates(inputs, settled, skipped) {
     const updated = [];
     const failed = [];
@@ -136,7 +144,7 @@ function summarizeSettledUpdates(inputs, settled, skipped) {
 exports.summarizeSettledUpdates = summarizeSettledUpdates;
 async function updateEntityById(entityName, id, changes, options = {}) {
     const client = new sdk_1.Entities();
-    const { skipHydrateRelationships, suffix, apiVersion } = options;
+    const { skipHydrateRelationships, suffix, apiVersion, force } = options;
     return client.update({
         entityName,
         id,

package/dist/helpers/propertySpecifications/propertySpecifications.js

@@ -68,8 +68,7 @@ function parsePropertySpecificationsIntoMap(propertySpecifications) {
             // ids is the one comma-split exception: it is a list of ids by definition
             // and ids never contain commas.
             const idsArray = value.split(',').map((id) => id.trim());
-            const existingIds = propertySpecBySlug.get(slug);
-            propertySpecBySlug.set(slug, Array.isArray(existingIds) ? [...existingIds, ...idsArray] : idsArray);
+            propertySpecBySlug.set(slug, idsArray);
             continue;
         }
         const existing = propertySpecBySlug.get(slug);

package/dist/helpers/propertySpecifications/propertySpecifications.spec.js

@@ -190,11 +190,6 @@ describe('parsePropertySpecificationsIntoMap', () => {
         expect(result.get('ids')).toEqual(['1', '2', '3']);
         expect(result.get('name')).toBe('Test');
     });
-    it('should accumulate repeated ids keys', () => {
-        const specs = ['ids=1,2', 'ids=3'];
-        const result = (0, propertySpecifications_1.parsePropertySpecificationsIntoMap)(specs);
-        expect(result.get('ids')).toEqual(['1', '2', '3']);
-    });
     it('should handle ids as single value', () => {
         const specs = ['ids=123'];
         const result = (0, propertySpecifications_1.parsePropertySpecificationsIntoMap)(specs);

package/oclif.manifest.json

@@ -1,5 +1,5 @@
 {
-  "version": "3.6.0-alpha-8-alpha.de819d3",
+  "version": "3.6.0-alpha-8-alpha.afa500a",
   "commands": {
     "app:create": {
       "id": "app:create",
@@ -1077,7 +1077,7 @@
           "name": "dryRun",
           "type": "boolean",
           "char": "d",
-          "description": "If true, does not update the entities but simulates the update",
+          "description": "If true, does not create the entity but simulates the creation",
           "required": false,
           "allowNo": false
         },
@@ -1085,10 +1085,18 @@
           "name": "inputFile",
           "type": "option",
           "char": "i",
-          "description": "A file with a JSON array of {id, ...} objects; each object is one entity to update.",
+          "description": "File containing properties to update the entity (optional)",
           "required": false,
           "multiple": false
         },
+        "force": {
+          "name": "force",
+          "type": "boolean",
+          "char": "F",
+          "description": "Force update without validation. Internal use only",
+          "required": false,
+          "allowNo": false
+        },
         "suffix": {
           "name": "suffix",
           "type": "option",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/cli",
-  "version": "3.6.0-alpha-8-alpha.de819d3",
+  "version": "3.6.0-alpha-8-alpha.afa500a",
   "description": "VibeIQ's Contrail Platform CLI.",
   "bin": {
     "contrail-cli": "./bin/run",