Skip to content

fix(frontend): code review polish (CR01-CR04)#16

Merged
ozymandiashh merged 1 commit intomainfrom
fix/code-review-polish
Mar 22, 2026
Merged

fix(frontend): code review polish (CR01-CR04)#16
ozymandiashh merged 1 commit intomainfrom
fix/code-review-polish

Conversation

@ozymandiashh
Copy link
Copy Markdown
Owner

Summary

4 low-severity fixes from code review:

  • CR01: wizard.html still said "Sync" instead of "Mirror"
  • CR02/CR03: [F314] and [F312] console.log fired every 2s poll, now removed
  • CR04: peakSpeed sessionStorage cleared on new transfer start

Test plan

  • Wizard mode selection shows "Mirror" not "Sync"
  • Browser console clean during transfer (no spam every 2s)
  • Start new transfer → peak speed starts fresh
  • python3 -m pytest -x -q passes (501 passed)

…peak speed

- CR01: wizard.html Sync → Mirror label
- CR02: remove [F314] console.log from poll cycle
- CR03: remove [F312] console.log from poll cycle
- CR04: clear peakSpeed sessionStorage on new transfer

Co-Authored-By: Claude <noreply@anthropic.com>
@ozymandiashh ozymandiashh merged commit 8433d86 into main Mar 22, 2026
5 of 15 checks passed
@ozymandiashh ozymandiashh deleted the fix/code-review-polish branch March 22, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant