chore: Make workspace slug not null after migrate

apps/api/src/routes/v1_keys_getVerifications.security.test.ts

@@ -75,10 +75,12 @@ test("cannot read keys from a different workspace", async (t) => {
   const h = await IntegrationHarness.init(t);
 
   const workspaceId = newId("workspace");
+  const name = randomUUID();
   await h.db.primary.insert(schema.workspaces).values({
     id: workspaceId,
     orgId: randomUUID(),
-    name: randomUUID(),
+    name: name,
+    slug: name,
     features: {},
     betaFeatures: {},
   });

apps/dashboard/app/(app)/gateway-new/page.tsx

@@ -8,6 +8,7 @@
  * 4. The user is redirected to create their API
  */
 
+import { randomInt } from "node:crypto";
 import { getAuth } from "@/lib/auth";
 import { db, schema } from "@/lib/db";
 import { freeTierQuotas } from "@/lib/quotas";
@@ -28,6 +29,7 @@ export default async function Page() {
     await db.insert(schema.workspaces).values({
       id,
       name: "Personal Workspace",
+      slug: `personal-workspace-${randomInt(100000)}`,
       orgId,
       betaFeatures: {},
       features: {},

deployment/04-seed-workspace.sql

@@ -8,13 +8,15 @@ INSERT INTO workspaces (
   id,
   org_id,
   name,
+  slug,
   created_at_m,
   beta_features,
   features
 ) VALUES (
   'ws_local_root',
   'user_REPLACE_ME',
   'Unkey',
+  'unkey',
   UNIX_TIMESTAMP() * 1000,
   '{}',
   '{}'

go/apps/api/routes/v2_identities_list_identities/200_test.go

@@ -292,6 +292,7 @@ func TestSuccess(t *testing.T) {
 		err = db.Query.InsertWorkspace(ctx, tx, db.InsertWorkspaceParams{
 			ID:        singleWorkspaceID,
 			Name:      "Single Identity Workspace",
+			Slug:      uid.New("slug"),
 			OrgID:     uid.New(uid.OrgPrefix),
 			CreatedAt: time.Now().UnixMilli(),
 		})

go/apps/api/routes/v2_identities_list_identities/400_test.go

@@ -2,6 +2,7 @@ package handler_test
 
 import (
 	"fmt"
+	"maps"
 	"net/http"
 	"strings"
 	"testing"
@@ -80,9 +81,7 @@ func TestBadRequests(t *testing.T) {
 
 	t.Run("malformed JSON body", func(t *testing.T) {
 		customHeaders := make(http.Header)
-		for k, v := range headers {
-			customHeaders[k] = v
-		}
+		maps.Copy(customHeaders, headers)
 		customHeaders.Set("Content-Type", "application/json")
 
 		// Create a malformed JSON string

go/apps/api/routes/v2_identities_list_identities/cross_workspace_test.go

@@ -46,6 +46,7 @@ func TestCrossWorkspaceForbidden(t *testing.T) {
 	err = db.Query.InsertWorkspace(ctx, tx, db.InsertWorkspaceParams{
 		ID:        workspaceB,
 		Name:      "Test Workspace B",
+		Slug:      uid.New("slug"),
 		CreatedAt: time.Now().UnixMilli(),
 		OrgID:     uid.New("org"),
 	})

go/apps/api/routes/v2_keys_add_permissions/404_test.go

@@ -81,6 +81,7 @@ func TestNotFoundErrors(t *testing.T) {
 			ID:        otherWorkspaceID,
 			OrgID:     uid.New("test_org"),
 			Name:      "Other Workspace",
+			Slug:      uid.New("slug"),
 			CreatedAt: time.Now().UnixMilli(),
 		})
 		require.NoError(t, err)

go/apps/api/routes/v2_keys_remove_permissions/403_test.go

@@ -114,6 +114,7 @@ func TestAuthorizationErrors(t *testing.T) {
 			ID:        otherWorkspaceID,
 			OrgID:     uid.New("test_org"),
 			Name:      "Other Workspace",
+			Slug:      uid.New("slug"),
 			CreatedAt: time.Now().UnixMilli(),
 		})
 		require.NoError(t, err)

go/apps/api/routes/v2_keys_remove_permissions/404_test.go

@@ -118,6 +118,7 @@ func TestNotFoundErrors(t *testing.T) {
 			ID:        otherWorkspaceID,
 			OrgID:     uid.New("test_org"),
 			Name:      "Other Workspace",
+			Slug:      uid.New("slug"),
 			CreatedAt: time.Now().UnixMilli(),
 		})
 		require.NoError(t, err)
@@ -189,6 +190,7 @@ func TestNotFoundErrors(t *testing.T) {
 			ID:        otherWorkspaceID,
 			OrgID:     uid.New("test_org"),
 			Name:      "Other Workspace",
+			Slug:      uid.New("slug"),
 			CreatedAt: time.Now().UnixMilli(),
 		})
 		require.NoError(t, err)

go/apps/api/routes/v2_keys_set_roles/404_test.go

@@ -152,6 +152,7 @@ func TestNotFoundErrors(t *testing.T) {
 			ID:        otherWorkspaceID,
 			OrgID:     uid.New("test_org"),
 			Name:      uid.New("test_name"),
+			Slug:      uid.New("slug"),
 			CreatedAt: time.Now().UnixMilli(),
 		})
 		require.NoError(t, err)
@@ -215,6 +216,7 @@ func TestNotFoundErrors(t *testing.T) {
 			ID:        otherWorkspaceID,
 			OrgID:     uid.New("test_org"),
 			Name:      uid.New("test_name"),
+			Slug:      uid.New("slug"),
 			CreatedAt: time.Now().UnixMilli(),
 		})
 		require.NoError(t, err)

go/pkg/db/queries/workspace_insert.sql

@@ -3,6 +3,7 @@ INSERT INTO `workspaces` (
     id,
     org_id,
     name,
+    slug,
     created_at_m,
     tier,
     beta_features,
@@ -14,7 +15,8 @@ VALUES (
     sqlc.arg(id),
     sqlc.arg(org_id),
     sqlc.arg(name),
-     sqlc.arg(created_at),
+    sqlc.arg(slug),
+    sqlc.arg(created_at),
     'Free',
     '{}',
     '{}',

go/pkg/db/retry_test.go

@@ -144,6 +144,7 @@ func TestWithRetry_Integration(t *testing.T) {
 		ID:        workspaceID,
 		OrgID:     workspaceID,
 		Name:      "Test Workspace",
+		Slug:      uid.New("slug"),
 		CreatedAt: time.Now().UnixMilli(),
 	})
 	require.NoError(t, err)

go/pkg/db/schema.sql

@@ -182,7 +182,7 @@ CREATE TABLE `workspaces` (
 	`id` varchar(256) NOT NULL,
 	`org_id` varchar(256) NOT NULL,
 	`name` varchar(256) NOT NULL,
-	`slug` varchar(64),
+	`slug` varchar(64) NOT NULL,
 	`partition_id` varchar(256),
 	`plan` enum('free','pro','enterprise') DEFAULT 'free',
 	`tier` varchar(256) DEFAULT 'Free',

go/pkg/testutil/seed/seed.go

@@ -50,6 +50,7 @@ func (s *Seeder) CreateWorkspace(ctx context.Context) db.Workspace {
 		ID:        uid.New("test_ws"),
 		OrgID:     uid.New("test_org"),
 		Name:      uid.New("test_name"),
+		Slug:      uid.New("slug"),
 		CreatedAt: time.Now().UnixMilli(),
 	}
 

internal/db/src/schema/workspaces.ts

@@ -20,7 +20,7 @@ export const workspaces = mysqlTable("workspaces", {
   name: varchar("name", { length: 256 }).notNull(),
 
   // slug is used for the workspace URL
-  slug: varchar("slug", { length: 64 }).unique(),
+  slug: varchar("slug", { length: 64 }).notNull().unique(),
 
   // Deployment platform - which partition this workspace deploys to
   partitionId: varchar("partition_id", { length: 256 }),

internal/resend/emails/api_v1_migration.tsx

@@ -16,7 +16,11 @@ export type Props = {
   deprecatedEndpoints: string[];
 };
 
-export function ApiV1Migration({ username, workspaceName, deprecatedEndpoints }: Props) {
+export function ApiV1Migration({
+  username = "User",
+  workspaceName = "Your Workspace",
+  deprecatedEndpoints = [],
+}: Props) {
   return (
     <Layout>
       <Heading className="font-sans text-3xl text-semibold text-center">