feat(claw-app): default audit replay to 2x

[Nikhil (shadowfax92)]

Summary

• Start Audit Replay playback at 2× by default.
• Keep the existing 1×, 2×, and 4× speed controls selectable.
• Cover the visible default selection and manual speed changes with a focused transport test.

Design

The replay transport now names 2× as its default alongside the supported speed list, and usePlayback initializes from that default. The existing controlled toggle group and rrweb speed synchronization remain unchanged.

Test plan

• bun test apps/claw-app/screens/replay/PlaybackTransport.test.tsx
• bun run test from apps/claw-app
• bun run --filter @browseros/claw-app typecheck
• Biome check on all touched files

Baseline notes

• Repository-wide bun run check still reports the two existing noUnsafeOptionalChaining errors in apps/server/tests/lib/agents/acpx-runtime.test.ts; that file is identical to origin/main.
• Repository-wide bun run test still reports the existing stale setup-uv@v8.2.0 assertion in scripts/release/release-claw-server-rust-workflow.test.ts; that file is identical to origin/main.

[Nikhil (shadowfax92)]

Greptile (@greptileai) review

[Nikhil (shadowfax92)]

Claude (@claude) review

[claude]

Claude finished Nikhil (@shadowfax92)'s task in 1m 37s —— View job

---

Review

• Gather context (diff, changed files, CLAUDE.md)
• Review code changes
• Post review feedback

Scope: PLAYBACK_SPEEDS[0] → DEFAULT_PLAYBACK_SPEED (2) in use-playback.ts, a new exported constant in replay.helpers.ts, and a focused RTL-style test for PlaybackTransport. Small, well-targeted diff.

Findings:

• use-playback.ts:26 — usePlayback now seeds speed from DEFAULT_PLAYBACK_SPEED; setPlaybackSpeed still gates on PLAYBACK_SPEEDS.includes(next) (use-playback.ts:36), and 2 is a member of PLAYBACK_SPEEDS, so this is internally consistent.
• Checked the rrweb sync path in Replay.tsx for a stale-speed bug: onPlayerReady (Replay.tsx:157-164) explicitly calls handle.setSpeed(playbackSpeedRef.current) at the moment the player becomes ready, so the new 2x default is correctly applied to the underlying Replayer even though it's constructed with a hardcoded speed: 1 in ReplayViewport.tsx:134. No regression there.
• PlaybackTransport.test.tsx reuses the exact DOM/linkedom/IS_REACT_ACT_ENVIRONMENT harness pattern already used in Replay.test.tsx and ReplayViewport.test.tsx, so it's consistent with existing conventions rather than a one-off. It verifies the 2× button starts aria-pressed="true" and that clicking 1×/4× correctly toggles pressed state.
• Minor nit (non-blocking): DEFAULT_PLAYBACK_SPEED = 2 is a standalone literal rather than derived from PLAYBACK_SPEEDS (e.g. PLAYBACK_SPEEDS[1]). Fine as-is since it's colocated and exported right next to the array, just flagging that the two could silently drift if PLAYBACK_SPEEDS's order ever changes.

Not verified: I wasn't able to execute bun test / bun run check in this sandbox (Bash tool calls beyond git require interactive approval that isn't available here), so I reviewed statically only. The PR body's listed test/typecheck/lint results were not independently re-run.

No correctness, security, or style issues found. LGTM.

[greptile-apps]

Greptile Summary

The PR changes Audit Replay’s initial playback speed from 1× to 2× while retaining the existing 1×, 2×, and 4× controls, and adds a focused transport test covering the default selection and manual changes.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The new default is consistently exposed through usePlayback, the existing speed options remain selectable, and no changed path introduces a concrete playback, build, or test failure.

Important Files Changed

Filename	Overview
packages/browseros-agent/apps/claw-app/screens/replay/replay.helpers.ts	Adds a named 2× default alongside the unchanged supported playback-speed list.
packages/browseros-agent/apps/claw-app/screens/replay/use-playback.ts	Initializes playback speed from the new default constant without changing playback controls or timing logic.
packages/browseros-agent/apps/claw-app/screens/replay/PlaybackTransport.test.tsx	Adds focused coverage confirming the visible 2× default and selection of the remaining supported speeds.

_{Reviews (1): Last reviewed commit: "feat(claw-app): default replays to 2x" | Re-trigger Greptile}

[greptile-apps]

Greptile Summary

Updates Audit Replay to begin playback at 2× by default.

• Adds a named DEFAULT_PLAYBACK_SPEED alongside the existing 1×, 2×, and 4× options.
• Initializes usePlayback with the new default.
• Adds a focused transport test covering the initial selection and manual speed changes.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The new default flows through the existing playback-speed synchronization path, all supported controls remain selectable, and the focused test follows the repository’s established isolated DOM-test pattern.

Important Files Changed

Filename	Overview
packages/browseros-agent/apps/claw-app/screens/replay/replay.helpers.ts	Adds the explicit 2× default without changing the supported playback-speed options.
packages/browseros-agent/apps/claw-app/screens/replay/use-playback.ts	Initializes playback speed from the new default while preserving existing playback behavior.
packages/browseros-agent/apps/claw-app/screens/replay/PlaybackTransport.test.tsx	Verifies the visible 2× default and continued selection of the 1× and 4× controls.

_{Reviews (2): Last reviewed commit: "feat(claw-app): default replays to 2x" | Re-trigger Greptile}

[github-actions]

✅ Tests passed — 1695/1699

Suite	Passed	Failed	Skipped
✅ agent	350/350	0	0
✅ build	34/34	0	0
✅ claw-app	215/215	0	0
⚠️ claw-mcp	0/0	0	0
⚠️ claw-onboard	0/0	0	0
⚠️ claw-server-rust-quality	0/0	0	0
⚠️ claw-server-rust	0/0	0	0
✅ server-agent	314/314	0	0
✅ server-api	171/171	0	0
✅ server-browser	10/10	0	0
✅ server-integration	10/10	0	0
✅ server-lib	299/300	0	1
✅ server-root	38/41	0	3
✅ server-tools	254/254	0	0

View workflow run