@things-factory/board-import 10.0.0-beta.72 → 10.0.0-beta.74

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@things-factory/board-import",
-  "version": "10.0.0-beta.72",
+  "version": "10.0.0-beta.74",
   "main": "dist-server/index.js",
   "things-factory": true,
   "author": "heartyoh",
@@ -31,5 +31,5 @@
     "@things-factory/shell": "^10.0.0-beta.72",
     "dxf-parser": "^1.1.2"
   },
-  "gitHead": "df28904355d85974279d2be2c51b1ffa39768953"
+  "gitHead": "589dd8dab5bb2db400e5f10a1f455c3fb69bf9f4"
 }

package/server/service/converters/generic-to.test.ts

@@ -6,14 +6,27 @@
  */
 import {
   convertPlaceholderTo,
-  attachGlbToPlaceholder,
   isPlaceholderType,
   CORE_PLACEHOLDER_TYPES
 } from './generic-to'
+import { GENERIC_OBJECT_TYPE } from '../pipeline/stage2-mapping'
 
 // 호출자가 도메인 추상까지 인정하는 경우의 fixtures.
 const EXTRA = ['GenericTransport', 'GenericFacility'] as const
 
+// ── 계약 어설션 ─────────────────────────────────────────────────────
+// things-scene 의 CORE_PLACEHOLDER_TYPES 와 mirror. client/server 경계로
+// 직접 import 불가 — 이 테스트가 양쪽 값 일치를 보증.
+describe('CORE_PLACEHOLDER_TYPES contract', () => {
+  test('GENERIC_OBJECT_TYPE 은 반드시 placeholder 고정', () => {
+    expect(GENERIC_OBJECT_TYPE).toBe('placeholder')
+  })
+
+  test('CORE_PLACEHOLDER_TYPES 는 [placeholder] 고정 — things-scene 과 동기화', () => {
+    expect(Array.from(CORE_PLACEHOLDER_TYPES)).toEqual(['placeholder'])
+  })
+})
+
 describe('isPlaceholderType', () => {
   test('코어 type 들은 인정', () => {
     for (const t of CORE_PLACEHOLDER_TYPES) {
@@ -90,56 +103,3 @@ describe('convertPlaceholderTo', () => {
   })
 })
 
-describe('attachGlbToPlaceholder', () => {
-  const base = {
-    type: 'placeholder',
-    id: 'a',
-    left: 0,
-    top: 0,
-    width: 100,
-    height: 100
-  }
-
-  test('placeholder 는 type 유지 + src set', () => {
-    const r = attachGlbToPlaceholder(base as any, { src: '/glb/foo.glb' }) as any
-    expect(r.type).toBe('placeholder')
-    expect(r.src).toBe('/glb/foo.glb')
-  })
-
-  test('autoFit 옵션 전파', () => {
-    const r = attachGlbToPlaceholder(base as any, {
-      src: '/glb/foo.glb',
-      autoFit: true
-    }) as any
-    expect(r.autoFit).toBe(true)
-  })
-
-  test('convertedFrom 메타 없음', () => {
-    const r = attachGlbToPlaceholder(base as any, { src: '/glb/foo.glb' }) as any
-    expect(r.convertedFrom).toBeUndefined()
-  })
-
-  test('도메인 추상 (extraPlaceholderTypes) 은 type 유지', () => {
-    const tx = { ...base, type: 'GenericTransport' }
-    const r = attachGlbToPlaceholder(tx as any, {
-      src: '/glb/forklift.glb',
-      extraPlaceholderTypes: ['GenericTransport']
-    }) as any
-    expect(r.type).toBe('GenericTransport')
-    expect(r.src).toBe('/glb/forklift.glb')
-  })
-
-  test('명시 toType 지정 시 그것으로', () => {
-    const r = attachGlbToPlaceholder(base as any, {
-      src: '/glb/foo.glb',
-      toType: 'gltf-object'
-    }) as any
-    expect(r.type).toBe('gltf-object')
-  })
-
-  test('placeholder 가 아닌 type 은 throw', () => {
-    expect(() =>
-      attachGlbToPlaceholder({ type: 'rect', id: 'x' } as any, { src: '/glb/foo.glb' })
-    ).toThrow(/not a generic placeholder/)
-  })
-})

package/server/service/converters/generic-to.ts

@@ -1,14 +1,14 @@
 /**
  * Generic placeholder → 도메인 컴포넌트 변환.
  *
- * Import 시 매칭되지 않은 entity 는 추상 placeholder 로 보드에 배치된다. 사용자가 사후에
- * 모델러 UI 에서 "Convert to ..." 액션으로 호출하거나, GLB 를 attach 해 GLB 기반 컴포넌트로
- * morph 한다. `placeholder` 는 GltfComponent mixin 을 갖고 있어 GLB attach 시 type 유지.
+ * Import 시 매칭되지 않은 entity 는 `placeholder` 타입으로 보드에 배치된다.
+ * 사용자가 사후에 모델러 UI 에서 "Convert to ..." 액션으로 도메인 컴포넌트로 변환.
  *
- * 인정되는 placeholder type:
- *   - `placeholder` — 도메인 비종속 fallback (things-scene 등록). GLB attach 지원.
- *   - 도메인 패키지의 generic-* 추상 — 호출자가 추가 list 로 명시 (예: 'GenericTransport',
- *     'GenericFacility' 등). 코어 list 외의 추상도 같은 helper 로 변환 가능.
+ * 아키텍처 원칙:
+ *   - `placeholder` — 도메인 비종속 임시 추상. 다른 type 으로 변환하는 기능만.
+ *     GLB 지원 없음 (GltfComponent mixin 제거됨).
+ *   - 도메인 generic 추상 (GenericTransport, GenericFacility 등) — 영구 도메인 추상.
+ *     GLB 바디를 직접 채우는 방식으로 사용. 변환 대상 아님.
  *
  * 변환 시:
  *   - 좌표/크기/회전 보존
@@ -32,7 +32,13 @@ export interface ConvertOptions {
   extraPlaceholderTypes?: ReadonlyArray<string>
 }
 
-/** things-scene 코어가 등록한 도메인 비종속 placeholder type. */
+/**
+ * things-scene 코어가 등록한 도메인 비종속 placeholder type.
+ *
+ * @sync things-scene/src/components/generic-placeholders.ts CORE_PLACEHOLDER_TYPES
+ * client/server 경계로 직접 import 불가 — 값을 바꾸면 양쪽 동시 수정 필수.
+ * 계약은 generic-to.test.ts 의 'CORE_PLACEHOLDER_TYPES contract' 어설션으로 보증.
+ */
 export const CORE_PLACEHOLDER_TYPES = [GENERIC_OBJECT_TYPE] as const
 
 export function isPlaceholderType(
@@ -98,58 +104,6 @@ export function convertPlaceholderTo(
   }
 }
 
-/**
- * Generic placeholder 에 GLB 를 attach 해 GLB 기반 컴포넌트로 morph.
- *
- * morph 정책:
- *   - `placeholder` — type 유지 (GltfComponent mixin 이 있어 GLB 직접 수용).
- *   - 도메인 추상 (`GenericTransport`, `GenericFacility` 등) — type 유지 + src set.
- *     호출자가 toType 명시 가능 (예: 'GenericTransport'). 미지정이면 placeholder type 유지.
- */
-export interface AttachGlbOptions extends ConvertOptions {
-  /** GLB 파일 URL 또는 Attachment src. */
-  src: string
-  /** 자동으로 모델 크기에 맞춰 width/height/depth 보정 (gltf-object autoFit). */
-  autoFit?: boolean
-  /**
-   * 명시 변환 type — 도메인 추상 (`GenericTransport` 등) 으로 morph 시 지정.
-   * 미지정 시: placeholder type 유지 (placeholder 는 GltfComponent mixin 으로 GLB 직접 수용).
-   */
-  toType?: string
-}
-
-export function attachGlbToPlaceholder(
-  placeholder: BoardComponent,
-  options: AttachGlbOptions
-): BoardComponent {
-  const extra = options.extraPlaceholderTypes ?? []
-  if (!isPlaceholderType(placeholder.type, extra)) {
-    throw new Error(
-      `Source component is not a generic placeholder (type=${placeholder.type}). Cannot attach GLB.`
-    )
-  }
-
-  const {
-    strokeStyle: _strokeStyle,
-    strokeDashArray: _strokeDashArray,
-    fillStyle: _fillStyle,
-    label: _label,
-    placeholderLabel: _placeholderLabel,
-    placeholderHint: _placeholderHint,
-    ...preservedState
-  } = placeholder as any
-
-  // morph 후 type 결정 — placeholder 는 GltfComponent mixin 보유, 도메인 추상도 type 유지
-  const nextType = options.toType ?? placeholder.type
-
-  return {
-    ...preservedState,
-    type: nextType,
-    src: options.src,
-    ...(options.autoFit ? { autoFit: true } : {})
-  }
-}
-
 // ── Backward-compat alias — 기존 호출처 호환 ──────────────────────
 
 /**

package/server/service/index.ts

@@ -28,11 +28,10 @@ export type { BindingOptions } from './pipeline/stage4-binding'
 export {
   convertGenericTo,         // deprecated alias — keep for backward-compat
   convertPlaceholderTo,
-  attachGlbToPlaceholder,
   isPlaceholderType,
   CORE_PLACEHOLDER_TYPES
 } from './converters/generic-to'
-export type { ConvertOptions, AttachGlbOptions } from './converters/generic-to'
+export type { ConvertOptions } from './converters/generic-to'
 
 // Adapters
 export { DxfAdapter } from './adapters/dxf-adapter'