@simplysm/sd-cli 14.0.17 → 14.0.19

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

@@ -70,6 +70,17 @@ vi.mock("fs/promises", () => ({
   symlink: mockSymlink,
 }));
 
+vi.mock("module", () => ({
+  createRequire: () => ({
+    resolve: (id: string) => {
+      if (id.includes("capacitor-plugin-auto-update")) {
+        return path.resolve("/fake/workspace/packages/capacitor-plugin-auto-update/package.json");
+      }
+      throw new Error(`Cannot find module '${id}'`);
+    },
+  }),
+}));
+
 vi.mock("consola", () => ({
   consola: {
     withTag: () => ({ debug: vi.fn(), warn: vi.fn() }),
@@ -155,7 +166,7 @@ describe("workspace:* 플러그인 해석", () => {
     setupDefaultMocks();
   });
 
-  it("workspace:* 플러그인을 package.json에 추가하지 않고 symlink로 연결한다", async () => {
+  it("workspace:* 플러그인이 link: 프로토콜로 package.json에 등록된다", async () => {
     const { Capacitor } = await import("../../src/capacitor/capacitor.js");
 
     const cap = await Capacitor.create(PKG_PATH, {
@@ -170,7 +181,6 @@ describe("workspace:* 플러그인 해석", () => {
 
     await cap.initialize();
 
-    // 1. .capacitor/package.json에 workspace:* 플러그인이 포함되지 않아야 한다
     const writeJsonCalls = mockFsxWriteJson.mock.calls;
     const capPkgWrite = writeJsonCalls.find(
       (call) => typeof call[0] === "string" && call[0].includes("package.json"),
@@ -180,18 +190,18 @@ describe("workspace:* 플러그인 해석", () => {
       string,
       string
     >;
-    expect(deps["@simplysm/capacitor-plugin-auto-update"]).toBeUndefined();
 
-    // 2. 일반 npm 플러그인은 정상 추가되어야 한다
+    // 1. workspace:* 플러그인이 link: 프로토콜로 등록되어야 한다
+    expect(deps["@simplysm/capacitor-plugin-auto-update"]).toMatch(/^link:/);
+
+    // 2. link: 경로는 상대경로여야 한다 (절대경로가 아님)
+    const linkPath = deps["@simplysm/capacitor-plugin-auto-update"].replace(/^link:/, "");
+    expect(linkPath).not.toMatch(/^[A-Z]:|^\//i);
+
+    // 3. 일반 npm 플러그인은 버전 문자열로 등록되어야 한다
     expect(deps["@capacitor/camera"]).toBe("^7.0.0");
 
-    // 3. symlink가 생성되어야 한다
-    expect(mockSymlink).toHaveBeenCalled();
-    const symlinkCall = mockSymlink.mock.calls.find(
-      (call) =>
-        typeof call[1] === "string" &&
-        call[1].replace(/\\/g, "/").includes("@simplysm/capacitor-plugin-auto-update"),
-    );
-    expect(symlinkCall).toBeDefined();
+    // 4. 수동 symlink 생성이 호출되지 않아야 한다
+    expect(mockSymlink).not.toHaveBeenCalled();
   });
 });

package/tests/commands/device.spec.ts

@@ -166,19 +166,19 @@ 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 () => {
+  // Acceptance: Scenario "server가 string일 때 서버 패키지의 .dev-port에서 포트 읽기"
+  it("reads .dev-port from server package directory when server is a string", async () => {
     vi.mocked(loadSdConfig).mockResolvedValue({
       packages: {
         "client-devtool": {
           target: "client",
-          server: "server",
+          server: "my-server",
           electron: { appId: "com.test.electron" },
         },
       },
     });
 
-    mockReadFileSync.mockReturnValue("5173");
+    mockReadFileSync.mockReturnValue("3000");
 
     // HTTP 헬스체크 성공 mock
     mockHttpGet.mockImplementation((_url: string, cb: Function) => {
@@ -189,8 +189,13 @@ describe("runDevice", () => {
 
     await runDevice({ target: "client-devtool", options: [] });
 
+    // 서버 패키지 경로의 .dev-port를 읽어야 함
+    expect(mockReadFileSync).toHaveBeenCalledWith(
+      expect.stringContaining("my-server"),
+      "utf-8",
+    );
     expect(mockElectronInstance.run).toHaveBeenCalledWith(
-      "http://localhost:5173/client-devtool/",
+      "http://localhost:3000/client-devtool/",
     );
   });
 
@@ -200,7 +205,7 @@ describe("runDevice", () => {
       packages: {
         "client-devtool": {
           target: "client",
-          server: "server",
+          server: "my-server",
           electron: { appId: "com.test.electron" },
         },
       },
@@ -221,7 +226,7 @@ describe("runDevice", () => {
       packages: {
         "client-devtool": {
           target: "client",
-          server: "server",
+          server: "my-server",
           electron: { appId: "com.test.electron" },
         },
       },

package/tests/electron/electron.spec.ts

@@ -245,6 +245,15 @@ describe("Electron", () => {
       expect(deps["sharp"]).toBe("^0.34.0");
     });
 
+    it("package.json에 type: module이 설정된다", async () => {
+      const { Electron } = await import("../../src/electron/electron.js");
+
+      const electron = await Electron.create(PKG_PATH, { appId: "com.test.app" });
+      await electron.initialize();
+
+      expect(findElectronPackageJson()!["type"]).toBe("module");
+    });
+
     it("postInstallScript가 설정되면 scripts.postinstall에 포함한다", async () => {
       const { Electron } = await import("../../src/electron/electron.js");
 
@@ -277,7 +286,7 @@ describe("Electron", () => {
         expect.objectContaining({
           platform: "node",
           target: "node20",
-          format: "cjs",
+          format: "esm",
           bundle: true,
           external: expect.arrayContaining(["electron", "better-sqlite3"]),
         }),
@@ -314,6 +323,18 @@ describe("Electron", () => {
       expect(banner).not.toContain("ELECTRON_DEV_URL");
       expect(callArgs.define).toBeUndefined();
     });
+
+    it("ESM 배너에 createRequire shim이 포함된다", async () => {
+      const { Electron } = await import("../../src/electron/electron.js");
+
+      const electron = await Electron.create(PKG_PATH, { appId: "com.test.app" });
+      await electron.build("/fake/out");
+
+      const callArgs = mockEsbuildBuild.mock.calls[0][0];
+      const banner = callArgs.banner?.js as string;
+      expect(banner).toContain("createRequire");
+      expect(banner).toContain("import.meta.url");
+    });
   });
 
   //#endregion
@@ -503,9 +524,13 @@ describe("Electron", () => {
 
       expect(callArgs.platform).toBe("node");
       expect(callArgs.target).toBe("node20");
-      expect(callArgs.format).toBe("cjs");
+      expect(callArgs.format).toBe("esm");
       expect(callArgs.bundle).toBe(true);
       expect(callArgs.external).toContain("electron");
+
+      // ESM 배너에 createRequire shim 포함
+      expect(banner).toContain("createRequire");
+      expect(banner).toContain("import.meta.url");
     }, 10_000);
 
     it("throws when electron-main.ts entry point is missing", async () => {