ginkgobioworks / react-json-schema-form-builder / 20673325228

72%

Build Type

push

github

Committed by raymond-lam

Commit Message

Refactor codebase: modernize stack, reorganize structure, and improve consistency

Stack modernization:
- Upgrade from React 18 to React 19 with automatic JSX runtime
- Replace Bootstrap/reactstrap with Material-UI (MUI) v7
- Replace @hello-pangea/dnd with @dnd-kit for drag-and-drop
- Replace react-jss with MUI styling system
- Replace Enzyme with React Testing Library
- Migrate from Rollup to tsup for library bundling
- Migrate example app from Next.js to Vite
- Update TypeScript to ES2022 with bundler module resolution
- Modernize ESLint to flat config format

Code changes:
- Convert all class components to functional components with hooks
- Improve type safety: replace 'any' with JsonSchema/UiSchema types
- Convert all interfaces to types
- Remove redundant nested containers
- Add SortableItem component with drag handle support
- Improve memoization with useMemo and useCallback
- Restructure example app to follow standard Vite directory structure
- Export types from library and update example app imports (Fixes #604)
- Reorganize component files: move FBCheckbox and PlaceholderInput to formBuilder root
- Standardize label casing to Title Case throughout (Default Value, Column Size, Dependencies, etc.)
- Optimize re-renders by fixing dependency arrays (remove setState from deps, fix CardModal useEffect, fix hideAddButton)
- Remove unnecessary container elements and standardize fragment syntax (React.Fragment -> <>)
- Update dependencies to latest versions and clean up code

Documentation:
- Update Contributing.md: replace Enzyme with React Testing Library, fix types.js to types.ts, remove outdated Travis CI references, restore coveralls.io reference
- Update all documentation and examples

Issues addressed:
- #369: Array field cursor loses focus when typing
- #467: Description field lost when changing Input Type
- #476: Unreachable code in ValueSelector
- #478: False positive unit tests and validator issues
- #412: Alert not exported from reactstrap... (continued)

Run Details

788 of 1120 branches covered (70.36%)

Branch coverage included in aggregate %.

357 of 490 new or added lines in 24 files covered. (72.86%)

8 existing lines in 1 file now uncovered.

882 of 1185 relevant lines covered (74.43%)

22.68 hits per line