refactor: devScripts update

eslint.config.mjs

@@ -0,0 +1 @@
+export { default } from 'eslint-config-salesforce-typescript';

package.json

@@ -12,14 +12,16 @@
     "@salesforce/kit": "^4.0.0",
     "@salesforce/packaging": "^5.0.4",
     "@salesforce/sf-plugins-core": "^13.0.0",
-    "chalk": "^5.6.2",
-    "@salesforce/ts-types": "^3.0.1"
+    "@salesforce/ts-types": "^3.0.1",
+    "chalk": "^5.6.2"
   },
   "devDependencies": {
     "@oclif/plugin-command-snapshot": "^5.3.34",
     "@salesforce/cli-plugins-testkit": "^5.3.62",
-    "@salesforce/dev-scripts": "^11.0.4",
+    "@salesforce/dev-scripts": "^13.0.1",
     "@salesforce/plugin-command-reference": "^3.1.122",
+    "eslint": "^10.4.0",
+    "eslint-config-salesforce-typescript": "^6.0.0",
     "eslint-plugin-sf-plugin": "^1.20.32",
     "minimatch": "^10.2.5",
     "oclif": "^4.23.27",
@@ -170,6 +172,7 @@
         "src/**/*.ts",
         "test/**/*.ts",
         "messages/**",
+        "**/eslint.config.*",
         "**/.eslint*",
         "**/tsconfig.json"
       ],

src/commands/package/bundle/install.ts

@@ -155,7 +155,7 @@ export class PackageBundlesInstall extends SfCommand<BundleSObjects.PkgBundleVer
       default:
         this.log(
           messages.getMessage('bundleInstallInProgress', [
-            camelCaseToTitleCase(result.InstallStatus as string),
+            camelCaseToTitleCase(result.InstallStatus),
             result.Id,
             targetOrg.getUsername() ?? '',
           ])

src/commands/package/bundle/uninstall.ts

@@ -129,7 +129,7 @@ export class PackageBundleUninstallCommand extends SfCommand<BundleUninstallResu
       default:
         this.log(
           messages.getMessage('bundleUninstallInProgress', [
-            camelCaseToTitleCase(result.UninstallStatus as string),
+            camelCaseToTitleCase(result.UninstallStatus),
             result.Id,
             targetOrg.getUsername() ?? '',
           ])

src/commands/package/bundle/version/create.ts

@@ -166,7 +166,7 @@ export class PackageBundlesCreate extends SfCommand<BundleSObjects.PackageBundle
         break;
       }
       default:
-        this.log(messages.getMessage('InProgress', [camelCaseToTitleCase(result.RequestStatus as string), result.Id]));
+        this.log(messages.getMessage('InProgress', [camelCaseToTitleCase(result.RequestStatus), result.Id]));
     }
     return result;
   }

src/commands/package/convert.ts

@@ -135,13 +135,13 @@ export class PackageConvert extends SfCommand<PackageVersionCreateRequestResult>
       flags['target-dev-hub'].getConnection(flags['api-version']),
       {
         wait: flags.wait,
-        installationKey: flags['installation-key'] as string,
-        definitionfile: flags['definition-file'] as string,
+        installationKey: flags['installation-key'],
+        definitionfile: flags['definition-file'],
         installationKeyBypass: flags['installation-key-bypass'],
-        buildInstance: flags['build-instance'] as string,
-        seedMetadata: flags['seed-metadata'] as string,
-        patchversion: flags['patch-version'] as string,
-        codecoverage: flags['code-coverage'] as boolean,
+        buildInstance: flags['build-instance'],
+        seedMetadata: flags['seed-metadata'],
+        patchversion: flags['patch-version'],
+        codecoverage: flags['code-coverage'],
       },
       this.project
     );

src/commands/package/installed/list.ts

@@ -60,7 +60,7 @@ export class PackageInstalledListCommand extends SfCommand<PackageInstalledComma
 
       // Calculate Version Settings from the data already in the result
       if (r.SubscriberPackageVersion) {
-        const isManaged = r.SubscriberPackageVersion.IsManaged as boolean;
+        const isManaged = r.SubscriberPackageVersion.IsManaged;
         const package2ContainerOptions = r.SubscriberPackageVersion.Package2ContainerOptions as string;
 
         // Use namespace for 1GP managed packages. For 2GP managed packages use packageId. For anything else,

src/commands/package/version/create.ts

@@ -315,7 +315,7 @@ export class PackageVersionCreateCommand extends SfCommand<PackageVersionCommand
         break;
       default:
         this.log(
-          messages.getMessage('InProgress', [camelCaseToTitleCase(result.Status as string), this.config.bin, result.Id])
+          messages.getMessage('InProgress', [camelCaseToTitleCase(result.Status), this.config.bin, result.Id])
         );
     }
     return result;

src/commands/package/version/create/report.ts

@@ -129,7 +129,7 @@ export class PackageVersionCreateReportCommand extends SfCommand<ReportCommandRe
       // Check if errors were truncated.  If so, inform the user with
       // instructions on how to retrieve the remaining errors.
       if (record.Error.length > ERROR_LIMIT) {
-        this.warn(messages.getMessage('truncatedErrors', [this.config.bin, requestId, devOrg.getUsername() as string]));
+        this.warn(messages.getMessage('truncatedErrors', [this.config.bin, requestId, devOrg.getUsername()]));
       }
     }
   }

src/commands/package/version/promote.ts

@@ -73,7 +73,7 @@ export class PackageVersionPromoteCommand extends SfCommand<PackageSaveResult> {
       this.log(messages.getMessage('humanSuccess', [result.id]));
       return result;
     } catch (e) {
-      const err = SfError.wrap(e as Error);
+      const err = SfError.wrap(e);
       if (err.name === 'DUPLICATE_VALUE' && err.message.includes('previously released')) {
         err.message = messages.getMessage('previouslyReleasedMessage');
         err.actions = [messages.getMessage('previouslyReleasedAction', [this.config.bin, this.config.bin])];

src/commands/package/version/report.ts

@@ -195,9 +195,9 @@ export class PackageVersionReportCommand extends SfCommand<PackageVersionReportR
         },
         {
           key: 'Metadata File Size',
-          value: (record.TotalSizeOfMetadataFiles
+          value: record.TotalSizeOfMetadataFiles
             ? `${Math.ceil(record.TotalSizeOfMetadataFiles / (1024 * 1024))} MB`
-            : '') as unknown as string,
+            : '',
         }
       );
       if (Number(connection.version) >= 67) {

test/commands/bundle/bundleCreate.test.ts

@@ -49,7 +49,7 @@ describe('package:bundle:create - tests', () => {
 
     await cmd.run();
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
   });
 

test/commands/bundle/bundleInstallReport.test.ts

@@ -64,7 +64,7 @@ describe('package:bundle:install:report - tests', () => {
 
     expect(getInstallStatusStub.calledOnce).to.be.true;
     expect(getInstallStatusStub.firstCall.args[0]).to.equal(requestId);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
   });
 
@@ -89,7 +89,7 @@ describe('package:bundle:install:report - tests', () => {
 
     expect(getInstallStatusStub.calledOnce).to.be.true;
     expect(getInstallStatusStub.firstCall.args[0]).to.equal(requestId);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
   });
 

test/commands/bundle/bundleInstalledList.test.ts

@@ -71,9 +71,9 @@ describe('package:bundle:installed:list - tests', () => {
     await cmd.run();
 
     expect(getInstalledBundlesStub.calledOnce).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.called).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.callCount).to.equal(2); // One for bundle info, one for components
   });
 
@@ -128,9 +128,9 @@ describe('package:bundle:installed:list - tests', () => {
     await cmd.run();
 
     expect(getInstalledBundlesStub.calledOnce).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.called).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.callCount).to.equal(4); // Two bundles × (info + components)
   });
 
@@ -165,7 +165,7 @@ describe('package:bundle:installed:list - tests', () => {
     await cmd.run();
 
     expect(getInstalledBundlesStub.calledOnce).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.called).to.be.true;
   });
 
@@ -200,7 +200,7 @@ describe('package:bundle:installed:list - tests', () => {
     await cmd.run();
 
     expect(getInstalledBundlesStub.calledOnce).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.called).to.be.true;
   });
 
@@ -212,9 +212,9 @@ describe('package:bundle:installed:list - tests', () => {
     await cmd.run();
 
     expect(getInstalledBundlesStub.calledOnce).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.warn.calledOnce).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.warn.firstCall.args[0]).to.include('No installed package bundles found');
   });
 
@@ -242,9 +242,9 @@ describe('package:bundle:installed:list - tests', () => {
     await cmd.run();
 
     expect(getInstalledBundlesStub.calledOnce).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.called).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.callCount).to.equal(1); // Only bundle info, no components table
   });
 

test/commands/bundle/bundleList.test.ts

@@ -46,7 +46,7 @@ describe('package:bundle:list - tests', () => {
 
     await cmd.run();
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
   });
 

test/commands/bundle/bundleVersionCreateList.test.ts

@@ -58,7 +58,7 @@ describe('package:bundle:version:create:list - tests', () => {
 
     await cmd.run();
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
     expect(getCreateStatusesStub.calledOnce).to.be.true;
   });
@@ -70,7 +70,7 @@ describe('package:bundle:version:create:list - tests', () => {
 
     await cmd.run();
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.warn.calledOnce).to.be.true;
     expect(sfCommandStubs.warn.firstCall.args[0]).to.equal('No results found');
   });

test/commands/bundle/bundleVersionCreateReport.test.ts

@@ -68,7 +68,7 @@ describe('package:bundle:version:create:report - tests', () => {
 
     expect(getCreateStatusStub.calledOnce).to.be.true;
     expect(getCreateStatusStub.firstCall.args[0]).to.equal(requestId);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
   });
 
@@ -100,7 +100,7 @@ describe('package:bundle:version:create:report - tests', () => {
 
     expect(getCreateStatusStub.calledOnce).to.be.true;
     expect(getCreateStatusStub.firstCall.args[0]).to.equal(requestId);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
   });
 

test/commands/bundle/bundleVersionList.test.ts

@@ -70,7 +70,7 @@ describe('package:bundle:version:list - tests', () => {
 
     await cmd.run();
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
   });
 

test/commands/bundle/bundleVersionReport.test.ts

@@ -145,7 +145,7 @@ describe('package:bundle:version:report - tests', () => {
     expect(reportStub.firstCall.args[1]).to.equal(bundleVersionId);
     expect(componentPackagesStub.calledOnce).to.be.true;
     expect(componentPackagesStub.firstCall.args[1]).to.equal(bundleVersionId);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledTwice).to.be.true;
   });
 
@@ -244,7 +244,7 @@ describe('package:bundle:version:report - tests', () => {
     expect(reportStub.firstCall.args[1]).to.equal(bundleVersionId);
     expect(componentPackagesStub.calledOnce).to.be.true;
     expect(componentPackagesStub.firstCall.args[1]).to.equal(bundleVersionId);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledTwice).to.be.true;
   });
 
@@ -350,7 +350,7 @@ describe('package:bundle:version:report - tests', () => {
     expect(reportStub.firstCall.args[1]).to.equal(bundleVersionId);
     expect(componentPackagesStub.calledOnce).to.be.true;
     expect(componentPackagesStub.firstCall.args[1]).to.equal(bundleVersionId);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
   });
 });

test/commands/package/install.test.ts

@@ -395,11 +395,11 @@ describe('package:install', () => {
       installStub = $$.SANDBOX.stub(SubscriberPackageVersion.prototype, 'install').callsFake(async () => {
         await Lifecycle.getInstance().emit(
           PackageEvents.install['subscriber-status'],
-          'PACKAGE_UNAVAILABLE' as InstallValidationStatus
+          'PACKAGE_UNAVAILABLE'
         );
         await Lifecycle.getInstance().emit(
           PackageEvents.install['subscriber-status'],
-          'NO_ERRORS_DETECTED' as InstallValidationStatus
+          'NO_ERRORS_DETECTED'
         );
         await Lifecycle.getInstance().emit(PackageEvents.install.status, pkgInstallRequest);
         await Lifecycle.getInstance().emit(PackageEvents.install.status, successRequest);

test/commands/package/installReport.test.ts

@@ -111,7 +111,7 @@ describe('package:install:report', () => {
     });
     getInstallRequestStub.restore();
     getInstallRequestStub = $$.SANDBOXES.DEFAULT.stub(SubscriberPackageVersion, 'getInstallRequest').resolves(
-      request as PackagingSObjects.PackageInstallRequest
+      request
     );
 
     getInstallRequestStub.resolves(request);

test/commands/package/packagePushUpgradeAbort.test.ts

@@ -49,11 +49,11 @@ describe('PackagePushUpgradeAbortCommand', () => {
     const res = await cmd.run();
 
     expect(res.success).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(abortStub.calledOnce).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(logStub.callCount).to.equal(1);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(logStub.args[0]).to.deep.equal([`Scheduled push upgrade ID [${pushRequestId}] was cancelled.`]);
   });
 

test/commands/package/packagePushUpgradeList.test.ts

@@ -86,9 +86,9 @@ describe('package:pushupgrade:list - tests', () => {
     await cmd.run();
 
     expect(envSpy.calledOnce).to.equal(false);
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.table.calledOnce).to.be.true;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.warn.calledOnce).to.be.false;
   });
 
@@ -100,7 +100,7 @@ describe('package:pushupgrade:list - tests', () => {
 
     await cmd.run();
 
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
+     
     expect(sfCommandStubs.warn.calledOnce).to.be.true;
   });