fix(protocol-designer): document that pipettes.tiprackURI is a labwareId, not a URI

Author: ddcc4

step-generation/src/types.ts

@@ -707,6 +707,7 @@ export interface TimelineFrame {
     pipettes: {
       [pipetteId: string]: {
         hasTip: boolean
+        /** TODO: tiprackURI is a misnomer: it's a labwareId, not a URI */
         tiprackURI: string | null
       }
     }

step-generation/src/utils/safePipetteMovements.ts

@@ -344,14 +344,10 @@ export const getIsSafePipetteMovement = (args: {
     return true
   }
 
-  const tiprackURI = tipState.pipettes[pipetteId]?.tiprackURI
-  const tiprackEntityId =
-    tiprackURI != null
-      ? Object.keys(labwareEntities).find(lwKey => lwKey.includes(tiprackURI))
-      : null
+  const tiprackEntityId = tipState.pipettes[pipetteId]?.tiprackURI
   const tiprackTipLength =
     tiprackEntityId != null
-      ? labwareEntities[tiprackEntityId].def.parameters.tipLength
+      ? labwareEntities[tiprackEntityId]?.def.parameters.tipLength
       : 0
   const stagingAreaSlots = Object.values(stagingAreaEntities).map(
     stagingArea => stagingArea.location as string