docuking-mcp 2.13.0 → 2.14.0

package/handlers/sync.js

@@ -806,82 +806,23 @@ docuking_init을 먼저 실행하세요.`,
   }
 
   // ========================================
-  // 4. 서버에만 있고 로컬에 없는 파일 soft-delete (deleted_at 기반)
+  // 4. 자동 삭제 비활성화 (2026-01-17 버그 수정)
+  // ========================================
+  // 이전 로직: "서버에 있고 로컬에 없으면 삭제"
+  // 문제: 다른 프로젝트에서 Pull하면 서버 파일이 삭제됨
+  // 해결: 자동 삭제 제거, docuking_delete로만 명시적 삭제
+  //
+  // 삭제는 명시적으로만:
+  // - 사용자가 "이 파일 삭제해줘" 요청 시
+  // - docuking_delete MCP 도구 사용
   // ========================================
-  // 단, 협업자 폴더(yy_Coworker_*)는 삭제하지 않음 (오너가 협업자 파일을 삭제하면 안됨)
-  // source: 'web' 파일은 MCP에서 삭제 시도하지 않음 (클라이언트에서 필터링)
-  // 중요: localPathsBeforePull 사용 (Pull 전에 수집한 목록)
-  //       processedLocalPaths는 Pull로 내려받은 파일도 포함하므로 사용 금지
   let deleted = 0;
   const deletedFilePaths = [];
-  let protectedFiles = []; // source: 'web' 파일 (보호됨)
-  const webProtectedPaths = []; // MCP에서 미리 필터링한 웹 파일
-
-  if (serverAllPaths.length > 0 && !isCoworker) {
-    // 오너만 삭제 수행
-    // yy_Coworker_*로 시작하는 경로는 삭제 대상에서 제외
-    // localPathsBeforePull: Pull 전에 수집한 로컬 파일 목록 (좀비 파일 방지)
-    //
-    // ★ 핵심: source: 'web' 파일은 삭제 대상에서 미리 제외!
-    // 이렇게 하면 웹에서 생성한 파일이 절대 삭제되지 않음
-    const pathsToDelete = serverAllPaths.filter(p => {
-      // 1. 로컬에 있으면 제외 (삭제 대상 아님)
-      if (localPathsBeforePull.has(p)) return false;
-
-      // 2. 협업자 폴더는 제외
-      if (p.startsWith('yy_Coworker_')) return false;
-
-      // 3. source: 'web' 파일은 제외 (웹에서 생성된 파일 보호)
-      const meta = serverPathToMeta[p];
-      if (meta && meta.source === 'web') {
-        webProtectedPaths.push(p);
-        return false;
-      }
-
-      return true;
-    });
-
-    // 웹 보호 파일 로그
-    if (webProtectedPaths.length > 0) {
-      console.error(`[DocuKing] source:web 파일 ${webProtectedPaths.length}개 삭제에서 제외:`, webProtectedPaths.slice(0, 5));
-    }
+  let protectedFiles = [];
 
-    console.error(`[DocuKing] soft-delete 대상: ${pathsToDelete.length}개 (서버에만 있고 Pull 전 로컬에 없던 파일, source:local만)`);
-
-    if (pathsToDelete.length > 0) {
-      try {
-        // soft-delete API 호출 (deleted_at 설정, 3일 후 hard delete)
-        const deleteResponse = await fetch(`${API_ENDPOINT}/files/soft-delete`, {
-          method: 'POST',
-          headers: {
-            'Content-Type': 'application/json',
-            'Authorization': `Bearer ${apiKey}`,
-          },
-          body: JSON.stringify({
-            projectId,
-            paths: pathsToDelete,
-          }),
-        });
-
-        if (deleteResponse.ok) {
-          const deleteResult = await deleteResponse.json();
-          deleted = deleteResult.deleted || 0;
-          deletedFilePaths.push(...(deleteResult.deletedPaths || []));
-          protectedFiles = deleteResult.protected || [];
-
-          // 보호된 파일이 있으면 로그 출력 (Backend에서 추가로 보호한 파일)
-          if (protectedFiles.length > 0) {
-            console.error(`[DocuKing] Backend에서 추가 보호된 파일 ${protectedFiles.length}개:`, protectedFiles.slice(0, 5));
-          }
-        }
-      } catch (e) {
-        console.error('[DocuKing] 파일 soft-delete 실패:', e.message);
-      }
-    }
-
-    // MCP에서 미리 제외한 웹 파일도 보호 목록에 추가 (결과 표시용)
-    protectedFiles = [...protectedFiles, ...webProtectedPaths];
-  }
+  // 자동 삭제 로직 비활성화 - 아래 코드는 주석 처리
+  // "로컬에 없으면 삭제" 로직은 위험하므로 제거
+  console.error(`[DocuKing] 자동 삭제 비활성화됨 (명시적 삭제만 허용: docuking_delete 사용)`);
 
   // 5. 빈 폴더 생성
   let createdEmptyFolders = 0;
@@ -1300,11 +1241,24 @@ export async function handlePullInternal(args) {
           current++;
           continue;
         }
+
+        // ========================================
+        // [버그 수정] 로컬 파일 우선 정책
+        // 해시가 다르면 → 로컬 수정이 있다는 뜻
+        // 서버 파일로 덮어쓰지 않고 로컬 유지
+        // 다음 Push에서 로컬 → 서버로 업로드됨
+        // ========================================
+        console.error(`[DocuKing] 로컬 우선: ${file.path} (해시 다름 → 로컬 유지, Push 시 업로드됨)`);
+        results.push({ type: 'skip', path: file.path, reason: 'local-modified' });
+        skipped++;
+        current++;
+        continue;
       } catch (e) {
         // 해시 계산 실패 시 다운로드 대상
       }
     }
 
+    // 로컬에 파일이 없는 경우만 다운로드
     filesToDownload.push({ ...file, fullPath });
   }
 
@@ -1437,29 +1391,18 @@ export async function handlePullInternal(args) {
   }
 
   // ========================================
-  // mark-as-pulled 호출 (source: 'web' → 'local' 변경)
-  // Pull된 웹 파일/폴더는 이제 로컬이 관리권한을 가짐
+  // mark-as-pulled 비활성화 (2026-01-17 정책 변경)
+  // ========================================
+  // 이전 로직: source를 'web' → 'local'로 변경
+  // 문제: source는 "출생신고"로, 한 번 정해지면 불변이어야 함
+  // 변경하면 "출처 위조"가 됨
+  //
+  // 새 정책: source는 절대 변경하지 않음
+  // - web에서 태어난 파일은 Pull해도 source: 'web' 유지
+  // - local에서 태어난 파일은 Push해도 source: 'local' 유지
   // ========================================
   if (filesToMarkAsPulled.length > 0) {
-    try {
-      const markResponse = await fetch(`${API_ENDPOINT}/files/mark-as-pulled`, {
-        method: 'POST',
-        headers: {
-          'Content-Type': 'application/json',
-          'Authorization': `Bearer ${apiKey}`,
-        },
-        body: JSON.stringify({
-          projectId,
-          paths: filesToMarkAsPulled,
-        }),
-      });
-
-      if (markResponse.ok) {
-        console.error(`[DocuKing] mark-as-pulled 완료: ${filesToMarkAsPulled.length}개`);
-      }
-    } catch (e) {
-      console.error('[DocuKing] mark-as-pulled 실패:', e.message);
-    }
+    console.error(`[DocuKing] mark-as-pulled 비활성화됨 (source 불변 정책): ${filesToMarkAsPulled.length}개 파일의 source 유지`);
   }
 
   // ========================================

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "docuking-mcp",
-  "version": "2.13.0",
+  "version": "2.14.0",
   "description": "DocuKing MCP Server - AI 시대의 문서 협업 플랫폼",
   "type": "module",
   "main": "index.js",