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

@@ -29,10 +29,16 @@ export const useModelServingMetrics = (
   refreshInterval: RefreshIntervalTitle,
   namespace: string,
 ): {
-  data: Record<
-    ServerMetricType | ModelMetricType,
-    ContextResourceData<PrometheusQueryRangeResultValue | PrometheusQueryRangeResponseDataResult>
-  >;
+  data: {
+    [ServerMetricType.REQUEST_COUNT]: ContextResourceData<PrometheusQueryRangeResultValue>;
+    [ServerMetricType.AVG_RESPONSE_TIME]: ContextResourceData<PrometheusQueryRangeResponseDataResult>;
+    [ServerMetricType.CPU_UTILIZATION]: ContextResourceData<PrometheusQueryRangeResultValue>;
+    [ServerMetricType.MEMORY_UTILIZATION]: ContextResourceData<PrometheusQueryRangeResultValue>;
+    [ModelMetricType.REQUEST_COUNT_FAILED]: ContextResourceData<PrometheusQueryRangeResultValue>;
+    [ModelMetricType.REQUEST_COUNT_SUCCESS]: ContextResourceData<PrometheusQueryRangeResultValue>;
+    [ModelMetricType.TRUSTY_AI_SPD]: ContextResourceData<PrometheusQueryRangeResponseDataResult>;
+    [ModelMetricType.TRUSTY_AI_DIR]: ContextResourceData<PrometheusQueryRangeResponseDataResult>;
+  };
   refresh: () => void;
 } => {
   const [end, setEnd] = React.useState(lastUpdateTime);
@@ -43,7 +49,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 +59,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 +70,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 +79,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 +90,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 +101,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 +112,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 +122,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) ?? OdhDocumentType.Documentation;
   const docName = odhDoc.metadata.name;
   const duration = odhDoc.spec.durationMinutes;
 

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,7 +26,8 @@ const OdhDocListItem: React.FC<OdhDocCardProps> = ({ odhDoc, favorite, updateFav
   };
 
   const renderTypeBadge = () => {
-    const docType = odhDoc.spec.type as OdhDocumentType;
+    const docType =
+      asEnumMember(odhDoc.spec.type, OdhDocumentType) ?? OdhDocumentType.Documentation;
     const typeBadgeClasses = classNames('odh-list-item__partner-badge odh-m-doc', {
       'odh-m-documentation': docType === OdhDocumentType.Documentation,
       'odh-m-tutorial': docType === OdhDocumentType.Tutorial,

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

@@ -15,8 +15,7 @@ const getFlags = (dashboardConfigSpec: DashboardConfigKind['spec']): FlagState =
     typeof value === 'boolean';
 
   return {
-    ...Object.keys(flags).reduce<FlagState>((acc, key) => {
-      const value = flags[key as FeatureFlag];
+    ...Object.entries(flags).reduce<FlagState>((acc, [key, value]) => {
       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,10 @@ const DashboardSearchField: React.FC<DashboardSearchFieldProps> = ({
         }))}
         value={searchType}
         onChange={(key) => {
-          onSearchTypeChange(key as SearchType);
+          const enumMember = asEnumMember(key, SearchType);
+          if (enumMember !== null) {
+            onSearchTypeChange(enumMember);
+          }
         }}
         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,12 +77,13 @@ const PipelineAndVersionContextProvider: React.FC<PipelineAndVersionContextProvi
       }),
       getResourcesForDeletion: () => ({
         pipelines: selectedPipelines,
-        versions: (Object.values(selectedVersions) as SelectedVersion[])
-          .map((selectedVersion) =>
-            selectedVersion.versions.map((version) => ({
-              pipelineName: selectedVersion.pipelineName,
-              version,
-            })),
+        versions: Object.values(selectedVersions)
+          .map(
+            (selectedVersion) =>
+              selectedVersion?.versions.map((version) => ({
+                pipelineName: selectedVersion.pipelineName,
+                version,
+              })) ?? [],
           )
           .flat()
           .filter((selection) => {

frontend/src/concepts/pipelines/content/compareRuns/metricsSection/PipelineRunArtifactSelect.tsx

@@ -49,10 +49,10 @@ export const PipelineRunArtifactSelect = <T,>({
             isOpen={isOpen}
             selected={selectedItemTitles}
             onSelect={(_event, value) => {
-              if (selectedItemTitles.includes(value as string)) {
+              if (typeof value === 'string' && selectedItemTitles.includes(value)) {
                 setSelectedItemTitles(selectedItemTitles.filter((id) => id !== value));
-              } else {
-                setSelectedItemTitles([...selectedItemTitles, value as string]);
+              } else if (typeof value === 'string') {
+                setSelectedItemTitles([...selectedItemTitles, value]);
               }
             }}
             onOpenChange={(nextOpen: boolean) => setIsOpen(nextOpen)}