@dxos/plugin-space 0.7.5-main.e9bb01b → 0.7.5-staging.2ff1350

package/src/SpacePlugin.tsx

@@ -12,10 +12,12 @@ import {
   Events,
   oneOf,
 } from '@dxos/app-framework';
+import { S } from '@dxos/echo-schema';
 import { RefArray } from '@dxos/live-object';
 import { AttentionEvents } from '@dxos/plugin-attention';
-import { ClientCapabilities, ClientEvents } from '@dxos/plugin-client';
-import { DeckCapabilities } from '@dxos/plugin-deck';
+import { ClientEvents } from '@dxos/plugin-client';
+import { DeckCapabilities, DeckEvents } from '@dxos/plugin-deck';
+import { isEchoObject, getSpace } from '@dxos/react-client/echo';
 import { osTranslations } from '@dxos/shell/react';
 
 import {
@@ -25,6 +27,8 @@ import {
   IntentResolver,
   ReactRoot,
   ReactSurface,
+  Schema,
+  SpaceCapabilities,
   SpaceSettings,
   SpacesReady,
   SpaceState,
@@ -32,7 +36,7 @@ import {
 import { SpaceEvents } from './events';
 import { meta, SPACE_PLUGIN } from './meta';
 import translations from './translations';
-import { CollectionAction, CollectionType } from './types';
+import { CollectionAction, CollectionType, defineObjectForm } from './types';
 
 export type SpacePluginOptions = {
   /**
@@ -44,11 +48,17 @@ export type SpacePluginOptions = {
    * Query parameter for the invitation code.
    */
   invitationParam?: string;
+
+  /**
+   * Whether to send observability events.
+   */
+  observability?: boolean;
 };
 
 export const SpacePlugin = ({
   invitationUrl = window.location.origin,
   invitationParam = 'spaceInvitationCode',
+  observability = true,
 }: SpacePluginOptions = {}) => {
   const createInvitationUrl = (invitationCode: string) => {
     const baseUrl = new URL(invitationUrl);
@@ -59,7 +69,10 @@ export const SpacePlugin = ({
   return definePlugin(meta, [
     defineModule({
       id: `${meta.id}/module/state`,
-      activatesOn: oneOf(Events.Startup, Events.SetupAppGraph),
+      // TODO(wittjosiah): Does not integrate with settings store.
+      //   Should this be a different event?
+      //   Should settings store be renamed to be more generic?
+      activatesOn: oneOf(Events.SetupSettings, Events.SetupAppGraph),
       activatesAfter: [SpaceEvents.StateReady],
       activate: SpaceState,
     }),
@@ -80,7 +93,6 @@ export const SpacePlugin = ({
         contributes(Capabilities.Metadata, {
           id: CollectionType.typename,
           metadata: {
-            createObject: (props: { name?: string }) => createIntent(CollectionAction.Create, props),
             placeholder: ['unnamed collection label', { ns: SPACE_PLUGIN }],
             icon: 'ph--cards-three--regular',
             // TODO(wittjosiah): Move out of metadata.
@@ -89,20 +101,35 @@ export const SpacePlugin = ({
           },
         }),
     }),
+    defineModule({
+      id: `${meta.id}/module/object-form`,
+      activatesOn: ClientEvents.SetupSchema,
+      activate: () =>
+        contributes(
+          SpaceCapabilities.ObjectForm,
+          defineObjectForm({
+            objectSchema: CollectionType,
+            formSchema: S.Struct({ name: S.optional(S.String) }),
+            getIntent: (props) => createIntent(CollectionAction.Create, props),
+          }),
+        ),
+    }),
     defineModule({
       id: `${meta.id}/module/complementary-panel`,
-      activatesOn: Events.Startup,
+      activatesOn: DeckEvents.SetupComplementaryPanels,
       activate: () =>
         contributes(DeckCapabilities.ComplementaryPanel, {
           id: 'settings',
           label: ['settings panel label', { ns: SPACE_PLUGIN }],
           icon: 'ph--sliders--regular',
+          filter: (node) => isEchoObject(node.data) && !!getSpace(node.data),
         }),
     }),
     defineModule({
       id: `${meta.id}/module/schema`,
-      activatesOn: ClientEvents.SetupSchema,
-      activate: () => contributes(ClientCapabilities.Schema, [CollectionType]),
+      activatesOn: ClientEvents.ClientReady,
+      activatesBefore: [ClientEvents.SetupSchema],
+      activate: Schema,
     }),
     defineModule({
       id: `${meta.id}/module/react-root`,
@@ -111,13 +138,15 @@ export const SpacePlugin = ({
     }),
     defineModule({
       id: `${meta.id}/module/react-surface`,
-      activatesOn: Events.SetupSurfaces,
+      activatesOn: Events.SetupReactSurface,
+      // TODO(wittjosiah): Should occur before the settings dialog is loaded when surfaces activation is more granular.
+      activatesBefore: [SpaceEvents.SetupSettingsPanel],
       activate: () => ReactSurface({ createInvitationUrl }),
     }),
     defineModule({
       id: `${meta.id}/module/intent-resolver`,
-      activatesOn: Events.SetupIntents,
-      activate: (context) => IntentResolver({ createInvitationUrl, context }),
+      activatesOn: Events.SetupIntentResolver,
+      activate: (context) => IntentResolver({ createInvitationUrl, context, observability }),
     }),
     defineModule({
       id: `${meta.id}/module/app-graph-builder`,
@@ -127,7 +156,7 @@ export const SpacePlugin = ({
     // TODO(wittjosiah): This could probably be deferred.
     defineModule({
       id: `${meta.id}/module/app-graph-serializer`,
-      activatesOn: Events.Startup,
+      activatesOn: Events.AppGraphReady,
       activate: AppGraphSerializer,
     }),
     defineModule({

package/src/capabilities/app-graph-builder.ts

@@ -11,17 +11,16 @@ import {
   isEchoObject,
   isSpace,
   OBJECT_ID_LENGTH,
-  parseId,
   type ReactiveEchoObject,
   SPACE_ID_LENGTH,
   SpaceState,
   type Space,
 } from '@dxos/client/echo';
-import { getTypename, isDeleted } from '@dxos/live-object';
+import { isDeleted } from '@dxos/live-object';
 import { log } from '@dxos/log';
 import { ClientCapabilities } from '@dxos/plugin-client';
-import { createExtension, toSignal, type Node } from '@dxos/plugin-graph';
-import { nonNullable } from '@dxos/util';
+import { createExtension, toSignal, type Node, type InvokeParams } from '@dxos/plugin-graph';
+import { isNonNullable } from '@dxos/util';
 
 import { SpaceCapabilities } from './capabilities';
 import { SPACE_PLUGIN } from '../meta';
@@ -85,29 +84,17 @@ export default (context: PluginsContext) => {
       filter: (node): node is Node<null> => node.id === 'root',
       actions: () => [
         {
-          id: SpaceAction.OpenCreateSpace._tag,
-          data: async () => {
+          id: SpaceAction.AddSpace._tag,
+          data: async (params: InvokeParams) => {
             const { dispatchPromise: dispatch } = context.requestCapability(Capabilities.IntentDispatcher);
-            await dispatch(createIntent(SpaceAction.OpenCreateSpace));
+            await dispatch(createIntent(SpaceAction.AddSpace));
           },
           properties: {
-            label: ['create space label', { ns: SPACE_PLUGIN }],
+            label: ['add space label', { ns: SPACE_PLUGIN }],
             icon: 'ph--plus--regular',
-            testId: 'spacePlugin.createSpace',
-            disposition: 'item',
-          },
-        },
-        {
-          id: SpaceAction.Join._tag,
-          data: async () => {
-            const { dispatchPromise: dispatch } = context.requestCapability(Capabilities.IntentDispatcher);
-            await dispatch(createIntent(SpaceAction.Join));
-          },
-          properties: {
-            label: ['join space label', { ns: SPACE_PLUGIN }],
-            icon: 'ph--sign-in--regular',
-            testId: 'spacePlugin.joinSpace',
+            testId: 'spacePlugin.addSpace',
             disposition: 'item',
+            position: 'fallback',
           },
         },
       ],
@@ -197,6 +184,8 @@ export default (context: PluginsContext) => {
 
         if (space.state.get() === SpaceState.SPACE_INACTIVE) {
           return false;
+        } else if (space.state.get() !== SpaceState.SPACE_READY) {
+          return undefined;
         } else {
           const state = context.requestCapability(SpaceCapabilities.State);
           return constructSpaceNode({
@@ -252,9 +241,9 @@ export default (context: PluginsContext) => {
 
         return collection.objects
           .map((object) => object.target)
-          .filter(nonNullable)
+          .filter(isNonNullable)
           .map((object) => createObjectNode({ object, space, resolve, navigable: state.navigableCollections }))
-          .filter(nonNullable);
+          .filter(isNonNullable);
       },
     }),
 
@@ -273,9 +262,9 @@ export default (context: PluginsContext) => {
 
         return collection.objects
           .map((object) => object.target)
-          .filter(nonNullable)
+          .filter(isNonNullable)
           .map((object) => createObjectNode({ object, space, resolve, navigable: state.navigableCollections }))
-          .filter(nonNullable);
+          .filter(isNonNullable);
       },
       resolver: ({ id }) => {
         if (id.length !== FQ_ID_LENGTH) {
@@ -326,69 +315,5 @@ export default (context: PluginsContext) => {
         return constructObjectActions({ node, dispatch, navigable: state.navigableCollections });
       },
     }),
-
-    // Create nodes for object settings.
-    createExtension({
-      id: `${SPACE_PLUGIN}/settings-for-subject`,
-      resolver: ({ id }) => {
-        // TODO(Zan): Find util (or make one).
-        if (!id.endsWith('~settings')) {
-          return;
-        }
-
-        const type = 'orphan-settings-for-subject';
-        const icon = 'ph--gear--regular';
-
-        const [subjectId] = id.split('~');
-        const { spaceId, objectId } = parseId(subjectId);
-        const client = context.requestCapability(ClientCapabilities.Client);
-        const spaces = toSignal(
-          (onChange) => client.spaces.subscribe(() => onChange()).unsubscribe,
-          () => client.spaces.get(),
-        );
-        const space = spaces?.find((space) => space.id === spaceId && space.state.get() === SpaceState.SPACE_READY);
-        if (!objectId) {
-          const label = space
-            ? space.properties.name || ['unnamed space label', { ns: SPACE_PLUGIN }]
-            : ['unnamed object settings label', { ns: SPACE_PLUGIN }];
-
-          // TODO(wittjosiah): Support comments for arbitrary subjects.
-          //   This is to ensure that the comments panel is not stuck on an old object.
-          return {
-            id,
-            type,
-            data: null,
-            properties: {
-              icon,
-              label,
-              showResolvedThreads: false,
-              object: null,
-              space,
-            },
-          };
-        }
-
-        const [object] = memoizeQuery(space, { id: objectId });
-        if (!object || !subjectId) {
-          return;
-        }
-
-        const meta = resolve(getTypename(object) ?? '');
-        const label = meta.label?.(object) ||
-          object.name ||
-          meta.placeholder || ['unnamed object settings label', { ns: SPACE_PLUGIN }];
-
-        return {
-          id,
-          type,
-          data: null,
-          properties: {
-            icon,
-            label,
-            object,
-          },
-        };
-      },
-    }),
   ]);
 };

package/src/capabilities/capabilities.ts

@@ -6,7 +6,7 @@ import { defineCapability, type Label } from '@dxos/app-framework';
 import { type DeepReadonly } from '@dxos/util';
 
 import { SPACE_PLUGIN } from '../meta';
-import { type PluginState } from '../types';
+import { type ObjectForm, type PluginState } from '../types';
 
 export namespace SpaceCapabilities {
   export const State = defineCapability<DeepReadonly<PluginState>>(`${SPACE_PLUGIN}/capability/state`);
@@ -14,6 +14,8 @@ export namespace SpaceCapabilities {
 
   export type SpaceSettingsPanel = { id: string; label: Label };
   export const SettingsPanel = defineCapability<SpaceSettingsPanel>(`${SPACE_PLUGIN}/capability/settings-panel`);
+
+  export const ObjectForm = defineCapability<ObjectForm<any>>(`${SPACE_PLUGIN}/capability/object-form`);
 }
 
 // TODO(wittjosiah): Factor out.

package/src/capabilities/index.ts

@@ -10,6 +10,7 @@ export const IdentityCreated = lazy(() => import('./identity-created'));
 export const IntentResolver = lazy(() => import('./intent-resolver'));
 export const ReactRoot = lazy(() => import('./react-root'));
 export const ReactSurface = lazy(() => import('./react-surface'));
+export const Schema = lazy(() => import('./schema'));
 export const SpaceSettings = lazy(() => import('./settings'));
 export const SpaceState = lazy(() => import('./state'));
 export const SpacesReady = lazy(() => import('./spaces-ready'));

package/src/capabilities/intent-resolver.ts

@@ -31,6 +31,7 @@ import {
   CREATE_OBJECT_DIALOG,
   type CreateObjectDialogProps,
   POPOVER_RENAME_OBJECT,
+  POPOVER_ADD_SPACE,
 } from '../components';
 import { SPACE_PLUGIN } from '../meta';
 import { CollectionAction, CollectionType, SpaceAction } from '../types';
@@ -42,9 +43,10 @@ const SPACE_MAX_OBJECTS = 500;
 type IntentResolverOptions = {
   createInvitationUrl: (invitationCode: string) => string;
   context: PluginsContext;
+  observability?: boolean;
 };
 
-export default ({ createInvitationUrl, context }: IntentResolverOptions) => {
+export default ({ createInvitationUrl, context, observability }: IntentResolverOptions) => {
   const resolve = (typename: string) =>
     context.requestCapabilities(Capabilities.Metadata).find(({ id }) => id === typename)?.metadata ?? {};
 
@@ -86,12 +88,16 @@ export default ({ createInvitationUrl, context }: IntentResolverOptions) => {
             space,
           },
           intents: [
-            createIntent(ObservabilityAction.SendEvent, {
-              name: 'space.create',
-              properties: {
-                spaceId: space.id,
-              },
-            }),
+            ...(observability
+              ? [
+                  createIntent(ObservabilityAction.SendEvent, {
+                    name: 'space.create',
+                    properties: {
+                      spaceId: space.id,
+                    },
+                  }),
+                ]
+              : []),
           ],
         };
       },
@@ -136,12 +142,16 @@ export default ({ createInvitationUrl, context }: IntentResolverOptions) => {
                 } satisfies Partial<SpaceSettingsDialogProps>,
               },
             }),
-            createIntent(ObservabilityAction.SendEvent, {
-              name: 'space.share',
-              properties: {
-                space: space.id,
-              },
-            }),
+            ...(observability
+              ? [
+                  createIntent(ObservabilityAction.SendEvent, {
+                    name: 'space.share',
+                    properties: {
+                      space: space.id,
+                    },
+                  }),
+                ]
+              : []),
           ],
         };
       },
@@ -152,12 +162,16 @@ export default ({ createInvitationUrl, context }: IntentResolverOptions) => {
         space.properties[COMPOSER_SPACE_LOCK] = true;
         return {
           intents: [
-            createIntent(ObservabilityAction.SendEvent, {
-              name: 'space.lock',
-              properties: {
-                spaceId: space.id,
-              },
-            }),
+            ...(observability
+              ? [
+                  createIntent(ObservabilityAction.SendEvent, {
+                    name: 'space.lock',
+                    properties: {
+                      spaceId: space.id,
+                    },
+                  }),
+                ]
+              : []),
           ],
         };
       },
@@ -168,12 +182,16 @@ export default ({ createInvitationUrl, context }: IntentResolverOptions) => {
         space.properties[COMPOSER_SPACE_LOCK] = false;
         return {
           intents: [
-            createIntent(ObservabilityAction.SendEvent, {
-              name: 'space.unlock',
-              properties: {
-                spaceId: space.id,
-              },
-            }),
+            ...(observability
+              ? [
+                  createIntent(ObservabilityAction.SendEvent, {
+                    name: 'space.unlock',
+                    properties: {
+                      spaceId: space.id,
+                    },
+                  }),
+                ]
+              : []),
           ],
         };
       },
@@ -195,6 +213,23 @@ export default ({ createInvitationUrl, context }: IntentResolverOptions) => {
         };
       },
     }),
+    createResolver({
+      intent: SpaceAction.AddSpace,
+      resolve: () => {
+        return {
+          intents: [
+            createIntent(LayoutAction.UpdatePopover, {
+              part: 'popover',
+              subject: POPOVER_ADD_SPACE,
+              options: {
+                anchorId: SpaceAction.AddSpace._tag,
+                side: 'right',
+              },
+            }),
+          ],
+        };
+      },
+    }),
     createResolver({
       intent: SpaceAction.OpenSettings,
       resolve: ({ space }) => {
@@ -242,13 +277,17 @@ export default ({ createInvitationUrl, context }: IntentResolverOptions) => {
         return {
           data: result,
           intents: [
-            createIntent(ObservabilityAction.SendEvent, {
-              name: 'space.migrate',
-              properties: {
-                spaceId: space.id,
-                version,
-              },
-            }),
+            ...(observability
+              ? [
+                  createIntent(ObservabilityAction.SendEvent, {
+                    name: 'space.migrate',
+                    properties: {
+                      spaceId: space.id,
+                      version,
+                    },
+                  }),
+                ]
+              : []),
           ],
         };
       },
@@ -301,12 +340,16 @@ export default ({ createInvitationUrl, context }: IntentResolverOptions) => {
                   onAction: () => space.db.coreDatabase.unlinkDeletedObjects(),
                 },
               }),
-              createIntent(ObservabilityAction.SendEvent, {
-                name: 'space.limit',
-                properties: {
-                  spaceId: space.id,
-                },
-              }),
+              ...(observability
+                ? [
+                    createIntent(ObservabilityAction.SendEvent, {
+                      name: 'space.limit',
+                      properties: {
+                        spaceId: space.id,
+                      },
+                    }),
+                  ]
+                : []),
             ],
           };
         }
@@ -331,14 +374,18 @@ export default ({ createInvitationUrl, context }: IntentResolverOptions) => {
             object: object as HasId,
           },
           intents: [
-            createIntent(ObservabilityAction.SendEvent, {
-              name: 'space.object.add',
-              properties: {
-                spaceId: space.id,
-                objectId: object.id,
-                typename: getTypename(object),
-              },
-            }),
+            ...(observability
+              ? [
+                  createIntent(ObservabilityAction.SendEvent, {
+                    name: 'space.object.add',
+                    properties: {
+                      spaceId: space.id,
+                      objectId: object.id,
+                      typename: getTypename(object),
+                    },
+                  }),
+                ]
+              : []),
           ],
         };
       },

package/src/capabilities/react-surface.tsx

@@ -2,19 +2,10 @@
 // Copyright 2025 DXOS.org
 //
 
-import React, { useCallback } from 'react';
+import React from 'react';
 
-import {
-  Capabilities,
-  contributes,
-  createSurface,
-  Surface,
-  useCapability,
-  useCapabilities,
-  usePluginManager,
-} from '@dxos/app-framework';
+import { Capabilities, contributes, createSurface, Surface, useCapability } from '@dxos/app-framework';
 import { SettingsStore } from '@dxos/local-storage';
-import { ClientCapabilities } from '@dxos/plugin-client';
 import {
   getSpace,
   isEchoObject,
@@ -53,6 +44,8 @@ import {
   SyncStatus,
   type CreateObjectDialogProps,
   type SpaceSettingsDialogProps,
+  POPOVER_ADD_SPACE,
+  PopoverAddSpace,
 } from '../components';
 import { SPACE_PLUGIN } from '../meta';
 import { CollectionType, type SpaceSettingsProps } from '../types';
@@ -126,23 +119,8 @@ export default ({ createInvitationUrl }: ReactSurfaceOptions) =>
     createSurface({
       id: CREATE_OBJECT_DIALOG,
       role: 'dialog',
-      filter: (data): data is { props: Partial<CreateObjectDialogProps> } => data.component === CREATE_OBJECT_DIALOG,
-      component: ({ data }) => {
-        const schemas = useCapabilities(ClientCapabilities.Schema).flat();
-        const manager = usePluginManager();
-
-        const resolve = useCallback(
-          (typename: string) => {
-            return (
-              manager.context.requestCapabilities(Capabilities.Metadata).find(({ id }) => id === typename)?.metadata ??
-              {}
-            );
-          },
-          [manager],
-        );
-
-        return <CreateObjectDialog schemas={schemas} resolve={resolve} {...data.props} />;
-      },
+      filter: (data): data is { props: CreateObjectDialogProps } => data.component === CREATE_OBJECT_DIALOG,
+      component: ({ data }) => <CreateObjectDialog {...data.props} />,
     }),
     createSurface({
       id: POPOVER_RENAME_SPACE,
@@ -157,6 +135,12 @@ export default ({ createInvitationUrl }: ReactSurfaceOptions) =>
         data.component === POPOVER_RENAME_OBJECT && isReactiveObject(data.props),
       component: ({ data }) => <PopoverRenameObject object={data.props} />,
     }),
+    createSurface({
+      id: POPOVER_ADD_SPACE,
+      role: 'popover',
+      filter: (data): data is any => data.component === POPOVER_ADD_SPACE,
+      component: () => <PopoverAddSpace />,
+    }),
     createSurface({
       id: `${SPACE_PLUGIN}/navtree-presence`,
       role: 'navtree-item-end',

package/src/capabilities/schema.ts

@@ -0,0 +1,27 @@
+//
+// Copyright 2025 DXOS.org
+//
+
+import { effect } from '@preact/signals-core';
+
+import { Capabilities, contributes, type PluginsContext } from '@dxos/app-framework';
+import { ClientCapabilities } from '@dxos/plugin-client';
+
+import { SpaceCapabilities } from './capabilities';
+import { type ObjectForm } from '../types';
+
+export default (context: PluginsContext) => {
+  const client = context.requestCapability(ClientCapabilities.Client);
+
+  // TODO(wittjosiah): Unregister schemas when they are disabled.
+  let previous: ObjectForm[] = [];
+  const unsubscribe = effect(() => {
+    const forms = Array.from(new Set(context.requestCapabilities(SpaceCapabilities.ObjectForm)));
+    // TODO(wittjosiah): Filter out schemas which the client has already registered.
+    const newSchemas = forms.filter((form) => !previous.includes(form)).map((form) => form.objectSchema);
+    previous = forms;
+    client.addTypes(newSchemas);
+  });
+
+  return contributes(Capabilities.Null, null, () => unsubscribe());
+};

package/src/components/BaseObjectSettings.tsx

@@ -15,17 +15,24 @@ export type BaseObjectSettingsProps = {
 
 export const BaseObjectSettings = ({ object }: BaseObjectSettingsProps) => {
   const { t } = useTranslation(SPACE_PLUGIN);
+  const inputRef = React.useRef<HTMLInputElement>(null);
   // TODO(burdon): Standardize forms.
   return (
     <div role='form' className='flex flex-col w-full p-2 gap-1'>
       <Input.Root>
         <Input.Label>{t('name label')}</Input.Label>
         <Input.TextInput
+          ref={inputRef}
           placeholder={t('name placeholder')}
           value={object.name ?? ''}
           onChange={(event) => {
             object.name = event.target.value;
           }}
+          onKeyDown={(event) => {
+            if (event.key === 'Enter') {
+              inputRef.current?.blur();
+            }
+          }}
         />
       </Input.Root>
     </div>

package/src/components/CreateDialog/CreateObjectDialog.stories.tsx

@@ -40,9 +40,7 @@ const meta: Meta<typeof CreateObjectDialog> = {
   parameters: {
     translations: [...translations, osTranslations],
   },
-  args: {
-    schemas: [CollectionType],
-  },
+  args: {},
 };
 
 export default meta;