@titan-design/react-ui 0.6.0 → 0.7.0

package/src/components/custom/Workout/VelocityStrip.tsx

@@ -1,8 +1,18 @@
 // Font mapping: font-heading=Space Grotesk, font-body=Nunito Sans (UI), font-sans=Inter (body)
 import { useEffect, useState } from 'react'
-import { View, Text, Pressable, Animated, Easing, type ViewProps, type ViewStyle } from 'react-native'
+import {
+  View,
+  Text,
+  Pressable,
+  Animated,
+  Easing,
+  type ViewProps,
+  type ViewStyle,
+} from 'react-native'
 import { WORKOUT_TOKENS } from '../../../theme/workout-tokens'
+import { primitiveColors, primitiveRamps, sequentialEffort } from '../../../theme/tokens/primitives'
 import { formatVelocity } from '../../../utils/workout-format'
+import { SET_STRIP_VARIABLE_COLOR } from './SetStrip'
 
 /**
  * Structural velocity-zone band accepted from an upstream analytics source
@@ -32,8 +42,20 @@ export interface VelocityStripProps extends ViewProps {
    * Drives both bar height and — in the default path — zone color. Callers must
    * feed mean (not peak) concentric velocity so bar color, height, and the
    * summary label all describe one metric.
+   *
+   * Optional: provide exactly one of `velocities` or {@link VelocityStripProps.set}.
+   * When neither is given the strip renders nothing.
    */
-  velocities: number[]
+  velocities?: number[]
+  /**
+   * Optional structured strength-training set descriptor (see {@link VelocitySet}).
+   * When present it supersedes `velocities`: the done-velocity array is derived
+   * from the set (so mean / loss / zone summaries still work) and the strip is
+   * drawn as a typed slot list — grey todo, cyan variable / continue windows and
+   * the wide-gap chunking that distinguishes drop / myo / cluster sets. Provide
+   * exactly one of `velocities` or `set`.
+   */
+  set?: VelocitySet
   /**
    * Optional velocity-zone bands (shape-compatible with WA's
    * `VelocityZones.bands`). When provided, bar color and the summary zone label
@@ -46,13 +68,36 @@ export interface VelocityStripProps extends ViewProps {
    * Live mode: index of the most-recently-completed rep. That bar animates in
    * with a "pop"; if it is also the current set peak (a new best), it "bounces"
    * instead. Only the latest bar animates. Honors `prefers-reduced-motion`.
-   * Full variant only.
+   * Framed `expanded` chart only (interactive tap-to-expand use).
    */
   liveRepIndex?: number
+  /**
+   * `expanded` variant: whether the chart is OPEN. Default true (the variant shows
+   * its chart). Toggle it (with {@link onToggle}) for the interactive tap-to-expand
+   * collapse↔chart animation; a static open chart (no `onToggle`) does not animate.
+   */
   expanded?: boolean
   onToggle?: () => void
   onRepPress?: (index: number, velocity: number) => void
-  variant?: 'full' | 'mini'
+  /**
+   * `mini` — a flat 3px static strip (set-type aware). `expanded` — the velocity-
+   * HEIGHT bar chart (rounded tops), whose chrome is prop-driven: with
+   * {@link showNumbers} or {@link showInfo} on it's the framed chart (raised surface,
+   * padding, per-bar m/s labels, mean/loss info row, interactive collapse); with both
+   * off it's a bare strip — the active-set "spotlight" of {@link ExerciseCard}.
+   */
+  variant?: 'mini' | 'expanded'
+  /** `expanded` plot height in px (bars scale to this). Default 60. */
+  height?: number
+  /**
+   * `expanded` bar scaling. `peak` (default) scales to the set's own max (+15%
+   * headroom). `fixed` scales to a fixed velocity ceiling so bar heights read the
+   * same absolute velocity across sets (the spotlight).
+   */
+  scale?: 'peak' | 'fixed'
+  /** `expanded` framed chart: per-bar m/s labels. Default true. */
+  showNumbers?: boolean
+  /** `expanded` framed chart: the mean/loss info row. Default true. */
   showInfo?: boolean
   className?: string
 }
@@ -62,26 +107,25 @@ export interface VelocityStripProps extends ViewProps {
 const VEL_COLORS = WORKOUT_TOKENS.scale
 
 /**
- * Map a velocity-zone id (WA's 5-band taxonomy) onto the canonical 4-color
- * scale. The scale has 4 hues but the taxonomy has 5 bands, so the two slowest
- * bands — `maximalStrength` and `grinding` — intentionally share `red`: both
- * read as "heavy / effortful" and there is no distinct 5th data-viz hue. The
- * top three bands align 1:1 with the legacy default scale so an exercise moving
- * from default to profile-derived zones never shifts its fast-end colors.
- *
- * Legacy default parity: speed=green, power=yellow, strengthSpeed=orange, and
- * the old sub-0.5 "Strength" band (now split into maximalStrength + grinding)
- * stays red.
+ * Map a velocity-zone id (WA's 5-band taxonomy) directly onto the 6-stop
+ * `sequentialEffort` ramp. The default strip uses the 4-color {@link VEL_COLORS}
+ * scale (a subsample of the same ramp), but the 5-band taxonomy has one more
+ * level than that scale carries, so it samples 5 stops: the top four align with
+ * the default scale (speed/power/strengthSpeed/maximalStrength = green/gold/
+ * orange/red) and `grinding` takes the ramp's darker red. `maximalStrength` and
+ * `grinding` therefore stay DISTINCT — they no longer collapse onto one red for
+ * lack of a 5th hue. An exercise moving from default to profile-derived zones
+ * never shifts its fast-end colors.
  *
  * Unknown ids fall back to `green` (matching the historical default-path
  * fallback), so a forward-compatible band id never renders an empty bar.
  */
-const zoneIdToScaleToken: Record<string, keyof typeof VEL_COLORS> = {
-  speed: 'green',
-  power: 'yellow',
-  strengthSpeed: 'orange',
-  maximalStrength: 'red',
-  grinding: 'red',
+const bandIdToEffortColor: Record<string, string> = {
+  speed: sequentialEffort[0], // green-300
+  power: sequentialEffort[2], // amber-300 (gold)
+  strengthSpeed: sequentialEffort[3], // orange-400
+  maximalStrength: sequentialEffort[4], // red-600
+  grinding: sequentialEffort[5], // red-700 — the distinct 5th band
 }
 
 // --- Default (no-zones) scale ------------------------------------------------
@@ -131,7 +175,7 @@ export function calculateMeanVelocity(velocities: number[]): number {
 /** Classify a velocity into its band (slow → fast, min inclusive / max exclusive). */
 function classifyBand(
   velocity: number,
-  bands: readonly VelocityZoneBandProp[],
+  bands: readonly VelocityZoneBandProp[]
 ): VelocityZoneBandProp | undefined {
   for (const band of bands) {
     if (band.max === null || velocity < band.max) return band
@@ -140,8 +184,7 @@ function classifyBand(
 }
 
 function bandColor(band: VelocityZoneBandProp): string {
-  const token = zoneIdToScaleToken[band.id]
-  return token ? VEL_COLORS[token] : VEL_COLORS.green
+  return bandIdToEffortColor[band.id] ?? VEL_COLORS.green
 }
 
 function getLossStyle(loss: number): Record<string, string> | null {
@@ -150,6 +193,197 @@ function getLossStyle(loss: number): Record<string, string> | null {
   return null
 }
 
+// --- Set-type slot model -----------------------------------------------------
+// The strength-training set vocabulary (operator-approved in the Set Modalities
+// exploration): one strip encoding for every per-rep set type. A set resolves to
+// an ordered list of slots; the slot `kind` picks the color and `leadingGap`
+// encodes the chunk pattern (rep gap vs the wide notch that splits drop sub-loads
+// / myo clusters / cluster intra-rests). Set-LEVEL types (pyramid / superset /
+// tempo) belong to the heading, not the strip.
+
+/**
+ * A structured strength-training set. Supply as {@link VelocityStripProps.set} to
+ * render its set-type encoding; the done-velocity array (used for the mean / loss
+ * / zone summary) is derived from the type's velocity fields.
+ */
+export type VelocitySet =
+  /** Fixed reps at a fixed load: done reps colored + a grey todo remainder to `planned`. */
+  | { type: 'straight'; velocities: number[]; planned?: number }
+  /** Bounded reps: committed slots `0..floor`, then a cyan variable window `floor..max`. */
+  | { type: 'range'; velocities: number[]; floor: number; max: number }
+  /** As-many-reps-as-possible: done reps + a trailing cyan "continue" slot that never closes. */
+  | { type: 'amrap'; velocities: number[]; target?: number }
+  /** Load drops with no rest: each sub-load's reps, split by a WIDE notch gap. */
+  | { type: 'drop'; subloads: number[][] }
+  /** Rest-pause: an activation chunk + mini-clusters split by WIDE gaps; `open` adds a cyan continue. */
+  | { type: 'myo'; activation: number[]; clusters: number[][]; open?: boolean }
+  /** Fixed count broken by intra-set rests: reps grouped by `groupSize` with WIDE intra-rest gaps. */
+  | { type: 'cluster'; velocities: number[]; groupSize: number; planned?: number }
+
+/** One drawn cell of the strip. `kind` picks the color; `leadingGap` the spacing before it. */
+interface VelocitySlot {
+  kind: 'rep' | 'todo' | 'variable' | 'continue'
+  velocity?: number
+  leadingGap: number
+}
+
+/** Butted reps carry this gap; chunk boundaries carry {@link WIDE_GAP}. */
+const REP_GAP = 2
+/**
+ * The between-group notch (drop sub-loads / myo clusters / cluster intra-rests): a
+ * fixed 8px total gap that reads as a group break without dominating a narrow strip.
+ * 4× the rep gap — enough to separate groups at session-rail scale AND wall scale
+ * without the notch ballooning at large widths. In the mini variant the container's
+ * 2px gap covers part of it, so per-slot margin adds the remaining 6px (2 + 6 = 8).
+ */
+const WIDE_GAP = 8
+
+/** Grey fill for planned-but-unperformed reps (charcoal placeholder, literal hex). */
+const TODO_COLOR = primitiveColors.charcoal[300]
+/** The thin cyan-800 outline on the open-ended "continue" slot — reads as "keep going". */
+const CONTINUE_OUTLINE = primitiveRamps.cyan[800]
+
+/** The done-velocity array a set contributes to the mean / loss / zone summary. */
+function deriveDoneVelocities(set: VelocitySet): number[] {
+  switch (set.type) {
+    case 'drop':
+      return set.subloads.flat()
+    case 'myo':
+      return [...set.activation, ...set.clusters.flat()]
+    default:
+      return set.velocities
+  }
+}
+
+/** Flatten velocity chunks into rep slots, carving a WIDE gap before each chunk but the first. */
+function chunkedRepSlots(chunks: number[][]): VelocitySlot[] {
+  const slots: VelocitySlot[] = []
+  chunks.forEach((chunk) => {
+    chunk.forEach((velocity, ri) => {
+      const first = slots.length === 0
+      slots.push({ kind: 'rep', velocity, leadingGap: first ? 0 : ri === 0 ? WIDE_GAP : REP_GAP })
+    })
+  })
+  return slots
+}
+
+/** Resolve a set to its ordered slot list — the exact vocabulary of the Set Modalities sheet. */
+function buildSlots(set: VelocitySet): VelocitySlot[] {
+  switch (set.type) {
+    case 'straight': {
+      const total = Math.max(set.velocities.length, set.planned ?? set.velocities.length)
+      return Array.from({ length: total }, (_, i) => {
+        const done = i < set.velocities.length
+        return {
+          kind: done ? 'rep' : 'todo',
+          velocity: done ? set.velocities[i] : undefined,
+          leadingGap: i === 0 ? 0 : REP_GAP,
+        }
+      })
+    }
+    case 'range': {
+      const total = Math.max(set.max, set.velocities.length)
+      return Array.from({ length: total }, (_, i) => {
+        const kind: VelocitySlot['kind'] =
+          i < set.velocities.length ? 'rep' : i < set.floor ? 'todo' : 'variable'
+        return {
+          kind,
+          velocity: kind === 'rep' ? set.velocities[i] : undefined,
+          leadingGap: i === 0 ? 0 : REP_GAP,
+        }
+      })
+    }
+    case 'amrap': {
+      const reps = set.velocities.map<VelocitySlot>((velocity, i) => ({
+        kind: 'rep',
+        velocity,
+        leadingGap: i === 0 ? 0 : REP_GAP,
+      }))
+      reps.push({ kind: 'continue', leadingGap: reps.length === 0 ? 0 : REP_GAP })
+      return reps
+    }
+    case 'drop':
+      return chunkedRepSlots(set.subloads)
+    case 'myo': {
+      const slots = chunkedRepSlots([set.activation, ...set.clusters])
+      if (set.open) slots.push({ kind: 'continue', leadingGap: slots.length === 0 ? 0 : REP_GAP })
+      return slots
+    }
+    case 'cluster': {
+      const total = Math.max(set.velocities.length, set.planned ?? set.velocities.length)
+      return Array.from({ length: total }, (_, i) => {
+        const done = i < set.velocities.length
+        const boundary = i > 0 && i % set.groupSize === 0
+        return {
+          kind: done ? 'rep' : 'todo',
+          velocity: done ? set.velocities[i] : undefined,
+          leadingGap: i === 0 ? 0 : boundary ? WIDE_GAP : REP_GAP,
+        }
+      })
+    }
+  }
+}
+
+/** A concise, set-type-aware summary for the strip's accessibility label. */
+function setAccessibilityLabel(set: VelocitySet, repCount: number): string {
+  switch (set.type) {
+    case 'straight':
+      return `Velocity strip, straight set, ${repCount} reps`
+    case 'range':
+      return `Velocity strip, rep-range set, floor ${set.floor} to ${set.max}, ${repCount} reps done`
+    case 'amrap':
+      return `Velocity strip, AMRAP set, ${repCount} reps and counting`
+    case 'drop':
+      return `Velocity strip, drop set, ${set.subloads.length} loads, ${repCount} reps`
+    case 'myo':
+      return `Velocity strip, myo-reps set, ${set.clusters.length} clusters${set.open ? ', open' : ''}, ${repCount} reps`
+    case 'cluster':
+      return `Velocity strip, cluster set, groups of ${set.groupSize}, ${repCount} reps`
+  }
+}
+
+/** Per-slot accessibility label for the expanded set-type bars. */
+function slotAccessibilityLabel(slot: VelocitySlot, index: number): string {
+  switch (slot.kind) {
+    case 'rep':
+      return `Rep ${index + 1}: ${formatVelocity(slot.velocity ?? 0)} meters per second`
+    case 'todo':
+      return `Rep ${index + 1}: planned`
+    case 'variable':
+      return `Rep ${index + 1}: variable`
+    case 'continue':
+      return 'Keep going'
+  }
+}
+
+/** Straight-set expanded stub height for a planned (todo) rep, as a % of the plot. */
+const EXPANDED_TODO_STUB_PCT = 16
+/** Expanded height for advanced set-types (drop / myo / cluster / range / amrap): a short mini-style bar. */
+const EXPANDED_ENCODED_PCT = 45
+
+/** Default framed `expanded` chart height (px). */
+const EXPANDED_HEIGHT = 60
+/**
+ * The `scale="fixed"` velocity ceiling (m/s). `peak` scales to the set's own max
+ * (+15% headroom); `fixed` uses this constant so a bar's height reads the same
+ * absolute velocity across sets (the spotlight).
+ */
+const FIXED_MAX_VELOCITY = 1.15
+/** Minimum bare-strip bar height (px) — a planned / variable / continue stub, or a near-zero rep. */
+const BARE_STUB_HEIGHT = 3
+
+/** The bar-height scaling denominator for the given `scale` (guarded ≥ 0 by callers). */
+function scaleDenominator(scale: 'peak' | 'fixed', maxVelocity: number): number {
+  return scale === 'fixed' ? FIXED_MAX_VELOCITY : maxVelocity * 1.15
+}
+
+/** Bare-strip bar height (px): velocity-scaled for a performed rep, a short stub otherwise. */
+function bareSlotHeight(slot: VelocitySlot, height: number, denom: number): number {
+  if (slot.kind !== 'rep' || denom <= 0) return BARE_STUB_HEIGHT
+  const ratio = Math.min(1, (slot.velocity ?? 0) / denom)
+  return Math.max(BARE_STUB_HEIGHT, ratio * height)
+}
+
 function getReducedMotionPreference(): boolean {
   if (typeof window === 'undefined' || typeof window.matchMedia !== 'function') return false
   return window.matchMedia('(prefers-reduced-motion: reduce)').matches
@@ -175,38 +409,64 @@ const POP_EASING = Easing.bezier(0.34, 1.56, 0.64, 1)
 
 export function VelocityStrip({
   velocities,
+  set,
   zones,
   liveRepIndex,
-  expanded = false,
+  expanded = true,
   onToggle,
   onRepPress,
-  variant = 'full',
+  variant = 'expanded',
+  height = EXPANDED_HEIGHT,
+  scale = 'peak',
+  showNumbers = true,
   showInfo = true,
   className,
   ...props
 }: VelocityStripProps) {
-  const maxVelocity = Math.max(...velocities, 0)
-  const meanVelocity = calculateMeanVelocity(velocities)
-  const loss = calculateVelocityLoss(velocities)
+  // A `set` descriptor derives its own done-velocity array; the legacy
+  // `velocities` path stays the source of truth otherwise. Every summary calc
+  // (mean / loss / zone) runs on this one array so the info row works either way.
+  const doneVelocities = set ? deriveDoneVelocities(set) : (velocities ?? [])
+  const slots: VelocitySlot[] = set
+    ? buildSlots(set)
+    : doneVelocities.map((v, i) => ({
+        kind: 'rep',
+        velocity: v,
+        leadingGap: i === 0 ? 0 : REP_GAP,
+      }))
+
+  const maxVelocity = Math.max(...doneVelocities, 0)
+  const meanVelocity = calculateMeanVelocity(doneVelocities)
+  const loss = calculateVelocityLoss(doneVelocities)
+
+  // The framed chart (raised box, labels, info) vs the bare spotlight strip is the
+  // only fork in the `expanded` variant — keyed by whether any chrome is requested.
+  const framed = showNumbers || showInfo
+  const scaleDenom = scaleDenominator(scale, maxVelocity)
 
   const hasZones = zones != null && zones.length > 0
   const barColorFor = (v: number): string =>
     hasZones ? bandColor(classifyBand(v, zones)!) : zoneHexMap[getVelocityZoneColor(v)]
+  const slotColor = (slot: VelocitySlot): string => {
+    if (slot.kind === 'rep') return barColorFor(slot.velocity ?? 0)
+    if (slot.kind === 'todo') return TODO_COLOR
+    return SET_STRIP_VARIABLE_COLOR
+  }
   const meanZone = hasZones
     ? (classifyBand(meanVelocity, zones)?.label ?? '')
     : getVelocityZoneName(meanVelocity)
 
   const prefersReducedMotion = usePrefersReducedMotion()
-  const [heightAnim] = useState(() => new Animated.Value(expanded ? 60 : 3))
+  const [heightAnim] = useState(() => new Animated.Value(expanded ? height : 3))
   const [labelOpacity] = useState(() => new Animated.Value(expanded ? 1 : 0))
   const [infoOpacity] = useState(() => new Animated.Value(expanded ? 1 : 0))
   const [liveScale] = useState(() => new Animated.Value(1))
 
   // Newest-rep animation: pop for a normal rep, bounce when it sets a new peak.
-  const liveVelocity = liveRepIndex != null ? velocities[liveRepIndex] : undefined
+  const liveVelocity = liveRepIndex != null ? doneVelocities[liveRepIndex] : undefined
   const isNewPeak = liveVelocity != null && maxVelocity > 0 && liveVelocity === maxVelocity
   useEffect(() => {
-    if (variant === 'mini' || liveRepIndex == null || liveVelocity == null) return
+    if (variant !== 'expanded' || !framed || liveRepIndex == null || liveVelocity == null) return
     if (prefersReducedMotion) {
       liveScale.setValue(1)
       return
@@ -236,14 +496,14 @@ export function VelocityStrip({
         useNativeDriver: true,
       }).start()
     }
-  }, [variant, liveRepIndex, liveVelocity, isNewPeak, prefersReducedMotion, liveScale])
+  }, [variant, framed, liveRepIndex, liveVelocity, isNewPeak, prefersReducedMotion, liveScale])
 
   useEffect(() => {
-    if (variant === 'mini') return
+    if (variant !== 'expanded' || !framed) return
 
     Animated.parallel([
       Animated.timing(heightAnim, {
-        toValue: expanded ? 60 : 3,
+        toValue: expanded ? height : 3,
         duration: ANIMATION_DURATION,
         easing: ANIMATION_EASING,
         useNativeDriver: false,
@@ -261,42 +521,148 @@ export function VelocityStrip({
         useNativeDriver: false,
       }),
     ]).start()
-  }, [expanded, variant, heightAnim, labelOpacity, infoOpacity])
+  }, [expanded, variant, framed, height, heightAnim, labelOpacity, infoOpacity])
+
+  // Nothing to draw: neither a legacy velocity array nor a set descriptor.
+  if (set == null && velocities == null) return null
+
+  const repCount = doneVelocities.length
+  const miniLabel = set ? setAccessibilityLabel(set, repCount) : `Velocity strip, ${repCount} reps`
 
   if (variant === 'mini') {
     const { style: externalStyle, ...restProps } = props
+    // The container keeps the uniform REP_GAP (so a no-`set` strip is byte-identical
+    // to prior releases and holds HTML-parity); per-slot `marginLeft` carries only
+    // the EXTRA spacing for the WIDE notch that chunks drop / myo / cluster sets.
     return (
       <View
         className={className}
-        style={[{ flexDirection: 'row', height: 3, gap: 2, borderRadius: 2, overflow: 'hidden' }, externalStyle]}
+        style={[
+          { flexDirection: 'row', height: 3, gap: REP_GAP, borderRadius: 2, overflow: 'hidden' },
+          externalStyle,
+        ]}
         accessibilityRole="image"
-        accessibilityLabel={`Velocity strip, ${velocities.length} reps`}
+        accessibilityLabel={miniLabel}
         testID="velocity-strip-mini"
         {...restProps}
       >
-        {velocities.map((v, i) => (
+        {slots.map((slot, i) => (
           <View
             key={i}
             style={{
               flex: 1,
-              backgroundColor: barColorFor(v),
+              backgroundColor: slotColor(slot),
               borderRadius: 1,
               minWidth: 4,
               height: '100%' as unknown as number,
+              // The container's uniform 2px gap covers rep spacing; a wide slot adds
+              // only the EXTRA (WIDE_GAP − REP_GAP) so the notch totals WIDE_GAP.
+              marginLeft: Math.max(0, slot.leadingGap - REP_GAP),
+              ...(slot.kind === 'continue'
+                ? { borderWidth: 1, borderColor: CONTINUE_OUTLINE }
+                : {}),
             }}
             accessibilityElementsHidden
-            testID={`velocity-bar-${i}`}
+            testID={slot.kind === 'rep' ? `velocity-bar-${i}` : `velocity-slot-${slot.kind}`}
           />
         ))}
       </View>
     )
   }
 
-  const stripLabel = `Velocity chart for set, ${velocities.length} reps, tap to ${expanded ? 'collapse' : 'expand'}`
+  // Bare `expanded` strip (both chrome flags off): the velocity-HEIGHT spotlight —
+  // px bar heights, no raised box / labels / info row / animation. Set-type gaps carry
+  // over from the slot model exactly as in `mini` (container REP_GAP + per-slot extra
+  // for a WIDE notch); planned / variable / continue slots draw as stubs.
+  if (!framed) {
+    const { style: externalStyle, ...restProps } = props
+    return (
+      <View
+        className={className}
+        style={[
+          { flexDirection: 'row', height, gap: REP_GAP, alignItems: 'flex-end' },
+          externalStyle,
+        ]}
+        accessibilityRole="image"
+        accessibilityLabel={miniLabel}
+        testID="velocity-strip-compact"
+        {...restProps}
+      >
+        {slots.map((slot, i) => (
+          <View
+            key={i}
+            style={{
+              flex: 1,
+              minWidth: 4,
+              height: bareSlotHeight(slot, height, scaleDenom),
+              // Top-rounded to match the framed chart's bars (the "rounded tops from
+              // the numbers one"); square bottoms since bars sit on the baseline.
+              borderTopLeftRadius: 2,
+              borderTopRightRadius: 2,
+              backgroundColor: slotColor(slot),
+              marginLeft: Math.max(0, slot.leadingGap - REP_GAP),
+              ...(slot.kind === 'continue'
+                ? { borderWidth: 1, borderColor: CONTINUE_OUTLINE }
+                : {}),
+            }}
+            accessibilityElementsHidden
+            testID={slot.kind === 'rep' ? `velocity-bar-${i}` : `velocity-slot-${slot.kind}`}
+          />
+        ))}
+      </View>
+    )
+  }
+
+  const stripLabel = set
+    ? `${setAccessibilityLabel(set, repCount)}, tap to ${expanded ? 'collapse' : 'expand'}`
+    : `Velocity chart for set, ${repCount} reps, tap to ${expanded ? 'collapse' : 'expand'}`
   // When onToggle wraps the strip or individual reps are interactive, the container itself is not a button
   const hasInteractiveContainer = onToggle != null
   const hasInteractiveReps = onRepPress != null && expanded
 
+  // Expanded set-type bars: `straight` is the active-set spotlight (velocity-height
+  // done reps + short grey planned stubs); the advanced types render as a short,
+  // mini-style encoding (the gaps + colors carry identity, height is uniform).
+  const setBars =
+    set != null &&
+    slots.map((slot, i) => {
+      const isStraightRep = set.type === 'straight' && slot.kind === 'rep'
+      const heightPct = isStraightRep
+        ? scaleDenom > 0
+          ? Math.round(((slot.velocity ?? 0) / scaleDenom) * 100)
+          : 0
+        : set.type === 'straight'
+          ? EXPANDED_TODO_STUB_PCT
+          : EXPANDED_ENCODED_PCT
+      return (
+        <View
+          key={i}
+          style={{
+            flex: 1,
+            height: '100%',
+            justifyContent: 'flex-end',
+            marginLeft: slot.leadingGap,
+          }}
+        >
+          <View
+            style={{
+              height: `${heightPct}%`,
+              minHeight: 2,
+              borderTopLeftRadius: 2,
+              borderTopRightRadius: 2,
+              backgroundColor: slotColor(slot),
+              ...(slot.kind === 'continue'
+                ? { borderWidth: 1, borderColor: CONTINUE_OUTLINE }
+                : {}),
+            }}
+            testID={slot.kind === 'rep' ? `velocity-bar-${i}` : `velocity-slot-${slot.kind}`}
+            accessibilityRole="image"
+            accessibilityLabel={slotAccessibilityLabel(slot, i)}
+          />
+        </View>
+      )
+    })
+
   const stripContent = (
     <Animated.View
       className={[className, expanded ? 'bg-surface-raised' : 'bg-transparent']
@@ -310,7 +676,9 @@ export function VelocityStrip({
           borderRadius: expanded ? 6 : 2,
           paddingTop: expanded ? 16 : 0,
           paddingBottom: expanded ? 24 : 0,
-          paddingHorizontal: expanded ? 6 : 0,
+          // No horizontal inset — the framed chart's bars span full width, matching
+          // the bare spotlight and mini. Numbers/info sit in the vertical padding.
+          paddingHorizontal: 0,
           paddingVertical: expanded ? undefined : 8,
           overflow: expanded ? 'visible' : 'hidden',
         },
@@ -320,86 +688,106 @@ export function VelocityStrip({
       testID="velocity-strip"
       {...props}
     >
-      <View
-        style={{ flexDirection: 'row', flex: 1, gap: 2, alignItems: 'flex-end' }}
-      >
-        {velocities.map((v, i) => {
-          const barBackground = barColorFor(v)
-          // Guard all-zero velocities (idle / pre-rep): maxVelocity === 0 makes
-          // this 0 / 0 === NaN and emits height:'NaN%'. Flatten the bars instead.
-          const barHeightPct =
-            maxVelocity > 0 ? Math.round((v / (maxVelocity * 1.15)) * 100) : 0
-          const isLive = liveRepIndex === i
-          const liveLabelSuffix = isLive
-            ? isNewPeak
-              ? ', latest rep, new set peak'
-              : ', latest rep'
-            : ''
-
-          const barStyle: ViewStyle = expanded
-            ? { height: `${barHeightPct}%`, minHeight: 2, borderTopLeftRadius: 2, borderTopRightRadius: 2, backgroundColor: barBackground }
-            : {
-                minHeight: '100%' as unknown as number,
-                borderTopLeftRadius: 2,
-                borderTopRightRadius: 2,
-                borderBottomLeftRadius: 0,
-                borderBottomRightRadius: 0,
-                backgroundColor: barBackground,
+      <View style={{ flexDirection: 'row', flex: 1, gap: set ? 0 : 2, alignItems: 'flex-end' }}>
+        {set
+          ? setBars
+          : doneVelocities.map((v, i) => {
+              const barBackground = barColorFor(v)
+              // Guard all-zero velocities (idle / pre-rep): a 0 denominator makes
+              // this 0 / 0 === NaN and emits height:'NaN%'. Flatten the bars instead.
+              const barHeightPct = scaleDenom > 0 ? Math.round((v / scaleDenom) * 100) : 0
+              const isLive = liveRepIndex === i
+              const liveLabelSuffix = isLive
+                ? isNewPeak
+                  ? ', latest rep, new set peak'
+                  : ', latest rep'
+                : ''
+
+              const barStyle: ViewStyle = expanded
+                ? {
+                    height: `${barHeightPct}%`,
+                    minHeight: 2,
+                    borderTopLeftRadius: 2,
+                    borderTopRightRadius: 2,
+                    backgroundColor: barBackground,
+                  }
+                : {
+                    minHeight: '100%' as unknown as number,
+                    borderTopLeftRadius: 2,
+                    borderTopRightRadius: 2,
+                    borderBottomLeftRadius: 0,
+                    borderBottomRightRadius: 0,
+                    backgroundColor: barBackground,
+                  }
+
+              const barInner = isLive ? (
+                <Animated.View
+                  style={[barStyle, { transform: [{ scale: liveScale }] }]}
+                  testID={`velocity-bar-${i}`}
+                  accessibilityRole="image"
+                  accessibilityLabel={`Rep ${i + 1}: ${formatVelocity(v)} meters per second${liveLabelSuffix}`}
+                />
+              ) : (
+                <View
+                  style={barStyle}
+                  testID={`velocity-bar-${i}`}
+                  accessibilityRole="image"
+                  accessibilityLabel={`Rep ${i + 1}: ${formatVelocity(v)} meters per second`}
+                />
+              )
+
+              const bar = (
+                <View
+                  key={i}
+                  style={{
+                    flex: 1,
+                    height: '100%',
+                    justifyContent: 'flex-end',
+                    position: 'relative',
+                  }}
+                >
+                  {expanded && showNumbers && (
+                    <Animated.View
+                      style={{
+                        opacity: labelOpacity,
+                        alignItems: 'center',
+                        position: 'absolute',
+                        top: -13,
+                        left: 0,
+                        right: 0,
+                      }}
+                      accessibilityElementsHidden
+                    >
+                      <Text
+                        className="text-text-secondary"
+                        style={{ fontSize: 8, fontWeight: '600' }}
+                        testID={`velocity-label-${i}`}
+                      >
+                        {formatVelocity(v)}
+                      </Text>
+                    </Animated.View>
+                  )}
+                  {barInner}
+                </View>
+              )
+
+              if (onRepPress && expanded) {
+                return (
+                  <Pressable
+                    key={i}
+                    style={{ flex: 1, height: '100%' }}
+                    onPress={() => onRepPress(i, v)}
+                    accessibilityRole="button"
+                    accessibilityLabel={`Rep ${i + 1}: ${formatVelocity(v)} meters per second, tap for details`}
+                    testID={`velocity-bar-pressable-${i}`}
+                  >
+                    {bar}
+                  </Pressable>
+                )
               }
 
-          const barInner = isLive ? (
-            <Animated.View
-              style={[barStyle, { transform: [{ scale: liveScale }] }]}
-              testID={`velocity-bar-${i}`}
-              accessibilityRole="image"
-              accessibilityLabel={`Rep ${i + 1}: ${formatVelocity(v)} meters per second${liveLabelSuffix}`}
-            />
-          ) : (
-            <View
-              style={barStyle}
-              testID={`velocity-bar-${i}`}
-              accessibilityRole="image"
-              accessibilityLabel={`Rep ${i + 1}: ${formatVelocity(v)} meters per second`}
-            />
-          )
-
-          const bar = (
-            <View
-              key={i}
-              style={{ flex: 1, height: '100%', justifyContent: 'flex-end', position: 'relative' }}
-            >
-              {expanded && (
-                <Animated.View style={{ opacity: labelOpacity, alignItems: 'center', position: 'absolute', top: -13, left: 0, right: 0 }} accessibilityElementsHidden>
-                  <Text
-                    className="text-text-secondary"
-                    style={{ fontSize: 8, fontWeight: '600' }}
-                    testID={`velocity-label-${i}`}
-                  >
-                    {formatVelocity(v)}
-                  </Text>
-                </Animated.View>
-              )}
-              {barInner}
-            </View>
-          )
-
-          if (onRepPress && expanded) {
-            return (
-              <Pressable
-                key={i}
-                style={{ flex: 1, height: '100%' }}
-                onPress={() => onRepPress(i, v)}
-                accessibilityRole="button"
-                accessibilityLabel={`Rep ${i + 1}: ${formatVelocity(v)} meters per second, tap for details`}
-                testID={`velocity-bar-pressable-${i}`}
-              >
-                {bar}
-              </Pressable>
-            )
-          }
-
-          return bar
-        })}
+              return bar
+            })}
       </View>
       {expanded && showInfo && (
         <Animated.View