chore: 2.0.3 버전 범프

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

.gitignore

@@ -210,3 +210,4 @@ dist-web
 .ionide
 
 # End of https://www.toptal.com/developers/gitignore/api/macos,git,visualstudiocode,node
+.gstack/

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pieroot/nfd2nfc",
-  "version": "2.0.2",
+  "version": "2.0.3",
   "description": "macOS 한글 파일명 NFD→NFC 실시간 변환 트레이 앱 + CLI + 라이브러리",
   "main": "out/lib/index.js",
   "types": "out/lib/index.d.ts",

src/main/__tests__/watcher.test.ts

@@ -0,0 +1,103 @@
+import { EventEmitter } from 'events';
+import { vi, describe, it, expect, beforeEach, afterEach } from 'vitest';
+
+vi.mock('chokidar');
+vi.mock('electron', () => ({ BrowserWindow: { getAllWindows: () => [] } }));
+
+const mockAppendUndoEntries = vi.fn().mockResolvedValue(undefined);
+vi.mock('../store', () => ({ appendUndoEntries: mockAppendUndoEntries }));
+
+const mockQueueRenamedNotification = vi.fn().mockResolvedValue(undefined);
+vi.mock('../notifier', () => ({
+  queueRenamedNotification: mockQueueRenamedNotification,
+  notifyManualQueue: vi.fn().mockResolvedValue(undefined),
+}));
+
+vi.mock('../nanoid', () => ({ nanoid: () => 'test-id' }));
+
+const mockNormalizeEntry = vi.fn();
+vi.mock('../../core/normalizer', () => ({ normalizeEntry: mockNormalizeEntry }));
+
+import chokidar from 'chokidar';
+import type { WatchedDir } from '../../core/types';
+
+// 테스트마다 고유 경로를 사용해 모듈 레벨 recentlyRenamed 상태 충돌 방지
+function makePaths(slug: string) {
+  const nfc = `/test/watch/${slug}.txt`;
+  return { nfc, nfd: nfc.normalize('NFD') };
+}
+
+function makeDir(id: string): WatchedDir {
+  return { id, path: '/test/watch', recursive: false, mode: 'auto', enabled: true, customRanges: [] };
+}
+
+describe('watcher — in-flight race condition', () => {
+  let mockWatcher: EventEmitter & { close: ReturnType<typeof vi.fn> };
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+    mockWatcher = Object.assign(new EventEmitter(), { close: vi.fn().mockResolvedValue(undefined) });
+    vi.mocked(chokidar.watch).mockReturnValue(mockWatcher as unknown as ReturnType<typeof chokidar.watch>);
+  });
+
+  afterEach(async () => {
+    const { stopDir } = await import('../watcher');
+    // 각 테스트에서 사용한 id 정리
+    for (const id of ['dir-burst', 'dir-nfc-reentry', 'dir-nfd-reentry']) {
+      await stopDir(id);
+    }
+  });
+
+  it('동일 NFD 경로에 burst add 이벤트가 와도 normalizeEntry가 1번만 호출된다', async () => {
+    const { nfc, nfd } = makePaths('강의자료-burst');
+    mockNormalizeEntry.mockResolvedValue({ type: 'file', status: 'renamed', oldPath: nfd, newPath: nfc });
+
+    const { startDir } = await import('../watcher');
+    await startDir(makeDir('dir-burst'));
+
+    // 같은 경로를 동기적으로 두 번 emit — race 시나리오 재현
+    mockWatcher.emit('add', nfd);
+    mockWatcher.emit('add', nfd);
+    await new Promise((r) => setTimeout(r, 0));
+
+    expect(mockNormalizeEntry).toHaveBeenCalledTimes(1);
+    expect(mockAppendUndoEntries).toHaveBeenCalledTimes(1);
+  });
+
+  it('rename 완료 후 NFC 경로로 다시 add 이벤트가 와도 recentlyRenamed 가드에 막힌다', async () => {
+    const { nfc, nfd } = makePaths('강의자료-nfc-reentry');
+    mockNormalizeEntry.mockResolvedValue({ type: 'file', status: 'renamed', oldPath: nfd, newPath: nfc });
+
+    const { startDir } = await import('../watcher');
+    await startDir(makeDir('dir-nfc-reentry'));
+
+    mockWatcher.emit('add', nfd);
+    await new Promise((r) => setTimeout(r, 0));
+    expect(mockNormalizeEntry).toHaveBeenCalledTimes(1);
+
+    vi.clearAllMocks();
+
+    // RENAME_TTL(2000ms) 이내에 NFC 경로로 재진입 — 가드에 막혀야 함
+    mockWatcher.emit('add', nfc);
+    await new Promise((r) => setTimeout(r, 0));
+    expect(mockNormalizeEntry).not.toHaveBeenCalled();
+  });
+
+  it('rename 완료 후 NFD 경로로 재진입해도 recentlyRenamed 가드에 막힌다', async () => {
+    const { nfc, nfd } = makePaths('강의자료-nfd-reentry');
+    mockNormalizeEntry.mockResolvedValue({ type: 'file', status: 'renamed', oldPath: nfd, newPath: nfc });
+
+    const { startDir } = await import('../watcher');
+    await startDir(makeDir('dir-nfd-reentry'));
+
+    mockWatcher.emit('add', nfd);
+    await new Promise((r) => setTimeout(r, 0));
+    expect(mockNormalizeEntry).toHaveBeenCalledTimes(1);
+
+    vi.clearAllMocks();
+
+    mockWatcher.emit('add', nfd);
+    await new Promise((r) => setTimeout(r, 0));
+    expect(mockNormalizeEntry).not.toHaveBeenCalled();
+  });
+});

src/main/watcher.ts

@@ -12,6 +12,7 @@ type ActivityListener = (event: ActivityEvent) => void;
 
 const watchers = new Map<string, FSWatcher>();
 const recentlyRenamed = new Map<string, number>();
+const inFlight = new Set<string>();
 const RENAME_TTL = 2000;
 const listeners = new Set<ActivityListener>();
 const manualQueue = new Map<string, string[]>();
@@ -51,20 +52,29 @@ export async function startDir(dir: WatchedDir): Promise<void> {
     persistent: true,
     depth: dir.recursive ? undefined : 0,
     ignored: /(^|[/\\])\../,
+    awaitWriteFinish: { stabilityThreshold: 300, pollInterval: 100 },
   });
 
   async function handlePath(filePath: string, type: 'file' | 'directory'): Promise<void> {
     const now = Date.now();
-    const lastRenamed = recentlyRenamed.get(filePath);
+    const key = filePath.normalize('NFC');
+
+    // 동일 경로가 이미 처리 중이면 즉시 return (burst add 이벤트로 인한 race 차단)
+    if (inFlight.has(key)) return;
+    const lastRenamed = recentlyRenamed.get(key);
     if (lastRenamed && now - lastRenamed < RENAME_TTL) return;
 
     const basename = filePath.split('/').pop() ?? '';
     if (!shouldNormalize(basename, filterOpts)) return;
 
+    inFlight.add(key);
+    try {
       if (dir.mode === 'auto') {
         const result = await normalizeEntry(filePath, type, filterOpts);
         if (result.status === 'renamed' || result.status === 'noop-same-inode') {
-        recentlyRenamed.set(result.newPath, Date.now());
+          // oldPath와 newPath 모두 등록 — chokidar가 NFD/NFC 변형 중 어느 쪽으로 이벤트를 보내도 막힘
+          recentlyRenamed.set(result.oldPath.normalize('NFC'), Date.now());
+          recentlyRenamed.set(result.newPath.normalize('NFC'), Date.now());
           if (result.status === 'renamed') {
             await store.appendUndoEntries([
               { id: nanoid(), ts: now, oldPath: result.oldPath, newPath: result.newPath, reverted: false },
@@ -85,6 +95,9 @@ export async function startDir(dir: WatchedDir): Promise<void> {
         }
         await notifier.notifyManualQueue(queue.length, dir.path);
       }
+    } finally {
+      inFlight.delete(key);
+    }
   }
 
   w.on('add', (p) => handlePath(p, 'file').catch(console.error))
@@ -147,7 +160,8 @@ export async function applyManualQueue(dirId: string, dir: WatchedDir): Promise<
       const result = await normalizeEntry(filePath, type, filterOpts);
       results.push(result);
       if (result.status === 'renamed') {
-        recentlyRenamed.set(result.newPath, Date.now());
+        recentlyRenamed.set(result.oldPath.normalize('NFC'), Date.now());
+        recentlyRenamed.set(result.newPath.normalize('NFC'), Date.now());
         undoEntries.push({ id: nanoid(), ts: now, oldPath: result.oldPath, newPath: result.newPath, reverted: false });
       }
     } catch (err) {