first round of updates to disable type assertions

frontend/.eslintrc

@@ -315,6 +315,18 @@
           }
         ]
       }
+    },
+    {
+      "files": ["*.ts", "*.tsx"],
+      "excludedFiles": ["**/__mocks__/**", "**/__tests__/**"],
+      "rules": {
+        "@typescript-eslint/consistent-type-assertions": [
+          "error",
+          {
+            "assertionStyle": "never"
+          }
+        ]
+      }
     }
   ]
 }

frontend/src/__tests__/unit/testUtils/hooks.ts

@@ -67,8 +67,8 @@ export const renderHook = <
   options?: RenderHookOptions<Props, Q, Container, BaseElement>,
 ): RenderHookResultExt<Result, Props> => {
   let updateCount = 0;
-  let prevResult: Result | undefined;
-  let currentResult: Result | undefined;
+  let prevResult: Result;
+  let currentResult: Result;
 
   const renderResult = renderHookRTL((props) => {
     updateCount++;
@@ -80,8 +80,7 @@ export const renderHook = <
   const renderResultExt: RenderHookResultExt<Result, Props> = {
     ...renderResult,
 
-    getPreviousResult: () =>
-      updateCount > 1 ? (prevResult as Result) : renderResult.result.current,
+    getPreviousResult: () => (updateCount > 1 ? prevResult : renderResult.result.current),
 
     getUpdateCount: () => updateCount,
 

frontend/src/api/errorUtils.ts

@@ -2,7 +2,7 @@ import { K8sStatus } from '@openshift/dynamic-plugin-sdk-utils';
 import { AxiosError } from 'axios';
 
 export const isK8sStatus = (data: unknown): data is K8sStatus =>
-  (data as K8sStatus).kind === 'Status';
+  typeof data === 'object' && data !== null && 'kind' in data && data.kind === 'Status';
 
 export class K8sStatusError extends Error {
   public statusObject: K8sStatus;
@@ -18,6 +18,7 @@ const isAxiosErrorWithResponseMessage = (
   error?: Error | AxiosError,
 ): error is AxiosError<{ message: string }> =>
   Boolean(
+    // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
     error && typeof (error as AxiosError<{ message: string }>).response?.data.message === 'string',
   );
 

frontend/src/api/modelRegistry/errorUtils.ts

@@ -2,7 +2,7 @@ import { ModelRegistryError } from '~/concepts/modelRegistry/types';
 import { isCommonStateError } from '~/utilities/useFetchState';
 
 const isError = (e: unknown): e is ModelRegistryError =>
-  ['code', 'message'].every((key) => key in (e as ModelRegistryError));
+  typeof e === 'object' && e !== null && ['code', 'message'].every((key) => key in e);
 
 export const handleModelRegistryFailures = <T>(promise: Promise<T>): Promise<T> =>
   promise

frontend/src/api/pipelines/errorUtils.ts

@@ -14,10 +14,12 @@ type ResultErrorKF = {
 };
 
 const isErrorKF = (e: unknown): e is ErrorKF =>
-  ['error', 'code', 'message'].every((key) => key in (e as ErrorKF));
+  typeof e === 'object' && e !== null && ['error', 'code', 'message'].every((key) => key in e);
 
 const isErrorDetailsKF = (result: unknown): result is ResultErrorKF =>
-  ['error_details', 'error_message'].every((key) => key in (result as ResultErrorKF));
+  typeof result === 'object' &&
+  result !== null &&
+  ['error_details', 'error_message'].every((key) => key in result);
 
 export const handlePipelineFailures = <T>(promise: Promise<T>): Promise<T> =>
   promise

frontend/src/api/prometheus/serving.ts

@@ -43,7 +43,7 @@ export const useModelServingMetrics = (
 
   const serverRequestCount = useQueryRangeResourceData(
     performanceMetricsAreaAvailable && type === PerformanceMetricType.SERVER,
-    (queries as { [key in ServerMetricType]: string })[ServerMetricType.REQUEST_COUNT],
+    ServerMetricType.REQUEST_COUNT in queries ? queries[ServerMetricType.REQUEST_COUNT] : '',
     end,
     timeframe,
     defaultResponsePredicate,
@@ -53,7 +53,9 @@ export const useModelServingMetrics = (
   const serverAverageResponseTime =
     useQueryRangeResourceData<PrometheusQueryRangeResponseDataResult>(
       performanceMetricsAreaAvailable && type === PerformanceMetricType.SERVER,
-      (queries as { [key in ServerMetricType]: string })[ServerMetricType.AVG_RESPONSE_TIME],
+      ServerMetricType.AVG_RESPONSE_TIME in queries
+        ? queries[ServerMetricType.AVG_RESPONSE_TIME]
+        : '',
       end,
       timeframe,
       prometheusQueryRangeResponsePredicate,
@@ -62,7 +64,7 @@ export const useModelServingMetrics = (
 
   const serverCPUUtilization = useQueryRangeResourceData(
     performanceMetricsAreaAvailable && type === PerformanceMetricType.SERVER,
-    (queries as { [key in ServerMetricType]: string })[ServerMetricType.CPU_UTILIZATION],
+    ServerMetricType.CPU_UTILIZATION in queries ? queries[ServerMetricType.CPU_UTILIZATION] : '',
     end,
     timeframe,
     defaultResponsePredicate,
@@ -71,7 +73,9 @@ export const useModelServingMetrics = (
 
   const serverMemoryUtilization = useQueryRangeResourceData(
     performanceMetricsAreaAvailable && type === PerformanceMetricType.SERVER,
-    (queries as { [key in ServerMetricType]: string })[ServerMetricType.MEMORY_UTILIZATION],
+    ServerMetricType.MEMORY_UTILIZATION in queries
+      ? queries[ServerMetricType.MEMORY_UTILIZATION]
+      : '',
     end,
     timeframe,
     defaultResponsePredicate,
@@ -80,7 +84,9 @@ export const useModelServingMetrics = (
 
   const modelRequestSuccessCount = useQueryRangeResourceData(
     performanceMetricsAreaAvailable && type === PerformanceMetricType.MODEL,
-    (queries as { [key in ModelMetricType]: string })[ModelMetricType.REQUEST_COUNT_SUCCESS],
+    ModelMetricType.REQUEST_COUNT_SUCCESS in queries
+      ? queries[ModelMetricType.REQUEST_COUNT_SUCCESS]
+      : '',
     end,
     timeframe,
     defaultResponsePredicate,
@@ -89,7 +95,9 @@ export const useModelServingMetrics = (
 
   const modelRequestFailedCount = useQueryRangeResourceData(
     performanceMetricsAreaAvailable && type === PerformanceMetricType.MODEL,
-    (queries as { [key in ModelMetricType]: string })[ModelMetricType.REQUEST_COUNT_FAILED],
+    ModelMetricType.REQUEST_COUNT_FAILED in queries
+      ? queries[ModelMetricType.REQUEST_COUNT_FAILED]
+      : '',
     end,
     timeframe,
     defaultResponsePredicate,
@@ -98,7 +106,7 @@ export const useModelServingMetrics = (
 
   const modelTrustyAISPD = useQueryRangeResourceData(
     biasMetricsAreaAvailable && type === PerformanceMetricType.MODEL,
-    (queries as { [key in ModelMetricType]: string })[ModelMetricType.TRUSTY_AI_SPD],
+    ModelMetricType.TRUSTY_AI_SPD in queries ? queries[ModelMetricType.TRUSTY_AI_SPD] : '',
     end,
     timeframe,
     prometheusQueryRangeResponsePredicate,
@@ -108,7 +116,7 @@ export const useModelServingMetrics = (
 
   const modelTrustyAIDIR = useQueryRangeResourceData(
     biasMetricsAreaAvailable && type === PerformanceMetricType.MODEL,
-    (queries as { [key in ModelMetricType]: string })[ModelMetricType.TRUSTY_AI_DIR],
+    ModelMetricType.TRUSTY_AI_DIR in queries ? queries[ModelMetricType.TRUSTY_AI_DIR] : '',
     end,
     timeframe,
     prometheusQueryRangeResponsePredicate,

frontend/src/api/trustyai/errorUtils.ts

@@ -9,8 +9,7 @@ type TrustyAIClientErrorViolation = {
 };
 
 const isTrustyAIClientError = (e: unknown): e is TrustyAIClientError =>
-  typeof e === 'object' &&
-  ['title', 'status', 'violations'].every((key) => key in (e as TrustyAIClientError));
+  typeof e === 'object' && e !== null && ['title', 'status', 'violations'].every((key) => key in e);
 
 export const handleTrustyAIFailures = <T>(promise: Promise<T>): Promise<T> =>
   promise.then((result) => {

frontend/src/app/AppContext.ts

@@ -8,13 +8,7 @@ type AppContextProps = {
   storageClasses: StorageClassKind[];
 };
 
-const defaultAppContext: AppContextProps = {
-  buildStatuses: [],
-  // At runtime dashboardConfig is never null -- DO NOT DO THIS usually
-  dashboardConfig: null as unknown as DashboardConfigKind,
-  storageClasses: [] as StorageClassKind[],
-};
-
-export const AppContext = React.createContext(defaultAppContext);
+// eslint-disable-next-line @typescript-eslint/consistent-type-assertions
+export const AppContext = React.createContext({} as AppContextProps);
 
 export const useAppContext = (): AppContextProps => React.useContext(AppContext);

frontend/src/components/DocCardBadges.tsx

@@ -5,7 +5,7 @@ import { QuickStartContext, QuickStartContextValues } from '@patternfly/quicksta
 import { OdhDocument, OdhDocumentType } from '~/types';
 import { getQuickStartCompletionStatus, CompletionStatusEnum } from '~/utilities/quickStartUtils';
 import { DOC_TYPE_TOOLTIPS } from '~/utilities/const';
-import { getLabelColorForDocType, getDuration } from '~/utilities/utils';
+import { getLabelColorForDocType, getDuration, asEnumMember } from '~/utilities/utils';
 import { DOC_TYPE_LABEL } from '~/pages/learningCenter/const';
 
 import './OdhCard.scss';
@@ -19,7 +19,7 @@ const DocCardBadges: React.FC<DocCardBadgesProps> = ({ odhDoc }) => {
   const [completionStatus, setCompletionStatus] = React.useState<
     CompletionStatusEnum | undefined
   >();
-  const docType = odhDoc.spec.type as OdhDocumentType;
+  const docType = asEnumMember(odhDoc.spec.type, OdhDocumentType);
   const docName = odhDoc.metadata.name;
   const duration = odhDoc.spec.durationMinutes;
 
@@ -32,13 +32,15 @@ const DocCardBadges: React.FC<DocCardBadgesProps> = ({ odhDoc }) => {
     }
   }, [qsContext, docType, docName]);
 
-  const label = DOC_TYPE_LABEL[docType] || 'Documentation';
+  const label = docType !== null ? DOC_TYPE_LABEL[docType] || 'Documentation' : null;
 
   return (
     <LabelGroup defaultIsOpen numLabels={3}>
-      <Tooltip content={DOC_TYPE_TOOLTIPS[docType]}>
-        <Label color={getLabelColorForDocType(docType)}>{label}</Label>
-      </Tooltip>
+      {docType && (
+        <Tooltip content={DOC_TYPE_TOOLTIPS[docType]}>
+          <Label color={getLabelColorForDocType(docType)}>{label}</Label>
+        </Tooltip>
+      )}
       {duration ? (
         <Label variant="outline" color="grey">
           {getDuration(duration)}
@@ -61,6 +63,8 @@ const DocCardBadges: React.FC<DocCardBadgesProps> = ({ odhDoc }) => {
       ) : null}
     </LabelGroup>
   );
+
+  return null;
 };
 
 export default DocCardBadges;

frontend/src/components/OdhDocListItem.tsx

@@ -5,7 +5,7 @@ import { ExternalLinkAltIcon } from '@patternfly/react-icons';
 import { OdhDocument, OdhDocumentType } from '~/types';
 import { getQuickStartLabel, launchQuickStart } from '~/utilities/quickStartUtils';
 import { DOC_TYPE_TOOLTIPS } from '~/utilities/const';
-import { getDuration } from '~/utilities/utils';
+import { asEnumMember, getDuration } from '~/utilities/utils';
 import { useQuickStartCardSelected } from './useQuickStartCardSelected';
 import FavoriteButton from './FavoriteButton';
 
@@ -26,18 +26,21 @@ const OdhDocListItem: React.FC<OdhDocCardProps> = ({ odhDoc, favorite, updateFav
   };
 
   const renderTypeBadge = () => {
-    const docType = odhDoc.spec.type as OdhDocumentType;
-    const typeBadgeClasses = classNames('odh-list-item__partner-badge odh-m-doc', {
-      'odh-m-documentation': docType === OdhDocumentType.Documentation,
-      'odh-m-tutorial': docType === OdhDocumentType.Tutorial,
-      'odh-m-quick-start': docType === OdhDocumentType.QuickStart,
-      'odh-m-how-to': docType === OdhDocumentType.HowTo,
-    });
-    return (
-      <Tooltip content={DOC_TYPE_TOOLTIPS[docType]}>
-        <div className={typeBadgeClasses}>{odhDoc.spec.type}</div>
-      </Tooltip>
-    );
+    const docType = asEnumMember(odhDoc.spec.type, OdhDocumentType);
+    if (docType !== null) {
+      const typeBadgeClasses = classNames('odh-list-item__partner-badge odh-m-doc', {
+        'odh-m-documentation': docType === OdhDocumentType.Documentation,
+        'odh-m-tutorial': docType === OdhDocumentType.Tutorial,
+        'odh-m-quick-start': docType === OdhDocumentType.QuickStart,
+        'odh-m-how-to': docType === OdhDocumentType.HowTo,
+      });
+      return (
+        <Tooltip content={DOC_TYPE_TOOLTIPS[docType]}>
+          <div className={typeBadgeClasses}>{odhDoc.spec.type}</div>
+        </Tooltip>
+      );
+    }
+    return null;
   };
 
   const renderDocLink = () => {

frontend/src/components/ToastNotification.tsx

@@ -3,6 +3,7 @@ import { Alert, AlertActionCloseButton, AlertVariant } from '@patternfly/react-c
 import { AppNotification } from '~/redux/types';
 import { ackNotification, hideNotification } from '~/redux/actions/actions';
 import { useAppDispatch } from '~/redux/hooks';
+import { asEnumMember } from '~/utilities/utils';
 
 const TOAST_NOTIFICATION_TIMEOUT = 8 * 1000;
 
@@ -36,7 +37,7 @@ const ToastNotification: React.FC<ToastNotificationProps> = ({ notification }) =
 
   return (
     <Alert
-      variant={notification.status as AlertVariant}
+      variant={asEnumMember(notification.status, AlertVariant) ?? undefined}
       title={notification.title}
       actionClose={
         <AlertActionCloseButton onClose={() => dispatch(ackNotification(notification))} />

frontend/src/components/browserStorage/BrowserStorageContext.tsx

@@ -48,6 +48,7 @@ export const useBrowserStorage = <T,>(
     [isSessionStorage, jsonify, setJSONValue, setStringValue, storageKey],
   );
 
+  // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
   return [(getValue(storageKey, jsonify, isSessionStorage) as T) ?? defaultValue, setValue];
 };
 

frontend/src/components/table/useTableColumnSort.ts

@@ -105,7 +105,9 @@ const useTableColumnSort = <T>(
             return 0;
           }
 
+          // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
           const dataValueA = a[columnField.field as keyof T];
+          // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
           const dataValueB = b[columnField.field as keyof T];
           if (typeof dataValueA === 'string' && typeof dataValueB === 'string') {
             return dataValueA.localeCompare(dataValueB);

frontend/src/concepts/areas/utils.ts

@@ -16,6 +16,7 @@ const getFlags = (dashboardConfigSpec: DashboardConfigKind['spec']): FlagState =
 
   return {
     ...Object.keys(flags).reduce<FlagState>((acc, key) => {
+      // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
       const value = flags[key as FeatureFlag];
       if (isFeatureFlag(key, value)) {
         acc[key] = key.startsWith('disable') ? !value : value;

frontend/src/concepts/dashboard/DashboardSearchField.tsx

@@ -1,6 +1,7 @@
 import * as React from 'react';
 import { InputGroup, SearchInput, InputGroupItem } from '@patternfly/react-core';
 import SimpleDropdownSelect from '~/components/SimpleDropdownSelect';
+import { asEnumMember } from '~/utilities/utils';
 
 // List all the possible search fields here
 export enum SearchType {
@@ -48,7 +49,7 @@ const DashboardSearchField: React.FC<DashboardSearchFieldProps> = ({
         }))}
         value={searchType}
         onChange={(key) => {
-          onSearchTypeChange(key as SearchType);
+          onSearchTypeChange(asEnumMember(key, SearchType)!);
         }}
         icon={icon}
       />

frontend/src/concepts/distributedWorkloads/utils.tsx

@@ -34,6 +34,7 @@ import {
   convertToUnit,
 } from '~/utilities/valueUnits';
 import { WorkloadWithUsage } from '~/api';
+import { isEnumMember } from '~/utilities/utils';
 
 export enum WorkloadStatusType {
   Pending = 'Pending',
@@ -166,7 +167,7 @@ export const getWorkloadName = (workload: WorkloadKind): string =>
   workload.metadata?.name || 'Unnamed';
 
 export const isKnownWorkloadOwnerType = (s: string): s is WorkloadOwnerType =>
-  (Object.values(WorkloadOwnerType) as string[]).includes(s);
+  isEnumMember(s, WorkloadOwnerType);
 
 export const getWorkloadOwner = (
   workload: WorkloadKind,

frontend/src/concepts/docResources/docUtils.ts

@@ -30,6 +30,7 @@ export const getQuickStartDocs = (
   // Get doc cards for the quick starts
   const docs: OdhDocument[] = quickStarts.map(
     (quickStart) =>
+      // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
       _.merge({}, quickStart, {
         spec: {
           type: OdhDocumentType.QuickStart,

frontend/src/concepts/metrics/utils.ts

@@ -1,12 +1,12 @@
 import { SelectOptionObject } from '@patternfly/react-core/deprecated';
 import { TimeframeTitle, RefreshIntervalTitle } from '~/concepts/metrics/types';
+import { isEnumMember } from '~/utilities/utils';
 
 export const isTimeframeTitle = (
   timeframe: string | SelectOptionObject,
-): timeframe is TimeframeTitle =>
-  Object.values(TimeframeTitle).includes(timeframe as TimeframeTitle);
+): timeframe is TimeframeTitle => isEnumMember(timeframe.toString(), TimeframeTitle);
 
 export const isRefreshIntervalTitle = (
   refreshInterval: string | SelectOptionObject,
 ): refreshInterval is RefreshIntervalTitle =>
-  Object.values(RefreshIntervalTitle).includes(refreshInterval as RefreshIntervalTitle);
+  isEnumMember(refreshInterval.toString(), RefreshIntervalTitle);

frontend/src/concepts/modelRegistry/context/ModelRegistryContext.tsx

@@ -28,6 +28,7 @@ export const ModelRegistryContext = React.createContext<ModelRegistryContextType
   hasCR: false,
   crInitializing: false,
   serverTimedOut: false,
+  // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
   apiState: { apiAvailable: false, api: null as unknown as ModelRegistryAPIState['api'] },
   ignoreTimedOut: () => undefined,
   refreshState: async () => undefined,

frontend/src/concepts/pipelines/content/PipelineAndVersionContext.tsx

@@ -77,10 +77,10 @@ const PipelineAndVersionContextProvider: React.FC<PipelineAndVersionContextProvi
       }),
       getResourcesForDeletion: () => ({
         pipelines: selectedPipelines,
-        versions: (Object.values(selectedVersions) as SelectedVersion[])
+        versions: Object.values(selectedVersions)
           .map((selectedVersion) =>
-            selectedVersion.versions.map((version) => ({
-              pipelineName: selectedVersion.pipelineName,
+            selectedVersion!.versions.map((version) => ({
+              pipelineName: selectedVersion!.pipelineName,
               version,
             })),
           )