1 files changed, 26 insertions, 15 deletions

diff --git a/components/ship-vendor-document/new-revision-dialog.tsx b/components/ship-vendor-document/new-revision-dialog.tsx
index bdbb1bc6..39025fa4 100644
--- a/components/ship-vendor-document/new-revision-dialog.tsx
+++ b/components/ship-vendor-document/new-revision-dialog.tsx
@@ -248,6 +248,14 @@ export function NewRevisionDialog({
     // Serial No 조회
     const fetchNextSerialNo = React.useCallback(async () => {
       console.log('🔍 fetchNextSerialNo called with documentId:', documentId)
+      
+      // documentId 유효성 검사
+      if (!documentId || documentId === undefined || documentId === null) {
+        console.warn('⚠️ Invalid documentId, using default serialNo: 1')
+        setNextSerialNo("1")
+        return
+      }
+      
       setIsLoadingSerialNo(true)
       try {
         const apiUrl = `/api/revisions/max-serial-no?documentId=${documentId}`
@@ -267,11 +275,16 @@ export function NewRevisionDialog({
           setNextSerialNo(serialNoString)
           console.log('🔍 nextSerialNo state updated')
         } else {
-          console.error('🔍 API call failed with status:', response.status)
+          const errorData = await response.json().catch(() => ({}))
+          console.error('🔍 API call failed with status:', response.status, errorData)
+          // API 실패 시 기본값 1 사용
+          console.warn('⚠️ Using default serialNo: 1')
+          setNextSerialNo("1")
         }
       } catch (error) {
         console.error('❌ Failed to fetch serial no:', error)
         // 에러 시 기본값 1 사용
+        console.warn('⚠️ Using default serialNo: 1 due to error')
         setNextSerialNo("1")
       } finally {
         setIsLoadingSerialNo(false)
@@ -280,12 +293,15 @@ export function NewRevisionDialog({
 
       // Dialog 열릴 때 Serial No 조회
   React.useEffect(() => {
-    console.log('🎯 useEffect triggered - open:', open, 'documentId:', documentId)
-    if (open && documentId) {
-      console.log('🎯 Calling fetchNextSerialNo')
-      fetchNextSerialNo()
-    } else {
-      console.log('🎯 Conditions not met for fetchNextSerialNo')
+    console.log('🎯 useEffect triggered - open:', open, 'documentId:', documentId, 'type:', typeof documentId)
+    if (open) {
+      if (documentId && typeof documentId === 'number' && documentId > 0) {
+        console.log('🎯 Calling fetchNextSerialNo')
+        fetchNextSerialNo()
+      } else {
+        console.warn('🎯 Invalid documentId, using default serialNo: 1')
+        setNextSerialNo("1")
+      }
     }
   }, [open, documentId, fetchNextSerialNo])
 
@@ -475,17 +491,12 @@ export function NewRevisionDialog({
             <DialogDescription className="text-sm space-y-1">
               <div>Document: {documentTitle}</div>
               <div className="text-xs text-muted-foreground">
-                Drawing Type: {drawingKind} | Serial No: {nextSerialNo}
-                {isLoadingSerialNo && (
+                Drawing Type: {drawingKind} | Serial No: {isLoadingSerialNo ? (
                   <>
-                    <Loader2 className="inline-block ml-2 h-3 w-3 animate-spin" />
+                    <Loader2 className="inline-block ml-1 h-3 w-3 animate-spin" />
                     <span className="ml-1">Loading...</span>
                   </>
-                )}
-                {/* 디버그용 임시 표시 */}
-                <div className="mt-1 text-xs text-orange-600">
-                  Debug: nextSerialNo={nextSerialNo}, isLoading={isLoadingSerialNo}
-                </div>
+                ) : nextSerialNo}
               </div>
             </DialogDescription>
           )}