@things-factory/figure-service 10.1.20 → 10.1.24

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@things-factory/figure-service",
-  "version": "10.1.20",
+  "version": "10.1.24",
   "publishConfig": {
     "access": "public",
     "@things-factory:registry": "https://registry.npmjs.org"
@@ -23,15 +23,15 @@
     "migration:create": "node ../../node_modules/typeorm/cli.js migration:create ./server/migrations/migration"
   },
   "dependencies": {
-    "@hatiolab/figure-model": "^0.1.20",
+    "@hatiolab/figure-model": "^0.1.21",
     "@operato/typeorm-history": "^10.0.0",
     "@things-factory/ai-client-base": "^10.1.20",
-    "@things-factory/auth-base": "^10.1.20",
+    "@things-factory/auth-base": "^10.1.24",
     "@things-factory/env": "^10.1.20",
     "@things-factory/shell": "^10.1.20",
     "graphql-scalars": "^1.22.4",
     "graphql-upload": "^15.0.2",
     "koa-router": "^7.4.0"
   },
-  "gitHead": "6bd01660a4b28554bbe6ae4ee58f3ff844c5af69"
+  "gitHead": "0d88a504e0ffa63cd91279b88c6998d4bf1b3d86"
 }

package/server/service/figure/figure-propose.test.ts

@@ -13,7 +13,9 @@
 import {
   ANCHOR_RULES,
   DETAIL_LEVELS,
+  CAPABILITY_NEEDS,
   CHANNEL_PATHS,
+  FIGURE_CAPABILITIES,
   FIGURE_PLACEMENTS,
   INTERPOLATIONS,
   LABEL_WHENS,
@@ -570,6 +572,44 @@ describe('정책을 넘은 후보는 막지 않고 알린다', () => {
  * These guard the wording, because the wording is the defect. A prompt that hides the one door
  * leaves the model to invent a door.
  */
+/*
+ * The capability vocabulary, which the prompt used to leave open.
+ *
+ * It said "as plain names the scene knows" -- an instruction to guess. The model guessed
+ * `capabilities: ['hoist']`, and because the list lived only in things-scene, figure-model's
+ * validate could not refuse it: the asset passed validation, passed publication, and threw at
+ * render. ADR-0051 (4) moved the list into the format, so the prompt can now state it.
+ */
+describe('능력 이름을 형식에서 받아 적는다 — 「씬이 아는 이름」은 지어내라는 말이다', () => {
+  const prompt = () => rulesPrompt(PALETTE)
+
+  it('닫힌 목록이라고 못 박는다', () => {
+    const text = prompt()
+    expect(text).toMatch(/capabilities \(top level\)[\s\S]*CLOSED LIST/)
+    expect(text).not.toMatch(/plain names the scene knows/)
+  })
+
+  it('목록을 형식에서 읽는다 — 손으로 적지 않는다', () => {
+    /* 이름 하나를 여기 적어 두면 형식에 능력이 늘 때 이 시험이 조용히 통과한다. */
+    const text = prompt()
+    for (const name of FIGURE_CAPABILITIES) {
+      expect(text).toContain(name)
+    }
+  })
+
+  it('혼자 못 서는 능력도 함께 적는다 — 그것도 형식의 선언이다', () => {
+    const text = prompt()
+    for (const [name, needs] of Object.entries(CAPABILITY_NEEDS)) {
+      expect(text).toContain(`${name} cannot stand alone`)
+      for (const one of needs) expect(text).toContain(one)
+    }
+  })
+
+  it('움직임을 능력으로 답하지 말라고 한다 — 실제로 나왔던 답이다', () => {
+    expect(prompt()).toMatch(/Do NOT answer it with a capability/)
+  })
+})
+
 describe('값이 만드는 자세를 parameters 로 가르친다', () => {
   const prompt = () => rulesPrompt(PALETTE)
 

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

@@ -1,7 +1,9 @@
 import {
   ANCHOR_RULES,
   AXES,
+  CAPABILITY_NEEDS,
   CHANNEL_PATHS,
+  FIGURE_CAPABILITIES,
   DETAIL_LEVELS,
   INTERPOLATIONS,
   LABEL_WHENS,
@@ -144,7 +146,8 @@ export function rulesPrompt(palette: string[]): string {
     'Answer with a single JSON object and nothing else — no prose, no code fence.',
     '',
     'The JSON is a FigureSource:',
-    '  { type, base: {x,y,z}, placement?, detailLevel, parts: [...], animations?: [...], capabilities?: [...] }',
+    '  { type, base: {x,y,z}, placement?, detailLevel, parts: [...],',
+    '    animations?: [...], parameters?: [...], capabilities?: [...] }',
     '',
     'A part is:',
     '  { name, primitive, transform: { position: {x,y,z}, size: {x,y,z}, rotation?: {x,y,z} },',
@@ -162,6 +165,10 @@ export function rulesPrompt(palette: string[]): string {
     `  anchor.<axis>   ${ANCHOR_RULES.join(' | ')}   (axis is one of ${AXES.join(', ')})`,
     `  repeat.axis     ${AXES.join(' | ')}`,
     `  capability.roles  ${PART_ROLES.join(' | ')}`,
+    `  capabilities      ${FIGURE_CAPABILITIES.join(' | ')}`,
+    ...Object.entries(CAPABILITY_NEEDS).map(
+      ([name, needs]) => `    ${name} cannot stand alone; give it ${(needs as readonly string[]).join(' or ')} as well`
+    ),
     `  label.when      ${LABEL_WHENS.join(' | ')}`,
     `  animation.path  ${CHANNEL_PATHS.join(' | ')}`,
     `  animation.interpolation ${INTERPOLATIONS.join(' | ')}`,
@@ -224,7 +231,8 @@ export function rulesPrompt(palette: string[]): string {
     '    A part may hold several roles: an AGV deck is a slot that things also enter and leave.',
     '    Omit capacity when the part repeats — the count then follows the instance size.',
     '    direction is the way a port faces, in the part\'s own axes; default is +z.',
-    '  capabilities (top level) is what the whole figure can do, as plain names the scene knows.',
+    '  capabilities (top level) is what the whole figure can do. The names are a CLOSED LIST',
+    `    (below) -- anything else is refused, so do not invent one for a motion or a shape.`,
     '  materialSlot + material.emissive make a status lamp: the token is the OFF look and',
     '  emissive { token, intensity, on? } is the ON look. A dark lamp is not a red lamp dimmed.',
     '  label is a place a number is shown on the figure: { name, source, when }. The figure says',