@things-factory/figure-service 10.1.6 → 10.1.8

package/server/service/figure/figure-tools.ts

@@ -55,7 +55,8 @@ function getDeps(): any {
       proposeFigure: propose.proposeFigure,
       compile: model.compile,
       costOf: model.costOf,
-      scoreOf: model.scoreOf
+      scoreOf: model.scoreOf,
+      validate: model.validate
     }
   }
   return _deps
@@ -92,6 +93,51 @@ async function figureInDomain(figureId: string, domainId: string): Promise<any>
   return found
 }
 
+/**
+ * 모델러가 아직 저장하지 않은 source를 도구가 읽는 유일한 길.
+ *
+ * host context는 브라우저가 보낸 값이므로 권한 근거가 될 수 없다. 먼저 `row`를 현재 도메인에서
+ * 찾은 뒤에만 보며, Figure의 불변 type과 형식 검증을 다시 확인한다. 검증되지 않은 큰 객체를 LLM
+ * 프롬프트로 옮기면 비용만 태우고, DB에 있던 Figure와 다른 것을 고친다고 잘못 말할 수 있다.
+ */
+function sourceForTool(row: any, host: any): { source: any; draft: boolean } {
+  const draft = host?.draftSource
+  if (!draft) {
+    return { source: row.source ? JSON.parse(row.source) : null, draft: false }
+  }
+
+  if (typeof draft !== 'object' || Array.isArray(draft)) {
+    throw new Error('the unsaved draft is not a FigureSource object')
+  }
+
+  const serialized = JSON.stringify(draft)
+  if (serialized.length > 100_000) {
+    throw new Error('the unsaved draft is too large to safely review')
+  }
+
+  const { validate } = getDeps()
+  const checked = validate(draft)
+  if (checked.errors.length > 0) {
+    throw new Error(`the unsaved draft is invalid: ${checked.errors.map((one: any) => one.message).join(' · ')}`)
+  }
+  if (draft.type !== row.type) {
+    throw new Error(`the unsaved draft type '${draft.type}' does not match Figure '${row.type}'`)
+  }
+
+  return { source: draft, draft: true }
+}
+
+/**
+ * 후보를 냈던 바로 그 Figure의 세션 반응만 후보 생성기에 넘긴다.
+ *
+ * 도크의 host context는 브라우저 값이므로 이 함수는 접근 권한을 판단하지 않는다. 이미 현재
+ * 도메인에서 Figure를 찾은 뒤, `proposeFigure`가 다시 길이·형식을 정리할 참고 데이터만 건넨다.
+ */
+function feedbackForTool(host: any, figureId?: string): unknown[] | undefined {
+  if (!figureId || host?.figureId !== figureId || !Array.isArray(host?.proposalFeedback)) return undefined
+  return host.proposalFeedback.slice(-5)
+}
+
 /**
  * 무엇을 고치라는 것인지 대화면 문맥에서 해소한다.
  *
@@ -163,21 +209,22 @@ function figureToolSpecs(): ToolSpec[] {
         const figureId = figureIdFrom(args, ctx?.state?.host)
         const row = await figureInDomain(figureId, domainId)
 
-        if (!row.source) {
+        const current = sourceForTool(row, ctx?.state?.host)
+        if (!current.source) {
           // 저장된 정본이 없는 상태가 실제로 있다(새로 만들고 저장 안 한 것). 지어내지 않는다.
           return { id: row.id, type: row.type, name: row.name, source: null, cost: null }
         }
 
         const { compile, costOf, scoreOf } = getDeps()
-        const source = JSON.parse(row.source)
-        const blueprint = compile(source)
+        const blueprint = compile(current.source)
 
         return {
           id: row.id,
           type: row.type,
           name: row.name,
           state: row.state,
-          source,
+          source: current.source,
+          draft: current.draft,
           cost: costOf(blueprint, 100),
           score: scoreOf(blueprint)
         }
@@ -196,8 +243,9 @@ function figureToolSpecs(): ToolSpec[] {
         const figureId = figureIdFrom(args, ctx?.state?.host)
         const row = await figureInDomain(figureId, domainId)
 
+        const current = sourceForTool(row, ctx?.state?.host)
         const { inspectSource } = getDeps()
-        return inspectSource(row.source)
+        return { ...inspectSource(current.source ? JSON.stringify(current.source) : null), draft: current.draft }
       }
     },
     {
@@ -244,7 +292,7 @@ function figureToolSpecs(): ToolSpec[] {
           const open = typeof args?.figureId === 'string' ? args.figureId : ctx?.state?.host?.figureId
           if (typeof open === 'string' && open.trim()) {
             const row = await figureInDomain(open.trim(), domainId)
-            if (row.source) base = JSON.parse(row.source)
+            base = sourceForTool(row, ctx?.state?.host).source ?? undefined
           }
         }
 
@@ -258,11 +306,14 @@ function figureToolSpecs(): ToolSpec[] {
         }
 
         const { proposeFigure } = getDeps()
+        const feedback = feedbackForTool(ctx?.state?.host, base ? figureIdFrom(args, ctx?.state?.host) : undefined)
         const result = await proposeFigure({
           prompt,
           palette,
           base,
-          type: typeof args?.type === 'string' ? args.type : undefined
+          type: typeof args?.type === 'string' ? args.type : undefined,
+          feedback,
+          refine: !!feedback?.length
         })
 
         /*
@@ -283,6 +334,7 @@ function figureToolSpecs(): ToolSpec[] {
           source: result.source,
           score: result.score,
           cost: result.cost,
+          quality: result.quality,
           violations: result.violations,
           attempts: result.attempts
         }

package/server/service/figure/figure-type.ts

@@ -22,6 +22,9 @@ export class NewFigure {
   @Field({ nullable: true, description: 'Classification code used to group figures in the palette.' })
   category?: string
 
+  @Field(type => [String], { nullable: true, description: 'Free-form tags. A figure may carry several.' })
+  tags?: string[]
+
   @Field({ description: 'The canonical FigureSource JSON.' })
   source: string
 
@@ -52,6 +55,13 @@ export class FigurePatch {
   @Field({ nullable: true, description: 'The new classification code.' })
   category?: string
 
+  /*
+    빈 배열과 안 준 것을 가른다. `undefined` 는 「건드리지 않는다」, `[]` 는 「전부 지운다」다.
+    둘을 뭉개면 태그를 다 지우려는 저작자가 지울 방법을 잃는다.
+  */
+  @Field(type => [String], { nullable: true, description: 'The new tags. An empty list clears them; omitting the field leaves them.' })
+  tags?: string[]
+
   @Field({ nullable: true, description: 'The new FigureSource JSON.' })
   source?: string
 
@@ -118,6 +128,31 @@ export class FigureSaveResult {
  * 것들이라 그 칸이 필요 없었다. 발행 판정은 막는 것과 알리는 것을 함께 내므로, 어느 쪽인지를 값이
  * 스스로 말해야 한다 — 코드를 보고 화면이 다시 분류하면 규칙이 두 벌이 된다.
  */
+/** A measured, one-anchor repair candidate for a release-gate sizing split. */
+@ObjectType({ description: 'A reversible anchor change measured to reduce a sizing split.' })
+export class FigureGateFix {
+  @Field({ description: 'Sizing test that this candidate improves, for example y or x-y.' })
+  findingWay: string
+
+  @Field({ description: 'Stable FigureSource part name to change.' })
+  part: string
+
+  @Field({ description: 'Axis whose anchor will change.' })
+  axis: string
+
+  @Field({ nullable: true, description: 'Current explicit anchor, if one is present.' })
+  from?: string
+
+  @Field({ description: 'Suggested explicit anchor: min, center, max, or span.' })
+  to: string
+
+  @Field(type => Int, { description: 'Number of blocking sizing-split checks removed by this one change.' })
+  resolved: number
+
+  @Field({ description: 'True only when every blocking sizing check is clear after remeasurement.' })
+  safe: boolean
+}
+
 @ObjectType({ description: 'One finding from the release gate.' })
 export class FigureGateFinding {
   @Field({ description: 'Stable code — figure-model validation codes, or sizing-splits / sizing-loose.' })
@@ -135,6 +170,12 @@ export class FigureGateFinding {
   @Field({ nullable: true, description: 'Where it points: part names that came apart, or a path in the source.' })
   at?: string
 
+  @Field(type => [FigureGateFix], {
+    nullable: true,
+    description: 'Measured repair candidates. They are previews until the author explicitly applies one.'
+  })
+  fixes?: FigureGateFix[]
+
   @Field({ description: 'True when this finding blocks the release.' })
   blocking: boolean
 }

package/server/service/figure/figure.ts

@@ -89,11 +89,31 @@ export class Figure {
   @Field({ nullable: true, description: 'A detailed description for the figure.' })
   description?: string
 
-  /** 분류 코드. 팔레트에서 묶어 보이는 기준. */
+  /**
+   * 분류 하나. **팔레트에서 묶어 보이는 기준이다.**
+   *
+   * 도형 하나에 하나뿐이다 — 팔레트는 목록을 한 벌로 나눠 보여 주므로, 한 도형이 두 묶음에
+   * 동시에 있으면 저작자가 같은 것을 두 번 만난다.
+   */
   @Column({ nullable: true })
-  @Field({ nullable: true, description: 'Classification code used to group figures in the palette.' })
+  @Field({ nullable: true, description: 'Classification code used to group figures in the palette. One per figure.' })
   category?: string
 
+  /**
+   * 태그. **여럿 붙는다.**
+   *
+   * 분류와 다른 일을 한다. 분류는 「이것이 무엇인가」를 한 번 정하는 것이고, 태그는 「어디에
+   * 걸리는가」를 여러 갈래로 적는 것이다 — 컨베이어이면서 반출 구역의 것일 수 있다. 그래서
+   * 분류로는 못 하는 찾기가 된다.
+   *
+   * `simple-json` 으로 담는다. 사내 선례가 그것이고(`ops-master` · `organization` ·
+   * `personalization`), 드라이버마다 배열 타입이 다른 것을 피한다.
+   */
+  /* `simple-json` 과 `default: null` 은 사내 선례 그대로다 — board-template · attachment 가 같은 모양이다. */
+  @Column('simple-json', { nullable: true, default: null })
+  @Field(type => [String], { nullable: true, description: 'Free-form tags. A figure may carry several.' })
+  tags?: string[]
+
   @Column({ nullable: true, default: 'draft' })
   @Field({ nullable: true, description: "Lifecycle state — 'draft' figures do not appear in the board palette." })
   state?: 'draft' | 'released'