feat(app): add tip rack to DropTipLocation as a new option

[koji]

Overview

add tip rack to TipDropLocation as a new option

close AUTH-2183

Test Plan and Hands on Testing

Changelog

Review requests

Risk assessment

[codecov]

Codecov Report

❌ Patch coverage is 3.30579% with 117 lines in your changes missing coverage. Please review.
✅ Project coverage is 25.55%. Comparing base (38816ee) to head (827786f).
⚠️ Report is 18 commits behind head on edge.

Files with missing lines	Patch %	Lines
...ickTransferFlow/utils/retrieveLiquidClassValues.ts	0.00%	29 Missing ⚠️
...ickTransferFlow/utils/generateQuickTransferArgs.ts	0.00%	27 Missing ⚠️
.../QuickTransferFlow/utils/convertBlowoutLocation.ts	0.00%	23 Missing ⚠️
...ms/ODD/QuickTransferFlow/SelectTipDropLocation.tsx	0.00%	22 Missing ⚠️
...p/src/organisms/ODD/QuickTransferFlow/Overview.tsx	0.00%	16 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             edge   #19897      +/-   ##
==========================================
+ Coverage   24.23%   25.55%   +1.31%     
==========================================
  Files        3540     3516      -24     
  Lines      298345   297424     -921     
  Branches    39863    40444     +581     
==========================================
+ Hits        72298    76000    +3702     
+ Misses     226029   221391    -4638     
- Partials       18       33      +15     

Flag	Coverage Δ
protocol-designer	18.76% <2.50%> (+0.05%)	⬆️
step-generation	5.46% <3.33%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
app/src/organisms/ODD/QuickTransferFlow/types.ts	100.00% <ø> (ø)
...eration/src/commandCreators/compound/replaceTip.ts	72.27% <100.00%> (-0.13%)	⬇️
...ion/src/getNextRobotStateAndWarnings/forDropTip.ts	70.45% <100.00%> (ø)
...p/src/organisms/ODD/QuickTransferFlow/Overview.tsx	0.00% <0.00%> (ø)
...ms/ODD/QuickTransferFlow/SelectTipDropLocation.tsx	0.00% <0.00%> (ø)
.../QuickTransferFlow/utils/convertBlowoutLocation.ts	0.00% <0.00%> (ø)
...ickTransferFlow/utils/generateQuickTransferArgs.ts	0.00% <0.00%> (ø)
...ickTransferFlow/utils/retrieveLiquidClassValues.ts	0.00% <0.00%> (ø)

... and 314 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[jerader]

a bit confused by the extension of stepArgs 🤔 left a handful of comments on how to avoid the new stepArg you created.

[ddcc4]

What does this mean? When happens when dropTipLocation IS a string? And what value would dropTipLocation contain if it is a string?

[ncdiehl11]

maybe refactor CutoutConfig | string to type for reuse

[ncdiehl11]

see type comment above

[ncdiehl11]

Unless I'm misreading these changes, it looks like dropTipLocation is never properly passed as the tiprack URI

[koji]

debug-protocol_name-1761662992975.py

"dropTipLocation":"opentrons/opentrons_flex_96_tiprack_200ul/1"

[koji]

trashbin case
debug-protocol_name-1761663196949.py

[ncdiehl11]

Looks good if tested on robot

Nick approved