Skip to content

BrainWaves modernization#189

Merged
teonbrooks merged 7 commits intomainfrom
modernization
Mar 14, 2026
Merged

BrainWaves modernization#189
teonbrooks merged 7 commits intomainfrom
modernization

Conversation

@teonbrooks
Copy link
Collaborator

@teonbrooks teonbrooks commented Mar 7, 2026

This is a continuation of the efforts to modernize BrainWaves with the latest tech.

Key features of this modernization are:

  • migration from SemanticUI to Shadcn/ui
  • migration from connected-react-router to react-router
  • update Pyodide to the latest version
  • add testing

Also trying our hand at using Claude Code and Manus to assist with the modernization efforts

@teonbrooks teonbrooks marked this pull request as draft March 7, 2026 20:44
@teonbrooks teonbrooks mentioned this pull request Mar 7, 2026
5 tasks
- Install @testing-library/dom peer dep (all 25 tests now pass)
- Move @import before @tailwind directives in app.global.css
- Rewrite README dev/build sections: npm, electron-vite, Pyodide note

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jdpigeon jdpigeon marked this pull request as ready for review March 14, 2026 14:54
jdpigeon and others added 5 commits March 14, 2026 11:27
…n system

- Delete all 4 CSS module files (common, collect, topnavbar, secondarynav)
- Add brand/accent/signal color tokens to tailwind.config.js
- Update Button default variant to use brand teal instead of blue
- Add shadcn Select and Badge ui primitives; install @radix-ui/react-select
- Migrate all 20 components to shadcn Card, DropdownMenu, Button, Badge
- Replace nav CSS class string props with typed status/active props
- All 25 tests pass, clean build

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add .worktrees/** to global ignores (was causing 13k false positives)
- Disable all jsx-a11y rules (desktop Electron app, not a public website)
- Fix label-has-associated-control violations in InputCollect.tsx and
  CustomDesignComponent.tsx by adding htmlFor/id pairs
- Upgrade @typescript-eslint/no-explicit-any to error level; existing
  uses already had eslint-disable comments from prior work
- Run lint-fix (prettier formatting across 52 files)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@teonbrooks
Copy link
Collaborator Author

merging into main. thanks @jdpigeon!

@teonbrooks teonbrooks merged commit 9538278 into main Mar 14, 2026
3 checks passed
@teonbrooks teonbrooks deleted the modernization branch March 14, 2026 21:15
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.

2 participants