fix(app): clone run with RTPs from HistoricalProtocolRun

api-client/src/runs/constants.ts

@@ -0,0 +1,11 @@
+import {
+  RUN_STATUS_FAILED,
+  RUN_STATUS_STOPPED,
+  RUN_STATUS_SUCCEEDED,
+} from './types'
+
+export const RUN_STATUSES_TERMINAL = [
+  RUN_STATUS_SUCCEEDED,
+  RUN_STATUS_FAILED,
+  RUN_STATUS_STOPPED,
+]

api-client/src/runs/index.ts

@@ -10,6 +10,6 @@ export { getCommands } from './commands/getCommands'
 export { createRunAction } from './createRunAction'
 export * from './createLabwareOffset'
 export * from './createLabwareDefinition'
-
+export * from './constants'
 export * from './types'
 export type { CreateRunData } from './createRun'

api-client/src/runs/types.ts

@@ -4,6 +4,7 @@ import type {
   LoadedPipette,
   ModuleModel,
   RunTimeCommand,
+  RunTimeParameter,
 } from '@opentrons/shared-data'
 import type { ResourceLink, ErrorDetails } from '../types'
 export * from './commands/types'
@@ -47,7 +48,7 @@ export interface LegacyGoodRunData {
   modules: LoadedModule[]
   protocolId?: string
   labwareOffsets?: LabwareOffset[]
-  runTimeParameterValues?: RunTimeParameterCreateData
+  runTimeParameters: RunTimeParameter[]
 }
 
 export interface KnownGoodRunData extends LegacyGoodRunData {

app/src/organisms/Devices/ProtocolRun/ProtocolRunHeader.tsx

@@ -15,6 +15,7 @@ import {
   RUN_STATUS_SUCCEEDED,
   RUN_STATUS_BLOCKED_BY_OPEN_DOOR,
   RUN_STATUS_AWAITING_RECOVERY,
+  RUN_STATUSES_TERMINAL,
 } from '@opentrons/api-client'
 import {
   useModulesQuery,
@@ -128,11 +129,6 @@ const CANCELLABLE_STATUSES = [
   RUN_STATUS_IDLE,
   RUN_STATUS_AWAITING_RECOVERY,
 ]
-const RUN_OVER_STATUSES: RunStatus[] = [
-  RUN_STATUS_FAILED,
-  RUN_STATUS_STOPPED,
-  RUN_STATUS_SUCCEEDED,
-]
 
 interface ProtocolRunHeaderProps {
   protocolRunHeaderRef: React.RefObject<HTMLDivElement> | null
@@ -215,7 +211,11 @@ export function ProtocolRunHeader({
       if (runStatus === RUN_STATUS_IDLE) {
         setShowDropTipBanner(true)
         setPipettesWithTip([])
-      } else if (runStatus != null && RUN_OVER_STATUSES.includes(runStatus)) {
+      } else if (
+        runStatus != null &&
+        // @ts-expect-error runStatus expected to possibly not be terminal
+        RUN_STATUSES_TERMINAL.includes(runStatus)
+      ) {
         getPipettesWithTipAttached({
           host,
           runId,

app/src/organisms/Devices/ProtocolRun/ProtocolRunRunTimeParameters.tsx

@@ -1,6 +1,11 @@
 import * as React from 'react'
 import { useTranslation } from 'react-i18next'
 import styled, { css } from 'styled-components'
+import {
+  RUN_ACTION_TYPE_PLAY,
+  RUN_STATUS_STOPPED,
+  RUN_STATUSES_TERMINAL,
+} from '@opentrons/api-client'
 import { formatRunTimeParameterValue } from '@opentrons/shared-data'
 import {
   ALIGN_CENTER,
@@ -23,8 +28,11 @@ import { Banner } from '../../../atoms/Banner'
 import { Divider } from '../../../atoms/structure'
 import { Tooltip } from '../../../atoms/Tooltip'
 import { useMostRecentCompletedAnalysis } from '../../LabwarePositionCheck/useMostRecentCompletedAnalysis'
+import { useRunStatus } from '../../RunTimeControl/hooks'
+import { useNotifyRunQuery } from '../../../resources/runs'
 
 import type { RunTimeParameter } from '@opentrons/shared-data'
+import type { RunStatus } from '@opentrons/api-client'
 
 interface ProtocolRunRuntimeParametersProps {
   runId: string
@@ -34,13 +42,29 @@ export function ProtocolRunRuntimeParameters({
 }: ProtocolRunRuntimeParametersProps): JSX.Element {
   const { t } = useTranslation('protocol_setup')
   const mostRecentAnalysis = useMostRecentCompletedAnalysis(runId)
-  const runTimeParameters = mostRecentAnalysis?.runTimeParameters ?? []
-  const hasParameter = runTimeParameters.length > 0
-
-  const hasCustomValues = runTimeParameters.some(
+  const runStatus = useRunStatus(runId)
+  const isRunTerminal =
+    runStatus == null
+      ? false
+      : (RUN_STATUSES_TERMINAL as RunStatus[]).includes(runStatus)
+  // we access runTimeParameters from the run record rather than the most recent analysis
+  // because the most recent analysis may not reflect the selected run (e.g. cloning a run
+  // from a historical protocol run from the device details page)
+  const run = useNotifyRunQuery(runId).data
+  const runTimeParameters =
+    (isRunTerminal
+      ? run?.data?.runTimeParameters
+      : mostRecentAnalysis?.runTimeParameters) ?? []
+  const hasRunTimeParameters = runTimeParameters.length > 0
+  const hasCustomRunTimeParameterValues = runTimeParameters.some(
     parameter => parameter.value !== parameter.default
   )
 
+  const runActions = run?.data.actions
+  const hasRunStarted = runActions?.some(
+    action => action.actionType === RUN_ACTION_TYPE_PLAY
+  )
+
   return (
     <>
       <Flex
@@ -56,13 +80,15 @@ export function ProtocolRunRuntimeParameters({
           <StyledText as="h3" fontWeight={TYPOGRAPHY.fontWeightSemiBold}>
             {t('parameters')}
           </StyledText>
-          {hasParameter ? (
+          {hasRunTimeParameters ? (
             <StyledText as="label" color={COLORS.grey60}>
-              {hasCustomValues ? t('custom_values') : t('default_values')}
+              {hasCustomRunTimeParameterValues
+                ? t('custom_values')
+                : t('default_values')}
             </StyledText>
           ) : null}
         </Flex>
-        {hasParameter ? (
+        {hasRunTimeParameters ? (
           <Banner
             type="informing"
             width="100%"
@@ -77,9 +103,15 @@ export function ProtocolRunRuntimeParameters({
           </Banner>
         ) : null}
       </Flex>
-      {!hasParameter ? (
+      {!hasRunTimeParameters ? (
         <Flex padding={SPACING.spacing16}>
-          <InfoScreen contentType="parameters" />
+          <InfoScreen
+            contentType={
+              !hasRunStarted && runStatus === RUN_STATUS_STOPPED
+                ? 'runNotStarted'
+                : 'parameters'
+            }
+          />
         </Flex>
       ) : (
         <>

app/src/organisms/Devices/ProtocolRun/__tests__/ProtocolRunRuntimeParameters.test.tsx

@@ -1,12 +1,17 @@
 import * as React from 'react'
+import { UseQueryResult } from 'react-query'
 import { describe, it, vi, beforeEach, afterEach, expect } from 'vitest'
 import { screen } from '@testing-library/react'
 import { when } from 'vitest-when'
 
+import { Run } from '@opentrons/api-client'
 import { InfoScreen } from '@opentrons/components'
 import { renderWithProviders } from '../../../../__testing-utils__'
 import { i18n } from '../../../../i18n'
 import { useMostRecentCompletedAnalysis } from '../../../LabwarePositionCheck/useMostRecentCompletedAnalysis'
+import { useRunStatus } from '../../../RunTimeControl/hooks'
+import { useNotifyRunQuery } from '../../../../resources/runs'
+import { mockSucceededRun } from '../../../RunTimeControl/__fixtures__'
 
 import { ProtocolRunRuntimeParameters } from '../ProtocolRunRunTimeParameters'
 
@@ -23,6 +28,8 @@ vi.mock('@opentrons/components', async importOriginal => {
   }
 })
 vi.mock('../../../LabwarePositionCheck/useMostRecentCompletedAnalysis')
+vi.mock('../../../RunTimeControl/hooks')
+vi.mock('../../../../resources/runs')
 
 const RUN_ID = 'mockId'
 
@@ -100,13 +107,17 @@ describe('ProtocolRunRuntimeParameters', () => {
       .thenReturn({
         runTimeParameters: mockRunTimeParameterData,
       } as CompletedProtocolAnalysis)
+    vi.mocked(useRunStatus).mockReturnValue('running')
+    vi.mocked(useNotifyRunQuery).mockReturnValue(({
+      data: { data: mockSucceededRun },
+    } as unknown) as UseQueryResult<Run>)
   })
 
   afterEach(() => {
     vi.resetAllMocks()
   })
 
-  it('should render title, and banner when RunTimeParameters are note empty and all values are default', () => {
+  it('should render title, and banner when RunTimeParameters are not empty and all values are default', () => {
     render(props)
     screen.getByText('Parameters')
     screen.getByText('Default values')
@@ -116,7 +127,7 @@ describe('ProtocolRunRuntimeParameters', () => {
     screen.getByText('Value')
   })
 
-  it('should render title, and banner when RunTimeParameters are note empty and some value is changed', () => {
+  it('should render title, and banner when RunTimeParameters are not empty and some value is changed', () => {
     vi.mocked(useMostRecentCompletedAnalysis).mockReturnValue({
       runTimeParameters: [
         ...mockRunTimeParameterData,
@@ -139,7 +150,7 @@ describe('ProtocolRunRuntimeParameters', () => {
     screen.getByText('Value')
   })
 
-  it('should render RunTimeParameters when RunTimeParameters are note empty', () => {
+  it('should render RunTimeParameters when RunTimeParameters are not empty', () => {
     render(props)
     screen.getByText('Dry Run')
     screen.getByText('Off')

app/src/organisms/Devices/hooks/useRunStatuses.ts

@@ -1,15 +1,15 @@
 import {
+  RUN_STATUSES_TERMINAL,
   RUN_STATUS_AWAITING_RECOVERY,
-  RUN_STATUS_FAILED,
   RUN_STATUS_IDLE,
   RUN_STATUS_PAUSED,
   RUN_STATUS_RUNNING,
-  RUN_STATUS_STOPPED,
-  RUN_STATUS_SUCCEEDED,
 } from '@opentrons/api-client'
 import { useCurrentRunId } from '../../ProtocolUpload/hooks'
 import { useRunStatus } from '../../RunTimeControl/hooks'
 
+import type { RunStatus } from '@opentrons/api-client'
+
 interface RunStatusesInfo {
   isRunStill: boolean
   isRunTerminal: boolean
@@ -29,9 +29,9 @@ export function useRunStatuses(): RunStatusesInfo {
     runStatus === RUN_STATUS_RUNNING ||
     runStatus === RUN_STATUS_AWAITING_RECOVERY
   const isRunTerminal =
-    runStatus === RUN_STATUS_SUCCEEDED ||
-    runStatus === RUN_STATUS_STOPPED ||
-    runStatus === RUN_STATUS_FAILED
+    runStatus != null
+      ? (RUN_STATUSES_TERMINAL as RunStatus[]).includes(runStatus)
+      : false
   const isRunStill = isRunTerminal || isRunIdle
 
   return { isRunStill, isRunTerminal, isRunIdle, isRunRunning }

app/src/organisms/InterventionModal/__fixtures__/index.ts

@@ -188,6 +188,7 @@ export const mockRunData: RunData = {
   pipettes: [],
   labware: [mockLabwareOnModule, mockLabwareOnSlot, mockLabwareOffDeck],
   modules: [mockModule],
+  runTimeParameters: [],
 }
 
 export const mockLabwareRenderInfo = [

app/src/organisms/OnDeviceDisplay/RobotDashboard/__tests__/RecentRunProtocolCarousel.test.tsx

@@ -26,6 +26,7 @@ const mockRun = {
   pipettes: [],
   protocolId: 'mockSortedProtocolID',
   status: 'stopped',
+  runTimeParameters: [],
 }
 
 const render = (

app/src/organisms/ProtocolUpload/hooks/__tests__/useCloneRun.test.tsx

@@ -4,7 +4,11 @@ import { renderHook } from '@testing-library/react'
 import { QueryClient, QueryClientProvider } from 'react-query'
 import { describe, it, beforeEach, afterEach, vi, expect } from 'vitest'
 
-import { useHost, useCreateRunMutation } from '@opentrons/react-api-client'
+import {
+  useHost,
+  useCreateRunMutation,
+  useCreateProtocolAnalysisMutation,
+} from '@opentrons/react-api-client'
 
 import { useCloneRun } from '../useCloneRun'
 import { useNotifyRunQuery } from '../../../../resources/runs'
@@ -30,13 +34,16 @@ describe('useCloneRun hook', () => {
             id: RUN_ID,
             protocolId: 'protocolId',
             labwareOffsets: 'someOffset',
-            runTimeParameterValues: 'someRtp',
+            runTimeParameters: [],
           },
         },
       } as any)
     when(vi.mocked(useCreateRunMutation))
       .calledWith(expect.anything())
       .thenReturn({ createRun: vi.fn() } as any)
+    vi.mocked(useCreateProtocolAnalysisMutation).mockReturnValue({
+      createProtocolAnalysis: vi.fn(),
+    } as any)
 
     const queryClient = new QueryClient()
     const clientProvider: React.FunctionComponent<{
@@ -61,7 +68,7 @@ describe('useCloneRun hook', () => {
     expect(mockCreateRun).toHaveBeenCalledWith({
       protocolId: 'protocolId',
       labwareOffsets: 'someOffset',
-      runTimeParameterValues: 'someRtp',
+      runTimeParameterValues: {},
     })
   })
 })

app/src/organisms/ProtocolUpload/hooks/useCloneRun.ts

@@ -1,10 +1,13 @@
 import { useQueryClient } from 'react-query'
 
-import { useHost, useCreateRunMutation } from '@opentrons/react-api-client'
-
+import {
+  useHost,
+  useCreateRunMutation,
+  useCreateProtocolAnalysisMutation,
+} from '@opentrons/react-api-client'
 import { useNotifyRunQuery } from '../../../resources/runs'
 
-import type { Run } from '@opentrons/api-client'
+import type { Run, RunTimeParameterCreateData } from '@opentrons/api-client'
 
 interface UseCloneRunResult {
   cloneRun: () => void
@@ -13,28 +16,45 @@ interface UseCloneRunResult {
 
 export function useCloneRun(
   runId: string | null,
-  onSuccessCallback?: (createRunResponse: Run) => unknown
+  onSuccessCallback?: (createRunResponse: Run) => unknown,
+  triggerAnalysis: boolean = false
 ): UseCloneRunResult {
   const host = useHost()
   const queryClient = useQueryClient()
   const { data: runRecord } = useNotifyRunQuery(runId)
+  const protocolKey = runRecord?.data.protocolId ?? null
+
   const { createRun, isLoading } = useCreateRunMutation({
     onSuccess: response => {
-      queryClient
-        .invalidateQueries([host, 'runs'])
-        .catch((e: Error) =>
-          console.error(`error invalidating runs query: ${e.message}`)
-        )
+      const invalidateRuns = queryClient.invalidateQueries([host, 'runs'])
+      const invalidateProtocols = queryClient.invalidateQueries([
+        host,
+        'protocols',
+        protocolKey,
+      ])
+      Promise.all([invalidateRuns, invalidateProtocols]).catch((e: Error) =>
+        console.error(`error invalidating runs query: ${e.message}`)
+      )
       if (onSuccessCallback != null) onSuccessCallback(response)
     },
   })
+  const { createProtocolAnalysis } = useCreateProtocolAnalysisMutation(
+    protocolKey,
+    host
+  )
   const cloneRun = (): void => {
     if (runRecord != null) {
-      const {
-        protocolId,
-        labwareOffsets,
-        runTimeParameterValues,
-      } = runRecord.data
+      const { protocolId, labwareOffsets, runTimeParameters } = runRecord.data
+      const runTimeParameterValues = runTimeParameters.reduce<RunTimeParameterCreateData>(
+        (acc, param) =>
+          param.value !== param.default
+            ? { ...acc, [param.variableName]: param.value }
+            : acc,
+        {}
+      )
+      if (triggerAnalysis && protocolKey != null) {
+        createProtocolAnalysis({ protocolKey, runTimeParameterValues })
+      }
       createRun({ protocolId, labwareOffsets, runTimeParameterValues })
     } else {
       console.info('failed to clone run record, source run record not found')