From b1a63e98ab7faf8758521bc0f008a12241cff321 Mon Sep 17 00:00:00 2001
From: Daniel Bedeleanu <bede@tfm.ro>
Date: Wed, 22 Apr 2026 18:50:06 +0300
Subject: [PATCH] fix(6): fix TypeScript type consistency issues

- Make Item.id required (items from DB always have id)
- Use shared Item type from db.ts in QuantityAdjustmentModal
- Show full npm build output instead of silencing errors
- Ensures all modals use consistent type definitions
---
 .../components/inventory/QuantityAdjustmentModal.tsx  | 11 +----------
 frontend/lib/db.ts                                    |  2 +-
 start_servers.py                                      |  7 ++++---
 3 files changed, 6 insertions(+), 14 deletions(-)

diff --git a/frontend/components/inventory/QuantityAdjustmentModal.tsx b/frontend/components/inventory/QuantityAdjustmentModal.tsx
index 8134db06..8ed96b8b 100644
--- a/frontend/components/inventory/QuantityAdjustmentModal.tsx
+++ b/frontend/components/inventory/QuantityAdjustmentModal.tsx
@@ -4,16 +4,7 @@ import { useState, useEffect } from 'react';
 import { X } from 'lucide-react';
 import QuantityDisplay from './QuantityDisplay';
 import axios from 'axios';
-
-interface Item {
-  id: number;
-  name: string;
-  quantity: number;
-  part_number?: string;
-  barcode?: string;
-  category?: string;
-  description?: string;
-}
+import { Item } from '@/lib/db';
 
 interface QuantityAdjustmentModalProps {
   item: Item | null;
diff --git a/frontend/lib/db.ts b/frontend/lib/db.ts
index 541dfa31..add6a541 100644
--- a/frontend/lib/db.ts
+++ b/frontend/lib/db.ts
@@ -1,7 +1,7 @@
 import Dexie, { Table } from 'dexie';
 
 export interface Item {
-  id?: number;
+  id: number;
   barcode: string;
   name: string;
   category: string;
diff --git a/start_servers.py b/start_servers.py
index bc812be2..f3c61bb0 100755
--- a/start_servers.py
+++ b/start_servers.py
@@ -112,12 +112,13 @@ class StandaloneServerManager:
         # Build if needed
         if not (frontend_dir / ".next" / "standalone").exists():
             self.log("Building Next.js application...")
-            subprocess.run(
+            result = subprocess.run(
                 ["npm", "run", "build"],
                 cwd=str(frontend_dir),
-                check=True,
-                capture_output=True
+                capture_output=False
             )
+            if result.returncode != 0:
+                self.error(f"Frontend build failed with exit code {result.returncode}")
 
     def start_backend(self):
         """Start FastAPI backend server using uvicorn from project root"""