@simplysm/sd-cli 14.0.8 → 14.0.9

package/tests/engines/server-esbuild-engine.spec.ts

@@ -52,8 +52,7 @@ function createMockPkg(overrides: Partial<ServerPackageInfo> = {}): ServerPackag
 
 function setupDefaultBuildResult(): void {
   mockWorker.build.mockResolvedValue({
-    js: { success: true, errors: undefined, warnings: undefined },
-    dts: { success: true, errors: undefined, diagnostics: [] },
+    build: { success: true, errors: undefined, warnings: undefined, diagnostics: [] },
     mainJsPath: "/packages/test-server/dist/main.js",
   });
   mockWorker.terminate.mockResolvedValue(undefined);
@@ -91,16 +90,14 @@ describe("ServerEsbuildEngine", () => {
         }),
       );
       expect(result.success).toBe(true);
-      expect(result.js.success).toBe(true);
-      expect(result.dts.success).toBe(true);
+      expect(result.build.success).toBe(true);
       await engine.stop();
     });
 
     // Acceptance: maps ServerBuildResult to EngineResult
-    it("maps worker result to EngineResult with js/dts separation", async () => {
+    it("maps worker result to EngineResult with build field", async () => {
       mockWorker.build.mockResolvedValue({
-        js: { success: true, errors: undefined, warnings: ["warn1"] },
-        dts: { success: false, errors: ["type error"], diagnostics: [{ code: 2345, category: 1 }] },
+        build: { success: false, errors: ["type error"], warnings: ["warn1"], diagnostics: [{ code: 2345, category: 1 }] },
         mainJsPath: "/packages/test-server/dist/main.js",
       });
 
@@ -108,17 +105,16 @@ describe("ServerEsbuildEngine", () => {
       const result = await engine.run({ js: true, dts: false });
 
       expect(result.success).toBe(false);
-      expect(result.js.warnings).toEqual(["warn1"]);
-      expect(result.dts.success).toBe(false);
-      expect(result.dts.diagnostics).toEqual([{ code: 2345, category: 1 }]);
+      expect(result.build.warnings).toEqual(["warn1"]);
+      expect(result.build.success).toBe(false);
+      expect(result.build.diagnostics).toEqual([{ code: 2345, category: 1 }]);
       await engine.stop();
     });
 
-    // Unit: JS failure
-    it("reflects JS build failure in result", async () => {
+    // Unit: build failure
+    it("reflects build failure in result", async () => {
       mockWorker.build.mockResolvedValue({
-        js: { success: false, errors: ["esbuild error"], warnings: undefined },
-        dts: { success: true, errors: undefined, diagnostics: [] },
+        build: { success: false, errors: ["esbuild error"], warnings: undefined, diagnostics: [] },
         mainJsPath: "/packages/test-server/dist/main.js",
       });
 
@@ -126,9 +122,8 @@ describe("ServerEsbuildEngine", () => {
       const result = await engine.run({ js: true, dts: false });
 
       expect(result.success).toBe(false);
-      expect(result.js.success).toBe(false);
-      expect(result.js.errors).toEqual(["esbuild error"]);
-      expect(result.dts.success).toBe(true);
+      expect(result.build.success).toBe(false);
+      expect(result.build.errors).toEqual(["esbuild error"]);
       await engine.stop();
     });
   });
@@ -141,8 +136,7 @@ describe("ServerEsbuildEngine", () => {
           (call: any[]) => call[0] === "build",
         )?.[1];
         buildHandler?.({
-          js: { success: true },
-          dts: { success: true },
+          build: { success: true },
           mainJsPath: "/packages/test-server/dist/main.js",
         });
       });
@@ -158,7 +152,7 @@ describe("ServerEsbuildEngine", () => {
     });
 
     // Acceptance: reports to ResultCollector
-    it("reports build and dts results separately to ResultCollector", async () => {
+    it("reports build result to ResultCollector", async () => {
       const mockResultCollector = { add: vi.fn() };
 
       mockWorker.startWatch.mockImplementation(() => {
@@ -166,8 +160,7 @@ describe("ServerEsbuildEngine", () => {
           (call: any[]) => call[0] === "build",
         )?.[1];
         buildHandler?.({
-          js: { success: true, errors: undefined },
-          dts: { success: false, errors: ["type error"] },
+          build: { success: false, errors: ["type error"] },
           mainJsPath: "/packages/test-server/dist/main.js",
         });
       });
@@ -182,12 +175,9 @@ describe("ServerEsbuildEngine", () => {
 
       const addCalls = mockResultCollector.add.mock.calls;
       const buildResult = addCalls.find((c: any[]) => c[0].type === "build");
-      const dtsResult = addCalls.find((c: any[]) => c[0].type === "dts");
 
       expect(buildResult).toBeDefined();
-      expect(buildResult![0].status).toBe("success");
-      expect(dtsResult).toBeDefined();
-      expect(dtsResult![0].status).toBe("error");
+      expect(buildResult![0].status).toBe("error");
 
       await engine.stop();
     });
@@ -199,8 +189,7 @@ describe("ServerEsbuildEngine", () => {
           (call: any[]) => call[0] === "build",
         )?.[1];
         buildHandler?.({
-          js: { success: true },
-          dts: { success: true },
+          build: { success: true },
           mainJsPath: "/packages/test-server/dist/main.js",
         });
       });
@@ -228,7 +217,7 @@ describe("ServerEsbuildEngine", () => {
         const buildHandler = mockWorker.on.mock.calls.find(
           (call: any[]) => call[0] === "build",
         )?.[1];
-        buildHandler?.({ js: { success: true }, dts: { success: true }, mainJsPath: "x" });
+        buildHandler?.({ build: { success: true }, mainJsPath: "x" });
       });
 
       const engine = new ServerEsbuildEngine({ cwd: "/root", pkg: createMockPkg() });

package/tests/engines/tsc-engine.spec.ts

@@ -45,8 +45,7 @@ function createMockPkg(overrides: Partial<BuildPackageInfo> = {}): BuildPackageI
 
 function setupDefaultBuildResult(): void {
   mockWorker.build.mockResolvedValue({
-    js: { success: true, errors: undefined, warnings: undefined },
-    dts: { success: true, errors: undefined, diagnostics: [] },
+    build: { success: true, errors: undefined, warnings: undefined, diagnostics: [] },
   });
   mockWorker.terminate.mockResolvedValue(undefined);
   mockWorker.stopWatch.mockResolvedValue(undefined);
@@ -77,37 +76,34 @@ describe("TscEngine", () => {
         }),
       );
       expect(result.success).toBe(true);
-      expect(result.js.success).toBe(true);
-      expect(result.dts.success).toBe(true);
+      expect(result.build.success).toBe(true);
       await engine.stop();
     });
 
-    it("maps worker result to EngineResult with js/dts separation", async () => {
+    it("maps worker result to EngineResult with build field", async () => {
       mockWorker.build.mockResolvedValue({
-        js: { success: true, errors: undefined, warnings: ["warn1"] },
-        dts: { success: true, errors: undefined, diagnostics: [{ code: 1, category: 0 }] },
+        build: { success: true, errors: undefined, warnings: ["warn1"], diagnostics: [{ code: 1, category: 0 }] },
       });
 
       const engine = new TscEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: true });
 
-      expect(result.js.warnings).toEqual(["warn1"]);
-      expect(result.dts.diagnostics).toEqual([{ code: 1, category: 0 }]);
+      expect(result.build.warnings).toEqual(["warn1"]);
+      expect(result.build.diagnostics).toEqual([{ code: 1, category: 0 }]);
       await engine.stop();
     });
 
     it("reflects failure when tsc reports errors", async () => {
       mockWorker.build.mockResolvedValue({
-        js: { success: false, errors: ["type error"], warnings: undefined },
-        dts: { success: false, errors: ["type error"], diagnostics: [{ code: 2322, category: 1 }] },
+        build: { success: false, errors: ["type error"], warnings: undefined, diagnostics: [{ code: 2322, category: 1 }] },
       });
 
       const engine = new TscEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: true });
 
       expect(result.success).toBe(false);
-      expect(result.js.errors).toEqual(["type error"]);
-      expect(result.dts.diagnostics).toHaveLength(1);
+      expect(result.build.errors).toEqual(["type error"]);
+      expect(result.build.diagnostics).toHaveLength(1);
       await engine.stop();
     });
 
@@ -129,8 +125,7 @@ describe("TscEngine", () => {
           (call: any[]) => call[0] === "build",
         )?.[1];
         buildHandler?.({
-          js: { success: true },
-          dts: { success: true },
+          build: { success: true },
         });
       });
 
@@ -144,7 +139,7 @@ describe("TscEngine", () => {
       await engine.stop();
     });
 
-    it("reports build and dts results separately to ResultCollector", async () => {
+    it("reports build result to ResultCollector", async () => {
       const mockResultCollector = { add: vi.fn() };
 
       mockWorker.startWatch.mockImplementation(() => {
@@ -152,8 +147,7 @@ describe("TscEngine", () => {
           (call: any[]) => call[0] === "build",
         )?.[1];
         buildHandler?.({
-          js: { success: true, errors: undefined },
-          dts: { success: false, errors: ["type error"] },
+          build: { success: false, errors: ["type error"] },
         });
       });
 
@@ -167,12 +161,9 @@ describe("TscEngine", () => {
 
       const addCalls = mockResultCollector.add.mock.calls;
       const buildResult = addCalls.find((c: any[]) => c[0].type === "build");
-      const dtsResult = addCalls.find((c: any[]) => c[0].type === "dts");
 
       expect(buildResult).toBeDefined();
-      expect(buildResult![0].status).toBe("success");
-      expect(dtsResult).toBeDefined();
-      expect(dtsResult![0].status).toBe("error");
+      expect(buildResult![0].status).toBe("error");
 
       await engine.stop();
     });
@@ -185,7 +176,7 @@ describe("TscEngine", () => {
         const buildHandler = mockWorker.on.mock.calls.find(
           (call: any[]) => call[0] === "build",
         )?.[1];
-        buildHandler?.({ js: { success: true }, dts: { success: true } });
+        buildHandler?.({ build: { success: true } });
       });
 
       const engine = new TscEngine({ cwd: "/root", pkg: createMockPkg() });

package/tests/engines/vite-engine.spec.ts

@@ -75,10 +75,9 @@ describe("ViteEngine", () => {
         }),
       );
       expect(result.success).toBe(true);
-      expect(result.js.success).toBe(true);
-      expect(result.js.errors).toEqual([]);
-      expect(result.dts.success).toBe(true);
-      expect(result.dts.diagnostics).toEqual([]);
+      expect(result.build.success).toBe(true);
+      expect(result.build.errors).toEqual([]);
+      expect(result.build.diagnostics).toEqual([]);
       await engine.stop();
     });
 
@@ -93,8 +92,8 @@ describe("ViteEngine", () => {
       const result = await engine.run({ js: true, dts: false });
 
       expect(result.success).toBe(false);
-      expect(result.js.success).toBe(false);
-      expect(result.js.errors).toContain("TS2345: Argument of type...");
+      expect(result.build.success).toBe(false);
+      expect(result.build.errors).toContain("TS2345: Argument of type...");
       await engine.stop();
     });
 
@@ -108,7 +107,7 @@ describe("ViteEngine", () => {
       const engine = new ViteEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: false });
 
-      expect(result.js.warnings).toEqual(["deprecation warning"]);
+      expect(result.build.warnings).toEqual(["deprecation warning"]);
       await engine.stop();
     });
 
@@ -192,8 +191,8 @@ describe("ViteEngine", () => {
       await engine.stop();
     });
 
-    // Unit: dts is always success for client packages
-    it("always returns dts success regardless of build outcome", async () => {
+    // Unit: build failure reflects in result
+    it("reflects build failure in result", async () => {
       mockWorker.build.mockResolvedValue({
         success: false,
         errors: ["error"],
@@ -202,8 +201,7 @@ describe("ViteEngine", () => {
       const engine = new ViteEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: false });
 
-      expect(result.dts.success).toBe(true);
-      expect(result.dts.diagnostics).toEqual([]);
+      expect(result.build.success).toBe(false);
       await engine.stop();
     });
   });
@@ -268,7 +266,7 @@ describe("ViteEngine", () => {
     });
 
     // Acceptance: Scenario "ResultCollector에 결과 보고"
-    it("reports build and dts results to ResultCollector", async () => {
+    it("reports build result to ResultCollector", async () => {
       const mockResultCollector = { add: vi.fn() };
 
       mockWorker.startWatch.mockResolvedValue({ success: true });
@@ -283,12 +281,9 @@ describe("ViteEngine", () => {
 
       const addCalls = mockResultCollector.add.mock.calls;
       const buildResult = addCalls.find((c: any[]) => c[0].type === "build");
-      const dtsResult = addCalls.find((c: any[]) => c[0].type === "dts");
 
       expect(buildResult).toBeDefined();
       expect(buildResult![0].status).toBe("success");
-      expect(dtsResult).toBeDefined();
-      expect(dtsResult![0].status).toBe("success");
 
       await engine.stop();
     });

package/tests/infra/result-collector.spec.ts

@@ -23,12 +23,12 @@ describe("ResultCollector", () => {
   it("returns all results via toMap", () => {
     const collector = new ResultCollector();
     collector.add({ name: "core", target: "node", type: "build", status: "success" });
-    collector.add({ name: "core", target: "node", type: "dts", status: "building" });
+    collector.add({ name: "core", target: "node", type: "lint", status: "building" });
 
     const map = collector.toMap();
     expect(map.size).toBe(2);
     expect(map.has("core:build")).toBe(true);
-    expect(map.has("core:dts")).toBe(true);
+    expect(map.has("core:lint")).toBe(true);
   });
 
   it("returns undefined for non-existent key", () => {

package/tests/orchestrators/build-orchestrator.spec.ts

@@ -51,6 +51,9 @@ vi.mock("@simplysm/core-node", () => ({
   fsx: {
     rm: vi.fn().mockResolvedValue(undefined),
   },
+  pathx: {
+    posixResolve: vi.fn((...args: string[]) => args.join("/").replace(/\\/g, "/")),
+  },
 }));
 
 const mockEngines: Array<{
@@ -64,8 +67,7 @@ vi.mock("../../src/engines/index", () => ({
     const engine = {
       run: vi.fn().mockResolvedValue({
         success: true,
-        js: { success: true, errors: [], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: true, errors: [], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn().mockResolvedValue(undefined),
       stop: vi.fn().mockResolvedValue(undefined),
@@ -158,8 +160,7 @@ beforeEach(() => {
     const engine = {
       run: vi.fn().mockResolvedValue({
         success: true,
-        js: { success: true, errors: [], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: true, errors: [], warnings: [], diagnostics: [] },
 
       }),
       startWatch: vi.fn().mockResolvedValue(undefined),
@@ -413,8 +414,7 @@ describe("BuildOrchestrator.start", () => {
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: false,
-        js: { success: false, errors: ["Module not found"], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: false, errors: ["Module not found"], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
@@ -464,8 +464,7 @@ describe("BuildOrchestrator.start", () => {
           active--;
           return {
             success: true,
-            js: { success: true, errors: [], warnings: [] },
-            dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+            build: { success: true, errors: [], warnings: [], diagnostics: [] },
           };
         }),
         startWatch: vi.fn(),
@@ -498,8 +497,7 @@ describe("BuildOrchestrator.start", () => {
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: true,
-        js: { success: true, errors: [], warnings: ["Unused variable"] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: true, errors: [], warnings: ["Unused variable"], diagnostics: [] },
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
@@ -606,8 +604,7 @@ describe("BuildOrchestrator client build", () => {
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: true,
-        js: { success: true, errors: [], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: true, errors: [], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
@@ -641,8 +638,7 @@ describe("BuildOrchestrator client build", () => {
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: false,
-        js: { success: false, errors: ["Template error"], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: false, errors: ["Template error"], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
@@ -962,8 +958,7 @@ describe("BuildOrchestrator native build integration (Slice 1)", () => {
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: false,
-        js: { success: false, errors: ["Build error"], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: false, errors: ["Build error"], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
@@ -1032,8 +1027,7 @@ describe("BuildOrchestrator lint integration", () => {
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: true,
-        js: { success: true, errors: [], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: true, errors: [], warnings: [], diagnostics: [] },
         lint: { success: false, errorCount: 3, warningCount: 0, formattedOutput: "lint errors here" },
       }),
       startWatch: vi.fn(),
@@ -1080,8 +1074,7 @@ describe("Feature 2.1: build에서 lint 에러가 'lint' 라벨로 출력", () =
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: true,
-        js: { success: true, errors: [], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: true, errors: [], warnings: [], diagnostics: [] },
         lint: { success: false, errorCount: 1, warningCount: 0, formattedOutput: "no-unused-vars" },
       }),
       startWatch: vi.fn(),
@@ -1109,8 +1102,7 @@ describe("Feature 2.1: build에서 lint 에러가 'lint' 라벨로 출력", () =
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: false,
-        js: { success: false, errors: ["Module not found"], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: false, errors: ["Module not found"], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
@@ -1126,7 +1118,7 @@ describe("Feature 2.1: build에서 lint 에러가 'lint' 라벨로 출력", () =
     expect(jsCall).toBeDefined();
   });
 
-  it("build에서 DTS 에러는 'dts' 라벨 유지", async () => {
+  it("build에서 빌드 에러는 target 라벨 유지", async () => {
     setupDefaults({
       packages: {
         "core-common": { target: "neutral", publish: { type: "npm" } },
@@ -1135,8 +1127,7 @@ describe("Feature 2.1: build에서 lint 에러가 'lint' 라벨로 출력", () =
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: false,
-        js: { success: true, errors: [], warnings: [] },
-        dts: { success: false, errors: ["Type error"], warnings: [], diagnostics: [] },
+        build: { success: false, errors: ["Type error"], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
@@ -1148,8 +1139,8 @@ describe("Feature 2.1: build에서 lint 에러가 'lint' 라벨로 출력", () =
 
     const { formatBuildMessages } = await import("../../src/utils/output-utils");
     const fmtCalls = vi.mocked(formatBuildMessages).mock.calls;
-    const dtsCall = fmtCalls.find((c) => c[1] === "dts");
-    expect(dtsCall).toBeDefined();
+    const buildCall = fmtCalls.find((c) => c[1] === "neutral");
+    expect(buildCall).toBeDefined();
   });
 
   it("build에서 lint 성공 시 에러 출력 없음", async () => {
@@ -1161,8 +1152,7 @@ describe("Feature 2.1: build에서 lint 에러가 'lint' 라벨로 출력", () =
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
         success: true,
-        js: { success: true, errors: [], warnings: [] },
-        dts: { success: true, errors: [], warnings: [], diagnostics: [] },
+        build: { success: true, errors: [], warnings: [], diagnostics: [] },
         lint: { success: true, errorCount: 0, warningCount: 0, formattedOutput: "" },
       }),
       startWatch: vi.fn(),