@apollo/client-ai-apps 0.3.1 → 0.3.2

package/dist/vite/index.js

@@ -105,6 +105,11 @@ var ApplicationManifestPlugin = () => {
         if (!name2) {
           throw new Error("'name' argument must be supplied for @tool");
         }
+        if (name2.indexOf(" ") > -1) {
+          throw new Error(
+            `Tool with name "${name2}" contains spaces which is not allowed.`
+          );
+        }
         if (!description) {
           throw new Error(
             "'description' argument must be supplied for @tool"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@apollo/client-ai-apps",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "publishConfig": {
     "access": "public"
   },

package/src/vite/__tests__/application_manifest_plugin.test.ts

@@ -562,6 +562,32 @@ describe("buildStart", () => {
     );
   });
 
+  test("Should error when tool name contains spaces", async () => {
+    vi.spyOn(fs, "readFileSync").mockImplementation((path) => {
+      if (path === "package.json") {
+        return JSON.stringify({});
+      } else if (path === "my-component.tsx") {
+        return `
+            const MY_QUERY = gql\`query HelloWorldQuery @tool(name: "hello world", description: "A tool") { helloWorld }\`;
+        `;
+      }
+    });
+    vi.spyOn(glob, "glob").mockImplementation(() =>
+      Promise.resolve(["my-component.tsx"])
+    );
+    vi.spyOn(path, "resolve").mockImplementation((_, file) => file);
+    vi.spyOn(fs, "writeFileSync");
+
+    const plugin = ApplicationManifestPlugin();
+    plugin.configResolved({ command: "serve", server: {} });
+
+    await expect(
+      async () => await plugin.buildStart()
+    ).rejects.toThrowErrorMatchingInlineSnapshot(
+      `[Error: Tool with name "hello world" contains spaces which is not allowed.]`
+    );
+  });
+
   test("Should error when tool name is not a string", async () => {
     vi.spyOn(fs, "readFileSync").mockImplementation((path) => {
       if (path === "package.json") {

package/src/vite/application_manifest_plugin.ts

@@ -140,6 +140,12 @@ export const ApplicationManifestPlugin = () => {
             throw new Error("'name' argument must be supplied for @tool");
           }
 
+          if (name.indexOf(" ") > -1) {
+            throw new Error(
+              `Tool with name "${name}" contains spaces which is not allowed.`
+            );
+          }
+
           if (!description) {
             throw new Error(
               "'description' argument must be supplied for @tool"