@simplysm/sd-cli 14.0.10 → 14.0.12

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
 }
 
 /**
- * Print only errors
- * @param results Build result status per package
+ * 에러만 출력한다.
+ * @param results 패키지별 빌드 결과 상태
  */
 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 {
 }
 
 /**
- * Print server URLs only
- * @param results Build result status per package
- * @param serverClientsMap List of connected clients per server
+ * 서버 URL만 출력한다.
+ * @param results 패키지별 빌드 결과 상태
+ * @param serverClientsMap 서버별 연결된 클라이언트 목록
  */
 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") {
-        // If server has connected clients, print only client URLs
+        // 서버에 연결된 클라이언트가 있으면 클라이언트 URL만 출력
         const clients = serverClientsMap?.get(server.name) ?? [];
         if (clients.length > 0) {
           for (const clientName of clients) {
             consola.info(`[server] http://localhost:${server.port}/${clientName}/`);
           }
         } else {
-          // If no connected clients, print server root URL
+          // 연결된 클라이언트가 없으면 서버 루트 URL 출력
           consola.info(`[server] http://localhost:${server.port}/`);
         }
       } else {
-        // Standalone client: print with name included
+        // 독립형 클라이언트: 이름 포함하여 출력
         consola.info(`[server] http://localhost:${server.port}/${server.name}/`);
       }
     }

package/src/utils/package-utils.ts

@@ -33,7 +33,7 @@ export function iteratePackages(
 }
 
 /**
- * Walk up from import.meta.dirname to find package.json and return package root
+ * import.meta.dirname에서 위로 탐색하여 package.json을 찾고 패키지 루트를 반환한다.
  */
 export function findPackageRoot(startDir: string): string {
   let dir = startDir;
@@ -46,8 +46,8 @@ export function findPackageRoot(startDir: string): string {
 }
 
 /**
- * Discover all workspace packages from packages/ and tests/ directories.
- * Returns a map of directory name → relative path (e.g., "orm" → "tests/orm").
+ * packages/ 및 tests/ 디렉토리에서 모든 워크스페이스 패키지를 탐색한다.
+ * 디렉토리명 → 상대 경로의 맵을 반환한다 (예: "orm" → "tests/orm").
  */
 export function discoverWorkspacePackages(cwd: string): Map<string, string> {
   logger.debug("워크스페이스 패키지 탐색 시작");
@@ -71,13 +71,7 @@ export function discoverWorkspacePackages(cwd: string): Map<string, string> {
 }
 
 /**
- * Merge tests/ packages discovered from workspace into sd.config.ts packages.
- * Tests packages are assigned `{ target: "node" }` by default.
- * Also builds a pathMap (name → relative path) for all packages.
- * Throws SdError if a tests package name collides with an sd.config.ts package name.
- */
-/**
- * Build pathMap from sd.config.ts packages only (without tests packages).
+ * sd.config.ts 패키지만으로 pathMap을 구성한다 (tests 패키지 제외).
  */
 export function buildPathMapFromConfig(
   configPackages: Record<string, SdPackageConfig | undefined>,
@@ -89,6 +83,12 @@ export function buildPathMapFromConfig(
   return pathMap;
 }
 
+/**
+ * workspace에서 발견된 tests/ 패키지를 sd.config.ts 패키지에 병합한다.
+ * tests 패키지는 기본적으로 `{ target: "node" }`가 할당된다.
+ * 모든 패키지의 pathMap(name → 상대 경로)도 함께 구성한다.
+ * tests 패키지명이 sd.config.ts 패키지명과 충돌하면 SdError를 던진다.
+ */
 export function mergeTestsPackagesIntoConfig(
   configPackages: Record<string, SdPackageConfig | undefined>,
   workspacePackages: Map<string, string>,
@@ -97,12 +97,12 @@ export function mergeTestsPackagesIntoConfig(
   const pathMap = new Map<string, string>();
   const merged: Record<string, SdPackageConfig | undefined> = { ...configPackages };
 
-  // Set default paths for config packages
+  // config 패키지의 기본 경로 설정
   for (const name of Object.keys(configPackages)) {
     pathMap.set(name, `packages/${name}`);
   }
 
-  // Add tests packages
+  // tests 패키지 추가
   for (const [name, relPath] of workspacePackages) {
     if (!relPath.startsWith("tests/")) continue;
 
@@ -125,6 +125,22 @@ export interface DepsResult {
   replaceDeps: string[];
 }
 
+/**
+ * pnpm-workspace.yaml 기반으로 workspace 패키지의 name → 상대 디렉토리 맵을 구성한다.
+ * 예: "@simplysm/core-node" → "packages/core-node"
+ */
+function buildWorkspacePkgMap(cwd: string): Map<string, string> {
+  const map = new Map<string, string>();
+  const wsPkgs = discoverWorkspacePackages(cwd);
+  for (const [, relDir] of wsPkgs) {
+    const pkgJsonPath = pathx.posix(path.join(cwd, relDir, "package.json"));
+    if (!fs.existsSync(pkgJsonPath)) continue;
+    const pkgJson = JSON.parse(fs.readFileSync(pkgJsonPath, "utf-8")) as { name: string };
+    map.set(pkgJson.name, relDir);
+  }
+  return map;
+}
+
 export function collectDeps(
   pkgDir: string,
   cwd: string,
@@ -132,10 +148,9 @@ export function collectDeps(
 ): DepsResult {
   const startTime = performance.now();
   logger.debug("의존성 수집 시작");
-  const rootPkgJsonPath = pathx.posix(path.join(cwd, "package.json"));
-  const rootPkgJson = JSON.parse(fs.readFileSync(rootPkgJsonPath, "utf-8")) as { name: string };
-  const scopeMatch = rootPkgJson.name.match(/^(@[^/]+)\//);
-  const workspaceScope = scopeMatch != null ? scopeMatch[1] : undefined;
+
+  // pnpm-workspace.yaml에서 workspace 패키지 디렉토리 패턴을 읽어 실제 패키지 맵 구성
+  const workspacePkgMap = buildWorkspacePkgMap(cwd);
 
   const replaceDepsPatterns: Array<{ regex: RegExp }> = [];
   if (replaceDepsConfig != null) {
@@ -162,18 +177,15 @@ export function collectDeps(
       if (visited.has(dep)) continue;
       visited.add(dep);
 
-      // Check for workspace package
-      if (workspaceScope != null && dep.startsWith(workspaceScope + "/")) {
-        const dirName = dep.slice(workspaceScope.length + 1);
-        const depDir = pathx.posix(path.join(cwd, "packages", dirName));
-        if (fs.existsSync(pathx.posix(path.join(depDir, "package.json")))) {
-          workspaceDeps.push(dirName);
-          traverse(depDir);
-          continue;
-        }
+      // 워크스페이스 패키지 확인
+      const wsDir = workspacePkgMap.get(dep);
+      if (wsDir != null) {
+        workspaceDeps.push(path.basename(wsDir));
+        traverse(pathx.posix(path.join(cwd, wsDir)));
+        continue;
       }
 
-      // Check replaceDeps pattern
+      // replaceDeps 패턴 확인
       const matched = replaceDepsPatterns.find((p) => p.regex.test(dep));
       if (matched != null) {
         replaceDeps.push(dep);
@@ -194,8 +206,8 @@ export function collectDeps(
 }
 
 /**
- * Check if package.json in the given directory has @angular/core
- * in dependencies or peerDependencies.
+ * 지정된 디렉토리의 package.json에 @angular/core가
+ * dependencies 또는 peerDependencies에 있는지 확인한다.
  */
 export function hasAngularCoreDependency(pkgDir: string): boolean {
   const pkgJsonPath = pathx.posix(path.join(pkgDir, "package.json"));
@@ -215,11 +227,11 @@ export function hasAngularCoreDependency(pkgDir: string): boolean {
 }
 
 /**
- * Validate that all target names exist in the sdConfig packages.
- * Throws SdError if any unknown targets are found.
- * Does nothing when targets is empty.
- * @param targets - package name list to validate
- * @param packages - sdConfig.packages object
+ * 모든 target 이름이 sdConfig 패키지에 존재하는지 검증한다.
+ * 알 수 없는 target이 발견되면 SdError를 던진다.
+ * targets가 비어있으면 아무 동작도 하지 않는다.
+ * @param targets - 검증할 패키지명 목록
+ * @param packages - sdConfig.packages 객체
  */
 export function validateTargets(
   targets: string[],
@@ -234,11 +246,11 @@ export function validateTargets(
 }
 
 /**
- * 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
+ * targets로 패키지 설정을 필터링한다 (scripts target 제외)
+ * @param packages 패키지 설정 맵
+ * @param targets 필터링할 패키지명 목록. 빈 배열이면 scripts를 제외한 모든 패키지 반환
+ * @returns 필터링된 패키지 설정 맵
+ * @internal 테스트용으로 export
  */
 export function filterPackagesByTargets(
   packages: Record<string, SdPackageConfig | undefined>,
@@ -250,16 +262,16 @@ export function filterPackagesByTargets(
   for (const [name, config] of Object.entries(packages)) {
     if (config == null) continue;
 
-    // Exclude scripts target unless watch hook is configured
+    // watch hook이 설정되지 않은 scripts target 제외
     if (config.target === "scripts" && config.watch == null) continue;
 
-    // If targets is empty, include all packages
+    // targets가 비어있으면 모든 패키지 포함
     if (targets.length === 0) {
       result[name] = config;
       continue;
     }
 
-    // Filter only packages included in targets
+    // targets에 포함된 패키지만 필터링
     if (targets.includes(name)) {
       result[name] = config;
     }
@@ -336,7 +348,7 @@ export function classifyDevPackages(
 
   const entries = iteratePackages(allPackages, []);
 
-  // First pass: collect server names
+  // 1차 패스: 서버 이름 수집
   const serverNames = new Set<string>();
   for (const { name, config } of entries) {
     if (config.target === "server") {
@@ -344,7 +356,7 @@ export function classifyDevPackages(
     }
   }
 
-  // Second pass: classify all packages
+  // 2차 패스: 모든 패키지 분류
   for (const { name, config } of entries) {
     const relPath = pathMap.get(name) ?? `packages/${name}`;
     const pkgDir = pathx.posix(path.join(cwd, relPath));
@@ -361,7 +373,7 @@ export function classifyDevPackages(
         config: config,
       });
 
-      // Build server-client mapping
+      // 서버-클라이언트 매핑 구성
       const clientConfig = config;
       if (typeof clientConfig.server === "string") {
         if (serverNames.has(clientConfig.server)) {
@@ -375,7 +387,7 @@ export function classifyDevPackages(
         }
       }
     }
-    // Library and scripts packages are excluded from dev mode
+    // 라이브러리 및 scripts 패키지는 dev 모드에서 제외
   }
 
   logger.debug(`dev 패키지 분류 완료 (server: ${serverPackages.length}, client: ${clientPackages.length})`);