@things-factory/figure-ui 10.1.167 → 10.1.169

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@things-factory/figure-ui",
-  "version": "10.1.167",
+  "version": "10.1.169",
   "publishConfig": {
     "access": "public",
     "@things-factory:registry": "https://registry.npmjs.org"
@@ -27,8 +27,8 @@
     "clean": "npm run clean:server && npm run clean:client"
   },
   "dependencies": {
-    "@hatiolab/figure-model": "^0.6.6",
-    "@hatiolab/things-scene": "^10.1.86",
+    "@hatiolab/figure-model": "^0.7.1",
+    "@hatiolab/things-scene": "^10.1.90",
     "@material/web": "^2.0.0",
     "@operato/data-grist": "^10.0.0",
     "@operato/graphql": "^10.0.0",
@@ -41,11 +41,11 @@
     "@operato/slot-layout-editor": "^10.29.25",
     "@operato/styles": "^10.0.0",
     "@operato/utils": "^10.0.0",
-    "@things-factory/ai-assistant": "^10.1.167",
-    "@things-factory/figure-service": "^10.1.167",
-    "@things-factory/figure-viewer": "^10.1.166",
+    "@things-factory/ai-assistant": "^10.1.169",
+    "@things-factory/figure-service": "^10.1.169",
+    "@things-factory/figure-viewer": "^10.1.169",
     "@things-factory/setting-base": "^10.1.167",
     "three": "^0.185.1"
   },
-  "gitHead": "86283c5d35042880be6827f777067fc5e6417eed"
+  "gitHead": "9d71d02b443f3e0080b25de1768b982c51a64111"
 }

package/test/ai-proposal.test.ts

@@ -9,7 +9,8 @@ import { fileURLToPath } from 'node:url'
 import { createFigureAsset, addFigureBox, serializeFigureAsset } from '@hatiolab/figure-model'
 
 import { createFigureProductSession } from '../client/modeller/product-session.js'
-import { pendNewFigure, runFigureProposal, takePendingNewFigure, type FigureApplyDetail } from '../client/modeller/ai-proposal.js'
+import type { FigureProposalStep } from '@hatiolab/figure-model'
+import { askingToLeave, pendNewFigure, runFigureProposal, takeFigureOffer, takePendingNewFigure, type FigureApplyDetail } from '../client/modeller/ai-proposal.js'
 
 const ko = JSON.parse(readFileSync(fileURLToPath(new URL('../translations/ko.json', import.meta.url)), 'utf8'))
 const say = (key: string, params: Record<string, string> = {}) => {
@@ -51,9 +52,18 @@ test('a person\'s edit after the proposal was asked makes it stale -- refused, n
   assert.equal(s.read().name, 'A')
 })
 
-const taker = (result: Partial<FigureApplyDetail>) => ({
+const taker = (result: Partial<FigureApplyDetail>, saved: Array<{ figureId: string; type?: string; name?: string }> = []) => ({
   offered: [] as FigureApplyDetail[],
   opened: [] as unknown[],
+  asked: [] as string[],
+  shown: [] as string[],
+  async savedInto(key: string) {
+    this.asked.push(key)
+    return saved
+  },
+  openSaved(figureId: string) {
+    this.shown.push(figureId)
+  },
   offer(detail: FigureApplyDetail) {
     this.offered.push(detail)
     return Object.assign(detail, result)
@@ -72,12 +82,26 @@ test('applied: said with how many commands, and the card carries the version to
   assert.equal(target.offered[0]!.revision, 3)
 })
 
+test('a revision carries the figure it was made for, and only a revision does (audit N3)', async () => {
+  const target = taker({ applied: true })
+  await runFigureProposal({ steps, figureId: 'a', context: 'c', revision: 0 }, target, say)
+  assert.equal(target.offered[0]!.figureId, 'a')
+  const fresh = taker({ applied: true })
+  await runFigureProposal({ steps, create: true, figureId: 'a' }, fresh, say)
+  assert.equal(fresh.offered[0]!.figureId, undefined)
+})
+
 test('refused: a stale version is said as what happened; any other refusal carries its reason', async () => {
   const stale = await runFigureProposal({ steps }, taker({ applied: false, code: 'REVISION_CONFLICT', error: 'editor changed' }), say)
   assert.equal(stale.verdict, 'refused')
   assert.match(stale.note!, /모델이 바뀌어/)
   const other = await runFigureProposal({ steps }, taker({ applied: false, code: 'ATTACH_FACE', error: 'faces along x' }), say)
-  assert.match(other.note!, /적용하지 않았습니다: faces along x/)
+  /* Said from the code in the reader's words, as the modeller says an edit it refuses -- never the kernel's English (audit N6). */
+  assert.match(other.note!, /적용하지 않았습니다: 이 편집은 할 수 없습니다\. \(ATTACH_FACE\)/)
+  assert.doesNotMatch(other.note!, /faces along x/)
+  const busy = await runFigureProposal({ steps }, taker({ applied: false, code: 'SESSION_SAVING', error: 'save first' }), say)
+  assert.match(busy.note!, /저장이 끝난 뒤/)
+  assert.doesNotMatch(busy.note!, /save first/)
 })
 
 test('a revision with no modeller to take it is not applied anywhere else', async () => {
@@ -94,8 +118,71 @@ test('a new figure with no new modeller on screen opens one', async () => {
   assert.equal(target.opened.length, 1)
 })
 
-test('an empty card is not applied', async () => {
-  assert.equal((await runFigureProposal({ steps: [] }, taker({ applied: true }), say)).verdict, 'unavailable')
+/*
+  The same proposal is one figure (ADR-0061 decision 3, ruled 2026-10-16): a new figure taken from it in the modeller keeps
+  the figure's own creation id, so what says the proposal was already made is the save's record of it, read before the
+  card runs again.
+*/
+test('a new figure\'s card run again after it was saved opens that figure, and makes none', async () => {
+  const K = '11111111-1111-4111-8111-111111111111'
+  const target = taker({ applied: true }, [{ figureId: 'R', type: 'box', name: 'Box with lid' }])
+  const out = await runFigureProposal({ key: K, steps, create: true }, target, say)
+  assert.deepEqual(target.asked, [K])
+  assert.deepEqual(target.shown, ['R'])
+  assert.equal(target.offered.length + target.opened.length, 0, 'not offered to a modeller, no new one opened')
+  assert.deepEqual(out, { verdict: 'accepted', note: say('figure.text.proposal-saved-into', { names: 'Box with lid' }) })
+})
+
+test('a new figure\'s card not saved anywhere yet runs as before; a revision is not looked up', async () => {
+  const K = '11111111-1111-4111-8111-111111111111'
+  const fresh = taker({})
+  assert.equal((await runFigureProposal({ key: K, steps, create: true }, fresh, say)).verdict, 'accepted')
+  assert.deepEqual({ asked: fresh.asked, opened: fresh.opened.length }, { asked: [K], opened: 1 })
+  const revision = taker({ applied: true }, [{ figureId: 'R' }])
+  await runFigureProposal({ key: K, steps, context: 'c', revision: 3 }, revision, say)
+  assert.deepEqual({ asked: revision.asked, shown: revision.shown }, { asked: [], shown: [] })
+})
+
+/*
+  Open (ADR-0061 decision 3, ruled 2026-10-16, not prevented): the same new figure's card run in another tab before the
+  first figure is saved finds no record yet, and opens a second figure under the proposal's key.
+*/
+test('the same new figure\'s card in another tab before the first save makes a second figure under its key', async () => {
+  const K = '11111111-1111-4111-8111-111111111111'
+  const ids: string[] = []
+  const tab = () =>
+    createFigureProductSession({
+      load: store.load,
+      save: async (q: { id?: string; source: string; creationId?: string }) => (ids.push(q.creationId!), { id: q.id ?? q.creationId!, source: q.source })
+    })
+  const made = [{ kind: 'add-part', part: { id: 'lid', kind: 'box', dimensions: { width: 100, height: 10, depth: 100 } } }] as any[]
+  const first = tab()
+  first.startNew()
+  const detail: FigureApplyDetail = { key: K, steps: made, create: true }
+  const here = { ...taker({}), offer: (d: FigureApplyDetail) => (takeFigureOffer(first, d, { onSavedFigure: false, say }), d) }
+  assert.equal((await runFigureProposal(detail, here, say)).verdict, 'accepted')
+
+  const second = tab()
+  const there = {
+    ...taker({}),
+    async openNew(s: FigureProposalStep[], key?: string) {
+      second.startNew()
+      second.createUnder(key!)
+      second.applyProposal(s, second.read().revision!, second.read().context!, key)
+      return { applied: true }
+    }
+  }
+  assert.equal((await runFigureProposal({ key: K, steps: made, create: true }, there, say)).verdict, 'accepted')
+  await first.save()
+  await second.save()
+  assert.deepEqual(ids, [first.read().creationId, K])
+  assert.notEqual(ids[0], ids[1], 'two figures')
+})
+
+test('an empty card is not applied, and says so (audit N7)', async () => {
+  const out = await runFigureProposal({ steps: [] }, taker({ applied: true }), say)
+  assert.equal(out.verdict, 'unavailable')
+  assert.match(out.note ?? '', /명령이 없어/)
 })
 
 test('a new figure\'s steps wait for the page once, and lapse if no page takes them', async () => {
@@ -110,3 +197,26 @@ test('a new figure\'s steps wait for the page once, and lapse if no page takes t
   assert.deepEqual(await lapsing, { applied: false, code: 'not-opened' })
   assert.equal(takePendingNewFigure(), undefined, 'a later new figure does not receive them')
 })
+
+/*
+  A new figure's card run from a modeller with unsaved edits waits behind the question whether to leave them (figure
+  audit 2026-10-16 N4). Its clock stopped for that question: before, a person who took longer than it to say leave lost
+  their edits and got a blank new figure, the steps gone; one who said stay got "could not be opened" ten seconds later.
+*/
+test('a new figure waiting behind the question whether to leave: no clock while asked, said at once when the person stays', async () => {
+  const leaving = pendNewFigure(steps, undefined, 10)
+  const answer = askingToLeave()!
+  await new Promise(resolve => setTimeout(resolve, 40))
+  answer(true)
+  const taken = takePendingNewFigure()
+  assert.ok(taken, 'still waiting for the page that opens')
+  taken!.settle({ applied: true })
+  assert.deepEqual(await leaving, { applied: true })
+
+  const staying = pendNewFigure(steps, undefined, 10_000)
+  askingToLeave()!(false)
+  assert.deepEqual(await staying, { applied: false, code: 'not-opened' })
+  assert.equal(takePendingNewFigure(), undefined)
+
+  assert.equal(askingToLeave(), undefined, 'nothing waiting: nothing to hold')
+})

package/test/browser/acceptance-page.js

@@ -202,7 +202,9 @@ async function start() {
   })
   const dock = {
     offer: detail => (window.dispatchEvent(new CustomEvent('figure-apply', { detail })), detail),
-    openNew: async () => ({ applied: false, code: 'not-opened' })
+    openNew: async () => ({ applied: false, code: 'not-opened' }),
+    savedInto: async () => [],
+    openSaved: () => {}
   }
   window.acceptance = {
     open: async id => (await session.open(id), editor.requestUpdate(), editor.dispatchEvent(new CustomEvent('changed'))),

package/test/browser/product-page.check.mjs

@@ -541,14 +541,30 @@ describe('the page around the figure: the shell, the dock, the properties', () =
     assert.equal(await beforeUnload(), true)
   })
 
-  test('a new figure gives the dock nothing to talk about, and the rail does not mark the catalogue', async () => {
+  /*
+    A figure not saved yet is a subject (ADR-0061 decision 3): its conversation is anchored on the creation id its first
+    save inserts the row under, so the first save changes nothing the dock knows -- the same conversation goes on.
+  */
+  test('a new figure: the dock talks about it by its creation id, and its first save keeps the conversation', async () => {
     await press('button[save]', 'save')
     await until(() => !product.view().dirty, 'saved')
     await read(() => assistant.forget())
     await open(null)
-    assert.deepEqual(await said(), [{ call: 'clear' }], 'the saved figure\'s conversation taken back, none given')
-    assert.equal(await read(() => assistant.surface), null)
+    const creationId = await read(() => product.page().session.read().creationId)
+    assert.match(creationId, /^[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/)
+    assert.deepEqual((await said()).filter(s => s.call !== 'notify'), [{ call: 'provide', key: `figure:${creationId}` }])
+    assert.deepEqual(await read(() => assistant.surface.anchor()), { anchorType: 'figure', anchorId: creationId })
+    assert.deepEqual(await read(() => { const { figureId, editorMode } = assistant.surface.hostContext(); return { figureId, editorMode } }), { figureId: creationId, editorMode: 'new' })
     assert.equal((await read(() => product.context())).menu, undefined)
+    await rename('Gate')
+    await read(() => { assistant.forget(); product.forget() })
+    await press('button[save]', 'save')
+    await press('figure-first-save >>> md-filled-button[slot=confirm]', 'the first save')
+    await until(() => product.view().id && !product.view().dirty, 'saved')
+    assert.equal((await callsTo('saveFigure'))[0].creationId, creationId)
+    assert.equal(await read(() => product.view().id), creationId)
+    assert.deepEqual((await said()).filter(s => s.call !== 'notify'), [], 'no new conversation, none taken back')
+    assert.deepEqual(await read(() => { const { figureId, editorMode } = assistant.surface.hostContext(); return { figureId, editorMode } }), { figureId: creationId, editorMode: 'existing' })
   })
 
   test('inactive: the surface is taken back', async () => {
@@ -643,6 +659,7 @@ describe('AI proposals reaching the page', () => {
     const outcome = await read(key => assistant.surface.proposalOutcome(key), K1)
     assert.deepEqual(outcome, { verdict: 'accepted', note: 'figure.text.proposal-saved-into {"names":"Door"}' })
     assert.deepEqual(await callsTo('figureProposalOutcome'), [{ proposalKey: K1 }])
+    await read(() => product.answer('figureProposalOutcome'))
   })
 
   test("a new figure proposed: the modeller opens a new one, takes the steps under the proposal's key, and the dock is told", async () => {
@@ -661,6 +678,7 @@ describe('AI proposals reaching the page', () => {
       outcome: { verdict: 'accepted', applied: 1, note: 'figure.text.proposal-applied {"n":"1"}' },
       told: [{ level: 'info', note: 'figure.text.proposal-applied {"n":"1"}' }]
     })
+    assert.deepEqual(await callsTo('figureProposalOutcome'), [{ proposalKey: K2 }], 'asked whether it was saved before')
     assert.deepEqual((await read(() => product.navigations())).map(n => n.path), ['/figure-modeller'])
     const made = await read(() => product.view())
     assert.equal(made.id, undefined)
@@ -671,6 +689,48 @@ describe('AI proposals reaching the page', () => {
     const [saved] = await callsTo('saveFigure')
     assert.deepEqual({ creationId: saved.creationId, proposalKeys: saved.proposalKeys }, { creationId: K2, proposalKeys: [K2] })
   })
+
+  /*
+    The draft the dock made on a new figure stays that figure's conversation (ADR-0061 decision 3, ruled 2026-10-16): the
+    figure keeps its creation id, and the save records the proposal. Run again once saved, the card opens that figure --
+    the record, not the row id, is what makes one proposal one figure.
+  */
+  test('a new figure\'s proposal taken on a new figure keeps its conversation; run again after the save, it opens that figure', async () => {
+    /* After the one above: the figure on the page is saved, so a new one opens without a question. */
+    await until(() => product.view().id && !product.view().dirty, 'the one above saved')
+    await open(null)
+    const creationId = await read(() => product.page().session.read().creationId)
+    await read(() => { assistant.forget(); product.forget() })
+    const run = () =>
+      read(
+        async ({ key, steps }) => {
+          const told = []
+          const outcome = await assistant.handlers.proposeFigureCandidate({ key, steps, create: true }, { notify: (level, note) => told.push({ level, note }) })
+          return { outcome, told }
+        },
+        { key: K1, steps: LID }
+      )
+    assert.deepEqual((await run()).outcome, { verdict: 'accepted', applied: 1, note: 'figure.text.proposal-applied {"n":"1"}' })
+    assert.equal(await read(() => product.page().session.read().creationId), creationId, 'the proposal\'s key is not the figure\'s id')
+    assert.deepEqual((await said()).filter(s => s.call !== 'notify'), [], 'the same conversation: no surface given or taken back')
+    assert.deepEqual(await read(() => product.navigations()), [], 'taken where it was asked')
+    await press('button[save]', 'save')
+    await press('figure-first-save >>> md-filled-button[slot=confirm]', 'the first save')
+    await until(() => product.view().id && !product.view().dirty, 'saved')
+    const [saved] = await callsTo('saveFigure')
+    assert.deepEqual({ creationId: saved.creationId, proposalKeys: saved.proposalKeys }, { creationId, proposalKeys: [K1] })
+
+    await open(null)
+    await read(() => product.forget())
+    const before = await read(() => product.rows().length)
+    const names = await read(id => product.rows().find(r => r.id === id).name, creationId)
+    const note = `figure.text.proposal-saved-into ${JSON.stringify({ names })}`
+    assert.deepEqual(await run(), { outcome: { verdict: 'accepted', note }, told: [{ level: 'info', note }] })
+    assert.deepEqual(await callsTo('figureProposalOutcome'), [{ proposalKey: K1 }])
+    assert.deepEqual((await read(() => product.navigations())).map(n => n.path), [`/figure-modeller/${creationId}`])
+    assert.equal(await read(() => product.rows().length), before, 'no second figure')
+    assert.deepEqual(await callsTo('saveFigure'), [])
+  })
 })
 
 describe('the catalogue preview', () => {

package/test/browser/product-page.js

@@ -61,6 +61,8 @@ const calls = []
 /* Answers a check wants instead of the server's: field -> function(variables) returning data or throwing. */
 const overrides = new Map()
 let made = 0
+/* What the saves recorded of AI proposals: { proposalKey, figureId }. */
+const records = []
 
 function row(id) {
   const found = rows.get(id)
@@ -71,8 +73,10 @@ function row(id) {
 const answers = {
   figure: ({ id }) => ({ __typename: 'Figure', properties: null, tags: null, category: null, description: null, score: null, triangles: null, groups: null, thumbnailUpdatedAt: null, updatedAt: null, updater: null, thumbnailHash: null, ...row(id) }),
   figureTypeNames: () => [...rows.values()].map(r => r.type),
-  saveFigure: async ({ id, source, type, creationId, thumbnail }) => {
+  saveFigure: async ({ id, source, type, creationId, thumbnail, proposalKeys }) => {
     const saved = id ? row(id) : { id: creationId ?? `made-${++made}`, namespace: null, type: type ?? `figure-${made}`, state: 'draft', version: 0 }
+    /* The save's record of the proposals its source holds, as the server keeps it (figure-proposal.ts). */
+    for (const key of proposalKeys ?? []) if (!records.some(r => r.proposalKey === key && r.figureId === saved.id)) records.push({ proposalKey: key, figureId: saved.id })
     /* The picture is kept by the hash of its bytes, as the server keeps it (ADR-0104 decision 6). */
     Object.assign(saved, { source, name: JSON.parse(source).name, state: 'draft' }, thumbnail === undefined ? {} : { thumbnailHash: await thumbnailHashOf(thumbnail) })
     rows.set(saved.id, saved)
@@ -87,7 +91,8 @@ const answers = {
     return { __typename: 'Figure', id, version: released.version, state: 'released', updatedAt: null }
   },
   figureRecipeTemplates: () => [],
-  figureProposalOutcome: () => [],
+  figureProposalOutcome: ({ proposalKey }) =>
+    records.filter(r => r.proposalKey === proposalKey).map(r => ({ figureId: r.figureId, type: rows.get(r.figureId)?.type, name: rows.get(r.figureId)?.name, recordedAt: null })),
   figureReplacedDrafts: () => [],
   figureVersions: () => [],
   figureReleased: () => null,

package/test/figure-assistant-surface.test.ts

@@ -2,7 +2,7 @@
  * The modeller's AI - the framework dock talks about the figure on screen (design/ai/shared-dock.md §3).
  *
  * What the page does with the dock -- ① a saved figure provides a surface anchored on its row ② the draft is read at the
- * moment of sending ③ an unsaved figure provides nothing ④ the surface is taken back when the page goes inactive or
+ * moment of sending ③ an unsaved figure provides one anchored on its creation id ④ the surface is taken back when the page goes inactive or
  * leaves ⑤ a proposal is taken through the page, and a new figure's opens a new figure -- is checked on the page itself
  * in test/browser/product-page.check.mjs. Here: the offer the page takes it by, and the words every outcome is said in.
  */
@@ -43,6 +43,70 @@ test('the dock applies a proposal through the page, on the version the model saw
   assert.equal(create.applied, undefined)
 })
 
+/*
+  The dock reads the draft when the person sends (figure audit 2026-10-16 N5). A save in flight is no edit: reading the
+  draft then was refused with the edit guard's Korean "save first" line, which the dock showed as the turn's error.
+*/
+test('the draft is read while a save is in flight', async () => {
+  const source = serializeFigureAsset(addFigureBox(createFigureAsset('A'), { id: 'body', size: { x: 100, y: 100, z: 100 }, position: { x: 0, y: 50, z: 0 } }))
+  let finish: (value: { id: string; source: string }) => void = () => {}
+  const session = createFigureProductSession({
+    load: async (id: string) => ({ id, source }),
+    save: (q: { id?: string; source: string }) => new Promise(resolve => (finish = () => resolve({ id: q.id ?? 'a', source: q.source })))
+  })
+  await session.open('a')
+  session.command({ kind: 'rename', name: 'B' } as never)
+  const saving = session.save()
+  assert.equal(session.read().saving, true)
+  assert.equal(JSON.parse(session.source()).name, 'B')
+  finish({ id: 'a', source })
+  await saving
+})
+
+/*
+  A revision is for the figure the model read, and a page with another figure open leaves it untaken (figure audit
+  2026-10-16 N3): the handler then says it needs that figure open. The version alone did not tell them apart -- two
+  figures opened in two page loads were both at their first context and revision 0.
+*/
+test('a revision made for another figure is left untaken, whatever version it names', async () => {
+  const source = serializeFigureAsset(addFigureBox(createFigureAsset('A'), { id: 'body', size: { x: 100, y: 100, z: 100 }, position: { x: 0, y: 50, z: 0 } }))
+  const session = createFigureProductSession({ load: async (id: string) => ({ id, source }), save: async (q: { id?: string; source: string }) => ({ id: q.id ?? 'a', source: q.source }) })
+  await session.open('a')
+  const now = session.read()
+  const other: FigureApplyDetail = { key: K1, steps: lid, create: false, figureId: 'b', context: now.context, revision: now.revision }
+  assert.equal(takeFigureOffer(session, other, { onSavedFigure: true, say }), false)
+  assert.equal(other.applied, undefined)
+  assert.equal(session.read().revision, now.revision)
+  const mine: FigureApplyDetail = { key: K1, steps: lid, create: false, figureId: 'a', context: now.context, revision: now.revision }
+  assert.equal(takeFigureOffer(session, mine, { onSavedFigure: true, say }), true)
+})
+
+/*
+  A figure not saved yet is a subject too (ADR-0061 decision 3): its conversation is anchored on the creation id the
+  first save inserts the row under, so the anchor is the same before and after that save. Taking a new figure's
+  proposal on it must not change that id -- the conversation that asked for the draft would be left behind.
+*/
+test('a new figure keeps its creation id through a new figure\'s proposal and the first save', async () => {
+  const sent: Array<{ id?: string; creationId?: string; proposalKeys?: string[] }> = []
+  const session = createFigureProductSession({
+    load: async (id: string) => ({ id, source: '' }),
+    save: async (q: { id?: string; source: string; creationId?: string; proposalKeys?: string[] }) => {
+      sent.push({ id: q.id, creationId: q.creationId, proposalKeys: q.proposalKeys })
+      return { id: q.id ?? q.creationId!, source: q.source }
+    }
+  })
+  session.startNew()
+  const creationId = session.read().creationId
+  assert.match(creationId ?? '', /^[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/)
+  const create: FigureApplyDetail = { key: K1, steps: lid, create: true }
+  assert.equal(takeFigureOffer(session, create, { onSavedFigure: false, say }), true)
+  assert.equal(session.read().creationId, creationId, 'the proposal\'s key is not the figure\'s id')
+  assert.deepEqual(session.proposalsInEffect(), [K1], 'the save still records the proposal')
+  await session.save()
+  assert.deepEqual(sent, [{ id: undefined, creationId, proposalKeys: [K1] }])
+  assert.deepEqual({ id: session.read().id, creationId: session.read().creationId }, { id: creationId, creationId })
+})
+
 test('a figure proposal is answered by its handler, in words, never left without one', () => {
   for (const lang of ['en', 'ko', 'ja', 'zh', 'ms', 'es']) {
     const words = JSON.parse(readFileSync(fileURLToPath(new URL(`../translations/${lang}.json`, import.meta.url)), 'utf8'))

package/test/proposal-record.test.ts

@@ -1,6 +1,6 @@
 /*
  * What became of an AI proposal (ADR-0097 §5.6 item 5; AI lane, 2026-09-27). A save names the proposals whose steps its
- * text holds -- undone, a proposal is not in it; a new figure made from a proposal is created under the proposal's key.
+ * text holds -- undone, a proposal is not in it; a new figure opened for a proposal is created under the proposal's key.
  * The server side (recorded in the save's transaction, one figure per key) is tested on sqlite in figure-service
  * (figure-proposal-db).
  */
@@ -59,16 +59,28 @@ test('a save names the proposals its text holds: undone, not; redone, again; und
   assert.deepEqual(s.proposalsInEffect(), [K1])
 })
 
-test('a new figure made from a proposal is created under its key; one being made keeps its own', async () => {
+/*
+  Two ways a new figure comes from a proposal (ADR-0061 decision 3, ruled 2026-10-16). Opened for a proposal from another
+  screen, it is created under the proposal's key. Taken on the new figure the modeller has open, its conversation is
+  anchored on that figure's creation id, which stays; the save records the proposal either way.
+*/
+test('a new figure opened for a proposal is created under its key; one taken on the open new figure keeps its id', async () => {
   const { store, saves } = server()
   const s = createFigureProductSession(store)
   s.startNew()
-  const detail: FigureApplyDetail = { key: K1, steps: lid, create: true }
-  assert.equal(takeFigureOffer(s, detail, { onSavedFigure: false, say }), true)
+  s.createUnder(K1)
+  s.applyProposal(lid, s.read().revision!, s.read().context!, K1)
   await s.save({ type: 'LIDDED' })
   assert.equal(saves.at(-1)!.creationId, K1, 'created under the proposal key: taken twice, it meets the first figure')
   assert.deepEqual(saves.at(-1)!.proposalKeys, [K1])
 
+  s.startNew({ discard: true })
+  const own = s.read().creationId
+  const detail: FigureApplyDetail = { key: K2, steps: lid, create: true }
+  assert.equal(takeFigureOffer(s, detail, { onSavedFigure: false, say }), true)
+  await s.save({ type: 'LIDDED_2' })
+  assert.deepEqual({ creationId: saves.at(-1)!.creationId, proposalKeys: saves.at(-1)!.proposalKeys }, { creationId: own, proposalKeys: [K2] })
+
   /* A new figure the person has touched is theirs: the proposal is not put on it (NEW_FIGURE_UNSAVED), its id stays. */
   s.startNew({ discard: true })
   s.command({ kind: 'rename', name: 'Mine' } as never)
@@ -84,13 +96,13 @@ test('the key travels from the card to the modeller, on a revision and on a new
   const opened: Array<string | undefined> = []
   await runFigureProposal(
     { key: K1, steps: lid, create: false, context: 'c', revision: 1 },
-    { offer: d => (offered.push(d), { ...d, applied: true }), openNew: async () => ({ applied: false }) },
+    { offer: d => (offered.push(d), { ...d, applied: true }), openNew: async () => ({ applied: false }), savedInto: async () => [], openSaved: () => {} },
     say
   )
   assert.equal(offered[0]!.key, K1)
   await runFigureProposal(
     { key: K2, steps: lid, create: true },
-    { offer: d => d, openNew: async (_steps, key) => (opened.push(key), { applied: true }) },
+    { offer: d => d, openNew: async (_steps, key) => (opened.push(key), { applied: true }), savedInto: async () => [], openSaved: () => {} },
     say
   )
   assert.deepEqual(opened, [K2])

package/translations/en.json

@@ -712,6 +712,7 @@
   "figure.text.proposal-applied": "applied {n} commands as one step.",
   "figure.text.proposal-saved-into": "Saved into: {names}",
   "figure.text.proposal-needs-modeller": "this proposal can be applied only while its figure is open in the modeller.",
+  "figure.text.proposal-empty": "this proposal has no commands, so nothing was applied. ask again.",
   "figure.text.proposal-new-figure-unsaved": "a new figure is being made, so the proposal was not applied. save or discard it, then run it again.",
   "figure.text.proposal-not-opened": "a new figure could not be opened in the modeller, so the proposal was not applied.",
   "figure.text.proposal-refused": "the modeller did not apply the proposal: {reason}",
@@ -754,6 +755,11 @@
   "figure.text.removal-bindings": "{places} went with it.",
   "figure.text.removal-mounting-face": "The mounting face is released.",
   "figure.text.edit-refused": "This edit cannot be made. ({code})",
+  "figure.text.session-not-open": "open the model first.",
+  "figure.text.session-saving": "wait for the save to finish, then edit.",
+  "figure.text.session-version-missing": "an AI edit needs the model version it was asked on.",
+  "figure.text.session-new-figure-touched": "this can be used only on a new model not yet edited.",
+  "figure.text.session-proposal-key": "the proposal's identifier is not valid.",
   "figure.text.edit-refused-node-limit": "The figure is too large to add more. It holds up to {max} nodes.",
   "figure.text.document-too-large": "The figure is {at} KiB, over the {max} KiB it can be saved at.",
   "figure.text.document-too-large-unmeasured": "The figure is over the {max} KiB it can be saved at.",
@@ -973,18 +979,6 @@
   "figure.text.detail-unreleased": "Not released yet. Showing the saved draft.",
   "figure.button.detail-edit": "Open editor",
   "figure.button.download-package": "download package",
-  "figure.label.group-carrier": "Carrier",
-  "figure.label.carrier-type": "Carrier type",
-  "figure.text.carrier-type-placeholder": "CARRIERTYPE",
-  "figure.label.carrier-size": "Physical size",
-  "figure.label.carrier-width": "Width",
-  "figure.label.carrier-length": "Length",
-  "figure.label.carrier-height": "Height",
-  "figure.button.declare-carrier": "Declare carrier",
-  "figure.button.clear-carrier": "Clear",
-  "figure.text.carrier-type-required": "Enter the carrier type code.",
-  "figure.text.carrier-type-too-long": "A carrier type code is at most {max} characters.",
-  "figure.text.carrier-length-positive": "{axis} must be greater than 0.",
   "figure.button.import-mesh": "import GLB",
   "figure.text.import-mesh-hint": "brings a GLB file in as a part: the whole file, or one part of it.",
   "figure.text.mesh-whole-file": "whole file",

package/translations/es.json

@@ -712,6 +712,7 @@
   "figure.text.proposal-applied": "se aplicaron {n} comandos como un solo paso.",
   "figure.text.proposal-saved-into": "Guardado en: {names}",
   "figure.text.proposal-needs-modeller": "esta propuesta solo se puede aplicar mientras su figura esté abierta en el modelador.",
+  "figure.text.proposal-empty": "esta propuesta no tiene comandos, por lo que no se aplicó. vuelva a pedirla.",
   "figure.text.proposal-new-figure-unsaved": "se está creando una nueva figura, por lo que no se aplicó la propuesta. guárdela o descártela y vuelva a ejecutarla.",
   "figure.text.proposal-not-opened": "no se pudo abrir una nueva figura en el modelador, por lo que no se aplicó la propuesta.",
   "figure.text.proposal-refused": "el modelador no aplicó la propuesta: {reason}",
@@ -754,6 +755,11 @@
   "figure.text.removal-bindings": "{places} se eliminaron con ella.",
   "figure.text.removal-mounting-face": "La cara de montaje se ha liberado.",
   "figure.text.edit-refused": "No se puede realizar esta edición. ({code})",
+  "figure.text.session-not-open": "abra primero el modelo.",
+  "figure.text.session-saving": "espere a que termine de guardarse y luego edite.",
+  "figure.text.session-version-missing": "una edición de IA necesita la versión del modelo en la que se pidió.",
+  "figure.text.session-new-figure-touched": "solo se puede usar en un modelo nuevo que aún no se ha editado.",
+  "figure.text.session-proposal-key": "el identificador de la propuesta no es válido.",
   "figure.text.edit-refused-node-limit": "La figura es demasiado grande para añadir más. Admite hasta {max} nodos.",
   "figure.text.document-too-large": "La figura ocupa {at} KiB, más que los {max} KiB con los que se puede guardar.",
   "figure.text.document-too-large-unmeasured": "La figura supera los {max} KiB con los que se puede guardar.",
@@ -973,18 +979,6 @@
   "figure.text.detail-unreleased": "Aún no publicada. Se muestra el borrador guardado.",
   "figure.button.detail-edit": "Abrir editor",
   "figure.button.download-package": "descargar paquete",
-  "figure.label.group-carrier": "Portador",
-  "figure.label.carrier-type": "Tipo de portador",
-  "figure.text.carrier-type-placeholder": "CARRIERTYPE",
-  "figure.label.carrier-size": "Tamaño físico",
-  "figure.label.carrier-width": "Ancho",
-  "figure.label.carrier-length": "Largo",
-  "figure.label.carrier-height": "Alto",
-  "figure.button.declare-carrier": "Declarar portador",
-  "figure.button.clear-carrier": "Quitar",
-  "figure.text.carrier-type-required": "Introduzca el código del tipo de portador.",
-  "figure.text.carrier-type-too-long": "El código del tipo de portador tiene como máximo {max} caracteres.",
-  "figure.text.carrier-length-positive": "{axis} debe ser mayor que 0.",
   "figure.button.import-mesh": "importar GLB",
   "figure.text.import-mesh-hint": "trae un archivo GLB como pieza: el archivo entero o una parte de él.",
   "figure.text.mesh-whole-file": "archivo entero",

package/translations/ja.json

@@ -712,6 +712,7 @@
   "figure.text.proposal-applied": "{n} 個のコマンドを 1 ステップとして適用しました。",
   "figure.text.proposal-saved-into": "保存された図形に含まれています: {names}",
   "figure.text.proposal-needs-modeller": "この提案は、その図形がモデラーで開いているときだけ適用できます。",
+  "figure.text.proposal-empty": "この提案にはコマンドがないため、適用しませんでした。もう一度依頼してください。",
   "figure.text.proposal-new-figure-unsaved": "作成中の新しい図形があるため、提案を適用しませんでした。保存または破棄してから、もう一度実行してください。",
   "figure.text.proposal-not-opened": "新しい図形をモデラーで開けなかったため、提案を適用しませんでした。",
   "figure.text.proposal-refused": "モデラーは提案を適用しませんでした: {reason}",
@@ -754,6 +755,11 @@
   "figure.text.removal-bindings": "{places} も一緒に削除しました。",
   "figure.text.removal-mounting-face": "取付面が外れました。",
   "figure.text.edit-refused": "この編集はできません。({code})",
+  "figure.text.session-not-open": "先にモデルを正しく開いてください。",
+  "figure.text.session-saving": "保存が終わってから編集してください。",
+  "figure.text.session-version-missing": "AI の編集には、依頼した時点のモデルのバージョンが必要です。",
+  "figure.text.session-new-figure-touched": "まだ編集していない新しいモデルにだけ使えます。",
+  "figure.text.session-proposal-key": "提案の識別子が正しくありません。",
   "figure.text.edit-refused-node-limit": "図形が大きいためこれ以上追加できません。ノードは {max} 個までです。",
   "figure.text.document-too-large": "図形が {at} KiB で、保存できる上限 {max} KiB を超えています。",
   "figure.text.document-too-large-unmeasured": "図形が保存できる上限 {max} KiB を超えています。",
@@ -973,18 +979,6 @@
   "figure.text.detail-unreleased": "まだ公開されていません。保存された下書きを表示します。",
   "figure.button.detail-edit": "編集画面を開く",
   "figure.button.download-package": "パッケージをダウンロード",
-  "figure.label.group-carrier": "キャリア",
-  "figure.label.carrier-type": "キャリアタイプ",
-  "figure.text.carrier-type-placeholder": "CARRIERTYPE",
-  "figure.label.carrier-size": "実寸",
-  "figure.label.carrier-width": "幅",
-  "figure.label.carrier-length": "長さ",
-  "figure.label.carrier-height": "高さ",
-  "figure.button.declare-carrier": "キャリアとして宣言",
-  "figure.button.clear-carrier": "宣言を解除",
-  "figure.text.carrier-type-required": "キャリアタイプのコードを入力してください。",
-  "figure.text.carrier-type-too-long": "キャリアタイプのコードは {max} 文字までです。",
-  "figure.text.carrier-length-positive": "{axis} は 0 より大きい値にしてください。",
   "figure.button.import-mesh": "GLB を取り込む",
   "figure.text.import-mesh-hint": "GLB ファイルを部品として取り込みます。ファイル全体か、その中の 1 つを選びます。",
   "figure.text.mesh-whole-file": "ファイル全体",

package/translations/ko.json

@@ -712,6 +712,7 @@
   "figure.text.proposal-applied": "명령 {n}개를 한 단계로 적용했습니다.",
   "figure.text.proposal-saved-into": "저장된 도형에 들어 있습니다: {names}",
   "figure.text.proposal-needs-modeller": "이 제안은 모델러에 그 도형이 열려 있을 때만 적용할 수 있습니다.",
+  "figure.text.proposal-empty": "이 제안에는 명령이 없어 적용하지 않았습니다. 다시 요청해 주세요.",
   "figure.text.proposal-new-figure-unsaved": "만들고 있는 새 Figure가 있어 제안을 적용하지 않았습니다. 저장하거나 버린 뒤 다시 실행해 주세요.",
   "figure.text.proposal-not-opened": "새 Figure를 모델러에 열지 못해 제안을 적용하지 않았습니다.",
   "figure.text.proposal-refused": "모델러가 제안을 적용하지 않았습니다: {reason}",
@@ -754,6 +755,11 @@
   "figure.text.removal-bindings": "{places} 도 함께 지웠습니다.",
   "figure.text.removal-mounting-face": "장착면이 풀렸습니다.",
   "figure.text.edit-refused": "이 편집은 할 수 없습니다. ({code})",
+  "figure.text.session-not-open": "먼저 모델을 정상적으로 열어 주세요.",
+  "figure.text.session-saving": "저장이 끝난 뒤 편집해 주세요.",
+  "figure.text.session-version-missing": "AI 편집에는 요청 당시의 모델 문맥과 리비전이 필요합니다.",
+  "figure.text.session-new-figure-touched": "손대지 않은 새 모델에만 쓸 수 있습니다.",
+  "figure.text.session-proposal-key": "제안 식별자가 올바르지 않습니다.",
   "figure.text.edit-refused-node-limit": "노드 수 제한으로 부품을 더 추가할 수 없습니다. 최대 노드 수는 {max}개입니다.",
   "figure.text.document-too-large": "도형이 {at} KiB 로 저장 한도 {max} KiB 를 넘습니다.",
   "figure.text.document-too-large-unmeasured": "도형이 저장 한도 {max} KiB 를 넘습니다.",
@@ -973,18 +979,6 @@
   "figure.text.detail-unreleased": "아직 발행되지 않았습니다. 저장된 초안을 보여줍니다.",
   "figure.button.detail-edit": "편집 화면 열기",
   "figure.button.download-package": "꾸러미 내려받기",
-  "figure.label.group-carrier": "캐리어",
-  "figure.label.carrier-type": "캐리어 유형",
-  "figure.text.carrier-type-placeholder": "CARRIERTYPE",
-  "figure.label.carrier-size": "실물 크기",
-  "figure.label.carrier-width": "폭",
-  "figure.label.carrier-length": "길이",
-  "figure.label.carrier-height": "높이",
-  "figure.button.declare-carrier": "캐리어로 선언",
-  "figure.button.clear-carrier": "선언 해제",
-  "figure.text.carrier-type-required": "캐리어 유형 코드를 입력해 주세요.",
-  "figure.text.carrier-type-too-long": "캐리어 유형 코드는 {max}자까지입니다.",
-  "figure.text.carrier-length-positive": "{axis} 값은 0보다 커야 합니다.",
   "figure.button.import-mesh": "GLB 가져오기",
   "figure.text.import-mesh-hint": "GLB 파일을 부품으로 가져옵니다. 파일 전체나 그 안의 한 부분을 고릅니다.",
   "figure.text.mesh-whole-file": "파일 전체",