@simplysm/sd-cli 14.0.63 → 14.0.65

package/tests/utils/external-modules.spec.ts

@@ -1,217 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import path from "path";
-
-// Mock fs (readFileSync, existsSync)
-const mockReadFileSync = vi.fn();
-const mockExistsSync = vi.fn();
-
-vi.mock("fs", () => ({
-  readFileSync: (...args: any[]) => mockReadFileSync(...args),
-  existsSync: (...args: any[]) => mockExistsSync(...args),
-}));
-
-// Mock createRequire to control module resolution
-const mockResolve = vi.fn();
-vi.mock("module", () => ({
-  createRequire: vi.fn(() => ({
-    resolve: mockResolve,
-  })),
-}));
-
-const { collectUninstalledOptionalPeerDeps, collectNativeModuleExternals, collectAllDependencyExternals } =
-  await import("../../src/esbuild/esbuild-config");
-
-describe("collectUninstalledOptionalPeerDeps", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it("detects uninstalled optional peer dependencies", () => {
-    // Root package.json with one dependency
-    mockReadFileSync.mockImplementation((filePath: string) => {
-      if (filePath === path.join("/pkg", "package.json")) {
-        return JSON.stringify({ dependencies: { "some-lib": "^1.0.0" } });
-      }
-      // some-lib's package.json with optional peer dep
-      return JSON.stringify({
-        peerDependencies: { "optional-peer": "^2.0.0" },
-        peerDependenciesMeta: { "optional-peer": { optional: true } },
-      });
-    });
-
-    // Resolving some-lib/package.json succeeds
-    mockResolve.mockImplementation((name: string) => {
-      if (name === "some-lib/package.json") {
-        return "/node_modules/some-lib/package.json";
-      }
-      // optional-peer cannot be resolved (not installed)
-      throw new Error("MODULE_NOT_FOUND");
-    });
-
-    const result = collectUninstalledOptionalPeerDeps("/pkg");
-    expect(result).toContain("optional-peer");
-  });
-
-  it("does not include installed optional peer dependencies", () => {
-    mockReadFileSync.mockImplementation((filePath: string) => {
-      if (filePath === path.join("/pkg", "package.json")) {
-        return JSON.stringify({ dependencies: { "some-lib": "^1.0.0" } });
-      }
-      return JSON.stringify({
-        peerDependencies: { "installed-peer": "^2.0.0" },
-        peerDependenciesMeta: { "installed-peer": { optional: true } },
-      });
-    });
-
-    mockResolve.mockImplementation((name: string) => {
-      if (name === "some-lib/package.json") {
-        return "/node_modules/some-lib/package.json";
-      }
-      // installed-peer resolves successfully
-      if (name === "installed-peer") {
-        return "/node_modules/installed-peer/index.js";
-      }
-      throw new Error("MODULE_NOT_FOUND");
-    });
-
-    const result = collectUninstalledOptionalPeerDeps("/pkg");
-    expect(result).not.toContain("installed-peer");
-  });
-
-  it("returns empty array when no optional peer deps exist", () => {
-    mockReadFileSync.mockImplementation((filePath: string) => {
-      if (filePath === path.join("/pkg", "package.json")) {
-        return JSON.stringify({ dependencies: { "some-lib": "^1.0.0" } });
-      }
-      return JSON.stringify({ dependencies: {} });
-    });
-
-    mockResolve.mockImplementation((name: string) => {
-      if (name === "some-lib/package.json") {
-        return "/node_modules/some-lib/package.json";
-      }
-      throw new Error("MODULE_NOT_FOUND");
-    });
-
-    const result = collectUninstalledOptionalPeerDeps("/pkg");
-    expect(result).toEqual([]);
-  });
-});
-
-describe("collectNativeModuleExternals", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it("detects native modules with binding.gyp", () => {
-    mockReadFileSync.mockImplementation((filePath: string) => {
-      if (filePath === path.join("/pkg", "package.json")) {
-        return JSON.stringify({ dependencies: { bcrypt: "^5.0.0" } });
-      }
-      return JSON.stringify({ dependencies: {} });
-    });
-
-    mockResolve.mockImplementation((name: string) => {
-      if (name === "bcrypt/package.json") {
-        return "/node_modules/bcrypt/package.json";
-      }
-      throw new Error("MODULE_NOT_FOUND");
-    });
-
-    mockExistsSync.mockImplementation((filePath: string) => {
-      return filePath === path.join("/node_modules/bcrypt", "binding.gyp");
-    });
-
-    const result = collectNativeModuleExternals("/pkg");
-    expect(result).toContain("bcrypt");
-  });
-
-  it("does not include non-native modules", () => {
-    mockReadFileSync.mockImplementation((filePath: string) => {
-      if (filePath === path.join("/pkg", "package.json")) {
-        return JSON.stringify({ dependencies: { lodash: "^4.0.0" } });
-      }
-      return JSON.stringify({ dependencies: {} });
-    });
-
-    mockResolve.mockImplementation((name: string) => {
-      if (name === "lodash/package.json") {
-        return "/node_modules/lodash/package.json";
-      }
-      throw new Error("MODULE_NOT_FOUND");
-    });
-
-    mockExistsSync.mockReturnValue(false);
-
-    const result = collectNativeModuleExternals("/pkg");
-    expect(result).not.toContain("lodash");
-    expect(result).toEqual([]);
-  });
-});
-
-describe("collectAllDependencyExternals", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it("collects both optional peer deps and native modules in a single pass", () => {
-    mockReadFileSync.mockImplementation((filePath: string) => {
-      if (filePath === path.join("/pkg", "package.json")) {
-        return JSON.stringify({
-          dependencies: { "native-lib": "^1.0.0", "peer-lib": "^2.0.0" },
-        });
-      }
-      if (filePath === "/node_modules/native-lib/package.json") {
-        return JSON.stringify({ dependencies: {} });
-      }
-      if (filePath === "/node_modules/peer-lib/package.json") {
-        return JSON.stringify({
-          peerDependencies: { "opt-peer": "^3.0.0" },
-          peerDependenciesMeta: { "opt-peer": { optional: true } },
-        });
-      }
-      return JSON.stringify({});
-    });
-
-    mockResolve.mockImplementation((name: string) => {
-      if (name === "native-lib/package.json") {
-        return "/node_modules/native-lib/package.json";
-      }
-      if (name === "peer-lib/package.json") {
-        return "/node_modules/peer-lib/package.json";
-      }
-      // opt-peer not installed
-      throw new Error("MODULE_NOT_FOUND");
-    });
-
-    mockExistsSync.mockImplementation((filePath: string) => {
-      return filePath === path.join("/node_modules/native-lib", "binding.gyp");
-    });
-
-    const result = collectAllDependencyExternals("/pkg");
-    expect(result.optionalPeerDeps).toContain("opt-peer");
-    expect(result.nativeModules).toContain("native-lib");
-  });
-
-  it("returns empty arrays when no externals are found", () => {
-    mockReadFileSync.mockImplementation((filePath: string) => {
-      if (filePath === path.join("/pkg", "package.json")) {
-        return JSON.stringify({ dependencies: { "some-lib": "^1.0.0" } });
-      }
-      return JSON.stringify({ dependencies: {} });
-    });
-
-    mockResolve.mockImplementation((name: string) => {
-      if (name === "some-lib/package.json") {
-        return "/node_modules/some-lib/package.json";
-      }
-      throw new Error("MODULE_NOT_FOUND");
-    });
-
-    mockExistsSync.mockReturnValue(false);
-
-    const result = collectAllDependencyExternals("/pkg");
-    expect(result.optionalPeerDeps).toEqual([]);
-    expect(result.nativeModules).toEqual([]);
-  });
-});

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

@@ -1,201 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-
-// --- Mocks ---
-
-// SdTsCompiler mock (js=false path) — lint result is part of compileAsync result
-const mockLintResult = {
-  success: true,
-  errorCount: 0,
-  warningCount: 0,
-  formattedOutput: "",
-};
-
-const mockCompileAsync = vi.fn(() => Promise.resolve({
-  program: { getSourceFiles: () => [] },
-  builderProgram: {},
-  isForAngular: false,
-  affectedFiles: undefined,
-  diagnostics: [],
-  errorCount: 0,
-  warningCount: 0,
-  errors: undefined as string[] | undefined,
-  emitResults: undefined,
-  lint: undefined as typeof mockLintResult | undefined,
-  scssErrors: [],
-  scssDependencies: new Map(),
-}));
-
-const MockSdTsCompiler = vi.fn().mockImplementation(function () {
-  return { compileAsync: mockCompileAsync };
-});
-
-vi.mock("../../src/ts-compiler/SdTsCompiler", () => ({
-  SdTsCompiler: MockSdTsCompiler,
-}));
-
-// tsc plugin mock (build() js=true path)
-const mockTscPlugin = {
-  plugin: { name: "sd-tsc", setup: vi.fn() },
-  getProgram: vi.fn(),
-  getAffectedFiles: vi.fn(),
-  getDiagnostics: vi.fn((): unknown[] => []),
-  getErrors: vi.fn((): string[] | undefined => undefined),
-  getLintResult: vi.fn((): typeof mockLintResult | undefined => undefined),
-  resetBuilderProgram: vi.fn(),
-};
-
-vi.mock("../../src/esbuild/esbuild-tsc-plugin", () => ({
-  createTscPlugin: vi.fn(() => mockTscPlugin),
-}));
-
-vi.mock("../../src/utils/tsconfig", () => ({
-  parseTsconfig: vi.fn(() => ({ options: {}, fileNames: [] })),
-  getPackageSourceFiles: vi.fn(() => []),
-}));
-
-vi.mock("../../src/esbuild/esbuild-config", () => ({
-  createServerEsbuildOptions: vi.fn(() => ({})),
-  collectAllDependencyExternals: vi.fn(() => ({ optionalPeerDeps: [], nativeModules: [] })),
-  writeChangedOutputFiles: vi.fn(),
-}));
-
-vi.mock("esbuild", () => ({
-  default: {
-    build: vi.fn().mockResolvedValue({ errors: [], warnings: [], outputFiles: [] }),
-    context: vi.fn().mockResolvedValue({ rebuild: vi.fn(), dispose: vi.fn() }),
-  },
-}));
-
-vi.mock("../../src/runtime/worker-utils", () => ({
-  registerCleanupHandlers: vi.fn(),
-  createOnceGuard: vi.fn(() => vi.fn()),
-  setupWorkerConsola: vi.fn(),
-}));
-
-vi.mock("../../src/utils/copy-public", () => ({
-  copyPublicFiles: vi.fn(),
-  watchPublicFiles: vi.fn(),
-}));
-
-vi.mock("../../src/deps/replace-deps/collect-deps", () => ({
-  collectDeps: vi.fn(() => ({ workspaceDeps: [], replaceDeps: [] })),
-}));
-
-vi.mock("@simplysm/core-node", () => ({
-  createWorker: vi.fn(
-    (methods: Record<string, Function>) => {
-      Object.assign(workerMethods, methods);
-      return { send: vi.fn() };
-    },
-  ),
-  FsWatcher: { watch: vi.fn() },
-  pathx: {
-    posix: vi.fn((p: string) => p.replace(/\\/g, "/")),
-    posixResolve: vi.fn((...args: string[]) => args.join("/").replace(/\/+/g, "/").replace(/\\/g, "/")),
-  },
-}));
-
-vi.mock("fs", () => ({
-  default: {
-    existsSync: vi.fn(() => false),
-    readFileSync: vi.fn(() => '{"name":"test","version":"1.0.0","type":"module"}'),
-    writeFileSync: vi.fn(),
-    mkdirSync: vi.fn(),
-  },
-}));
-
-vi.mock("execa", () => ({
-  execaSync: vi.fn(() => ({ stdout: "v20.0.0" })),
-}));
-
-const workerMethods: Record<string, Function> = {};
-
-await import("../../src/workers/server-build.worker");
-
-// --- Tests ---
-
-beforeEach(() => {
-  vi.clearAllMocks();
-  mockCompileAsync.mockResolvedValue({
-    program: { getSourceFiles: () => [] },
-    builderProgram: {},
-    isForAngular: false,
-    affectedFiles: undefined,
-    diagnostics: [],
-    errorCount: 0,
-    warningCount: 0,
-    errors: undefined,
-    emitResults: undefined,
-    lint: undefined,
-    scssErrors: [],
-    scssDependencies: new Map(),
-  });
-
-  // Reset tsc plugin mock
-  mockTscPlugin.getProgram.mockReset();
-  mockTscPlugin.getAffectedFiles.mockReset();
-  mockTscPlugin.getDiagnostics.mockReset().mockReturnValue([]);
-  mockTscPlugin.getErrors.mockReset().mockReturnValue(undefined);
-  mockTscPlugin.getLintResult.mockReset().mockReturnValue(undefined);
-  mockTscPlugin.resetBuilderProgram.mockReset();
-});
-
-describe("server-build.worker lint integration (Slice 3)", () => {
-  describe("Scenario: js=false lint via SdTsCompiler", () => {
-    it("returns lint result from SdTsCompiler when lint is enabled", async () => {
-      mockCompileAsync.mockResolvedValueOnce({
-        program: { getSourceFiles: () => [] },
-        builderProgram: {},
-        isForAngular: false,
-        affectedFiles: undefined,
-        diagnostics: [],
-        errorCount: 0,
-        warningCount: 0,
-        errors: undefined,
-        emitResults: undefined,
-        lint: mockLintResult,
-        scssErrors: [],
-        scssDependencies: new Map(),
-      });
-
-      const result = await workerMethods["build"]({
-        name: "my-server",
-        cwd: "/workspace",
-        pkgDir: "/workspace/packages/my-server",
-        output: { js: false, dts: false, lint: true },
-      });
-
-      expect(result).toHaveProperty("lint");
-      expect(result.lint).toEqual(mockLintResult);
-    });
-  });
-
-  describe("Scenario: js=true lint via tscPlugin.getLintResult()", () => {
-    it("returns lint result from tsc plugin when js=true and lint enabled", async () => {
-      mockTscPlugin.getLintResult.mockReturnValue(mockLintResult);
-
-      const result = await workerMethods["build"]({
-        name: "my-server",
-        cwd: "/workspace",
-        pkgDir: "/workspace/packages/my-server",
-        output: { js: true, dts: false, lint: true },
-      });
-
-      expect(result).toHaveProperty("lint");
-      expect(result.lint).toEqual(mockLintResult);
-    });
-  });
-
-  describe("Scenario: lint disabled", () => {
-    it("does not run lint when output.lint is not set", async () => {
-      const result = await workerMethods["build"]({
-        name: "my-server",
-        cwd: "/workspace",
-        pkgDir: "/workspace/packages/my-server",
-        output: { js: false, dts: false },
-      });
-
-      expect(result.lint).toBeUndefined();
-    });
-  });
-});