@dxos/functions 0.5.3-main.6f960f5 → 0.5.3-main.79e0565

package/dist/lib/browser/index.mjs

@@ -14,65 +14,19 @@ import {
   process
 } from "@dxos/node-std/inject-globals";
 
-// packages/core/functions/src/handler.ts
-import { PublicKey } from "@dxos/client";
-import { log } from "@dxos/log";
-import { nonNullable } from "@dxos/util";
-var __dxlog_file = "/home/runner/work/dxos/dxos/packages/core/functions/src/handler.ts";
-var subscriptionHandler = (handler) => {
-  return ({ event: { data }, context, ...rest }) => {
-    const { client } = context;
-    const space = data.spaceKey ? client.spaces.get(PublicKey.from(data.spaceKey)) : void 0;
-    const objects = space ? data.objects?.map((id) => space.db.getObjectById(id)).filter(nonNullable) : [];
-    if (!!data.spaceKey && !space) {
-      log.warn("invalid space", {
-        data
-      }, {
-        F: __dxlog_file,
-        L: 91,
-        S: void 0,
-        C: (f, a) => f(...a)
-      });
-    } else {
-      log.info("handler", {
-        space: space?.key.truncate(),
-        objects: objects?.length
-      }, {
-        F: __dxlog_file,
-        L: 93,
-        S: void 0,
-        C: (f, a) => f(...a)
-      });
-    }
-    return handler({
-      event: {
-        data: {
-          ...data,
-          space,
-          objects
-        }
-      },
-      context,
-      ...rest
-    });
-  };
-};
-
-// packages/core/functions/src/registry/function-registry.ts
+// packages/core/functions/src/function/function-registry.ts
 import { Event } from "@dxos/async";
 import { create, Filter } from "@dxos/client/echo";
 import { Resource } from "@dxos/context";
-import { PublicKey as PublicKey2 } from "@dxos/keys";
+import { PublicKey } from "@dxos/keys";
+import { log } from "@dxos/log";
 import { ComplexMap } from "@dxos/util";
 
 // packages/core/functions/src/types.ts
-import { AST, S, TypedObject } from "@dxos/echo-schema";
-var omitEchoId = (schema) => S.make(AST.omit(schema.ast, [
-  "id"
-]));
+import { RawObject, S, TypedObject } from "@dxos/echo-schema";
 var SubscriptionTriggerSchema = S.struct({
   type: S.literal("subscription"),
-  // TODO(burdon): Define query DSL.
+  // TODO(burdon): Define query DSL (from ECHO).
   filter: S.array(S.struct({
     type: S.string,
     props: S.optional(S.record(S.string, S.any))
@@ -115,49 +69,79 @@ var FunctionTrigger = class extends TypedObject({
   typename: "dxos.org/type/FunctionTrigger",
   version: "0.1.0"
 })({
-  function: S.string.pipe(S.description("Function ID/URI.")),
-  // Context passed to a function.
-  meta: S.optional(S.record(S.string, S.any)),
+  function: S.string.pipe(S.description("Function URI.")),
+  // Context is merged into the event data passed to the function.
+  meta: S.optional(S.object),
   spec: TriggerSpecSchema
 }) {
 };
 var FunctionManifestSchema = S.struct({
-  functions: S.optional(S.mutable(S.array(omitEchoId(FunctionDef)))),
-  triggers: S.optional(S.mutable(S.array(omitEchoId(FunctionTrigger))))
+  functions: S.optional(S.mutable(S.array(RawObject(FunctionDef)))),
+  triggers: S.optional(S.mutable(S.array(RawObject(FunctionTrigger))))
 });
 
-// packages/core/functions/src/registry/function-registry.ts
+// packages/core/functions/src/util.ts
+var diff = (previous, next, comparator) => {
+  const remaining = [
+    ...previous
+  ];
+  const result = {
+    added: [],
+    updated: [],
+    removed: remaining
+  };
+  for (const object of next) {
+    const index = remaining.findIndex((item) => comparator(item, object));
+    if (index === -1) {
+      result.added.push(object);
+    } else {
+      result.updated.push(remaining[index]);
+      remaining.splice(index, 1);
+    }
+  }
+  return result;
+};
+var intersection = (a, b, comparator) => a.filter((a2) => b.find((b2) => comparator(a2, b2)) !== void 0);
+
+// packages/core/functions/src/function/function-registry.ts
+var __dxlog_file = "/home/runner/work/dxos/dxos/packages/core/functions/src/function/function-registry.ts";
 var FunctionRegistry = class extends Resource {
   constructor(_client) {
     super();
     this._client = _client;
-    this._functionBySpaceKey = new ComplexMap(PublicKey2.hash);
-    this.onFunctionsRegistered = new Event();
+    this._functionBySpaceKey = new ComplexMap(PublicKey.hash);
+    this.registered = new Event();
   }
   getFunctions(space) {
     return this._functionBySpaceKey.get(space.key) ?? [];
   }
   /**
-  * The method loads function definitions from the manifest into the space.
+  * Loads function definitions from the manifest into the space.
   * We first load all the definitions from the space to deduplicate by functionId.
   */
-  // TODO(burdon): This should not be space specific (they are static for the agent).
-  async register(space, manifest) {
-    if (!manifest.functions?.length) {
+  async register(space, functions) {
+    log("register", {
+      space: space.key,
+      functions: functions?.length ?? 0
+    }, {
+      F: __dxlog_file,
+      L: 39,
+      S: this,
+      C: (f, a) => f(...a)
+    });
+    if (!functions?.length) {
       return;
     }
-    if (!space.db.graph.runtimeSchemaRegistry.isSchemaRegistered(FunctionDef)) {
+    if (!space.db.graph.runtimeSchemaRegistry.hasSchema(FunctionDef)) {
       space.db.graph.runtimeSchemaRegistry.registerSchema(FunctionDef);
     }
-    const { objects: existingDefinitions } = await space.db.query(Filter.schema(FunctionDef)).run();
-    const newDefinitions = getNewDefinitions(manifest.functions, existingDefinitions);
-    const reactiveObjects = newDefinitions.map((template) => create(FunctionDef, {
-      ...template
-    }));
-    reactiveObjects.forEach((obj) => space.db.add(obj));
+    const { objects: existing } = await space.db.query(Filter.schema(FunctionDef)).run();
+    const { added, removed } = diff(existing, functions, (a, b) => a.uri === b.uri);
+    added.forEach((def) => space.db.add(create(FunctionDef, def)));
+    removed.forEach((def) => space.db.remove(def));
   }
   async _open() {
-    const spaceListSubscription = this._client.spaces.subscribe(async (spaces) => {
+    const spacesSubscription = this._client.spaces.subscribe(async (spaces) => {
       for (const space of spaces) {
         if (this._functionBySpaceKey.has(space.key)) {
           continue;
@@ -168,27 +152,67 @@ var FunctionRegistry = class extends Resource {
         if (this._ctx.disposed) {
           break;
         }
-        const functionsSubscription = space.db.query(Filter.schema(FunctionDef)).subscribe((definitions) => {
-          const newFunctions = getNewDefinitions(definitions.objects, registered);
-          if (newFunctions.length > 0) {
-            registered.push(...newFunctions);
-            this.onFunctionsRegistered.emit({
+        this._ctx.onDispose(space.db.query(Filter.schema(FunctionDef)).subscribe(({ objects }) => {
+          const { added } = diff(registered, objects, (a, b) => a.uri === b.uri);
+          if (added.length > 0) {
+            registered.push(...added);
+            this.registered.emit({
               space,
-              newFunctions
+              added
             });
           }
-        });
-        this._ctx.onDispose(functionsSubscription);
+        }));
       }
     });
-    this._ctx.onDispose(() => spaceListSubscription.unsubscribe());
+    this._ctx.onDispose(() => spacesSubscription.unsubscribe());
   }
   async _close(_) {
     this._functionBySpaceKey.clear();
   }
 };
-var getNewDefinitions = (candidateList, existing) => {
-  return candidateList.filter((candidate) => existing.find((def) => def.uri === candidate.uri) == null);
+
+// packages/core/functions/src/handler.ts
+import { PublicKey as PublicKey2 } from "@dxos/client";
+import { log as log2 } from "@dxos/log";
+import { nonNullable } from "@dxos/util";
+var __dxlog_file2 = "/home/runner/work/dxos/dxos/packages/core/functions/src/handler.ts";
+var subscriptionHandler = (handler) => {
+  return ({ event: { data }, context, ...rest }) => {
+    const { client } = context;
+    const space = data.spaceKey ? client.spaces.get(PublicKey2.from(data.spaceKey)) : void 0;
+    const objects = space ? data.objects?.map((id) => space.db.getObjectById(id)).filter(nonNullable) : [];
+    if (!!data.spaceKey && !space) {
+      log2.warn("invalid space", {
+        data
+      }, {
+        F: __dxlog_file2,
+        L: 91,
+        S: void 0,
+        C: (f, a) => f(...a)
+      });
+    } else {
+      log2.info("handler", {
+        space: space?.key.truncate(),
+        objects: objects?.length
+      }, {
+        F: __dxlog_file2,
+        L: 93,
+        S: void 0,
+        C: (f, a) => f(...a)
+      });
+    }
+    return handler({
+      event: {
+        data: {
+          ...data,
+          space,
+          objects
+        }
+      },
+      context,
+      ...rest
+    });
+  };
 };
 
 // packages/core/functions/src/runtime/dev-server.ts
@@ -198,10 +222,9 @@ import { join } from "@dxos/node-std/path";
 import { Event as Event2, Trigger } from "@dxos/async";
 import { Context } from "@dxos/context";
 import { invariant } from "@dxos/invariant";
-import { log as log2 } from "@dxos/log";
-var __dxlog_file2 = "/home/runner/work/dxos/dxos/packages/core/functions/src/runtime/dev-server.ts";
+import { log as log3 } from "@dxos/log";
+var __dxlog_file3 = "/home/runner/work/dxos/dxos/packages/core/functions/src/runtime/dev-server.ts";
 var DevServer = class {
-  // prettier-ignore
   constructor(_client, _functionsRegistry, _options) {
     this._client = _client;
     this._functionsRegistry = _functionsRegistry;
@@ -210,14 +233,14 @@ var DevServer = class {
     this._handlers = {};
     this._seq = 0;
     this.update = new Event2();
-    this._functionsRegistry.onFunctionsRegistered.on(async ({ newFunctions }) => {
-      newFunctions.forEach((def) => this._load(def));
+    this._functionsRegistry.registered.on(async ({ added }) => {
+      added.forEach((def) => this._load(def));
       await this._safeUpdateRegistration();
-      log2("new functions loaded", {
-        newFunctions
+      log3("new functions loaded", {
+        added
       }, {
-        F: __dxlog_file2,
-        L: 53,
+        F: __dxlog_file3,
+        L: 52,
         S: this,
         C: (f, a) => f(...a)
       });
@@ -230,8 +253,8 @@ var DevServer = class {
   }
   get endpoint() {
     invariant(this._port, void 0, {
-      F: __dxlog_file2,
-      L: 64,
+      F: __dxlog_file3,
+      L: 63,
       S: this,
       A: [
         "this._port",
@@ -248,17 +271,17 @@ var DevServer = class {
   }
   async start() {
     invariant(!this._server, void 0, {
-      F: __dxlog_file2,
-      L: 77,
+      F: __dxlog_file3,
+      L: 76,
       S: this,
       A: [
         "!this._server",
         ""
       ]
     });
-    log2.info("starting...", void 0, {
-      F: __dxlog_file2,
-      L: 78,
+    log3.info("starting...", void 0, {
+      F: __dxlog_file3,
+      L: 77,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -268,11 +291,11 @@ var DevServer = class {
     app.post("/:path", async (req, res) => {
       const { path: path2 } = req.params;
       try {
-        log2.info("calling", {
+        log3.info("calling", {
           path: path2
         }, {
-          F: __dxlog_file2,
-          L: 88,
+          F: __dxlog_file3,
+          L: 87,
           S: this,
           C: (f, a) => f(...a)
         });
@@ -283,9 +306,9 @@ var DevServer = class {
         res.statusCode = await this.invoke("/" + path2, req.body);
         res.end();
       } catch (err) {
-        log2.catch(err, void 0, {
-          F: __dxlog_file2,
-          L: 98,
+        log3.catch(err, void 0, {
+          F: __dxlog_file3,
+          L: 97,
           S: this,
           C: (f, a) => f(...a)
         });
@@ -306,11 +329,11 @@ var DevServer = class {
       const { registrationId, endpoint } = await this._client.services.services.FunctionRegistryService.register({
         endpoint: this.endpoint
       });
-      log2.info("registered", {
+      log3.info("registered", {
         endpoint
       }, {
-        F: __dxlog_file2,
-        L: 113,
+        F: __dxlog_file3,
+        L: 112,
         S: this,
         C: (f, a) => f(...a)
       });
@@ -321,44 +344,44 @@ var DevServer = class {
       await this.stop();
       throw new Error("FunctionRegistryService not available (check plugin is configured).");
     }
-    log2.info("started", {
+    log3.info("started", {
       port: this._port
     }, {
-      F: __dxlog_file2,
-      L: 124,
+      F: __dxlog_file3,
+      L: 123,
       S: this,
       C: (f, a) => f(...a)
     });
   }
   async stop() {
     invariant(this._server, void 0, {
-      F: __dxlog_file2,
-      L: 128,
+      F: __dxlog_file3,
+      L: 127,
       S: this,
       A: [
         "this._server",
         ""
       ]
     });
-    log2.info("stopping...", void 0, {
-      F: __dxlog_file2,
-      L: 129,
+    log3.info("stopping...", void 0, {
+      F: __dxlog_file3,
+      L: 128,
       S: this,
       C: (f, a) => f(...a)
     });
     const trigger = new Trigger();
     this._server.close(async () => {
-      log2.info("server stopped", void 0, {
-        F: __dxlog_file2,
-        L: 133,
+      log3.info("server stopped", void 0, {
+        F: __dxlog_file3,
+        L: 132,
         S: this,
         C: (f, a) => f(...a)
       });
       try {
         if (this._functionServiceRegistration) {
           invariant(this._client.services.services.FunctionRegistryService, void 0, {
-            F: __dxlog_file2,
-            L: 136,
+            F: __dxlog_file3,
+            L: 135,
             S: this,
             A: [
               "this._client.services.services.FunctionRegistryService",
@@ -368,11 +391,11 @@ var DevServer = class {
           await this._client.services.services.FunctionRegistryService.unregister({
             registrationId: this._functionServiceRegistration
           });
-          log2.info("unregistered", {
+          log3.info("unregistered", {
             registrationId: this._functionServiceRegistration
           }, {
-            F: __dxlog_file2,
-            L: 141,
+            F: __dxlog_file3,
+            L: 140,
             S: this,
             C: (f, a) => f(...a)
           });
@@ -387,9 +410,9 @@ var DevServer = class {
     await trigger.wait();
     this._port = void 0;
     this._server = void 0;
-    log2.info("stopped", void 0, {
-      F: __dxlog_file2,
-      L: 155,
+    log3.info("stopped", void 0, {
+      F: __dxlog_file3,
+      L: 154,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -400,12 +423,12 @@ var DevServer = class {
   async _load(def, force = false) {
     const { uri, route, handler } = def;
     const filePath = join(this._options.baseDir, handler);
-    log2.info("loading", {
+    log3.info("loading", {
       uri,
       force
     }, {
-      F: __dxlog_file2,
-      L: 164,
+      F: __dxlog_file3,
+      L: 163,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -425,8 +448,8 @@ var DevServer = class {
   }
   async _safeUpdateRegistration() {
     invariant(this._functionServiceRegistration, void 0, {
-      F: __dxlog_file2,
-      L: 186,
+      F: __dxlog_file3,
+      L: 185,
       S: this,
       A: [
         "this._functionServiceRegistration",
@@ -442,9 +465,9 @@ var DevServer = class {
         }))
       });
     } catch (e) {
-      log2.catch(e, void 0, {
-        F: __dxlog_file2,
-        L: 193,
+      log3.catch(e, void 0, {
+        F: __dxlog_file3,
+        L: 192,
         S: this,
         C: (f, a) => f(...a)
       });
@@ -456,26 +479,26 @@ var DevServer = class {
   async invoke(path2, data) {
     const seq = ++this._seq;
     const now = Date.now();
-    log2.info("req", {
+    log3.info("req", {
       seq,
       path: path2
     }, {
-      F: __dxlog_file2,
-      L: 204,
+      F: __dxlog_file3,
+      L: 203,
       S: this,
       C: (f, a) => f(...a)
     });
     const statusCode = await this._invoke(path2, {
       data
     });
-    log2.info("res", {
+    log3.info("res", {
       seq,
       path: path2,
       statusCode,
       duration: Date.now() - now
     }, {
-      F: __dxlog_file2,
-      L: 207,
+      F: __dxlog_file3,
+      L: 206,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -485,8 +508,8 @@ var DevServer = class {
   async _invoke(path2, event) {
     const { handler } = this._handlers[path2] ?? {};
     invariant(handler, `invalid path: ${path2}`, {
-      F: __dxlog_file2,
-      L: 214,
+      F: __dxlog_file3,
+      L: 213,
       S: this,
       A: [
         "handler",
@@ -518,17 +541,19 @@ var createContext = () => new Context({
 
 // packages/core/functions/src/runtime/scheduler.ts
 import path from "@dxos/node-std/path";
+import { Mutex } from "@dxos/async";
 import { Context as Context2 } from "@dxos/context";
-import { log as log3 } from "@dxos/log";
-var __dxlog_file3 = "/home/runner/work/dxos/dxos/packages/core/functions/src/runtime/scheduler.ts";
+import { log as log4 } from "@dxos/log";
+var __dxlog_file4 = "/home/runner/work/dxos/dxos/packages/core/functions/src/runtime/scheduler.ts";
 var Scheduler = class {
   constructor(functions, triggers, _options = {}) {
     this.functions = functions;
     this.triggers = triggers;
     this._options = _options;
     this._ctx = createContext2();
-    this.functions.onFunctionsRegistered.on(async ({ space, newFunctions }) => {
-      await this._safeActivateTriggers(space, this.triggers.getInactiveTriggers(space), newFunctions);
+    this._callMutex = new Mutex();
+    this.functions.registered.on(async ({ space, added }) => {
+      await this._safeActivateTriggers(space, this.triggers.getInactiveTriggers(space), added);
     });
     this.triggers.registered.on(async ({ space, triggers: triggers2 }) => {
       await this._safeActivateTriggers(space, triggers2, this.functions.getFunctions(space));
@@ -545,24 +570,25 @@ var Scheduler = class {
     await this.functions.close();
     await this.triggers.close();
   }
+  // TODO(burdon): Remove and update registries directly.
   async register(space, manifest) {
-    await this.functions.register(space, manifest);
+    await this.functions.register(space, manifest.functions);
     await this.triggers.register(space, manifest);
   }
   async _safeActivateTriggers(space, triggers, functions) {
     const mountTasks = triggers.map((trigger) => {
       return this.activate(space, functions, trigger);
     });
-    await Promise.all(mountTasks).catch(log3.catch);
+    await Promise.all(mountTasks).catch(log4.catch);
   }
   async activate(space, functions, fnTrigger) {
     const definition = functions.find((def) => def.uri === fnTrigger.function);
     if (!definition) {
-      log3.info("function is not found for trigger", {
+      log4.info("function is not found for trigger", {
         fnTrigger
       }, {
-        F: __dxlog_file3,
-        L: 74,
+        F: __dxlog_file4,
+        L: 78,
         S: this,
         C: (f, a) => f(...a)
       });
@@ -571,25 +597,27 @@ var Scheduler = class {
     await this.triggers.activate({
       space
     }, fnTrigger, async (args) => {
-      return this._execFunction(definition, {
-        meta: fnTrigger.meta,
-        data: {
-          ...args,
-          spaceKey: space.key
-        }
+      return this._callMutex.executeSynchronized(() => {
+        return this._execFunction(definition, fnTrigger, {
+          meta: fnTrigger.meta,
+          data: {
+            ...args,
+            spaceKey: space.key
+          }
+        });
       });
     });
-    log3("activated trigger", {
+    log4("activated trigger", {
       space: space.key,
       trigger: fnTrigger
     }, {
-      F: __dxlog_file3,
-      L: 84,
+      F: __dxlog_file4,
+      L: 91,
       S: this,
       C: (f, a) => f(...a)
     });
   }
-  async _execFunction(def, { data, meta }) {
+  async _execFunction(def, trigger, { data, meta }) {
     let status = 0;
     try {
       const payload = Object.assign({}, meta && {
@@ -598,12 +626,13 @@ var Scheduler = class {
       const { endpoint, callback } = this._options;
       if (endpoint) {
         const url = path.join(endpoint, def.route);
-        log3.info("exec", {
+        log4.info("exec", {
           function: def.uri,
-          url
+          url,
+          triggerType: trigger.spec.type
         }, {
-          F: __dxlog_file3,
-          L: 100,
+          F: __dxlog_file4,
+          L: 108,
           S: this,
           C: (f, a) => f(...a)
         });
@@ -616,11 +645,11 @@ var Scheduler = class {
         });
         status = response.status;
       } else if (callback) {
-        log3.info("exec", {
+        log4.info("exec", {
           function: def.uri
         }, {
-          F: __dxlog_file3,
-          L: 111,
+          F: __dxlog_file4,
+          L: 119,
           S: this,
           C: (f, a) => f(...a)
         });
@@ -629,22 +658,22 @@ var Scheduler = class {
       if (status && status >= 400) {
         throw new Error(`Response: ${status}`);
       }
-      log3.info("done", {
+      log4.info("done", {
         function: def.uri,
         status
       }, {
-        F: __dxlog_file3,
-        L: 121,
+        F: __dxlog_file4,
+        L: 129,
         S: this,
         C: (f, a) => f(...a)
       });
     } catch (err) {
-      log3.error("error", {
+      log4.error("error", {
         function: def.uri,
         error: err.message
       }, {
-        F: __dxlog_file3,
-        L: 123,
+        F: __dxlog_file4,
+        L: 131,
         S: this,
         C: (f, a) => f(...a)
       });
@@ -659,58 +688,65 @@ var createContext2 = () => new Context2({
 
 // packages/core/functions/src/trigger/trigger-registry.ts
 import { Event as Event3 } from "@dxos/async";
-import { create as create2, Filter as Filter3 } from "@dxos/client/echo";
+import { create as create2, Filter as Filter3, getMeta } from "@dxos/client/echo";
 import { Context as Context3, Resource as Resource2 } from "@dxos/context";
+import { ECHO_ATTR_META, foreignKey, foreignKeyEquals, splitMeta } from "@dxos/echo-schema";
 import { invariant as invariant2 } from "@dxos/invariant";
 import { PublicKey as PublicKey3 } from "@dxos/keys";
-import { log as log8 } from "@dxos/log";
+import { log as log9 } from "@dxos/log";
 import { ComplexMap as ComplexMap2 } from "@dxos/util";
 
 // packages/core/functions/src/trigger/type/subscription-trigger.ts
 import { TextV0Type } from "@braneframe/types";
-import { debounce, DeferredTask } from "@dxos/async";
+import { debounce, UpdateScheduler } from "@dxos/async";
 import { createSubscription, Filter as Filter2, getAutomergeObjectCore } from "@dxos/echo-db";
-import { log as log4 } from "@dxos/log";
-var __dxlog_file4 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/type/subscription-trigger.ts";
+import { log as log5 } from "@dxos/log";
+var __dxlog_file5 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/type/subscription-trigger.ts";
 var createSubscriptionTrigger = async (ctx, triggerCtx, spec, callback) => {
   const objectIds = /* @__PURE__ */ new Set();
-  const task = new DeferredTask(ctx, async () => {
+  const task = new UpdateScheduler(ctx, async () => {
     if (objectIds.size > 0) {
+      const objects = Array.from(objectIds);
+      objectIds.clear();
       await callback({
-        objects: Array.from(objectIds)
+        objects
       });
-      objectIds.clear();
     }
+  }, {
+    maxFrequency: 4
   });
   const subscriptions = [];
   const subscription = createSubscription(({ added, updated }) => {
-    log4.info("updated", {
-      added: added.length,
-      updated: updated.length
-    }, {
-      F: __dxlog_file4,
-      L: 32,
-      S: void 0,
-      C: (f, a) => f(...a)
-    });
+    const sizeBefore = objectIds.size;
     for (const object of added) {
       objectIds.add(object.id);
     }
     for (const object of updated) {
       objectIds.add(object.id);
     }
-    task.schedule();
+    if (objectIds.size > sizeBefore) {
+      log5.info("updated", {
+        added: added.length,
+        updated: updated.length
+      }, {
+        F: __dxlog_file5,
+        L: 45,
+        S: void 0,
+        C: (f, a) => f(...a)
+      });
+      task.trigger();
+    }
   });
   subscriptions.push(() => subscription.unsubscribe());
   const { filter, options: { deep, delay } = {} } = spec;
   const update = ({ objects }) => {
     subscription.update(objects);
     if (deep) {
-      log4.info("update", {
+      log5.info("update", {
         objects: objects.length
       }, {
-        F: __dxlog_file4,
-        L: 52,
+        F: __dxlog_file5,
+        L: 59,
         S: void 0,
         C: (f, a) => f(...a)
       });
@@ -733,11 +769,11 @@ var createSubscriptionTrigger = async (ctx, triggerCtx, spec, callback) => {
 
 // packages/core/functions/src/trigger/type/timer-trigger.ts
 import { CronJob } from "cron";
-import { DeferredTask as DeferredTask2 } from "@dxos/async";
-import { log as log5 } from "@dxos/log";
-var __dxlog_file5 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/type/timer-trigger.ts";
+import { DeferredTask } from "@dxos/async";
+import { log as log6 } from "@dxos/log";
+var __dxlog_file6 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/type/timer-trigger.ts";
 var createTimerTrigger = async (ctx, triggerContext, spec, callback) => {
-  const task = new DeferredTask2(ctx, async () => {
+  const task = new DeferredTask(ctx, async () => {
     await callback({});
   });
   let last = 0;
@@ -750,12 +786,12 @@ var createTimerTrigger = async (ctx, triggerContext, spec, callback) => {
       const delta = last ? now - last : 0;
       last = now;
       run++;
-      log5.info("tick", {
+      log6.info("tick", {
         space: triggerContext.space.key.truncate(),
         count: run,
         delta
       }, {
-        F: __dxlog_file5,
+        F: __dxlog_file6,
         L: 37,
         S: void 0,
         C: (f, a) => f(...a)
@@ -770,8 +806,8 @@ var createTimerTrigger = async (ctx, triggerContext, spec, callback) => {
 // packages/core/functions/src/trigger/type/webhook-trigger.ts
 import { getPort as getPort2 } from "get-port-please";
 import http from "@dxos/node-std/http";
-import { log as log6 } from "@dxos/log";
-var __dxlog_file6 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/type/webhook-trigger.ts";
+import { log as log7 } from "@dxos/log";
+var __dxlog_file7 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/type/webhook-trigger.ts";
 var createWebhookTrigger = async (ctx, _, spec, callback) => {
   const server = http.createServer(async (req, res) => {
     if (req.method !== spec.method) {
@@ -785,10 +821,10 @@ var createWebhookTrigger = async (ctx, _, spec, callback) => {
     random: true
   });
   server.listen(port, () => {
-    log6.info("started webhook", {
+    log7.info("started webhook", {
       port
     }, {
-      F: __dxlog_file6,
+      F: __dxlog_file7,
       L: 40,
       S: void 0,
       C: (f, a) => f(...a)
@@ -803,8 +839,8 @@ var createWebhookTrigger = async (ctx, _, spec, callback) => {
 // packages/core/functions/src/trigger/type/websocket-trigger.ts
 import WebSocket from "ws";
 import { sleep, Trigger as Trigger2 } from "@dxos/async";
-import { log as log7 } from "@dxos/log";
-var __dxlog_file7 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/type/websocket-trigger.ts";
+import { log as log8 } from "@dxos/log";
+var __dxlog_file8 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/type/websocket-trigger.ts";
 var createWebsocketTrigger = async (ctx, triggerCtx, spec, callback, options = {
   retryDelay: 2,
   maxAttempts: 5
@@ -816,10 +852,10 @@ var createWebsocketTrigger = async (ctx, triggerCtx, spec, callback, options = {
     ws = new WebSocket(url);
     Object.assign(ws, {
       onopen: () => {
-        log7.info("opened", {
+        log8.info("opened", {
           url
         }, {
-          F: __dxlog_file7,
+          F: __dxlog_file8,
           L: 39,
           S: void 0,
           C: (f, a) => f(...a)
@@ -830,21 +866,21 @@ var createWebsocketTrigger = async (ctx, triggerCtx, spec, callback, options = {
         open.wake(true);
       },
       onclose: (event) => {
-        log7.info("closed", {
+        log8.info("closed", {
           url,
           code: event.code
         }, {
-          F: __dxlog_file7,
+          F: __dxlog_file8,
           L: 48,
           S: void 0,
           C: (f, a) => f(...a)
         });
         if (event.code === 1006) {
           setTimeout(async () => {
-            log7.info(`reconnecting in ${options.retryDelay}s...`, {
+            log8.info(`reconnecting in ${options.retryDelay}s...`, {
               url
             }, {
-              F: __dxlog_file7,
+              F: __dxlog_file8,
               L: 53,
               S: void 0,
               C: (f, a) => f(...a)
@@ -855,10 +891,10 @@ var createWebsocketTrigger = async (ctx, triggerCtx, spec, callback, options = {
         open.wake(false);
       },
       onerror: (event) => {
-        log7.catch(event.error, {
+        log8.catch(event.error, {
           url
         }, {
-          F: __dxlog_file7,
+          F: __dxlog_file8,
           L: 62,
           S: void 0,
           C: (f, a) => f(...a)
@@ -866,8 +902,8 @@ var createWebsocketTrigger = async (ctx, triggerCtx, spec, callback, options = {
       },
       onmessage: async (event) => {
         try {
-          log7.info("message", void 0, {
-            F: __dxlog_file7,
+          log8.info("message", void 0, {
+            F: __dxlog_file8,
             L: 67,
             S: void 0,
             C: (f, a) => f(...a)
@@ -877,10 +913,10 @@ var createWebsocketTrigger = async (ctx, triggerCtx, spec, callback, options = {
             data
           });
         } catch (err) {
-          log7.catch(err, {
+          log8.catch(err, {
             url
           }, {
-            F: __dxlog_file7,
+            F: __dxlog_file8,
             L: 71,
             S: void 0,
             C: (f, a) => f(...a)
@@ -894,10 +930,10 @@ var createWebsocketTrigger = async (ctx, triggerCtx, spec, callback, options = {
     } else {
       const wait = Math.pow(attempt, 2) * options.retryDelay;
       if (attempt < options.maxAttempts) {
-        log7.warn(`failed to connect; trying again in ${wait}s`, {
+        log8.warn(`failed to connect; trying again in ${wait}s`, {
           attempt
         }, {
-          F: __dxlog_file7,
+          F: __dxlog_file8,
           L: 82,
           S: void 0,
           C: (f, a) => f(...a)
@@ -912,7 +948,7 @@ var createWebsocketTrigger = async (ctx, triggerCtx, spec, callback, options = {
 };
 
 // packages/core/functions/src/trigger/trigger-registry.ts
-var __dxlog_file8 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/trigger-registry.ts";
+var __dxlog_file9 = "/home/runner/work/dxos/dxos/packages/core/functions/src/trigger/trigger-registry.ts";
 var triggerHandlers = {
   subscription: createSubscriptionTrigger,
   timer: createTimerTrigger,
@@ -935,12 +971,12 @@ var TriggerRegistry = class extends Resource2 {
     return this._getTriggers(space, (t) => t.activationCtx == null);
   }
   async activate(triggerCtx, trigger, callback) {
-    log8("activate", {
+    log9("activate", {
       space: triggerCtx.space.key,
       trigger
     }, {
-      F: __dxlog_file8,
-      L: 73,
+      F: __dxlog_file9,
+      L: 75,
       S: this,
       C: (f, a) => f(...a)
     });
@@ -950,8 +986,8 @@ var TriggerRegistry = class extends Resource2 {
     this._ctx.onDispose(() => activationCtx.dispose());
     const registeredTrigger = this._triggersBySpaceKey.get(triggerCtx.space.key)?.find((reg) => reg.trigger.id === trigger.id);
     invariant2(registeredTrigger, `Trigger is not registered: ${trigger.function}`, {
-      F: __dxlog_file8,
-      L: 79,
+      F: __dxlog_file9,
+      L: 81,
       S: this,
       A: [
         "registeredTrigger",
@@ -971,24 +1007,35 @@ var TriggerRegistry = class extends Resource2 {
   * Loads triggers from the manifest into the space.
   */
   async register(space, manifest) {
-    log8("register", {
+    log9("register", {
       space: space.key
     }, {
-      F: __dxlog_file8,
-      L: 95,
+      F: __dxlog_file9,
+      L: 97,
       S: this,
       C: (f, a) => f(...a)
     });
     if (!manifest.triggers?.length) {
       return;
     }
-    if (!space.db.graph.runtimeSchemaRegistry.isSchemaRegistered(FunctionTrigger)) {
+    if (!space.db.graph.runtimeSchemaRegistry.hasSchema(FunctionTrigger)) {
       space.db.graph.runtimeSchemaRegistry.registerSchema(FunctionTrigger);
     }
-    const reactiveObjects = manifest.triggers.map((template) => create2(FunctionTrigger, {
-      ...template
-    }));
-    reactiveObjects.forEach((obj) => space.db.add(obj));
+    const { objects: existing } = await space.db.query(Filter3.schema(FunctionTrigger)).run();
+    const { added, removed } = diff(existing, manifest.triggers, (a, b) => {
+      const keys = b[ECHO_ATTR_META]?.keys ?? [
+        foreignKey("manifest", [
+          b.function,
+          b.spec.type
+        ].join("-"))
+      ];
+      return intersection(getMeta(a)?.keys ?? [], keys, foreignKeyEquals).length > 0;
+    });
+    added.forEach((trigger) => {
+      const { meta, object } = splitMeta(trigger);
+      space.db.add(create2(FunctionTrigger, object, meta));
+    });
+    removed.forEach((trigger) => space.db.remove(trigger));
   }
   async _open() {
     const spaceListSubscription = this._client.spaces.subscribe(async (spaces) => {
@@ -1023,12 +1070,12 @@ var TriggerRegistry = class extends Resource2 {
         trigger
       }));
       registered.push(...newRegisteredTriggers);
-      log8("registered new triggers", () => ({
+      log9("registered new triggers", () => ({
         spaceKey: space.key,
         functions: newTriggers.map((t) => t.function)
       }), {
-        F: __dxlog_file8,
-        L: 146,
+        F: __dxlog_file9,
+        L: 159,
         S: this,
         C: (f, a) => f(...a)
       });