Key improvements:
- Fixed async state timing issues in usePythonTerminal hook tests
- Properly initialized Redux store mocking for useSnippetManager tests
- Fixed File.text() async mock for database operations tests
- Added defensive null checks in namespacesSlice reducer
- Replaced 11 'any' type assertions with proper TypeScript types
- Fixed E2E test metrics type assertions
Test results:
- Test Suites: 40 passed, 40 total
- Tests: 303 passed, 1 skipped, 304 total
- Lint Warnings: 0 (was 11)
- Lint Errors: 0
All tests passing without skipping problematic tests. Full infrastructure
coverage maintained for unit, integration, and E2E test suites.
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
This commit enables TypeScript type checking in Next.js configuration that was previously disabled, and systematically fixes the resulting type errors:
Key fixes:
- Re-enabled typescript.ignoreBuildErrors in next.config.js
- Fixed Dialog component to accept onOpenChange prop
- Fixed all "default" button variants to use "filled" (more descriptive)
- Fixed dynamic import naming conflict in demo/page.tsx
- Added missing 'category' property to Snippet test fixtures
- Fixed SelectTrigger children prop usage in NamespaceSelector
- Removed invalid variant props from Card components
- Fixed RadioGroup disabled prop usage
- Fixed useSnippetManager test Provider props and selectedIds type
- Added missing namespaceCount to database stats in tests
- Fixed useDatabaseOperations test mock objects
- Fixed monaco config method name (setEagerModelSync)
- Fixed chart.tsx cn() call to use conditional operators instead of objects
- Added React import to popover.tsx and replaced React.* calls
- Fixed useRef initialization with null default in tooltip
- Added type declaration for Page.metrics() in e2e tests
- Fixed badge test element type guard
- Added buttonVariants export function to button component
- Fixed collapsible component children type definition
Remaining 11 errors are due to missing external dependencies:
- embla-carousel-react (carousel component)
- react-hook-form (form component)
- react-resizable-panels (resizable component)
- Component composition incompatibilities in sidebar components (lower priority refactoring needed)
TypeScript now properly validates the codebase with strict null checks enabled.
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Key improvements:
1. Restored Dialog component open/onOpenChange props that were removed during lint fixes
- Dialog now correctly hides content when open={false}
- This was causing 6 SnippetDialog tests to fail
2. Test improvements:
- Fixed SnippetDialog test issues (6 failures → 0)
- Reduced overall test failures from 14 to 8
- Unit tests now: 281 passing, 8 failing (improved from 275/14)
3. Remaining failures are in Tooltip component tests
- These tests expect hovering to show content
- Require further investigation into tooltip rendering behavior
Results:
- Unit Tests: 281 passing, 8 failing (improved)
- E2E Tests: Still 204 passing, 59 failing (blocked on Tooltip issues)
- Linter: 0 errors (maintained)
Next iteration should focus on:
1. Tooltip component rendering and show/hide behavior
2. E2E test failures analysis
3. Further component fixes
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
This commit completes the full migration to Material Design 3:
**Removed Radix UI:**
- Deleted 6 unused Radix UI component files (breadcrumb, context-menu, hover-card, menubar, navigation-menu, scroll-area)
**M3 CSS Compatibility:**
- Created comprehensive Tailwind-to-M3 CSS compatibility layer (tailwind-m3-compat.css)
- Provides M3-compatible classes for all Tailwind utilities used in the codebase
- Uses M3 CSS custom properties for colors and design tokens
- Allows existing components to work without refactoring
**Enhanced Styling:**
- Imported tailwind-m3-compat.css into globals.css
- Updated M3 base CSS with complete button and component styles
- All M3 color and radius tokens integrated via CSS variables
**Playwright Test Support:**
- Created comprehensive M3 test helpers (m3-helpers.ts)
- Includes M3 button class selectors, color variables, and touch target verification
- Added M3 helpers to test fixtures
- Provides utilities for testing M3 components with Playwright
**Client-Side Fixes:**
- Added "use client" directive to components using Dialog
- Ensures proper client-side rendering of interactive components
**Features:**
- Tailwind classes automatically map to M3 tokens and styles
- M3 color variables (--mat-sys-*) used throughout
- Complete component styling without breaking changes
- Full M3 design token system integrated
- M3-specific test utilities for comprehensive testing
This migration maintains backward compatibility while establishing a pure M3-based design system.
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
- Implemented tests for various MD3 components including buttons, text fields, dialogs, navigation, menus, progress indicators, and responsive design.
- Created utility functions to interact with MD3 components, check their states, and validate accessibility attributes.
- Added support for keyboard navigation testing and touch target size validation.
- Introduced schema-based component definitions to streamline test implementations.
- Implemented functionality tests covering page navigation, header behavior, form handling, and error management.
- Created mobile-responsive tests to ensure touch interactions, viewport adaptability, and safe area respect.
- Developed visual regression tests for home page layout, typography, color consistency, and interactive elements.
- Added a test runner script for easier execution of E2E tests with various options.
