@frockbot/plugin-bot-template 0.3.5 → 0.3.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@frockbot/plugin-bot-template",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "private": false,
   "type": "module",
   "exports": {
@@ -24,20 +24,20 @@
     "typecheck": "vue-tsc --noEmit -p tsconfig.json"
   },
   "dependencies": {
-    "@frockbot/client-core": "0.3.5",
-    "@frockbot/client-ui": "0.3.5",
-    "@frockbot/configuration-core": "0.3.5",
-    "@frockbot/connection-core": "0.3.5",
-    "@frockbot/kernel-agent-loop": "0.3.5",
-    "@frockbot/kernel-contracts": "0.3.5",
-    "@frockbot/plugin-settings": "0.3.5",
-    "@frockbot/plugin-shell": "0.3.5",
-    "@frockbot/template-core": "0.3.5",
+    "@frockbot/client-core": "0.3.6",
+    "@frockbot/client-ui": "0.3.6",
+    "@frockbot/configuration-core": "0.3.6",
+    "@frockbot/connection-core": "0.3.6",
+    "@frockbot/kernel-agent-loop": "0.3.6",
+    "@frockbot/kernel-contracts": "0.3.6",
+    "@frockbot/plugin-settings": "0.3.6",
+    "@frockbot/plugin-shell": "0.3.6",
+    "@frockbot/template-core": "0.3.6",
     "cordis": "4.0.0-rc.8",
     "vue": "3.5.41"
   },
   "devDependencies": {
-    "@frockbot/plugin-tools": "0.3.5",
+    "@frockbot/plugin-tools": "0.3.6",
     "@types/bun": "1.4.0",
     "@vitejs/plugin-vue": "6.0.8",
     "typescript": "5.9.3",

package/src/agent.test.ts

@@ -81,10 +81,16 @@ function contextFor(turnType: TurnTypeV1): ToolExecutionContext {
   };
 }
 
+// The tool lives in the `frockbot` namespace, so it is reached the way every
+// namespaced tool is: through the `call_dynamic_tool` envelope (ADR 0023).
 const call: ToolCall = {
   id: "call-1",
-  name: BOT_EXPORT_TEMPLATE_TOOL_V1,
-  input: {},
+  name: "call_dynamic_tool",
+  input: {
+    namespace: "frockbot",
+    toolName: BOT_EXPORT_TEMPLATE_TOOL_V1,
+    arguments: {},
+  },
 };
 
 async function invoke(
@@ -98,15 +104,39 @@ async function invoke(
 }
 
 describe("bot_export_template", () => {
-  it("is offered on chat turns only", async () => {
+  it("is discoverable on chat turns only", async () => {
+    // F36: journey 10 step 1 never called this tool — the Bot invented
+    // `package_inspect_self` instead. Registered outside a namespace it
+    // appeared in no `<dynamic_tool_catalog>` entry, no
+    // `get_dynamic_tools({namespace})` result and no pattern search, so a Bot
+    // asked to share itself had nothing to find.
     const mounted = await mount();
     try {
-      const names = (turnType: TurnTypeV1) =>
-        mounted.root.tools.schemas({ turnType }).map((tool) => tool.name);
-      expect(names("chat")).toContain(BOT_EXPORT_TEMPLATE_TOOL_V1);
+      const discovered = async (turnType: TurnTypeV1) => {
+        const context = contextFor(turnType);
+        const discovery: ToolCall = {
+          id: "call-discover",
+          name: "get_dynamic_tools",
+          input: { pattern: "template" },
+        };
+        const preparation = await mounted.root.tools.prepare(
+          discovery,
+          context,
+        );
+        if (preparation.kind === "denied") return preparation.result.content;
+        return (await mounted.root.tools.executePrepared(preparation, context))
+          .content;
+      };
+      expect(await discovered("chat")).toContain(BOT_EXPORT_TEMPLATE_TOOL_V1);
       for (const turnType of ["automation", "subagent"] as const) {
-        expect(names(turnType)).not.toContain(BOT_EXPORT_TEMPLATE_TOOL_V1);
+        expect(await discovered(turnType)).not.toContain(
+          BOT_EXPORT_TEMPLATE_TOOL_V1,
+        );
       }
+      // And it is not offered as a top-level schema, because it is namespaced.
+      expect(
+        mounted.root.tools.schemas({ turnType: "chat" }).map((t) => t.name),
+      ).not.toContain(BOT_EXPORT_TEMPLATE_TOOL_V1);
     } finally {
       await mounted.dispose();
     }

package/src/agent.ts

@@ -134,6 +134,13 @@ export function createBotExportTemplateTool(
 ): ToolDefinition {
   return {
     name: BOT_EXPORT_TEMPLATE_TOOL_V1,
+    // The `frockbot` namespace, so progressive disclosure (ADR 0023) can find
+    // it: it is listed in `<dynamic_tool_catalog>`, returned by
+    // `get_dynamic_tools({namespace:"frockbot"})`, and matched by a pattern
+    // search for "template" or "export". Registered outside a namespace it was
+    // invisible to every one of those, and a Bot asked to share itself
+    // invented `package_inspect_self` instead (journey 10 step 1, F36).
+    namespace: "frockbot",
     description: DESCRIPTION,
     inputSchema: {
       type: "object",

package/src/client/BotTemplateImportSection.vue

@@ -69,11 +69,7 @@ async function plan(): Promise<void> {
       /></span>
       <span class="import__intro">
         <strong>Import a Bot template</strong>
-        <small>
-          Paste a template link to see exactly what it would create before
-          anything happens. Importing never brings across Memory, credentials,
-          or Connections.
-        </small>
+        <small> Paste a template link to see what it would create. </small>
       </span>
     </header>
 
@@ -150,8 +146,8 @@ async function plan(): Promise<void> {
         <h4>Missing from your catalog</h4>
         <ul>
           <li v-for="entry in missing" :key="entry.catalogId">
-            {{ entry.displayName }} — not in the Catalog generation you are
-            pinned to, so it will be skipped.
+            {{ entry.displayName }} — not available in your catalog, so it will
+            be skipped.
           </li>
         </ul>
       </section>

package/src/client/BotTemplateSection.vue

@@ -101,8 +101,8 @@ async function copyLink(share: TemplateShareRecordV1): Promise<void> {
         <span class="templates__intro">
           <strong>Share this Bot</strong>
           <small>
-            Packs the profile, Bot-authored Skills, Routines, and required
-            Packages. Memory, credentials, Connections, and Computer files stay
+            Packs the profile, Skills, Routines, and the plugins it needs.
+            Memory, credentials, connected accounts and computer files stay
             private.
           </small>
         </span>
@@ -125,11 +125,11 @@ async function copyLink(share: TemplateShareRecordV1): Promise<void> {
       >
         Packed {{ templates.summary.skills }} Skill(s),
         {{ templates.summary.routines }} Routine(s),
-        {{ templates.summary.packages }} Package(s) and
+        {{ templates.summary.packages }} plugin(s) and
         {{ templates.summary.publicServers }} public MCP server(s).
         <template v-if="templates.summary.needsConnection > 0">
-          {{ templates.summary.needsConnection }} server(s) are left as a
-          placeholder the importer fills with their own Connection.
+          {{ templates.summary.needsConnection }} server(s) are left for whoever
+          imports this to connect themselves.
         </template>
         Memory, credentials, and Connections were not included.
       </p>