b524fa5e

By: Michael Lynch <git@mtlynch.io>

Extract readVideoUploadParts so the parser never sees a nil tmpPath

The nilable *UploadPath and its three roles (duplicate-file detection,
error-path cleanup, missing-file validation) now live entirely inside
readVideoUploadParts, which guarantees a valid temp path on success.
parseVideoUploadPostRequest works purely with values.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Suite timing

Time to Start Worker time Duration Time to finish Idle
Config 2s 10s 10s 13s 2s
Eval 9s 1m27s 1m27s 1m37s 0s
Build 1m26s 6m44s 2m34s 4m01s 0s
Suite 2s 8m22s 3m59s 4m01s 2s

Timeline

0s20s40s1m1m20s1m40s2m2m20s2m40s3m3m20s3m40s4m