@simplysm/sd-cli 13.0.68 → 13.0.70

package/src/utils/esbuild-config.ts

@@ -7,11 +7,11 @@ import { solidPlugin } from "esbuild-plugin-solid";
 import type { TypecheckEnv } from "./tsconfig";
 
 /**
- * esbuild outputFiles 중 실제로 변경된 파일만 디스크에 쓴다.
+ * Write only changed files from esbuild outputFiles to disk
  *
- * - .js 파일: ESM 상대 import 경로에 .js 확장자를 추가한 후 비교
- * - 그 외 파일(.js.map 등): 원본 그대로 비교
- * - 기존 파일과 내용이 동일하면 쓰기를 건너뛰어 타임스탬프를 유지한다.
+ * - .js files: Add .js extension to ESM relative import paths before comparing
+ * - Other files (.js.map etc): Compare original content as-is
+ * - Skip writing if content matches existing file to preserve timestamps
  */
 export async function writeChangedOutputFiles(outputFiles: esbuild.OutputFile[]): Promise<boolean> {
   let hasChanges = false;
@@ -43,9 +43,9 @@ export async function writeChangedOutputFiles(outputFiles: esbuild.OutputFile[])
 }
 
 /**
- * Library 빌드용 esbuild 옵션
- * - bundle: false (개별 파일 트랜스파일)
- * - platform: target에 따라 node 또는 browser
+ * esbuild options for Library build
+ * - bundle: false (transpile individual files)
+ * - platform: node or browser depending on target
  */
 export interface LibraryEsbuildOptions {
   pkgDir: string;
@@ -55,20 +55,20 @@ export interface LibraryEsbuildOptions {
 }
 
 /**
- * Server 빌드용 esbuild 옵션
- * - bundle: true (모든 의존성 포함한 단일 번들)
+ * esbuild options for Server build
+ * - bundle: true (single bundle with all dependencies)
  */
 export interface ServerEsbuildOptions {
   pkgDir: string;
   entryPoints: string[];
   compilerOptions: Record<string, unknown>;
   env?: Record<string, string>;
-  /** 번들에서 제외할 외부 모듈 */
+  /** External modules to exclude from bundle */
   external?: string[];
 }
 
 /**
- * package.json에서 solid-js 의존성 감지
+ * Detect solid-js dependency in package.json
  */
 function hasSolidDependency(pkgDir: string): boolean {
   const pkgJson = JSON.parse(readFileSync(path.join(pkgDir, "package.json"), "utf-8")) as PkgJson;
@@ -77,12 +77,12 @@ function hasSolidDependency(pkgDir: string): boolean {
 }
 
 /**
- * Library용 esbuild 설정 생성
+ * Create esbuild config for Library build
  *
- * node/browser/neutral 타겟의 라이브러리 패키지 빌드에 사용합니다.
- * - bundle: false (개별 파일을 각각 트랜스파일)
- * - platform: target이 node면 node, 그 외는 browser
- * - target: node면 node20, 그 외는 chrome84
+ * Used to build library packages with node/browser/neutral targets
+ * - bundle: false (transpile each file individually)
+ * - platform: node if target is node, otherwise browser
+ * - target: node20 if target is node, otherwise chrome84
  */
 export function createLibraryEsbuildOptions(options: LibraryEsbuildOptions): esbuild.BuildOptions {
   const plugins: esbuild.Plugin[] = [];
@@ -110,13 +110,13 @@ export function createLibraryEsbuildOptions(options: LibraryEsbuildOptions): esb
 }
 
 /**
- * Server용 esbuild 설정 생성
+ * Create esbuild config for Server build
  *
- * 서버 패키지 빌드에 사용합니다.
- * - bundle: true (모든 의존성 포함한 단일 번들)
- * - minify: true (코드 보호를 위한 압축)
- * - banner: CJS 패키지의 require() 지원을 위한 createRequire shim
- * - env를 define 옵션으로 치환 (process.env.KEY 형태)
+ * Used for server package builds
+ * - bundle: true (single bundle with all dependencies)
+ * - minify: true (minify for code protection)
+ * - banner: createRequire shim for CJS package require() support
+ * - Replace env with define option (process.env.KEY format)
  */
 export function createServerEsbuildOptions(options: ServerEsbuildOptions): esbuild.BuildOptions {
   const define: Record<string, string> = {};
@@ -147,10 +147,10 @@ export function createServerEsbuildOptions(options: ServerEsbuildOptions): esbui
 }
 
 /**
- * 빌드 타겟에서 TypecheckEnv 추출
+ * Extract TypecheckEnv from build target
  *
- * 빌드용이므로 neutral은 browser로 처리합니다.
- * (neutral 패키지는 Node/브라우저 공용이지만, 빌드 시에는 browser 환경 기준으로 처리)
+ * Neutral is treated as browser for builds.
+ * (neutral packages are Node/browser universal, but we treat as browser for build)
  */
 export function getTypecheckEnvFromTarget(target: "node" | "browser" | "neutral"): TypecheckEnv {
   return target === "node" ? "node" : "browser";
@@ -165,10 +165,10 @@ interface PkgJson {
 }
 
 /**
- * 의존성 트리에서 미설치 optional peer dep 수집
+ * Collect uninstalled optional peer deps from dependency tree
  *
- * 서버 빌드(bundle: true) 시 설치되지 않은 optional peer dependency를
- * esbuild external로 지정하여 빌드 실패를 방지한다.
+ * For server builds (bundle: true), specify uninstalled optional peer dependencies
+ * as esbuild externals to prevent build failures
  */
 export function collectUninstalledOptionalPeerDeps(pkgDir: string): string[] {
   const external = new Set<string>();
@@ -227,10 +227,10 @@ function scanOptionalPeerDeps(
 //#region Native Module Externals
 
 /**
- * 의존성 중 binding.gyp가 있는 네이티브 모듈 수집
+ * Collect native modules with binding.gyp from dependencies
  *
- * node-gyp로 빌드되는 네이티브 모듈은 esbuild가 번들링할 수 없으므로
- * external로 지정해야 한다.
+ * Native modules built with node-gyp cannot be bundled by esbuild,
+ * so they must be specified as externals.
  */
 export function collectNativeModuleExternals(pkgDir: string): string[] {
   const external = new Set<string>();
@@ -264,12 +264,12 @@ function scanNativeModules(
 
   const depDir = path.dirname(pkgJsonPath);
 
-  // binding.gyp 존재 여부로 네이티브 모듈 감지
+  // Detect native modules by checking for binding.gyp
   if (existsSync(path.join(depDir, "binding.gyp"))) {
     external.add(pkgName);
   }
 
-  // 하위 dependencies도 재귀 탐색
+  // Recursively traverse sub-dependencies
   const depPkgJson = JSON.parse(readFileSync(pkgJsonPath, "utf-8")) as PkgJson;
   for (const dep of Object.keys(depPkgJson.dependencies ?? {})) {
     scanNativeModules(dep, depDir, external, visited);

package/src/utils/output-utils.ts

@@ -2,7 +2,7 @@ import { consola } from "consola";
 import type { BuildResult } from "../infra/ResultCollector";
 
 /**
- * 빌드 경고/에러 메시지를 포맷팅한다.
+ * Format build warning/error messages
  */
 export function formatBuildMessages(name: string, label: string, messages: string[]): string {
   const lines: string[] = [`${name} (${label})`];
@@ -15,8 +15,8 @@ export function formatBuildMessages(name: string, label: string, messages: strin
 }
 
 /**
- * 에러만 출력한다.
- * @param results 패키지별 빌드 결과 상태
+ * Print only errors
+ * @param results Build result status per package
  */
 export function printErrors(results: Map<string, BuildResult>): void {
   for (const result of results.values()) {
@@ -32,34 +32,34 @@ export function printErrors(results: Map<string, BuildResult>): void {
 }
 
 /**
- * 서버 URL만 출력한다.
- * @param results 패키지별 빌드 결과 상태
- * @param serverClientsMap 서버별 연결된 클라이언트 목록
+ * Print server URLs only
+ * @param results Build result status per package
+ * @param serverClientsMap List of connected clients per server
  */
 export function printServers(
   results: Map<string, BuildResult>,
   serverClientsMap?: Map<string, string[]>,
 ): void {
-  // 서버 정보 수집
+  // Collect server info
   const servers = [...results.values()].filter((r) => r.status === "running" && r.port != null);
 
-  // 서버 정보 출력 (있으면 앞에 빈 줄 추가)
+  // Print server info (add blank line before if present)
   if (servers.length > 0) {
     process.stdout.write("\n");
     for (const server of servers) {
       if (server.target === "server") {
-        // 서버에 연결된 클라이언트가 있으면 클라이언트 URL만 출력
+        // If server has connected clients, print only client URLs
         const clients = serverClientsMap?.get(server.name) ?? [];
         if (clients.length > 0) {
           for (const clientName of clients) {
             consola.info(`[server] http://localhost:${server.port}/${clientName}/`);
           }
         } else {
-          // 연결된 클라이언트가 없으면 서버 루트 URL 출력
+          // If no connected clients, print server root URL
           consola.info(`[server] http://localhost:${server.port}/`);
         }
       } else {
-        // standalone client는 이름 포함해서 출력
+        // Standalone client: print with name included
         consola.info(`[server] http://localhost:${server.port}/${server.name}/`);
       }
     }

package/src/utils/package-utils.ts

@@ -3,13 +3,13 @@ import fs from "fs";
 import type { SdPackageConfig } from "../sd-config.types";
 
 /**
- * import.meta.dirname에서 상위로 올라가며 package.json을 찾아 패키지 루트를 반환한다.
+ * Walk up from import.meta.dirname to find package.json and return package root
  */
 export function findPackageRoot(startDir: string): string {
   let dir = startDir;
   while (!fs.existsSync(path.join(dir, "package.json"))) {
     const parent = path.dirname(dir);
-    if (parent === dir) throw new Error("package.json을 찾을 수 없습니다.");
+    if (parent === dir) throw new Error("package.json not found");
     dir = parent;
   }
   return dir;
@@ -55,7 +55,7 @@ export function collectDeps(
       if (visited.has(dep)) continue;
       visited.add(dep);
 
-      // workspace package check
+      // Check for workspace package
       if (workspaceScope != null && dep.startsWith(workspaceScope + "/")) {
         const dirName = dep.slice(workspaceScope.length + 1);
         const depDir = path.join(cwd, "packages", dirName);
@@ -66,7 +66,7 @@ export function collectDeps(
         }
       }
 
-      // replaceDeps pattern check
+      // Check replaceDeps pattern
       const matched = replaceDepsPatterns.find((p) => p.regex.test(dep));
       if (matched != null) {
         replaceDeps.push(dep);
@@ -84,11 +84,11 @@ export function collectDeps(
 }
 
 /**
- * 패키지 설정에서 targets 필터링 (scripts 타겟 제외)
- * @param packages 패키지 설정 맵
- * @param targets 필터링할 패키지 이름 목록. 빈 배열이면 scripts를 제외한 모든 패키지 반환
- * @returns 필터링된 패키지 설정 맵
- * @internal 테스트용으로 export
+ * Filter packages config by targets (excluding scripts target)
+ * @param packages Package config map
+ * @param targets List of package names to filter. If empty array, return all packages except scripts
+ * @returns Filtered package config map
+ * @internal exported for testing
  */
 export function filterPackagesByTargets(
   packages: Record<string, SdPackageConfig | undefined>,
@@ -99,16 +99,16 @@ export function filterPackagesByTargets(
   for (const [name, config] of Object.entries(packages)) {
     if (config == null) continue;
 
-    // scripts 타겟은 watch/dev 대상에서 제외
+    // Exclude scripts target from watch/dev targets
     if (config.target === "scripts") continue;
 
-    // targets가 비어있으면 모든 패키지 포함
+    // If targets is empty, include all packages
     if (targets.length === 0) {
       result[name] = config;
       continue;
     }
 
-    // targets에 포함된 패키지만 필터링
+    // Filter only packages included in targets
     if (targets.includes(name)) {
       result[name] = config;
     }

package/src/utils/rebuild-manager.ts

@@ -45,18 +45,18 @@ export class RebuildManager extends EventEmitter<RebuildManagerEvents> {
 
     const tasks = Array.from(batchBuilds.entries());
     const titles = tasks.map(([, { title }]) => title).join(", ");
-    this._logger.start(`리빌드 진행 중... (${titles})`);
+    this._logger.start(`Rebuilding... (${titles})`);
 
     const results = await Promise.allSettled(tasks.map(([, { promise }]) => promise));
 
     const failed = results.filter((r): r is PromiseRejectedResult => r.status === "rejected");
     if (failed.length > 0) {
       for (const result of failed) {
-        this._logger.error("리빌드 중 오류 발생", { error: String(result.reason) });
+        this._logger.error("Error during rebuild", { error: String(result.reason) });
       }
     }
 
-    this._logger.success(`리빌드 완료 (${titles})`);
+    this._logger.success(`Rebuild completed (${titles})`);
 
     this.emit("batchComplete");