@dxos/functions 0.5.3-main.8b66fe2 → 0.5.3-main.8ffbbae

package/schema/functions.json

@@ -1,24 +1,26 @@
 {
   "$schema": "http://json-schema.org/draft-07/schema#",
   "type": "object",
-  "required": [],
+  "required": [
+    "functions"
+  ],
   "properties": {
     "functions": {
       "type": "array",
       "items": {
         "type": "object",
         "required": [
-          "uri",
-          "route",
+          "id",
+          "path",
           "handler"
         ],
         "properties": {
-          "uri": {
+          "id": {
             "type": "string",
             "description": "a string",
             "title": "string"
           },
-          "route": {
+          "path": {
             "type": "string",
             "description": "a string",
             "title": "string"
@@ -42,165 +44,140 @@
       "items": {
         "type": "object",
         "required": [
-          "function",
-          "spec"
+          "function"
         ],
         "properties": {
           "function": {
             "type": "string",
-            "description": "Function URI.",
+            "description": "Function ID/URI.",
             "title": "string"
           },
           "meta": {
-            "$ref": "#/$defs/object"
+            "type": "object",
+            "required": [],
+            "properties": {},
+            "additionalProperties": {
+              "$id": "/schemas/any",
+              "title": "any"
+            }
           },
-          "spec": {
-            "anyOf": [
-              {
+          "timer": {
+            "type": "object",
+            "required": [
+              "cron"
+            ],
+            "properties": {
+              "cron": {
+                "type": "string",
+                "description": "a string",
+                "title": "string"
+              }
+            },
+            "additionalProperties": false
+          },
+          "webhook": {
+            "type": "object",
+            "required": [
+              "method"
+            ],
+            "properties": {
+              "method": {
+                "type": "string",
+                "description": "a string",
+                "title": "string"
+              },
+              "port": {
+                "type": "number",
+                "description": "a number",
+                "title": "number"
+              }
+            },
+            "additionalProperties": false
+          },
+          "websocket": {
+            "type": "object",
+            "required": [
+              "url"
+            ],
+            "properties": {
+              "url": {
+                "type": "string",
+                "description": "a string",
+                "title": "string"
+              },
+              "init": {
                 "type": "object",
-                "required": [
-                  "type",
-                  "cron"
-                ],
-                "properties": {
-                  "type": {
-                    "const": "timer"
+                "required": [],
+                "properties": {},
+                "additionalProperties": {
+                  "$id": "/schemas/any",
+                  "title": "any"
+                }
+              }
+            },
+            "additionalProperties": false
+          },
+          "subscription": {
+            "type": "object",
+            "required": [
+              "filter"
+            ],
+            "properties": {
+              "spaceKey": {
+                "type": "string",
+                "description": "a string",
+                "title": "string"
+              },
+              "filter": {
+                "type": "array",
+                "items": {
+                  "type": "object",
+                  "required": [
+                    "type"
+                  ],
+                  "properties": {
+                    "type": {
+                      "type": "string",
+                      "description": "a string",
+                      "title": "string"
+                    },
+                    "props": {
+                      "type": "object",
+                      "required": [],
+                      "properties": {},
+                      "additionalProperties": {
+                        "$id": "/schemas/any",
+                        "title": "any"
+                      }
+                    }
                   },
-                  "cron": {
-                    "type": "string",
-                    "description": "a string",
-                    "title": "string"
-                  }
-                },
-                "additionalProperties": false
+                  "additionalProperties": false
+                }
               },
-              {
+              "options": {
                 "type": "object",
-                "required": [
-                  "type",
-                  "method"
-                ],
+                "required": [],
                 "properties": {
-                  "type": {
-                    "const": "webhook"
-                  },
-                  "method": {
-                    "type": "string",
-                    "description": "a string",
-                    "title": "string"
+                  "deep": {
+                    "type": "boolean",
+                    "description": "a boolean",
+                    "title": "boolean"
                   },
-                  "port": {
+                  "delay": {
                     "type": "number",
                     "description": "a number",
                     "title": "number"
                   }
                 },
                 "additionalProperties": false
-              },
-              {
-                "type": "object",
-                "required": [
-                  "type",
-                  "url"
-                ],
-                "properties": {
-                  "type": {
-                    "const": "websocket"
-                  },
-                  "url": {
-                    "type": "string",
-                    "description": "a string",
-                    "title": "string"
-                  },
-                  "init": {
-                    "type": "object",
-                    "required": [],
-                    "properties": {},
-                    "additionalProperties": {
-                      "$id": "/schemas/any",
-                      "title": "any"
-                    }
-                  }
-                },
-                "additionalProperties": false
-              },
-              {
-                "type": "object",
-                "required": [
-                  "type",
-                  "filter"
-                ],
-                "properties": {
-                  "type": {
-                    "const": "subscription"
-                  },
-                  "filter": {
-                    "type": "array",
-                    "items": {
-                      "type": "object",
-                      "required": [
-                        "type"
-                      ],
-                      "properties": {
-                        "type": {
-                          "type": "string",
-                          "description": "a string",
-                          "title": "string"
-                        },
-                        "props": {
-                          "type": "object",
-                          "required": [],
-                          "properties": {},
-                          "additionalProperties": {
-                            "$id": "/schemas/any",
-                            "title": "any"
-                          }
-                        }
-                      },
-                      "additionalProperties": false
-                    }
-                  },
-                  "options": {
-                    "type": "object",
-                    "required": [],
-                    "properties": {
-                      "deep": {
-                        "type": "boolean",
-                        "description": "a boolean",
-                        "title": "boolean"
-                      },
-                      "delay": {
-                        "type": "number",
-                        "description": "a number",
-                        "title": "number"
-                      }
-                    },
-                    "additionalProperties": false
-                  }
-                },
-                "additionalProperties": false
               }
-            ]
+            },
+            "additionalProperties": false
           }
         },
         "additionalProperties": false
       }
     }
   },
-  "additionalProperties": false,
-  "$defs": {
-    "object": {
-      "$id": "/schemas/object",
-      "oneOf": [
-        {
-          "type": "object"
-        },
-        {
-          "type": "array"
-        }
-      ],
-      "description": "an object in the TypeScript meaning, i.e. the `object` type",
-      "title": "object"
-    }
-  }
+  "additionalProperties": false
 }

package/src/index.ts

@@ -2,8 +2,6 @@
 // Copyright 2023 DXOS.org
 //
 
-export * from './function';
 export * from './handler';
 export * from './runtime';
-export * from './trigger';
 export * from './types';

package/src/runtime/dev-server.test.ts

@@ -5,14 +5,12 @@
 import { expect } from 'chai';
 import path from 'path';
 
-import { waitForCondition } from '@dxos/async';
-import { type Client } from '@dxos/client';
+import { FunctionsPlugin } from '@dxos/agent';
+import { Client, Config } from '@dxos/client';
 import { TestBuilder } from '@dxos/client/testing';
-import { describe, test } from '@dxos/test';
+import { describe, openAndClose, test } from '@dxos/test';
 
 import { DevServer } from './dev-server';
-import { FunctionRegistry } from '../function';
-import { createFunctionRuntime } from '../testing';
 import { type FunctionManifest } from '../types';
 
 describe('dev server', () => {
@@ -20,10 +18,35 @@ describe('dev server', () => {
   let testBuilder: TestBuilder;
   before(async () => {
     testBuilder = new TestBuilder();
-    client = await createFunctionRuntime(testBuilder);
+    const config = new Config({
+      runtime: {
+        agent: {
+          plugins: [
+            {
+              id: 'dxos.org/agent/plugin/functions',
+              config: {
+                port: 8080,
+              },
+            },
+          ],
+        },
+      },
+    });
+
+    const services = testBuilder.createLocalClientServices();
+    client = new Client({ config, services });
+
+    await client.initialize();
+    await client.halo.createIdentity();
+    testBuilder.ctx.onDispose(() => client.destroy());
+
+    // TODO(burdon): Better way to configure plugin? (Rationalize chess.test).
+    const functionsPlugin = new FunctionsPlugin();
+    await functionsPlugin.initialize({ client, clientServices: services });
+    await openAndClose(functionsPlugin);
+
     expect(client.services.services.FunctionRegistryService).to.exist;
   });
-
   after(async () => {
     await testBuilder.destroy();
   });
@@ -32,19 +55,18 @@ describe('dev server', () => {
     const manifest: FunctionManifest = {
       functions: [
         {
-          uri: 'example.com/function/test',
-          route: 'test',
+          id: 'example.com/function/test',
+          path: 'test',
           handler: 'test',
         },
       ],
     };
 
-    const registry = new FunctionRegistry(client);
-    const server = new DevServer(client, registry, {
+    const server = new DevServer(client, {
+      manifest,
       baseDir: path.join(__dirname, '../testing'),
     });
-    const space = await client.spaces.create();
-    await registry.register(space, manifest.functions);
+    await server.initialize();
     await server.start();
 
     // TODO(burdon): Doesn't shut down cleanly.
@@ -52,8 +74,6 @@ describe('dev server', () => {
     testBuilder.ctx.onDispose(() => server.stop());
     expect(server).to.exist;
 
-    await waitForCondition({ condition: () => server.functions.length > 0 });
-
     await server.invoke('test', {});
     expect(server.stats.seq).to.eq(1);
   });

package/src/runtime/dev-server.ts

@@ -9,15 +9,14 @@ import { join } from 'node:path';
 
 import { Event, Trigger } from '@dxos/async';
 import { type Client } from '@dxos/client';
-import { Context } from '@dxos/context';
 import { invariant } from '@dxos/invariant';
 import { log } from '@dxos/log';
 
-import { type FunctionRegistry } from '../function';
 import { type FunctionContext, type FunctionEvent, type FunctionHandler, type FunctionResponse } from '../handler';
-import { type FunctionDef } from '../types';
+import { type FunctionDef, type FunctionManifest } from '../types';
 
 export type DevServerOptions = {
+  manifest: FunctionManifest;
   baseDir: string;
   port?: number;
   reload?: boolean;
@@ -28,8 +27,6 @@ export type DevServerOptions = {
  * Functions dev server provides a local HTTP server for testing functions.
  */
 export class DevServer {
-  private _ctx = createContext();
-
   // Function handlers indexed by name (URL path).
   private readonly _handlers: Record<string, { def: FunctionDef; handler: FunctionHandler<any> }> = {};
 
@@ -41,17 +38,11 @@ export class DevServer {
 
   public readonly update = new Event<number>();
 
+  // prettier-ignore
   constructor(
     private readonly _client: Client,
-    private readonly _functionsRegistry: FunctionRegistry,
     private readonly _options: DevServerOptions,
-  ) {
-    this._functionsRegistry.registered.on(async ({ added }) => {
-      added.forEach((def) => this._load(def));
-      await this._safeUpdateRegistration();
-      log('new functions loaded', { added });
-    });
-  }
+  ) {}
 
   get stats() {
     return {
@@ -72,10 +63,19 @@ export class DevServer {
     return Object.values(this._handlers);
   }
 
+  async initialize() {
+    for (const def of this._options.manifest.functions) {
+      try {
+        await this._load(def);
+      } catch (err) {
+        log.error('parsing function (check manifest)', err);
+      }
+    }
+  }
+
   async start() {
     invariant(!this._server);
     log.info('starting...');
-    this._ctx = createContext();
 
     // TODO(burdon): Move to hono.
     const app = express();
@@ -107,14 +107,12 @@ export class DevServer {
       // Register functions.
       const { registrationId, endpoint } = await this._client.services.services.FunctionRegistryService!.register({
         endpoint: this.endpoint,
+        functions: this.functions.map(({ def: { id, path } }) => ({ id, path })),
       });
 
       log.info('registered', { endpoint });
       this._proxy = endpoint;
       this._functionServiceRegistration = registrationId;
-
-      // Open after registration, so that it can be updated with the list of function definitions.
-      await this._functionsRegistry.open(this._ctx);
     } catch (err: any) {
       await this.stop();
       throw new Error('FunctionRegistryService not available (check plugin is configured).');
@@ -157,10 +155,10 @@ export class DevServer {
   /**
    * Load function.
    */
-  private async _load(def: FunctionDef, force?: boolean | undefined) {
-    const { uri, route, handler } = def;
+  private async _load(def: FunctionDef, force = false) {
+    const { id, path, handler } = def;
     const filePath = join(this._options.baseDir, handler);
-    log.info('loading', { uri, force });
+    log.info('loading', { id, force });
 
     // Remove from cache.
     if (force) {
@@ -171,26 +169,13 @@ export class DevServer {
         });
     }
 
-    // TODO(burdon): Import types.
     // eslint-disable-next-line @typescript-eslint/no-var-requires
     const module = require(filePath);
     if (typeof module.default !== 'function') {
-      throw new Error(`Handler must export default function: ${uri}`);
+      throw new Error(`Handler must export default function: ${id}`);
     }
 
-    this._handlers[route] = { def, handler: module.default };
-  }
-
-  private async _safeUpdateRegistration(): Promise<void> {
-    invariant(this._functionServiceRegistration);
-    try {
-      await this._client.services.services.FunctionRegistryService!.updateRegistration({
-        registrationId: this._functionServiceRegistration,
-        functions: this.functions.map(({ def: { id, route } }) => ({ id, route })),
-      });
-    } catch (e) {
-      log.catch(e);
-    }
+    this._handlers[path] = { def, handler: module.default };
   }
 
   /**
@@ -229,5 +214,3 @@ export class DevServer {
     return statusCode;
   }
 }
-
-const createContext = () => new Context({ name: 'DevServer' });