@apollo/client-ai-apps 0.2.2 → 0.2.3

package/dist/hooks/useToolName.d.ts

@@ -1 +1 @@
-export declare const useToolName: () => any;
+export declare const useToolName: () => string | undefined;

package/dist/vite/application_manifest_plugin.d.ts

@@ -1,5 +1,4 @@
-import { TypeNode, type DocumentNode } from "graphql";
-export declare function getTypeName(type: TypeNode): string;
+import { type DocumentNode } from "graphql";
 export declare const ApplicationManifestPlugin: () => {
     name: string;
     configResolved(resolvedConfig: any): Promise<void>;

package/dist/vite/index.js

@@ -1,5 +1,5 @@
 // src/vite/application_manifest_plugin.ts
-import { readFileSync, writeFileSync } from "fs";
+import { readFileSync, writeFileSync, mkdirSync } from "fs";
 import { glob } from "glob";
 import { gqlPluckFromCodeStringSync } from "@graphql-tools/graphql-tag-pluck";
 import { createHash } from "crypto";
@@ -12,7 +12,6 @@ import {
 import { ApolloClient, ApolloLink, InMemoryCache } from "@apollo/client";
 import Observable from "rxjs";
 import path from "path";
-import fs from "fs";
 var root = process.cwd();
 var getRawValue = (node) => {
   switch (node.kind) {
@@ -26,8 +25,25 @@ var getRawValue = (node) => {
         acc[field.name.value] = getRawValue(field.value);
         return acc;
       }, {});
+    default:
+      throw new Error(`Error when parsing directive values: unexpected type '${node.kind}'`);
   }
 };
+var getTypedDirectiveArgument = (argumentName, expectedType, directiveArguments) => {
+  if (!directiveArguments || directiveArguments.length === 0) {
+    return void 0;
+  }
+  let argument = directiveArguments.find((directiveArgument) => directiveArgument.name.value === argumentName);
+  if (!argument) {
+    return void 0;
+  }
+  if (argument.value.kind != expectedType) {
+    throw new Error(
+      `Expected argument '${argumentName}' to be of type '${expectedType}' but found '${argument.value.kind}' instead.`
+    );
+  }
+  return getRawValue(argument.value);
+};
 function getTypeName(type) {
   let t = type;
   while (t.kind === "NonNullType" || t.kind === "ListType") {
@@ -54,11 +70,19 @@ var ApplicationManifestPlugin = () => {
       const id = createHash("sha256").update(body).digest("hex");
       const prefetchID = prefetch ? "__anonymous" : void 0;
       const tools = operation.query.definitions.find((d) => d.kind === "OperationDefinition").directives?.filter((d) => d.name.value === "tool").map((directive) => {
-        const directiveArguments = directive.arguments?.reduce((obj, arg) => ({ ...obj, [arg.name.value]: getRawValue(arg.value) }), {}) ?? {};
+        const name2 = getTypedDirectiveArgument("name", Kind.STRING, directive.arguments);
+        const description = getTypedDirectiveArgument("description", Kind.STRING, directive.arguments);
+        const extraInputs = getTypedDirectiveArgument("extraInputs", Kind.LIST, directive.arguments);
+        if (!name2) {
+          throw new Error("'name' argument must be supplied for @tool");
+        }
+        if (!description) {
+          throw new Error("'description' argument must be supplied for @tool");
+        }
         return {
-          name: directiveArguments["name"],
-          description: directiveArguments["description"],
-          extraInputs: directiveArguments["extraInputs"]
+          name: name2,
+          description,
+          extraInputs
         };
       });
       return Observable.of({ data: { id, name, type, body, variables, prefetch, prefetchID, tools } });
@@ -135,7 +159,7 @@ var ApplicationManifestPlugin = () => {
     };
     if (config.command === "build") {
       const dest = path.resolve(root, config.build.outDir, ".application-manifest.json");
-      fs.mkdirSync(path.dirname(dest), { recursive: true });
+      mkdirSync(path.dirname(dest), { recursive: true });
       writeFileSync(dest, JSON.stringify(manifest));
     }
     writeFileSync(".application-manifest.json", JSON.stringify(manifest));
@@ -209,6 +233,5 @@ function removeClientDirective(doc) {
 }
 export {
   ApplicationManifestPlugin,
-  getTypeName,
   sortTopLevelDefinitions
 };

package/package.json

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

package/src/hooks/useToolInput.test.ts

@@ -1,6 +1,6 @@
 import { expect, test, vi } from "vitest";
 import { useToolInput } from "./useToolInput";
-import { renderHook, act } from "@testing-library/react";
+import { renderHook } from "@testing-library/react";
 
 test("Should return tool input when called", async () => {
   vi.stubGlobal("openai", {

package/src/hooks/useToolInput.ts

@@ -1,6 +1,6 @@
 import { useOpenAiGlobal } from "./useOpenAiGlobal";
 
-export const useToolInput = () => {
+export const useToolInput = (): any => {
   const toolInput = useOpenAiGlobal("toolInput");
 
   return toolInput;

package/src/hooks/useToolName.test.ts

@@ -0,0 +1,13 @@
+import { expect, test, vi } from "vitest";
+import { useToolName } from "./useToolName";
+import { renderHook } from "@testing-library/react";
+
+test("Should return tool input when called", async () => {
+  vi.stubGlobal("openai", {
+    toolResponseMetadata: { toolName: "get-products" },
+  });
+
+  const { result } = renderHook(() => useToolName());
+
+  expect(result.current).toBe("get-products");
+});

package/src/hooks/useToolName.ts

@@ -1,6 +1,6 @@
 import { useOpenAiGlobal } from "./useOpenAiGlobal";
 
-export const useToolName = () => {
+export const useToolName = (): string | undefined => {
   const toolResponseMetadata = useOpenAiGlobal("toolResponseMetadata");
 
   return toolResponseMetadata?.toolName;

package/src/vite/application_manifest_plugin.test.ts

@@ -0,0 +1,830 @@
+import { expect, test, vi, describe, beforeEach, Mock } from "vitest";
+import { ApplicationManifestPlugin } from "./application_manifest_plugin";
+import fs from "fs";
+import * as glob from "glob";
+import path from "path";
+
+vi.mock(import("fs"), async (importOriginal) => {
+  const actual = await importOriginal();
+  return {
+    default: {
+      ...actual.default,
+      readFileSync: vi.fn(),
+      writeFileSync: vi.fn(),
+      mkdirSync: vi.fn(),
+    },
+  };
+});
+
+vi.mock(import("path"), async (importOriginal) => {
+  const actual = await importOriginal();
+  return {
+    default: {
+      ...actual.default,
+      resolve: vi.fn(),
+      dirname: vi.fn(),
+    },
+  };
+});
+
+vi.mock(import("glob"), async (importOriginal) => {
+  const actual = await importOriginal();
+  return {
+    glob: vi.fn(),
+  };
+});
+
+beforeEach(() => {
+  vi.clearAllMocks();
+});
+
+describe("buildStart", () => {
+  test("Should write to dev application manifest file when using a serve command", 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($name: string!) @tool(name: "hello-world", description: "This is an awesome tool!", extraInputs: [{
+              name: "doStuff",
+              type: "boolean",
+              description: "Should we do stuff?"
+            }]) { helloWorld(name: $name) }\`;
+        `;
+      }
+    });
+    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 plugin.buildStart();
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+
+    // Ignore the hash so we can do a snapshot that doesn't constantly change
+    let contentObj = JSON.parse(content);
+    contentObj.hash = "abc";
+
+    expect(fs.writeFileSync).toHaveBeenCalledOnce();
+    expect(file).toBe(".application-manifest.json");
+    expect(contentObj).toMatchInlineSnapshot(`
+      {
+        "csp": {
+          "connectDomains": [],
+          "resourceDomains": [],
+        },
+        "format": "apollo-ai-app-manifest",
+        "hash": "abc",
+        "operations": [
+          {
+            "body": "query HelloWorldQuery($name: string!) {
+        helloWorld(name: $name)
+      }",
+            "id": "c2ceb00338549909d9a8cd5cc601bda78d8c27654294dfe408a6c3735beb26a6",
+            "name": "HelloWorldQuery",
+            "prefetch": false,
+            "tools": [
+              {
+                "description": "This is an awesome tool!",
+                "extraInputs": [
+                  {
+                    "description": "Should we do stuff?",
+                    "name": "doStuff",
+                    "type": "boolean",
+                  },
+                ],
+                "name": "hello-world",
+              },
+            ],
+            "type": "query",
+            "variables": {
+              "name": "string",
+            },
+          },
+        ],
+        "resource": "http://localhost:undefined",
+        "version": "1",
+      }
+    `);
+  });
+
+  test("Should NOT write to dev application manifest file when using a build command", 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: "This is an awesome 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: "build", server: {} });
+    await plugin.buildStart();
+
+    expect(fs.writeFileSync).not.toHaveBeenCalled();
+  });
+
+  test("Should not process files that do not contain gql tags", async () => {
+    vi.spyOn(fs, "readFileSync").mockImplementation((path) => {
+      if (path === "package.json") {
+        return JSON.stringify({});
+      } else if (path === "my-component.tsx") {
+        return `
+            const MY_QUERY = \`query HelloWorldQuery @tool(name: "hello-world", description: "This is an awesome 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 plugin.buildStart();
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+
+    // Ignore the hash so we can do a snapshot that doesn't constantly change
+    let contentObj = JSON.parse(content);
+    contentObj.hash = "abc";
+
+    expect(fs.writeFileSync).toHaveBeenCalledOnce();
+    expect(file).toBe(".application-manifest.json");
+    expect(contentObj).toMatchInlineSnapshot(`
+      {
+        "csp": {
+          "connectDomains": [],
+          "resourceDomains": [],
+        },
+        "format": "apollo-ai-app-manifest",
+        "hash": "abc",
+        "operations": [],
+        "resource": "http://localhost:undefined",
+        "version": "1",
+      }
+    `);
+  });
+
+  test("Should capture queries when writing to manifest file", 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 { 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 plugin.buildStart();
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+
+    // Ignore the hash so we can do a snapshot that doesn't constantly change
+    let contentObj = JSON.parse(content);
+    contentObj.hash = "abc";
+
+    expect(fs.writeFileSync).toHaveBeenCalledOnce();
+    expect(file).toBe(".application-manifest.json");
+    expect(contentObj).toMatchInlineSnapshot(`
+      {
+        "csp": {
+          "connectDomains": [],
+          "resourceDomains": [],
+        },
+        "format": "apollo-ai-app-manifest",
+        "hash": "abc",
+        "operations": [
+          {
+            "body": "query HelloWorldQuery {
+        helloWorld
+      }",
+            "id": "f8604bba13e2f589608c0eb36c3039c5ef3a4c5747bc1596f9dbcbe924dc90f9",
+            "name": "HelloWorldQuery",
+            "prefetch": false,
+            "tools": [],
+            "type": "query",
+            "variables": {},
+          },
+        ],
+        "resource": "http://localhost:undefined",
+        "version": "1",
+      }
+    `);
+  });
+
+  test("Should capture queries as prefetch when query is marked with @prefetch directive", 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 @prefetch { 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 plugin.buildStart();
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+
+    // Ignore the hash so we can do a snapshot that doesn't constantly change
+    let contentObj = JSON.parse(content);
+    contentObj.hash = "abc";
+
+    expect(fs.writeFileSync).toHaveBeenCalledOnce();
+    expect(file).toBe(".application-manifest.json");
+    expect(contentObj).toMatchInlineSnapshot(`
+      {
+        "csp": {
+          "connectDomains": [],
+          "resourceDomains": [],
+        },
+        "format": "apollo-ai-app-manifest",
+        "hash": "abc",
+        "operations": [
+          {
+            "body": "query HelloWorldQuery {
+        helloWorld
+      }",
+            "id": "f8604bba13e2f589608c0eb36c3039c5ef3a4c5747bc1596f9dbcbe924dc90f9",
+            "name": "HelloWorldQuery",
+            "prefetch": true,
+            "prefetchID": "__anonymous",
+            "tools": [],
+            "type": "query",
+            "variables": {},
+          },
+        ],
+        "resource": "http://localhost:undefined",
+        "version": "1",
+      }
+    `);
+  });
+
+  test("Should error when multiple operations are marked with @prefetch", 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 @prefetch { helloWorld }\`;
+            const MY_QUERY2 = gql\`query HelloWorldQuery2 @prefetch { 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: Found multiple operations marked as \`@prefetch\`. You can only mark 1 operation with \`@prefetch\`.]`
+    );
+  });
+
+  test("Should capture mutations when writing to manifest file", 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\`mutation HelloWorldQuery @tool(name: "hello-world", description: "This is an awesome 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 plugin.buildStart();
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+
+    // Ignore the hash so we can do a snapshot that doesn't constantly change
+    let contentObj = JSON.parse(content);
+    contentObj.hash = "abc";
+
+    expect(fs.writeFileSync).toHaveBeenCalledOnce();
+    expect(file).toBe(".application-manifest.json");
+    expect(contentObj).toMatchInlineSnapshot(`
+      {
+        "csp": {
+          "connectDomains": [],
+          "resourceDomains": [],
+        },
+        "format": "apollo-ai-app-manifest",
+        "hash": "abc",
+        "operations": [
+          {
+            "body": "mutation HelloWorldQuery {
+        helloWorld
+      }",
+            "id": "0c98e15f08542215c9c268192aaff732800bc33b79dddea7dc6fdf69c21b61a7",
+            "name": "HelloWorldQuery",
+            "prefetch": false,
+            "tools": [
+              {
+                "description": "This is an awesome tool!",
+                "name": "hello-world",
+              },
+            ],
+            "type": "mutation",
+            "variables": {},
+          },
+        ],
+        "resource": "http://localhost:undefined",
+        "version": "1",
+      }
+    `);
+  });
+
+  test("Should throw error when a subscription operation type is discovered", 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\`subscription HelloWorldQuery @tool(name: "hello-world", description: "This is an awesome 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: Found an unsupported operation type. Only Query and Mutation are supported.]`
+    );
+  });
+
+  test("Should use custom entry point when in serve mode and provided in package.json", async () => {
+    vi.spyOn(fs, "readFileSync").mockImplementation((path) => {
+      if (path === "package.json") {
+        return JSON.stringify({
+          entry: {
+            staging: "http://staging.awesome.com",
+          },
+        });
+      } else if (path === "my-component.tsx") {
+        return `
+            const MY_QUERY = gql\`query HelloWorldQuery @tool(name: "hello-world", description: "This is an awesome 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", mode: "staging", server: {} });
+    await plugin.buildStart();
+
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+    let contentObj = JSON.parse(content);
+
+    expect(contentObj.resource).toBe("http://staging.awesome.com");
+  });
+
+  test("Should use https when enabled in server config", 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: "This is an awesome 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: { https: {}, port: "5678" } });
+    await plugin.buildStart();
+
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+    let contentObj = JSON.parse(content);
+
+    expect(contentObj.resource).toBe("https://localhost:5678");
+  });
+
+  test("Should use custom host when specified in server config", 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: "This is an awesome 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: { port: "5678", host: "awesome.com" } });
+    await plugin.buildStart();
+
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+    let contentObj = JSON.parse(content);
+
+    expect(contentObj.resource).toBe("http://awesome.com:5678");
+  });
+
+  test("Should error when tool name is not provided", 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 { 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: 'name' argument must be supplied for @tool]`
+    );
+  });
+
+  test("Should error when tool description is not provided", 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") { 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: 'description' argument must be supplied for @tool]`
+    );
+  });
+
+  test("Should error when tool name is not a string", 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: true) { 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: Expected argument 'name' to be of type 'StringValue' but found 'BooleanValue' instead.]`
+    );
+  });
+
+  test("Should error when tool description is not a string", 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: false) { 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: Expected argument 'description' to be of type 'StringValue' but found 'BooleanValue' instead.]`
+    );
+  });
+
+  test("Should error when extraInputs is not an array", 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: "hello", extraInputs: false ) { 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: Expected argument 'extraInputs' to be of type 'ListValue' but found 'BooleanValue' instead.]`
+    );
+  });
+
+  test("Should error when an unknown type is discovered", 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: "hello", extraInputs: [{
+              name: 3.1
+            }] ) { 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: Error when parsing directive values: unexpected type 'FloatValue']`
+    );
+  });
+
+  test("Should order operations and fragments when generating normalized operation", 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\`
+              fragment A on User { firstName } 
+              fragment B on User { lastName } 
+              query HelloWorldQuery @tool(name: "hello-world", description: "This is an awesome tool!") { 
+                helloWorld {
+                  ...B
+                  ...A
+                  ...C
+                } 
+              
+              fragment C on User { middleName } 
+              }\`;
+        `;
+      }
+    });
+    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 plugin.buildStart();
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+
+    // Ignore the hash so we can do a snapshot that doesn't constantly change
+    let contentObj = JSON.parse(content);
+    contentObj.hash = "abc";
+
+    expect(fs.writeFileSync).toHaveBeenCalledOnce();
+    expect(file).toBe(".application-manifest.json");
+    expect(contentObj).toMatchInlineSnapshot(`
+      {
+        "csp": {
+          "connectDomains": [],
+          "resourceDomains": [],
+        },
+        "format": "apollo-ai-app-manifest",
+        "hash": "abc",
+        "operations": [
+          {
+            "body": "query HelloWorldQuery {
+        helloWorld {
+          ...B
+          ...A
+          ...C
+          __typename
+        }
+        fragment
+        C
+        on
+        User {
+          middleName
+          __typename
+        }
+      }
+
+      fragment A on User {
+        firstName
+        __typename
+      }
+
+      fragment B on User {
+        lastName
+        __typename
+      }",
+            "id": "58359ad006a8e1a6cdabe4b49c0322e8a41d71c5194a796e6432be055220b9ec",
+            "name": "HelloWorldQuery",
+            "prefetch": false,
+            "tools": [
+              {
+                "description": "This is an awesome tool!",
+                "name": "hello-world",
+              },
+            ],
+            "type": "query",
+            "variables": {},
+          },
+        ],
+        "resource": "http://localhost:undefined",
+        "version": "1",
+      }
+    `);
+  });
+});
+
+describe("writeBundle", () => {
+  test("Should use custom entry point when in build mode and provided in package.json", async () => {
+    vi.spyOn(fs, "readFileSync").mockImplementation((path) => {
+      if (path === "package.json") {
+        return JSON.stringify({
+          entry: {
+            staging: "http://staging.awesome.com",
+          },
+        });
+      } else if (path === "my-component.tsx") {
+        return `
+            const MY_QUERY = gql\`query HelloWorldQuery @tool(name: "hello-world", description: "This is an awesome tool!") { helloWorld }\`;
+        `;
+      }
+    });
+    vi.spyOn(glob, "glob").mockImplementation(() => Promise.resolve(["my-component.tsx"]));
+    vi.spyOn(path, "resolve").mockImplementation((_, file) => file);
+    vi.spyOn(path, "dirname").mockImplementation(() => "/dist");
+    vi.spyOn(fs, "writeFileSync");
+
+    const plugin = ApplicationManifestPlugin();
+    plugin.configResolved({ command: "build", mode: "staging", server: {}, build: { outDir: "/dist/" } });
+    await plugin.buildStart();
+    await plugin.writeBundle();
+
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+    let contentObj = JSON.parse(content);
+
+    expect(contentObj.resource).toBe("http://staging.awesome.com");
+  });
+
+  test("Should use index.html when in build production and not provided in package.json", 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: "This is an awesome tool!") { helloWorld }\`;
+        `;
+      }
+    });
+    vi.spyOn(glob, "glob").mockImplementation(() => Promise.resolve(["my-component.tsx"]));
+    vi.spyOn(path, "resolve").mockImplementation((_, file) => file);
+    vi.spyOn(path, "dirname").mockImplementation(() => "/dist");
+    vi.spyOn(fs, "writeFileSync");
+
+    const plugin = ApplicationManifestPlugin();
+    plugin.configResolved({ command: "build", mode: "production", server: {}, build: { outDir: "/dist/" } });
+    await plugin.buildStart();
+    await plugin.writeBundle();
+
+    let [file, content] = (fs.writeFileSync as unknown as Mock).mock.calls[0];
+    let contentObj = JSON.parse(content);
+
+    expect(contentObj.resource).toBe("index.html");
+  });
+
+  test("Should throw an error when in build mode and using a mode that is not production and not provided in package.json", 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: "This is an awesome tool!") { helloWorld }\`;
+        `;
+      }
+    });
+    vi.spyOn(glob, "glob").mockImplementation(() => Promise.resolve(["my-component.tsx"]));
+    vi.spyOn(path, "resolve").mockImplementation((_, file) => file);
+    vi.spyOn(path, "dirname").mockImplementation(() => "/dist");
+    vi.spyOn(fs, "writeFileSync");
+
+    const plugin = ApplicationManifestPlugin();
+    plugin.configResolved({ command: "build", mode: "staging", server: {}, build: { outDir: "/dist/" } });
+    await plugin.buildStart();
+
+    await expect(async () => await plugin.writeBundle()).rejects.toThrowErrorMatchingInlineSnapshot(
+      `[Error: No entry point found for mode "staging". Entry points other than "development" and "production" must be defined in package.json file.]`
+    );
+  });
+
+  test("Should always write to both locations when running in build mode", 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: "This is an awesome tool!") { helloWorld }\`;
+        `;
+      }
+    });
+    vi.spyOn(glob, "glob").mockImplementation(() => Promise.resolve(["my-component.tsx"]));
+    vi.spyOn(path, "resolve").mockImplementation((_, file) => file);
+    vi.spyOn(path, "dirname").mockImplementation(() => "/dist");
+    vi.spyOn(fs, "writeFileSync");
+
+    const plugin = ApplicationManifestPlugin();
+    plugin.configResolved({ command: "build", mode: "production", server: {}, build: { outDir: "/dist/" } });
+    await plugin.buildStart();
+    await plugin.writeBundle();
+
+    expect(fs.writeFileSync).toBeCalledTimes(2);
+  });
+});
+
+describe("configureServer", () => {
+  test("Should write to manifest file when package.json or file is updated", 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($name: string!) @tool(name: "hello-world", description: "This is an awesome tool!", extraInputs: [{
+              name: "doStuff",
+              type: "boolean",
+              description: "Should we do stuff?"
+            }]) { helloWorld(name: $name) }\`;
+        `;
+      }
+    });
+    vi.spyOn(glob, "glob").mockImplementation(() => Promise.resolve(["my-component.tsx"]));
+    vi.spyOn(path, "resolve").mockImplementation((_, file) => file);
+    vi.spyOn(fs, "writeFileSync");
+
+    const server = {
+      watcher: {
+        init: () => {
+          this._callbacks = [];
+        },
+        on: (_event: string, callback: Function) => {
+          this._callbacks.push(callback);
+        },
+        trigger: async (file) => {
+          for (const callback of this._callbacks) {
+            await callback(file);
+          }
+        },
+      },
+    };
+    server.watcher.init();
+
+    const plugin = ApplicationManifestPlugin();
+    plugin.configResolved({ command: "serve", server: {} });
+    await plugin.buildStart();
+    await plugin.configureServer(server);
+    await server.watcher.trigger("package.json");
+    await server.watcher.trigger("my-component.tsx");
+
+    expect(fs.writeFileSync).toBeCalledTimes(3);
+  });
+});

package/src/vite/application_manifest_plugin.ts

@@ -1,8 +1,9 @@
-import { readFileSync, writeFileSync } from "fs";
+import { readFileSync, writeFileSync, mkdirSync } from "fs";
 import { glob } from "glob";
 import { gqlPluckFromCodeStringSync } from "@graphql-tools/graphql-tag-pluck";
 import { createHash } from "crypto";
 import {
+  ArgumentNode,
   Kind,
   ListTypeNode,
   NamedTypeNode,
@@ -18,11 +19,9 @@ import {
 import { ApolloClient, ApolloLink, InMemoryCache } from "@apollo/client";
 import Observable from "rxjs";
 import path from "path";
-import fs from "fs";
 
 const root = process.cwd();
 
-// TODO: Do we need "validation" of the types for the different properties? Probably?
 const getRawValue = (node: ValueNode): any => {
   switch (node.kind) {
     case Kind.STRING:
@@ -35,10 +34,36 @@ const getRawValue = (node: ValueNode): any => {
         acc[field.name.value] = getRawValue(field.value);
         return acc;
       }, {});
+    default:
+      throw new Error(`Error when parsing directive values: unexpected type '${node.kind}'`);
   }
 };
 
-export function getTypeName(type: TypeNode): string {
+const getTypedDirectiveArgument = (
+  argumentName: string,
+  expectedType: Kind,
+  directiveArguments: readonly ArgumentNode[] | undefined
+) => {
+  if (!directiveArguments || directiveArguments.length === 0) {
+    return undefined;
+  }
+
+  let argument = directiveArguments.find((directiveArgument) => directiveArgument.name.value === argumentName);
+
+  if (!argument) {
+    return undefined;
+  }
+
+  if (argument.value.kind != expectedType) {
+    throw new Error(
+      `Expected argument '${argumentName}' to be of type '${expectedType}' but found '${argument.value.kind}' instead.`
+    );
+  }
+
+  return getRawValue(argument.value);
+};
+
+function getTypeName(type: TypeNode): string {
   let t = type;
   while (t.kind === "NonNullType" || t.kind === "ListType") {
     t = (t as NonNullTypeNode | ListTypeNode).type;
@@ -78,12 +103,22 @@ export const ApplicationManifestPlugin = () => {
       ).directives
         ?.filter((d) => d.name.value === "tool")
         .map((directive) => {
-          const directiveArguments: Record<string, any> =
-            directive.arguments?.reduce((obj, arg) => ({ ...obj, [arg.name.value]: getRawValue(arg.value) }), {}) ?? {};
+          const name = getTypedDirectiveArgument("name", Kind.STRING, directive.arguments);
+          const description = getTypedDirectiveArgument("description", Kind.STRING, directive.arguments);
+          const extraInputs = getTypedDirectiveArgument("extraInputs", Kind.LIST, directive.arguments);
+
+          if (!name) {
+            throw new Error("'name' argument must be supplied for @tool");
+          }
+
+          if (!description) {
+            throw new Error("'description' argument must be supplied for @tool");
+          }
+
           return {
-            name: directiveArguments["name"],
-            description: directiveArguments["description"],
-            extraInputs: directiveArguments["extraInputs"],
+            name,
+            description,
+            extraInputs,
           };
         });
 
@@ -174,7 +209,7 @@ export const ApplicationManifestPlugin = () => {
 
     if (config.command === "build") {
       const dest = path.resolve(root, config.build.outDir, ".application-manifest.json");
-      fs.mkdirSync(path.dirname(dest), { recursive: true });
+      mkdirSync(path.dirname(dest), { recursive: true });
       writeFileSync(dest, JSON.stringify(manifest));
     }
     // Always write to the dev location so that the app can bundle the manifest content