@dxos/compute 0.8.4-main.bc674ce → 0.8.4-main.bcb3aa67d6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dxos/compute",
-  "version": "0.8.4-main.bc674ce",
+  "version": "0.8.4-main.bcb3aa67d6",
   "description": "Low-level compute API",
   "homepage": "https://dxos.org",
   "bugs": "https://github.com/dxos/dxos/issues",
@@ -16,14 +16,12 @@
     ".": {
       "source": "./src/index.ts",
       "types": "./dist/types/src/index.d.ts",
-      "browser": "./dist/lib/browser/index.mjs",
-      "node": "./dist/lib/node-esm/index.mjs"
+      "default": "./dist/lib/neutral/index.mjs"
     },
     "./testing": {
       "source": "./src/testing/index.ts",
       "types": "./dist/types/src/testing/index.d.ts",
-      "browser": "./dist/lib/browser/testing/index.mjs",
-      "node": "./dist/lib/node-esm/testing/index.mjs"
+      "default": "./dist/lib/neutral/testing/index.mjs"
     }
   },
   "types": "dist/types/src/index.d.ts",
@@ -39,34 +37,35 @@
     "src"
   ],
   "dependencies": {
-    "effect": "3.19.11",
+    "effect": "3.20.0",
     "lodash.defaultsdeep": "^4.6.1",
-    "@dxos/ai": "0.8.4-main.bc674ce",
-    "@dxos/async": "0.8.4-main.bc674ce",
-    "@dxos/context": "0.8.4-main.bc674ce",
-    "@dxos/debug": "0.8.4-main.bc674ce",
-    "@dxos/client": "0.8.4-main.bc674ce",
-    "@dxos/echo": "0.8.4-main.bc674ce",
-    "@dxos/functions": "0.8.4-main.bc674ce",
-    "@dxos/functions-runtime": "0.8.4-main.bc674ce",
-    "@dxos/invariant": "0.8.4-main.bc674ce",
-    "@dxos/keys": "0.8.4-main.bc674ce",
-    "@dxos/log": "0.8.4-main.bc674ce",
-    "@dxos/plugin-automation": "0.8.4-main.bc674ce",
-    "@dxos/schema": "0.8.4-main.bc674ce",
-    "@dxos/util": "0.8.4-main.bc674ce",
-    "@dxos/protocols": "0.8.4-main.bc674ce",
-    "@dxos/vendor-hyperformula": "0.8.4-main.bc674ce"
+    "@dxos/async": "0.8.4-main.bcb3aa67d6",
+    "@dxos/context": "0.8.4-main.bcb3aa67d6",
+    "@dxos/debug": "0.8.4-main.bcb3aa67d6",
+    "@dxos/echo": "0.8.4-main.bcb3aa67d6",
+    "@dxos/ai": "0.8.4-main.bcb3aa67d6",
+    "@dxos/client": "0.8.4-main.bcb3aa67d6",
+    "@dxos/functions": "0.8.4-main.bcb3aa67d6",
+    "@dxos/invariant": "0.8.4-main.bcb3aa67d6",
+    "@dxos/keys": "0.8.4-main.bcb3aa67d6",
+    "@dxos/functions-runtime": "0.8.4-main.bcb3aa67d6",
+    "@dxos/operation": "0.8.4-main.bcb3aa67d6",
+    "@dxos/log": "0.8.4-main.bcb3aa67d6",
+    "@dxos/schema": "0.8.4-main.bcb3aa67d6",
+    "@dxos/vendor-hyperformula": "0.8.4-main.bcb3aa67d6",
+    "@dxos/plugin-automation": "0.8.4-main.bcb3aa67d6",
+    "@dxos/util": "0.8.4-main.bcb3aa67d6",
+    "@dxos/protocols": "0.8.4-main.bcb3aa67d6"
   },
   "devDependencies": {
     "@tldraw/indices": "^2.0.0-alpha.14",
     "@types/lodash.defaultsdeep": "^4.6.6",
-    "@dxos/echo": "0.8.4-main.bc674ce",
-    "@dxos/echo-db": "0.8.4-main.bc674ce",
-    "@dxos/random": "0.8.4-main.bc674ce"
+    "@dxos/echo": "0.8.4-main.bcb3aa67d6",
+    "@dxos/echo-db": "0.8.4-main.bcb3aa67d6",
+    "@dxos/random": "0.8.4-main.bcb3aa67d6"
   },
   "peerDependencies": {
-    "effect": "3.19.11"
+    "effect": "3.20.0"
   },
   "publishConfig": {
     "access": "public"

package/src/compute-graph-registry.test.ts

@@ -2,32 +2,43 @@
 // Copyright 2025 DXOS.org
 //
 
+import * as Effect from 'effect/Effect';
 import * as Schema from 'effect/Schema';
 import { describe, expect, onTestFinished, test } from 'vitest';
 
 import { Trigger } from '@dxos/async';
 import { Obj } from '@dxos/echo';
-import { Function, FunctionDefinition } from '@dxos/functions';
+import { Operation, OperationHandlerSet } from '@dxos/operation';
 
 import { ComputeGraphRegistry, defaultPlugins } from './compute-graph-registry';
 import { TestBuilder, createMockedComputeRuntimeProvider } from './testing';
 
-describe('ComputeGraphRegistry', () => {
-  const add = FunctionDefinition.make({
+const AddDefinition = Operation.make({
+  meta: {
     key: 'add',
     name: 'add',
     description: 'Adds two numbers',
-    inputSchema: Schema.Struct({ a: Schema.Number, b: Schema.Number }),
-    outputSchema: Schema.Number,
-    handler: ({ data }) => data.a + data.b,
-  });
+  },
+  input: Schema.Struct({ a: Schema.Number, b: Schema.Number }),
+  output: Schema.Number,
+  services: [],
+});
 
+const add = AddDefinition.pipe(
+  Operation.withHandler(
+    Effect.fn(function* ({ a, b }) {
+      return a + b;
+    }),
+  ),
+);
+
+describe('ComputeGraphRegistry', () => {
   test('invokes user function through compute graph', async () => {
     const computeRuntime = createMockedComputeRuntimeProvider({
-      functions: [add],
+      functions: OperationHandlerSet.make(add),
     });
     const testBuilder = new TestBuilder({
-      types: [Function.Function],
+      types: [Operation.PersistentOperation],
       computeRuntime,
     });
     await testBuilder.open();
@@ -53,10 +64,9 @@ describe('ComputeGraphRegistry', () => {
       await graph.close();
     });
 
-    // Create a function object in ECHO with binding that maps to DX calls.
-    const functionObj = FunctionDefinition.serialize(add);
-    Obj.change(functionObj, (f) => {
-      f.binding = 'ADD';
+    const functionObj = Operation.serialize(add);
+    Obj.change(functionObj, (obj) => {
+      obj.binding = 'ADD';
     });
     space.db.add(functionObj);
 
@@ -84,10 +94,10 @@ describe('ComputeGraphRegistry', () => {
 
   test('adding a function binding updates autocomplete and enables execution', async () => {
     const computeRuntime = createMockedComputeRuntimeProvider({
-      functions: [add],
+      functions: OperationHandlerSet.make(add),
     });
     const testBuilder = new TestBuilder({
-      types: [Function.Function],
+      types: [Operation.PersistentOperation],
       computeRuntime,
     });
     await testBuilder.open();
@@ -124,10 +134,9 @@ describe('ComputeGraphRegistry', () => {
     });
     onTestFinished(unsubscribe);
 
-    // Add a bound function to the space; graph should pick it up.
-    const functionObj = FunctionDefinition.serialize(add);
-    Obj.change(functionObj, (f) => {
-      f.binding = 'ADD';
+    const functionObj = Operation.serialize(add);
+    Obj.change(functionObj, (obj) => {
+      obj.binding = 'ADD';
     });
     space.db.add(functionObj);
 

package/src/compute-graph.test.ts

@@ -6,7 +6,7 @@ import { afterEach, beforeEach, describe, expect, test } from 'vitest';
 
 import { Trigger } from '@dxos/async';
 import { Obj } from '@dxos/echo';
-import { Function } from '@dxos/functions';
+import { Operation } from '@dxos/operation';
 import { type CellValue } from '@dxos/vendor-hyperformula';
 import { DetailedCellError } from '@dxos/vendor-hyperformula';
 
@@ -15,7 +15,7 @@ import { TestBuilder } from './testing';
 describe('ComputeGraph', () => {
   let testBuilder: TestBuilder;
   beforeEach(async () => {
-    testBuilder = new TestBuilder({ types: [Function.Function] });
+    testBuilder = new TestBuilder({ types: [Operation.PersistentOperation] });
     await testBuilder.open();
   });
   afterEach(async () => {
@@ -30,13 +30,45 @@ describe('ComputeGraph', () => {
     // Create script.
     const trigger = new Trigger();
     graph.update.once(() => trigger.wake());
-    const functionObject = space.db.add(Function.make({ name: 'test', version: '0.0.1', binding: 'TEST' }));
+    const functionObject = space.db.add(
+      Obj.make(Operation.PersistentOperation, { name: 'test', version: '0.0.1', binding: 'TEST' }),
+    );
     await trigger.wait();
     const functions = graph.getFunctions({ echo: true });
     expect(functions).to.toHaveLength(1);
 
     const id = graph.mapFunctionBindingToId('TEST()');
     expect(id).to.eq(`${Obj.getDXN(functionObject).toString()}()`);
+
+    const restored = graph.mapFunctionBindingFromId(id);
+    expect(restored).to.eq('TEST()');
+  });
+
+  test('map functions roundtrip with args', async () => {
+    const space = await testBuilder.client.spaces.create();
+    const graph = testBuilder.registry.createGraph(space);
+    await graph.open();
+
+    const trigger = new Trigger();
+    graph.update.once(() => trigger.wake());
+    space.db.add(Obj.make(Operation.PersistentOperation, { name: 'forex', version: '0.0.1', binding: 'FOREX' }));
+    await trigger.wait();
+
+    const stored = graph.mapFunctionBindingToId('=FOREX(C6,C7)');
+    expect(stored).to.include('dxn:echo:');
+    expect(stored).not.to.include('FOREX');
+
+    const restored = graph.mapFunctionBindingFromId(stored);
+    expect(restored).to.eq('=FOREX(C6,C7)');
+  });
+
+  test('map functions unknown binding returns undefined', async () => {
+    const space = await testBuilder.client.spaces.create();
+    const graph = testBuilder.registry.createGraph(space);
+    await graph.open();
+
+    const result = graph.mapFunctionBindingFromId('=dxn:echo:@:01ABCDEF01ABCDEF01ABCDEF01(A1,B1)');
+    expect(result).to.be.undefined;
   });
 
   test('cross-node references', async () => {

package/src/compute-graph.ts

@@ -6,10 +6,10 @@ import type * as ManagedRuntime from 'effect/ManagedRuntime';
 
 import { Event } from '@dxos/async';
 import { Filter, type Space } from '@dxos/client/echo';
-import { FQ_ID_LENGTH } from '@dxos/client/echo';
 import { Resource } from '@dxos/context';
 import { Obj } from '@dxos/echo';
-import { Function, type FunctionInvocationService } from '@dxos/functions';
+import { type FunctionInvocationService } from '@dxos/functions';
+import { Operation } from '@dxos/operation';
 import { invariant } from '@dxos/invariant';
 import { PublicKey } from '@dxos/keys';
 import { log } from '@dxos/log';
@@ -37,7 +37,7 @@ type ObjectRef = { type: string; id: string };
 
 /**
  * Marker for sheets that are managed by an ECHO object.
- * Sheet ID: `dxos.org/type/SheetType@1234`
+ * Sheet ID: `org.dxos.type.sheet-type@1234`
  */
 export const createSheetName = ({ type, id }: ObjectRef) => `${type}@${id}`;
 export const parseSheetName = (name: string): Partial<ObjectRef> => {
@@ -60,7 +60,7 @@ export class ComputeGraph extends Resource {
   private readonly _nodes = new Map<number, ComputeNode>();
 
   // Cached function objects.
-  private _remoteFunctions: Function.Function[] = [];
+  private _remoteFunctions: Operation.PersistentOperation[] = [];
 
   public readonly update = new Event<{ type: ComputeGraphEvent }>();
 
@@ -153,7 +153,7 @@ export class ComputeGraph extends Resource {
       formula
         //
         // Map cross-sheet references by name onto sheet stored by ECHO object/model.
-        // Example: "Test Sheet"!A0 => "dxos.org/type/SheetType@1234"!A0
+        // Example: "Test Sheet"!A0 => "org.dxos.type.sheet-type@1234"!A0
         // https://hyperformula.handsontable.com/guide/cell-references.html#cell-references
         //
         .replace(/['"]?([ \w]+)['"]?!/, (_match, name) => {
@@ -198,8 +198,8 @@ export class ComputeGraph extends Resource {
   }
 
   /**
-   * Map from binding to fully qualified ECHO ID (to store).
-   * E.g., HELLO() => spaceId:objectId()
+   * Map from binding to fully qualified ECHO DXN (to store).
+   * E.g., HELLO() => dxn:echo:spaceId:objectId()
    */
   mapFunctionBindingToId(formula: string): string {
     return formula.replace(/(\w+)\((.*)\)/g, (match, binding, args) => {
@@ -218,23 +218,22 @@ export class ComputeGraph extends Resource {
   }
 
   /**
-   * Map from fully qualified ECHO ID to binding (from store).
-   * E.g., spaceId:objectId() => HELLO()
+   * Map from fully qualified ECHO DXN to binding (from store).
+   * E.g., dxn:echo:spaceId:objectId() => HELLO()
    */
   mapFunctionBindingFromId(formula: string): string | undefined {
-    const binding = formula.replace(/(\w+):([a-zA-Z0-9]+)\((.*)\)/g, (match, spaceId, objectId, args) => {
-      const id = `${spaceId}:${objectId}`;
-      if (id.length !== FQ_ID_LENGTH) {
-        return match;
-      }
-
-      const fn = this._remoteFunctions.find((fn) => Obj.getDXN(fn).toString() === id);
-      if (fn?.binding) {
-        return `${fn.binding}(${args})`;
-      } else {
-        return UNKNOWN_BINDING;
-      }
-    });
+    const binding = formula.replace(
+      /dxn:([^:]+):([^:(]+):([a-zA-Z0-9]+)\((.*)\)/g,
+      (_match, kind, spaceTag, objectId, args) => {
+        const dxn = `dxn:${kind}:${spaceTag}:${objectId}`;
+        const fn = this._remoteFunctions.find((fn) => Obj.getDXN(fn).toString() === dxn);
+        if (fn?.binding) {
+          return `${fn.binding}(${args})`;
+        } else {
+          return UNKNOWN_BINDING;
+        }
+      },
+    );
 
     if (binding.startsWith(`=${UNKNOWN_BINDING}`)) {
       return undefined;
@@ -246,11 +245,14 @@ export class ComputeGraph extends Resource {
   protected override async _open(): Promise<void> {
     if (this._space) {
       // Subscribe to remote function definitions.
-      const query = this._space.db.query(Filter.type(Function.Function));
-      const unsubscribe = query.subscribe(() => {
-        this._remoteFunctions = query.results.filter((fn) => fn.binding);
-        this.update.emit({ type: 'functionsUpdated' });
-      });
+      const query = this._space.db.query(Filter.type(Operation.PersistentOperation));
+      const unsubscribe = query.subscribe(
+        () => {
+          this._remoteFunctions = query.results.filter((fn) => fn.binding);
+          this.update.emit({ type: 'functionsUpdated' });
+        },
+        { fire: true },
+      );
 
       this._ctx.onDispose(unsubscribe);
     }

package/src/functions/edge-function.ts

@@ -7,8 +7,8 @@ import * as SchemaAST from 'effect/SchemaAST';
 
 import { Filter } from '@dxos/client/echo';
 import { JsonSchema, Obj } from '@dxos/echo';
-import { Function, FunctionInvocationService, TracingService } from '@dxos/functions';
-import { FunctionDefinition } from '@dxos/functions';
+import { FunctionInvocationService, TracingService } from '@dxos/functions';
+import { Operation } from '@dxos/operation';
 import { log } from '@dxos/log';
 import { isNonNullable } from '@dxos/util';
 import { type ProcedureAst } from '@dxos/vendor-hyperformula';
@@ -35,7 +35,7 @@ export class EdgeFunctionPlugin extends AsyncFunctionPlugin {
           return new CellError(ErrorType.REF, 'Missing space');
         }
 
-        const [fn] = await space.db.query(Filter.type(Function.Function, { binding })).run();
+        const [fn] = await space.db.query(Filter.type(Operation.PersistentOperation, { binding })).run();
         if (!fn) {
           log.info('Function not found', { binding });
           return new CellError(ErrorType.REF, 'Function not found');
@@ -55,7 +55,7 @@ export class EdgeFunctionPlugin extends AsyncFunctionPlugin {
         }
 
         const runtime = this.context.runtime;
-        const functionDef = FunctionDefinition.deserialize(fn);
+        const functionDef = Operation.deserialize(fn);
         // If input schema exists, construct an object from args using the schema props order, otherwise pass { args }.
         let input: any;
         if (fn.inputSchema) {

package/src/testing/test-runtime.ts

@@ -7,9 +7,10 @@ import * as ManagedRuntime from 'effect/ManagedRuntime';
 
 import { AiService } from '@dxos/ai';
 import { type SpaceId } from '@dxos/client/echo';
-import { Database } from '@dxos/echo';
-import { CredentialsService, type FunctionDefinition, QueueService } from '@dxos/functions';
+import { Database, Feed } from '@dxos/echo';
+import { CredentialsService, QueueService } from '@dxos/functions';
 import { FunctionInvocationServiceLayerTest } from '@dxos/functions-runtime/testing';
+import { OperationHandlerSet } from '@dxos/operation';
 
 import { type FunctionsRuntimeProvider } from '../compute-graph-registry';
 
@@ -19,15 +20,16 @@ import { type FunctionsRuntimeProvider } from '../compute-graph-registry';
  */
 export const createMockedComputeRuntimeProvider = ({
   functions,
-}: { functions?: FunctionDefinition.Any[] } = {}): FunctionsRuntimeProvider => {
+}: { functions?: OperationHandlerSet.OperationHandlerSet } = {}): FunctionsRuntimeProvider => {
   return {
     getRuntime: (_spaceId: SpaceId) =>
       ManagedRuntime.make(
         FunctionInvocationServiceLayerTest({ functions }).pipe(
           Layer.provide(AiService.notAvailable),
           Layer.provide(CredentialsService.configuredLayer([])),
-          Layer.provide(Database.Service.notAvailable),
+          Layer.provide(Database.notAvailable),
           Layer.provide(QueueService.notAvailable),
+          Layer.provide(Feed.notAvailable),
         ),
       ),
   };