@simplysm/sd-cli 14.0.16 → 14.0.18

package/tests/angular/vite-angular-plugin-scss-hmr.spec.ts

@@ -0,0 +1,87 @@
+import { describe, it, expect, beforeAll, afterAll } from "vitest";
+import path from "path";
+import { sdAngularPlugin } from "../../src/angular/vite-angular-plugin.js";
+
+const FIXTURE_DIR = path.resolve(import.meta.dirname, "fixtures/basic-app");
+const TSCONFIG_PATH = path.join(FIXTURE_DIR, "tsconfig.json");
+
+function mockEnvironmentContext() {
+  return {
+    environment: {
+      moduleGraph: {
+        getModulesByFile: (file: string) => {
+          return new Set([{ file, id: file }]);
+        },
+      },
+    },
+  };
+}
+
+describe("sdAngularPlugin SCSS @use HMR", () => {
+  let plugin: ReturnType<typeof sdAngularPlugin>;
+
+  beforeAll(async () => {
+    plugin = sdAngularPlugin({ tsconfig: TSCONFIG_PATH, dev: true });
+    await (plugin as any).buildStart?.call({});
+  });
+
+  afterAll(async () => {
+    await (plugin as any).buildEnd?.call({});
+  });
+
+  // Acceptance: inline styles의 직접 @use 의존성 변경 시 재컴파일
+  it("recompiles when inline SCSS @use dependency changes", async () => {
+    const variablesPath = path
+      .join(FIXTURE_DIR, "scss/_variables.scss")
+      .replace(/\\/g, "/");
+
+    const ctx = mockEnvironmentContext();
+    const result = await (plugin as any).hotUpdate?.call(ctx, {
+      file: variablesPath,
+      modules: [],
+      server: {},
+      timestamp: Date.now(),
+      read: () => Promise.resolve(""),
+    });
+
+    expect(result).toBeDefined();
+    expect(result!.length).toBeGreaterThan(0);
+  });
+
+  // Acceptance: inline styles의 간접 @use 의존성 변경 시 재컴파일 (체이닝)
+  it("recompiles when chained @use dependency changes", async () => {
+    const colorsPath = path
+      .join(FIXTURE_DIR, "scss/_colors.scss")
+      .replace(/\\/g, "/");
+
+    const ctx = mockEnvironmentContext();
+    const result = await (plugin as any).hotUpdate?.call(ctx, {
+      file: colorsPath,
+      modules: [],
+      server: {},
+      timestamp: Date.now(),
+      read: () => Promise.resolve(""),
+    });
+
+    expect(result).toBeDefined();
+    expect(result!.length).toBeGreaterThan(0);
+  });
+
+  // Acceptance: 무관한 SCSS 변경 시 재빌드하지 않음
+  it("does not recompile when unrelated SCSS changes", async () => {
+    const unrelatedPath = path
+      .join(FIXTURE_DIR, "scss/_unrelated.scss")
+      .replace(/\\/g, "/");
+
+    const ctx = mockEnvironmentContext();
+    const result = await (plugin as any).hotUpdate?.call(ctx, {
+      file: unrelatedPath,
+      modules: [],
+      server: {},
+      timestamp: Date.now(),
+      read: () => Promise.resolve(""),
+    });
+
+    expect(result).toBeUndefined();
+  });
+});

package/tests/angular/vite-angular-plugin.spec.ts

@@ -54,7 +54,7 @@ describe("sdAngularPlugin", () => {
   });
 
   // Scenario: Angular 컴포넌트 .ts 파일 수정 시 컴포넌트 HMR (Acceptance — Feature 3.3)
-  it("updates emit cache and returns affected modules when handleHotUpdate is called", async () => {
+  it("updates emit cache and returns affected modules when hotUpdate is called", async () => {
     const onBuildStart = vi.fn();
     const onBuild = vi.fn();
 
@@ -77,12 +77,12 @@ describe("sdAngularPlugin", () => {
     expect(initialResult).toBeDefined();
     expect(initialResult.code.length).toBeGreaterThan(0);
 
-    // handleHotUpdate must exist
-    expect((plugin as any).handleHotUpdate).toBeDefined();
+    // hotUpdate must exist
+    expect((plugin as any).hotUpdate).toBeDefined();
 
     // Simulate file change
     const mockModule = { file: appComponentPath, id: appComponentPath };
-    const hmrResult = await (plugin as any).handleHotUpdate?.({
+    const hmrResult = await (plugin as any).hotUpdate?.({
       file: appComponentPath,
       modules: [mockModule],
       server: { watcher: { emit: vi.fn() } },
@@ -122,7 +122,7 @@ describe("sdAngularPlugin", () => {
   });
 
   // Scenario: 컴파일 에러 발생 및 복구 (Acceptance — Feature 3.3)
-  it("calls onBuild with success=false when handleHotUpdate encounters compile error", async () => {
+  it("calls onBuild with success=false when hotUpdate encounters compile error", async () => {
     const onBuild = vi.fn();
     const plugin = sdAngularPlugin({
       tsconfig: TSCONFIG_PATH,
@@ -132,8 +132,8 @@ describe("sdAngularPlugin", () => {
 
     await (plugin as any).buildStart?.call({});
 
-    // handleHotUpdate with a non-existent file — facade.update() should handle gracefully
-    const _hmrResult = await (plugin as any).handleHotUpdate?.({
+    // hotUpdate with a non-existent file — facade.update() should handle gracefully
+    const _hmrResult = await (plugin as any).hotUpdate?.({
       file: path.join(FIXTURE_DIR, "src/nonexistent-file.ts").replace(/\\/g, "/"),
       modules: [],
       server: { watcher: { emit: vi.fn() } },
@@ -147,8 +147,8 @@ describe("sdAngularPlugin", () => {
     await (plugin as any).buildEnd?.call({});
   });
 
-  // Scenario: non-Angular .ts 파일 수정 — handleHotUpdate passes through
-  it("handleHotUpdate skips non-ts/html/scss files", async () => {
+  // Scenario: non-Angular .ts 파일 수정 — hotUpdate passes through
+  it("hotUpdate skips non-ts/html/scss files", async () => {
     const onBuildStart = vi.fn();
     const plugin = sdAngularPlugin({
       tsconfig: TSCONFIG_PATH,
@@ -159,7 +159,7 @@ describe("sdAngularPlugin", () => {
     await (plugin as any).buildStart?.call({});
 
     // .json file should be ignored
-    const result = await (plugin as any).handleHotUpdate?.({
+    const result = await (plugin as any).hotUpdate?.({
       file: "/some/file.json",
       modules: [],
       server: {},
@@ -192,16 +192,16 @@ describe("sdAngularPlugin", () => {
     // (in real use, Vite server close triggers this)
   });
 
-  // Scenario: optimizeDeps에 Angular Linker esbuild 플러그인이 등록된다 (Feature 1.1 Angular Linker)
-  it("registers angular-vite-optimize-deps esbuild plugin in optimizeDeps config", () => {
+  // Scenario: optimizeDeps에 Angular Linker Rolldown 플러그인이 등록된다
+  it("registers angular-vite-optimize-deps rolldown plugin in optimizeDeps config", () => {
     const plugin = sdAngularPlugin({ tsconfig: TSCONFIG_PATH, dev: true });
     const config = (plugin as any).config?.();
 
-    const esbuildPlugins = config?.optimizeDeps?.esbuildOptions?.plugins as
+    const rolldownPlugins = config?.optimizeDeps?.rolldownOptions?.plugins as
       | { name: string }[]
       | undefined;
-    expect(esbuildPlugins).toBeDefined();
-    expect(esbuildPlugins!.some((p) => p.name === "angular-vite-optimize-deps")).toBe(true);
+    expect(rolldownPlugins).toBeDefined();
+    expect(rolldownPlugins!.some((p) => p.name === "angular-vite-optimize-deps")).toBe(true);
   });
 
   // Scenario: .mjs 파일이 JavaScriptTransformer를 통과한다 (Feature 1.1 Angular Linker)

package/tests/capacitor/capacitor-icon.spec.ts

@@ -35,11 +35,9 @@ vi.mock("@simplysm/core-node", () => ({
 }));
 
 // env mock
-let mockEnv: Record<string, unknown> = {};
+let mockEnv: Record<string, string | undefined> = {};
 vi.mock("@simplysm/core-common", () => ({
-  env: new Proxy({} as Record<string, unknown>, {
-    get: (_target, prop) => mockEnv[prop as string],
-  }),
+  env: (key: string) => mockEnv[key],
 }));
 
 // cpx mock (was execa)

package/tests/capacitor/capacitor-init.spec.ts

@@ -35,11 +35,9 @@ vi.mock("@simplysm/core-node", () => ({
   },
 }));
 
-let mockEnv: Record<string, unknown> = {};
+let mockEnv: Record<string, string | undefined> = {};
 vi.mock("@simplysm/core-common", () => ({
-  env: new Proxy({} as Record<string, unknown>, {
-    get: (_target, prop) => mockEnv[prop as string],
-  }),
+  env: (key: string) => mockEnv[key],
 }));
 
 const execaCalls: { command: string; args: string[] }[] = [];

package/tests/capacitor/capacitor-workspace.spec.ts

@@ -35,11 +35,9 @@ vi.mock("@simplysm/core-node", () => ({
   },
 }));
 
-let mockEnv: Record<string, unknown> = {};
+let mockEnv: Record<string, string | undefined> = {};
 vi.mock("@simplysm/core-common", () => ({
-  env: new Proxy({} as Record<string, unknown>, {
-    get: (_target, prop) => mockEnv[prop as string],
-  }),
+  env: (key: string) => mockEnv[key],
 }));
 
 const execaCalls: { command: string; args: string[] }[] = [];

package/tests/commands/device.spec.ts

@@ -31,6 +31,27 @@ vi.mock("../../src/utils/sd-config", () => ({
   loadSdConfig: vi.fn(),
 }));
 
+// fs mock (포트 파일 읽기용)
+const mockReadFileSync = vi.fn();
+const mockExistsSync = vi.fn().mockReturnValue(false);
+vi.mock("node:fs", () => ({
+  default: {
+    readFileSync: (...args: any[]) => mockReadFileSync(...args),
+    existsSync: (...args: any[]) => mockExistsSync(...args),
+  },
+  readFileSync: (...args: any[]) => mockReadFileSync(...args),
+  existsSync: (...args: any[]) => mockExistsSync(...args),
+}));
+
+// http mock (헬스체크용)
+const mockHttpGet = vi.fn();
+vi.mock("node:http", () => ({
+  default: {
+    get: (...args: any[]) => mockHttpGet(...args),
+  },
+  get: (...args: any[]) => mockHttpGet(...args),
+}));
+
 const { Capacitor } = await import("../../src/capacitor/capacitor");
 const { Electron } = await import("../../src/electron/electron");
 const { loadSdConfig } = await import("../../src/utils/sd-config");
@@ -144,4 +165,83 @@ describe("runDevice", () => {
 
     await expect(runDevice({ target: "my-server", options: [] })).rejects.toThrow();
   });
+
+  // Acceptance: Scenario "dev 서버 실행 중 device 실행 시 URL 자동 생성"
+  it("auto-detects URL from .dev-port when server is a string", async () => {
+    vi.mocked(loadSdConfig).mockResolvedValue({
+      packages: {
+        "client-devtool": {
+          target: "client",
+          server: "server",
+          electron: { appId: "com.test.electron" },
+        },
+      },
+    });
+
+    mockReadFileSync.mockReturnValue("5173");
+
+    // HTTP 헬스체크 성공 mock
+    mockHttpGet.mockImplementation((_url: string, cb: Function) => {
+      const res = { resume: vi.fn() };
+      cb(res);
+      return { on: vi.fn(), setTimeout: vi.fn() };
+    });
+
+    await runDevice({ target: "client-devtool", options: [] });
+
+    expect(mockElectronInstance.run).toHaveBeenCalledWith(
+      "http://localhost:5173/client-devtool/",
+    );
+  });
+
+  // Acceptance: Scenario "dev 서버 미실행 시 에러"
+  it("throws when .dev-port file does not exist and server is a string", async () => {
+    vi.mocked(loadSdConfig).mockResolvedValue({
+      packages: {
+        "client-devtool": {
+          target: "client",
+          server: "server",
+          electron: { appId: "com.test.electron" },
+        },
+      },
+    });
+
+    mockReadFileSync.mockImplementation(() => {
+      throw new Error("ENOENT");
+    });
+
+    await expect(runDevice({ target: "client-devtool", options: [] })).rejects.toThrow(
+      "dev 서버가 실행 중이 아닙니다",
+    );
+  });
+
+  // Acceptance: Scenario "stale 포트 파일 존재 시 헬스체크 실패 에러"
+  it("throws when .dev-port exists but health check fails", async () => {
+    vi.mocked(loadSdConfig).mockResolvedValue({
+      packages: {
+        "client-devtool": {
+          target: "client",
+          server: "server",
+          electron: { appId: "com.test.electron" },
+        },
+      },
+    });
+
+    mockReadFileSync.mockReturnValue("5173");
+
+    // HTTP 헬스체크 실패 mock
+    mockHttpGet.mockImplementation((_url: string, _cb: Function) => {
+      const req = {
+        on: vi.fn((event: string, handler: Function) => {
+          if (event === "error") handler(new Error("ECONNREFUSED"));
+        }),
+        setTimeout: vi.fn(),
+      };
+      return req;
+    });
+
+    await expect(runDevice({ target: "client-devtool", options: [] })).rejects.toThrow(
+      "dev 서버가 응답하지 않습니다",
+    );
+  });
 });

package/tests/electron/electron.spec.ts

@@ -7,6 +7,7 @@ import { describe, it, expect, vi, beforeEach } from "vitest";
 const mockFsxExists = vi.fn();
 const mockFsxReadJson = vi.fn();
 const mockFsxWriteJson = vi.fn().mockResolvedValue(undefined);
+const mockFsxWrite = vi.fn().mockResolvedValue(undefined);
 const mockFsxMkdir = vi.fn().mockResolvedValue(undefined);
 const mockFsxCopy = vi.fn().mockResolvedValue(undefined);
 const mockFsxReaddir = vi.fn();
@@ -17,6 +18,7 @@ vi.mock("@simplysm/core-node", () => ({
     exists: mockFsxExists,
     readJson: mockFsxReadJson,
     writeJson: mockFsxWriteJson,
+    write: mockFsxWrite,
     mkdir: mockFsxMkdir,
     copy: mockFsxCopy,
     readdir: mockFsxReaddir,
@@ -96,6 +98,11 @@ function setupDefaultMocks() {
       "better-sqlite3": "^11.0.0",
       "sharp": "^0.34.0",
     },
+    devDependencies: {
+      "electron": "^35.0.0",
+      "@electron/rebuild": "^4.0.0",
+      "electron-builder": "^26.0.0",
+    },
   });
   mockFsxReaddir.mockResolvedValue(["index.html", "assets", "electron"]);
   // Default: glob returns one exe file matching the builder output
@@ -169,7 +176,7 @@ describe("Electron", () => {
   //#region Rule: Electron 프로젝트를 초기화한다
 
   describe("인수 테스트: 초기화", () => {
-    it("package.json 생성 + npm install + electron-rebuild를 실행한다", async () => {
+    it("package.json 생성 + pnpm install + electron-rebuild를 실행한다", async () => {
       const { Electron } = await import("../../src/electron/electron.js");
 
       const electron = await Electron.create(PKG_PATH, {
@@ -181,13 +188,13 @@ describe("Electron", () => {
 
       expect(findElectronPackageJson()).toBeDefined();
 
-      const execaCalls = mockCpxSpawn.mock.calls;
+      const spawnCalls = mockCpxSpawn.mock.calls;
       expect(
-        execaCalls.find((c) => c[0] === "npm" && (c[1] as string[]).includes("install")),
+        spawnCalls.find((c) => c[0] === "pnpm" && (c[1] as string[]).includes("install")),
       ).toBeDefined();
       expect(
-        execaCalls.find(
-          (c) => typeof c[0] === "string" && c[0].includes("electron-rebuild"),
+        spawnCalls.find(
+          (c) => c[0] === "pnpm" && (c[1] as string[]).includes("electron-rebuild"),
         ),
       ).toBeDefined();
     });
@@ -199,7 +206,7 @@ describe("Electron", () => {
       await electron.initialize();
 
       const rebuildCall = mockCpxSpawn.mock.calls.find(
-        (c) => typeof c[0] === "string" && c[0].includes("electron-rebuild"),
+        (c) => c[0] === "pnpm" && (c[1] as string[]).includes("electron-rebuild"),
       );
       expect(rebuildCall).toBeUndefined();
     });
@@ -455,9 +462,9 @@ describe("Electron", () => {
       const electronKill = vi.fn();
       let resolveElectron: () => void = () => {};
 
-      mockCpxSpawn.mockImplementation((cmd: string) => {
-        if (typeof cmd === "string" && cmd.includes("electron")) {
-          // Electron process: create a deferred promise we can resolve externally
+      mockCpxSpawn.mockImplementation((cmd: string, args: string[]) => {
+        // pnpm exec electron . → Electron 프로세스
+        if (cmd === "pnpm" && args[0] === "exec" && args[1] === "electron" && args[2] === ".") {
           const p = new Promise<void>((resolve) => {
             resolveElectron = resolve;
           }) as any;
@@ -537,8 +544,8 @@ describe("Electron", () => {
   describe("단위: run() 플러그인 동작", () => {
     it("passes custom env and ELECTRON_DEV_URL via esbuild banner", async () => {
       let resolveElectron: () => void = () => {};
-      mockCpxSpawn.mockImplementation((cmd: string) => {
-        if (typeof cmd === "string" && cmd.includes("electron")) {
+      mockCpxSpawn.mockImplementation((cmd: string, args: string[]) => {
+        if (cmd === "pnpm" && args[0] === "exec" && args[1] === "electron" && args[2] === ".") {
           const p = new Promise<void>((resolve) => {
             resolveElectron = resolve;
           }) as any;
@@ -570,8 +577,8 @@ describe("Electron", () => {
 
     it("calls initialize() before starting esbuild context", async () => {
       let resolveElectron: () => void = () => {};
-      mockCpxSpawn.mockImplementation((cmd: string) => {
-        if (typeof cmd === "string" && cmd.includes("electron")) {
+      mockCpxSpawn.mockImplementation((cmd: string, args: string[]) => {
+        if (cmd === "pnpm" && args[0] === "exec" && args[1] === "electron" && args[2] === ".") {
           const p = new Promise<void>((resolve) => {
             resolveElectron = resolve;
           }) as any;
@@ -589,11 +596,11 @@ describe("Electron", () => {
       resolveElectron();
       await runPromise;
 
-      // initialize calls npm install -> execa should have been called with npm install
-      const npmInstallCall = mockCpxSpawn.mock.calls.find(
-        (c: any[]) => c[0] === "npm" && (c[1] as string[]).includes("install"),
+      // initialize calls pnpm install
+      const pnpmInstallCall = mockCpxSpawn.mock.calls.find(
+        (c: any[]) => c[0] === "pnpm" && (c[1] as string[]).includes("install"),
       );
-      expect(npmInstallCall).toBeDefined();
+      expect(pnpmInstallCall).toBeDefined();
     }, 10_000);
   });
 

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

@@ -78,7 +78,6 @@ describe("NgtscEngine", () => {
           output: { js: true, dts: true },
         }),
       );
-      expect(result.success).toBe(true);
       expect(result.build.success).toBe(true);
       expect(result.build.diagnostics).toEqual([]);
       await engine.stop();
@@ -133,7 +132,6 @@ describe("NgtscEngine", () => {
       const engine = new NgtscEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: true });
 
-      expect(result.success).toBe(false);
       expect(result.build.success).toBe(false);
       expect(result.build.errors).toContain("TS2322: Type error");
       await engine.stop();
@@ -148,7 +146,6 @@ describe("NgtscEngine", () => {
       const engine = new NgtscEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: true });
 
-      expect(result.success).toBe(false);
       expect(result.build.success).toBe(false);
       expect(result.build.errors).toEqual(["ngtsc compilation error"]);
       await engine.stop();

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

@@ -89,7 +89,6 @@ describe("ServerEsbuildEngine", () => {
           packageManager: "mise",
         }),
       );
-      expect(result.success).toBe(true);
       expect(result.build.success).toBe(true);
       await engine.stop();
     });
@@ -104,7 +103,6 @@ describe("ServerEsbuildEngine", () => {
       const engine = new ServerEsbuildEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: false });
 
-      expect(result.success).toBe(false);
       expect(result.build.warnings).toEqual(["warn1"]);
       expect(result.build.success).toBe(false);
       expect(result.build.diagnostics).toEqual([{ code: 2345, category: 1 }]);
@@ -121,7 +119,6 @@ describe("ServerEsbuildEngine", () => {
       const engine = new ServerEsbuildEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: false });
 
-      expect(result.success).toBe(false);
       expect(result.build.success).toBe(false);
       expect(result.build.errors).toEqual(["esbuild error"]);
       await engine.stop();

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

@@ -75,7 +75,6 @@ describe("TscEngine", () => {
           output: { js: true, dts: true },
         }),
       );
-      expect(result.success).toBe(true);
       expect(result.build.success).toBe(true);
       await engine.stop();
     });
@@ -101,7 +100,7 @@ describe("TscEngine", () => {
       const engine = new TscEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: true });
 
-      expect(result.success).toBe(false);
+      expect(result.build.success).toBe(false);
       expect(result.build.errors).toEqual(["type error"]);
       expect(result.build.diagnostics).toHaveLength(1);
       await engine.stop();

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

@@ -74,7 +74,6 @@ describe("ViteEngine", () => {
           pkgDir: "/packages/my-client",
         }),
       );
-      expect(result.success).toBe(true);
       expect(result.build.success).toBe(true);
       expect(result.build.errors).toEqual([]);
       expect(result.build.diagnostics).toEqual([]);
@@ -91,7 +90,6 @@ describe("ViteEngine", () => {
       const engine = new ViteEngine({ cwd: "/root", pkg: createMockPkg() });
       const result = await engine.run({ js: true, dts: false });
 
-      expect(result.success).toBe(false);
       expect(result.build.success).toBe(false);
       expect(result.build.errors).toContain("TS2345: Argument of type...");
       await engine.stop();

package/tests/infra/signal-handler.spec.ts

@@ -1,23 +1,13 @@
-import { describe, it, expect } from "vitest";
+import { describe, it } from "vitest";
 import { SignalHandler } from "../../src/infra/SignalHandler";
 
 describe("SignalHandler", () => {
   it("resolves waitForTermination on requestTermination", async () => {
     const handler = new SignalHandler();
 
-    expect(handler.isTerminated()).toBe(false);
-
     handler.requestTermination();
 
     await handler.waitForTermination();
-    expect(handler.isTerminated()).toBe(true);
-  });
-
-  it("is not terminated initially", () => {
-    const handler = new SignalHandler();
-    expect(handler.isTerminated()).toBe(false);
-    // Clean up: prevent dangling signal listeners
-    handler.requestTermination();
   });
 
   it("handles double requestTermination gracefully", async () => {
@@ -27,6 +17,5 @@ describe("SignalHandler", () => {
     handler.requestTermination();
 
     await handler.waitForTermination();
-    expect(handler.isTerminated()).toBe(true);
   });
 });

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

@@ -66,7 +66,6 @@ vi.mock("../../src/engines/index", () => ({
   createBuildEngine: vi.fn(() => {
     const engine = {
       run: vi.fn().mockResolvedValue({
-        success: true,
         build: { success: true, errors: [], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn().mockResolvedValue(undefined),
@@ -159,7 +158,6 @@ beforeEach(() => {
   vi.mocked(createBuildEngine).mockImplementation(() => {
     const engine = {
       run: vi.fn().mockResolvedValue({
-        success: true,
         build: { success: true, errors: [], warnings: [], diagnostics: [] },
 
       }),
@@ -413,7 +411,6 @@ describe("BuildOrchestrator.start", () => {
     });
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
-        success: false,
         build: { success: false, errors: ["Module not found"], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),
@@ -603,7 +600,6 @@ describe("BuildOrchestrator client build", () => {
     });
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
-        success: true,
         build: { success: true, errors: [], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),
@@ -637,7 +633,6 @@ describe("BuildOrchestrator client build", () => {
     });
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
-        success: false,
         build: { success: false, errors: ["Template error"], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),
@@ -957,7 +952,6 @@ describe("BuildOrchestrator native build integration (Slice 1)", () => {
     });
     vi.mocked(createBuildEngine).mockReturnValue({
       run: vi.fn().mockResolvedValue({
-        success: false,
         build: { success: false, errors: ["Build error"], warnings: [], diagnostics: [] },
       }),
       startWatch: vi.fn(),