Make owner optional in experiment POST endpoint.

packages/back-end/src/api/experiments/postExperiment.ts

@@ -17,7 +17,7 @@ export const postExperiment = createApiRequestHandler(postExperimentValidator)(
   async (req): Promise<PostExperimentResponse> => {
     req.checkPermissions("createAnalyses", req.body.project);
 
-    const { datasourceId, owner } = req.body;
+    const { datasourceId, owner: ownerEmail } = req.body;
 
     const datasource = await getDataSourceById(req.context, datasourceId);
 
@@ -47,20 +47,26 @@ export const postExperiment = createApiRequestHandler(postExperimentValidator)(
       );
     }
 
-    const user = await getUserByEmail(owner);
+    const ownerId = await (async () => {
+      if (!ownerEmail) return req.context.userId;
 
-    // check if the user is a member of the organization
-    const isMember = req.organization.members.some(
-      (member) => member.id === user?.id
-    );
+      const user = await getUserByEmail(ownerEmail);
 
-    if (!isMember || !user) {
-      throw new Error(`Unable to find user: ${owner}.`);
-    }
+      // check if the user is a member of the organization
+      const isMember = req.organization.members.some(
+        (member) => member.id === user?.id
+      );
+
+      if (!isMember || !user) {
+        throw new Error(`Unable to find user: ${ownerEmail}.`);
+      }
+
+      return user.id;
+    })();
 
     // transform into exp interface; set sane defaults
     const newExperiment = postExperimentApiPayloadToInterface(
-      { ...req.body, owner: user.id },
+      { ...req.body, ...(ownerId ? { owner: ownerId } : {}) },
       req.organization,
       datasource
     );
@@ -70,13 +76,15 @@ export const postExperiment = createApiRequestHandler(postExperimentValidator)(
       context: req.context,
     });
 
-    // add owner as watcher
-    await upsertWatch({
-      userId: user.id,
-      organization: req.organization.id,
-      item: experiment.id,
-      type: "experiments",
-    });
+    if (ownerId) {
+      // add owner as watcher
+      await upsertWatch({
+        userId: ownerId,
+        organization: req.organization.id,
+        item: experiment.id,
+        type: "experiments",
+      });
+    }
 
     const apiExperiment = await toExperimentApiInterface(
       req.context,

packages/back-end/src/api/openapi/payload-schemas/PostExperimentPayload.yaml

@@ -2,7 +2,6 @@ type: object
 required:
   - name
   - variations
-  - owner
   - datasourceId
   - trackingKey
   - assignmentQueryId