@simplysm/sd-cli 14.0.49 → 14.0.52

package/tests/esbuild/esbuild-tsc-plugin.acc.spec.ts

@@ -41,7 +41,7 @@ function setupPlugin(plugin: esbuild.Plugin) {
         outputFiles: [],
         metafile: { inputs: {}, outputs: {} },
         ...result,
-      } as esbuild.BuildResult)) ?? null;
+      })) ?? null;
     },
   };
 }
@@ -73,7 +73,7 @@ function createErrorCompileResult(): ISdTsCompilerResult {
   return {
     ...createSuccessCompileResult(),
     errors: ["TS2322: Type 'string' is not assignable to type 'number'"],
-    diagnostics: [{ category: 1, code: 2322, messageText: "Type mismatch" }] as any,
+    diagnostics: [{ category: 1, code: 2322, messageText: "Type mismatch" }],
     errorCount: 1,
   };
 }

package/tests/esbuild/esbuild-tsc-plugin.spec.ts

@@ -41,7 +41,7 @@ function setupPlugin(plugin: esbuild.Plugin) {
         outputFiles: [],
         metafile: { inputs: {}, outputs: {} },
         ...result,
-      } as esbuild.BuildResult)) ?? null;
+      })) ?? null;
     },
   };
 }
@@ -147,7 +147,7 @@ describe("createTscPlugin — Unit Tests", () => {
     it("env, includeTests 옵션을 SdTsCompiler에 전달한다", async () => {
       const result = createTscPlugin({
         ...baseOptions,
-        env: "node" as any,
+        env: "node",
         includeTests: true,
       });
       const lifecycle = setupPlugin(result.plugin);
@@ -172,7 +172,7 @@ describe("createTscPlugin — Unit Tests", () => {
       ];
       mockCompileAsync.mockResolvedValue({
         ...createSuccessCompileResult(),
-        diagnostics: diagnostics as any,
+        diagnostics: diagnostics,
       });
 
       const result = createTscPlugin(baseOptions);

package/tests/esbuild/esbuild-worker-plugin.spec.ts

@@ -44,7 +44,7 @@ function createTrackedBuild(overrides?: Partial<esbuild.BuildOptions>): {
       transformSyncCalls.push(args);
       return esbuild.transformSync(...args);
     },
-  } as typeof esbuild;
+  };
 
   const build = {
     esbuild: trackedEsbuild,

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

@@ -129,7 +129,7 @@ function createMockWorkerProxy(overrides: Partial<MockWorkerProxy> = {}): MockWo
 }
 
 function setupDefaults(config: Partial<SdConfig> = {}): void {
-  vi.mocked(loadSdConfig).mockResolvedValue({ packages: {}, ...config } as SdConfig);
+  vi.mocked(loadSdConfig).mockResolvedValue({ packages: {}, ...config });
   vi.mocked(getVersion).mockResolvedValue("1.0.0");
 }
 
@@ -372,7 +372,7 @@ describe("BuildOrchestrator.start", () => {
       run: vi.fn().mockRejectedValue(new Error("build failed")),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
-    } as any);
+    });
     const mockProxy = createMockWorkerProxy();
     vi.mocked(Worker.create).mockReturnValue(mockProxy as any);
 
@@ -413,7 +413,7 @@ describe("BuildOrchestrator.start", () => {
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
-    } as any);
+    });
     const mockProxy = createMockWorkerProxy();
     vi.mocked(Worker.create).mockReturnValue(mockProxy as any);
 
@@ -466,7 +466,7 @@ describe("BuildOrchestrator.start", () => {
         stop: vi.fn().mockResolvedValue(undefined),
       };
       mockEngines.push(engine);
-      return engine as any;
+      return engine;
     });
     const mockProxy = createMockWorkerProxy();
     vi.mocked(Worker.create).mockReturnValue(mockProxy as any);
@@ -496,7 +496,7 @@ describe("BuildOrchestrator.start", () => {
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
-    } as any);
+    });
     const mockProxy = createMockWorkerProxy();
     vi.mocked(Worker.create).mockReturnValue(mockProxy as any);
 
@@ -602,7 +602,7 @@ describe("BuildOrchestrator client build", () => {
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
-    } as any);
+    });
     const mockProxy = createMockWorkerProxy();
     vi.mocked(Worker.create).mockReturnValue(mockProxy as any);
 
@@ -635,7 +635,7 @@ describe("BuildOrchestrator client build", () => {
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
-    } as any);
+    });
     const mockProxy = createMockWorkerProxy();
     vi.mocked(Worker.create).mockReturnValue(mockProxy as any);
 
@@ -720,7 +720,7 @@ describe("BuildOrchestrator client build", () => {
       run: vi.fn().mockRejectedValue(new Error("build crashed")),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
-    } as any);
+    });
     const mockProxy = createMockWorkerProxy();
     vi.mocked(Worker.create).mockReturnValue(mockProxy as any);
 
@@ -954,7 +954,7 @@ describe("BuildOrchestrator native build integration (Slice 1)", () => {
       }),
       startWatch: vi.fn(),
       stop: vi.fn().mockResolvedValue(undefined),
-    } as any);
+    });
     const mockProxy = createMockWorkerProxy();
     vi.mocked(Worker.create).mockReturnValue(mockProxy as any);
 

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

@@ -187,7 +187,7 @@ function setupEngineWithResult(status: "success" | "error" = "success"): void {
       _pkgName: pkg.name,
     };
     mockBuildEngines.push(engine);
-    return engine as any;
+    return engine;
   });
 }
 
@@ -275,7 +275,7 @@ describe("DevOrchestrator", () => {
         _pkgName: pkg.name,
       };
       mockBuildEngines.push(engine);
-      return engine as any;
+      return engine;
     });
   });
 
@@ -471,13 +471,13 @@ describe("DevOrchestrator", () => {
   // --- Unit: disposes replaceDeps on shutdown ---
   it("disposes replaceDeps watcher on dev shutdown", async () => {
     const mockDispose = vi.fn();
-    vi.mocked(watchReplaceDeps).mockResolvedValue({ entries: [], dispose: mockDispose } as any);
+    vi.mocked(watchReplaceDeps).mockResolvedValue({ entries: [], dispose: mockDispose });
 
     setupDefaults(createConfig({
       packages: { "service-server": { target: "server" } },
       replaceDeps: { "@simplysm/*": "packages/*/src" },
     }));
-    vi.mocked(watchReplaceDeps).mockResolvedValue({ entries: [], dispose: mockDispose } as any);
+    vi.mocked(watchReplaceDeps).mockResolvedValue({ entries: [], dispose: mockDispose });
     setupEngineWithResult("success");
 
     const orchestrator = new DevOrchestrator({ targets: [], options: [] });

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

@@ -156,7 +156,7 @@ describe("WatchOrchestrator", () => {
         _pkgName: pkg.name,
       };
       mockBuildEngines.push(engine);
-      return engine as any;
+      return engine;
     });
   });
 
@@ -494,7 +494,7 @@ describe("WatchOrchestrator", () => {
 
   it("disposes replaceDepWatcher even when initialize fails after watchReplaceDeps", async () => {
     const mockDispose = vi.fn();
-    vi.mocked(watchReplaceDeps).mockResolvedValue({ entries: [], dispose: mockDispose } as any);
+    vi.mocked(watchReplaceDeps).mockResolvedValue({ entries: [], dispose: mockDispose });
 
     // loadSdConfig succeeds but we'll make classifyWatchPackages fail
     // by having watchReplaceDeps succeed first, then causing a later failure

package/tests/ts-compiler/SdTsCompiler-crash-handling.verify.md

@@ -0,0 +1,24 @@
+# SdTsCompiler 크래시 핸들링 — LLM 검증
+
+## 검증 항목
+
+### 구조적 검증
+
+- 단계별 try-catch: compileAsync 내 analyzeAsync, findAffectedFiles, emit, collectDiagnostics, lintAndGlobalScss 각각이 개별 try-catch로 감싸져 있는지 확인
+- 바깥 try-catch 유지: 단계별 try-catch를 감싸는 최종 안전망 try-catch가 존재하는지 확인
+- SdError 사용: catch 블록에서 SdError(원본에러, context)로 감싸는지 확인
+- crashDiagnostics 누적: 각 단계 catch에서 SerializedDiagnostic을 배열에 누적하고, 최종 결과에 합산하는지 확인
+- per-file 프로브 제거: _probeCrashPerFileAngular, _probeCrashPerFileTsc 메서드가 삭제되었는지 확인
+
+### 파일 추적 검증
+
+- _findAffectedFilesForTsc: ignoreSourceFile 콜백으로 _setCrashContext("findAffectedFiles", sourceFile.fileName)를 호출하는지 확인
+- _findAffectedFilesForAngular: 기존 콜백에 _setCrashContext("findAffectedFiles", sourceFile.fileName) 추가, 기존 ngtypecheck 필터링 로직 유지 확인
+- _collectDiagnosticsForTsc: 파일 단위 루프로 분리되어 _setCrashContext("collectDiagnostics", sourceFile.fileName)를 호출하는지 확인
+- _collectDiagnosticsForAngular: 기존 파일 추적 유지 확인
+
+### 단계 간 의존성 검증
+
+- analyzeAsync 크래시 시: _ngtscProgram 미설정 → Angular emit/diagnostics에서 null 체크로 스킵하는지 확인
+- findAffectedFiles 크래시 시: affectedFiles=undefined(전체), _affectedSourceFiles 빈 Set → emit/diagnostics가 전체 대상으로 동작하는지 확인
+- _createCrashDiagnostic: SdError cause chain으로 원본 에러 메시지+스택이 보존되는지 확인

package/tests/utils/copy-src.spec.ts

@@ -35,8 +35,8 @@ const distDir = toPosix(path.join(pkgDir, "dist"));
 describe("copySrcFiles", () => {
   beforeEach(() => {
     vi.clearAllMocks();
-    vi.mocked(fsx.mkdir).mockResolvedValue(undefined as any);
-    vi.mocked(fsx.copy).mockResolvedValue(undefined as any);
+    vi.mocked(fsx.mkdir).mockResolvedValue(undefined);
+    vi.mocked(fsx.copy).mockResolvedValue(undefined);
   });
 
   it("copies files matching glob patterns preserving relative paths", async () => {
@@ -84,9 +84,9 @@ describe("watchCopySrcFiles", () => {
   beforeEach(() => {
     vi.clearAllMocks();
     vi.mocked(fsx.glob).mockResolvedValue([]);
-    vi.mocked(fsx.mkdir).mockResolvedValue(undefined as any);
-    vi.mocked(fsx.copy).mockResolvedValue(undefined as any);
-    vi.mocked(fsx.rm).mockResolvedValue(undefined as any);
+    vi.mocked(fsx.mkdir).mockResolvedValue(undefined);
+    vi.mocked(fsx.copy).mockResolvedValue(undefined);
+    vi.mocked(fsx.rm).mockResolvedValue(undefined);
   });
 
   it("performs initial copy then starts watch", async () => {

package/tests/utils/esbuild-client-config.acc.spec.ts

@@ -423,7 +423,7 @@ describe("createClientEsbuildContext — Acceptance", () => {
       pkgDir: "/workspace/packages/my-app",
       cwd: "/workspace",
       mode: "dev",
-      plugins: [customPlugin as any],
+      plugins: [customPlugin],
       onEnd: vi.fn(),
     });
 

package/tests/utils/esbuild-client-config.spec.ts

@@ -215,7 +215,7 @@ describe("createClientEsbuildContext — PostCSS 플러그인 통합", () => {
     await createClientEsbuildContext({
       ...baseBuild,
       postcssPlugins: [["autoprefixer"]],
-      plugins: [customPlugin as any],
+      plugins: [customPlugin],
       legacyModule: true,
       onEnd: vi.fn(),
     });
@@ -294,7 +294,7 @@ describe("createClientEsbuildContext — 추가 옵션", () => {
     const customPlugin = { name: "custom", setup: vi.fn() };
     await createClientEsbuildContext({
       ...baseDev,
-      plugins: [customPlugin as any],
+      plugins: [customPlugin],
     });
     const opts = vi.mocked(esbuild.context).mock.calls[0][0];
     expect(opts.plugins).toContainEqual(customPlugin);
@@ -460,7 +460,7 @@ describe("createClientEsbuildContext — onEnd 플러그인", () => {
   it("customPlugins가 angularPlugin 이전에 위치 (onStart에서 sourceFileCache 무효화 선행)", async () => {
     await createClientEsbuildContext({
       ...baseDev,
-      plugins: [{ name: "custom", setup: vi.fn() } as any],
+      plugins: [{ name: "custom", setup: vi.fn() }],
       onEnd: vi.fn(),
     });
     const opts = vi.mocked(esbuild.context).mock.calls[0][0];
@@ -506,7 +506,7 @@ describe("createClientEsbuildContext — SCSS 플러그인 통합", () => {
     const customPlugin = { name: "custom", setup: vi.fn() };
     await createClientEsbuildContext({
       ...baseDev,
-      plugins: [customPlugin as any],
+      plugins: [customPlugin],
     });
     const opts = vi.mocked(esbuild.context).mock.calls[0][0];
     const pluginNames = opts.plugins!.map((p: any) => p.name);

package/tests/utils/esbuild-config.spec.ts

@@ -123,7 +123,7 @@ describe("createEnvBanner", () => {
 describe("writeChangedOutputFiles", () => {
   beforeEach(() => {
     vi.clearAllMocks();
-    vi.mocked(mockFs.mkdir).mockResolvedValue(undefined as any);
+    vi.mocked(mockFs.mkdir).mockResolvedValue(undefined);
     vi.mocked(mockFs.writeFile).mockResolvedValue();
   });
 
@@ -160,7 +160,7 @@ describe("writeChangedOutputFiles", () => {
   });
 
   it("skips writing when transformed content matches existing file", async () => {
-    vi.mocked(mockFs.readFile).mockResolvedValue('import { bar } from "./bar.js";' as any);
+    vi.mocked(mockFs.readFile).mockResolvedValue('import { bar } from "./bar.js";');
 
     await writeChangedOutputFiles([
       {
@@ -173,7 +173,7 @@ describe("writeChangedOutputFiles", () => {
   });
 
   it("writes file when content changed", async () => {
-    vi.mocked(mockFs.readFile).mockResolvedValue('import { old } from "./old.js";' as any);
+    vi.mocked(mockFs.readFile).mockResolvedValue('import { old } from "./old.js";');
 
     await writeChangedOutputFiles([
       {

package/tests/utils/esbuild-postcss-plugin.spec.ts

@@ -86,7 +86,7 @@ describe("createPostcssPlugin — metafile 가드", () => {
       metafile: undefined,
       outputFiles: [],
       mangleCache: {},
-    } as unknown as esbuild.BuildResult);
+    });
 
     expect(fs.readFileSync(cssFile, "utf-8")).toBe(original);
   });

package/tests/utils/hmr-client-script.acc.spec.ts

@@ -64,7 +64,7 @@ describe("HMR 클라이언트 스크립트 통합", () => {
         globalThis: {} as Record<string, unknown>,
         JSON,
       };
-      sandbox.globalThis = sandbox as Record<string, unknown>;
+      sandbox.globalThis = sandbox;
 
       return {
         sandbox,

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

@@ -8,7 +8,7 @@ const mockCollectDiagnostics = vi.fn().mockReturnValue([]);
 const mockEmitAffectedFiles = vi.fn().mockReturnValue([]);
 const mockGetTsProgram = vi.fn().mockReturnValue({
   getSourceFiles: () => [],
-} as unknown as ts.Program);
+});
 
 const angularCompilerConstructorSpy = vi.fn();
 

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

@@ -97,8 +97,8 @@ describe("mergeTestsPackagesIntoConfig", () => {
   // Acceptance: targets 없이 watch 실행 시 tests 패키지가 포함된다
   it("merges tests packages into config packages with target node", () => {
     const configPackages: Record<string, SdPackageConfig | undefined> = {
-      "core-common": { target: "neutral" } as SdBuildPackageConfig,
-      "core-node": { target: "node" } as SdBuildPackageConfig,
+      "core-common": { target: "neutral" },
+      "core-node": { target: "node" },
     };
     const workspacePackages = new Map([
       ["core-common", "packages/core-common"],
@@ -124,7 +124,7 @@ describe("mergeTestsPackagesIntoConfig", () => {
   // Acceptance: tests 패키지를 target으로 지정하여 watch 실행 (validateTargets에서 통합 맵 사용)
   it("makes tests packages available for validateTargets", () => {
     const configPackages: Record<string, SdPackageConfig | undefined> = {
-      "core-common": { target: "neutral" } as SdBuildPackageConfig,
+      "core-common": { target: "neutral" },
     };
     const workspacePackages = new Map([
       ["core-common", "packages/core-common"],
@@ -140,7 +140,7 @@ describe("mergeTestsPackagesIntoConfig", () => {
   // Unit: packages/ entries in workspacePackages are not added to merged (already in config)
   it("does not duplicate packages/ entries from workspacePackages", () => {
     const configPackages: Record<string, SdPackageConfig | undefined> = {
-      "core-common": { target: "neutral" } as SdBuildPackageConfig,
+      "core-common": { target: "neutral" },
     };
     const workspacePackages = new Map([
       ["core-common", "packages/core-common"],
@@ -156,7 +156,7 @@ describe("mergeTestsPackagesIntoConfig", () => {
   // Unit: empty workspacePackages returns config as-is
   it("returns config unchanged when no workspace packages", () => {
     const configPackages: Record<string, SdPackageConfig | undefined> = {
-      "core-common": { target: "neutral" } as SdBuildPackageConfig,
+      "core-common": { target: "neutral" },
     };
     const workspacePackages = new Map<string, string>();
 
@@ -169,7 +169,7 @@ describe("mergeTestsPackagesIntoConfig", () => {
   // Acceptance: 이름 충돌 시 에러 발생
   it("throws when config package name collides with tests package name", () => {
     const configPackages: Record<string, SdPackageConfig | undefined> = {
-      "orm": { target: "node" } as SdBuildPackageConfig,
+      "orm": { target: "node" },
     };
     const workspacePackages = new Map([
       ["orm", "tests/orm"],

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

@@ -317,7 +317,7 @@ describe("server-build.worker build()", () => {
       builderProgram: {},
       isForAngular: false,
       affectedFiles: undefined,
-      diagnostics: [{ code: 2345, category: 1 }] as any,
+      diagnostics: [{ code: 2345, category: 1 }],
       errorCount: 1,
       warningCount: 0,
       errors: ["TS2345: type error"],