CodeIssuesPull RequestsActionsSecurityInsights
✨ AI
More
Settings

fix: bugs in new landing page components (LiveDemo, ROICalculator, SEO pages) #4745

Merged⚡ AI-generatedXSccantynz wants to mergeclaude/session-improvements-1781303024mainopened Jun 12, 20260/5 tasks
ccantynzcommented Jun 12, 2026

Originally written by @ccantynz-alt on GitHub.
Imported from https://github.com/ccantynz-alt/voxlen/pull/56


Summary

  • LiveDemo transcript bug: prev + final || interim evaluated as (prev + final) || interim due to operator precedence — when final was empty the transcript was silently reset to just the interim result instead of accumulating. Fixed to prev + (final || interim).
  • ROICalculator NaN/Infinity: sessionsPerDay = (hoursPerDay * 60) / typingMinutes could divide by zero, producing InfinityNaN down the chain → rendered "NaN×" in the UI. Added typingMinutes > 0 guard and isFinite check.
  • SEOPage <title> JSX noop: <title> rendered as JSX has no effect on the actual browser tab or search result title. Replaced with useEffect(() => { document.title = ... }).
  • Intel Mac download: detectPlatform() returned "mac-arm" for all Macs, so Intel Mac users got the wrong download binary. Now detects Intel via UA string /Intel/i or /x86_64/i.

Test plan

  • TypeScript clean (npx tsc --noEmit)
  • LiveDemo: speech recognition transcript accumulates correctly across multiple finals/interims
  • ROICalculator: no "NaN×" renders at min input values
  • SEO pages: browser tab title updates when navigating to /voice-dictation-software etc.
  • Intel Mac: detectPlatform() returns "mac-intel" on Intel hardware

https://claude.ai/code/session_011MFNHeFKrhfuSrEYVgrdyr


Generated by Claude Code

ccantynzcommented Jun 12, 2026

Originally written by @vercel[bot] on GitHub.


The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
voxlen Ready Ready Preview, Comment Jun 12, 2026 10:40pm

Cross-repo impact

See what breaks downstream if this PR merges.

Analyze →
⮌ Merged

This pull request was merged into main.

c comment · e edit title · m merge · a approve · r request changes · ? shortcuts