@adminforth/agent 1.2.1 → 1.3.0

package/agent/systemPrompt.ts

@@ -26,6 +26,9 @@ export const DEFAULT_AGENT_SYSTEM_PROMPT = [
   "Do not add extra explanations or suggestions unless the user asks.",
   "Adapt to the user's tone and style of speaking, mirroring their vibe and wording.",
   "if the user speaks casually, you should respond casually too",
+  "Never mutate data without a fresh user confirmation for that exact mutation.",
+  "A previous confirmation does not carry over to later create, update, delete, or action calls.",
+  "Each separate mutation or explicitly described batch needs its own confirmation immediately before the tool call.",
 
   
 ].join(" ");
@@ -59,8 +62,13 @@ export async function buildAgentSystemPrompt(adminforth: IAdminForth) {
     "You have next default skills which you can fallback to if primary skill set does not provide a good skill:\n" +
       formatSkills(defaultSkills, "skill_name"),
     "Before using any skill, call fetch_skill to load its full instructions.",
+    "The fetched skill response starts with 'Tools mentioned in this skill'. Read that list first.",
     "You can use get_resource immediately to inspect resource structure and column names.",
+    "If the user wants to create, update, delete, or run actions on records, load mutate_data first.",
+    "If the user wants to fetch records, load fetch_data first. If the user wants analytics or charts, load data-analytics first.",
     "Only call fetch_tool_schema for tool names that are explicitly mentioned in a fetched skill and are not already available as base tools.",
+    "If a fetched skill lists a non-base tool you need, call fetch_tool_schema for it immediately instead of telling the user the tool is unavailable.",
+    "For example: for record creation load mutate_data, read its tool list, call fetch_tool_schema for create_record, and then use create_record after confirmation.",
     "When fetch_tool_schema succeeds, that tool becomes available on the next step.",
     "Try to call as many tools as possible in parallel in one step.",
   ];

package/agent/tools/fetchToolSchema.ts

@@ -43,7 +43,7 @@ export async function createFetchToolSchemaTool(
     {
       name: "fetch_tool_schema",
       description:
-        "Fetch the schema for an API-based AdminForth tool by name and load it for later use.",
+        "Fetch the schema for an API-based AdminForth tool by name and load it for later use. Use this right after fetch_skill when the skill mentions non-base tools.",
       schema: fetchToolSchemaSchema,
     },
   );

package/build.log

@@ -29,5 +29,5 @@ custom/skills/fetch_data/SKILL.md
 custom/skills/mutate_data/
 custom/skills/mutate_data/SKILL.md
 
-sent 167,715 bytes  received 413 bytes  336,256.00 bytes/sec
-total size is 166,041  speedup is 0.99
+sent 169,588 bytes  received 413 bytes  340,002.00 bytes/sec
+total size is 167,914  speedup is 0.99

package/custom/SessionsHistory.vue

@@ -6,7 +6,7 @@
     "
   >
     <h3 :class="h3Style">{{ $t('Chat history') }}</h3>
-    <Button @click="agentStore.createPreSession(); agentStore.setSessionHistoryOpen(false)" :disabled="agentStore.isResponseInProgress" class="w-[360px] mx-4 my-2 mb-4 rounded-3xl text-gray-800 dark:text-gray-200">
+    <Button @click="agentStore.createPreSession(); agentStore.setSessionHistoryOpen(false); agentStore.focusTextInput();" :disabled="agentStore.isResponseInProgress" class="w-[360px] mx-4 my-2 mb-4 rounded-3xl text-gray-800 dark:text-gray-200">
       <IconPlusOutline class="w-5 h-5" />
       {{ $t('New chat') }}
     </Button>

package/custom/chat.ts

@@ -1,3 +1,16 @@
+/**
+ * This file is used to fix circular module initialization between ai and @ai-sdk/vue
+ * These files are depending on each other, but vite put them in different chunks, so they are not initialized at the same time, which causes the circular module initialization issue
+ * So I get rid of the @ai-sdk/vue only fixes are:
+ * 1) Change vite config to put these files in the same chunk
+ * 2) Get rid of the circular module initialization by moving the Chat class to this file
+ * 
+ * Maybe there is a better way to fix this issue
+ * 
+ * If you were updating "ai" package and plugin broke, probably you need to update this file as well
+ * Or resolve the circular module initialization issue in a better way
+ */
+
 import {
   AbstractChat,
   ChatInit as BaseChatInit,

package/custom/skills/mutate_data/SKILL.md

@@ -2,6 +2,16 @@ name: mutate_data
 description:  Create/update/delete some record of resource or call actions on one or multiple records
 ---
 
+# Involved tools
+
+Use `create_record` for creating records.
+
+Use `update_record` for editing records.
+
+Use `delete_record` for deleting records.
+
+Use `start_custom_action` and `start_custom_bulk_action` for resource actions.
+
 # General rules
 
 - if there is a dedicated action for some routine (result of `get_resource` tool call, field actions), prefer this to manual updating of records, for example, if you want to approve some comment, prefer calling `approve` action instead of updating `approved` field of comment record (because in action there might be some additional logic like sending notification to user, updating some counters and so on)
@@ -16,7 +26,19 @@ And in the same message ask user for final confirmation.
 
 When creating new record, show user all data which you gona create and in same message ask for confirmation.
 
-Accept any positive confirmation from user like "yes", "sure", "+", anything non-negative call to action, can be considered as confirmation. 
+Accept any positive confirmation from user like "yes", "sure", "+", anything non-negative call to action, can be considered as confirmation.
+
+A confirmation is valid only for the exact mutation plan from the immediately previous assistant message.
+
+Never reuse an older confirmation for a later mutation.
+
+After one mutation is executed, confirmation is consumed and reset.
+
+If you want to perform another create/update/delete/action after that, ask for confirmation again even if the user previously said "yes".
+
+If the mutation plan changes in any way (different record, different fields, different values, different number of records, different action), the old confirmation is invalid and you must ask again.
+
+If you are creating or deleting multiple records in one batch, you may ask once only for that exact batch, but you must list the whole batch explicitly in the confirmation message. Any extra record outside that described batch requires a new confirmation.
 
 # Calling actions
 
@@ -105,4 +127,4 @@ I am going to create user:
 View [John Doe](/admin/resource/users/show/421) # 421 is id of new created record
 
 Are you sure?
-```
+```

package/custom/useAgentStore.ts

@@ -155,10 +155,14 @@ export const useAgentStore = defineStore('agent', () => {
   function openChat() {
     isChatOpen.value = true;
     nextTick(() => {
-      textInput.value?.focus();
+      focusTextInput();
     });
   }
 
+  function focusTextInput() {
+      textInput.value?.focus();
+  }
+
   function setIsChatOpen(isOpen: boolean) {
     isOpen ? openChat() : closeChat();
   }
@@ -354,5 +358,6 @@ export const useAgentStore = defineStore('agent', () => {
     setIsTeleportedToBody,
     chatWidth,
     setChatWidth,
+    focusTextInput
   }
 })

package/dist/agent/systemPrompt.js

@@ -25,6 +25,9 @@ export const DEFAULT_AGENT_SYSTEM_PROMPT = [
     "Do not add extra explanations or suggestions unless the user asks.",
     "Adapt to the user's tone and style of speaking, mirroring their vibe and wording.",
     "if the user speaks casually, you should respond casually too",
+    "Never mutate data without a fresh user confirmation for that exact mutation.",
+    "A previous confirmation does not carry over to later create, update, delete, or action calls.",
+    "Each separate mutation or explicitly described batch needs its own confirmation immediately before the tool call.",
 ].join(" ");
 function formatResources(resources) {
     return resources
@@ -54,8 +57,13 @@ export function buildAgentSystemPrompt(adminforth) {
             "You have next default skills which you can fallback to if primary skill set does not provide a good skill:\n" +
                 formatSkills(defaultSkills, "skill_name"),
             "Before using any skill, call fetch_skill to load its full instructions.",
+            "The fetched skill response starts with 'Tools mentioned in this skill'. Read that list first.",
             "You can use get_resource immediately to inspect resource structure and column names.",
+            "If the user wants to create, update, delete, or run actions on records, load mutate_data first.",
+            "If the user wants to fetch records, load fetch_data first. If the user wants analytics or charts, load data-analytics first.",
             "Only call fetch_tool_schema for tool names that are explicitly mentioned in a fetched skill and are not already available as base tools.",
+            "If a fetched skill lists a non-base tool you need, call fetch_tool_schema for it immediately instead of telling the user the tool is unavailable.",
+            "For example: for record creation load mutate_data, read its tool list, call fetch_tool_schema for create_record, and then use create_record after confirmation.",
             "When fetch_tool_schema succeeds, that tool becomes available on the next step.",
             "Try to call as many tools as possible in parallel in one step.",
         ];

package/dist/agent/tools/fetchToolSchema.js

@@ -29,7 +29,7 @@ export function createFetchToolSchemaTool(apiBasedTools) {
             return JSON.stringify(Object.assign({ status: 200, name: toolName }, serializeApiBasedTool(toolDefinition)), null, 2);
         }), {
             name: "fetch_tool_schema",
-            description: "Fetch the schema for an API-based AdminForth tool by name and load it for later use.",
+            description: "Fetch the schema for an API-based AdminForth tool by name and load it for later use. Use this right after fetch_skill when the skill mentions non-base tools.",
             schema: fetchToolSchemaSchema,
         });
     });

package/dist/custom/SessionsHistory.vue

@@ -6,7 +6,7 @@
     "
   >
     <h3 :class="h3Style">{{ $t('Chat history') }}</h3>
-    <Button @click="agentStore.createPreSession(); agentStore.setSessionHistoryOpen(false)" :disabled="agentStore.isResponseInProgress" class="w-[360px] mx-4 my-2 mb-4 rounded-3xl text-gray-800 dark:text-gray-200">
+    <Button @click="agentStore.createPreSession(); agentStore.setSessionHistoryOpen(false); agentStore.focusTextInput();" :disabled="agentStore.isResponseInProgress" class="w-[360px] mx-4 my-2 mb-4 rounded-3xl text-gray-800 dark:text-gray-200">
       <IconPlusOutline class="w-5 h-5" />
       {{ $t('New chat') }}
     </Button>

package/dist/custom/chat.ts

@@ -1,3 +1,16 @@
+/**
+ * This file is used to fix circular module initialization between ai and @ai-sdk/vue
+ * These files are depending on each other, but vite put them in different chunks, so they are not initialized at the same time, which causes the circular module initialization issue
+ * So I get rid of the @ai-sdk/vue only fixes are:
+ * 1) Change vite config to put these files in the same chunk
+ * 2) Get rid of the circular module initialization by moving the Chat class to this file
+ * 
+ * Maybe there is a better way to fix this issue
+ * 
+ * If you were updating "ai" package and plugin broke, probably you need to update this file as well
+ * Or resolve the circular module initialization issue in a better way
+ */
+
 import {
   AbstractChat,
   ChatInit as BaseChatInit,

package/dist/custom/skills/mutate_data/SKILL.md

@@ -2,6 +2,16 @@ name: mutate_data
 description:  Create/update/delete some record of resource or call actions on one or multiple records
 ---
 
+# Involved tools
+
+Use `create_record` for creating records.
+
+Use `update_record` for editing records.
+
+Use `delete_record` for deleting records.
+
+Use `start_custom_action` and `start_custom_bulk_action` for resource actions.
+
 # General rules
 
 - if there is a dedicated action for some routine (result of `get_resource` tool call, field actions), prefer this to manual updating of records, for example, if you want to approve some comment, prefer calling `approve` action instead of updating `approved` field of comment record (because in action there might be some additional logic like sending notification to user, updating some counters and so on)
@@ -16,7 +26,19 @@ And in the same message ask user for final confirmation.
 
 When creating new record, show user all data which you gona create and in same message ask for confirmation.
 
-Accept any positive confirmation from user like "yes", "sure", "+", anything non-negative call to action, can be considered as confirmation. 
+Accept any positive confirmation from user like "yes", "sure", "+", anything non-negative call to action, can be considered as confirmation.
+
+A confirmation is valid only for the exact mutation plan from the immediately previous assistant message.
+
+Never reuse an older confirmation for a later mutation.
+
+After one mutation is executed, confirmation is consumed and reset.
+
+If you want to perform another create/update/delete/action after that, ask for confirmation again even if the user previously said "yes".
+
+If the mutation plan changes in any way (different record, different fields, different values, different number of records, different action), the old confirmation is invalid and you must ask again.
+
+If you are creating or deleting multiple records in one batch, you may ask once only for that exact batch, but you must list the whole batch explicitly in the confirmation message. Any extra record outside that described batch requires a new confirmation.
 
 # Calling actions
 
@@ -105,4 +127,4 @@ I am going to create user:
 View [John Doe](/admin/resource/users/show/421) # 421 is id of new created record
 
 Are you sure?
-```
+```

package/dist/custom/useAgentStore.ts

@@ -155,10 +155,14 @@ export const useAgentStore = defineStore('agent', () => {
   function openChat() {
     isChatOpen.value = true;
     nextTick(() => {
-      textInput.value?.focus();
+      focusTextInput();
     });
   }
 
+  function focusTextInput() {
+      textInput.value?.focus();
+  }
+
   function setIsChatOpen(isOpen: boolean) {
     isOpen ? openChat() : closeChat();
   }
@@ -354,5 +358,6 @@ export const useAgentStore = defineStore('agent', () => {
     setIsTeleportedToBody,
     chatWidth,
     setChatWidth,
+    focusTextInput
   }
 })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adminforth/agent",
-  "version": "1.2.1",
+  "version": "1.3.0",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
   "type": "module",