@contrail/cli 3.6.0-alpha-8-alpha.6d0c626 → 3.6.0-alpha-8-alpha.e4ae6bd

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.6d0c626 linux-x64 node-v24.20.0
+@contrail/cli/3.6.0-alpha-8-alpha.e4ae6bd linux-x64 node-v24.21.0
 $ contrail --help [COMMAND]
 USAGE
   $ contrail COMMAND
@@ -52,9 +52,9 @@ To get instructions on how to set up autocomplete for your shell, run `contrail
 * [`contrail config set`](#contrail-config-set)
 * [`contrail config use [NAME]`](#contrail-config-use-name)
 * [`contrail entities create PATH`](#contrail-entities-create-path)
-* [`contrail entities delete ENTITYNAME`](#contrail-entities-delete-entityname)
-* [`contrail entities get TYPEPATH`](#contrail-entities-get-typepath)
-* [`contrail entities update ENTITYNAME [ID]`](#contrail-entities-update-entityname-id)
+* [`contrail entities delete ENTITYNAME [IDS]`](#contrail-entities-delete-entityname-ids)
+* [`contrail entities get TYPEPATH [IDS]`](#contrail-entities-get-typepath-ids)
+* [`contrail entities update ENTITYNAME [IDS]`](#contrail-entities-update-entityname-ids)
 * [`contrail file upload FILELOCATION CONTENTTYPE`](#contrail-file-upload-filelocation-contenttype)
 * [`contrail groups export`](#contrail-groups-export)
 * [`contrail groups import GROUPSFILE`](#contrail-groups-import-groupsfile)
@@ -548,12 +548,13 @@ EXAMPLES
 ```
 USAGE
   $ contrail entities create PATH [-l error|warn|info|debug] [-e true|false] [-n <value>] [-d] [-i <value>] [-o
-    <value>] [-g] [-r] [-s]
+    <value>] [-g] [-r] [-S]
 
 ARGUMENTS
   PATH  Path of the entity type to create
 
 FLAGS
+  -S, --skipHydrateRelationships   If true, does not hydrate relationships for the created entities
   -d, --dryRun                     If true, does not create the entity but simulates the creation
   -e, --alwaysConfirmEnv=<option>  [default: true] The Environment variables
                                    <options: true|false>
@@ -567,7 +568,6 @@ FLAGS
   -n, --count=<value>              Number of random entities to create
   -o, --outputFile=<value>         File to save the output (optional)
   -r, --readableIds                If true, uses readable IDs for the created entities
-  -s, --skipHydrateRelationships   If true, does not hydrate relationships for the created entities
 
 DESCRIPTION
   (BETA) Create entities
@@ -592,46 +592,50 @@ EXAMPLES
   $ contrail entities create color -o /tmp/colors.json -- name=Blue green=1 red=2 blue=3 # creates a color and saves the output to /tmp/colors.json
 ```
 
-## `contrail entities delete ENTITYNAME`
+## `contrail entities delete ENTITYNAME [IDS]`
 
-(BETA) Delete an entity
+(BETA) Delete entities by id
 
 ```
 USAGE
-  $ contrail entities delete ENTITYNAME [-l error|warn|info|debug] [-e true|false] [-d] [-i <value>] [--id <value>]
+  $ contrail entities delete ENTITYNAME [IDS] [-l error|warn|info|debug] [-e true|false] [-d] [-i <value>]
 
 ARGUMENTS
   ENTITYNAME  The name of the entity type to delete
+  IDS         One or more entity ids to delete
 
 FLAGS
-  -d, --dryRun                     If true, does not delete the entity but simulates the deletion
+  -d, --dryRun                     If true, does not delete the entities but simulates the deletion
   -e, --alwaysConfirmEnv=<option>  [default: true] The Environment variables
                                    <options: true|false>
-  -i, --input=<value>              A JSON array of objects where each object is one entity to create. Cannot be used
-                                   with --count. Unless --precise is specified, any missing values will be populated
-                                   with random values.
+  -i, --inputFile=<value>          A file with a JSON array of objects; only each object's id is used. Deletes every
+                                   entity in the file.
   -l, --logLevel=<option>          The Log Level
                                    <options: error|warn|info|debug>
-  --id=<value>                     The unique identifier of the entity
 
 DESCRIPTION
-  (BETA) Delete an entity
+  (BETA) Delete entities by id
 
 EXAMPLES
   $ contrail entities delete color 123
+
+  $ contrail entities delete color id1 id2 id3
+
+  $ contrail entities delete item -i ids.json
 ```
 
-## `contrail entities get TYPEPATH`
+## `contrail entities get TYPEPATH [IDS]`
 
 (BETA) Get an entity
 
 ```
 USAGE
-  $ contrail entities get TYPEPATH [-l error|warn|info|debug] [-e true|false] [-o <value>] [-p <value>] [-P] [-s
-    <value>] [-S] [-t <value>] [-r <value>]
+  $ contrail entities get TYPEPATH [IDS] [-l error|warn|info|debug] [-e true|false] [-o <value>] [-p <value>] [-P]
+    [-s <value>] [-S] [-t <value>] [-r <value>]
 
 ARGUMENTS
   TYPEPATH  The name of the entity type to fetch
+  IDS       One or more entity ids; shorthand for the id/ids criteria
 
 FLAGS
   -P, --paginated                  If true, fetches all entities in a paginated manner
@@ -650,6 +654,10 @@ DESCRIPTION
   (BETA) Get an entity
 
 EXAMPLES
+  $ contrail entities get color 123
+
+  $ contrail entities get color id1 id2 id3
+
   $ contrail entities get color -- id=123
 
   $ contrail entities get loader-processes -- createdOn=2025-09-15..2025-09-16
@@ -659,27 +667,27 @@ EXAMPLES
   $ contrail entities get events -- timestamp=..2025-09-16T23:59:59Z
 ```
 
-## `contrail entities update ENTITYNAME [ID]`
+## `contrail entities update ENTITYNAME [IDS]`
 
 (BETA) Update an entity
 
 ```
 USAGE
-  $ contrail entities update ENTITYNAME [ID] [-l error|warn|info|debug] [-e true|false] [-s] [-d] [-i <value>] [-F]
+  $ contrail entities update ENTITYNAME [IDS] [-l error|warn|info|debug] [-e true|false] [-S] [-d] [-i <value>] [-F]
 
 ARGUMENTS
   ENTITYNAME  The name of the entity type to update
-  ID          The unique identifier of the entity
+  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 create the entity but simulates the creation
   -e, --alwaysConfirmEnv=<option>  [default: true] The Environment variables
                                    <options: true|false>
   -i, --inputFile=<value>          File containing properties to update the entity (optional)
   -l, --logLevel=<option>          The Log Level
                                    <options: error|warn|info|debug>
-  -s, --skipHydrateRelationships   Skip hydrating relationships when updating the entity
 
 DESCRIPTION
   (BETA) Update an entity
@@ -687,6 +695,8 @@ DESCRIPTION
 EXAMPLES
   $ contrail entities update color 123 -- name=obsidian hex=000000
 
+  $ contrail entities update item id1 id2 id3 -- status=archived # apply the same change to several entities
+
   $ contrail entities update color 123 -- description= notes='""' # Unset description, set notes to empty string
 ```
 

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

@@ -1,4 +1,4 @@
-import { PropertyLevel } from '@contrail/types';
+import { PropertyLevel, PropertyType } from '@contrail/types';
 import { ColonDelimited, TypeProperty as ExtendedTypeProperty, TypeReferencesCacheByPath } from '../../api/types';
 import { TypeWithProperties } from '../../api/types';
 import { BaseCommand } from '../../basecommand';
@@ -23,6 +23,14 @@ export default class CreateEntity extends BaseCommand {
     static strict: boolean;
     run(): Promise<void>;
 }
+export type CreateSummary = {
+    created: any[];
+    failed: {
+        input: any;
+        error: string;
+    }[];
+};
+export declare function summarizeSettledCreates(inputs: any[], settled: PromiseSettledResult<any>[]): CreateSummary;
 export type WithId = {
     id: string;
     [key: string]: any;
@@ -30,6 +38,7 @@ export type WithId = {
 export type ObjectReferenceCache = Map<string, WithId>;
 type ObjectReferenceCachesByTypePath = Map<string, ObjectReferenceCache>;
 export declare function pickRandomValueForProperty(property: ExtendedTypeProperty, objectReferencesCache: ObjectReferenceCachesByTypePath, typeReferencesCache: TypeReferencesCacheByPath): number | string | string[] | boolean | null;
+export declare function castValueToPropertyType(value: string | number | string[] | number[], propertyType: PropertyType): any;
 export declare function getReferencedObjectPaths(type: TypeWithProperties): ColonDelimited[];
 export declare function buildObjectReferenceCache(entities: any[]): ObjectReferenceCache;
 export declare function selectRandomObject(cache: ObjectReferenceCache): WithId;

package/dist/commands/entities/create.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.doesPropertyLevelMatchRole = exports.selectRandomObject = exports.buildObjectReferenceCache = exports.getReferencedObjectPaths = exports.pickRandomValueForProperty = void 0;
+exports.doesPropertyLevelMatchRole = exports.selectRandomObject = exports.buildObjectReferenceCache = exports.getReferencedObjectPaths = exports.castValueToPropertyType = exports.pickRandomValueForProperty = exports.summarizeSettledCreates = void 0;
 const tslib_1 = require("tslib");
 const fs_1 = tslib_1.__importDefault(require("fs"));
 const unique_names_generator_1 = require("unique-names-generator");
@@ -10,6 +10,7 @@ const types_2 = require("../../api/types");
 const core_1 = require("@oclif/core");
 const types_3 = require("../../api/types");
 const login_1 = require("../../auth/login");
+const common_1 = require("../../common");
 const basecommand_1 = require("../../basecommand");
 const sdk_1 = require("@contrail/sdk");
 const entities_1 = require("../../helpers/entities");
@@ -26,7 +27,7 @@ const baseValidationMap = {
     itemId: async (_slug, itemId) => (0, entities_1.throwIfInvalidItemId)(itemId),
     projectId: async (_slug, projectId) => (0, entities_1.throwIfInvalidProjectId)(projectId),
 };
-const throwIfInvalidPropertyLevelFn = async (_slug, maybeRoles) => maybeRoles.forEach((maybeRole) => (0, entities_1.throwIfInvalidItemLevel)(maybeRole));
+const throwIfInvalidPropertyLevelFn = async (_slug, maybeRoles) => [].concat(maybeRoles).forEach((maybeRole) => (0, entities_1.throwIfInvalidItemLevel)(maybeRole));
 const throwIfInvalidUserOrgRoleFn = async (_slug, maybeRole) => {
     if (maybeRole !== 'MEMBER' && maybeRole !== 'ADMIN') {
         throw new Error(`${maybeRole} is not a valid User Org Role`); // TODO: get these roles from a shared library
@@ -194,7 +195,7 @@ class CreateEntity extends basecommand_1.BaseCommand {
             default: false,
         }),
         skipHydrateRelationships: core_1.Flags.boolean({
-            char: 's',
+            char: 'S',
             description: 'If true, does not hydrate relationships for the created entities',
             required: false,
             default: false,
@@ -209,7 +210,8 @@ class CreateEntity extends basecommand_1.BaseCommand {
     };
     static strict = false;
     async run() {
-        this.log('🚨 This command is in BETA and may change or be removed at any time.');
+        (0, common_1.routeConsoleLogToStderr)();
+        console.error('🚨 This command is in BETA and may change or be removed at any time.');
         await (0, login_1.authUser)();
         const { flags, argv, args } = await this.parse(CreateEntity);
         const { count, dryRun: isDryRun, inputFile: inputFilename, outputFile: outputFilename, generateValues: shouldGenerateMissingValues, readableIds: shouldUseReadableIds, skipHydrateRelationships, } = flags;
@@ -221,10 +223,11 @@ class CreateEntity extends basecommand_1.BaseCommand {
         SHOULD_USE_READABLE_IDS = shouldUseReadableIds;
         SHOULD_SKIP_HYDRATE_RELATIONSHIPS = skipHydrateRelationships;
         SHOULD_GENERATE_MISSING_VALUES = shouldGenerateMissingValues;
-        const propertySpecificationArgs = argv
-            .slice(1, argv.length)
-            .filter(propertySpecifications_1.isValidPropertySpecification);
-        const propertySpecBySlug = (0, propertySpecifications_1.parsePropertySpecificationsIntoMap)(propertySpecificationArgs);
+        const { ids: strayTokens, specs } = (0, propertySpecifications_1.partitionIdsAndSpecs)(argv.slice(1));
+        if (strayTokens.length > 0) {
+            this.error(`create takes no positional ids (got: ${strayTokens.join(' ')}); did you forget -- before property values?`);
+        }
+        const propertySpecBySlug = (0, propertySpecifications_1.parsePropertySpecificationsIntoMap)(specs);
         // Validate args
         if (!entityTypePath) {
             this.error('You must specify an entity type path.');
@@ -257,13 +260,17 @@ class CreateEntity extends basecommand_1.BaseCommand {
         }
         try {
             const entitiesToCreate = await buildEntitiesToCreate(entityTypePath, partialEntities, propertySpecBySlug);
-            const createdEntities = await createEntities(entityName, entitiesToCreate);
+            const summary = await createEntities(entityName, entitiesToCreate);
             if (outputFilename) {
-                fs_1.default.writeFileSync(outputFilename, JSON.stringify(createdEntities, null, 2));
-                this.log(`Output saved to ${outputFilename}`);
+                fs_1.default.writeFileSync(outputFilename, JSON.stringify(summary, null, 2));
+                console.error(`Output saved to ${outputFilename}`);
             }
             else {
-                console.log(`Created ${entityTypePath}(s), Entity: ${JSON.stringify(createdEntities, null, 2)}`);
+                this.log(JSON.stringify(summary, null, 2));
+            }
+            if (summary.failed.length > 0) {
+                console.error(`${summary.failed.length} of ${entitiesToCreate.length} entities failed.`);
+                process.exitCode = 1;
             }
         }
         catch (error) {
@@ -272,15 +279,30 @@ 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 = [];
+    settled.forEach((result, index) => {
+        if (result.status === 'fulfilled') {
+            created.push(result.value);
+        }
+        else {
+            failed.push({ input: inputs[index], error: (0, common_1.sdkErrorMessage)(result.reason) });
+        }
+    });
+    return { created, failed };
+}
+exports.summarizeSettledCreates = summarizeSettledCreates;
 async function createEntities(entityName, entities) {
-    const limit = (0, p_limit_1.default)(20);
-    const createdEntities = IS_DRY_RUN
-        ? entities
-        : await Promise.allSettled(entities.map(async (e) => limit(() => createEntity(entityName, e))));
     if (IS_DRY_RUN) {
-        console.log(`Dry run mode: not creating entities, but simulating creation for ${entityName}`);
+        console.error(`Dry run mode: not creating entities, but simulating creation for ${entityName}`);
+        return { created: entities, failed: [] };
     }
-    return createdEntities;
+    const limit = (0, p_limit_1.default)(20);
+    const settled = await Promise.allSettled(entities.map(async (e) => limit(() => createEntity(entityName, e))));
+    return summarizeSettledCreates(entities, settled);
 }
 async function buildEntitiesToCreate(entityTypePath, entities, propertySpecificationBySlug) {
     let type = await (0, types_3.getTypeByPath)(entityTypePath);
@@ -303,7 +325,7 @@ async function buildEntitiesToCreate(entityTypePath, entities, propertySpecifica
     await Promise.all(populatedEntities.map(async () => limit(() => throwIfEntityIsInvalid)));
     // Skip system value generation unless -g flag is set (generateValues mode)
     if (!SHOULD_GENERATE_MISSING_VALUES) {
-        console.log('Using only specified properties. Use --generate-values to auto-generate missing values.');
+        console.error('Using only specified properties. Use --generate-values to auto-generate missing values.');
         return populatedEntities;
     }
     // Add system values if needed
@@ -312,13 +334,13 @@ async function buildEntitiesToCreate(entityTypePath, entities, propertySpecifica
     for (const { slug, findValidValuesFn, applyValueToEntityFn } of valueFinderApplierForEntity) {
         // If the property is already specified in command-line args, skip it
         if (propertySpecificationBySlug?.has(slug)) {
-            console.log(`Skipping property ${slug} because it is already specified.`);
+            console.error(`Skipping property ${slug} because it is already specified.`);
             continue;
         }
         // If all entities already have this property, skip finding values
         const allEntitiesHaveProperty = populatedEntities.every((entity) => entity[slug] !== undefined && entity[slug] !== null);
         if (allEntitiesHaveProperty) {
-            console.log(`Skipping property ${slug} because all entities already have this value.`);
+            console.error(`Skipping property ${slug} because all entities already have this value.`);
             continue;
         }
         const values = await findValidValuesFn(populatedEntities, entities.length);
@@ -332,7 +354,8 @@ async function buildEntitiesToCreate(entityTypePath, entities, propertySpecifica
     return populatedEntities;
 }
 function createEntityWithPopulatedValues(type, typeReferencesCache, objectReferencesCache, propertySpecificationBySlug) {
-    const roles = propertySpecificationBySlug?.get('roles');
+    const rolesValue = propertySpecificationBySlug?.get('roles');
+    const roles = rolesValue === undefined ? undefined : [].concat(rolesValue);
     const entity = SHOULD_GENERATE_MISSING_VALUES
         ? pickRandomPropertyValues(type, objectReferencesCache, typeReferencesCache, roles)
         : {};
@@ -348,6 +371,16 @@ function createEntityWithPopulatedValues(type, typeReferencesCache, objectRefere
     if (SHOULD_USE_READABLE_IDS) {
         entity.specifiedId = generateRandomReadableString();
     }
+    // roles is a multi-value system property with no type property to drive
+    // coercion; a single value still has to reach the API as an array
+    if (typeof entity.roles === 'string') {
+        if (entity.roles.includes(',')) {
+            console.error(`roles value "${entity.roles}" contains a comma and is treated as one literal value. To pass several roles, repeat the key: roles=${entity.roles
+                .split(',')
+                .join(' roles=')}`);
+        }
+        entity.roles = [entity.roles];
+    }
     // Support specifiedId
     if (propertySpecificationBySlug?.has('specifiedId')) {
         const specifiedId = propertySpecificationBySlug.get('specifiedId');
@@ -490,10 +523,24 @@ function castValueToPropertyType(value, propertyType) {
             return numValue;
         case types_1.PropertyType.Date:
             return new Date(value.toString()).toISOString();
+        case types_1.PropertyType.MultiSelect:
+            if (Array.isArray(value)) {
+                return value;
+            }
+            // 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(',')
+                    .join(' slug=')}`);
+            }
+            return [value];
         default:
             return value; // For string and other types, return as is
     }
 }
+exports.castValueToPropertyType = castValueToPropertyType;
 async function createEntity(entityName, entity) {
     const client = new sdk_1.Entities();
     const skipHydrateRelationships = SHOULD_SKIP_HYDRATE_RELATIONSHIPS;

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

@@ -270,8 +270,8 @@ describe('Entity Creation E2E Happy Path', () => {
                 expect(result.get('quantity')).toBe('10');
                 expect(result.get('isActive')).toBe('true');
             });
-            it('should handle roles as comma-separated array', () => {
-                const specs = ['roles=option,family', 'name=Test'];
+            it('should build roles arrays from repeated keys', () => {
+                const specs = ['roles=option', 'roles=family', 'name=Test'];
                 const result = (0, propertySpecifications_1.parsePropertySpecificationsIntoMap)(specs);
                 expect(result.get('roles')).toEqual(['option', 'family']);
                 expect(result.get('name')).toBe('Test');
@@ -333,7 +333,9 @@ describe('Entity Creation E2E Happy Path', () => {
                 const propertySpecs = (0, propertySpecifications_1.parsePropertySpecificationsIntoMap)([
                     'roles=option',
                 ]);
-                const roles = propertySpecs.get('roles');
+                // 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);
                 expect((0, create_1.doesPropertyLevelMatchRole)(types_1.PropertyLevel.FAMILY, roles[0])).toBe(false);
@@ -514,3 +516,28 @@ describe('Entity Creation E2E Happy Path', () => {
         });
     });
 });
+describe('castValueToPropertyType array coercion', () => {
+    it('should coerce a single value on a multi-value property to a one-element array', () => {
+        expect((0, create_1.castValueToPropertyType)('option', types_1.PropertyType.MultiSelect)).toEqual(['option']);
+    });
+    it('should pass arrays through on multi-value properties', () => {
+        expect((0, create_1.castValueToPropertyType)(['option', 'family'], types_1.PropertyType.MultiSelect)).toEqual([
+            'option',
+            'family',
+        ]);
+    });
+    it('should keep a comma value literal on a multi-value property and warn with the repeated-key form', () => {
+        const consoleError = jest.spyOn(console, 'error').mockImplementation(() => { });
+        expect((0, create_1.castValueToPropertyType)('option,family', types_1.PropertyType.MultiSelect)).toEqual([
+            'option,family',
+        ]);
+        expect(consoleError).toHaveBeenCalledWith(expect.stringContaining('slug=option slug=family'));
+        consoleError.mockRestore();
+    });
+    it('should keep commas literal on string properties without warning', () => {
+        const consoleError = jest.spyOn(console, 'error').mockImplementation(() => { });
+        expect((0, create_1.castValueToPropertyType)('my,name', types_1.PropertyType.String)).toBe('my,name');
+        expect(consoleError).not.toHaveBeenCalled();
+        consoleError.mockRestore();
+    });
+});

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

@@ -4,13 +4,14 @@ export default class DeleteEntity extends BaseCommand {
     static examples: string[];
     static flags: {
         dryRun: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
-        input: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces/parser").CustomOptions>;
-        id: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces/parser").CustomOptions>;
+        inputFile: 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>;
         logLevel: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces/parser").CustomOptions>;
     };
     static args: {
         entityName: import("@oclif/core/lib/interfaces/parser").Arg<string, Record<string, unknown>>;
+        ids: import("@oclif/core/lib/interfaces/parser").Arg<string, Record<string, unknown>>;
     };
+    static strict: boolean;
     run(): Promise<void>;
 }

package/dist/commands/entities/delete.js

@@ -6,63 +6,87 @@ const p_limit_1 = tslib_1.__importDefault(require("p-limit"));
 const basecommand_1 = require("../../basecommand");
 const sdk_1 = require("@contrail/sdk");
 const login_1 = require("../../auth/login");
+const common_1 = require("../../common");
 const file_helpers_1 = require("../../helpers/file-helpers");
 let IS_DRY_RUN = false;
 class DeleteEntity extends basecommand_1.BaseCommand {
-    static description = '(BETA) Delete an entity';
-    static examples = ['contrail entities delete color 123'];
+    static description = '(BETA) Delete entities by id';
+    static examples = [
+        'contrail entities delete color 123',
+        'contrail entities delete color id1 id2 id3',
+        'contrail entities delete item -i ids.json',
+    ];
     static flags = {
         ...basecommand_1.BaseCommand.baseFlags,
         dryRun: core_1.Flags.boolean({
             char: 'd',
-            description: 'If true, does not delete the entity but simulates the deletion',
+            description: 'If true, does not delete the entities but simulates the deletion',
             required: false,
             default: false,
         }),
-        input: core_1.Flags.string({
+        inputFile: core_1.Flags.string({
             char: 'i',
-            description: 'A JSON array of objects where each object is one entity to create. Cannot be used with --count. Unless --precise is specified, any missing values will be populated with random values.',
+            description: 'A file with a JSON array of objects; only each object\'s id is used. Deletes every entity in the file.',
             required: false,
             multiple: false,
         }),
-        id: core_1.Flags.string({
-            description: 'The unique identifier of the entity',
-            required: false,
-        }),
     };
     static args = {
         entityName: core_1.Args.string({
             description: 'The name of the entity type to delete',
             required: true,
         }),
+        ids: core_1.Args.string({
+            description: 'One or more entity ids to delete',
+            required: false,
+        }),
     };
+    static strict = false;
     async run() {
-        this.log('🚨 This command is in BETA and may change or be removed at any time.');
+        (0, common_1.routeConsoleLogToStderr)();
+        console.error('🚨 This command is in BETA and may change or be removed at any time.');
         await (0, login_1.authUser)();
-        const { args, flags } = await this.parse(DeleteEntity);
+        const { args, flags, argv } = await this.parse(DeleteEntity);
         const { entityName } = args;
-        const { id, input: inputFilename } = flags;
+        const { inputFile: inputFilename } = flags;
         if (flags.dryRun) {
             IS_DRY_RUN = true;
-            this.log('Running in dry run mode. No entities will be deleted.');
+            console.error('Running in dry run mode. No entities will be deleted.');
+        }
+        const tokens = argv.slice(1);
+        const valueTokens = tokens.filter((token) => token.includes('='));
+        if (valueTokens.length > 0) {
+            this.error(`delete takes no -- values (got: ${valueTokens.join(' ')}); pass ids positionally or use an input file (-i).`);
+        }
+        const positionalIds = tokens;
+        if (inputFilename && positionalIds.length > 0) {
+            this.error('You cannot use --inputFile and positional ids together. Please specify one or the other.');
+        }
+        if (!inputFilename && positionalIds.length === 0) {
+            this.error('You must provide one or more ids or an input file (-i) of entities to delete.');
         }
         const jsonInput = inputFilename ? (0, file_helpers_1.parseFileAsJsonArray)(inputFilename) : undefined;
-        if (!jsonInput && !id) {
-            this.error('You must provide either an ID or a JSON input file with entities to delete.');
+        const rowsWithId = jsonInput?.filter((item) => item.id);
+        if (jsonInput && rowsWithId.length < jsonInput.length) {
+            console.error(`${jsonInput.length - rowsWithId.length} row(s) without an id were skipped.`);
         }
-        const ids = jsonInput ? jsonInput.map((item) => item.id) : [id];
+        const ids = rowsWithId ? rowsWithId.map((item) => item.id) : positionalIds;
         try {
             const response = await deleteEntityByIds(entityName, ids);
             if (IS_DRY_RUN) {
-                this.log('simulated deletion of entities:');
+                console.error('simulated deletion of entities:');
             }
             else {
-                this.log('deleted entities:');
+                console.error('deleted entities:');
             }
             this.log(JSON.stringify(response, null, 2));
+            if (response.idsErrored.length > 0) {
+                console.error(`${response.idsErrored.length} of ${ids.length} deletes failed.`);
+                process.exitCode = 1;
+            }
         }
         catch (e) {
-            this.error(`Error fetching entity: ${e.message}`);
+            this.error(`Error deleting entities: ${e.message}`);
         }
     }
 }

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

@@ -15,6 +15,7 @@ export default class GetEntity extends BaseCommand {
     };
     static args: {
         typePath: import("@oclif/core/lib/interfaces/parser").Arg<string, Record<string, unknown>>;
+        ids: import("@oclif/core/lib/interfaces/parser").Arg<string, Record<string, unknown>>;
     };
     static strict: boolean;
     run(): Promise<void>;