@simplysm/sd-cli 14.0.42 → 14.0.44

package/tests/utils/ngtsc-build-core-write-emit.spec.ts

@@ -22,7 +22,7 @@ describe("writeEmitResults", () => {
   });
 
   it("emitResults를 output-path-rewriting 적용하여 파일로 쓴다", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     const distDir = path.join(pkgDir, "dist");
     // Simulate nested output: dist/my-pkg/src/index.js
@@ -40,7 +40,7 @@ describe("writeEmitResults", () => {
   });
 
   it("output-path-rewriter가 null을 반환하면 파일을 쓰지 않는다", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     const distDir = path.join(pkgDir, "dist");
     // Nested output from OTHER package: dist/other-pkg/src/index.js
@@ -59,7 +59,7 @@ describe("writeEmitResults", () => {
   });
 
   it("이미 flat 구조인 경로는 그대로 쓴다", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     const distDir = path.join(pkgDir, "dist");
     const flatPath = path.join(distDir, "index.js");
@@ -74,7 +74,7 @@ describe("writeEmitResults", () => {
   });
 
   it("디렉��리가 없으면 ��동 생성한다", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     const distDir = path.join(pkgDir, "dist");
     // Nested path under new subdirectory
@@ -109,7 +109,7 @@ describe("writeEmitResults with side-effect SCSS", () => {
   });
 
   it("compiles side-effect SCSS to CSS and rewrites import path in JS", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     // Create source SCSS file
     const scssPath = path.join(srcDir, "ui", "layout", "sd-flex.scss");
@@ -149,7 +149,7 @@ describe("writeEmitResults with side-effect SCSS", () => {
 
   // Acceptance: Scenario "SCSS @use 의존성 해석"
   it("resolves SCSS @use dependencies via loadPaths", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     // Create SCSS load path with a shared variables file
     const scssLoadDir = path.join(pkgDir, "scss");
@@ -194,7 +194,7 @@ describe("writeEmitResults with side-effect SCSS", () => {
 
   // Acceptance: Scenario "SCSS 컴파일 에러"
   it("reports SCSS compilation error in scssErrors", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     // Create SCSS file with syntax error
     const scssPath = path.join(srcDir, "ui", "layout", "broken.scss");
@@ -224,7 +224,7 @@ describe("writeEmitResults with side-effect SCSS", () => {
 
   // Acceptance: Scenario "참조하는 SCSS 파일이 존재하지 않음"
   it("reports error when referenced SCSS file does not exist", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     const distDir = path.join(pkgDir, "dist");
     const jsPath = path.join(distDir, "my-pkg", "src", "ui", "layout", "missing.directive.js");
@@ -267,7 +267,7 @@ describe("writeEmitResults with registry", () => {
   });
 
   it("registers side-effect SCSS entries in registry when provided", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     // Create source SCSS file
     const scssPath = path.join(srcDir, "ui", "layout", "sd-flex.scss");
@@ -304,7 +304,7 @@ describe("writeEmitResults with registry", () => {
   });
 
   it("clears stale registry entries for re-emitted source file before registering new ones", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     const scssPath = path.join(srcDir, "ui", "layout", "sd-flex.scss");
     fs.writeFileSync(scssPath, ".sd-flex { display: flex; }", "utf-8");
@@ -345,7 +345,7 @@ describe("writeEmitResults with registry", () => {
   });
 
   it("removes registry entry when source file is re-emitted without SCSS imports", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     const distDir = path.join(pkgDir, "dist");
     const sourceFileName = path.join(srcDir, "ui", "layout", "sd-flex.directive.ts");
@@ -382,7 +382,7 @@ describe("writeEmitResults with registry", () => {
   });
 
   it("preserves registry entries for non-emitted source files", async () => {
-    const { writeEmitResults } = await import("../../src/angular/angular-build-pipeline");
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
 
     const distDir = path.join(pkgDir, "dist");
     const sourceFileA = path.join(srcDir, "ui", "layout", "sd-flex.directive.ts");
@@ -428,6 +428,130 @@ describe("writeEmitResults with registry", () => {
   });
 });
 
+// Feature 1.1 (review fix): writeEmitResults에서 sideEffectScssDeps 갱신
+describe("writeEmitResults with sideEffectScssDeps", () => {
+  let tmpDir: string;
+  let pkgDir: string;
+  let srcDir: string;
+
+  beforeEach(() => {
+    tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "write-emit-se-deps-"));
+    pkgDir = path.join(tmpDir, "my-pkg");
+    srcDir = path.join(pkgDir, "src");
+    fs.mkdirSync(path.join(pkgDir, "dist"), { recursive: true });
+    fs.mkdirSync(path.join(srcDir, "ui", "layout"), { recursive: true });
+  });
+
+  afterEach(() => {
+    fs.rmSync(tmpDir, { recursive: true, force: true });
+  });
+
+  // Acceptance: 새 side-effect SCSS의 의존성이 sideEffectScssDeps에 기록된다
+  it("records side-effect SCSS dependencies in sideEffectScssDeps after compileScssFile", async () => {
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
+
+    // Create shared variables file
+    const scssLoadDir = path.join(pkgDir, "scss");
+    fs.mkdirSync(path.join(scssLoadDir, "commons"), { recursive: true });
+    fs.writeFileSync(
+      path.join(scssLoadDir, "commons", "_variables.scss"),
+      "$primary: #ff0000;",
+      "utf-8",
+    );
+
+    // Create source SCSS that uses @use
+    const scssPath = path.join(srcDir, "ui", "layout", "sd-card.scss");
+    fs.writeFileSync(
+      scssPath,
+      '@use "commons/variables" as vars;\n.sd-card { color: vars.$primary; }',
+      "utf-8",
+    );
+
+    const distDir = path.join(pkgDir, "dist");
+    const jsPath = path.join(distDir, "my-pkg", "src", "ui", "layout", "sd-card.directive.js");
+    const sourceFileName = path.join(srcDir, "ui", "layout", "sd-card.directive.ts");
+    const emitResults = [
+      {
+        filename: jsPath,
+        contents: 'import "./sd-card.scss";\nexport class SdCardDirective {}',
+        sourceFileName,
+      },
+    ];
+
+    const sideEffectScssDeps = new Map<string, Set<string>>();
+    const scss: SideEffectScssOptions = {
+      loadPaths: [scssLoadDir],
+      scssErrors: [],
+      scssDependencies: new Map(),
+      sideEffectScssDeps,
+    };
+    writeEmitResults(emitResults, pkgDir, scss);
+
+    // sideEffectScssDeps should have the SCSS entry with its dependencies
+    expect(sideEffectScssDeps.has(scssPath)).toBe(true);
+    const deps = sideEffectScssDeps.get(scssPath)!;
+    expect(deps.size).toBeGreaterThan(0);
+    expect([...deps].some((d) => d.includes("_variables.scss"))).toBe(true);
+  });
+
+  // Unit: sideEffectScssDeps 미제공 시 기존 동작 유지
+  it("does not crash when sideEffectScssDeps is not provided", async () => {
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
+
+    const scssPath = path.join(srcDir, "ui", "layout", "sd-flex.scss");
+    fs.writeFileSync(scssPath, ".sd-flex { display: flex; }", "utf-8");
+
+    const distDir = path.join(pkgDir, "dist");
+    const jsPath = path.join(distDir, "my-pkg", "src", "ui", "layout", "sd-flex.directive.js");
+    const emitResults = [
+      {
+        filename: jsPath,
+        contents: 'import "./sd-flex.scss";\nexport class SdFlexDirective {}',
+        sourceFileName: path.join(srcDir, "ui", "layout", "sd-flex.directive.ts"),
+      },
+    ];
+
+    const scss: SideEffectScssOptions = {
+      loadPaths: [],
+      scssErrors: [],
+      scssDependencies: new Map(),
+    };
+
+    // Should not throw
+    writeEmitResults(emitResults, pkgDir, scss);
+    expect(scss.scssErrors).toEqual([]);
+  });
+
+  // Unit: SCSS 컴파일 에러 시 sideEffectScssDeps에 기록하지 않는다
+  it("does not record in sideEffectScssDeps when SCSS compilation fails", async () => {
+    const { writeEmitResults } = await import("../../src/angular/ngtsc-build-core");
+
+    const distDir = path.join(pkgDir, "dist");
+    const jsPath = path.join(distDir, "my-pkg", "src", "ui", "layout", "missing.directive.js");
+    const emitResults = [
+      {
+        filename: jsPath,
+        contents: 'import "./missing.scss";\nexport class MissingDirective {}',
+        sourceFileName: path.join(srcDir, "ui", "layout", "missing.directive.ts"),
+      },
+    ];
+
+    const sideEffectScssDeps = new Map<string, Set<string>>();
+    const scss: SideEffectScssOptions = {
+      loadPaths: [],
+      scssErrors: [],
+      scssDependencies: new Map(),
+      sideEffectScssDeps,
+    };
+    writeEmitResults(emitResults, pkgDir, scss);
+
+    // Error should be reported
+    expect(scss.scssErrors.length).toBeGreaterThan(0);
+    // sideEffectScssDeps should be empty (no successful compilation)
+    expect(sideEffectScssDeps.size).toBe(0);
+  });
+});
+
 // Feature 1.2: compileSideEffectScss
 describe("compileSideEffectScss", () => {
   let tmpDir: string;

package/tests/utils/ngtsc-build-core.spec.ts

@@ -87,17 +87,10 @@ vi.mock("../../src/angular/angular-build", () => {
   };
 });
 
-const {
-  buildScssLoadPaths,
-} = await import("../../src/angular/ngtsc-build-core");
 const {
   createLibraryTransformStylesheet,
-} = await import("../../src/angular/angular-build-pipeline");
-
-import { join } from "node:path";
+} = await import("../../src/angular/ngtsc-build-core");
 
-const fakeCwd = "/workspace";
-const fakePkgDir = "/workspace/packages/my-angular-lib";
 
 // ─── createLibraryTransformStylesheet ───
 
@@ -193,39 +186,3 @@ describe("createLibraryTransformStylesheet", () => {
   });
 });
 
-// ─── buildScssLoadPaths ───
-
-describe("buildScssLoadPaths", () => {
-  it("returns [pkgDir/scss, cwd/node_modules] for given NgtscBuildInfo", () => {
-    const info = {
-      name: "my-angular-lib",
-      cwd: fakeCwd,
-      pkgDir: fakePkgDir,
-      output: { js: true, dts: false },
-    };
-
-    const result = buildScssLoadPaths(info);
-
-    expect(result).toEqual([
-      join(fakePkgDir, "scss"),
-      join(fakeCwd, "node_modules"),
-    ]);
-  });
-
-  it("uses cwd from info to construct node_modules path", () => {
-    const customCwd = "/custom/workspace";
-    const customPkgDir = "/custom/workspace/packages/my-pkg";
-
-    const result = buildScssLoadPaths({
-      name: "test",
-      cwd: customCwd,
-      pkgDir: customPkgDir,
-      output: { js: true, dts: false },
-    });
-
-    expect(result).toEqual([
-      join(customPkgDir, "scss"),
-      join(customCwd, "node_modules"),
-    ]);
-  });
-});

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

@@ -1,32 +1,112 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
 import { consola } from "consola";
-import { formatBuildMessages, printErrors, printServers } from "../../src/utils/output-utils";
 import type { BuildResult } from "../../src/runtime/ResultCollector";
+import type { PartialMessage } from "esbuild";
+
+// output-utils.ts가 모듈 로드 시 consola.withTag("sd:cli:output")로 로거를 생성하므로,
+// withTag가 consola 자체를 반환하도록 하여 기존 스파이가 태그 로거 호출을 캡처하게 한다.
+vi.mock("consola", async (importOriginal) => {
+  const mod = await importOriginal<typeof import("consola")>();
+  const orig = mod.consola;
+  vi.spyOn(orig, "withTag").mockReturnValue(orig);
+  return { consola: orig };
+});
+
+const { formatBuildMessages, formatEsbuildMessages, printDiagnostics, printServers } =
+  await import("../../src/utils/output-utils");
 
 vi.spyOn(consola, "error").mockImplementation(() => {});
+vi.spyOn(consola, "warn").mockImplementation(() => {});
 vi.spyOn(consola, "info").mockImplementation(() => {});
 
 describe("formatBuildMessages", () => {
   it("formats name, label, and messages into indented lines", () => {
     const result = formatBuildMessages("core", "node", ["error in file.ts"]);
-    expect(result).toBe("core (node)\n  → error in file.ts");
+    expect(result).toBe("core (node)\n  error in file.ts");
   });
 
   it("splits multiline messages into separate indented lines", () => {
     const result = formatBuildMessages("core", "node", ["line1\nline2"]);
-    expect(result).toContain("→ line1");
-    expect(result).toContain("→ line2");
+    expect(result).toContain("  line1");
+    expect(result).toContain("  line2");
   });
 
   it("handles multiple messages", () => {
     const result = formatBuildMessages("core", "node", ["err1", "err2"]);
     expect(result).toContain("core (node)");
-    expect(result).toContain("→ err1");
-    expect(result).toContain("→ err2");
+    expect(result).toContain("  err1");
+    expect(result).toContain("  err2");
+  });
+
+  it("preserves empty lines without arrow prefix", () => {
+    const result = formatBuildMessages("core", "node", ["line1\n\nline3"]);
+    const lines = result.split("\n");
+    expect(lines[0]).toBe("core (node)");
+    expect(lines[1]).toBe("  line1");
+    expect(lines[2]).toBe("");
+    expect(lines[3]).toBe("  line3");
   });
 });
 
-describe("printErrors", () => {
+describe("formatEsbuildMessages", () => {
+  it("빈 배열이면 빈 배열을 반환한다", () => {
+    expect(formatEsbuildMessages([], "error")).toEqual([]);
+  });
+
+  it("location이 없는 에러 메시지를 포맷한다", () => {
+    const msgs: PartialMessage[] = [{ text: "Some global error" }];
+    const result = formatEsbuildMessages(msgs, "error");
+    expect(result).toHaveLength(1);
+    expect(result[0]).toContain("Some global error");
+    expect(result[0]).not.toContain("[ERROR]");
+  });
+
+  it("location이 있으면 파일 경로와 코드 컨텍스트를 포함한다", () => {
+    const msgs: PartialMessage[] = [{
+      text: "Property 'id' does not exist",
+      location: {
+        file: "src/app/page.ts",
+        line: 10,
+        column: 4,
+        length: 2,
+        lineText: "    item.id;",
+      },
+    }];
+    const result = formatEsbuildMessages(msgs, "error");
+    expect(result).toHaveLength(1);
+    expect(result[0]).toContain("src/app/page.ts");
+    expect(result[0]).toContain("Property 'id' does not exist");
+  });
+
+  it("notes를 포함한다", () => {
+    const msgs: PartialMessage[] = [{
+      text: "Build failed",
+      notes: [{ text: "hint message" }],
+    }];
+    const result = formatEsbuildMessages(msgs, "error");
+    expect(result[0]).toContain("hint message");
+  });
+
+  it("warning kind에서 [WARNING] 접두사를 제거한다", () => {
+    const msgs: PartialMessage[] = [{ text: "Unused variable" }];
+    const result = formatEsbuildMessages(msgs, "warning");
+    expect(result[0]).not.toContain("[WARNING]");
+    expect(result[0]).toContain("Unused variable");
+  });
+
+  it("여러 메시지를 각각 포맷한다", () => {
+    const msgs: PartialMessage[] = [
+      { text: "Error A" },
+      { text: "Error B" },
+    ];
+    const result = formatEsbuildMessages(msgs, "error");
+    expect(result).toHaveLength(2);
+    expect(result[0]).toContain("Error A");
+    expect(result[1]).toContain("Error B");
+  });
+});
+
+describe("printDiagnostics", () => {
   beforeEach(() => {
     vi.clearAllMocks();
   });
@@ -35,7 +115,7 @@ describe("printErrors", () => {
     const results = new Map<string, BuildResult>([
       ["core:build", { name: "core", target: "node", type: "build", status: "error", message: "failed" }],
     ]);
-    printErrors(results);
+    printDiagnostics(results);
     expect(consola.error).toHaveBeenCalledOnce();
   });
 
@@ -43,7 +123,7 @@ describe("printErrors", () => {
     const results = new Map<string, BuildResult>([
       ["core:build", { name: "core", target: "node", type: "build", status: "error" }],
     ]);
-    printErrors(results);
+    printDiagnostics(results);
     expect(consola.error).toHaveBeenCalledOnce();
   });
 
@@ -51,7 +131,7 @@ describe("printErrors", () => {
     const results = new Map<string, BuildResult>([
       ["core:build", { name: "core", target: "node", type: "build", status: "success" }],
     ]);
-    printErrors(results);
+    printDiagnostics(results);
     expect(consola.error).not.toHaveBeenCalled();
   });
 
@@ -61,7 +141,7 @@ describe("printErrors", () => {
     const results = new Map<string, BuildResult>([
       ["my-server:lint", { name: "my-server", target: "server", type: "lint", status: "error", message: "no-unused-vars" }],
     ]);
-    printErrors(results);
+    printDiagnostics(results);
     const callArg = vi.mocked(consola.error).mock.calls[0][0] as string;
     expect(callArg).toContain("my-server (lint)");
   });
@@ -70,7 +150,7 @@ describe("printErrors", () => {
     const results = new Map<string, BuildResult>([
       ["core:build", { name: "core-common", target: "node", type: "build", status: "error", message: "build err" }],
     ]);
-    printErrors(results);
+    printDiagnostics(results);
     const callArg = vi.mocked(consola.error).mock.calls[0][0] as string;
     expect(callArg).toContain("core-common (node)");
   });
@@ -79,8 +159,48 @@ describe("printErrors", () => {
     const results = new Map<string, BuildResult>([
       ["core:lint", { name: "core-common", target: "node", type: "lint", status: "success" }],
     ]);
-    printErrors(results);
+    printDiagnostics(results);
+    expect(consola.error).not.toHaveBeenCalled();
+  });
+
+  //#endregion
+
+  //#region Feature 1.1 Slice 2: printDiagnostics() 경고 출력 확장
+
+  it("prints warnings with consola.warn", () => {
+    const results = new Map<string, BuildResult>([
+      ["core:build", { name: "core", target: "node", type: "build", status: "success", warnings: "unused var" }],
+    ]);
+    printDiagnostics(results);
+    expect(consola.error).not.toHaveBeenCalled();
+    expect(consola.warn).toHaveBeenCalledOnce();
+    const callArg = vi.mocked(consola.warn).mock.calls[0][0] as string;
+    expect(callArg).toContain("core (node)");
+    expect(callArg).toContain("unused var");
+  });
+
+  it("prints errors before warnings", () => {
+    const results = new Map<string, BuildResult>([
+      ["core:build", { name: "core", target: "node", type: "build", status: "error", message: "type error" }],
+      ["lib:build", { name: "lib", target: "browser", type: "build", status: "success", warnings: "deprecation" }],
+    ]);
+    printDiagnostics(results);
+    expect(consola.error).toHaveBeenCalledOnce();
+    expect(consola.warn).toHaveBeenCalledOnce();
+
+    // error가 warn보다 먼저 호출되었는지 확인
+    const errorOrder = vi.mocked(consola.error).mock.invocationCallOrder[0];
+    const warnOrder = vi.mocked(consola.warn).mock.invocationCallOrder[0];
+    expect(errorOrder).toBeLessThan(warnOrder);
+  });
+
+  it("does not print warnings when there are none", () => {
+    const results = new Map<string, BuildResult>([
+      ["core:build", { name: "core", target: "node", type: "build", status: "success" }],
+    ]);
+    printDiagnostics(results);
     expect(consola.error).not.toHaveBeenCalled();
+    expect(consola.warn).not.toHaveBeenCalled();
   });
 
   //#endregion

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

@@ -32,7 +32,13 @@ describe("watchReplaceDeps onChanged 콜백", () => {
     tmpDir = await fs.promises.mkdtemp(path.join(os.tmpdir(), "sd-replace-deps-"));
 
     // 소스 패키지 생성
-    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 a = 1;");
 

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-mtime-incremental.verify.md

@@ -0,0 +1,10 @@
+# client.worker mtime 증분 추적 — LLM 검증
+
+## 검증 항목
+
+- [x] createSourceFileCachePlugin이 IncrementalMtimeTracker를 사용하는지 확인: `client.worker.ts:198` — `const mtimeTracker = new IncrementalMtimeTracker();` 확인됨
+- [x] onStart에서 detectChanges 결과로 invalidate가 호출되는지 확인: `client.worker.ts:210-213` — `mtimeTracker.detectChanges(watchTargets)` 결과를 `sourceFileCache.invalidate(changedFiles)`에 전달. 기존 로직과 동작 동일
+- [x] onEnd에서 updateMtimes가 호출되는지 확인: `client.worker.ts:228` — `mtimeTracker.updateMtimes(watchTargets)` 호출 확인. `prevMtimes.clear()` + 전체 순회 제거됨
+- [x] esbuildResult == null 가드가 유지되는지 확인: onStart의 `if (esbuildResult != null)` (라인 202), onEnd의 `if (esbuildResult == null) return;` (라인 222) 모두 유지됨
+- [x] isInitialBuild 가드와 sender.send("buildStart")가 유지되는지 확인: `client.worker.ts:216-218` — `if (!isInitialBuild) { sender.send("buildStart", {}); }` 유지됨
+- [x] import 경로에 .js 확장자가 없는지 확인: `client.worker.ts:20` — `from "./incremental-mtime-tracker"` (확장자 없음)