@simplysm/sd-cli 14.0.41 → 14.0.43

package/tests/utils/replace-deps-watch.spec.ts

@@ -17,7 +17,13 @@ describe("watchReplaceDeps onChanged", () => {
     tmpDir = await fs.promises.mkdtemp(path.join(os.tmpdir(), "sd-replace-deps-unit-"));
 
     // 소스 패키지 생성
-    const sourceDir = path.join(tmpDir, "source-pkg", "src");
+    const sourcePkg = path.join(tmpDir, "source-pkg");
+    await fs.promises.mkdir(sourcePkg, { recursive: true });
+    await fs.promises.writeFile(
+      path.join(sourcePkg, "package.json"),
+      JSON.stringify({ name: "@test/pkg", files: ["src"] }),
+    );
+    const sourceDir = path.join(sourcePkg, "src");
     await fs.promises.mkdir(sourceDir, { recursive: true });
     await fs.promises.writeFile(path.join(sourceDir, "index.ts"), "export const v = 1;");
 
@@ -75,6 +81,56 @@ describe("watchReplaceDeps onChanged", () => {
     expect(callCount).toBe(1);
   }, 10_000);
 
+  it("files에 없는 파일의 변경은 감지되지 않는다", async () => {
+    const projectRoot = path.join(tmpDir, "project");
+    const sourcePath = path.join(tmpDir, "source-pkg");
+
+    // files에 없는 파일 생성
+    await fs.promises.writeFile(path.join(sourcePath, "tsconfig.json"), "{}");
+
+    let callCount = 0;
+
+    watchResult = await watchReplaceDeps(projectRoot, { "@test/pkg": sourcePath }, {
+      onChanged: () => {
+        callCount++;
+      },
+    });
+
+    // files에 없는 파일 변경
+    await fs.promises.writeFile(path.join(sourcePath, "tsconfig.json"), '{"strict": true}');
+
+    // 충분한 대기 시간 (300ms 배칭 + 여유)
+    await new Promise((r) => setTimeout(r, 1500));
+
+    // files에 없으므로 감지되지 않음
+    expect(callCount).toBe(0);
+  }, 10_000);
+
+  it("npm 기본 파일(README.md) 변경이 감지된다", async () => {
+    const projectRoot = path.join(tmpDir, "project");
+    const sourcePath = path.join(tmpDir, "source-pkg");
+
+    // 소스에 README.md 생성
+    await fs.promises.writeFile(path.join(sourcePath, "README.md"), "# README v1");
+
+    let resolveChanged: () => void;
+    const changedPromise = new Promise<void>((resolve) => {
+      resolveChanged = resolve;
+    });
+
+    watchResult = await watchReplaceDeps(projectRoot, { "@test/pkg": sourcePath }, {
+      onChanged: () => {
+        resolveChanged();
+      },
+    });
+
+    // README.md 변경
+    await fs.promises.writeFile(path.join(sourcePath, "README.md"), "# README v2");
+
+    // 콜백 호출 대기
+    await changedPromise;
+  }, 10_000);
+
   it("options 파라미터가 undefined일 때 에러가 발생하지 않는다", async () => {
     const projectRoot = path.join(tmpDir, "project");
     const sourcePath = path.join(tmpDir, "source-pkg");

package/tests/utils/worker-utils.spec.ts

@@ -11,7 +11,7 @@ describe("setupWorkerConsola", () => {
   afterEach(() => {
     consola.level = originalLevel;
     consola.options.reporters = originalReporters;
-    if (originalEnv === undefined) {
+    if (originalEnv == null) {
       delete process.env["SD_DEBUG"];
     } else {
       process.env["SD_DEBUG"] = originalEnv;
@@ -40,7 +40,13 @@ describe("createOnceGuard", () => {
   it("throws on second call", () => {
     const guard = createOnceGuard("startWatch");
     guard();
-    expect(() => guard()).toThrow("startWatch can only be called once per Worker");
+    expect(() => guard()).toThrow("Worker당 한 번만 호출할 수 있습니다: startWatch");
+  });
+
+  it("includes the label in Korean error message", () => {
+    const guard = createOnceGuard("myFunction");
+    guard();
+    expect(() => guard()).toThrow("Worker당 한 번만 호출할 수 있습니다: myFunction");
   });
 });
 

package/tests/workers/client-worker-initial-build-error.verify.md

@@ -3,6 +3,5 @@
 ## 검증 항목
 
 - [x] client.worker.ts onEnd에서 초기 빌드 시 errors 필드가 포함되는지: `client.worker.ts:285-291` — `initialBuildResolve`에 `errors: result.errors.map((e) => e.text)` 포함 확인
-- [x] EsbuildClientEngine.startWatch에서 반환값의 success 확인 후 에러 로깅하는지: `EsbuildClientEngine.ts:135-137` — `result != null && !result.success` 조건으로 `logger.error` 호출 확인. 테스트 실행 시 `[my-client] 초기 빌드 실패: Module not found: @angular/core; Syntax error in app.ts` 출력 확인
-- [x] esbuild-client-config.ts에서 dev 모드 logLevel이 "warning"인지: `esbuild-client-config.ts:169` — `logLevel: isDev ? "warning" : "silent"` 확인. `isDev = options.mode === "dev"` (line 52)
-- [x] esbuild-client-config.ts에서 build 모드 logLevel이 "silent"인지: 동일 라인 — 삼항 연산자의 false 분기가 `"silent"` 확인
+- [x] EsbuildClientEngine.startWatch에서 반환값의 success 확인 후 ResultCollector에 보고하는지: `EsbuildClientEngine.ts:129-138` — `!result.success` 조건으로 `resultCollector.add()` 호출 확인. logger.error는 Feature 1.2에서 제거됨 (중복 출력 방지)
+- [x] esbuild-client-config.ts에서 logLevel이 "silent"인지: `esbuild-client-config.ts:191` — `logLevel: "silent"` 확인. Feature 1.2에서 dev/build 모두 "silent"로 통일됨

package/tests/workers/client-worker-initial-build-warnings.verify.md

@@ -0,0 +1,7 @@
+# 초기 빌드 warnings 전달 — LLM 검증
+
+## 검증 항목
+
+- [x] client.worker.ts의 initialBuildResolve에 warnings 필드가 포함되는지: `client.worker.ts:305-315` — `initialBuildResolve`에 `warnings: result.warnings.length > 0 ? result.warnings.map(formatEsbuildMessage) : undefined` 포함 확인. 후속 빌드(line 295-298)와 동일 패턴
+- [x] warnings가 없을 때 undefined인지: `client.worker.ts:312-314` — `result.warnings.length > 0` 조건으로 빈 배열일 때 undefined 반환 확인
+- [x] ClientBuildResult 인터페이스에 warnings 필드가 있는지: `client.worker.ts:47` — `warnings?: string[]` 확인. 변경 불필요

package/tests/workers/client-worker-refactor.verify.md

@@ -0,0 +1,22 @@
+# client.worker.ts startWatch 서브함수 추출 — LLM 검증
+
+## 검증 항목
+
+- [x] createSourceFileCachePlugin이 모듈 스코프에 존재하고 esbuild.Plugin을 반환한다: lines 192-250, `function createSourceFileCachePlugin(): esbuild.Plugin` 반환 타입 명시, `name: "sd-build-start"` 유지
+- [x] createDevBuildEndHandler가 모듈 스코프에 존재하고 onEnd 콜백을 반환한다: lines 255-322, 반환 타입 `(result: esbuild.BuildResult) => Promise<void>`
+- [x] startWatch에서 추출 함수를 호출한다: line 385 `plugins: [createSourceFileCachePlugin()]`, line 386 `onEnd: createDevBuildEndHandler(basePath, actualPort, outdir, entryNames, info.pkgDir)`
+- [x] 파일 변경 감지 로직이 동일하다: createSourceFileCachePlugin onStart (lines 198-230) — watchFiles + typeScriptFileCache 순회, mtime 비교, invalidate 호출, buildStart 이벤트 전송 모두 원본과 동일
+- [x] mtime 기록 로직이 동일하다: createSourceFileCachePlugin onEnd (lines 232-247) — prevMtimes.clear() + mtime 기록 원본과 동일
+- [x] index.html 재생성 로직이 동일하다: createDevBuildEndHandler (lines 264-279) — lastMetafile 보관 + generateIndexHtml + writeFileSync 원본과 동일
+- [x] HMR 디스패치 로직이 동일하다: lines 281-284, `hmrService.onBuildEnd(result.metafile)` 조건부 호출 원본과 동일
+- [x] build 이벤트 전송 로직이 동일하다: lines 286-300, `sender.send("build", ...)` 조건부 호출 원본과 동일
+- [x] 초기 빌드 resolve 로직이 동일하다: lines 302-312, `isInitialBuild = false` + `initialBuildResolve?.()` 원본과 동일
+- [x] 에러 핸들링 로직이 동일하다: lines 313-320, catch 블록 원본과 동일
+- [x] 외부 인터페이스가 변경되지 않았다: export는 `export default sender`만 존재 (line 501), ClientBuildInfo/ClientBuildResult/ClientWorkerEvents 타입 변경 없음
+- [x] build 함수가 변경되지 않았다: lines 96-187, 원본과 동일
+- [x] stopWatch에 리셋 로직이 추가되었다: lines 472-475, `lastMetafile = undefined`, `isInitialBuild = true`, `initialBuildResolve = undefined`
+- [x] stopWatch 기존 정리 로직이 변경되지 않았다: lines 439-470, 5단계 정리(esbuild dispose, HMR 종료, HTTP 종료, public 감시 종료, index.html 감시 종료) 원본과 동일
+- [x] 모듈 스코프 변수가 올바르게 선언되었다: lines 67-69, 기존 변수(62-66) 바로 아래 배치
+- [x] startWatch가 ~110줄로 축소되었다: lines 327-434 = 107줄 (목표 ~110줄 달성)
+- [x] index.html watcher에서 lastMetafile 접근이 정상 동작한다: line 400, `if (lastMetafile == null) return;` — 모듈 스코프 변수로 접근
+- [x] 기존 테스트 6개 전부 통과: client-worker.spec.ts (3), client-worker.acc.spec.ts (3) — 회귀 없음

package/tests/workers/client-worker-ts-cache-invalidation.verify.md

@@ -0,0 +1,12 @@
+# sd-build-start 플러그인 TypeScript 파일 감시 — LLM 검증
+
+## 검증 항목
+
+- [x] onStart에서 `typeScriptFileCache.keys()`가 watchTargets에 포함: `client.worker.ts:266` — `...typeScriptFileCache.keys()`가 `watchTargets` 배열에 spread되어 포함됨
+- [x] onStart에서 `loadResultCache.watchFiles`도 여전히 watchTargets에 포함: `client.worker.ts:265` — `...loadResultCache.watchFiles`가 첫 번째로 spread됨 (기존 JS 동작 보존)
+- [x] onStart에서 watchTargets의 각 파일에 대해 mtime 비교 수행: `client.worker.ts:268-279` — `for (const file of watchTargets)` 루프에서 `fs.statSync(file).mtimeMs`와 `prevMtimes.get(file)` 비교
+- [x] 초기 빌드 직후 prevMtimes가 비어있으면 mtime 비교 스킵: `client.worker.ts:271-273` — `prev != null` 조건에 의해 `prevMtimes`에 없는 파일은 `changedFiles`에 추가되지 않음
+- [x] onEnd에서 `typeScriptFileCache.keys()`가 prevMtimes 기록 대상에 포함: `client.worker.ts:297` — `...esbuildResult.sourceFileCache.typeScriptFileCache.keys()`가 watchTargets에 spread됨
+- [x] onEnd에서 `loadResultCache.watchFiles`도 여전히 prevMtimes 기록 대상에 포함: `client.worker.ts:296` — `...esbuildResult.sourceFileCache.loadResultCache.watchFiles`가 첫 번째로 spread됨
+- [x] `typeScriptFileCache`가 `sourceFileCache`에서 destructuring으로 접근: `client.worker.ts:260` — `const { loadResultCache, typeScriptFileCache } = esbuildResult.sourceFileCache;` SourceFileCache의 public 속성 (`source-file-cache.js:52`)
+- [x] 삭제된 TS 파일에 대한 catch 처리: `client.worker.ts:275-279` — `catch` 블록에서 `prevMtimes.has(file)`인 경우 `changedFiles`에 추가 (기존 로직 동일)

package/tests/workers/client-worker.acc.spec.ts

@@ -122,10 +122,12 @@ describe("client.worker build() — Acceptance", () => {
     const result = await workerFns["build"](baseBuildInfo);
 
     expect(result.success).toBe(false);
-    expect(result.errors).toContain("Could not resolve 'missing-module'");
-    expect(result.errors).toContain("Syntax error in file.ts");
+    // formatEsbuildMessages가 ANSI 색상 코드를 포함하므로 부분 문자열 매칭
+    const errorsJoined = result.errors!.join("\n");
+    expect(errorsJoined).toContain("Could not resolve 'missing-module'");
+    expect(errorsJoined).toContain("Syntax error in file.ts");
     // 요약 메시지("Build failed with 2 errors")가 아닌 상세 에러만 포함
-    expect(result.errors).not.toContain("Build failed with 2 errors");
+    expect(errorsJoined).not.toContain("Build failed with 2 errors");
   });
 
   // Scenario: 일반 Error 발생 시 기존 폴백이 동작한다
@@ -183,3 +185,4 @@ describe("client.worker build() — browserSupport 전달", () => {
     );
   });
 });
+

package/tests/workers/library-build-lint.spec.ts

@@ -1,41 +1,24 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
-import * as tscBuildModule from "../../src/utils/tsc-build";
 
 // --- Mocks ---
 
-const { mockLintFn, MockLintWithProgramRunner } = vi.hoisted(() => {
-  const lintFn = vi.fn().mockResolvedValue({
-    success: true,
-    errorCount: 0,
-    warningCount: 0,
-    formattedOutput: "",
+const { mockCompileAsync, MockSdTsCompiler } = vi.hoisted(() => {
+  const compileAsync = vi.fn();
+  const Compiler = vi.fn().mockImplementation(function () {
+    return { compileAsync };
   });
-  const RunnerCls = vi.fn().mockImplementation(function () {
-    return { lint: lintFn };
-  });
-  return { mockLintFn: lintFn, MockLintWithProgramRunner: RunnerCls };
+  return { mockCompileAsync: compileAsync, MockSdTsCompiler: Compiler };
 });
 
-vi.mock("../../src/lint/lint-with-program", () => ({
-  LintWithProgramRunner: MockLintWithProgramRunner,
+vi.mock("../../src/ts-compiler/SdTsCompiler", () => ({
+  SdTsCompiler: MockSdTsCompiler,
 }));
 
-const mockTscResult = {
-  success: true,
-  errors: undefined,
-  diagnostics: [],
-  errorCount: 0,
-  warningCount: 0,
-  program: { getSourceFiles: () => [] },
-};
-
-const runTscSpy = vi.spyOn(tscBuildModule, "runTscPackageBuild")
-  .mockReturnValue(mockTscResult as any);
-
-vi.mock("../../src/runtime/worker-utils", () => ({
-  registerCleanupHandlers: vi.fn(),
-  createOnceGuard: vi.fn(() => vi.fn()),
-  setupWorkerConsola: vi.fn(),
+vi.mock("../../src/workers/shared-worker-lifecycle", () => ({
+  setupWorkerLifecycle: vi.fn(() => ({
+    logger: { debug: vi.fn(), warn: vi.fn() },
+    guardStartWatch: vi.fn(),
+  })),
 }));
 
 vi.mock("@simplysm/core-node", () => ({
@@ -52,6 +35,21 @@ vi.mock("../../src/deps/replace-deps/collect-deps", () => ({
   collectDeps: vi.fn(() => ({ workspaceDeps: [], replaceDeps: [] })),
 }));
 
+const defaultCompileResult = {
+  program: { getSourceFiles: () => [] },
+  builderProgram: {},
+  isForAngular: false,
+  affectedFiles: new Set<string>(),
+  diagnostics: [] as any[],
+  errorCount: 0,
+  warningCount: 0,
+  errors: undefined as string[] | undefined,
+  emitResults: undefined,
+  lint: undefined as any,
+  scssErrors: [] as string[],
+  scssDependencies: new Map<string, Set<string>>(),
+};
+
 const workerMethods: Record<string, Function> = {};
 
 await import("../../src/workers/library-build.worker");
@@ -60,33 +58,31 @@ await import("../../src/workers/library-build.worker");
 
 beforeEach(() => {
   vi.clearAllMocks();
-  runTscSpy.mockReturnValue(mockTscResult as any);
-  mockLintFn.mockResolvedValue({
-    success: true,
-    errorCount: 0,
-    warningCount: 0,
-    formattedOutput: "",
-  });
-  mockTscResult.program = { getSourceFiles: () => [] } as any;
+  mockCompileAsync.mockResolvedValue({ ...defaultCompileResult });
 });
 
 describe("library-build.worker lint integration (Slice 3)", () => {
   describe("Scenario: library-build.worker runs lint after typecheck", () => {
     it("returns lint result in build output when lint is enabled", async () => {
+      const lintResult = {
+        success: true,
+        errorCount: 0,
+        warningCount: 0,
+        formattedOutput: "",
+      };
+      mockCompileAsync.mockResolvedValueOnce({
+        ...defaultCompileResult,
+        lint: lintResult,
+      });
+
       const result = await workerMethods["build"]({
         name: "my-lib",
-        config: { target: "node" },
         cwd: "/workspace",
         pkgDir: "/workspace/packages/my-lib",
         output: { js: true, dts: true, lint: true },
       });
 
-      expect(result.lint).toEqual({
-        success: true,
-        errorCount: 0,
-        warningCount: 0,
-        formattedOutput: "",
-      });
+      expect(result.lint).toEqual(lintResult);
     });
   });
 
@@ -94,7 +90,6 @@ describe("library-build.worker lint integration (Slice 3)", () => {
     it("does not run lint when output.lint is false", async () => {
       const result = await workerMethods["build"]({
         name: "my-lib",
-        config: { target: "node" },
         cwd: "/workspace",
         pkgDir: "/workspace/packages/my-lib",
         output: { js: true, dts: true },