fix(eslint): remove unused in catch; remove unexpected expression; fix dangling promises

Author: gribnoysup

configs/webpack-config-compass/src/util.ts

@@ -38,7 +38,7 @@ export function entriesToHtml(
           fs.statSync(maybeTemplatePath);
           template = maybeTemplatePath;
           break;
-        } catch (e) {
+        } catch {
           // ignore and use default template, electron renderer entry will need
           // at least some kind of html page provided one way or the other
         }

packages/atlas-service/src/main.ts

@@ -244,7 +244,7 @@ export class CompassAuthService {
     throwIfAborted(signal);
     try {
       return (await this.introspect({ signal })).active;
-    } catch (err) {
+    } catch {
       return false;
     }
   }

packages/atlas-service/src/store/atlas-signin-reducer.ts

@@ -278,7 +278,7 @@ export const restoreSignInState = (): AtlasSignInThunkAction<Promise<void>> => {
       } else {
         dispatch({ type: AtlasSignInActions.RestoringFailed });
       }
-    } catch (err) {
+    } catch {
       // For the initial state check if failed to check auth for any reason we
       // will just allow user to sign in again, ignoring the error
       dispatch({ type: AtlasSignInActions.RestoringFailed });

packages/compass-aggregations/src/modules/count-documents.ts

@@ -132,7 +132,7 @@ export const countDocuments = (): PipelineBuilderThunkAction<Promise<void>> => {
         type: ActionTypes.CountFinished,
         count: Number(count),
       });
-    } catch (e) {
+    } catch {
       dispatch({
         type: ActionTypes.CountFailed,
       });

packages/compass-aggregations/src/modules/insights.ts

@@ -97,7 +97,7 @@ export const fetchExplainForPipeline = (): PipelineBuilderThunkAction<
       const explainPlan = new ExplainPlan(rawExplainPlan as Stage);
       dispatch({ type: FETCH_EXPLAIN_PLAN_SUCCESS, explainPlan });
       ExplainFetchAbortControllerMap.delete(id);
-    } catch (err) {
+    } catch {
       // We are only fetching this to get information about index usage for
       // insight badge, if this fails for any reason: server, cancel, error
       // getting pipeline from state, or parsing explain plan. Whatever it is,

packages/compass-aggregations/src/modules/pipeline-builder/pipeline-builder.ts

@@ -96,7 +96,7 @@ export class PipelineBuilder {
       if (!Array.isArray(this.pipeline)) {
         throw new Error('Pipeline should be an array');
       }
-    } catch (e) {
+    } catch {
       this.pipeline = null;
     }
   }

packages/compass-aggregations/src/modules/pipeline-builder/pipeline-parser/stage-parser.ts

@@ -26,7 +26,7 @@ export function isValidStageNode(node?: t.ObjectExpression): boolean {
       // before validating it again
       parseShellBSON(generate(node));
       return true;
-    } catch (err) {
+    } catch {
       return false;
     }
   }

packages/compass-aggregations/src/modules/pipeline-builder/stage-editor.ts

@@ -42,7 +42,7 @@ export const enum StageEditorActionTypes {
   StagePreviewFetchError = 'compass-aggregations/pipeline-builder/stage-editor/StagePreviewFetchError',
   StageRun = 'compass-aggregations/pipeline-builder/stage-editor/StageRun',
   StageRunSuccess = 'compass-aggregations/pipeline-builder/stage-editor/StageRunSuccess',
-  StageRunError = 'compass-aggregations/pipeline-builder/stage-editor/StagePreviewFetchError',
+  StageRunError = 'compass-aggregations/pipeline-builder/stage-editor/StageRunError',
   StageValueChange = 'compass-aggregations/pipeline-builder/stage-editor/StageValueChange',
   StageOperatorChange = 'compass-aggregations/pipeline-builder/stage-editor/StageOperatorChange',
   StageCollapsedChange = 'compass-aggregations/pipeline-builder/stage-editor/StageCollapsedChange',
@@ -886,13 +886,13 @@ const formatWizardValue = (value?: string): string => {
   let reIndented = value;
   try {
     reIndented = JSON.stringify(JSON.parse(value), null, 2);
-  } catch (e) {
+  } catch {
     // not valid json
   }
 
   try {
     return prettify(reIndented);
-  } catch (e) {
+  } catch {
     // not valid js (ie. the generated stage has placeholders for the user to fill etc ..)
     return reIndented;
   }

packages/compass-aggregations/src/modules/search-indexes.ts

@@ -109,7 +109,7 @@ export const fetchIndexes = (): PipelineBuilderThunkAction<Promise<void>> => {
         type: ActionTypes.FetchIndexesFinished,
         indexes,
       });
-    } catch (e) {
+    } catch {
       dispatch({
         type: ActionTypes.FetchIndexesFailed,
       });

packages/compass-components/src/components/actions/dropdown-menu-button.tsx

@@ -20,7 +20,7 @@ const hiddenOnNarrowStyles = css({
 
 export type DropdownMenuButtonProps<Action extends string> = {
   actions: MenuAction<Action>[];
-  onAction(actionName: Action): void;
+  onAction(this: void, actionName: Action): void;
   renderMode?: RenderMode;
   iconSize?: ItemActionButtonSize;
   isVisible?: boolean;
@@ -93,7 +93,7 @@ export function DropdownMenuButton<Action extends string>({
             data-testid={dataTestId ? `${dataTestId}-show-actions` : undefined}
             onClick={(evt) => {
               evt.stopPropagation();
-              onClick && onClick(evt);
+              onClick?.(evt);
             }}
             rightGlyph={<Icon glyph={'CaretDown'} />}
             title={buttonText}