eslint fix

Author: ad-shreya

src/common/outputService/aorOutputService.ts

@@ -24,7 +24,7 @@ export type AorOutputFlags = {
  *
  * @author JuanStenghele-sf
  */
-export interface AorOutputService extends OutputService {
+export type AorOutputService = OutputService & {
   /**
    * Prints the status of the given aor
    */
@@ -44,7 +44,7 @@ export interface AorOutputService extends OutputService {
    * Sets the aor id
    */
   getStatus(): AsyncOperationStatus | undefined;
-}
+};
 
 /**
  * Abstract class that implements AorOutputService interface

src/common/outputService/deploymentResultOutputService.ts

@@ -12,12 +12,12 @@ import { AbstractOutputService, OutputFlags, OutputService } from './outputServi
 /**
  * Service interface for printing the output of a deployment result.
  */
-export interface DeploymentResultOutputService extends OutputService {
+export type DeploymentResultOutputService = OutputService & {
   /**
    * Prints the deployment result.
    */
   printDeploymentResult(): void;
-}
+};
 
 export abstract class AbstractDeploymentResultOutputService<T extends OutputFlags>
   extends AbstractOutputService<T>

src/common/outputService/outputService.ts

@@ -19,12 +19,12 @@ export type OutputFlags = {
  *
  * @author JuanStenghele-sf
  */
-export interface OutputService {
+export type OutputService = {
   /**
    * Prints a summary of the operation being done
    */
   printOpSummary(): void;
-}
+};
 
 /**
  * Abstract class that implements OutputService interface

src/common/outputService/promoteOutputService.ts

@@ -25,7 +25,7 @@ export type PromoteOutputFlags = {
  *
  * @author JuanStenghele-sf
  */
-export interface PromoteOutputService extends ResumeOutputService {}
+export type PromoteOutputService = ResumeOutputService;
 
 /**
  * Abstract class that implements PromoteOutputService interface

src/common/outputService/reportOutputService.ts

@@ -22,7 +22,7 @@ const messages = Messages.loadMessages('@salesforce/plugin-devops-center', 'proj
 /**
  * Interface for output methods for report operations.
  */
-export interface ReportOutputService extends DeploymentResultOutputService {}
+export type ReportOutputService = DeploymentResultOutputService;
 
 /**
  * Base class.
@@ -55,10 +55,10 @@ export class PromoteReportOutputService extends AbstractReportOutputService<Outp
     // convert every field of the deployment result record into a key:value pair.
     const formattedDeploymentResult = sObjectToArrayOfKeyValue(this.deploymentResult)
       // rename User.Name field.
-      .map((x) => {
-        x.key = x.key === 'Name' ? messages.getMessage('report.key.created-by-name') : x.key;
-        return x;
-      })
+      .map((x) => ({
+        ...x,
+        key: x.key === 'Name' ? messages.getMessage('report.key.created-by-name') : x.key,
+      }))
       // sort by key.
       .sort((a, b) => (a.key < b.key ? -1 : 1));
 

src/common/outputService/resumeOutputService.ts

@@ -19,9 +19,9 @@ import { AorOutputService, AorOutputFlags, AbstractAorOutputService } from './ao
  *
  * @author JuanStenghele-sf
  */
-export interface ResumeOutputService extends AorOutputService {
+export type ResumeOutputService = AorOutputService & {
   displayEndResults(): void;
-}
+};
 
 /**
  * Abstract class that implements ResumeOutputService interface

src/common/types.ts

@@ -41,13 +41,13 @@ export enum TestLevel {
   RunAllTestsInOrg = 'RunAllTestsInOrg',
 }
 
-export interface ApiError extends Error {
+export type ApiError = Error & {
   errorCode: string;
-}
+};
 
-export interface ApiPromoteResponse {
+export type ApiPromoteResponse = {
   jobId: string;
-}
+};
 
 export type WorkItem = {
   Name: string;

src/common/utils.ts

@@ -182,12 +182,11 @@ export async function getFormattedDeployComponentsByAyncOpId(
     ? await selectDeployComponentsForCheckDeployByAsynchOpId(con, asyncOpId)
     : await selectDeployComponentsByAsyncOpId(con, asyncOpId);
 
-  components.forEach((component) => {
-    component.Type = component.sf_devops__Source_Component__c.split(':')[0];
-    component.Name = component.sf_devops__Source_Component__c.split(':')[1];
-  });
-
-  return components;
+  return components.map((component) => ({
+    ...component,
+    Type: component.sf_devops__Source_Component__c.split(':')[0],
+    Name: component.sf_devops__Source_Component__c.split(':')[1],
+  }));
 }
 
 /**

src/streamer/doceMonitor.ts

@@ -10,6 +10,8 @@ import { AnyJson } from '@salesforce/ts-types';
 /**
  * Common interface used to monitor the progress of something in the DevOps Center.
  **/
-export default interface DoceMonitor {
+type DoceMonitor = {
   monitor(): Promise<void | AnyJson>;
-}
+};
+
+export default DoceMonitor;

src/utils/createWorkItem.ts

@@ -9,20 +9,20 @@ import { Connection } from '@salesforce/core';
 
 const API_VERSION = 'v65.0';
 
-export interface CreateWorkItemParams {
+export type CreateWorkItemParams = {
   connection: Connection;
   projectId: string;
   subject: string;
   description: string;
-}
+};
 
-export interface CreateWorkItemResult {
+export type CreateWorkItemResult = {
   success: boolean;
   workItemId?: string;
   workItemName?: string;
   subject?: string;
   error?: string;
-}
+};
 
 /**
  * Creates a new DevOps Center Work Item in the specified project.
@@ -49,18 +49,20 @@ export async function createWorkItem(params: CreateWorkItemParams): Promise<Crea
       subject: (data.subject ?? data.Subject ?? subject) as string,
     };
   } catch (error: unknown) {
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    const err = error as any;
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-    const data = err.response?.data ?? err.body ?? err;
-    const message =
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-      (typeof data === 'object' && (data?.message ?? data?.error ?? data?.errorDescription)) ??
-      // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-      err.message ??
+    const err = error as Record<string, unknown> & { response?: { data?: unknown }; body?: unknown; message?: unknown };
+    const data: unknown = (err.response?.data ?? err.body ?? err) as unknown;
+    const message: unknown =
+      (typeof data === 'object' &&
+        data !== null &&
+        ((data as Record<string, unknown>).message ??
+          (data as Record<string, unknown>).error ??
+          (data as Record<string, unknown>).errorDescription)) ||
+      err.message ||
       'Unknown error';
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-    const details = Array.isArray(data?.body) ? (data.body as string[]).join('; ') : undefined;
+    const details =
+      typeof data === 'object' && data !== null && Array.isArray((data as Record<string, unknown>).body)
+        ? ((data as Record<string, unknown[]>).body as string[]).join('; ')
+        : undefined;
     return {
       success: false,
       error: details ? `${String(message)}: ${details}` : String(message),