feat: remove startDate and endDate from license provision command

Author: ukanoja-sf

README.md

@@ -115,16 +115,14 @@ Provision Permission Set Licenses (PSL) into a target org.
 
 ```
 USAGE
-  $ sf license provision -o <value> [-n <value>] [-l <value>] [-q <value>] [-s <value>] [-e <value>] [-f <value>] [--api-version <value>] [--json] [--flags-dir <value>]
+  $ sf license provision -o <value> [-n <value>] [-l <value>] [-q <value>] [-f <value>] [--api-version <value>] [--json] [--flags-dir <value>]
 
 FLAGS
-  -e, --end-date=<value>         License end date in YYYY-MM-DD format. Default is no expiration.
   -f, --definition-file=<value>  Path to a JSON file that contains the PSL provisioning request information.
   -l, --license=<value>          Permission Set License name.
   -n, --namespace=<value>        License package namespace.
   -o, --target-org=<value>       (required) Username or alias of the target org.
   -q, --quantity=<value>         Number of licenses to provision.
-  -s, --start-date=<value>       License start date in YYYY-MM-DD format. Defaults to today.
       --api-version=<value>      Override the api version used for api requests made by this command.
 
 GLOBAL FLAGS
@@ -141,7 +139,7 @@ DESCRIPTION
 EXAMPLES
   Provision a single Permission Set License into an org:
 
-    $ sf license provision --target-org myScratchOrg --namespace demo --license newLicense --quantity 5 --start-date '2026-03-30' --end-date '2027-03-30'
+    $ sf license provision --target-org myScratchOrg --namespace demo --license newLicense --quantity 5
 
   Use a JSON formatted input file to provision one or more Permission Set Licenses into an org:
 
@@ -186,5 +184,5 @@ sf package install --package <package-id> --target-org <scratch-org-username>
 
 sf package install report -i <install-request-id> -o <scratch-org-username>
 
-sf license provision -o <scratch-org-username> --license premium --namespace demo --quantity 10 --start-date '2026-03-20' --end-date '2027-03-20'
+sf license provision -o <scratch-org-username> --license premium --namespace demo --quantity 10
 ```

command-snapshot.json

@@ -3,19 +3,8 @@
     "alias": [],
     "command": "license:provision",
     "flagAliases": ["apiversion", "targetusername", "u"],
-    "flagChars": ["e", "f", "l", "n", "o", "q", "s"],
-    "flags": [
-      "api-version",
-      "definition-file",
-      "end-date",
-      "flags-dir",
-      "json",
-      "license",
-      "namespace",
-      "quantity",
-      "start-date",
-      "target-org"
-    ],
+    "flagChars": ["f", "l", "n", "o", "q"],
+    "flags": ["api-version", "definition-file", "flags-dir", "json", "license", "namespace", "quantity", "target-org"],
     "plugin": "@salesforce/plugin-license-management"
   }
 ]

messages/license.provision.md

@@ -22,14 +22,6 @@ Permission Set License name.
 
 Number of licenses to provision.
 
-# flags.start-date.summary
-
-License start date in YYYY-MM-DD format. Defaults to today.
-
-# flags.end-date.summary
-
-License end date in YYYY-MM-DD format. Default is no expiration.
-
 # flags.definition-file.summary
 
 Path to a JSON file that contains the PSL provisioning request information.
@@ -38,7 +30,7 @@ Path to a JSON file that contains the PSL provisioning request information.
 
 - Provision a single Permission Set License into an org:
 
-  <%= config.bin %> <%= command.id %> --target-org myScratchOrg --namespace demo --license newLicense --quantity 5 --start-date '2026-03-30' --end-date '2027-03-30'
+  <%= config.bin %> <%= command.id %> --target-org myScratchOrg --namespace demo --license newLicense --quantity 5
 
 - Use a JSON formatted input file to provision one or more Permission Set Licenses into an org:
 
@@ -54,7 +46,7 @@ Either --license or --definition-file is required.
 
 # error.mutuallyExclusiveFlags
 
-The --definition-file flag cannot be used with --namespace, --license, --quantity, --start-date, or --end-date.
+The --definition-file flag cannot be used with --namespace, --license, or --quantity.
 
 # error.emptyDefinitionFile
 

src/commands/license/provision.ts

@@ -25,8 +25,6 @@ type ProvisionLicenseSpec = {
   namespacePrefix?: string;
   permissionSetLicense?: string;
   quantity?: number;
-  startDate?: string;
-  endDate?: string;
 };
 
 type ProvisionPslRequest = {
@@ -72,14 +70,6 @@ export default class LicenseProvision extends SfCommand<LicenseProvisionResult>
       min: 0,
       max: Number.MAX_SAFE_INTEGER,
     }),
-    'start-date': Flags.string({
-      char: 's',
-      summary: messages.getMessage('flags.start-date.summary'),
-    }),
-    'end-date': Flags.string({
-      char: 'e',
-      summary: messages.getMessage('flags.end-date.summary'),
-    }),
     'definition-file': Flags.string({
       char: 'f',
       summary: messages.getMessage('flags.definition-file.summary'),
@@ -92,13 +82,7 @@ export default class LicenseProvision extends SfCommand<LicenseProvisionResult>
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
     flags: Record<string, any>
   ): Promise<ProvisionLicenseSpec[]> {
-    if (
-      flags['license'] ||
-      flags['namespace'] ||
-      flags['quantity'] !== undefined ||
-      flags['start-date'] ||
-      flags['end-date']
-    ) {
+    if (flags['license'] || flags['namespace'] || flags['quantity'] !== undefined) {
       throw messages.createError('error.mutuallyExclusiveFlags');
     }
 
@@ -118,15 +102,11 @@ export default class LicenseProvision extends SfCommand<LicenseProvisionResult>
       throw messages.createError('error.missingLicenseFlag');
     }
 
-    const startDate = (flags['start-date'] as string | undefined) ?? new Date().toISOString().slice(0, 10);
-
     return [
       {
         namespacePrefix: flags['namespace'] as string | undefined,
         permissionSetLicense: flags['license'] as string,
         quantity: flags['quantity'] as number | undefined,
-        startDate,
-        endDate: flags['end-date'] as string | undefined,
       },
     ];
   }

test/commands/license/provision.test.ts

@@ -65,10 +65,6 @@ describe('license provision', () => {
       'demo',
       '--quantity',
       '5',
-      '--start-date',
-      '2026-03-30',
-      '--end-date',
-      '2027-03-30',
     ]);
 
     const tableCall = sfCommandStubs.table.firstCall.args[0] as { data: Array<Record<string, string>>; title: string };
@@ -99,10 +95,6 @@ describe('license provision', () => {
       'demo',
       '--quantity',
       '5',
-      '--start-date',
-      '2026-03-30',
-      '--end-date',
-      '2027-03-30',
     ]);
 
     expect(requestStub.calledOnce).to.be.true;
@@ -116,21 +108,9 @@ describe('license provision', () => {
       namespacePrefix: 'demo',
       permissionSetLicense: 'newLicense',
       quantity: 5,
-      startDate: '2026-03-30',
-      endDate: '2027-03-30',
     });
   });
 
-  it('defaults start-date to today when not provided', async () => {
-    const today = new Date().toISOString().slice(0, 10);
-
-    await LicenseProvision.run(['--target-org', testOrg.username, '--license', 'myLicense']);
-
-    const callArgs = requestStub.firstCall.args[0] as { body: string };
-    const body = JSON.parse(callArgs.body) as { licenses: Array<{ startDate: string }> };
-    expect(body.licenses[0].startDate).to.equal(today);
-  });
-
   it('returns status:success result with traceId', async () => {
     const result = await LicenseProvision.run(['--target-org', testOrg.username, '--license', 'myLicense']);
     expect(result).to.deep.equal({ status: 'success', traceId: 'tm_test123' });
@@ -268,16 +248,16 @@ describe('license provision', () => {
 
   it('wraps a HTTP 400 provisioning error as SfError', async () => {
     requestStub.rejects(
-      Object.assign(new Error("Invalid endDate format for permissionSetLicense 'premium'"), {
-        name: 'INVALID_END_DATE',
+      Object.assign(new Error("Invalid quantity for permissionSetLicense 'premium'"), {
+        name: 'INVALID_QUANTITY',
       })
     );
 
     try {
       await LicenseProvision.run(['--target-org', testOrg.username, '--license', 'premium']);
       expect.fail('Expected an error to be thrown');
     } catch (error: unknown) {
-      expect((error as Error).message).to.include('Invalid endDate format');
+      expect((error as Error).message).to.include('Invalid quantity');
     }
   });