@things-factory/board-ai 10.1.176 → 10.1.178

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@things-factory/board-ai",
-  "version": "10.1.176",
+  "version": "10.1.178",
   "main": "dist-server/index.js",
   "browser": "dist-client/client/index.js",
   "things-factory": true,
@@ -32,12 +32,12 @@
     "@operato/i18n": "^10.0.0",
     "@operato/scene-ops": "^10.14.1",
     "@operato/styles": "^10.0.0",
-    "@things-factory/ai-assistant": "^10.1.176",
-    "@things-factory/ai-client-base": "^10.1.176",
-    "@things-factory/auth-base": "^10.1.176",
-    "@things-factory/board-import": "^10.1.176",
+    "@things-factory/ai-assistant": "^10.1.178",
+    "@things-factory/ai-client-base": "^10.1.178",
+    "@things-factory/auth-base": "^10.1.178",
+    "@things-factory/board-import": "^10.1.178",
     "@things-factory/env": "^10.1.149",
-    "@things-factory/shell": "^10.1.165",
+    "@things-factory/shell": "^10.1.178",
     "dompurify": "^3.0.0",
     "graphql-tag": "^2.12.6",
     "lit": "^3.1.2",
@@ -47,5 +47,5 @@
     "copyfiles": "^2.4.1",
     "rimraf": "^5.0.0"
   },
-  "gitHead": "24928f127d73de0e9946423bd4c3295f54cce03d"
+  "gitHead": "d57448d3f3045f72a9f24b877a3e306eb34cbaf1"
 }

package/server/service/assistant.test.ts

@@ -15,7 +15,7 @@ import {
   isActionTool,
   executeReadTool,
   summarizeBoard,
-  summarizeToolResult,
+  resultShapeOf,
   buildBoardEditTools,
   DefaultBoardAIAssistant
 } from './assistant'
@@ -1857,9 +1857,9 @@ describe('agentic loop (chatViaTools) — mock provider', () => {
     expect(u.name).toBe('getSelection')
     expect(u.kind).toBe('read')
     expect(u.arguments).toEqual({})
-    // result 는 summarized 객체
-    expect(u.result.count).toBe(1)
-    expect(u.result.selected[0].name).toBe('AGV-1')
+    // result 는 값이 아니라 모양 — 건수 · 키 이름 (ADR-0061 결정 8)
+    expect(u.result).toMatchObject({ count: 1, keys: expect.arrayContaining(['selected']) })
+    expect(JSON.stringify(u.result)).not.toMatch(/AGV-1/)
     expect(isGroundedAnalysisContract(r.analysis)).toBe(true)
     expect(r.analysis?.kind).toBe('board.analysis.v1')
     expect(r.analysis?.approvalRequired).toBe(true)
@@ -1885,7 +1885,7 @@ describe('agentic loop (chatViaTools) — mock provider', () => {
     const u = r.toolUsages![0]
     expect(u.name).toBe('addComponent')
     expect(u.kind).toBe('write')
-    expect(u.result.queued).toBe(true)
+    expect(u.result.keys).toContain('queued')
   })
 
   test('toolUsages 수집 — read + write 혼합 시간순 보존', async () => {
@@ -1929,7 +1929,7 @@ describe('agentic loop (chatViaTools) — mock provider', () => {
     const r = await a.chat([{ role: 'user', content: 'x' }], undefined)
     expect(r.toolUsages).toHaveLength(1)
     expect(r.toolUsages![0].kind).toBe('unknown')
-    expect(r.toolUsages![0].result.error).toMatch(/unknown/i)
+    expect(r.toolUsages![0].result.errorCode).toMatch(/unknown/i)
   })
 
   test('actions 수집 — selectComponents 호출 시 ChatResponse.actions 누적', async () => {
@@ -2011,43 +2011,28 @@ describe('agentic loop (chatViaTools) — mock provider', () => {
   })
 })
 
-describe('summarizeToolResult — UI 영속용 결과 압축', () => {
-  test('짧은 string / number / boolean 그대로', () => {
-    expect(summarizeToolResult('hello')).toBe('hello')
-    expect(summarizeToolResult(42)).toBe(42)
-    expect(summarizeToolResult(true)).toBe(true)
-    expect(summarizeToolResult(null)).toBeNull()
+describe('resultShapeOf — 대화 행에 남는 결과의 모양 (ADR-0061 결정 8)', () => {
+  test('값이 아니라 크기 · 건수 · 키 이름만 남는다', () => {
+    const pay = { read: 2, rows: [{ name: '김급여', net: 3100000 }, { name: '이급여', net: 2900000 }] }
+    const shape = resultShapeOf(pay)
+    expect(shape).toEqual({ size: JSON.stringify(pay).length, count: 2, keys: ['read', 'rows'] })
+    expect(JSON.stringify(shape)).not.toMatch(/김급여|3100000/)
   })
 
-  test('200자 초과 string truncate + 마커', () => {
-    const long = 'x'.repeat(500)
-    const r = summarizeToolResult(long)
-    expect(r).toMatch(/…\[\+\d+ chars\]$/)
-    expect(r.length).toBeLessThan(long.length)
+  test('목록은 건수, 값 하나는 크기만', () => {
+    expect(resultShapeOf([1, 2, 3])).toEqual({ size: 7, count: 3 })
+    expect(resultShapeOf('hello')).toEqual({ size: 7 })
+    expect(resultShapeOf(null)).toEqual({ size: 4 })
   })
 
-  test('5개 초과 array truncate + 마커', () => {
-    const big = Array.from({ length: 12 }, (_, i) => i)
-    const r = summarizeToolResult(big)
-    expect(r.length).toBe(6) // 5 + marker
-    expect(r[5]).toMatch(/items\]/)
+  test('거절과 실패는 코드로', () => {
+    expect(resultShapeOf({ rejected: true, error: 'query-failed', message: '김급여 unauthorized' })).toMatchObject({ errorCode: 'query-failed' })
+    expect(JSON.stringify(resultShapeOf({ error: 'tool-failed', message: 'the ledger of 김급여' }))).not.toMatch(/김급여/)
   })
 
-  test('object 의 nested 값 재귀 요약', () => {
-    const o = { a: 1, b: { c: 'x'.repeat(500) } }
-    const r = summarizeToolResult(o)
-    expect(r.a).toBe(1)
-    expect(r.b.c).toMatch(/…/)
-  })
-
-  test('depth 3 초과 object 는 키 목록 string 으로 압축', () => {
-    const deep = { l1: { l2: { l3: { l4: { x: 1, y: 2, z: 3 } } } } }
-    const r = summarizeToolResult(deep)
-    // l1 (d0) → l2 (d1) → l3 (d2) 까지 object 유지, l3 의 값은 d3 진입 → string
-    expect(typeof r.l1).toBe('object')
-    expect(typeof r.l1.l2).toBe('object')
-    // l3 자체가 d3 에서 평가되어 키 목록 문자열로 압축됨
-    expect(typeof r.l1.l2.l3).toBe('string')
-    expect(r.l1.l2.l3).toMatch(/^\{l4/)
+  test('키가 많으면 앞의 것만 이름으로, 나머지는 수로', () => {
+    const wide = Object.fromEntries(Array.from({ length: 25 }, (_, i) => [`k${i}`, i]))
+    expect(resultShapeOf(wide)).toMatchObject({ moreKeys: 5 })
+    expect(resultShapeOf(wide).keys).toHaveLength(20)
   })
 })

package/server/service/assistant.ts

@@ -41,7 +41,7 @@ import { buildComponentStyleSummary } from './styling/read-tools'
 import {
   runAgenticLoop,
   formatAbortNotice,
-  summarizeToolResult,
+  resultShapeOf,
   registryToolGate,
   extendToolGate,
   type ToolGate as NeutralToolGate,
@@ -270,7 +270,8 @@ export class DefaultBoardAIAssistant implements BoardAIAssistant {
         /* 문서에서 부품을 꺼내는 일은 **여기서** 한다 — 루프는 board 의 모양을 모른다. */
         validateWriteToolCall: (tc, ctx) => validateWriteToolCall(tc, (ctx.board?.components ?? []) as any[]),
         toolCallToOp: toolCallToSceneActionOp,
-        summarizeToolResult
+        /* The conversation row keeps each answer's shape, never its values (ADR-0061 decision 8). */
+        summarizeToolResult: resultShapeOf
       },
       context: { board: currentBoard, selectedRefids }
     })
@@ -2034,7 +2035,7 @@ export function listLayoutTemplateNames(): string[] {
 }
 
 /*
- * 중단 통지 · 도구 결과 요약 — **공용 패키지로 옮겼다** (2026-09-10).
+ * 중단 통지 · 도구 결과의 모양 — **공용 패키지에 있다** (2026-09-10, 결과는 값 대신 모양 2026-10-17).
  *
  * 둘 다 보드를 모르는 함수였다. 옮긴 이유는 중복이 아니라 **한쪽만 갖고 있었던 것**이다:
  * `twin-ai` 는 `summarizeToolResult: v => v` 로 요약을 아예 안 하고 있었다. 한 제품은 압축해서
@@ -2045,7 +2046,7 @@ export function listLayoutTemplateNames(): string[] {
  *
  * 여기 남은 것은 재수출이다. 이 패키지를 지나는 소비처가 이름을 그대로 쓰게 한다.
  */
-export { formatAbortNotice, summarizeToolResult }
+export { formatAbortNotice, resultShapeOf }
 
 
 export function isWriteTool(name: string): boolean {