Fix latest build issues from master

[ipodishima]

src/components/ConversationHistory.tsx:171:33 - error TS7006: Parameter 'e' implicitly has an 'any' type.

171                       onClick={(e) => handleDeleteClick(e, conversation)}
                                    ~

src/components/DebugLogViewer.tsx:352:27 - error TS7006: Parameter 'e' implicitly has an 'any' type.

352                 onClick={(e) => {
                              ~

src/components/DebugLogViewer.tsx:369:27 - error TS7006: Parameter 'e' implicitly has an 'any' type.

369                 onClick={(e) => {
                              ~

src/components/DebugLogViewer.tsx:386:27 - error TS7006: Parameter 'e' implicitly has an 'any' type.

386                 onClick={(e) => {
                              ~

src/components/DebugLogViewer.tsx:424:25 - error TS7006: Parameter 'e' implicitly has an 'any' type.

424               onClick={(e) => {
                            ~

src/components/ProjectSelector.tsx:123:31 - error TS7006: Parameter 'e' implicitly has an 'any' type.

123                     onClick={(e) => handleDeleteClick(e, project.name)}
                                  ~

src/components/ScheduleModal.tsx:338:35 - error TS7006: Parameter 'checked' implicitly has an 'any' type.

338                 onCheckedChange={(checked) =>
                                      ~~~~~~~

src/components/ThemeSelector.tsx:16:29 - error TS2503: Cannot find namespace 'NodeJS'.

16   const timeoutRef = useRef<NodeJS.Timeout | null>(null)
                               ~~~~~~

src/components/ui/alert.tsx:2:40 - error TS2307: Cannot find module 'class-variance-authority' or its corresponding type declarations.

2 import { cva, type VariantProps } from "class-variance-authority"
                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/badge.tsx:3:40 - error TS2307: Cannot find module 'class-variance-authority' or its corresponding type declarations.

3 import { cva, type VariantProps } from "class-variance-authority"
                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/button.tsx:3:40 - error TS2307: Cannot find module 'class-variance-authority' or its corresponding type declarations.

3 import { cva, type VariantProps } from "class-variance-authority"
                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/checkbox.tsx:2:36 - error TS2307: Cannot find module '@radix-ui/react-checkbox' or its corresponding type declarations.

2 import * as CheckboxPrimitive from "@radix-ui/react-checkbox"
                                     ~~~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/label.tsx:2:33 - error TS2307: Cannot find module '@radix-ui/react-label' or its corresponding type declarations.

2 import * as LabelPrimitive from "@radix-ui/react-label"
                                  ~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/popover.tsx:2:35 - error TS2307: Cannot find module '@radix-ui/react-popover' or its corresponding type declarations.

2 import * as PopoverPrimitive from "@radix-ui/react-popover"
                                    ~~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/radio-group.tsx:4:38 - error TS2307: Cannot find module '@radix-ui/react-radio-group' or its corresponding type declarations.

4 import * as RadioGroupPrimitive from "@radix-ui/react-radio-group"
                                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/scroll-area.tsx:2:38 - error TS2307: Cannot find module '@radix-ui/react-scroll-area' or its corresponding type declarations.

2 import * as ScrollAreaPrimitive from "@radix-ui/react-scroll-area"
                                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/select.tsx:4:34 - error TS2307: Cannot find module '@radix-ui/react-select' or its corresponding type declarations.

4 import * as SelectPrimitive from "@radix-ui/react-select"
                                   ~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/separator.tsx:4:37 - error TS2307: Cannot find module '@radix-ui/react-separator' or its corresponding type declarations.

4 import * as SeparatorPrimitive from "@radix-ui/react-separator"
                                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/switch.tsx:4:34 - error TS2307: Cannot find module '@radix-ui/react-switch' or its corresponding type declarations.

4 import * as SwitchPrimitive from "@radix-ui/react-switch"
                                   ~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/tabs.tsx:2:32 - error TS2307: Cannot find module '@radix-ui/react-tabs' or its corresponding type declarations.

2 import * as TabsPrimitive from "@radix-ui/react-tabs"
                                 ~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/tabs.tsx:3:40 - error TS2307: Cannot find module 'class-variance-authority' or its corresponding type declarations.

3 import { cva, type VariantProps } from "class-variance-authority"
                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/toggle.tsx:4:34 - error TS2307: Cannot find module '@radix-ui/react-toggle' or its corresponding type declarations.

4 import * as TogglePrimitive from "@radix-ui/react-toggle"
                                   ~~~~~~~~~~~~~~~~~~~~~~~~

src/components/ui/toggle.tsx:5:40 - error TS2307: Cannot find module 'class-variance-authority' or its corresponding type declarations.

5 import { cva, type VariantProps } from "class-variance-authority"
                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~

src/lib/utils.ts:2:25 - error TS2307: Cannot find module 'tailwind-merge' or its corresponding type declarations.

2 import { twMerge } from "tailwind-merge"
                          ~~~~~~~~~~~~~~~~

vite.config.ts:4:18 - error TS2307: Cannot find module 'path' or its corresponding type declarations.

4 import path from 'path'
                   ~~~~~~

vite.config.ts:7:17 - error TS2580: Cannot find name 'process'. Do you need to install type definitions for node? Try `npm i --save-dev @types/node`.

7 const apiPort = process.env.VITE_API_PORT || '8888'
                  ~~~~~~~

vite.config.ts:14:25 - error TS2304: Cannot find name '__dirname'.

14       '@': path.resolve(__dirname, './src'),
                           ~~~~~~~~~


Found 27 errors.

ERROR: Failed to build frontend

Summary by CodeRabbit

Release Notes

• Bug Fixes

  • Optimized npm dependency installation to skip unnecessary reinstalls when package dependencies are already current.

• Refactor

  • Strengthened type safety across the application by adding explicit type annotations to event handlers and component properties.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The changes improve type safety across React components by adding explicit event parameter type annotations, enhance npm dependency management by checking if packages are stale before reinstalling, and add Node.js type definitions to the TypeScript configuration.

Changes

Cohort / File(s)	Summary
React Component Type Refinements ui/src/components/ConversationHistory.tsx, ui/src/components/DebugLogViewer.tsx, ui/src/components/ProjectSelector.tsx, ui/src/components/ScheduleModal.tsx, ui/src/components/ThemeSelector.tsx	Added explicit type annotations to event handlers: React.MouseEvent for onClick handlers in ConversationHistory, DebugLogViewer, and ProjectSelector; `boolean
NPM Dependency Management start_ui.py	Modified install_npm_deps() to check staleness: skips reinstallation if node_modules exists and is newer than package.json/package-lock.json; otherwise installs to ensure dependencies reflect recent package changes.
TypeScript Configuration ui/tsconfig.node.json	Added "types": ["node"] to compilerOptions to include Node.js type definitions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 With types so precise and deps kept in check,
No stale packages cause a wreck,
Events now typed with care so fine,
React components in perfect line—
A whisker-twitch of TypeScript delight! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 16.67% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'Fix latest build issues from master' is vague and generic, using non-descriptive phrasing that doesn't clearly convey the specific changes (TypeScript type annotations and Node.js type configuration additions).	Replace with a more specific title that describes the actual changes, such as 'Add explicit TypeScript type annotations and Node.js type configuration to fix build errors' or 'Fix TypeScript implicit any errors and configure Node types'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[leonvanzyl]

Thanks