docuking-mcp 2.12.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));
-    }
-
-    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);
-      }
-    }
+  let protectedFiles = [];
 
-    // MCP에서 미리 제외한 웹 파일도 보호 목록에 추가 (결과 표시용)
-    protectedFiles = [...protectedFiles, ...webProtectedPaths];
-  }
+  // 자동 삭제 로직 비활성화 - 아래 코드는 주석 처리
+  // "로컬에 없으면 삭제" 로직은 위험하므로 제거
+  console.error(`[DocuKing] 자동 삭제 비활성화됨 (명시적 삭제만 허용: docuking_delete 사용)`);
 
   // 5. 빈 폴더 생성
   let createdEmptyFolders = 0;
@@ -1227,14 +1168,48 @@ export async function handlePullInternal(args) {
   const localIndex = loadIndex(localPath);
   const syncedFiles = []; // 다운로드/업데이트된 파일 목록
 
-  for (const file of sortedFiles) {
+  // ========================================
+  // 폴더 먼저 처리
+  // ========================================
+  const folders = sortedFiles.filter(f => f.type === 'folder');
+  const files_only = sortedFiles.filter(f => f.type !== 'folder');
+
+  for (const folder of folders) {
     current++;
-    const progress = `${current}/${total}`;
+    const serverMeta = serverPathToMeta[folder.path];
+    let fullPath;
+    if (folder.path.startsWith('xx_') ||
+        folder.path.startsWith('yy_All_Docu/') ||
+        folder.path.startsWith('yy_All_Docu') ||
+        folder.path.startsWith('yy_Coworker_') ||
+        folder.path.startsWith('zz_ai_')) {
+      fullPath = path.join(localPath, folder.path);
+    } else {
+      fullPath = path.join(mainFolderPath, folder.path);
+    }
+
+    if (fs.existsSync(fullPath)) {
+      results.push({ type: 'skip', path: folder.path, itemType: 'folder' });
+      skipped++;
+    } else {
+      fs.mkdirSync(fullPath, { recursive: true });
+      results.push({ type: 'download', path: folder.path, itemType: 'folder' });
+      foldersCreated++;
+      downloaded++;
+      if (serverMeta && serverMeta.source === 'web') {
+        filesToMarkAsPulled.push(folder.path);
+      }
+    }
+  }
 
-    // 서버 메타 정보 조회
-    const serverMeta = serverPathToMeta[file.path];
+  // ========================================
+  // 파일 처리 - Batch API 사용 (200개씩)
+  // ========================================
+  const BATCH_SIZE = 200;
 
-    // 서버 경로에 따라 로컬 저장 위치 결정
+  // 먼저 로컬 해시 기반으로 스킵할 파일 필터링
+  const filesToDownload = [];
+  for (const file of files_only) {
     let fullPath;
     if (file.path.startsWith('xx_') ||
         file.path.startsWith('yy_All_Docu/') ||
@@ -1246,146 +1221,188 @@ export async function handlePullInternal(args) {
       fullPath = path.join(mainFolderPath, file.path);
     }
 
-    try {
-      // ========================================
-      // 폴더 처리 (type === 'folder')
-      // ========================================
-      if (file.type === 'folder') {
-        if (fs.existsSync(fullPath)) {
-          // 이미 있음 → 스킵
-          results.push({ type: 'skip', path: file.path, itemType: 'folder' });
-          skipped++;
-        } else {
-          // 폴더 생성
-          fs.mkdirSync(fullPath, { recursive: true });
-          results.push({ type: 'download', path: file.path, itemType: 'folder' });
-          foldersCreated++;
-          downloaded++;
+    const serverHash = serverPathToHash[file.path];
 
-          // source: 'web' 폴더면 mark-as-pulled 대상
-          if (serverMeta && serverMeta.source === 'web') {
-            filesToMarkAsPulled.push(file.path);
-          }
+    // 로컬 파일 해시 비교
+    if (fs.existsSync(fullPath)) {
+      try {
+        const localStat = fs.statSync(fullPath);
+        if (localStat.isDirectory()) {
+          results.push({ type: 'skip', path: file.path, error: '같은 이름의 폴더 존재' });
+          skipped++;
+          current++;
+          continue;
         }
-        continue;
-      }
-
-      // ========================================
-      // 파일 처리 (type === 'file')
-      // ========================================
-      const response = await fetch(
-        `${API_ENDPOINT}/files?projectId=${projectId}&path=${encodeURIComponent(file.path)}`,
-        {
-          headers: {
-            'Authorization': `Bearer ${apiKey}`,
-          },
+        const localBuffer = fs.readFileSync(fullPath);
+        const localHash = crypto.createHash('sha256').update(localBuffer).digest('hex');
+        if (localHash === serverHash) {
+          results.push({ type: 'skip', path: file.path });
+          skipped++;
+          current++;
+          continue;
         }
-      );
 
-      if (!response.ok) {
-        results.push({ type: 'fail', path: file.path, error: await response.text() });
-        failed++;
+        // ========================================
+        // [버그 수정] 로컬 파일 우선 정책
+        // 해시가 다르면 → 로컬 수정이 있다는 뜻
+        // 서버 파일로 덮어쓰지 않고 로컬 유지
+        // 다음 Push에서 로컬 → 서버로 업로드됨
+        // ========================================
+        console.error(`[DocuKing] 로컬 우선: ${file.path} (해시 다름 → 로컬 유지, Push 시 업로드됨)`);
+        results.push({ type: 'skip', path: file.path, reason: 'local-modified' });
+        skipped++;
+        current++;
         continue;
+      } catch (e) {
+        // 해시 계산 실패 시 다운로드 대상
       }
+    }
 
-      const data = await response.json();
+    // 로컬에 파일이 없는 경우만 다운로드
+    filesToDownload.push({ ...file, fullPath });
+  }
 
-      // 인코딩에 따라 저장
-      const content = data.file?.content || data.content || '';
-      const encoding = data.file?.encoding || data.encoding || 'utf-8';
+  console.error(`[DocuKing] Batch Pull: ${filesToDownload.length}개 파일 다운로드 필요 (${Math.ceil(filesToDownload.length / BATCH_SIZE)}개 배치)`);
+
+  // Batch API로 파일 다운로드
+  for (let i = 0; i < filesToDownload.length; i += BATCH_SIZE) {
+    const batch = filesToDownload.slice(i, i + BATCH_SIZE);
+    const batchPaths = batch.map(f => f.path);
+    const batchNum = Math.floor(i / BATCH_SIZE) + 1;
+    const totalBatches = Math.ceil(filesToDownload.length / BATCH_SIZE);
+
+    console.error(`[DocuKing] Batch ${batchNum}/${totalBatches}: ${batch.length}개 파일 다운로드 중...`);
+
+    try {
+      const batchResponse = await fetch(`${API_ENDPOINT}/files/batch-pull`, {
+        method: 'POST',
+        headers: {
+          'Content-Type': 'application/json',
+          'Authorization': `Bearer ${apiKey}`,
+        },
+        body: JSON.stringify({
+          projectId,
+          paths: batchPaths,
+        }),
+      });
 
-      // 서버 파일 해시 계산
-      let serverBuffer;
-      if (encoding === 'base64') {
-        serverBuffer = Buffer.from(content, 'base64');
-      } else {
-        serverBuffer = Buffer.from(content, 'utf-8');
+      if (!batchResponse.ok) {
+        const errorText = await batchResponse.text();
+        console.error(`[DocuKing] Batch Pull 실패: ${errorText}`);
+        // 배치 실패 시 모든 파일 실패 처리
+        for (const file of batch) {
+          results.push({ type: 'fail', path: file.path, error: 'Batch request failed' });
+          failed++;
+          current++;
+        }
+        continue;
       }
-      const serverHash = crypto.createHash('sha256').update(serverBuffer).digest('hex');
 
-      // 로컬 파일 존재 여부 및 해시 비교
-      let localExists = fs.existsSync(fullPath);
-      let localHash = null;
+      const batchData = await batchResponse.json();
+      const filesData = batchData.files || [];
+
+      // 파일별 처리
+      for (const file of batch) {
+        current++;
+        const fileData = filesData.find(f => f.path === file.path);
+        const serverMeta = serverPathToMeta[file.path];
+
+        if (!fileData || fileData.error) {
+          results.push({ type: 'fail', path: file.path, error: fileData?.error || 'Not found' });
+          failed++;
+          continue;
+        }
 
-      if (localExists) {
         try {
-          const localStat = fs.statSync(fullPath);
-          if (localStat.isDirectory()) {
-            // 로컬에 같은 이름의 폴더가 있으면 건너뜀
-            results.push({ type: 'skip', path: file.path, error: '같은 이름의 폴더 존재' });
-            skipped++;
-            continue;
+          const content = fileData.content || '';
+          const encoding = fileData.encoding || 'utf-8';
+
+          // 서버 파일 버퍼
+          let serverBuffer;
+          if (encoding === 'base64') {
+            serverBuffer = Buffer.from(content, 'base64');
+          } else {
+            serverBuffer = Buffer.from(content, 'utf-8');
           }
-          const localBuffer = fs.readFileSync(fullPath);
-          localHash = crypto.createHash('sha256').update(localBuffer).digest('hex');
-        } catch (e) {
-          localExists = false;
-        }
-      }
+          const serverHash = fileData.hash || crypto.createHash('sha256').update(serverBuffer).digest('hex');
 
-      // 변경 감지
-      if (localExists && localHash === serverHash) {
-        // 변경 없음 - 스킵
-        results.push({ type: 'skip', path: file.path });
-        skipped++;
-        continue;
-      }
+          // 로컬 존재 여부
+          const localExists = fs.existsSync(file.fullPath);
 
-      // 디렉토리 생성
-      fs.mkdirSync(path.dirname(fullPath), { recursive: true });
+          // 디렉토리 생성
+          fs.mkdirSync(path.dirname(file.fullPath), { recursive: true });
 
-      // 파일 저장
-      if (encoding === 'base64') {
-        fs.writeFileSync(fullPath, serverBuffer);
-      } else {
-        fs.writeFileSync(fullPath, content, 'utf-8');
-      }
+          // 파일 저장
+          if (encoding === 'base64') {
+            fs.writeFileSync(file.fullPath, serverBuffer);
+          } else {
+            fs.writeFileSync(file.fullPath, content, 'utf-8');
+          }
 
-      if (localExists) {
-        results.push({ type: 'update', path: file.path });
-        updated++;
-      } else {
-        results.push({ type: 'download', path: file.path });
-        downloaded++;
-      }
+          if (localExists) {
+            results.push({ type: 'update', path: file.path });
+            updated++;
+          } else {
+            results.push({ type: 'download', path: file.path });
+            downloaded++;
+          }
 
-      // 인덱스에 다운로드된 파일 기록
-      syncedFiles.push({ serverPath: file.path, fullPath, hash: serverHash });
+          // 인덱스에 기록
+          syncedFiles.push({ serverPath: file.path, fullPath: file.fullPath, hash: serverHash });
 
-      // source: 'web' 파일이면 mark-as-pulled 대상
-      if (serverMeta && serverMeta.source === 'web') {
-        filesToMarkAsPulled.push(file.path);
+          // source: 'web' 파일이면 mark-as-pulled 대상
+          if (serverMeta && serverMeta.source === 'web') {
+            filesToMarkAsPulled.push(file.path);
+          }
+        } catch (e) {
+          results.push({ type: 'fail', path: file.path, error: e.message });
+          failed++;
+        }
       }
     } catch (e) {
-      results.push({ type: 'fail', path: file.path, error: e.message });
-      failed++;
+      console.error(`[DocuKing] Batch ${batchNum} 네트워크 오류: ${e.message}`);
+      // 배치 실패 시 개별 요청으로 폴백
+      for (const file of batch) {
+        current++;
+        try {
+          const response = await fetch(
+            `${API_ENDPOINT}/files?projectId=${projectId}&path=${encodeURIComponent(file.path)}`,
+            { headers: { 'Authorization': `Bearer ${apiKey}` } }
+          );
+          if (!response.ok) {
+            results.push({ type: 'fail', path: file.path, error: await response.text() });
+            failed++;
+            continue;
+          }
+          const data = await response.json();
+          const content = data.file?.content || data.content || '';
+          const encoding = data.file?.encoding || data.encoding || 'utf-8';
+          let serverBuffer = encoding === 'base64' ? Buffer.from(content, 'base64') : Buffer.from(content, 'utf-8');
+          fs.mkdirSync(path.dirname(file.fullPath), { recursive: true });
+          fs.writeFileSync(file.fullPath, encoding === 'base64' ? serverBuffer : content, encoding === 'base64' ? undefined : 'utf-8');
+          results.push({ type: 'download', path: file.path });
+          downloaded++;
+        } catch (fallbackErr) {
+          results.push({ type: 'fail', path: file.path, error: fallbackErr.message });
+          failed++;
+        }
+      }
     }
   }
 
   // ========================================
-  // 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.12.0",
+  "version": "2.14.0",
   "description": "DocuKing MCP Server - AI 시대의 문서 협업 플랫폼",
   "type": "module",
   "main": "index.js",