Skip to content

Update: comprehensive repo improvements — testing, linting, a11y, docs, and dep upgrades#12

Merged
Varnasr merged 1 commit intomainfrom
claude/repo-review-suggestions-Tpf7J
Apr 4, 2026
Merged

Update: comprehensive repo improvements — testing, linting, a11y, docs, and dep upgrades#12
Varnasr merged 1 commit intomainfrom
claude/repo-review-suggestions-Tpf7J

Conversation

@Varnasr
Copy link
Copy Markdown
Owner

@Varnasr Varnasr commented Apr 4, 2026

Summary

  • ESLint + Prettier configured with flat config for consistent code style
  • Vitest + React Testing Library setup with 21 passing tests
  • CI workflow updated to run lint, format check, tests, and build on every PR
  • Accessibility: skip-to-content link, ARIA roles/labels, semantic HTML, keyboard-navigable DataTable
  • Responsive design: Indicators page stacks on tablet/mobile, added 768px and 1024px breakpoints
  • Inline styles → CSS classes: section-title, value-sm/md/lg, card-padded, card-empty, etc.
  • Reusable components: DataTable and FilterBar
  • Dependency upgrades: React 19, React Router 7, Recharts 3
  • Docsify documentation site with architecture, API ref, components, best practices
  • GitHub Pages workflow for automatic docs deployment
  • TypeScript config for IDE support
  • .env.example documenting API configuration
  • Fixed silent error catching in Indicators and Schemes pages

Test plan

  • npm run lint passes (0 errors)
  • npm test passes (21/21 tests)
  • npm run build succeeds

https://claude.ai/code/session_0162zAikEeEs7xSqcq8Fbv4o

…s, and dep upgrades

- Add ESLint + Prettier with flat config for code quality enforcement
- Add Vitest + React Testing Library with 21 passing tests
- Update CI to run lint, format check, tests, and build
- Fix silent error swallowing in Indicators and Schemes pages
- Add accessibility: skip-to-content, ARIA roles, semantic HTML, keyboard nav
- Fix responsive design with tablet/mobile breakpoints for Indicators layout
- Move inline styles to CSS classes (section-title, value-sm, card-empty, etc.)
- Extract reusable DataTable and FilterBar components
- Add TypeScript config (tsconfig.json) for IDE support
- Add .env.example documenting API configuration
- Upgrade to React 19, React Router 7, Recharts 3
- Add Docsify documentation site with architecture, API, and best practices
- Add GitHub Pages workflow for docs deployment

https://claude.ai/code/session_0162zAikEeEs7xSqcq8Fbv4o
@Varnasr Varnasr merged commit 5fb3790 into main Apr 4, 2026
1 of 2 checks passed
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