@things-factory/board-ai 10.1.5 → 10.1.7

package/client/components/chat-echo-dedup.test.ts

@@ -1,230 +0,0 @@
-import { receiveRemoteMessage, reconcileSentLine, remoteMessageToLine, type DedupLine } from './chat-echo-dedup'
-
-const msg = (id: string, role = 'user', content = 'hi') => ({ id, role, content })
-
-describe('remoteMessageToLine', () => {
-  it('필드 매핑 + relatedPatchId/toolUsagesJson 변환', () => {
-    const line = remoteMessageToLine({
-      id: 'm1',
-      role: 'assistant',
-      content: 'done',
-      relatedPatchId: 'p1',
-      toolUsagesJson: [{ name: 'x' }]
-    })
-    expect(line).toMatchObject({ id: 'm1', role: 'assistant', content: 'done', patchId: 'p1' })
-    expect(line.toolUsages).toEqual([{ name: 'x' }])
-  })
-
-  it('toolUsagesJson 이 배열 아니면 undefined', () => {
-    expect(remoteMessageToLine({ id: 'm', role: 'user', content: 'x', toolUsagesJson: null }).toolUsages).toBeUndefined()
-  })
-
-  it('creator → senderEmail/senderName (그룹챗 발신자 표시·자기 판정)', () => {
-    const line = remoteMessageToLine({ id: 'm', role: 'user', content: 'x', creator: { name: 'Kim', email: 'u9@example.com' } })
-    expect(line.senderEmail).toBe('u9@example.com')
-    expect(line.senderName).toBe('Kim')
-  })
-
-  it('creator 없으면 sender 필드 undefined', () => {
-    const line = remoteMessageToLine({ id: 'm', role: 'user', content: 'x', creator: null })
-    expect(line.senderEmail).toBeUndefined()
-    expect(line.senderName).toBeUndefined()
-  })
-})
-
-describe('receiveRemoteMessage — 방송 수신/에코 차단', () => {
-  it('신규 id → append + seen 등록', () => {
-    const res = receiveRemoteMessage([], new Set(), msg('m1'))!
-    expect(res).not.toBeNull()
-    expect(res.lines).toHaveLength(1)
-    expect(res.lines[0].id).toBe('m1')
-    expect(res.seen.has('m1')).toBe(true)
-  })
-
-  it('이미 seen 인 id → null (에코·중복 차단)', () => {
-    expect(receiveRemoteMessage([], new Set(['m1']), msg('m1'))).toBeNull()
-  })
-
-  it('id 없는 메시지 → null', () => {
-    expect(receiveRemoteMessage([], new Set(), { role: 'user', content: 'x' } as any)).toBeNull()
-    expect(receiveRemoteMessage([], new Set(), null)).toBeNull()
-  })
-
-  it('불변 — 원본 배열/셋 미변경', () => {
-    const lines: DedupLine[] = []
-    const seen = new Set<string>()
-    receiveRemoteMessage(lines, seen, msg('m1'))
-    expect(lines).toHaveLength(0)
-    expect(seen.size).toBe(0)
-  })
-
-  it('다른 참여자 메시지 두 개 순차 append', () => {
-    let lines: DedupLine[] = []
-    let seen = new Set<string>()
-    ;({ lines, seen } = receiveRemoteMessage(lines, seen, msg('a'))!)
-    ;({ lines, seen } = receiveRemoteMessage(lines, seen, msg('b'))!)
-    expect(lines.map(l => l.id)).toEqual(['a', 'b'])
-  })
-})
-
-describe('reconcileSentLine — optimistic 라인 조정', () => {
-  const optimistic = (): DedupLine[] => [{ _localId: 'L1', role: 'user', content: 'typed' }]
-
-  it('정상(에코 전): serverId 부여 + seen 등록 + _localId 해제', () => {
-    const res = reconcileSentLine(optimistic(), new Set(), 'L1', 'srv1', {})
-    expect(res.lines).toHaveLength(1)
-    expect(res.lines[0].id).toBe('srv1')
-    expect(res.lines[0]._localId).toBeUndefined()
-    expect(res.seen.has('srv1')).toBe(true)
-  })
-
-  it('레이스(에코 먼저): serverId 가 이미 seen → optimistic 중복 제거', () => {
-    // 에코가 먼저 도착해 같은 메시지가 이미 append + seen 됨
-    const lines: DedupLine[] = [
-      { id: 'srv1', role: 'user', content: 'typed#enriched' }, // 방송 에코로 들어온 것
-      { _localId: 'L1', role: 'user', content: 'typed' } // optimistic 중복
-    ]
-    const res = reconcileSentLine(lines, new Set(['srv1']), 'L1', 'srv1', {})
-    expect(res.lines).toHaveLength(1)
-    expect(res.lines[0].id).toBe('srv1')
-    expect(res.lines.some(l => l._localId === 'L1')).toBe(false)
-  })
-
-  it('patch 병합 (AI 라인: pending → 실제 응답)', () => {
-    const lines: DedupLine[] = [{ _localId: 'A1', role: 'assistant', content: '', pending: true }]
-    const res = reconcileSentLine(lines, new Set(), 'A1', 'srv2', {
-      content: 'reply',
-      pending: false,
-      patchId: 'p9'
-    })
-    expect(res.lines[0]).toMatchObject({ id: 'srv2', content: 'reply', pending: false, patchId: 'p9' })
-  })
-
-  it('serverId 없음(ad-hoc 세션): 라인 유지 + _localId 만 해제, seen 불변', () => {
-    const seen = new Set<string>()
-    const res = reconcileSentLine(optimistic(), seen, 'L1', undefined, {})
-    expect(res.lines[0]._localId).toBeUndefined()
-    expect(res.lines[0].id).toBeUndefined()
-    expect(res.seen.size).toBe(0)
-  })
-
-  it('해당 _localId 없으면 변화 없음', () => {
-    const lines = optimistic()
-    const res = reconcileSentLine(lines, new Set(), 'NOPE', 'srv1', {})
-    expect(res.lines).toBe(lines)
-  })
-})
-
-describe('내 발신 에코 흡수 (대기 중 중복 렌더 방지)', () => {
-  /* 방송은 mutation 응답보다 먼저 올 수 있고, 그 응답은 LLM 이 끝난 뒤에 온다.
-   * 붙였다 나중에 지우면 사용자는 **대기 시간 내내** 자기 질문을 두 줄로 본다. */
-  const pending = (): DedupLine[] => [
-    { _localId: 'L1', role: 'user', content: '질문', senderEmail: 'me@x.com' },
-    { _localId: 'L2', role: 'assistant', content: '', pending: true }
-  ]
-
-  it('내 에코는 대기 중인 내 줄에 id 를 입힌다 — 줄 수가 늘지 않는다', () => {
-    const r = receiveRemoteMessage(pending(), new Set(), {
-      id: 'S1',
-      role: 'user',
-      content: '질문',
-      creator: { name: 'Me', email: 'me@x.com' }
-    })!
-    expect(r.lines).toHaveLength(2) // append 되지 않음
-    expect(r.lines[0].id).toBe('S1')
-    expect(r.lines[0]._localId).toBe('L1') // 이후 patch 병합을 위해 유지
-    expect(r.seen.has('S1')).toBe(true)
-  })
-
-  it('발신자 email 이 없는 방송(단일 사용자·구형)도 내 줄에 흡수된다', () => {
-    const r = receiveRemoteMessage(pending(), new Set(), { id: 'S1', role: 'user', content: '질문' })!
-    expect(r.lines).toHaveLength(2)
-    expect(r.lines[0].id).toBe('S1')
-  })
-
-  it('다른 사용자 메시지는 흡수하지 않고 append 한다 — 내 대기 줄을 삼키면 안 된다', () => {
-    const r = receiveRemoteMessage(pending(), new Set(), {
-      id: 'S9',
-      role: 'user',
-      content: '남의 말',
-      creator: { name: 'Other', email: 'other@x.com' }
-    })!
-    expect(r.lines).toHaveLength(3)
-    expect(r.lines[0].id).toBeUndefined() // 내 대기 줄은 그대로
-    expect(r.lines[2]).toMatchObject({ id: 'S9', senderEmail: 'other@x.com' })
-  })
-
-  /* 이 테스트는 예전에 "assistant 방송은 흡수하지 않고 붙인다" 를 규칙으로 굳혀 두고 있었다.
-   * 그것이 실제 결함이었다 — 붙이면 뒤이은 mutation 응답이 대기 줄을 지우고, 그 줄에만 있던
-   * **영속되지 않는 응답 데이터**(조치 제안 카드·접지 경고)가 함께 사라진다. 규칙을 바로잡는다. */
-  it('assistant 방송도 대기 중인 AI 줄에 흡수한다 — 붙이면 제안 카드가 사라진다', () => {
-    const lines = [
-      { _localId: 'L1', role: 'user', content: '질문', senderEmail: 'me@x.com' },
-      { _localId: 'L2', role: 'assistant', content: '', pending: true }
-    ]
-    const r = receiveRemoteMessage(lines as any, new Set(), { id: 'S2', role: 'assistant', content: '답' })!
-    expect(r.lines).toHaveLength(2)
-    expect(r.lines[1]).toMatchObject({ id: 'S2', content: '답', pending: false })
-  })
-
-  it('대기 중인 AI 줄이 없으면 붙인다 — 다른 참여자가 부른 답변', () => {
-    const settled = [{ id: 'S0', role: 'user', content: '질문' }]
-    const r = receiveRemoteMessage(settled as any, new Set(), { id: 'S2', role: 'assistant', content: '답' })!
-    expect(r.lines).toHaveLength(2)
-    expect(r.lines[1].id).toBe('S2')
-  })
-
-  it('흡수한 뒤 mutation 응답 reconcile 은 그 줄을 지우지 않고 patch 를 병합한다', () => {
-    const echo = receiveRemoteMessage(pending(), new Set(), {
-      id: 'S1',
-      role: 'user',
-      content: '질문',
-      creator: { email: 'me@x.com' }
-    })!
-    const after = reconcileSentLine(echo.lines, echo.seen, 'L1', 'S1', { content: '질문(보강)' })
-    expect(after.lines).toHaveLength(2) // 자기 줄을 지우지 않음
-    expect(after.lines[0]).toMatchObject({ id: 'S1', content: '질문(보강)' })
-    expect(after.lines[0]._localId).toBeUndefined()
-  })
-})
-
-describe('AI 답변 에코 흡수 — 영속되지 않는 응답 데이터를 잃지 않기', () => {
-  /* 실제 사고: 방송이 mutation 응답보다 먼저 와서 assistant 줄이 새로 붙고, 뒤이어 응답이 대기 줄을
-   * 지웠다. 대기 줄에만 있던 **조치 제안 카드**가 그때 사라졌다(도구는 분명히 호출됐는데도). */
-  const pendingAi = () => [
-    { id: 'u1', role: 'user', content: '지게차 한 대 추가해줘' },
-    { _localId: 'L2', role: 'assistant', content: '', pending: true, proposals: [{ command: 'resource.add' }] }
-  ]
-
-  it('대기 중인 AI 줄에 흡수한다 — 새 줄로 붙이지 않는다', () => {
-    const r = receiveRemoteMessage(pendingAi() as any, new Set(), {
-      id: 'a1',
-      role: 'assistant',
-      content: '제안해 두었습니다.'
-    })!
-    expect(r.lines).toHaveLength(2)
-    expect(r.lines[1].id).toBe('a1')
-    expect(r.lines[1].content).toBe('제안해 두었습니다.')
-  })
-
-  it('제안 카드를 유지한다 — 영속되지 않는 값이라 잃으면 화면에서 버튼이 사라진다', () => {
-    const r = receiveRemoteMessage(pendingAi() as any, new Set(), {
-      id: 'a1',
-      role: 'assistant',
-      content: '제안해 두었습니다.'
-    })!
-    expect(r.lines[1].proposals).toEqual([{ command: 'resource.add' }])
-  })
-
-  it('대기 표시를 푼다 — 방송이 먼저 오면 답이 더 빨리 보인다', () => {
-    const r = receiveRemoteMessage(pendingAi() as any, new Set(), { id: 'a1', role: 'assistant', content: '답' })!
-    expect(r.lines[1].pending).toBe(false)
-  })
-
-  it('대기 줄이 없으면(다른 참여자가 부른 AI 답변) 새 줄로 붙인다', () => {
-    const lines = [{ id: 'u1', role: 'user', content: '질문' }]
-    const r = receiveRemoteMessage(lines as any, new Set(), { id: 'a9', role: 'assistant', content: '답' })!
-    expect(r.lines).toHaveLength(2)
-    expect(r.lines[1].id).toBe('a9')
-  })
-})

package/client/components/chat-echo-dedup.ts

@@ -1,155 +0,0 @@
-/*
- * 그룹챗 메시지 에코 dedup — 순수 로직.
- *
- * 실시간 방송은 자기 발신 메시지도 되돌려준다(에코). 또 pubsub publish 가 mutation 응답보다
- * 먼저 도착할 수 있어(레이스) optimistic 라인과 방송 라인이 겹칠 수 있다. 이 모듈은 그 조정을
- * **id 기반**으로 결정한다 (서버가 멘션 마커로 content 를 enrich 하므로 content 매칭은 불가).
- *
- * 컴포넌트 상태(this.lines / this._seenMessageIds)에서 분리 → 단위테스트 가능.
- * (C2 의 patch-apply-gate.ts 와 동일한 "순수 게이트" 컨벤션.)
- */
-
-/** dedup 에 필요한 최소 라인 형태 (board-ai-chat 의 ChatLine 부분집합). */
-export interface DedupLine {
-  id?: string
-  _localId?: string
-  role: string
-  content: string
-  senderEmail?: string
-  senderName?: string
-  [k: string]: any
-}
-
-/** 서버 방송 메시지(필요 필드만). */
-export interface RemoteMessage {
-  id?: string
-  role: string
-  content: string
-  relatedPatchId?: string
-  toolUsagesJson?: any
-  /** 접지 경고(영속) — 답이 언급했으나 근거에 없던 식별자. 나중에 읽는 사람도 같은 주의를 봐야 한다. */
-  groundingWarningsJson?: any
-  /** 발신자 — 표시용 name + **자기 여부 판정용 email**(내부 관리키 id 는 쓰지 않는다). */
-  creator?: { name?: string; email?: string } | null
-}
-
-/** 방송 메시지 → 라인. */
-export function remoteMessageToLine(m: RemoteMessage): DedupLine {
-  return {
-    id: m.id,
-    role: m.role,
-    content: m.content,
-    patchId: m.relatedPatchId ?? undefined,
-    toolUsages: Array.isArray(m.toolUsagesJson) ? m.toolUsagesJson : undefined,
-    groundingWarnings: Array.isArray(m.groundingWarningsJson) ? m.groundingWarningsJson : undefined,
-    senderEmail: m.creator?.email ?? undefined,
-    senderName: m.creator?.name ?? undefined
-  }
-}
-
-/**
- * 원격(방송) 메시지 수신 결과를 계산.
- * - id 없음 / 이미 seen → 변화 없음(null).
- * - **내 발신 에코**이고 아직 서버 id 를 못 받은 optimistic 라인이 있으면 → 그 라인에 id 를 입힌다.
- * - 그 밖 → seen 에 추가 + 라인 append.
- *
- * 왜 '입히기'인가: 방송은 mutation 응답보다 먼저 도착할 수 있다. 예전에는 에코를 새 줄로 붙이고
- * 중복 제거를 mutation 응답 시점(reconcileSentLine)에 했는데, 그 응답은 **LLM 이 끝난 뒤** 오므로
- * 사용자는 대기 시간 내내 자기 질문을 두 줄로 본다. 그래서 붙이지 않고 대기 중인 줄에 흡수시킨다.
- *
- * `_localId` 는 유지한다 — 이후 mutation 응답이 patch(요약·patchId 등)를 그 줄에 병합할 수 있어야 한다.
- */
-export function receiveRemoteMessage(
-  lines: DedupLine[],
-  seen: Set<string>,
-  m: RemoteMessage | undefined | null
-): { lines: DedupLine[]; seen: Set<string> } | null {
-  if (!m || !m.id) return null
-  if (seen.has(m.id)) return null
-  const nextSeen = new Set(seen)
-  nextSeen.add(m.id)
-
-  /* 내 발신 에코 흡수 — `_localId` 가 있는 줄은 이 클라이언트가 만든 것뿐이다.
-   * 발신자 판정: 양쪽 email 이 있으면 같아야 하고, 방송에 발신자 email 이 없으면(단일 사용자·구형)
-   * 내 것으로 본다. 다른 사용자의 메시지가 내 대기 줄을 삼키지 않도록 email 이 다르면 흡수하지 않는다. */
-  /* AI 답변 에코 흡수 — 사용자 발화와 **같은 이유**로 붙이지 않고 대기 줄에 흡수한다.
-   *
-   * 왜 필요한가(실제 사고): 방송은 mutation 응답보다 먼저 도착할 수 있다. 예전에는 assistant 방송을
-   * 새 줄로 붙였고, 뒤이어 mutation 응답이 "같은 서버 id 를 가진 다른 줄이 있다" 며 **대기 줄을 지웠다**.
-   * 그런데 대기 줄에만 있는 것이 있다 — **영속되지 않는 응답 데이터**(조치 제안 카드·접지 경고).
-   * 그래서 화면에서 실행 버튼이 사라졌다(제안 도구는 분명히 호출됐는데도).
-   *
-   * 흡수하면 서버가 보낸 본문(멘션 마커 포함)을 그 줄에 입히면서 대기 표시를 풀 수 있어, 답이 더
-   * 빨리 보이는 효과도 있다. */
-  if (m.role === 'assistant') {
-    const idx = lines.findIndex(l => l.role === 'assistant' && !l.id && !!l._localId)
-    if (idx >= 0) {
-      const nextLines = [...lines]
-      nextLines[idx] = {
-        ...nextLines[idx],
-        id: m.id,
-        /* 본문·영속 데이터는 서버 것으로 갱신하고, 이 줄이 들고 있던 값(제안 등)은 유지한다. */
-        content: m.content,
-        patchId: m.relatedPatchId ?? nextLines[idx].patchId,
-        toolUsages: Array.isArray(m.toolUsagesJson) ? m.toolUsagesJson : nextLines[idx].toolUsages,
-        groundingWarnings: Array.isArray(m.groundingWarningsJson)
-          ? m.groundingWarningsJson
-          : nextLines[idx].groundingWarnings,
-        pending: false
-      }
-      return { lines: nextLines, seen: nextSeen }
-    }
-  }
-
-  if (m.role === 'user') {
-    const echoEmail = m.creator?.email
-    const idx = lines.findIndex(
-      l =>
-        l.role === 'user' &&
-        !l.id &&
-        !!l._localId &&
-        (!echoEmail || l.senderEmail == null || l.senderEmail === echoEmail)
-    )
-    if (idx >= 0) {
-      const nextLines = [...lines]
-      nextLines[idx] = {
-        ...nextLines[idx],
-        id: m.id,
-        senderEmail: echoEmail ?? nextLines[idx].senderEmail,
-        senderName: m.creator?.name ?? nextLines[idx].senderName
-      }
-      return { lines: nextLines, seen: nextSeen }
-    }
-  }
-
-  return { lines: [...lines, remoteMessageToLine(m)], seen: nextSeen }
-}
-
-/**
- * 전송한 optimistic 라인(_localId)을 서버 id 로 reconcile.
- * - **다른 줄**이 이미 그 serverId 를 갖고 있으면(옛 append 경로) optimistic 중복 라인 제거.
- * - 아니면 serverId 부여(+ seen 등록) + patch 병합 + _localId 해제.
- * - 해당 _localId 라인이 없으면 변화 없음.
- * 반환: 새 { lines, seen } (불변).
- */
-export function reconcileSentLine(
-  lines: DedupLine[],
-  seen: Set<string>,
-  localId: string,
-  serverId: string | undefined,
-  patch: Partial<DedupLine>
-): { lines: DedupLine[]; seen: Set<string> } {
-  const idx = lines.findIndex(l => l._localId === localId)
-  if (idx < 0) return { lines, seen }
-
-  const nextLines = [...lines]
-  /* seen 만 보고 지우면, 에코를 이 줄에 **흡수**한 경우(id 가 이미 이 줄에 있다) 자기 줄을 지운다.
-   * 실제 중복은 "다른 줄이 같은 id 를 가진" 상태일 때뿐이다. */
-  if (serverId && nextLines.some((l, i) => i !== idx && l.id === serverId)) {
-    nextLines.splice(idx, 1)
-    return { lines: nextLines, seen }
-  }
-  const nextSeen = serverId ? new Set(seen).add(serverId) : seen
-  nextLines[idx] = { ...nextLines[idx], ...patch, id: serverId ?? nextLines[idx].id, _localId: undefined }
-  return { lines: nextLines, seen: nextSeen }
-}

package/client/components/chat-input-builder.ts

@@ -1,69 +0,0 @@
-/**
- * boardAIChat GraphQL mutation 의 input 객체 빌더.
- *
- * board-ai-chat.ts (Lit / Material Web 의존) 와 분리 — pure 함수만 두면 jest 의 node
- * 환경에서도 import 가능. 회귀 테스트 (selectedRefids 누락 등) 가 단위 테스트로 잠긴다.
- */
-
-export interface ChatMutationInputArgs {
-  sessionId?: string
-  history: Array<{ role: string; content: string }>
-  liveBoard: any
-  scopes?: string[]
-  /** 편집으로 접은 지점(마지막으로 남긴 서버 메시지 id) — 서버가 그 뒤 이력을 에이전트 문맥에서 뺀다. */
-  truncateAfterMessageId?: string
-  /** 이 대화면이 무엇을 보고 있는지(예: { spaceId }) — 도구가 대상을 알 수 있게. 없으면 모델이 식별자를 되묻는다. */
-  hostContext?: any
-  /**
-   * 이 대화면이 쓸 수 있는 외부 도구 카테고리(registerToolCategory 이름).
-   * 미지정 = 등록된 전부(현행 동작). 지정하면 정의와 **실행** 모두 그 범위로 제한된다.
-   */
-  toolCategories?: string[]
-  /** 코어 보드 편집 도구(addComponent·스타일링·씬 조작) 사용 여부. 기본 true. */
-  boardTools?: boolean
-  /**
-   * 첫 턴에 도구 호출을 강제할지 — 라이브 상태를 다루는 대화면에서 켠다.
-   * 근거 없이 상태를 단언하는 답(도구를 아예 부르지 않은 답)을 원천에서 막는다.
-   */
-  requireGroundingTools?: boolean
-  knownTypes?: string[]
-  categories?: string[]
-  componentSchemas?: any
-  /**
-   * 선택된 컴포넌트의 `refid` 배열 — things-scene universal numeric handle.
-   * id (데이터 바인딩 이름) 와는 다른 개념.
-   */
-  selectedRefids: number[]
-  /**
-   * popup 에서 명시 선택한 #mention 매핑 — {token, refid}.
-   * 사용자 textarea 에는 #token 만 보이고, 정확한 refid 는 이 별도 채널로 server 에 전달.
-   * server 가 mention map 으로 LLM 에 정확한 refid 정보 첨부 (사용자 노출 0).
-   */
-  mentions?: Array<{ token: string; refid: number }>
-}
-
-/**
- * GraphQL mutation 의 input 객체를 빌드.
- *
- * 핵심 보호 — `selectedRefids` 가 항상 포함되어야 한다. 누락이 사용자에게 보고된
- * "AI 가 선택된 컴포넌트를 모른다" 의 root cause 였다.
- */
-export function buildChatMutationInput(args: ChatMutationInputArgs): Record<string, any> {
-  return {
-    sessionId: args.sessionId ?? null,
-    messages: args.history,
-    currentBoard: args.liveBoard ?? null,
-    scopes: args.scopes,
-    ...(args.truncateAfterMessageId ? { truncateAfterMessageId: args.truncateAfterMessageId } : {}),
-    ...(args.hostContext ? { hostContext: args.hostContext } : {}),
-    /* 미지정은 서버 기본(전부 허용)에 맡긴다 — null 을 보내 굳이 덮어쓰지 않는다. */
-    ...(args.toolCategories ? { toolCategories: args.toolCategories } : {}),
-    ...(args.boardTools === false ? { boardTools: false } : {}),
-    ...(args.requireGroundingTools ? { requireGroundingTools: true } : {}),
-    knownTypes: args.knownTypes,
-    categories: args.categories,
-    componentSchemas: args.componentSchemas,
-    selectedRefids: args.selectedRefids,
-    ...(args.mentions && args.mentions.length > 0 && { mentions: args.mentions })
-  }
-}