@jskit-ai/workspaces-core 0.1.113 → 0.1.114

package/package.descriptor.mjs

@@ -1,7 +1,7 @@
 export default Object.freeze({
   packageVersion: 1,
   packageId: "@jskit-ai/workspaces-core",
-  version: "0.1.113",
+  version: "0.1.114",
   kind: "runtime",
   description: "Workspace tenancy runtime plus HTTP routes, role catalog, and workspace config scaffolding.",
   dependsOn: [
@@ -150,7 +150,7 @@ export default Object.freeze({
         "@jskit-ai/json-rest-api-core": "0.1.79",
         "@jskit-ai/resource-core": "0.1.78",
         "@jskit-ai/resource-crud-core": "0.1.78",
-        "@jskit-ai/users-core": "0.1.146"
+        "@jskit-ai/users-core": "0.1.147"
       },
       dev: {}
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/workspaces-core",
-  "version": "0.1.113",
+  "version": "0.1.114",
   "type": "module",
   "scripts": {
     "test": "node --test"
@@ -24,7 +24,7 @@
     "@jskit-ai/kernel": "0.1.135",
     "@jskit-ai/resource-crud-core": "0.1.78",
     "@jskit-ai/resource-core": "0.1.78",
-    "@jskit-ai/users-core": "0.1.146",
+    "@jskit-ai/users-core": "0.1.147",
     "json-rest-schema": "1.x.x"
   }
 }

package/src/server/workspaceDirectory/bootWorkspaceDirectoryRoutes.js

@@ -61,7 +61,7 @@ function bootWorkspaceDirectoryRoutes(app) {
           actionId: "workspace.workspaces.create",
           input: {
             name: body.name,
-            slug: body.slug
+            ...(Object.hasOwn(body, "slug") ? { slug: body.slug } : {})
           }
         });
         reply.code(200).send(response);

package/test/workspaceService.test.js

@@ -338,6 +338,37 @@ test("workspaceService.createWorkspaceForAuthenticatedUser creates non-personal
   assert.equal(workspace.slug, "ops-team");
   assert.equal(calls.insert, 1);
   assert.equal(calls.ensureOwnerMembership, 1);
+  assert.equal(calls.ensureWorkspaceSettings, 1);
+  assert.equal(insertedPayloads[0].isPersonal, false);
+  assert.equal(insertedPayloads[0].ownerUserId, "7");
+});
+
+test("workspaceService.createWorkspaceForAuthenticatedUser generates an omitted slug and provisions ownership", async () => {
+  const { service, calls, insertedPayloads } = createWorkspaceServiceFixture({
+    tenancyMode: "workspaces",
+    tenancyPolicy: {
+      workspace: {
+        allowSelfCreate: true
+      }
+    }
+  });
+
+  const workspace = await service.createWorkspaceForAuthenticatedUser(
+    {
+      id: "7",
+      email: "chiaramobily@gmail.com",
+      displayName: "Chiara"
+    },
+    {
+      name: "Operations Team"
+    }
+  );
+
+  assert.equal(workspace.slug, "operations-team");
+  assert.equal(calls.insert, 1);
+  assert.equal(calls.ensureOwnerMembership, 1);
+  assert.equal(calls.ensureWorkspaceSettings, 1);
+  assert.equal(insertedPayloads[0].slug, "operations-team");
   assert.equal(insertedPayloads[0].isPersonal, false);
   assert.equal(insertedPayloads[0].ownerUserId, "7");
 });

package/test/workspacesRouteRequestInputValidator.test.js

@@ -340,6 +340,34 @@ test("workspaces-core boot skips invitation redeem/list routes when workspace in
   assert.equal(findRoute(routes, { method: "DELETE", path: "/api/w/:workspaceSlug/invites/:inviteId" }), null);
 });
 
+test("workspace create handler preserves an omitted optional slug", async () => {
+  const routes = await registerRoutes();
+  const workspaceCreate = findRoute(routes, {
+    method: "POST",
+    path: "/api/workspaces"
+  });
+  const calls = [];
+
+  await workspaceCreate.handler(
+    createActionRequest({
+      input: {
+        body: { name: "Operations Team" }
+      },
+      executeAction: async (payload) => {
+        calls.push(payload);
+        return {};
+      }
+    }),
+    createReplyDouble()
+  );
+
+  assert.deepEqual(calls, [{
+    actionId: "workspace.workspaces.create",
+    input: { name: "Operations Team" }
+  }]);
+  assert.equal(Object.hasOwn(calls[0].input, "slug"), false);
+});
+
 test("workspace invite and member handlers build action input from request.input", async () => {
   const routes = await registerRoutes();
   const workspaceCreate = findRoute(routes, {