fix(app): fix labware offset text alignment, fix historical run log missing timestamps (#11114)

Ensure that labware names are left aligned in the LabwareInfoOverlay, even if an offset is present.
Ensure that historical runs which involved more than 60 LPC (or otherwise pre-play) run commands are
able to correctly glean the first post play run command in the absence of a current command key.
Remove unnecessary polling of the protocol record from the run detail page.

Closes #11109, Closes #11108

Author: b-cooper

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

@@ -17,6 +17,7 @@ import {
   Icon,
   DIRECTION_ROW,
   ALIGN_FLEX_START,
+  JUSTIFY_SPACE_BETWEEN,
 } from '@opentrons/components'
 import { StyledText } from '../../../atoms/text'
 
@@ -56,7 +57,7 @@ const LabwareInfo = (props: LabwareInfoProps): JSX.Element | null => {
     >
       <Flex
         flexDirection={DIRECTION_ROW}
-        justifyContent={JUSTIFY_FLEX_END}
+        justifyContent={JUSTIFY_SPACE_BETWEEN}
         alignItems={ALIGN_FLEX_START}
         gridGap={SPACING.spacing2}
       >

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

@@ -78,6 +78,7 @@ export function RunLog({ robotName, runId }: RunLogProps): JSX.Element | null {
   const listInnerRef = React.useRef<HTMLDivElement>(null)
   const currentItemRef = React.useRef<HTMLDivElement>(null)
   const firstPostInitialPlayRunCommandIndex = React.useRef<number | null>(null)
+  const lastKnownPrePlayRunCommandIndex = React.useRef<number>(-1)
   const [isDeterministic, setIsDeterministic] = React.useState<boolean>(true)
   const [windowIndex, setWindowIndex] = React.useState<number>(0)
   const [
@@ -90,7 +91,8 @@ export function RunLog({ robotName, runId }: RunLogProps): JSX.Element | null {
   const prePlayCommandCount =
     firstPostInitialPlayRunCommandIndex.current != null
       ? firstPostInitialPlayRunCommandIndex.current
-      : 0
+      : lastKnownPrePlayRunCommandIndex.current + 1
+
   const isCurrentRun =
     useRunQuery(runId, { staleTime: Infinity }).data?.data.current ?? false
   const { data: commandsData } = useAllCommandsQuery(
@@ -141,6 +143,32 @@ export function RunLog({ robotName, runId }: RunLogProps): JSX.Element | null {
   )
 
   const runStartDateTime = runStartTime != null ? new Date(runStartTime) : null
+  React.useEffect(() => {
+    // if viewing an historical run log, because there is no current command
+    // we may get stuck with no  run commands as the default first window may
+    // contain only "pre-play" commands (e.g. more LPC commands than the window size)
+    // in this case, we'll walk the pointer to the first "post-play" command
+    if (
+      (runStatus === RUN_STATUS_FAILED ||
+        runStatus === RUN_STATUS_STOPPED ||
+        runStatus === RUN_STATUS_SUCCEEDED) &&
+      runStartDateTime != null &&
+      firstPostInitialPlayRunCommandIndex.current === null
+    ) {
+      const foundPostPlayRunCommandIndex = runCommands.findIndex(
+        command => new Date(command?.createdAt) > runStartDateTime
+      )
+      if (foundPostPlayRunCommandIndex >= 0) {
+        firstPostInitialPlayRunCommandIndex.current =
+          lastKnownPrePlayRunCommandIndex.current +
+          foundPostPlayRunCommandIndex +
+          1
+      } else {
+        lastKnownPrePlayRunCommandIndex.current =
+          lastKnownPrePlayRunCommandIndex.current + runCommands.length
+      }
+    }
+  }, [runId, runStatus, runCommands, runStartDateTime])
   const postInitialPlayRunCommands =
     runStartDateTime != null
       ? dropWhile(

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

@@ -1,4 +1,3 @@
-import * as React from 'react'
 import { useProtocolQuery } from '@opentrons/react-api-client'
 import { useCurrentRun } from './useCurrentRun'
 
@@ -7,12 +6,9 @@ import type { Protocol } from '@opentrons/api-client'
 export function useCurrentProtocol(): Protocol | null {
   const currentProtocolId = useCurrentRun()?.data?.protocolId ?? null
 
-  const enableProtocolPolling = React.useRef<boolean>(true)
-  const { data: protocolRecord } = useProtocolQuery(
-    currentProtocolId,
-    { staleTime: Infinity },
-    enableProtocolPolling.current
-  )
+  const { data: protocolRecord } = useProtocolQuery(currentProtocolId, {
+    staleTime: Infinity,
+  })
 
   return protocolRecord ?? null
 }