@things-factory/ai-client-base 10.1.207 → 10.1.208

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@things-factory/ai-client-base",
-  "version": "10.1.207",
+  "version": "10.1.208",
   "main": "dist-server/index.js",
   "things-factory": true,
   "author": "heartyoh",
@@ -24,11 +24,11 @@
   "dependencies": {
     "@anthropic-ai/sdk": "^0.40.0",
     "@google/generative-ai": "^0.24.0",
-    "@things-factory/auth-base": "^10.1.206",
+    "@things-factory/auth-base": "^10.1.208",
     "@things-factory/env": "^10.1.149",
-    "@things-factory/setting-base": "^10.1.206",
-    "@things-factory/shell": "^10.1.206",
+    "@things-factory/setting-base": "^10.1.208",
+    "@things-factory/shell": "^10.1.208",
     "openai": "^4.0.0"
   },
-  "gitHead": "e6e3fbf5c353629adbeb8850093150b14254e059"
+  "gitHead": "ffc1ceaa9471b9f4a9b86f46c3021f2fed72d17d"
 }

package/server/service/assistant/tool-budget.test.ts

@@ -136,6 +136,24 @@ describe('a scope larger than one call carries', () => {
     expect(result.openableTools).not.toContain('readList150')
   })
 
+  it('★ the list says its tools are the model\'s to use - not something to tell the person is missing', async () => {
+    const categories = registerLists(165)
+    const { client, seen } = recordingClient()
+    await registryAssistantChat(input({ client, toolCategories: categories, front: { categories: ['list-3'] } }))
+
+    const prompt = seen[0].options.systemPrompt
+    expect(prompt).toMatch(/never tell the person a tool on this list is missing/)
+    expect(prompt).toMatch(/never ask the person to choose instead of opening it/)
+  })
+
+  it('★ opening a tool that is already offered says so - call it now', async () => {
+    const categories = registerLists(165)
+    const { client, seen } = recordingClient([reply(undefined, [TC(OPEN_TOOLS, { names: ['readList3', 'readList150'] })]), reply('done')])
+    await registryAssistantChat(input({ client, toolCategories: categories, front: { categories: ['list-3'] } }))
+
+    expect(JSON.parse(seen[1].messages.at(-1).content[0].content)).toMatchObject({ opened: ['readList150'], alreadyOffered: ['readList3'] })
+  })
+
   it('★ a tool called before it is opened does not run - the model is told to open it', async () => {
     const categories = registerLists(165)
     const { client, seen } = recordingClient([reply(undefined, [TC('readList150')]), reply('done')])

package/server/service/assistant/tool-budget.ts

@@ -82,7 +82,14 @@ function oneLine(description: string | undefined): string {
 export function openableToolCatalog(openable: ReadonlyArray<ToolSpec>): string {
   if (openable.length === 0) return ''
   return [
-    `Tools you can open with ${OPEN_TOOLS} (name: what it does). Open the ones the question needs, then call them:`,
+    /*
+     * Office's model, offered a bank lines' total it could open, told the person that read was not open and asked them
+     * to pick a range (2026-10-22) - an app's prompt said a missing tool means a missing permission. What is on this
+     * list is the model's to use.
+     */
+    `Tools you can open with ${OPEN_TOOLS} (name: what it does). They are yours to use in this conversation, only not loaded yet. ` +
+      `When the question needs one, open it with ${OPEN_TOOLS} and call it: never tell the person a tool on this list is missing, ` +
+      'and never ask the person to choose instead of opening it.',
     ...openable.map(spec => `- ${spec.name}: ${oneLine(spec.description)}`)
   ].join('\n')
 }
@@ -119,6 +126,7 @@ export function toolOfferState(scope: ToolScope, front: ToolFront = {}) {
       return { rejected: true, error: 'tool-not-offered', names: unknown, message: `these are not in the list of tools you can open: ${unknown.join(', ')}. Use the exact names from the list.` }
     }
     const fresh = [...new Set(names)].filter(name => !offered.has(name))
+    const already = [...new Set(names)].filter(name => offered.has(name) && name !== OPEN_TOOLS)
     const room = MAX_OFFERED_TOOLS - offered.size
     if (fresh.length > room) {
       return {
@@ -130,7 +138,14 @@ export function toolOfferState(scope: ToolScope, front: ToolFront = {}) {
     }
     for (const name of fresh) offered.set(name, openable.get(name)!)
     const guidance = [...new Set(fresh.map(name => categoryOf.get(name)?.guidance).filter(Boolean))]
-    return { opened: fresh, ...(guidance.length ? { guidance: guidance.join('\n') } : {}), message: 'these tools are open from your next step on. Call them as the question needs.' }
+    return {
+      opened: fresh,
+      ...(already.length ? { alreadyOffered: already } : {}),
+      ...(guidance.length ? { guidance: guidance.join('\n') } : {}),
+      message: already.length
+        ? `${already.join(', ')} ${already.length > 1 ? 'are' : 'is'} already open - call ${already.length > 1 ? 'them' : 'it'} now. ${fresh.length ? 'The others are open from your next step on.' : ''}`.trim()
+        : 'these tools are open from your next step on. Call them as the question needs.'
+    }
   }
 
   return {

package/translations/ko.json

@@ -1,7 +1,7 @@
 {
   "error.ai-not-configured": "이 사업장에는 AI 가 설정되어 있지 않습니다.",
   "error.ai-agent-not-available": "선택한 AI 에이전트를 이 사업장에서 사용할 수 없습니다.",
-  "error.ai-agent-provider-unknown": "\"{provider}\" 은(는) 에이전트가 쓸 수 있는 제공처가 아닙니다.",
+  "error.ai-agent-provider-unknown": "\"{provider}\"은(는) 에이전트에서 지원하지 않는 제공자입니다.",
   "error.ai-agent-key-required": "\"{agent}\" 의 API 키를 입력해 주세요.",
   "error.ai-agent-name-required": "에이전트마다 이름이 있어야 합니다.",
   "error.ai-agent-model-required": "\"{agent}\" 의 모델을 선택해 주세요.",