"fix" (app): allow setup with no pipette

sfoster1 · sfoster1 · commit 43f48283f223 · 2025-08-08T16:14:58.000-04:00
We want to allow module setup flows when there aren't pipettes attached, if the module can be setup without pipettes. Which is some of them, some of the time, from 3 different places. "Closes" RQA-4481
diff --git a/app/src/App/hooks.ts b/app/src/App/hooks.ts
@@ -15,6 +15,10 @@ import {
   useCreateLiveCommandMutation,
   useHost,
 } from '@opentrons/react-api-client'
+import {
+  ABSORBANCE_READER_TYPE,
+  FLEX_STACKER_MODULE_TYPE,
+} from '@opentrons/shared-data'
 
 import { useToaster } from '/app/organisms/ToasterOven'
 import { checkShellUpdate } from '/app/redux/shell'
@@ -26,7 +30,10 @@ import { useCurrentRunId } from '../resources/runs'
 import { SharedScrollRefContext } from './ODDProviders/ScrollRefProvider'
 
 import type { AttachedModule } from '@opentrons/api-client'
-import type { SetStatusBarCreateCommand } from '@opentrons/shared-data'
+import type {
+  ModuleType,
+  SetStatusBarCreateCommand,
+} from '@opentrons/shared-data'
 import type { Dispatch } from '/app/redux/types'
 
 const UPDATE_RECHECK_INTERVAL_MS = 60000
@@ -151,12 +158,28 @@ export function useGetNewModules(): AttachedModule[] {
   return []
 }
 
+const MODULES_NOT_REQUIRING_PIPETTE_FOR_SETUP: ModuleType[] = [
+  ABSORBANCE_READER_TYPE,
+  FLEX_STACKER_MODULE_TYPE,
+]
+
 export function useModuleAttachedToast(
   launchModuleSetupCallback: () => void
 ): void {
   const newModules = useGetNewModules()
   const currentRunId = useCurrentRunId({ refetchInterval: CURRENT_RUN_POLL })
-  const attachedPipettes = useAttachedPipettes(newModules.length > 0)
+  const modulesNotRequiringPipette = newModules
+    .filter(thisModule =>
+      MODULES_NOT_REQUIRING_PIPETTE_FOR_SETUP.includes(thisModule.moduleType)
+    )
+    .map(thisModule => thisModule.serialNumber)
+  const moduleSerialsRequiringPipette = newModules
+    .map(thisModule => thisModule.serialNumber)
+    .filter(thisSerial => !modulesNotRequiringPipette.includes(thisSerial))
+
+  const attachedPipettes = useAttachedPipettes(
+    moduleSerialsRequiringPipette.length > 0
+  )
   const { t, i18n } = useTranslation(['module_wizard_flows', 'shared'])
   const { makeToast } = useToaster()
   const moduleSerials = newModules.map(m => m.serialNumber)
@@ -165,9 +188,19 @@ export function useModuleAttachedToast(
 
   useEffect(() => {
     const newModuleSerials = difference(moduleSerials, moduleSerialsRef.current)
+    const newModulesRequiringPipette = newModuleSerials.filter(serial =>
+      moduleSerialsRequiringPipette.includes(serial)
+    )
+    const newModulesNotRequiringPipette = newModuleSerials.filter(
+      serial => !moduleSerialsRequiringPipette.includes(serial)
+    )
     const hasPipette =
       attachedPipettes.left != null || attachedPipettes.right != null
-    if (!runInProgress && hasPipette && newModuleSerials.length > 0) {
+    if (
+      !runInProgress &&
+      ((hasPipette && newModulesRequiringPipette.length > 0) ||
+        newModulesNotRequiringPipette.length > 0)
+    ) {
       makeToast(t('module_added') as string, 'info', {
         buttonText: i18n.format(t('shared:close'), 'capitalize'),
         linkText: t('module_added_link'),
@@ -179,7 +212,7 @@ export function useModuleAttachedToast(
     moduleSerialsRef.current = moduleSerials
     // dont want this hook to rerun when other deps change
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [moduleSerials, runInProgress])
+  }, [moduleSerials, runInProgress, moduleSerialsRequiringPipette])
 }
 
 export function useScrollRef(): {
diff --git a/app/src/molecules/UpdateBanner/index.tsx b/app/src/molecules/UpdateBanner/index.tsx
@@ -51,14 +51,17 @@ export const UpdateBanner = ({
     return t('module_calibration_required')
   }
 
-  const canProceed =
-    updateType === 'firmware'
-      ? true
-      : !isEstopNotDisengaged &&
-        !isTooHot &&
-        !attachPipetteRequired &&
-        !calibratePipetteRequired &&
-        !updatePipetteFWRequired
+  const proceedChecks = {
+    firmware: () => true,
+    calibration: () =>
+      !isEstopNotDisengaged &&
+      !isTooHot &&
+      !attachPipetteRequired &&
+      !calibratePipetteRequired &&
+      !updatePipetteFWRequired,
+    setup: () => true,
+  }
+  const canProceed = proceedChecks[updateType]()
 
   const getMessage = (): string => {
     switch (updateType) {
diff --git a/app/src/organisms/ModuleWizardFlows/AttachProbe.tsx b/app/src/organisms/ModuleWizardFlows/AttachProbe.tsx
@@ -21,9 +21,9 @@ import { SimpleWizardInProgressBody } from '/app/molecules/SimpleWizardBody'
 import { getFixtureIdByCutoutId } from './getFixtureIdByCutoutId'
 
 import type { CreateCommand, DeckConfiguration } from '@opentrons/shared-data'
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardRequiresPipetteStepProps } from './types'
 
-interface AttachProbeProps extends ModuleSetupWizardStepProps {
+interface AttachProbeProps extends ModuleSetupWizardRequiresPipetteStepProps {
   adapterId: string | null
   deckConfig: DeckConfiguration
 }
diff --git a/app/src/organisms/ModuleWizardFlows/BeforeBeginning.tsx b/app/src/organisms/ModuleWizardFlows/BeforeBeginning.tsx
@@ -12,15 +12,15 @@ import { GenericWizardTile } from '/app/molecules/GenericWizardTile'
 import { WizardRequiredEquipmentList } from '/app/molecules/WizardRequiredEquipmentList'
 
 import type { AttachedModule } from '@opentrons/api-client'
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardMaybePipetteStepProps } from './types'
 
 interface EqipmentItem {
   loadName: string
   displayName: string
   subtitle?: string
 }
 
-interface BeforeBeginningProps extends ModuleSetupWizardStepProps {}
+interface BeforeBeginningProps extends ModuleSetupWizardMaybePipetteStepProps {}
 
 export function BeforeBeginning(props: BeforeBeginningProps): JSX.Element {
   const { proceed, attachedModule, setErrorMessage } = props
diff --git a/app/src/organisms/ModuleWizardFlows/CheckStackerInstall.tsx b/app/src/organisms/ModuleWizardFlows/CheckStackerInstall.tsx
@@ -11,9 +11,10 @@ import { useSendIdentifyStacker } from './hooks'
 
 import type { AttachedModule } from '@opentrons/api-client'
 import type { DeckConfiguration } from '@opentrons/shared-data'
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardMaybePipetteStepProps } from './types'
 
-interface CheckStackerInstallProps extends ModuleSetupWizardStepProps {
+interface CheckStackerInstallProps
+  extends ModuleSetupWizardMaybePipetteStepProps {
   deckConfig: DeckConfiguration
   attachedModules: AttachedModule[]
   doorOpenStatus: boolean
diff --git a/app/src/organisms/ModuleWizardFlows/CloseStackerDoor.tsx b/app/src/organisms/ModuleWizardFlows/CloseStackerDoor.tsx
@@ -13,9 +13,9 @@ import {
 } from '/app/molecules/SimpleWizardBody'
 
 import type { CreateCommand, DeckConfiguration } from '@opentrons/shared-data'
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardMaybePipetteStepProps } from './types'
 
-interface CloseDoorProps extends ModuleSetupWizardStepProps {
+interface CloseDoorProps extends ModuleSetupWizardMaybePipetteStepProps {
   deckConfig: DeckConfiguration
 }
 
diff --git a/app/src/organisms/ModuleWizardFlows/DetachProbe.tsx b/app/src/organisms/ModuleWizardFlows/DetachProbe.tsx
@@ -15,7 +15,7 @@ import detachProbe8 from '/app/assets/videos/pipette-wizard-flows/Pipette_Detach
 import detachProbe96 from '/app/assets/videos/pipette-wizard-flows/Pipette_Detach_Probe_96.webm'
 import { GenericWizardTile } from '/app/molecules/GenericWizardTile'
 
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardRequiresPipetteStepProps } from './types'
 
 const BODY_STYLE = css`
   ${TYPOGRAPHY.pRegular};
@@ -26,7 +26,9 @@ const BODY_STYLE = css`
   }
 `
 
-export function DetachProbe(props: ModuleSetupWizardStepProps): JSX.Element {
+export function DetachProbe(
+  props: ModuleSetupWizardRequiresPipetteStepProps
+): JSX.Element {
   const { attachedPipette, proceed, goBack } = props
   const { t, i18n } = useTranslation('module_wizard_flows')
 
diff --git a/app/src/organisms/ModuleWizardFlows/InstallShuttle.tsx b/app/src/organisms/ModuleWizardFlows/InstallShuttle.tsx
@@ -22,9 +22,9 @@ import { SimpleWizardBody } from '/app/molecules/SimpleWizardBody'
 
 import type { AttachedModule } from '@opentrons/api-client'
 import type { DeckConfiguration } from '@opentrons/shared-data'
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardMaybePipetteStepProps } from './types'
 
-interface InstallShuttleProps extends ModuleSetupWizardStepProps {
+interface InstallShuttleProps extends ModuleSetupWizardMaybePipetteStepProps {
   deckConfig: DeckConfiguration
   attachedModules: AttachedModule[]
 }
diff --git a/app/src/organisms/ModuleWizardFlows/PlaceAdapter.tsx b/app/src/organisms/ModuleWizardFlows/PlaceAdapter.tsx
@@ -33,9 +33,9 @@ import { SimpleWizardInProgressBody } from '/app/molecules/SimpleWizardBody'
 import { LEFT_SLOTS } from './constants'
 
 import type { CreateCommand, DeckConfiguration } from '@opentrons/shared-data'
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardRequiresPipetteStepProps } from './types'
 
-interface PlaceAdapterProps extends ModuleSetupWizardStepProps {
+interface PlaceAdapterProps extends ModuleSetupWizardRequiresPipetteStepProps {
   deckConfig: DeckConfiguration
   setCreatedAdapterId: (adapterId: string) => void
 }
diff --git a/app/src/organisms/ModuleWizardFlows/SelectLocation.tsx b/app/src/organisms/ModuleWizardFlows/SelectLocation.tsx
@@ -42,7 +42,7 @@ import type {
   CutoutId,
   DeckConfiguration,
 } from '@opentrons/shared-data'
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardMaybePipetteStepProps } from './types'
 
 export const BODY_STYLE = css`
   ${TYPOGRAPHY.pRegular};
@@ -52,7 +52,8 @@ export const BODY_STYLE = css`
     line-height: 1.75rem;
   }
 `
-export interface SelectLocationProps extends ModuleSetupWizardStepProps {
+export interface SelectLocationProps
+  extends ModuleSetupWizardMaybePipetteStepProps {
   deckConfig: DeckConfiguration
   createMaintenanceRun: CreateMaintenanceRunType
   isLoadedInRun: boolean
diff --git a/app/src/organisms/ModuleWizardFlows/SelectModule.tsx b/app/src/organisms/ModuleWizardFlows/SelectModule.tsx
@@ -12,7 +12,11 @@ import {
   RESPONSIVENESS,
   SPACING,
 } from '@opentrons/components'
-import { getModuleDisplayName } from '@opentrons/shared-data'
+import {
+  ABSORBANCE_READER_TYPE,
+  FLEX_STACKER_MODULE_TYPE,
+  getModuleDisplayName,
+} from '@opentrons/shared-data'
 
 import { useGetNewModules } from '/app/App/hooks'
 import { SmallButton } from '/app/atoms/buttons'
@@ -27,6 +31,8 @@ import {
 import { useSendIdentifyStacker } from './hooks'
 
 import type { AttachedModule } from '@opentrons/api-client'
+import type { ModuleType } from '@opentrons/shared-data'
+import type { PipetteInformation } from '/app/redux/pipettes'
 
 interface SelectModuleProps {
   buildFlowForSelectedModule: (module: AttachedModule) => void
@@ -35,31 +41,42 @@ interface SelectModuleProps {
   setSelectedModule: (module: AttachedModule | null) => void
   setShowLaunchSetup: (show: boolean) => void
   attachedModuleOnLaunch?: AttachedModule | null
+  attachedPipette?: PipetteInformation | null
 }
 
 interface ModuleNameAndPort {
   name: string
   port: string
 }
 
-export function SelectModule(props: SelectModuleProps): JSX.Element {
+export function SelectModule(props: SelectModuleProps): JSX.Element | null {
   const {
     buildFlowForSelectedModule,
     isOnDevice,
     selectedModule,
     setSelectedModule,
     setShowLaunchSetup,
     attachedModuleOnLaunch = null,
+    attachedPipette,
   } = props
   const { t } = useTranslation('module_wizard_flows')
 
   const { parseModuleUSBPort } = useModuleUSBPort()
   const availableModules = useGetNewModules()
-  const newModules =
+  const allNewModules =
     attachedModuleOnLaunch !== null
       ? [attachedModuleOnLaunch]
       : availableModules
 
+  const modulesNotRequiringPipette = allNewModules.filter(thisModule =>
+    ([
+      ABSORBANCE_READER_TYPE,
+      FLEX_STACKER_MODULE_TYPE,
+    ] as ModuleType[]).includes(thisModule.moduleType)
+  )
+  const newModules =
+    attachedPipette == null ? modulesNotRequiringPipette : allNewModules
+
   const isSingleModule = newModules.length === 1
   const sendIdentifyStacker = useSendIdentifyStacker()
 
@@ -114,8 +131,9 @@ export function SelectModule(props: SelectModuleProps): JSX.Element {
       padding-left: ${SPACING.spacing32};
     }
   `
-
-  if (isSingleModule && selectedModule != null) {
+  if (newModules.length === 0) {
+    return null
+  } else if (isSingleModule && selectedModule != null) {
     const m = getModuleNameAndPort(selectedModule)
     return (
       <SimpleWizardBody
diff --git a/app/src/organisms/ModuleWizardFlows/Success.tsx b/app/src/organisms/ModuleWizardFlows/Success.tsx
@@ -21,7 +21,7 @@ import { SimpleWizardBody } from '/app/molecules/SimpleWizardBody'
 import { useSendIdentifyStacker } from './hooks'
 
 import type { AttachedModule } from '@opentrons/api-client'
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardMaybePipetteStepProps } from './types'
 
 export const BODY_STYLE = css`
   ${TYPOGRAPHY.pRegular};
@@ -32,7 +32,7 @@ export const BODY_STYLE = css`
   }
 `
 
-interface SuccessProps extends ModuleSetupWizardStepProps {
+interface SuccessProps extends ModuleSetupWizardMaybePipetteStepProps {
   setSelectedModule: (module: AttachedModule | null) => void
   attachedModuleOnLaunch?: AttachedModule | null
 }
diff --git a/app/src/organisms/ModuleWizardFlows/UpdateFirmware.tsx b/app/src/organisms/ModuleWizardFlows/UpdateFirmware.tsx
@@ -24,12 +24,12 @@ import { useSendIdentifyStacker } from './hooks'
 
 import type { AttachedModule } from '@opentrons/api-client'
 import type { Dispatch, State } from '/app/redux/types'
-import type { ModuleSetupWizardStepProps } from './types'
+import type { ModuleSetupWizardMaybePipetteStepProps } from './types'
 
 const EQUIPMENT_POLL_MS = 3000
 const MODULE_TIMEOUT_MS = 60000
 const NO_UPDATE_FOUND_TIMEOUT_MS = 2000
-interface UpdateFirmwareProps extends ModuleSetupWizardStepProps {
+interface UpdateFirmwareProps extends ModuleSetupWizardMaybePipetteStepProps {
   robotName: string
   patchModuleAfterUpdate: (module: AttachedModule) => void
 }
diff --git a/app/src/organisms/ModuleWizardFlows/index.tsx b/app/src/organisms/ModuleWizardFlows/index.tsx
@@ -63,6 +63,7 @@ export function ModuleWizardFlows(
     buildFlowForSelectedModule,
     patchModuleAfterUpdate,
     deckConfig,
+    requirePipette,
   } = useModuleSetupWizard({ closeFlow, attachedModuleOnLaunch, onComplete })
 
   // build out flow if there is a module passed in at launch
@@ -89,7 +90,7 @@ export function ModuleWizardFlows(
 
   const doorStatus = useIsDoorOpen(robotName).isDoorOpen
 
-  if (wizardFlowBaseProps.attachedPipette == null) return null
+  if (wizardFlowBaseProps.attachedPipette == null && requirePipette) return null
   if (showLaunchSetup || wizardFlowBaseProps.attachedModule == null) {
     return (
       <ModuleWizardScreen
@@ -112,6 +113,7 @@ export function ModuleWizardFlows(
           setSelectedModule={setSelectedModule}
           setShowLaunchSetup={setShowLaunchSetup}
           attachedModuleOnLaunch={attachedModuleOnLaunch}
+          attachedPipette={wizardFlowBaseProps.attachedPipette}
         />
       </ModuleWizardScreen>
     )
@@ -217,7 +219,7 @@ export function ModuleWizardFlows(
         </ModuleWizardScreen>
       )
     case SECTIONS.PLACE_ADAPTER:
-      return (
+      return wizardFlowBaseProps.attachedPipette == null ? null : (
         <ModuleWizardScreen
           isRobotMoving={wizardFlowBaseProps.isRobotMoving}
           isModuleUpdating={wizardFlowBaseProps.isModuleUpdating}
@@ -236,7 +238,7 @@ export function ModuleWizardFlows(
         </ModuleWizardScreen>
       )
     case SECTIONS.ATTACH_PROBE:
-      return (
+      return wizardFlowBaseProps.attachedPipette == null ? null : (
         <ModuleWizardScreen
           isRobotMoving={wizardFlowBaseProps.isRobotMoving}
           isModuleUpdating={wizardFlowBaseProps.isModuleUpdating}
@@ -255,7 +257,7 @@ export function ModuleWizardFlows(
         </ModuleWizardScreen>
       )
     case SECTIONS.DETACH_PROBE:
-      return (
+      return wizardFlowBaseProps.attachedPipette == null ? null : (
         <ModuleWizardScreen
           isRobotMoving={wizardFlowBaseProps.isRobotMoving}
           isModuleUpdating={wizardFlowBaseProps.isModuleUpdating}
diff --git a/app/src/organisms/ModuleWizardFlows/types.ts b/app/src/organisms/ModuleWizardFlows/types.ts
diff --git a/app/src/organisms/ModuleWizardFlows/useModuleSetupWizard.ts b/app/src/organisms/ModuleWizardFlows/useModuleSetupWizard.ts

Original file line number	Diff line number	Diff line change
`@@ -13,9 +13,9 @@ import {`
`13`	`13`	`} from '/app/molecules/SimpleWizardBody'`
`14`	`14`
`15`	`15`	`import type { CreateCommand, DeckConfiguration } from '@opentrons/shared-data'`
`16`		`-import type { ModuleSetupWizardStepProps } from './types'`
	`16`	`+import type { ModuleSetupWizardMaybePipetteStepProps } from './types'`
`17`	`17`
`18`		`-interface CloseDoorProps extends ModuleSetupWizardStepProps {`
	`18`	`+interface CloseDoorProps extends ModuleSetupWizardMaybePipetteStepProps {`
`19`	`19`	`deckConfig: DeckConfiguration`
`20`	`20`	`}`
`21`	`21`