@simplysm/sd-cli 14.0.44 → 14.0.46

package/tests/esbuild/esbuild-angular-compiler-plugin-worker.verify.md

@@ -1,31 +1,59 @@
-# Web Worker 번들링 — LLM 검증
+# Web Worker 통합 (Feature 1.2) — LLM 검증
 
 ## 검증 항목
 
-### bundleWebWorker 함수
-
-- [x] buildSync 옵션: `esbuild-angular-compiler-plugin.ts:165-177` — platform: "browser", write: false, bundle: true, metafile: true, format: "esm", entryNames: "worker-[hash]", plugins: undefined, supported: undefined 모두 확인
-- [x] sourcemap 전달: `esbuild-angular-compiler-plugin.ts:174` — `sourcemap` 파라미터 직접 전달
-- [x] initialOptions spread: `esbuild-angular-compiler-plugin.ts:165` — `...build.initialOptions`로 target/absWorkingDir/outdir 전달
-- [x] catch — errors/warnings 있으면 반환: `esbuild-angular-compiler-plugin.ts:178-186` — `"errors" in error && "warnings" in error` 확인 후 `error as esbuild.BuildResult` 반환
-- [x] catch — 없으면 re-throw: `esbuild-angular-compiler-plugin.ts:187` — `throw error`
-
-### processWebWorker 콜백
-
-- [x] workerFile resolve: `esbuild-angular-compiler-plugin.ts:346` — `path.join(path.dirname(containingFile), workerFile)`
-- [x] 성공 — additionalResults 저장: `esbuild-angular-compiler-plugin.ts:365-368` — `additionalResults.set(fullWorkerPath, { outputFiles, metafile })`
-- [x] 성공 — FileReferenceTracker 등록: `esbuild-angular-compiler-plugin.ts:371-376` — `metafile.inputs` 키를 `path.join(cwd, input)`으로 변환하여 등록
-- [x] 성공 — 반환값: `esbuild-angular-compiler-plugin.ts:379-388` — `/^worker-[A-Z0-9]{8}\.[cm]?js$/` 패턴으로 검색, `path.relative(outdir, ...)` + `replaceAll("\\", "/")` forward slash 반환
-- [x] 에러 — errors push: `esbuild-angular-compiler-plugin.ts:352` — `errors.push(...workerResult.errors)`
-- [x] 에러 — warnings push: `esbuild-angular-compiler-plugin.ts:349` — `warnings.push(...workerResult.warnings)` (성공/실패 공통 실행)
-- [x] 에러 — FileReferenceTracker 등록: `esbuild-angular-compiler-plugin.ts:354-359` — `location?.file` 필터 후 `path.join(cwd, f)` 변환
-- [x] 에러 — additionalResults 저장: `esbuild-angular-compiler-plugin.ts:361` — `{ errors: workerResult.errors }`
-- [x] 에러 — 원본 반환: `esbuild-angular-compiler-plugin.ts:362` — `return workerFile`
-
-### Plugin 통합
-
-- [x] import: `esbuild-angular-compiler-plugin.ts:14` — `import { createWorkerTransformer } from "../angular/web-worker-transformer.js"`
-- [x] transformer 생성: `esbuild-angular-compiler-plugin.ts:392` — `createWorkerTransformer(processWebWorker)`
-- [x] emitAffectedFiles 전달: `esbuild-angular-compiler-plugin.ts:393-395` — `{ additionalTransformers: { before: [workerTransformer] } }`
-- [x] 클로저 접근: `esbuild-angular-compiler-plugin.ts:344-389` — onStart 내부에서 생성, `errors`/`warnings`/`additionalResults`/`referencedFileTracker`/`cwd`/`build` 접근 가능
-- [x] onEnd 병합: `esbuild-angular-compiler-plugin.ts:449-460` — additionalResults 순회에서 worker 결과 포함
+### 제거된 코드
+
+- [x] `createWorkerTransformer` import가 제거됨: `esbuild-angular-compiler-plugin.ts`에 `web-worker-transformer.js`로부터의 import가 존재하지 않음
+- [x] `AdditionalResult` 인터페이스 선언이 제거됨
+- [x] `bundleWebWorker` 함수 및 `#region bundleWebWorker` 주석이 제거됨
+- [x] `additionalResults` Map 선언이 제거됨
+- [x] `createWebWorkerProcessor` 함수가 제거됨
+- [x] onStart 내부의 `processWebWorker` + `workerTransformer` 생성 코드가 제거됨
+- [x] `compileAsync` 호출에서 `additionalTransformers` 옵션이 제거됨
+- [x] 증분 빌드 루프에서 `additionalResults.delete(file)` 호출이 제거됨
+
+### 추가된 코드
+
+- [x] `transformWorkerPatterns` import가 `./esbuild-worker-plugin`에서 추가됨 (같은 디렉토리, .js 확장자 미사용)
+- [x] setup 스코프에 `workerResultsByContainingFile = new Map<string, { outputFiles?: esbuild.OutputFile[]; metafile?: esbuild.Metafile }>()` 선언이 존재함 (증분 빌드 시 변경되지 않은 Worker metafile 유지 목적)
+
+### onStart — TS 파일 Worker 패턴 처리 (Rule 1)
+
+- [x] 증분 빌드 시 `expandedModifiedFiles` 각 파일에 대해 `workerResultsByContainingFile.delete(file)` 호출 (선택적 제거)
+- [x] `emitResults` 루프에서 각 파일에 대해 `transformWorkerPatterns(contents, normalized, build)` 호출
+- [x] `workerResult != null`이면 `typeScriptFileCache.set(normalized, workerResult.contents)` 저장
+- [x] `workerResult.errors`/`workerResult.warnings`를 onStart의 `errors`/`warnings` 배열에 push
+- [x] `workerResult.workerMetafile != null`이면 `referencedFileTracker.add(normalized, Object.keys(workerResult.workerMetafile.inputs).map(input => path.join(cwd, input)))` 호출
+- [x] `workerMetafile` 또는 `workerOutputFiles`가 있으면 `workerResultsByContainingFile.set(normalized, { outputFiles, metafile })` 저장
+- [x] `workerResult == null`이면 기존처럼 `typeScriptFileCache.set(normalized, contents)` 저장
+- [x] Worker 번들 에러가 errors 배열에 포함되어 onStart 결과에 반영됨 (Scenario: 에러 전파)
+
+### JS onLoad — .js 파일 Worker 패턴 처리 (Rule 2)
+
+- [x] `createCachedLoad` 콜백에서 `javascriptTransformer.transformFile` 호출 후 결과를 `TextDecoder`로 문자열로 변환
+- [x] 변환된 문자열에 `transformWorkerPatterns(textContents, request, build)` 적용
+- [x] `workerResult != null`일 때:
+  - `workerResult.workerMetafile`이 있으면 `referencedFileTracker.add` 호출
+  - `workerMetafile` 또는 `workerOutputFiles`가 있으면 `workerResultsByContainingFile.set(request, { outputFiles, metafile })` 저장
+  - 반환: `{ contents, loader: "js", resolveDir, errors (>0일때), warnings (>0일때) }` — TS/JS 에러 처리 일관성 확보 (L2 리뷰 반영)
+- [x] `workerResult == null`일 때 기존과 동일한 `{ contents, loader: "js", resolveDir }` 반환
+
+### onEnd — metafile 병합 (Rule 3)
+
+- [x] onEnd에서 `workerResultsByContainingFile.values()` 순회
+- [x] 각 항목의 `outputFiles`가 있으면 `result.outputFiles?.push(...outputFiles)`
+- [x] 각 항목의 `metafile`이 있으면 `Object.assign(result.metafile.inputs, wr.metafile.inputs)` + `Object.assign(result.metafile.outputs, wr.metafile.outputs)`
+- [x] onEnd에서 Map 전체 리셋 없음 — 증분 빌드에서 변경되지 않은 Worker 결과가 다음 빌드에서도 병합됨
+- [x] 기존 `additionalResults` 순회 루프가 제거됨
+
+### client config 무변경 (Rule 6)
+
+- [x] `esbuild-client-config.ts`에 Worker 플러그인 추가 코드가 존재하지 않음 (plugins 배열에 `createWorkerBundlePlugin` 또는 `sd-worker-bundle` 미포함)
+- [x] `esbuild-client-config.ts` 파일 자체가 Feature 1.2로 인해 변경되지 않음 (git diff 기준)
+
+### 회귀 방지
+
+- [x] `pnpm test --run packages/sd-cli/tests/esbuild/esbuild-angular-compiler-plugin.spec.ts` 통과 (기존 plugin 구조 테스트)
+- [x] `pnpm test --run packages/sd-cli/tests/esbuild/esbuild-worker-plugin.spec.ts` 통과 (Feature 1.1 Unit 테스트)
+- [x] `pnpm test --run packages/sd-cli/tests/esbuild/esbuild-worker-plugin.acc.spec.ts` 통과 (Feature 1.1 Acceptance 테스트)

package/tests/esbuild/esbuild-worker-plugin-node.verify.md

@@ -0,0 +1,11 @@
+# import.meta.resolve Worker 패턴 — LLM 검증
+
+## 검증 항목
+
+- [x] `NODE_WORKER_PATTERN` 정규식이 상대 경로만 감지: `esbuild-worker-plugin.ts:25` — `(\.\.?\/[^"']+)` 패턴으로 `./` 또는 `../`로 시작하는 경로만 캡처. `"some-package"` 같은 절대 모듈 경로는 매칭되지 않음.
+- [x] `bundleWorker`에 platform 파라미터 추가: `esbuild-worker-plugin.ts:42` — `platform: esbuild.Platform` 파라미터 사용. 기존 `"browser"` 하드코딩 제거.
+- [x] 브라우저 Worker 패턴이 `platform: "browser"`로 빌드: `esbuild-worker-plugin.ts:137` — `processWorkerBundle(fullWorkerPath, "browser")` 호출. 기존 동작 유지.
+- [x] Node.js Worker 패턴이 메인 빌드의 platform 계승: `esbuild-worker-plugin.ts:149` — `build.initialOptions.platform ?? "browser"` 사용. 서버 빌드(platform: "node")에서는 "node"로 빌드.
+- [x] 경로 치환이 `new URL("path", import.meta.url).href` 형태: `esbuild-worker-plugin.ts:153` — `new URL("${workerCodePath}", import.meta.url).href` 반환. file:// URL을 반환하므로 core-node Worker 호환.
+- [x] 기존 브라우저 Worker 테스트 22개 모두 통과: `pnpm test --run` 결과 32개 전체 통과 (기존 22 + 신규 10).
+- [x] `external: undefined` 설정 유지: `esbuild-worker-plugin.ts:62` — 메인 빌드의 external이 Worker 번들에 상속되지 않음.

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

@@ -0,0 +1,318 @@
+import { describe, it, expect } from "vitest";
+import esbuild from "esbuild";
+import path from "path";
+import fs from "fs";
+import os from "os";
+import { fileURLToPath } from "url";
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const fixturesDir = path.join(__dirname, "fixtures", "worker-plugin");
+
+const { createWorkerBundlePlugin } = await import(
+  "../../src/esbuild/esbuild-worker-plugin.js"
+);
+
+/**
+ * 임시 디렉토리에 entry 파일을 생성하고 esbuild로 빌드한다.
+ * Worker fixture 파일은 fixturesDir에서 참조된다.
+ */
+async function buildWithPlugin(
+  entryContent: string,
+  options?: { write?: boolean; entryExt?: string },
+): Promise<esbuild.BuildResult & { outdir: string }> {
+  const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "worker-plugin-test-"));
+  const ext = options?.entryExt ?? ".js";
+  const entryFile = path.join(tmpDir, `entry${ext}`);
+
+  // Worker fixture 파일을 tmpDir에 복사 (entry에서 상대경로 참조 가능하도록)
+  for (const f of ["worker.js", "worker2.js", "shared-worker.js", "worker-error.js", "node-worker.js"]) {
+    const src = path.join(fixturesDir, f);
+    if (fs.existsSync(src)) {
+      fs.copyFileSync(src, path.join(tmpDir, f));
+    }
+  }
+
+  fs.writeFileSync(entryFile, entryContent);
+
+  const outdir = path.join(tmpDir, "dist");
+  const write = options?.write ?? false;
+
+  const result = await esbuild.build({
+    entryPoints: [entryFile],
+    bundle: true,
+    write,
+    outdir,
+    format: "esm",
+    platform: "browser",
+    metafile: true,
+    logLevel: "silent",
+    plugins: [createWorkerBundlePlugin()],
+  });
+
+  return { ...result, outdir };
+}
+
+/**
+ * Node.js platform으로 esbuild 빌드한다 (서버 빌드 시뮬레이션).
+ */
+async function buildNodeWithPlugin(
+  entryContent: string,
+  options?: { write?: boolean },
+): Promise<esbuild.BuildResult & { outdir: string }> {
+  const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "worker-plugin-node-test-"));
+  const entryFile = path.join(tmpDir, "entry.js");
+
+  for (const f of ["worker.js", "worker2.js", "shared-worker.js", "worker-error.js", "node-worker.js"]) {
+    const src = path.join(fixturesDir, f);
+    if (fs.existsSync(src)) {
+      fs.copyFileSync(src, path.join(tmpDir, f));
+    }
+  }
+
+  fs.writeFileSync(entryFile, entryContent);
+
+  const outdir = path.join(tmpDir, "dist");
+  const write = options?.write ?? false;
+
+  const result = await esbuild.build({
+    entryPoints: [entryFile],
+    bundle: true,
+    write,
+    outdir,
+    format: "esm",
+    platform: "node",
+    metafile: true,
+    logLevel: "silent",
+    plugins: [createWorkerBundlePlugin()],
+  });
+
+  return { ...result, outdir };
+}
+
+describe("esbuild Worker Bundle Plugin — Acceptance", () => {
+  it("JS 파일의 Worker 패턴을 감지하여 번들링하고 URL을 치환한다", async () => {
+    const result = await buildWithPlugin(
+      'const w = new Worker(new URL("./worker.js", import.meta.url));',
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const mainOutput = result.outputFiles!.find((f) => path.basename(f.path).startsWith("entry"));
+    expect(mainOutput).toBeDefined();
+    const content = mainOutput!.text;
+
+    // 원본 경로가 치환됨
+    expect(content).not.toContain("./worker.js");
+    // worker-[HASH].js 패턴으로 치환
+    expect(content).toMatch(/worker-[A-Z0-9]+\.js/i);
+    // { type: "module" } 추가됨
+    expect(content).toContain("module");
+
+    // Worker 번들 파일이 outputFiles에 포함됨
+    const workerOutput = result.outputFiles!.find((f) =>
+      /worker-[a-z0-9]+\.js$/i.test(path.basename(f.path)),
+    );
+    expect(workerOutput).toBeDefined();
+  });
+
+  it("SharedWorker 패턴을 감지하여 번들링하고 URL을 치환한다", async () => {
+    const result = await buildWithPlugin(
+      'const sw = new SharedWorker(new URL("./shared-worker.js", import.meta.url));',
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const mainOutput = result.outputFiles!.find((f) => path.basename(f.path).startsWith("entry"));
+    const content = mainOutput!.text;
+
+    expect(content).not.toContain("./shared-worker.js");
+    expect(content).toMatch(/worker-[A-Z0-9]+\.js/i);
+  });
+
+  it("TS 파일의 Worker 패턴을 감지하여 번들링한다", async () => {
+    const result = await buildWithPlugin(
+      'const w: Worker = new Worker(new URL("./worker.js", import.meta.url));',
+      { entryExt: ".ts" },
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const mainOutput = result.outputFiles!.find((f) => path.basename(f.path).startsWith("entry"));
+    const content = mainOutput!.text;
+
+    expect(content).not.toContain("./worker.js");
+    expect(content).toMatch(/worker-[A-Z0-9]+\.js/i);
+  });
+
+  it("Worker 패턴 없는 파일은 변환 없이 통과한다", async () => {
+    const result = await buildWithPlugin(
+      'console.log("no worker");',
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const mainOutput = result.outputFiles!.find((f) => path.basename(f.path).startsWith("entry"));
+    const content = mainOutput!.text;
+
+    expect(content).toContain("no worker");
+    expect(content).not.toMatch(/worker-[A-Z0-9]+\.js/i);
+  });
+
+  it("한 파일에 복수 Worker 패턴이 있으면 모두 번들링한다", async () => {
+    const result = await buildWithPlugin(
+      [
+        'const w1 = new Worker(new URL("./worker.js", import.meta.url));',
+        'const w2 = new Worker(new URL("./worker2.js", import.meta.url));',
+      ].join("\n"),
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const mainOutput = result.outputFiles!.find((f) => path.basename(f.path).startsWith("entry"));
+    const content = mainOutput!.text;
+
+    expect(content).not.toContain("./worker.js");
+    expect(content).not.toContain("./worker2.js");
+
+    // 2개의 서로 다른 worker 번들이 있어야 함
+    const workerOutputs = result.outputFiles!.filter((f) =>
+      /worker-[a-z0-9]+\.js$/i.test(path.basename(f.path)),
+    );
+    expect(workerOutputs.length).toBeGreaterThanOrEqual(2);
+  });
+
+  it("기존 옵션이 있으면 유지하고 URL 경로만 치환한다", async () => {
+    const result = await buildWithPlugin(
+      'const w = new Worker(new URL("./worker.js", import.meta.url), { type: "module" });',
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const mainOutput = result.outputFiles!.find((f) => path.basename(f.path).startsWith("entry"));
+    const content = mainOutput!.text;
+
+    expect(content).not.toContain("./worker.js");
+    expect(content).toMatch(/worker-[A-Z0-9]+\.js/i);
+    // 기존 { type: "module" }이 유지됨
+    expect(content).toContain("module");
+  });
+
+  it("Worker 빌드 에러가 메인 빌드로 전파된다", async () => {
+    try {
+      await buildWithPlugin(
+        'const w = new Worker(new URL("./worker-error.js", import.meta.url));',
+      );
+      expect.fail("빌드 에러가 발생해야 한다");
+    } catch (e: unknown) {
+      // esbuild.build()는 에러 시 errors 프로퍼티를 가진 예외를 throw
+      const buildError = e as { errors?: esbuild.Message[] };
+      expect(buildError.errors).toBeDefined();
+      expect(buildError.errors!.length).toBeGreaterThan(0);
+    }
+  });
+
+  it("write: true 빌드에서 Worker 파일이 디스크에 기록된다", async () => {
+    const result = await buildWithPlugin(
+      'const w = new Worker(new URL("./worker.js", import.meta.url));',
+      { write: true },
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    // outdir에 worker-[HASH].js 파일이 존재
+    const files = fs.readdirSync(result.outdir);
+    const workerFile = files.find((f) => /worker-[a-z0-9]+\.js$/i.test(f));
+    expect(workerFile).toBeDefined();
+  });
+
+  it("write: false 빌드에서 Worker 파일이 outputFiles에 포함된다", async () => {
+    const result = await buildWithPlugin(
+      'const w = new Worker(new URL("./worker.js", import.meta.url));',
+      { write: false },
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const workerOutput = result.outputFiles!.find((f) =>
+      /worker-[a-z0-9]+\.js$/i.test(path.basename(f.path)),
+    );
+    expect(workerOutput).toBeDefined();
+  });
+});
+
+describe("esbuild Worker Bundle Plugin — Node.js import.meta.resolve Acceptance", () => {
+  it("import.meta.resolve 패턴을 감지하여 번들링하고 경로를 치환한다", async () => {
+    const result = await buildNodeWithPlugin(
+      'const p = import.meta.resolve("./node-worker.js");',
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const mainOutput = result.outputFiles!.find((f) => path.basename(f.path).startsWith("entry"));
+    expect(mainOutput).toBeDefined();
+    const content = mainOutput!.text;
+
+    // import.meta.resolve가 치환됨
+    expect(content).not.toContain("import.meta.resolve");
+    // new URL("worker-HASH.js", import.meta.url).href 패턴으로 치환
+    expect(content).toMatch(/new URL\("worker-[A-Z0-9]+\.js",\s*import\.meta\.url\)\.href/i);
+
+    // Worker 번들 파일이 outputFiles에 포함됨
+    const workerOutput = result.outputFiles!.find((f) =>
+      /worker-[a-z0-9]+\.js$/i.test(path.basename(f.path)),
+    );
+    expect(workerOutput).toBeDefined();
+  });
+
+  it("절대 모듈 경로의 import.meta.resolve는 무시한다", async () => {
+    const result = await buildNodeWithPlugin(
+      'const p = import.meta.resolve("path");',
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const mainOutput = result.outputFiles!.find((f) => path.basename(f.path).startsWith("entry"));
+    const content = mainOutput!.text;
+
+    // import.meta.resolve가 그대로 남음 (치환되지 않음)
+    expect(content).toContain("import.meta.resolve");
+  });
+
+  it("브라우저 + Node.js Worker 패턴 공존 시 모두 번들링한다", async () => {
+    const result = await buildNodeWithPlugin(
+      [
+        'const w = new Worker(new URL("./worker.js", import.meta.url));',
+        'const p = import.meta.resolve("./node-worker.js");',
+      ].join("\n"),
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const mainOutput = result.outputFiles!.find((f) => path.basename(f.path).startsWith("entry"));
+    const content = mainOutput!.text;
+
+    // 브라우저 Worker 치환
+    expect(content).not.toContain('"./worker.js"');
+    // Node.js resolve 치환
+    expect(content).not.toContain("import.meta.resolve");
+
+    // 최소 2개의 worker 번들
+    const workerOutputs = result.outputFiles!.filter((f) =>
+      /worker-[a-z0-9]+\.js$/i.test(path.basename(f.path)),
+    );
+    expect(workerOutputs.length).toBeGreaterThanOrEqual(2);
+  });
+
+  it("write: true 빌드에서 Worker 파일이 디스크에 기록된다", async () => {
+    const result = await buildNodeWithPlugin(
+      'const p = import.meta.resolve("./node-worker.js");',
+      { write: true },
+    );
+
+    expect(result.errors).toHaveLength(0);
+
+    const files = fs.readdirSync(result.outdir);
+    const workerFile = files.find((f) => /worker-[a-z0-9]+\.js$/i.test(f));
+    expect(workerFile).toBeDefined();
+  });
+});