refactor: no-param-reassign

src/utils/deploy.ts

@@ -145,13 +145,12 @@ export async function executeDeploy(
       await deploy.start();
     }
   } else {
-    // mdapi format deploys don't require a project, but at this point we need one
-    project ??= await SfProject.resolve();
     // instantiate source tracking
     // stl will decide, based on the org's properties, what needs to be done
     const stl = await SourceTracking.create({
       org,
-      project,
+      // mdapi format deploys don't require a project, but at this point we need one
+      project: project ?? (await SfProject.resolve()),
       subscribeSDREvents: true,
       ignoreConflicts: opts['ignore-conflicts'],
     });

src/utils/flags.ts

@@ -52,13 +52,13 @@ async function ensureDirectoryPath(path: string): Promise<string> {
   return resolvedPath;
 }
 
-function resolveZipFileName(zipFileName?: string): string {
+const resolveZipFileName = (zipFileName?: string): string => {
   if (!zipFileName) {
     return DEFAULT_ZIP_FILE_NAME;
   }
   // If no file extension was provided append, '.zip'
-  return !extname(zipFileName) ? (zipFileName += '.zip') : zipFileName;
-}
+  return !extname(zipFileName) ? `${zipFileName}.zip` : zipFileName;
+};
 
 export const DEFAULT_ZIP_FILE_NAME = 'unpackaged.zip';
 

test/nuts/convert/mdapi.nut.ts

@@ -14,15 +14,15 @@ import { ConvertMdapiJson } from '../../../src/utils/types.js';
 let session: TestSession;
 
 const writeManifest = (manifestPath: string, contents?: string) => {
-  contents ??= `<?xml version="1.0" encoding="UTF-8"?>
+  const defaultContents = `<?xml version="1.0" encoding="UTF-8"?>
 <Package xmlns="http://soap.sforce.com/2006/04/metadata">
     <types>
         <members>*</members>
         <name>ApexClass</name>
     </types>
     <version>53.0</version>
 </Package>`;
-  fs.writeFileSync(manifestPath, contents);
+  fs.writeFileSync(manifestPath, contents ?? defaultContents);
 };
 
 describe('project convert mdapi NUTs', () => {

test/nuts/convert/source.nut.ts

@@ -13,15 +13,15 @@ import { ConvertResultJson } from '../../../src/utils/types.js';
 let session: TestSession;
 
 const writeManifest = (manifestPath: string, contents?: string) => {
-  contents ??= `<?xml version="1.0" encoding="UTF-8"?>
+  const defaultContents = `<?xml version="1.0" encoding="UTF-8"?>
 <Package xmlns="http://soap.sforce.com/2006/04/metadata">
     <types>
         <members>*</members>
         <name>ApexClass</name>
     </types>
     <version>53.0</version>
 </Package>`;
-  fs.writeFileSync(manifestPath, contents);
+  fs.writeFileSync(manifestPath, contents ?? defaultContents);
 };
 
 describe('project convert source NUTs', () => {

test/nuts/digitalExperienceBundle/helper.ts

@@ -4,6 +4,9 @@
  * Licensed under the BSD 3-Clause license.
  * For full license text, see LICENSE.txt file in the repo root or https://opensource.org/licenses/BSD-3-Clause
  */
+// existing tests do a lot mutation.  I decided to leave rather than refactor
+/* eslint-disable no-param-reassign */
+
 import { join, relative } from 'node:path';
 import * as fs from 'node:fs';
 import { FileResponse } from '@salesforce/source-deploy-retrieve';

test/nuts/specialTypes/folderTypes.nut.ts

@@ -72,11 +72,10 @@ describe('metadata types that go in folders', () => {
     ];
 
     const getRelativeFileResponses = (resp: FileResponse[]) =>
-      resp.map((s) => {
-        // grab the last 2 directories with the file only
-        s.filePath = s.filePath?.split(path.sep).slice(-3).join(path.sep);
-        return s;
-      });
+      resp.map((fr) => ({
+        ...fr,
+        filePath: fr.filePath?.split(path.sep).slice(-3).join(path.sep),
+      }));
 
     it('can generate manifest for just the emailTemplates', () => {
       const pathToEmails = path.join('force-app', 'main', 'default', 'email');