Included the contributers changes

.cursor/rules/component-migration.mdc

@@ -1,216 +0,0 @@
----
-description: 
-globs: 
-alwaysApply: false
----
-# Component Migration Automation Rule
-
-## Overview
-This rule automates the migration of deprecated components to new component systems in React/TypeScript codebases.
-
-## Trigger
-When the user requests component migration (e.g., "migrate [DeprecatedComponent] to [NewComponent]" or "component migration").
-
-## Process
-
-### Step 1: Discovery and Planning
-1. **Identify migration parameters:**
-   - Ask user for deprecated component name (e.g., "Modal")
-   - Ask user for new component name(s) (e.g., "Dialog")
-   - Ask for any components to exclude (e.g., "ModalWithTabs")
-   - Ask for specific import paths if needed
-
-2. **Scan codebase** for deprecated components:
-   - Search for `import.*[DeprecatedComponent]` patterns
-   - Exclude specified components that should not be migrated
-   - List all found components with file paths
-   - Present numbered list to user for confirmation
-
-### Step 2: Component-by-Component Migration
-For each component, follow this exact sequence:
-
-#### 2.1 Component Migration
-- **Import changes:**
-  - Ask user to provide the new import structure
-  - Example transformation pattern:
-  ```typescript
-  // FROM:
-  import { [DeprecatedComponent] } from "@/components/ui/[DeprecatedComponent]"
-  
-  // TO:
-  import {
-    [NewComponent],
-    [NewComponentPart1],
-    [NewComponentPart2],
-    // ... other parts
-  } from "@/components/ui/[NewComponent]"
-  ```
-
-- **Props transformation:**
-  - Ask user for prop mapping rules (e.g., `open` → `open`, `setOpen` → `onOpenChange`)
-  - Ask for props to remove (e.g., `noPadding`, `closeOnOutsideClick`, `size`)
-  - Apply transformations based on user specifications
-
-- **Structure transformation:**
-  - Ask user for the new component structure pattern
-  - Apply the transformation maintaining all functionality
-  - Preserve all existing logic, state management, and event handlers
-
-#### 2.2 Wait for User Approval
-- Present the migration changes
-- Wait for explicit user approval before proceeding
-- If rejected, ask for specific feedback and iterate
-#### 2.3 Re-read and Apply Additional Changes
-- Re-read the component file to capture any user modifications
-- Apply any additional improvements the user made
-- Ensure all changes are incorporated
-
-#### 2.4 Test File Updates
-- **Find corresponding test file** (same name with `.test.tsx` or `.test.ts`)
-- **Update test mocks:**
-  - Ask user for new component mock structure
-  - Replace old component mocks with new ones
-  - Example pattern:
-  ```typescript
-  // Add to test setup:
-  jest.mock("@/components/ui/[NewComponent]", () => ({
-    [NewComponent]: ({ children, [props] }: any) => ([mock implementation]),
-    [NewComponentPart1]: ({ children }: any) => <div data-testid="[new-component-part1]">{children}</div>,
-    [NewComponentPart2]: ({ children }: any) => <div data-testid="[new-component-part2]">{children}</div>,
-    // ... other parts
-  }));
-  ```
-- **Update test expectations:**
-  - Change test IDs from old component to new component
-  - Update any component-specific assertions
-  - Ensure all new component parts used in the component are mocked
-
-#### 2.5 Run Tests and Optimize
-- Execute `Node package manager test -- ComponentName.test.tsx`
-- Fix any failing tests
-- Optimize code quality (imports, formatting, etc.)
-- Re-run tests until all pass
-- **Maximum 3 iterations** - if still failing, ask user for guidance
-
-#### 2.6 Wait for Final Approval
-- Present test results and any optimizations made
-- Wait for user approval of the complete migration
-- If rejected, iterate based on feedback
-
-#### 2.7 Git Commit
-- Run: `git add .`
-- Run: `git commit -m "migrate [ComponentName] from [DeprecatedComponent] to [NewComponent]"`
-- Confirm commit was successful
-
-### Step 3: Final Report Generation
-After all components are migrated, generate a comprehensive GitHub PR report:
-
-#### PR Title
-```
-feat: migrate [DeprecatedComponent] components to [NewComponent] system
-```
-
-#### PR Description Template
-```markdown
-## 🔄 [DeprecatedComponent] to [NewComponent] Migration
-
-### Overview
-Migrated [X] [DeprecatedComponent] components to the new [NewComponent] component system to modernize the UI architecture and improve consistency.
-
-### Components Migrated
-[List each component with file path]
-
-### Technical Changes
-- **Imports:** Replaced `[DeprecatedComponent]` with `[NewComponent], [NewComponentParts...]`
-- **Props:** [List prop transformations]
-- **Structure:** Implemented proper [NewComponent] component hierarchy
-- **Styling:** [Describe styling changes]
-- **Tests:** Updated all test mocks and expectations
-
-### Migration Pattern
-```typescript
-// Before
-<[DeprecatedComponent] [oldProps]>
-  [oldStructure]
-</[DeprecatedComponent]>
-
-// After
-<[NewComponent] [newProps]>
-  [newStructure]
-</[NewComponent]>
-```
-
-### Testing
-- ✅ All existing tests updated and passing
-- ✅ Component functionality preserved
-- ✅ UI/UX behavior maintained
-
-### How to Test This PR
-1. **Functional Testing:**
-   - Navigate to each migrated component's usage
-   - Verify [component] opens and closes correctly
-   - Test all interactive elements within [components]
-   - Confirm styling and layout are preserved
-
-2. **Automated Testing:**
-   ```bash
-   Node package manager test
-   ```
-
-3. **Visual Testing:**
-   - Check that all [components] maintain proper styling
-   - Verify responsive behavior
-   - Test keyboard navigation and accessibility
-
-### Breaking Changes
-[List any breaking changes or state "None - this is a drop-in replacement maintaining all existing functionality."]
-
-### Notes
-- [Any excluded components] were preserved as they already use [NewComponent] internally
-- All form validation and complex state management preserved
-- Enhanced code quality with better imports and formatting
-```
-
-## Special Considerations
-
-### Excluded Components
-- **DO NOT MIGRATE** components specified by user as exclusions
-- They may already use the new component internally or have other reasons
-- Inform user these are skipped and why
-
-### Complex Components
-- Preserve all existing functionality (forms, validation, state management)
-- Maintain prop interfaces
-- Keep all event handlers and callbacks
-- Preserve accessibility features
-
-### Test Coverage
-- Ensure all new component parts are mocked when used
-- Mock all new component parts that appear in the component
-- Update test IDs from old component to new component
-- Maintain all existing test scenarios
-
-### Error Handling
-- If tests fail after 3 iterations, stop and ask user for guidance
-- If component is too complex, ask user for specific guidance
-- If unsure about functionality preservation, ask for clarification
-
-### Migration Patterns
-- Always ask user for specific migration patterns before starting
-- Confirm import structures, prop mappings, and component hierarchies
-- Adapt to different component architectures (simple replacements, complex restructuring, etc.)
-
-## Success Criteria
-- All deprecated components successfully migrated to new components
-- All tests passing
-- No functionality lost
-- Code quality maintained or improved
-- User approval on each component
-- Successful git commits for each migration
-- Comprehensive PR report generated
-
-## Usage Examples
-- "migrate Modal to Dialog"
-- "migrate Button to NewButton" 
-- "migrate Card to ModernCard"
-- "component migration" (will prompt for details) 

.cursor/rules/documentations.mdc

@@ -1,23 +0,0 @@
----
-description: Guideline for writing end-user facing documentation in the apps/docs folder
-globs: 
-alwaysApply: false
----
-Follow these instructions and guidelines when asked to write documentation in the apps/docs folder
-
-Follow this structure to write the title, describtion and pick a matching icon and insert it at the top of the MDX file:
-
----
-title: "FEATURE NAME"
-description: "1 concise sentence to describe WHEN the feature is being used and FOR WHAT BENEFIT."
-icon: "link"
----
-
-- Description: 1 concise sentence to describe WHEN the feature is being used and FOR WHAT BENEFIT.
-- Make ample use of the Mintlify components you can find here https://mintlify.com/docs/llms.txt
-- In all Headlines, only capitalize the current feature and nothing else, to Camel Case
-- If a feature is part of the Enterprise Edition, use this note:
-
-<Note>
-  FEATURE NAME is part of the @Enterprise Edition. 
-</Note>

.github/ISSUE_TEMPLATE/feature_request.yml

@@ -1,7 +1,6 @@
 name: Feature request
 description: "Suggest an idea for this project \U0001F680"
 type: feature
-projects: "formbricks/21"
 body:
   - type: textarea
     id: problem-description

.github/ISSUE_TEMPLATE/task.yml

@@ -0,0 +1,11 @@
+name: Task (internal)
+description: "Template for creating a task. Used by the Formbricks Team only \U0001f4e5"
+type: task
+body:
+  - type: textarea
+    id: task-summary
+    attributes:
+      label: Task description
+      description: A clear detailed-rich description of the task.
+    validations:
+      required: true

apps/web/app/(app)/(onboarding)/organizations/[organizationId]/landing/components/landing-sidebar.test.tsx

@@ -94,7 +94,6 @@ describe("LandingSidebar component", () => {
       organizationId: "o1",
       redirect: true,
       callbackUrl: "/auth/login",
-      clearEnvironmentId: true,
     });
   });
 });

apps/web/app/(app)/(onboarding)/organizations/[organizationId]/landing/components/landing-sidebar.tsx

@@ -130,7 +130,6 @@ export const LandingSidebar = ({
                   organizationId: organization.id,
                   redirect: true,
                   callbackUrl: "/auth/login",
-                  clearEnvironmentId: true,
                 });
               }}
               icon={<LogOutIcon className="mr-2 h-4 w-4" strokeWidth={1.5} />}>

apps/web/app/(app)/environments/[environmentId]/actions/components/ActionRowData.tsx

@@ -11,21 +11,22 @@ export const ActionClassDataRow = ({
   locale: TUserLocale;
 }) => {
   return (
-    <div className="m-2 grid grid-cols-6 content-center rounded-lg transition-colors ease-in-out hover:bg-slate-100">
-      <div className="col-span-4 flex items-start py-3 pl-6 text-sm">
-        <div className="flex w-full items-center gap-4">
-          <div className="mt-1 h-5 w-5 flex-shrink-0 text-slate-500">
+    <div className="m-2 grid h-16 grid-cols-6 content-center rounded-lg transition-colors ease-in-out hover:bg-slate-100">
+      <div className="col-span-4 flex items-center pl-6 text-sm">
+        <div className="flex items-center">
+          <div className="h-5 w-5 flex-shrink-0 text-slate-500">
             {ACTION_TYPE_ICON_LOOKUP[actionClass.type]}
           </div>
-          <div className="text-left">
-            <div className="break-words font-medium text-slate-900">{actionClass.name}</div>
-            <div className="break-words text-xs text-slate-400">{actionClass.description}</div>
+          <div className="ml-4 text-left">
+            <div className="font-medium text-slate-900">{actionClass.name}</div>
+            <div className="text-xs text-slate-400">{actionClass.description}</div>
           </div>
         </div>
       </div>
       <div className="col-span-2 my-auto whitespace-nowrap text-center text-sm text-slate-500">
         {timeSince(actionClass.createdAt.toString(), locale)}
       </div>
+      <div className="text-center"></div>
     </div>
   );
 };

apps/web/app/(app)/environments/[environmentId]/components/MainNavigation.test.tsx

@@ -220,8 +220,6 @@ describe("MainNavigation", () => {
     const mockSignOut = vi.fn().mockResolvedValue({ url: "/auth/login" });
     vi.mocked(useSignOut).mockReturnValue({ signOut: mockSignOut });
 
-    // Set up localStorage spy on the mocked localStorage
-
     render(<MainNavigation {...defaultProps} />);
 
     // Find the avatar and get its parent div which acts as the trigger
@@ -248,9 +246,7 @@ describe("MainNavigation", () => {
       organizationId: "org1",
       redirect: false,
       callbackUrl: "/auth/login",
-      clearEnvironmentId: true,
     });
-
     await waitFor(() => {
       expect(mockRouterPush).toHaveBeenCalledWith("/auth/login");
     });

apps/web/app/(app)/environments/[environmentId]/components/MainNavigation.tsx

@@ -396,7 +396,6 @@ export const MainNavigation = ({
                         organizationId: organization.id,
                         redirect: false,
                         callbackUrl: "/auth/login",
-                        clearEnvironmentId: true,
                       });
                       router.push(route?.url || "/auth/login"); // NOSONAR // We want to check for empty strings
                     }}

apps/web/app/(app)/environments/[environmentId]/settings/(account)/notifications/components/NotificationSwitch.test.tsx

@@ -20,7 +20,7 @@ vi.mock("@/modules/ui/components/switch", () => ({
 }));
 
 vi.mock("../actions", () => ({
-  updateNotificationSettingsAction: vi.fn(() => Promise.resolve({ data: true })),
+  updateNotificationSettingsAction: vi.fn(() => Promise.resolve()),
 }));
 
 const surveyId = "survey1";
@@ -246,204 +246,4 @@ describe("NotificationSwitch", () => {
     });
     expect(updateNotificationSettingsAction).not.toHaveBeenCalled();
   });
-
-  test("shows error toast when updateNotificationSettingsAction fails for 'alert' type", async () => {
-    const mockErrorResponse = { serverError: "Failed to update notification settings" };
-    vi.mocked(updateNotificationSettingsAction).mockResolvedValueOnce(mockErrorResponse);
-
-    const initialSettings = { ...baseNotificationSettings, alert: { [surveyId]: false } };
-    renderSwitch({ notificationSettings: initialSettings, notificationType: "alert" });
-    const switchInput = screen.getByLabelText("toggle notification settings for alert");
-
-    await act(async () => {
-      await user.click(switchInput);
-    });
-
-    expect(updateNotificationSettingsAction).toHaveBeenCalledWith({
-      notificationSettings: { ...initialSettings, alert: { [surveyId]: true } },
-    });
-    expect(toast.error).toHaveBeenCalledWith("Failed to update notification settings", {
-      id: "notification-switch",
-    });
-    expect(toast.success).not.toHaveBeenCalled();
-  });
-
-  test("shows error toast when updateNotificationSettingsAction fails for 'weeklySummary' type", async () => {
-    const mockErrorResponse = { serverError: "Database connection failed" };
-    vi.mocked(updateNotificationSettingsAction).mockResolvedValueOnce(mockErrorResponse);
-
-    const initialSettings = { ...baseNotificationSettings, weeklySummary: { [projectId]: true } };
-    renderSwitch({
-      surveyOrProjectOrOrganizationId: projectId,
-      notificationSettings: initialSettings,
-      notificationType: "weeklySummary",
-    });
-    const switchInput = screen.getByLabelText("toggle notification settings for weeklySummary");
-
-    await act(async () => {
-      await user.click(switchInput);
-    });
-
-    expect(updateNotificationSettingsAction).toHaveBeenCalledWith({
-      notificationSettings: { ...initialSettings, weeklySummary: { [projectId]: false } },
-    });
-    expect(toast.error).toHaveBeenCalledWith("Database connection failed", {
-      id: "notification-switch",
-    });
-    expect(toast.success).not.toHaveBeenCalled();
-  });
-
-  test("shows error toast when updateNotificationSettingsAction fails for 'unsubscribedOrganizationIds' type", async () => {
-    const mockErrorResponse = { serverError: "Permission denied" };
-    vi.mocked(updateNotificationSettingsAction).mockResolvedValueOnce(mockErrorResponse);
-
-    const initialSettings = { ...baseNotificationSettings, unsubscribedOrganizationIds: [] };
-    renderSwitch({
-      surveyOrProjectOrOrganizationId: organizationId,
-      notificationSettings: initialSettings,
-      notificationType: "unsubscribedOrganizationIds",
-    });
-    const switchInput = screen.getByLabelText("toggle notification settings for unsubscribedOrganizationIds");
-
-    await act(async () => {
-      await user.click(switchInput);
-    });
-
-    expect(updateNotificationSettingsAction).toHaveBeenCalledWith({
-      notificationSettings: { ...initialSettings, unsubscribedOrganizationIds: [organizationId] },
-    });
-    expect(toast.error).toHaveBeenCalledWith("Permission denied", {
-      id: "notification-switch",
-    });
-    expect(toast.success).not.toHaveBeenCalled();
-  });
-
-  test("shows error toast when updateNotificationSettingsAction returns null", async () => {
-    const mockErrorResponse = { serverError: "An error occurred" };
-    vi.mocked(updateNotificationSettingsAction).mockResolvedValueOnce(mockErrorResponse);
-
-    const initialSettings = { ...baseNotificationSettings, alert: { [surveyId]: false } };
-    renderSwitch({ notificationSettings: initialSettings, notificationType: "alert" });
-    const switchInput = screen.getByLabelText("toggle notification settings for alert");
-
-    await act(async () => {
-      await user.click(switchInput);
-    });
-
-    expect(updateNotificationSettingsAction).toHaveBeenCalledWith({
-      notificationSettings: { ...initialSettings, alert: { [surveyId]: true } },
-    });
-    expect(toast.error).toHaveBeenCalledWith("An error occurred", {
-      id: "notification-switch",
-    });
-    expect(toast.success).not.toHaveBeenCalled();
-  });
-
-  test("shows error toast when updateNotificationSettingsAction returns undefined", async () => {
-    const mockErrorResponse = { serverError: "An error occurred" };
-    vi.mocked(updateNotificationSettingsAction).mockResolvedValueOnce(mockErrorResponse);
-
-    const initialSettings = { ...baseNotificationSettings, alert: { [surveyId]: false } };
-    renderSwitch({ notificationSettings: initialSettings, notificationType: "alert" });
-    const switchInput = screen.getByLabelText("toggle notification settings for alert");
-
-    await act(async () => {
-      await user.click(switchInput);
-    });
-
-    expect(updateNotificationSettingsAction).toHaveBeenCalledWith({
-      notificationSettings: { ...initialSettings, alert: { [surveyId]: true } },
-    });
-    expect(toast.error).toHaveBeenCalledWith("An error occurred", {
-      id: "notification-switch",
-    });
-    expect(toast.success).not.toHaveBeenCalled();
-  });
-
-  test("shows error toast when updateNotificationSettingsAction returns response without data property", async () => {
-    const mockErrorResponse = { validationErrors: { _errors: ["Invalid input"] } };
-    vi.mocked(updateNotificationSettingsAction).mockResolvedValueOnce(mockErrorResponse);
-
-    const initialSettings = { ...baseNotificationSettings, alert: { [surveyId]: false } };
-    renderSwitch({ notificationSettings: initialSettings, notificationType: "alert" });
-    const switchInput = screen.getByLabelText("toggle notification settings for alert");
-
-    await act(async () => {
-      await user.click(switchInput);
-    });
-
-    expect(updateNotificationSettingsAction).toHaveBeenCalledWith({
-      notificationSettings: { ...initialSettings, alert: { [surveyId]: true } },
-    });
-    expect(toast.error).toHaveBeenCalledWith("Invalid input", {
-      id: "notification-switch",
-    });
-    expect(toast.success).not.toHaveBeenCalled();
-  });
-
-  test("shows error toast when updateNotificationSettingsAction throws an exception", async () => {
-    const mockErrorResponse = { serverError: "Network error" };
-    vi.mocked(updateNotificationSettingsAction).mockResolvedValueOnce(mockErrorResponse);
-
-    const initialSettings = { ...baseNotificationSettings, alert: { [surveyId]: false } };
-    renderSwitch({ notificationSettings: initialSettings, notificationType: "alert" });
-    const switchInput = screen.getByLabelText("toggle notification settings for alert");
-
-    await act(async () => {
-      await user.click(switchInput);
-    });
-
-    expect(updateNotificationSettingsAction).toHaveBeenCalledWith({
-      notificationSettings: { ...initialSettings, alert: { [surveyId]: true } },
-    });
-    expect(toast.error).toHaveBeenCalledWith("Network error", {
-      id: "notification-switch",
-    });
-    expect(toast.success).not.toHaveBeenCalled();
-  });
-
-  test("switch remains enabled after error occurs", async () => {
-    const mockErrorResponse = { serverError: "Failed to update" };
-    vi.mocked(updateNotificationSettingsAction).mockResolvedValueOnce(mockErrorResponse);
-
-    const initialSettings = { ...baseNotificationSettings, alert: { [surveyId]: false } };
-    renderSwitch({ notificationSettings: initialSettings, notificationType: "alert" });
-    const switchInput = screen.getByLabelText("toggle notification settings for alert");
-
-    await act(async () => {
-      await user.click(switchInput);
-    });
-
-    expect(toast.error).toHaveBeenCalledWith("Failed to update", {
-      id: "notification-switch",
-    });
-    expect(switchInput).toBeEnabled(); // Switch should be re-enabled after error
-  });
-
-  test("shows error toast with validation errors for specific fields", async () => {
-    const mockErrorResponse = {
-      validationErrors: {
-        notificationSettings: {
-          _errors: ["Invalid notification settings"],
-        },
-      },
-    };
-    vi.mocked(updateNotificationSettingsAction).mockResolvedValueOnce(mockErrorResponse);
-
-    const initialSettings = { ...baseNotificationSettings, alert: { [surveyId]: false } };
-    renderSwitch({ notificationSettings: initialSettings, notificationType: "alert" });
-    const switchInput = screen.getByLabelText("toggle notification settings for alert");
-
-    await act(async () => {
-      await user.click(switchInput);
-    });
-
-    expect(updateNotificationSettingsAction).toHaveBeenCalledWith({
-      notificationSettings: { ...initialSettings, alert: { [surveyId]: true } },
-    });
-    expect(toast.error).toHaveBeenCalledWith("notificationSettingsInvalid notification settings", {
-      id: "notification-switch",
-    });
-    expect(toast.success).not.toHaveBeenCalled();
-  });
 });

apps/web/app/(app)/environments/[environmentId]/settings/(account)/notifications/components/NotificationSwitch.tsx

@@ -1,9 +1,7 @@
 "use client";
 
-import { getFormattedErrorMessage } from "@/lib/utils/helper";
 import { Switch } from "@/modules/ui/components/switch";
 import { useTranslate } from "@tolgee/react";
-import { useRouter } from "next/navigation";
 import { useEffect, useState } from "react";
 import toast from "react-hot-toast";
 import { TUserNotificationSettings } from "@formbricks/types/user";
@@ -26,7 +24,6 @@ export const NotificationSwitch = ({
 }: NotificationSwitchProps) => {
   const [isLoading, setIsLoading] = useState(false);
   const { t } = useTranslate();
-  const router = useRouter();
   const isChecked =
     notificationType === "unsubscribedOrganizationIds"
       ? !notificationSettings.unsubscribedOrganizationIds?.includes(surveyOrProjectOrOrganizationId)
@@ -53,20 +50,7 @@ export const NotificationSwitch = ({
         !updatedNotificationSettings[notificationType][surveyOrProjectOrOrganizationId];
     }
 
-    const updatedNotificationSettingsActionResponse = await updateNotificationSettingsAction({
-      notificationSettings: updatedNotificationSettings,
-    });
-    if (updatedNotificationSettingsActionResponse?.data) {
-      toast.success(t("environments.settings.notifications.notification_settings_updated"), {
-        id: "notification-switch",
-      });
-      router.refresh();
-    } else {
-      const errorMessage = getFormattedErrorMessage(updatedNotificationSettingsActionResponse);
-      toast.error(errorMessage, {
-        id: "notification-switch",
-      });
-    }
+    await updateNotificationSettingsAction({ notificationSettings: updatedNotificationSettings });
     setIsLoading(false);
   };
 
@@ -120,6 +104,9 @@ export const NotificationSwitch = ({
       disabled={isLoading}
       onCheckedChange={async () => {
         await handleSwitchChange();
+        toast.success(t("environments.settings.notifications.notification_settings_updated"), {
+          id: "notification-switch",
+        });
       }}
     />
   );

apps/web/app/(app)/environments/[environmentId]/settings/(account)/profile/actions.ts

@@ -13,7 +13,7 @@ import { AuthenticatedActionClientCtx } from "@/lib/utils/action-client/types/co
 import { rateLimit } from "@/lib/utils/rate-limit";
 import { updateBrevoCustomer } from "@/modules/auth/lib/brevo";
 import { withAuditLogging } from "@/modules/ee/audit-logs/lib/handler";
-import { sendForgotPasswordEmail, sendVerificationNewEmail } from "@/modules/email";
+import { sendVerificationNewEmail } from "@/modules/email";
 import { z } from "zod";
 import { ZId } from "@formbricks/types/common";
 import {
@@ -162,21 +162,3 @@ export const removeAvatarAction = authenticatedActionClient.schema(ZRemoveAvatar
     }
   )
 );
-
-export const resetPasswordAction = authenticatedActionClient.action(
-  withAuditLogging(
-    "passwordReset",
-    "user",
-    async ({ ctx }: { ctx: AuthenticatedActionClientCtx; parsedInput: undefined }) => {
-      if (ctx.user.identityProvider !== "email") {
-        throw new OperationNotAllowedError("auth.reset-password.not-allowed");
-      }
-
-      await sendForgotPasswordEmail(ctx.user);
-
-      ctx.auditLoggingCtx.userId = ctx.user.id;
-
-      return { success: true };
-    }
-  )
-);

apps/web/app/(app)/environments/[environmentId]/settings/(account)/profile/components/EditProfileDetailsForm.test.tsx

@@ -3,7 +3,7 @@ import userEvent from "@testing-library/user-event";
 import toast from "react-hot-toast";
 import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
 import { TUser } from "@formbricks/types/user";
-import { resetPasswordAction, updateUserAction } from "../actions";
+import { updateUserAction } from "../actions";
 import { EditProfileDetailsForm } from "./EditProfileDetailsForm";
 
 const mockUser = {
@@ -24,8 +24,6 @@ const mockUser = {
   objective: "other",
 } as unknown as TUser;
 
-vi.mock("next-auth/react", () => ({ signOut: vi.fn() }));
-
 // Mock window.location.reload
 const originalLocation = window.location;
 beforeEach(() => {
@@ -37,11 +35,6 @@ beforeEach(() => {
 
 vi.mock("@/app/(app)/environments/[environmentId]/settings/(account)/profile/actions", () => ({
   updateUserAction: vi.fn(),
-  resetPasswordAction: vi.fn(),
-}));
-
-vi.mock("@/modules/auth/forgot-password/actions", () => ({
-  forgotPasswordAction: vi.fn(),
 }));
 
 afterEach(() => {
@@ -57,13 +50,7 @@ describe("EditProfileDetailsForm", () => {
   test("renders with initial user data and updates successfully", async () => {
     vi.mocked(updateUserAction).mockResolvedValue({ ...mockUser, name: "New Name" } as any);
 
-    render(
-      <EditProfileDetailsForm
-        user={mockUser}
-        emailVerificationDisabled={true}
-        isPasswordResetEnabled={false}
-      />
-    );
+    render(<EditProfileDetailsForm user={mockUser} emailVerificationDisabled={true} />);
 
     const nameInput = screen.getByPlaceholderText("common.full_name");
     expect(nameInput).toHaveValue(mockUser.name);
@@ -104,13 +91,7 @@ describe("EditProfileDetailsForm", () => {
     const errorMessage = "Update failed";
     vi.mocked(updateUserAction).mockRejectedValue(new Error(errorMessage));
 
-    render(
-      <EditProfileDetailsForm
-        user={mockUser}
-        emailVerificationDisabled={false}
-        isPasswordResetEnabled={false}
-      />
-    );
+    render(<EditProfileDetailsForm user={mockUser} emailVerificationDisabled={false} />);
 
     const nameInput = screen.getByPlaceholderText("common.full_name");
     await userEvent.clear(nameInput);
@@ -128,13 +109,7 @@ describe("EditProfileDetailsForm", () => {
   });
 
   test("update button is disabled initially and enables on change", async () => {
-    render(
-      <EditProfileDetailsForm
-        user={mockUser}
-        emailVerificationDisabled={false}
-        isPasswordResetEnabled={false}
-      />
-    );
+    render(<EditProfileDetailsForm user={mockUser} emailVerificationDisabled={false} />);
     const updateButton = screen.getByText("common.update");
     expect(updateButton).toBeDisabled();
 
@@ -142,68 +117,4 @@ describe("EditProfileDetailsForm", () => {
     await userEvent.type(nameInput, " updated");
     expect(updateButton).toBeEnabled();
   });
-
-  test("reset password button works", async () => {
-    vi.mocked(resetPasswordAction).mockResolvedValue({ data: { success: true } });
-
-    render(
-      <EditProfileDetailsForm
-        user={mockUser}
-        emailVerificationDisabled={false}
-        isPasswordResetEnabled={true}
-      />
-    );
-
-    const resetButton = screen.getByRole("button", { name: "auth.forgot-password.reset_password" });
-    await userEvent.click(resetButton);
-
-    await waitFor(() => {
-      expect(resetPasswordAction).toHaveBeenCalled();
-    });
-
-    await waitFor(() => {
-      expect(toast.success).toHaveBeenCalledWith("auth.forgot-password.email-sent.heading");
-    });
-  });
-
-  test("reset password button handles error correctly", async () => {
-    const errorMessage = "Reset failed";
-    vi.mocked(resetPasswordAction).mockResolvedValue({ serverError: errorMessage });
-
-    render(
-      <EditProfileDetailsForm
-        user={mockUser}
-        emailVerificationDisabled={false}
-        isPasswordResetEnabled={true}
-      />
-    );
-
-    const resetButton = screen.getByRole("button", { name: "auth.forgot-password.reset_password" });
-    await userEvent.click(resetButton);
-
-    await waitFor(() => {
-      expect(resetPasswordAction).toHaveBeenCalled();
-    });
-
-    await waitFor(() => {
-      expect(toast.error).toHaveBeenCalledWith(errorMessage);
-    });
-  });
-
-  test("reset password button shows loading state", async () => {
-    vi.mocked(resetPasswordAction).mockImplementation(() => new Promise(() => {})); // Never resolves
-
-    render(
-      <EditProfileDetailsForm
-        user={mockUser}
-        emailVerificationDisabled={false}
-        isPasswordResetEnabled={true}
-      />
-    );
-
-    const resetButton = screen.getByRole("button", { name: "auth.forgot-password.reset_password" });
-    await userEvent.click(resetButton);
-
-    expect(resetButton).toBeDisabled();
-  });
 });

apps/web/app/(app)/environments/[environmentId]/settings/(account)/profile/components/EditProfileDetailsForm.tsx

@@ -14,7 +14,6 @@ import {
 } from "@/modules/ui/components/dropdown-menu";
 import { FormControl, FormError, FormField, FormItem, FormLabel } from "@/modules/ui/components/form";
 import { Input } from "@/modules/ui/components/input";
-import { Label } from "@/modules/ui/components/label";
 import { zodResolver } from "@hookform/resolvers/zod";
 import { useTranslate } from "@tolgee/react";
 import { ChevronDownIcon } from "lucide-react";
@@ -23,7 +22,7 @@ import { FormProvider, SubmitHandler, useForm } from "react-hook-form";
 import toast from "react-hot-toast";
 import { z } from "zod";
 import { TUser, TUserUpdateInput, ZUser, ZUserEmail } from "@formbricks/types/user";
-import { resetPasswordAction, updateUserAction } from "../actions";
+import { updateUserAction } from "../actions";
 
 // Schema & types
 const ZEditProfileNameFormSchema = ZUser.pick({ name: true, locale: true, email: true }).extend({
@@ -31,17 +30,13 @@ const ZEditProfileNameFormSchema = ZUser.pick({ name: true, locale: true, email:
 });
 type TEditProfileNameForm = z.infer<typeof ZEditProfileNameFormSchema>;
 
-interface IEditProfileDetailsFormProps {
-  user: TUser;
-  isPasswordResetEnabled?: boolean;
-  emailVerificationDisabled: boolean;
-}
-
 export const EditProfileDetailsForm = ({
   user,
-  isPasswordResetEnabled,
   emailVerificationDisabled,
-}: IEditProfileDetailsFormProps) => {
+}: {
+  user: TUser;
+  emailVerificationDisabled: boolean;
+}) => {
   const { t } = useTranslate();
 
   const form = useForm<TEditProfileNameForm>({
@@ -55,8 +50,6 @@ export const EditProfileDetailsForm = ({
   });
 
   const { isSubmitting, isDirty } = form.formState;
-
-  const [isResettingPassword, setIsResettingPassword] = useState(false);
   const [showModal, setShowModal] = useState(false);
   const { signOut: signOutWithAudit } = useSignOut({ id: user.id, email: user.email });
 
@@ -97,7 +90,6 @@ export const EditProfileDetailsForm = ({
           redirectUrl: "/email-change-without-verification-success",
           redirect: true,
           callbackUrl: "/email-change-without-verification-success",
-          clearEnvironmentId: true,
         });
         return;
       }
@@ -129,28 +121,6 @@ export const EditProfileDetailsForm = ({
     }
   };
 
-  const handleResetPassword = async () => {
-    setIsResettingPassword(true);
-
-    const result = await resetPasswordAction();
-    if (result?.data) {
-      toast.success(t("auth.forgot-password.email-sent.heading"));
-
-      await signOutWithAudit({
-        reason: "password_reset",
-        redirectUrl: "/auth/login",
-        redirect: true,
-        callbackUrl: "/auth/login",
-        clearEnvironmentId: true,
-      });
-    } else {
-      const errorMessage = getFormattedErrorMessage(result);
-      toast.error(t(errorMessage));
-    }
-
-    setIsResettingPassword(false);
-  };
-
   return (
     <>
       <FormProvider {...form}>
@@ -235,26 +205,6 @@ export const EditProfileDetailsForm = ({
             )}
           />
 
-          {isPasswordResetEnabled && (
-            <div className="mt-4 space-y-2">
-              <Label htmlFor="reset-password">{t("auth.forgot-password.reset_password")}</Label>
-              <p className="mt-1 text-sm text-slate-500">
-                {t("auth.forgot-password.reset_password_description")}
-              </p>
-              <div className="flex items-center justify-between gap-2">
-                <Input type="email" id="reset-password" defaultValue={user.email} disabled />
-                <Button
-                  onClick={handleResetPassword}
-                  loading={isResettingPassword}
-                  disabled={isResettingPassword}
-                  size="default"
-                  variant="secondary">
-                  {t("auth.forgot-password.reset_password")}
-                </Button>
-              </div>
-            </div>
-          )}
-
           <Button
             type="submit"
             className="mt-4"

apps/web/app/(app)/environments/[environmentId]/settings/(account)/profile/page.test.tsx

@@ -12,8 +12,7 @@ import Page from "./page";
 
 // Mock services and utils
 vi.mock("@/lib/constants", () => ({
-  IS_FORMBRICKS_CLOUD: 1,
-  PASSWORD_RESET_DISABLED: 1,
+  IS_FORMBRICKS_CLOUD: true,
   EMAIL_VERIFICATION_DISABLED: true,
 }));
 vi.mock("@/lib/organization/service", () => ({

apps/web/app/(app)/environments/[environmentId]/settings/(account)/profile/page.tsx

@@ -1,6 +1,6 @@
 import { AccountSettingsNavbar } from "@/app/(app)/environments/[environmentId]/settings/(account)/components/AccountSettingsNavbar";
 import { AccountSecurity } from "@/app/(app)/environments/[environmentId]/settings/(account)/profile/components/AccountSecurity";
-import { EMAIL_VERIFICATION_DISABLED, IS_FORMBRICKS_CLOUD, PASSWORD_RESET_DISABLED } from "@/lib/constants";
+import { EMAIL_VERIFICATION_DISABLED, IS_FORMBRICKS_CLOUD } from "@/lib/constants";
 import { getOrganizationsWhereUserIsSingleOwner } from "@/lib/organization/service";
 import { getUser } from "@/lib/user/service";
 import { getIsMultiOrgEnabled, getIsTwoFactorAuthEnabled } from "@/modules/ee/license-check/lib/utils";
@@ -32,8 +32,6 @@ const Page = async (props: { params: Promise<{ environmentId: string }> }) => {
     throw new Error(t("common.user_not_found"));
   }
 
-  const isPasswordResetEnabled = !PASSWORD_RESET_DISABLED && user.identityProvider === "email";
-
   return (
     <PageContentWrapper>
       <PageHeader pageTitle={t("common.account_settings")}>
@@ -44,11 +42,7 @@ const Page = async (props: { params: Promise<{ environmentId: string }> }) => {
           <SettingsCard
             title={t("environments.settings.profile.personal_information")}
             description={t("environments.settings.profile.update_personal_info")}>
-            <EditProfileDetailsForm
-              user={user}
-              emailVerificationDisabled={EMAIL_VERIFICATION_DISABLED}
-              isPasswordResetEnabled={isPasswordResetEnabled}
-            />
+            <EditProfileDetailsForm emailVerificationDisabled={EMAIL_VERIFICATION_DISABLED} user={user} />
           </SettingsCard>
           <SettingsCard
             title={t("common.avatar")}

apps/web/app/(app)/environments/[environmentId]/surveys/[surveyId]/(analysis)/summary/components/ShareEmbedSurvey.test.tsx

@@ -176,8 +176,8 @@ describe("ShareEmbedSurvey", () => {
     ));
   });
 
-  test("renders initial 'start' view correctly when open and modalView is 'start' for link survey", () => {
-    render(<ShareEmbedSurvey {...defaultProps} survey={mockSurveyLink} />);
+  test("renders initial 'start' view correctly when open and modalView is 'start'", () => {
+    render(<ShareEmbedSurvey {...defaultProps} />);
     expect(screen.getByText("environments.surveys.summary.your_survey_is_public 🎉")).toBeInTheDocument();
     expect(screen.getByText("ShareSurveyLinkMock")).toBeInTheDocument();
     expect(screen.getByText("environments.surveys.summary.whats_next")).toBeInTheDocument();
@@ -188,18 +188,6 @@ describe("ShareEmbedSurvey", () => {
     expect(screen.getByTestId("badge-mock")).toHaveTextContent("common.new");
   });
 
-  test("renders initial 'start' view correctly when open and modalView is 'start' for app survey", () => {
-    render(<ShareEmbedSurvey {...defaultProps} survey={mockSurveyWeb} />);
-    // For app surveys, ShareSurveyLink should not be rendered
-    expect(screen.queryByText("ShareSurveyLinkMock")).not.toBeInTheDocument();
-    expect(screen.getByText("environments.surveys.summary.whats_next")).toBeInTheDocument();
-    expect(screen.getByText("environments.surveys.summary.embed_survey")).toBeInTheDocument();
-    expect(screen.getByText("environments.surveys.summary.configure_alerts")).toBeInTheDocument();
-    expect(screen.getByText("environments.surveys.summary.setup_integrations")).toBeInTheDocument();
-    expect(screen.getByText("environments.surveys.summary.send_to_panel")).toBeInTheDocument();
-    expect(screen.getByTestId("badge-mock")).toHaveTextContent("common.new");
-  });
-
   test("switches to 'embed' view when 'Embed survey' button is clicked", async () => {
     render(<ShareEmbedSurvey {...defaultProps} />);
     const embedButton = screen.getByText("environments.surveys.summary.embed_survey");
@@ -217,7 +205,7 @@ describe("ShareEmbedSurvey", () => {
   });
 
   test("returns to 'start' view when handleInitialPageButton is triggered from EmbedView", async () => {
-    render(<ShareEmbedSurvey {...defaultProps} survey={mockSurveyLink} modalView="embed" />);
+    render(<ShareEmbedSurvey {...defaultProps} modalView="embed" />);
     expect(mockEmbedViewComponent).toHaveBeenCalled();
     expect(screen.getByText("EmbedViewMockContent")).toBeInTheDocument();
 
@@ -231,7 +219,7 @@ describe("ShareEmbedSurvey", () => {
   });
 
   test("returns to 'start' view when handleInitialPageButton is triggered from PanelInfoView", async () => {
-    render(<ShareEmbedSurvey {...defaultProps} survey={mockSurveyLink} modalView="panel" />);
+    render(<ShareEmbedSurvey {...defaultProps} modalView="panel" />);
     expect(mockPanelInfoViewComponent).toHaveBeenCalled();
     expect(screen.getByText("PanelInfoViewMockContent")).toBeInTheDocument();
 
@@ -269,8 +257,8 @@ describe("ShareEmbedSurvey", () => {
     };
     expect(embedViewProps.tabs.length).toBe(3);
     expect(embedViewProps.tabs.find((tab) => tab.id === "app")).toBeUndefined();
-    expect(embedViewProps.tabs[0].id).toBe("link");
-    expect(embedViewProps.activeId).toBe("link");
+    expect(embedViewProps.tabs[0].id).toBe("email");
+    expect(embedViewProps.activeId).toBe("email");
   });
 
   test("correctly configures for 'web' survey type in embed view", () => {

apps/web/app/(app)/environments/[environmentId]/surveys/[surveyId]/(analysis)/summary/components/ShareEmbedSurvey.tsx

@@ -47,14 +47,13 @@ export const ShareEmbedSurvey = ({
   const tabs = useMemo(
     () =>
       [
+        { id: "email", label: t("environments.surveys.summary.embed_in_an_email"), icon: MailIcon },
+        { id: "webpage", label: t("environments.surveys.summary.embed_on_website"), icon: Code2Icon },
         {
           id: "link",
           label: `${isSingleUseLinkSurvey ? t("environments.surveys.summary.single_use_links") : t("environments.surveys.summary.share_the_link")}`,
           icon: LinkIcon,
         },
-        { id: "email", label: t("environments.surveys.summary.embed_in_an_email"), icon: MailIcon },
-        { id: "webpage", label: t("environments.surveys.summary.embed_on_website"), icon: Code2Icon },
-
         { id: "app", label: t("environments.surveys.summary.embed_in_app"), icon: SmartphoneIcon },
       ].filter((tab) => !(survey.type === "link" && tab.id === "app")),
     [t, isSingleUseLinkSurvey, survey.type]
@@ -107,28 +106,27 @@ export const ShareEmbedSurvey = ({
 
   return (
     <Dialog open={open} onOpenChange={handleOpenChange}>
-      <DialogContent className="w-full bg-white p-0 lg:h-[700px]" width="wide">
+      <DialogTitle className="sr-only" />
+      <DialogContent className="w-full max-w-xl bg-white p-0 md:max-w-3xl lg:h-[700px] lg:max-w-5xl">
         {showView === "start" ? (
-          <div className="flex h-full max-w-full flex-col overflow-hidden">
-            {survey.type === "link" && (
-              <div className="flex h-2/5 w-full flex-col items-center justify-center space-y-6 p-8 text-center">
-                <DialogTitle>
-                  <p className="pt-2 text-xl font-semibold text-slate-800">
-                    {t("environments.surveys.summary.your_survey_is_public")} 🎉
-                  </p>
-                </DialogTitle>
-                <DialogDescription className="hidden" />
-                <ShareSurveyLink
-                  survey={survey}
-                  surveyUrl={surveyUrl}
-                  publicDomain={publicDomain}
-                  setSurveyUrl={setSurveyUrl}
-                  locale={user.locale}
-                />
-              </div>
-            )}
-            <div className="flex h-full flex-col items-center justify-center gap-4 rounded-b-lg bg-slate-50 px-8">
-              <p className="text-sm text-slate-500">{t("environments.surveys.summary.whats_next")}</p>
+          <div className="h-full max-w-full overflow-hidden">
+            <div className="flex h-[200px] w-full flex-col items-center justify-center space-y-6 p-8 text-center lg:h-2/5">
+              <DialogTitle>
+                <p className="pt-2 text-xl font-semibold text-slate-800">
+                  {t("environments.surveys.summary.your_survey_is_public")} 🎉
+                </p>
+              </DialogTitle>
+              <DialogDescription className="hidden" />
+              <ShareSurveyLink
+                survey={survey}
+                surveyUrl={surveyUrl}
+                publicDomain={publicDomain}
+                setSurveyUrl={setSurveyUrl}
+                locale={user.locale}
+              />
+            </div>
+            <div className="flex h-[300px] flex-col items-center justify-center gap-8 rounded-b-lg bg-slate-50 px-8 lg:h-3/5">
+              <p className="-mt-8 text-sm text-slate-500">{t("environments.surveys.summary.whats_next")}</p>
               <div className="grid grid-cols-4 gap-2">
                 <button
                   type="button"

apps/web/app/(app)/environments/[environmentId]/surveys/[surveyId]/(analysis)/summary/components/SummaryMetadata.tsx

@@ -1,6 +1,5 @@
 "use client";
 
-import { Button } from "@/modules/ui/components/button";
 import { Tooltip, TooltipContent, TooltipProvider, TooltipTrigger } from "@/modules/ui/components/tooltip";
 import { useTranslate } from "@tolgee/react";
 import { ChevronDownIcon, ChevronUpIcon } from "lucide-react";
@@ -118,13 +117,13 @@ export const SummaryMetadata = ({
                     )}
                   </span>
                   {!isLoading && (
-                    <Button variant="secondary" className="h-6 w-6">
+                    <span className="ml-1 flex items-center rounded-md bg-slate-800 px-2 py-1 text-xs text-slate-50 group-hover:bg-slate-700">
                       {showDropOffs ? (
                         <ChevronUpIcon className="h-4 w-4" />
                       ) : (
                         <ChevronDownIcon className="h-4 w-4" />
                       )}
-                    </Button>
+                    </span>
                   )}
                 </div>
               </div>

apps/web/app/(app)/environments/[environmentId]/surveys/[surveyId]/(analysis)/summary/components/SurveyAnalysisCTA.test.tsx

@@ -69,13 +69,13 @@ vi.mock("@/modules/survey/hooks/useSingleUseId", () => ({
 
 const mockSearchParams = new URLSearchParams();
 const mockPush = vi.fn();
-const mockReplace = vi.fn();
 
 // Mock next/navigation
 vi.mock("next/navigation", () => ({
-  useRouter: () => ({ push: mockPush, replace: mockReplace }),
+  useRouter: () => ({ push: mockPush }),
   useSearchParams: () => mockSearchParams,
-  usePathname: () => "/current-path",
+  usePathname: () => "/current",
+  useParams: () => ({ environmentId: "env123", surveyId: "survey123" }),
 }));
 
 // Mock copySurveyLink to return a predictable string
@@ -131,281 +131,280 @@ const dummySurvey = {
   id: "survey123",
   type: "link",
   environmentId: "env123",
-  status: "inProgress",
-  resultShareKey: null,
+  status: "active",
 } as unknown as TSurvey;
-
-const dummyAppSurvey = {
-  id: "survey123",
-  type: "app",
-  environmentId: "env123",
-  status: "inProgress",
-} as unknown as TSurvey;
-
 const dummyEnvironment = { id: "env123", appSetupCompleted: true } as TEnvironment;
 const dummyUser = { id: "user123", name: "Test User" } as TUser;
 
-describe("SurveyAnalysisCTA", () => {
+describe("SurveyAnalysisCTA - handleCopyLink", () => {
+  afterEach(() => {
+    cleanup();
+  });
+
+  test("calls copySurveyLink and clipboard.writeText on success", async () => {
+    render(
+      <SurveyAnalysisCTA
+        survey={dummySurvey}
+        environment={dummyEnvironment}
+        isReadOnly={false}
+        publicDomain={mockPublicDomain}
+        user={dummyUser}
+        responseCount={5}
+      />
+    );
+
+    const copyButton = screen.getByRole("button", { name: "common.copy_link" });
+    fireEvent.click(copyButton);
+
+    await waitFor(() => {
+      expect(refreshSingleUseIdSpy).toHaveBeenCalled();
+      expect(writeTextMock).toHaveBeenCalledWith("https://public-domain.com/s/survey123?suId=newSingleUseId");
+      expect(toast.success).toHaveBeenCalledWith("common.copied_to_clipboard");
+    });
+  });
+
+  test("shows error toast on failure", async () => {
+    refreshSingleUseIdSpy.mockImplementationOnce(() => Promise.reject(new Error("fail")));
+    render(
+      <SurveyAnalysisCTA
+        survey={dummySurvey}
+        environment={dummyEnvironment}
+        isReadOnly={false}
+        publicDomain={mockPublicDomain}
+        user={dummyUser}
+        responseCount={5}
+      />
+    );
+
+    const copyButton = screen.getByRole("button", { name: "common.copy_link" });
+    fireEvent.click(copyButton);
+
+    await waitFor(() => {
+      expect(refreshSingleUseIdSpy).toHaveBeenCalled();
+      expect(writeTextMock).not.toHaveBeenCalled();
+      expect(toast.error).toHaveBeenCalledWith("environments.surveys.summary.failed_to_copy_link");
+    });
+  });
+});
+
+// New tests for squarePenIcon and edit functionality
+describe("SurveyAnalysisCTA - Edit functionality", () => {
   beforeEach(() => {
     vi.resetAllMocks();
-    mockSearchParams.delete("share"); // reset params
   });
 
   afterEach(() => {
     cleanup();
   });
 
-  describe("Edit functionality", () => {
-    test("opens EditPublicSurveyAlertDialog when edit icon is clicked and response count > 0", async () => {
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
+  test("opens EditPublicSurveyAlertDialog when edit icon is clicked and response count > 0", async () => {
+    render(
+      <SurveyAnalysisCTA
+        survey={dummySurvey}
+        environment={dummyEnvironment}
+        isReadOnly={false}
+        publicDomain={mockPublicDomain}
+        user={dummyUser}
+        responseCount={5}
+      />
+    );
 
-      // Find the edit button
-      const editButton = screen.getByRole("button", { name: "common.edit" });
-      await fireEvent.click(editButton);
+    // Find the edit button
+    const editButton = screen.getByRole("button", { name: "common.edit" });
+    await fireEvent.click(editButton);
 
-      // Check if dialog is shown
-      const dialogTitle = screen.getByText("environments.surveys.edit.caution_edit_published_survey");
-      expect(dialogTitle).toBeInTheDocument();
-    });
-
-    test("navigates directly to edit page when response count = 0", async () => {
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={0}
-        />
-      );
-
-      // Find the edit button
-      const editButton = screen.getByRole("button", { name: "common.edit" });
-      await fireEvent.click(editButton);
-
-      // Should navigate directly to edit page
-      expect(mockPush).toHaveBeenCalledWith(
-        `/environments/${dummyEnvironment.id}/surveys/${dummySurvey.id}/edit`
-      );
-    });
-
-    test("doesn't show edit button when isReadOnly is true", () => {
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={true}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
-
-      const editButton = screen.queryByRole("button", { name: "common.edit" });
-      expect(editButton).not.toBeInTheDocument();
-    });
+    // Check if dialog is shown
+    const dialogTitle = screen.getByText("environments.surveys.edit.caution_edit_published_survey");
+    expect(dialogTitle).toBeInTheDocument();
   });
 
-  describe("Duplicate functionality", () => {
-    test("duplicates survey and redirects on primary button click", async () => {
-      mockCopySurveyToOtherEnvironmentAction.mockResolvedValue({
-        data: { id: "newSurvey456" },
-      });
+  test("navigates directly to edit page when response count = 0", async () => {
+    render(
+      <SurveyAnalysisCTA
+        survey={dummySurvey}
+        environment={dummyEnvironment}
+        isReadOnly={false}
+        publicDomain={mockPublicDomain}
+        user={dummyUser}
+        responseCount={0}
+      />
+    );
 
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
+    // Find the edit button
+    const editButton = screen.getByRole("button", { name: "common.edit" });
+    await fireEvent.click(editButton);
 
-      const editButton = screen.getByRole("button", { name: "common.edit" });
-      fireEvent.click(editButton);
-
-      const primaryButton = await screen.findByText("environments.surveys.edit.caution_edit_duplicate");
-      fireEvent.click(primaryButton);
-
-      await waitFor(() => {
-        expect(mockCopySurveyToOtherEnvironmentAction).toHaveBeenCalledWith({
-          environmentId: "env123",
-          surveyId: "survey123",
-          targetEnvironmentId: "env123",
-        });
-        expect(mockPush).toHaveBeenCalledWith("/environments/env123/surveys/newSurvey456/edit");
-        expect(toast.success).toHaveBeenCalledWith("environments.surveys.survey_duplicated_successfully");
-      });
-    });
-
-    test("shows error toast on duplication failure", async () => {
-      const error = { error: "Duplication failed" };
-      mockCopySurveyToOtherEnvironmentAction.mockResolvedValue(error);
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
-
-      const editButton = screen.getByRole("button", { name: "common.edit" });
-      fireEvent.click(editButton);
-
-      const primaryButton = await screen.findByText("environments.surveys.edit.caution_edit_duplicate");
-      fireEvent.click(primaryButton);
-
-      await waitFor(() => {
-        expect(toast.error).toHaveBeenCalledWith("Duplication failed");
-      });
-    });
+    // Should navigate directly to edit page
+    expect(mockPush).toHaveBeenCalledWith(
+      `/environments/${dummyEnvironment.id}/surveys/${dummySurvey.id}/edit`
+    );
   });
 
-  describe("Share button and modal", () => {
-    test("opens share modal when 'Share survey' button is clicked", async () => {
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
+  test("doesn't show edit button when isReadOnly is true", () => {
+    render(
+      <SurveyAnalysisCTA
+        survey={dummySurvey}
+        environment={dummyEnvironment}
+        isReadOnly={true}
+        publicDomain={mockPublicDomain}
+        user={dummyUser}
+        responseCount={5}
+      />
+    );
 
-      const shareButton = screen.getByText("environments.surveys.summary.share_survey");
-      fireEvent.click(shareButton);
+    // Try to find the edit button (it shouldn't exist)
+    const editButton = screen.queryByRole("button", { name: "common.edit" });
+    expect(editButton).not.toBeInTheDocument();
+  });
+});
 
-      // The share button opens the embed modal, not a URL
-      // We can verify this by checking that the ShareEmbedSurvey component is rendered
-      // with the embed modal open
-      expect(screen.getByText("environments.surveys.summary.share_survey")).toBeInTheDocument();
-    });
-
-    test("renders ShareEmbedSurvey component when share modal is open", async () => {
-      mockSearchParams.set("share", "true");
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
-
-      // Assuming ShareEmbedSurvey renders a dialog with a specific title when open
-      const dialog = await screen.findByRole("dialog");
-      expect(dialog).toBeInTheDocument();
-    });
+// Updated test description to mention EditPublicSurveyAlertDialog
+describe("SurveyAnalysisCTA - duplicateSurveyAndRoute and EditPublicSurveyAlertDialog", () => {
+  afterEach(() => {
+    cleanup();
   });
 
-  describe("General UI and visibility", () => {
-    test("shows public results badge when resultShareKey is present", () => {
-      const surveyWithShareKey = { ...dummySurvey, resultShareKey: "someKey" } as TSurvey;
-      render(
-        <SurveyAnalysisCTA
-          survey={surveyWithShareKey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
-
-      expect(screen.getByText("environments.surveys.summary.results_are_public")).toBeInTheDocument();
+  test("duplicates survey successfully and navigates to edit page", async () => {
+    // Mock the API response
+    mockCopySurveyToOtherEnvironmentAction.mockResolvedValueOnce({
+      data: { id: "duplicated-survey-456" },
     });
 
-    test("shows SurveyStatusDropdown for non-draft surveys", () => {
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
+    render(
+      <SurveyAnalysisCTA
+        survey={dummySurvey}
+        environment={dummyEnvironment}
+        isReadOnly={false}
+        publicDomain={mockPublicDomain}
+        user={dummyUser}
+        responseCount={5}
+      />
+    );
 
-      expect(screen.getByRole("combobox")).toBeInTheDocument();
+    // Find and click the edit button to show dialog
+    const editButton = screen.getByRole("button", { name: "common.edit" });
+    await fireEvent.click(editButton);
+
+    // Find and click the duplicate button in dialog
+    const duplicateButton = screen.getByRole("button", {
+      name: "environments.surveys.edit.caution_edit_duplicate",
+    });
+    await fireEvent.click(duplicateButton);
+
+    // Verify the API was called with correct parameters
+    expect(mockCopySurveyToOtherEnvironmentAction).toHaveBeenCalledWith({
+      environmentId: dummyEnvironment.id,
+      surveyId: dummySurvey.id,
+      targetEnvironmentId: dummyEnvironment.id,
     });
 
-    test("does not show SurveyStatusDropdown for draft surveys", () => {
-      const draftSurvey = { ...dummySurvey, status: "draft" } as TSurvey;
-      render(
-        <SurveyAnalysisCTA
-          survey={draftSurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
-      expect(screen.queryByRole("combobox")).not.toBeInTheDocument();
+    // Verify success toast was shown
+    expect(toast.success).toHaveBeenCalledWith("environments.surveys.survey_duplicated_successfully");
+
+    // Verify navigation to edit page
+    expect(mockPush).toHaveBeenCalledWith(
+      `/environments/${dummyEnvironment.id}/surveys/duplicated-survey-456/edit`
+    );
+  });
+
+  test("shows error toast when duplication fails with error object", async () => {
+    // Mock API failure with error object
+    mockCopySurveyToOtherEnvironmentAction.mockResolvedValueOnce({
+      error: "Test error message",
     });
 
-    test("hides status dropdown and edit actions when isReadOnly is true", () => {
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={true}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
+    render(
+      <SurveyAnalysisCTA
+        survey={dummySurvey}
+        environment={dummyEnvironment}
+        isReadOnly={false}
+        publicDomain={mockPublicDomain}
+        user={dummyUser}
+        responseCount={5}
+      />
+    );
 
-      expect(screen.queryByRole("combobox")).not.toBeInTheDocument();
-      expect(screen.queryByRole("button", { name: "common.edit" })).not.toBeInTheDocument();
+    // Open dialog
+    const editButton = screen.getByRole("button", { name: "common.edit" });
+    await fireEvent.click(editButton);
+
+    // Click duplicate
+    const duplicateButton = screen.getByRole("button", {
+      name: "environments.surveys.edit.caution_edit_duplicate",
+    });
+    await fireEvent.click(duplicateButton);
+
+    // Verify error toast
+    expect(toast.error).toHaveBeenCalledWith("Test error message");
+  });
+
+  test("navigates to edit page when cancel button is clicked in dialog", async () => {
+    render(
+      <SurveyAnalysisCTA
+        survey={dummySurvey}
+        environment={dummyEnvironment}
+        isReadOnly={false}
+        publicDomain={mockPublicDomain}
+        user={dummyUser}
+        responseCount={5}
+      />
+    );
+
+    // Open dialog
+    const editButton = screen.getByRole("button", { name: "common.edit" });
+    await fireEvent.click(editButton);
+
+    // Click edit (cancel) button
+    const editButtonInDialog = screen.getByRole("button", { name: "common.edit" });
+    await fireEvent.click(editButtonInDialog);
+
+    // Verify navigation
+    expect(mockPush).toHaveBeenCalledWith(
+      `/environments/${dummyEnvironment.id}/surveys/${dummySurvey.id}/edit`
+    );
+  });
+
+  test("shows loading state when duplicating survey", async () => {
+    // Create a promise that we can resolve manually
+    let resolvePromise: (value: any) => void;
+    const promise = new Promise((resolve) => {
+      resolvePromise = resolve;
     });
 
-    test("shows preview button for link surveys", () => {
-      render(
-        <SurveyAnalysisCTA
-          survey={dummySurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
-      expect(screen.getByRole("button", { name: "common.preview" })).toBeInTheDocument();
+    mockCopySurveyToOtherEnvironmentAction.mockImplementation(() => promise);
+
+    render(
+      <SurveyAnalysisCTA
+        survey={dummySurvey}
+        environment={dummyEnvironment}
+        isReadOnly={false}
+        publicDomain={mockPublicDomain}
+        user={dummyUser}
+        responseCount={5}
+      />
+    );
+
+    // Open dialog
+    const editButton = screen.getByRole("button", { name: "common.edit" });
+    await fireEvent.click(editButton);
+
+    // Click duplicate
+    const duplicateButton = screen.getByRole("button", {
+      name: "environments.surveys.edit.caution_edit_duplicate",
+    });
+    await fireEvent.click(duplicateButton);
+
+    // Button should now be in loading state
+    // expect(duplicateButton).toHaveAttribute("data-state", "loading");
+
+    // Resolve the promise
+    resolvePromise!({
+      data: { id: "duplicated-survey-456" },
     });
 
-    test("hides preview button for app surveys", () => {
-      render(
-        <SurveyAnalysisCTA
-          survey={dummyAppSurvey}
-          environment={dummyEnvironment}
-          isReadOnly={false}
-          publicDomain={mockPublicDomain}
-          user={dummyUser}
-          responseCount={5}
-        />
-      );
-      expect(screen.queryByRole("button", { name: "common.preview" })).not.toBeInTheDocument();
+    // Wait for the promise to resolve
+    await waitFor(() => {
+      expect(mockPush).toHaveBeenCalled();
     });
   });
 });

apps/web/app/(app)/environments/[environmentId]/surveys/[surveyId]/(analysis)/summary/components/SurveyAnalysisCTA.tsx

@@ -5,12 +5,13 @@ import { SuccessMessage } from "@/app/(app)/environments/[environmentId]/surveys
 import { SurveyStatusDropdown } from "@/app/(app)/environments/[environmentId]/surveys/[surveyId]/components/SurveyStatusDropdown";
 import { getFormattedErrorMessage } from "@/lib/utils/helper";
 import { EditPublicSurveyAlertDialog } from "@/modules/survey/components/edit-public-survey-alert-dialog";
+import { useSingleUseId } from "@/modules/survey/hooks/useSingleUseId";
+import { copySurveyLink } from "@/modules/survey/lib/client-utils";
 import { copySurveyToOtherEnvironmentAction } from "@/modules/survey/list/actions";
 import { Badge } from "@/modules/ui/components/badge";
-import { Button } from "@/modules/ui/components/button";
 import { IconBar } from "@/modules/ui/components/iconbar";
 import { useTranslate } from "@tolgee/react";
-import { BellRing, Eye, SquarePenIcon } from "lucide-react";
+import { BellRing, Code2Icon, Eye, LinkIcon, SquarePenIcon, UsersRound } from "lucide-react";
 import { usePathname, useRouter, useSearchParams } from "next/navigation";
 import { useEffect, useMemo, useState } from "react";
 import toast from "react-hot-toast";
@@ -56,6 +57,7 @@ export const SurveyAnalysisCTA = ({
   });
 
   const surveyUrl = useMemo(() => `${publicDomain}/s/${survey.id}`, [survey.id, publicDomain]);
+  const { refreshSingleUseId } = useSingleUseId(survey);
 
   const widgetSetupCompleted = survey.type === "app" && environment.appSetupCompleted;
 
@@ -77,6 +79,22 @@ export const SurveyAnalysisCTA = ({
     setModalState((prev) => ({ ...prev, share: open }));
   };
 
+  const handleCopyLink = () => {
+    refreshSingleUseId()
+      .then((newId) => {
+        const linkToCopy = copySurveyLink(surveyUrl, newId);
+        return navigator.clipboard.writeText(linkToCopy);
+      })
+      .then(() => {
+        toast.success(t("common.copied_to_clipboard"));
+      })
+      .catch((err) => {
+        toast.error(t("environments.surveys.summary.failed_to_copy_link"));
+        console.error(err);
+      });
+    setModalState((prev) => ({ ...prev, dropdown: false }));
+  };
+
   const duplicateSurveyAndRoute = async (surveyId: string) => {
     setLoading(true);
     const duplicatedSurveyResponse = await copySurveyToOtherEnvironmentAction({
@@ -116,6 +134,24 @@ export const SurveyAnalysisCTA = ({
   const [isCautionDialogOpen, setIsCautionDialogOpen] = useState(false);
 
   const iconActions = [
+    {
+      icon: Eye,
+      tooltip: t("common.preview"),
+      onClick: () => window.open(getPreviewUrl(), "_blank"),
+      isVisible: survey.type === "link",
+    },
+    {
+      icon: LinkIcon,
+      tooltip: t("common.copy_link"),
+      onClick: handleCopyLink,
+      isVisible: survey.type === "link",
+    },
+    {
+      icon: Code2Icon,
+      tooltip: t("common.embed"),
+      onClick: () => handleModalState("embed")(true),
+      isVisible: !isReadOnly,
+    },
     {
       icon: BellRing,
       tooltip: t("environments.surveys.summary.configure_alerts"),
@@ -123,10 +159,13 @@ export const SurveyAnalysisCTA = ({
       isVisible: !isReadOnly,
     },
     {
-      icon: Eye,
-      tooltip: t("common.preview"),
-      onClick: () => window.open(getPreviewUrl(), "_blank"),
-      isVisible: survey.type === "link",
+      icon: UsersRound,
+      tooltip: t("environments.surveys.summary.send_to_panel"),
+      onClick: () => {
+        handleModalState("panel")(true);
+        setModalState((prev) => ({ ...prev, dropdown: false }));
+      },
+      isVisible: !isReadOnly,
     },
     {
       icon: SquarePenIcon,
@@ -156,13 +195,6 @@ export const SurveyAnalysisCTA = ({
       )}
 
       <IconBar actions={iconActions} />
-      <Button
-        className="h-10"
-        onClick={() => {
-          setModalState((prev) => ({ ...prev, embed: true }));
-        }}>
-        {t("environments.surveys.summary.share_survey")}
-      </Button>
 
       {user && (
         <>

apps/web/app/ClientEnvironmentRedirect.test.tsx

@@ -14,64 +14,41 @@ describe("ClientEnvironmentRedirect", () => {
     cleanup();
   });
 
-  test("should redirect to the first environment ID when no last environment exists", () => {
+  test("should redirect to the provided environment ID when no last environment exists", () => {
     const mockPush = vi.fn();
     vi.mocked(useRouter).mockReturnValue({ push: mockPush } as any);
 
     // Mock localStorage
     const localStorageMock = {
       getItem: vi.fn().mockReturnValue(null),
-      removeItem: vi.fn(),
     };
-
     Object.defineProperty(window, "localStorage", {
       value: localStorageMock,
     });
 
-    render(<ClientEnvironmentRedirect userEnvironments={["test-env-id"]} />);
+    render(<ClientEnvironmentRedirect environmentId="test-env-id" />);
 
     expect(mockPush).toHaveBeenCalledWith("/environments/test-env-id");
   });
 
-  test("should redirect to the last environment ID when it exists in localStorage and is valid", () => {
+  test("should redirect to the last environment ID when it exists in localStorage", () => {
     const mockPush = vi.fn();
     vi.mocked(useRouter).mockReturnValue({ push: mockPush } as any);
 
     // Mock localStorage with a last environment ID
     const localStorageMock = {
       getItem: vi.fn().mockReturnValue("last-env-id"),
-      removeItem: vi.fn(),
     };
     Object.defineProperty(window, "localStorage", {
       value: localStorageMock,
     });
 
-    render(<ClientEnvironmentRedirect userEnvironments={["last-env-id", "other-env-id"]} />);
+    render(<ClientEnvironmentRedirect environmentId="test-env-id" />);
 
     expect(localStorageMock.getItem).toHaveBeenCalledWith(FORMBRICKS_ENVIRONMENT_ID_LS);
     expect(mockPush).toHaveBeenCalledWith("/environments/last-env-id");
   });
 
-  test("should clear invalid environment ID and redirect to default when stored ID is not in user environments", () => {
-    const mockPush = vi.fn();
-    vi.mocked(useRouter).mockReturnValue({ push: mockPush } as any);
-
-    // Mock localStorage with an invalid environment ID
-    const localStorageMock = {
-      getItem: vi.fn().mockReturnValue("invalid-env-id"),
-      removeItem: vi.fn(),
-    };
-    Object.defineProperty(window, "localStorage", {
-      value: localStorageMock,
-    });
-
-    render(<ClientEnvironmentRedirect userEnvironments={["valid-env-1", "valid-env-2"]} />);
-
-    expect(localStorageMock.getItem).toHaveBeenCalledWith(FORMBRICKS_ENVIRONMENT_ID_LS);
-    expect(localStorageMock.removeItem).toHaveBeenCalledWith(FORMBRICKS_ENVIRONMENT_ID_LS);
-    expect(mockPush).toHaveBeenCalledWith("/environments/valid-env-1");
-  });
-
   test("should update redirect when environment ID prop changes", () => {
     const mockPush = vi.fn();
     vi.mocked(useRouter).mockReturnValue({ push: mockPush } as any);
@@ -79,20 +56,19 @@ describe("ClientEnvironmentRedirect", () => {
     // Mock localStorage
     const localStorageMock = {
       getItem: vi.fn().mockReturnValue(null),
-      removeItem: vi.fn(),
     };
     Object.defineProperty(window, "localStorage", {
       value: localStorageMock,
     });
 
-    const { rerender } = render(<ClientEnvironmentRedirect userEnvironments={["initial-env-id"]} />);
+    const { rerender } = render(<ClientEnvironmentRedirect environmentId="initial-env-id" />);
     expect(mockPush).toHaveBeenCalledWith("/environments/initial-env-id");
 
     // Clear mock calls
     mockPush.mockClear();
 
     // Rerender with new environment ID
-    rerender(<ClientEnvironmentRedirect userEnvironments={["new-env-id"]} />);
+    rerender(<ClientEnvironmentRedirect environmentId="new-env-id" />);
     expect(mockPush).toHaveBeenCalledWith("/environments/new-env-id");
   });
 });

apps/web/app/ClientEnvironmentRedirect.tsx

@@ -5,23 +5,22 @@ import { useRouter } from "next/navigation";
 import { useEffect } from "react";
 
 interface ClientEnvironmentRedirectProps {
-  userEnvironments: string[];
+  environmentId: string;
 }
 
-const ClientEnvironmentRedirect = ({ userEnvironments }: ClientEnvironmentRedirectProps) => {
+const ClientEnvironmentRedirect = ({ environmentId }: ClientEnvironmentRedirectProps) => {
   const router = useRouter();
 
   useEffect(() => {
     const lastEnvironmentId = localStorage.getItem(FORMBRICKS_ENVIRONMENT_ID_LS);
 
-    if (lastEnvironmentId && userEnvironments.includes(lastEnvironmentId)) {
+    if (lastEnvironmentId) {
+      // Redirect to the last environment the user was in
       router.push(`/environments/${lastEnvironmentId}`);
     } else {
-      // If the last environmentId is not valid, remove it from localStorage and redirect to the provided environmentId
-      localStorage.removeItem(FORMBRICKS_ENVIRONMENT_ID_LS);
-      router.push(`/environments/${userEnvironments[0]}`);
+      router.push(`/environments/${environmentId}`);
     }
-  }, [userEnvironments, router]);
+  }, [environmentId, router]);
 
   return null;
 };

apps/web/app/api/v1/client/[environmentId]/environment/lib/environmentState.test.ts

@@ -274,7 +274,7 @@ describe("getEnvironmentState", () => {
 
     expect(withCache).toHaveBeenCalledWith(expect.any(Function), {
       key: `fb:env:${environmentId}:state`,
-      ttl: 5 * 60 * 1000, // 5 minutes in milliseconds
+      ttl: 60 * 30 * 1000, // 30 minutes in milliseconds
     });
   });
 });

apps/web/app/api/v1/client/[environmentId]/environment/lib/environmentState.ts

@@ -83,8 +83,9 @@ export const getEnvironmentState = async (
     {
       // Use enterprise-grade cache key pattern
       key: createCacheKey.environment.state(environmentId),
-      // This is a temporary fix for the invalidation issues, will be changed later with a proper solution
-      ttl: 5 * 60 * 1000, // 5 minutes in milliseconds
+      // 30 minutes TTL ensures fresh data for hourly SDK checks
+      // Balances performance with freshness requirements
+      ttl: 60 * 30 * 1000, // 30 minutes in milliseconds
     }
   );
 

apps/web/app/api/v1/management/action-classes/route.ts

@@ -52,6 +52,14 @@ export const POST = withApiLogging(
       }
 
       const inputValidation = ZActionClassInput.safeParse(actionClassInput);
+      const environmentId = actionClassInput.environmentId;
+
+      if (!hasPermission(authentication.environmentPermissions, environmentId, "POST")) {
+        return {
+          response: responses.unauthorizedResponse(),
+        };
+      }
+
       if (!inputValidation.success) {
         return {
           response: responses.badRequestResponse(
@@ -62,14 +70,6 @@ export const POST = withApiLogging(
         };
       }
 
-      const environmentId = inputValidation.data.environmentId;
-
-      if (!hasPermission(authentication.environmentPermissions, environmentId, "POST")) {
-        return {
-          response: responses.unauthorizedResponse(),
-        };
-      }
-
       const actionClass: TActionClass = await createActionClass(environmentId, inputValidation.data);
       auditLog.targetId = actionClass.id;
       auditLog.newObject = actionClass;

apps/web/app/api/v1/management/responses/lib/response.test.ts

@@ -186,18 +186,6 @@ describe("Response Lib Tests", () => {
       expect(logger.error).not.toHaveBeenCalled(); // Should be caught and re-thrown as DatabaseError
     });
 
-    test("should handle RelatedRecordDoesNotExist error with specific message", async () => {
-      const prismaError = new Prisma.PrismaClientKnownRequestError("Related record does not exist", {
-        code: "P2025", // PrismaErrorType.RelatedRecordDoesNotExist
-        clientVersion: "2.0",
-      });
-      vi.mocked(getOrganizationByEnvironmentId).mockResolvedValue(mockOrganization);
-      vi.mocked(prisma.response.create).mockRejectedValue(prismaError);
-
-      await expect(createResponse(mockResponseInput)).rejects.toThrow(DatabaseError);
-      await expect(createResponse(mockResponseInput)).rejects.toThrow("Display ID does not exist");
-    });
-
     test("should handle generic errors", async () => {
       const genericError = new Error("Something went wrong");
       vi.mocked(getOrganizationByEnvironmentId).mockResolvedValue(mockOrganization);

apps/web/app/api/v1/management/responses/lib/response.ts

@@ -12,7 +12,6 @@ import { validateInputs } from "@/lib/utils/validate";
 import { Prisma } from "@prisma/client";
 import { cache as reactCache } from "react";
 import { prisma } from "@formbricks/database";
-import { PrismaErrorType } from "@formbricks/database/types/error";
 import { logger } from "@formbricks/logger";
 import { ZId, ZOptionalNumber } from "@formbricks/types/common";
 import { TContactAttributes } from "@formbricks/types/contact-attribute";
@@ -177,9 +176,6 @@ export const createResponse = async (responseInput: TResponseInput): Promise<TRe
     return response;
   } catch (error) {
     if (error instanceof Prisma.PrismaClientKnownRequestError) {
-      if (error.code === PrismaErrorType.RelatedRecordDoesNotExist) {
-        throw new DatabaseError("Display ID does not exist");
-      }
       throw new DatabaseError(error.message);
     }
 

apps/web/app/api/v1/management/responses/route.ts

@@ -149,10 +149,6 @@ export const POST = withApiLogging(
           return {
             response: responses.badRequestResponse(error.message),
           };
-        } else if (error instanceof DatabaseError) {
-          return {
-            response: responses.badRequestResponse(error.message),
-          };
         }
         logger.error({ error, url: request.url }, "Error in POST /api/v1/management/responses");
         return {
@@ -162,7 +158,7 @@ export const POST = withApiLogging(
     } catch (error) {
       if (error instanceof DatabaseError) {
         return {
-          response: responses.badRequestResponse("An unexpected error occurred while creating the response"),
+          response: responses.badRequestResponse(error.message),
         };
       }
       throw error;

apps/web/app/api/v1/management/storage/lib/utils.test.ts

@@ -1,13 +1,13 @@
-import { authenticateRequest } from "@/app/api/v1/auth";
+import { checkForRequiredFields } from "./utils";
+import { describe, test, expect } from "vitest";
 import { responses } from "@/app/lib/api/response";
 import { hasUserEnvironmentAccess } from "@/lib/environment/auth";
 import { hasPermission } from "@/modules/organization/settings/api-keys/lib/utils";
-import { Session } from "next-auth";
 import { NextRequest } from "next/server";
-import { describe, expect, test } from "vitest";
+import { Session } from "next-auth";
 import { vi } from "vitest";
 import { TAuthenticationApiKey } from "@formbricks/types/auth";
-import { checkForRequiredFields } from "./utils";
+import { authenticateRequest } from "@/app/api/v1/auth";
 import { checkAuth } from "./utils";
 
 // Create mock response objects
@@ -16,197 +16,189 @@ const mockNotAuthenticatedResponse = new Response("Not authenticated", { status:
 const mockUnauthorizedResponse = new Response("Unauthorized", { status: 401 });
 
 vi.mock("@/app/api/v1/auth", () => ({
-  authenticateRequest: vi.fn(),
+    authenticateRequest: vi.fn(),
 }));
 
 vi.mock("@/lib/environment/auth", () => ({
-  hasUserEnvironmentAccess: vi.fn(),
+    hasUserEnvironmentAccess: vi.fn(),
 }));
 
 vi.mock("@/modules/organization/settings/api-keys/lib/utils", () => ({
-  hasPermission: vi.fn(),
+    hasPermission: vi.fn(),
 }));
 
 vi.mock("@/app/lib/api/response", () => ({
-  responses: {
-    badRequestResponse: vi.fn(() => mockBadRequestResponse),
-    notAuthenticatedResponse: vi.fn(() => mockNotAuthenticatedResponse),
-    unauthorizedResponse: vi.fn(() => mockUnauthorizedResponse),
-  },
+    responses: {
+        badRequestResponse: vi.fn(() => mockBadRequestResponse),
+        notAuthenticatedResponse: vi.fn(() => mockNotAuthenticatedResponse),
+        unauthorizedResponse: vi.fn(() => mockUnauthorizedResponse),
+    },
 }));
 
 describe("checkForRequiredFields", () => {
-  test("should return undefined when all required fields are present", () => {
-    const result = checkForRequiredFields("env-123", "image/png", "test-file.png");
-    expect(result).toBeUndefined();
-  });
+    test("should return undefined when all required fields are present", () => {
+        const result = checkForRequiredFields("env-123", "image/png", "test-file.png");
+        expect(result).toBeUndefined();
+    });
 
-  test("should return bad request response when environmentId is missing", () => {
-    const result = checkForRequiredFields("", "image/png", "test-file.png");
-    expect(responses.badRequestResponse).toHaveBeenCalledWith("environmentId is required");
-    expect(result).toBe(mockBadRequestResponse);
-  });
+    test("should return bad request response when environmentId is missing", () => {
+        const result = checkForRequiredFields("", "image/png", "test-file.png");
+        expect(responses.badRequestResponse).toHaveBeenCalledWith("environmentId is required");
+        expect(result).toBe(mockBadRequestResponse);
+    });
 
-  test("should return bad request response when fileType is missing", () => {
-    const result = checkForRequiredFields("env-123", "", "test-file.png");
-    expect(responses.badRequestResponse).toHaveBeenCalledWith("contentType is required");
-    expect(result).toBe(mockBadRequestResponse);
-  });
+    test("should return bad request response when fileType is missing", () => {
+        const result = checkForRequiredFields("env-123", "", "test-file.png");
+        expect(responses.badRequestResponse).toHaveBeenCalledWith("contentType is required");
+        expect(result).toBe(mockBadRequestResponse);
+    });
 
-  test("should return bad request response when encodedFileName is missing", () => {
-    const result = checkForRequiredFields("env-123", "image/png", "");
-    expect(responses.badRequestResponse).toHaveBeenCalledWith("fileName is required");
-    expect(result).toBe(mockBadRequestResponse);
-  });
+    test("should return bad request response when encodedFileName is missing", () => {
+        const result = checkForRequiredFields("env-123", "image/png", "");
+        expect(responses.badRequestResponse).toHaveBeenCalledWith("fileName is required");
+        expect(result).toBe(mockBadRequestResponse);
+    });
 
-  test("should return bad request response when environmentId is undefined", () => {
-    const result = checkForRequiredFields(undefined as any, "image/png", "test-file.png");
-    expect(responses.badRequestResponse).toHaveBeenCalledWith("environmentId is required");
-    expect(result).toBe(mockBadRequestResponse);
-  });
+    test("should return bad request response when environmentId is undefined", () => {
+        const result = checkForRequiredFields(undefined as any, "image/png", "test-file.png");
+        expect(responses.badRequestResponse).toHaveBeenCalledWith("environmentId is required");
+        expect(result).toBe(mockBadRequestResponse);
+    });
 
-  test("should return bad request response when fileType is undefined", () => {
-    const result = checkForRequiredFields("env-123", undefined as any, "test-file.png");
-    expect(responses.badRequestResponse).toHaveBeenCalledWith("contentType is required");
-    expect(result).toBe(mockBadRequestResponse);
-  });
+    test("should return bad request response when fileType is undefined", () => {
+        const result = checkForRequiredFields("env-123", undefined as any, "test-file.png");
+        expect(responses.badRequestResponse).toHaveBeenCalledWith("contentType is required");
+        expect(result).toBe(mockBadRequestResponse);
+    });
 
-  test("should return bad request response when encodedFileName is undefined", () => {
-    const result = checkForRequiredFields("env-123", "image/png", undefined as any);
-    expect(responses.badRequestResponse).toHaveBeenCalledWith("fileName is required");
-    expect(result).toBe(mockBadRequestResponse);
-  });
+    test("should return bad request response when encodedFileName is undefined", () => {
+        const result = checkForRequiredFields("env-123", "image/png", undefined as any);
+        expect(responses.badRequestResponse).toHaveBeenCalledWith("fileName is required");
+        expect(result).toBe(mockBadRequestResponse);
+    });
 });
 
 describe("checkAuth", () => {
-  const environmentId = "env-123";
-  const mockRequest = new NextRequest("http://localhost:3000/api/test");
+    const environmentId = "env-123";
+    const mockRequest = new NextRequest("http://localhost:3000/api/test");
 
-  test("returns notAuthenticatedResponse when no session and no authentication", async () => {
-    vi.mocked(authenticateRequest).mockResolvedValue(null);
+    test("returns notAuthenticatedResponse when no session and no authentication", async () => {
+        vi.mocked(authenticateRequest).mockResolvedValue(null);
 
-    const result = await checkAuth(null, environmentId, mockRequest);
+        const result = await checkAuth(null, environmentId, mockRequest);
 
-    expect(authenticateRequest).toHaveBeenCalledWith(mockRequest);
-    expect(responses.notAuthenticatedResponse).toHaveBeenCalled();
-    expect(result).toBe(mockNotAuthenticatedResponse);
-  });
+        expect(authenticateRequest).toHaveBeenCalledWith(mockRequest);
+        expect(responses.notAuthenticatedResponse).toHaveBeenCalled();
+        expect(result).toBe(mockNotAuthenticatedResponse);
+    });
 
-  test("returns unauthorizedResponse when no session and authentication lacks POST permission", async () => {
-    const mockAuthentication: TAuthenticationApiKey = {
-      type: "apiKey",
-      environmentPermissions: [
-        {
-          environmentId: "env-123",
-          permission: "read",
-          environmentType: "development",
-          projectId: "project-1",
-          projectName: "Project 1",
-        },
-      ],
-      hashedApiKey: "hashed-key",
-      apiKeyId: "api-key-id",
-      organizationId: "org-id",
-      organizationAccess: {
-        accessControl: {},
-      },
-    };
+    test("returns unauthorizedResponse when no session and authentication lacks POST permission", async () => {
+        const mockAuthentication: TAuthenticationApiKey = {
+            type: "apiKey",
+            environmentPermissions: [
+                {
+                    environmentId: "env-123",
+                    permission: "read",
+                    environmentType: "development",
+                    projectId: "project-1",
+                    projectName: "Project 1",
+                },
+            ],
+            hashedApiKey: "hashed-key",
+            apiKeyId: "api-key-id",
+            organizationId: "org-id",
+            organizationAccess: {
+                accessControl: {},
+            },
+        };
 
-    vi.mocked(authenticateRequest).mockResolvedValue(mockAuthentication);
-    vi.mocked(hasPermission).mockReturnValue(false);
+        vi.mocked(authenticateRequest).mockResolvedValue(mockAuthentication);
+        vi.mocked(hasPermission).mockReturnValue(false);
 
-    const result = await checkAuth(null, environmentId, mockRequest);
+        const result = await checkAuth(null, environmentId, mockRequest);
 
-    expect(authenticateRequest).toHaveBeenCalledWith(mockRequest);
-    expect(hasPermission).toHaveBeenCalledWith(
-      mockAuthentication.environmentPermissions,
-      environmentId,
-      "POST"
-    );
-    expect(responses.unauthorizedResponse).toHaveBeenCalled();
-    expect(result).toBe(mockUnauthorizedResponse);
-  });
+        expect(authenticateRequest).toHaveBeenCalledWith(mockRequest);
+        expect(hasPermission).toHaveBeenCalledWith(mockAuthentication.environmentPermissions, environmentId, "POST");
+        expect(responses.unauthorizedResponse).toHaveBeenCalled();
+        expect(result).toBe(mockUnauthorizedResponse);
+    });
 
-  test("returns undefined when no session and authentication has POST permission", async () => {
-    const mockAuthentication: TAuthenticationApiKey = {
-      type: "apiKey",
-      environmentPermissions: [
-        {
-          environmentId: "env-123",
-          permission: "write",
-          environmentType: "development",
-          projectId: "project-1",
-          projectName: "Project 1",
-        },
-      ],
-      hashedApiKey: "hashed-key",
-      apiKeyId: "api-key-id",
-      organizationId: "org-id",
-      organizationAccess: {
-        accessControl: {},
-      },
-    };
+    test("returns undefined when no session and authentication has POST permission", async () => {
+        const mockAuthentication: TAuthenticationApiKey = {
+            type: "apiKey",
+            environmentPermissions: [
+                {
+                    environmentId: "env-123",
+                    permission: "write",
+                    environmentType: "development",
+                    projectId: "project-1",
+                    projectName: "Project 1",
+                },
+            ],
+            hashedApiKey: "hashed-key",
+            apiKeyId: "api-key-id",
+            organizationId: "org-id",
+            organizationAccess: {
+                accessControl: {},
+            },
+        };
 
-    vi.mocked(authenticateRequest).mockResolvedValue(mockAuthentication);
-    vi.mocked(hasPermission).mockReturnValue(true);
+        vi.mocked(authenticateRequest).mockResolvedValue(mockAuthentication);
+        vi.mocked(hasPermission).mockReturnValue(true);
 
-    const result = await checkAuth(null, environmentId, mockRequest);
+        const result = await checkAuth(null, environmentId, mockRequest);
 
-    expect(authenticateRequest).toHaveBeenCalledWith(mockRequest);
-    expect(hasPermission).toHaveBeenCalledWith(
-      mockAuthentication.environmentPermissions,
-      environmentId,
-      "POST"
-    );
-    expect(result).toBeUndefined();
-  });
+        expect(authenticateRequest).toHaveBeenCalledWith(mockRequest);
+        expect(hasPermission).toHaveBeenCalledWith(mockAuthentication.environmentPermissions, environmentId, "POST");
+        expect(result).toBeUndefined();
+    });
 
-  test("returns unauthorizedResponse when session exists but user lacks environment access", async () => {
-    const mockSession: Session = {
-      user: {
-        id: "user-123",
-      },
-      expires: "2024-12-31T23:59:59.999Z",
-    };
+    test("returns unauthorizedResponse when session exists but user lacks environment access", async () => {
+        const mockSession: Session = {
+            user: {
+                id: "user-123",
+            },
+            expires: "2024-12-31T23:59:59.999Z",
+        };
 
-    vi.mocked(hasUserEnvironmentAccess).mockResolvedValue(false);
+        vi.mocked(hasUserEnvironmentAccess).mockResolvedValue(false);
 
-    const result = await checkAuth(mockSession, environmentId, mockRequest);
+        const result = await checkAuth(mockSession, environmentId, mockRequest);
 
-    expect(hasUserEnvironmentAccess).toHaveBeenCalledWith("user-123", environmentId);
-    expect(responses.unauthorizedResponse).toHaveBeenCalled();
-    expect(result).toBe(mockUnauthorizedResponse);
-  });
+        expect(hasUserEnvironmentAccess).toHaveBeenCalledWith("user-123", environmentId);
+        expect(responses.unauthorizedResponse).toHaveBeenCalled();
+        expect(result).toBe(mockUnauthorizedResponse);
+    });
 
-  test("returns undefined when session exists and user has environment access", async () => {
-    const mockSession: Session = {
-      user: {
-        id: "user-123",
-      },
-      expires: "2024-12-31T23:59:59.999Z",
-    };
+    test("returns undefined when session exists and user has environment access", async () => {
+        const mockSession: Session = {
+            user: {
+                id: "user-123",
+            },
+            expires: "2024-12-31T23:59:59.999Z",
+        };
 
-    vi.mocked(hasUserEnvironmentAccess).mockResolvedValue(true);
+        vi.mocked(hasUserEnvironmentAccess).mockResolvedValue(true);
 
-    const result = await checkAuth(mockSession, environmentId, mockRequest);
+        const result = await checkAuth(mockSession, environmentId, mockRequest);
 
-    expect(hasUserEnvironmentAccess).toHaveBeenCalledWith("user-123", environmentId);
-    expect(result).toBeUndefined();
-  });
+        expect(hasUserEnvironmentAccess).toHaveBeenCalledWith("user-123", environmentId);
+        expect(result).toBeUndefined();
+    });
 
-  test("does not call authenticateRequest when session exists", async () => {
-    const mockSession: Session = {
-      user: {
-        id: "user-123",
-      },
-      expires: "2024-12-31T23:59:59.999Z",
-    };
+    test("does not call authenticateRequest when session exists", async () => {
+        const mockSession: Session = {
+            user: {
+                id: "user-123",
+            },
+            expires: "2024-12-31T23:59:59.999Z",
+        };
 
-    vi.mocked(hasUserEnvironmentAccess).mockResolvedValue(true);
+        vi.mocked(hasUserEnvironmentAccess).mockResolvedValue(true);
 
-    await checkAuth(mockSession, environmentId, mockRequest);
+        await checkAuth(mockSession, environmentId, mockRequest);
 
-    expect(authenticateRequest).not.toHaveBeenCalled();
-    expect(hasUserEnvironmentAccess).toHaveBeenCalledWith("user-123", environmentId);
-  });
-});
+        expect(authenticateRequest).not.toHaveBeenCalled();
+        expect(hasUserEnvironmentAccess).toHaveBeenCalledWith("user-123", environmentId);
+    });
+}); 

apps/web/app/api/v1/management/storage/lib/utils.ts

@@ -1,41 +1,38 @@
 import { authenticateRequest } from "@/app/api/v1/auth";
 import { responses } from "@/app/lib/api/response";
 import { hasUserEnvironmentAccess } from "@/lib/environment/auth";
-import { hasPermission } from "@/modules/organization/settings/api-keys/lib/utils";
-import { Session } from "next-auth";
 import { NextRequest } from "next/server";
+import { Session } from "next-auth";
+import { hasPermission } from "@/modules/organization/settings/api-keys/lib/utils";
 
-export const checkForRequiredFields = (
-  environmentId: string,
-  fileType: string,
-  encodedFileName: string
-): Response | undefined => {
-  if (!environmentId) {
-    return responses.badRequestResponse("environmentId is required");
-  }
 
-  if (!fileType) {
-    return responses.badRequestResponse("contentType is required");
-  }
+export const checkForRequiredFields = (environmentId: string, fileType: string, encodedFileName: string): Response | undefined => {
+    if (!environmentId) {
+        return responses.badRequestResponse("environmentId is required");
+    }
 
-  if (!encodedFileName) {
-    return responses.badRequestResponse("fileName is required");
-  }
+    if (!fileType) {
+        return responses.badRequestResponse("contentType is required");
+    }
+
+    if (!encodedFileName) {
+        return responses.badRequestResponse("fileName is required");
+    }
 };
 
 export const checkAuth = async (session: Session | null, environmentId: string, request: NextRequest) => {
-  if (!session) {
-    //check whether its using API key
-    const authentication = await authenticateRequest(request);
-    if (!authentication) return responses.notAuthenticatedResponse();
+    if (!session) {
+        //check whether its using API key
+        const authentication = await authenticateRequest(request);
+        if (!authentication) return responses.notAuthenticatedResponse();
 
-    if (!hasPermission(authentication.environmentPermissions, environmentId, "POST")) {
-      return responses.unauthorizedResponse();
+        if (!hasPermission(authentication.environmentPermissions, environmentId, "POST")) {
+            return responses.unauthorizedResponse();
+        }
+    } else {
+        const isUserAuthorized = await hasUserEnvironmentAccess(session.user.id, environmentId);
+        if (!isUserAuthorized) {
+            return responses.unauthorizedResponse();
+        }
     }
-  } else {
-    const isUserAuthorized = await hasUserEnvironmentAccess(session.user.id, environmentId);
-    if (!isUserAuthorized) {
-      return responses.unauthorizedResponse();
-    }
-  }
-};
+};

apps/web/app/api/v1/management/storage/local/route.ts

@@ -1,7 +1,6 @@
 // headers -> "Content-Type" should be present and set to a valid MIME type
 // body -> should be a valid file object (buffer)
 // method -> PUT (to be the same as the signedUrl method)
-import { checkAuth, checkForRequiredFields } from "@/app/api/v1/management/storage/lib/utils";
 import { responses } from "@/app/lib/api/response";
 import { ENCRYPTION_KEY, UPLOADS_DIR } from "@/lib/constants";
 import { validateLocalSignedUrl } from "@/lib/crypto";
@@ -11,6 +10,7 @@ import { authOptions } from "@/modules/auth/lib/authOptions";
 import { getServerSession } from "next-auth";
 import { NextRequest } from "next/server";
 import { logger } from "@formbricks/logger";
+import { checkAuth, checkForRequiredFields } from "@/app/api/v1/management/storage/lib/utils";
 
 export const POST = async (req: NextRequest): Promise<Response> => {
   if (!ENCRYPTION_KEY) {

apps/web/app/api/v1/management/storage/route.ts

@@ -1,4 +1,3 @@
-import { checkAuth, checkForRequiredFields } from "@/app/api/v1/management/storage/lib/utils";
 import { responses } from "@/app/lib/api/response";
 import { validateFile } from "@/lib/fileValidation";
 import { authOptions } from "@/modules/auth/lib/authOptions";
@@ -6,6 +5,8 @@ import { getServerSession } from "next-auth";
 import { NextRequest } from "next/server";
 import { logger } from "@formbricks/logger";
 import { getSignedUrlForPublicFile } from "./lib/getSignedUrl";
+import { checkAuth, checkForRequiredFields } from "@/app/api/v1/management/storage/lib/utils";
+
 
 // api endpoint for uploading public files
 // uploaded files will be public, anyone can access the file
@@ -13,6 +14,7 @@ import { getSignedUrlForPublicFile } from "./lib/getSignedUrl";
 // use this to upload files for a specific resource, e.g. a user profile picture or a survey
 // this api endpoint will return a signed url for uploading the file to s3 and another url for uploading file to the local storage
 
+
 export const POST = async (request: NextRequest): Promise<Response> => {
   let storageInput;
 
@@ -32,6 +34,7 @@ export const POST = async (request: NextRequest): Promise<Response> => {
   const authResponse = await checkAuth(session, environmentId, request);
   if (authResponse) return authResponse;
 
+
   // Perform server-side file validation first to block dangerous file types
   const fileValidation = validateFile(fileName, fileType);
   if (!fileValidation.valid) {

apps/web/app/lib/survey-builder.test.ts

@@ -41,7 +41,7 @@ describe("Survey Builder", () => {
         buttonLabel: { default: "common.next" },
         backButtonLabel: { default: "common.back" },
         shuffleOption: "none",
-        required: false,
+        required: true,
       });
       expect(question.choices.length).toBe(3);
       expect(question.id).toBeDefined();
@@ -141,7 +141,7 @@ describe("Survey Builder", () => {
         inputType: "text",
         buttonLabel: { default: "common.next" },
         backButtonLabel: { default: "common.back" },
-        required: false,
+        required: true,
         charLimit: {
           enabled: false,
         },
@@ -204,7 +204,7 @@ describe("Survey Builder", () => {
         range: 5,
         buttonLabel: { default: "common.next" },
         backButtonLabel: { default: "common.back" },
-        required: false,
+        required: true,
         isColorCodingEnabled: false,
       });
       expect(question.id).toBeDefined();
@@ -265,7 +265,7 @@ describe("Survey Builder", () => {
         headline: { default: "NPS Question" },
         buttonLabel: { default: "common.next" },
         backButtonLabel: { default: "common.back" },
-        required: false,
+        required: true,
         isColorCodingEnabled: false,
       });
       expect(question.id).toBeDefined();
@@ -324,7 +324,7 @@ describe("Survey Builder", () => {
         label: { default: "I agree to terms" },
         buttonLabel: { default: "common.next" },
         backButtonLabel: { default: "common.back" },
-        required: false,
+        required: true,
       });
       expect(question.id).toBeDefined();
     });
@@ -377,7 +377,7 @@ describe("Survey Builder", () => {
         headline: { default: "CTA Question" },
         buttonLabel: { default: "common.next" },
         backButtonLabel: { default: "common.back" },
-        required: false,
+        required: true,
         buttonExternal: false,
       });
       expect(question.id).toBeDefined();

apps/web/app/lib/survey-builder.ts

@@ -66,7 +66,7 @@ export const buildMultipleChoiceQuestion = ({
     buttonLabel: createI18nString(buttonLabel || t(defaultButtonLabel), []),
     backButtonLabel: createI18nString(backButtonLabel || t(defaultBackButtonLabel), []),
     shuffleOption: shuffleOption || "none",
-    required: required ?? false,
+    required: required ?? true,
     logic,
   };
 };
@@ -105,7 +105,7 @@ export const buildOpenTextQuestion = ({
     headline: createI18nString(headline, []),
     buttonLabel: createI18nString(buttonLabel || t(defaultButtonLabel), []),
     backButtonLabel: createI18nString(backButtonLabel || t(defaultBackButtonLabel), []),
-    required: required ?? false,
+    required: required ?? true,
     longAnswer,
     logic,
     charLimit: {
@@ -153,7 +153,7 @@ export const buildRatingQuestion = ({
     range,
     buttonLabel: createI18nString(buttonLabel || t(defaultButtonLabel), []),
     backButtonLabel: createI18nString(backButtonLabel || t(defaultBackButtonLabel), []),
-    required: required ?? false,
+    required: required ?? true,
     isColorCodingEnabled,
     lowerLabel: lowerLabel ? createI18nString(lowerLabel, []) : undefined,
     upperLabel: upperLabel ? createI18nString(upperLabel, []) : undefined,
@@ -194,7 +194,7 @@ export const buildNPSQuestion = ({
     headline: createI18nString(headline, []),
     buttonLabel: createI18nString(buttonLabel || t(defaultButtonLabel), []),
     backButtonLabel: createI18nString(backButtonLabel || t(defaultBackButtonLabel), []),
-    required: required ?? false,
+    required: required ?? true,
     isColorCodingEnabled,
     lowerLabel: lowerLabel ? createI18nString(lowerLabel, []) : undefined,
     upperLabel: upperLabel ? createI18nString(upperLabel, []) : undefined,
@@ -230,7 +230,7 @@ export const buildConsentQuestion = ({
     headline: createI18nString(headline, []),
     buttonLabel: createI18nString(buttonLabel || t(defaultButtonLabel), []),
     backButtonLabel: createI18nString(backButtonLabel || t(defaultBackButtonLabel), []),
-    required: required ?? false,
+    required: required ?? true,
     label: createI18nString(label, []),
     logic,
   };
@@ -269,7 +269,7 @@ export const buildCTAQuestion = ({
     buttonLabel: createI18nString(buttonLabel || t(defaultButtonLabel), []),
     backButtonLabel: createI18nString(backButtonLabel || t(defaultBackButtonLabel), []),
     dismissButtonLabel: dismissButtonLabel ? createI18nString(dismissButtonLabel, []) : undefined,
-    required: required ?? false,
+    required: required ?? true,
     buttonExternal,
     buttonUrl,
     logic,

apps/web/app/lib/templates.ts

@@ -3006,7 +3006,12 @@ const understandLowEngagement = (t: TFnType): TTemplate => {
             t("templates.understand_low_engagement_question_1_choice_4"),
             t("templates.understand_low_engagement_question_1_choice_5"),
           ],
-          choiceIds: [reusableOptionIds[0], reusableOptionIds[1], reusableOptionIds[2], reusableOptionIds[3]],
+          choiceIds: [
+            reusableOptionIds[0],
+            reusableOptionIds[1],
+            reusableOptionIds[2],
+            reusableOptionIds[3],
+          ],
           headline: t("templates.understand_low_engagement_question_1_headline"),
           required: true,
           containsOther: true,

apps/web/app/page.test.tsx

@@ -3,12 +3,12 @@ import { cleanup } from "@testing-library/react";
 import { beforeEach, describe, expect, test, vi } from "vitest";
 import { TMembership } from "@formbricks/types/memberships";
 import { TOrganization } from "@formbricks/types/organizations";
-import { TProject } from "@formbricks/types/project";
 import { TUser } from "@formbricks/types/user";
 import Page from "./page";
 
-vi.mock("@/lib/project/service", () => ({
-  getUserProjectEnvironmentsByOrganizationIds: vi.fn(),
+// Mock dependencies
+vi.mock("@/lib/environment/service", () => ({
+  getFirstEnvironmentIdByUserId: vi.fn(),
 }));
 
 vi.mock("@/lib/instance/service", () => ({
@@ -48,11 +48,8 @@ vi.mock("@/modules/ui/components/client-logout", () => ({
 }));
 
 vi.mock("@/app/ClientEnvironmentRedirect", () => ({
-  default: ({ environmentId, userEnvironments }: { environmentId: string; userEnvironments?: string[] }) => (
-    <div data-testid="client-environment-redirect">
-      Environment ID: {environmentId}
-      {userEnvironments && ` | User Environments: ${userEnvironments.join(", ")}`}
-    </div>
+  default: ({ environmentId }: { environmentId: string }) => (
+    <div data-testid="client-environment-redirect">Environment ID: {environmentId}</div>
   ),
 }));
 
@@ -152,7 +149,7 @@ describe("Page", () => {
     const { getIsFreshInstance } = await import("@/lib/instance/service");
     const { getUser } = await import("@/lib/user/service");
     const { getOrganizationsByUserId } = await import("@/lib/organization/service");
-    const { getUserProjectEnvironmentsByOrganizationIds } = await import("@/lib/project/service");
+    const { getFirstEnvironmentIdByUserId } = await import("@/lib/environment/service");
     const { getMembershipByUserIdOrganizationId } = await import("@/lib/membership/service");
     const { getAccessFlags } = await import("@/lib/membership/utils");
     const { redirect } = await import("next/navigation");
@@ -207,23 +204,13 @@ describe("Page", () => {
       role: "owner",
     };
 
-    const mockUserProjects = [
-      {
-        id: "test-project-id",
-        name: "Test Project",
-        environments: [],
-      },
-    ];
-
     vi.mocked(getServerSession).mockResolvedValue({
       user: { id: "test-user-id" },
     } as any);
     vi.mocked(getIsFreshInstance).mockResolvedValue(false);
     vi.mocked(getUser).mockResolvedValue(mockUser);
-    vi.mocked(getUserProjectEnvironmentsByOrganizationIds).mockResolvedValue(
-      mockUserProjects as unknown as TProject[]
-    );
     vi.mocked(getOrganizationsByUserId).mockResolvedValue([mockOrganization]);
+    vi.mocked(getFirstEnvironmentIdByUserId).mockResolvedValue(null);
     vi.mocked(getMembershipByUserIdOrganizationId).mockResolvedValue(mockMembership);
     vi.mocked(getAccessFlags).mockReturnValue({
       isManager: false,
@@ -241,8 +228,8 @@ describe("Page", () => {
     const { getServerSession } = await import("next-auth");
     const { getIsFreshInstance } = await import("@/lib/instance/service");
     const { getUser } = await import("@/lib/user/service");
-    const { getUserProjectEnvironmentsByOrganizationIds } = await import("@/lib/project/service");
     const { getOrganizationsByUserId } = await import("@/lib/organization/service");
+    const { getFirstEnvironmentIdByUserId } = await import("@/lib/environment/service");
     const { getMembershipByUserIdOrganizationId } = await import("@/lib/membership/service");
     const { getAccessFlags } = await import("@/lib/membership/utils");
     const { redirect } = await import("next/navigation");
@@ -297,23 +284,13 @@ describe("Page", () => {
       role: "member",
     };
 
-    const mockUserProjects = [
-      {
-        id: "test-project-id",
-        name: "Test Project",
-        environments: [],
-      },
-    ];
-
     vi.mocked(getServerSession).mockResolvedValue({
       user: { id: "test-user-id" },
     } as any);
     vi.mocked(getIsFreshInstance).mockResolvedValue(false);
     vi.mocked(getUser).mockResolvedValue(mockUser);
-    vi.mocked(getUserProjectEnvironmentsByOrganizationIds).mockResolvedValue(
-      mockUserProjects as unknown as TProject[]
-    );
     vi.mocked(getOrganizationsByUserId).mockResolvedValue([mockOrganization]);
+    vi.mocked(getFirstEnvironmentIdByUserId).mockResolvedValue(null);
     vi.mocked(getMembershipByUserIdOrganizationId).mockResolvedValue(mockMembership);
     vi.mocked(getAccessFlags).mockReturnValue({
       isManager: false,
@@ -332,9 +309,9 @@ describe("Page", () => {
     const { getIsFreshInstance } = await import("@/lib/instance/service");
     const { getUser } = await import("@/lib/user/service");
     const { getOrganizationsByUserId } = await import("@/lib/organization/service");
+    const { getFirstEnvironmentIdByUserId } = await import("@/lib/environment/service");
     const { getMembershipByUserIdOrganizationId } = await import("@/lib/membership/service");
     const { getAccessFlags } = await import("@/lib/membership/utils");
-    const { getUserProjectEnvironmentsByOrganizationIds } = await import("@/lib/project/service");
     const { render } = await import("@testing-library/react");
 
     const mockUser: TUser = {
@@ -387,43 +364,7 @@ describe("Page", () => {
       role: "member",
     };
 
-    const mockUserProjects = [
-      {
-        id: "project-1",
-        name: "Test Project",
-        createdAt: new Date(),
-        updatedAt: new Date(),
-        organizationId: "test-org-id",
-        styling: { allowStyleOverwrite: true },
-        recontactDays: 0,
-        inAppSurveyBranding: false,
-        linkSurveyBranding: false,
-        config: { channel: "link" as const, industry: "saas" as const },
-        placement: "bottomRight" as const,
-        clickOutsideClose: false,
-        darkOverlay: false,
-        languages: [],
-        logo: null,
-        environments: [
-          {
-            id: "test-env-id",
-            type: "production" as const,
-            createdAt: new Date(),
-            updatedAt: new Date(),
-            projectId: "project-1",
-            appSetupCompleted: true,
-          },
-          {
-            id: "test-env-dev",
-            type: "development" as const,
-            createdAt: new Date(),
-            updatedAt: new Date(),
-            projectId: "project-1",
-            appSetupCompleted: true,
-          },
-        ],
-      },
-    ] as any;
+    const mockEnvironmentId = "test-env-id";
 
     vi.mocked(getServerSession).mockResolvedValue({
       user: { id: "test-user-id" },
@@ -431,8 +372,8 @@ describe("Page", () => {
     vi.mocked(getIsFreshInstance).mockResolvedValue(false);
     vi.mocked(getUser).mockResolvedValue(mockUser);
     vi.mocked(getOrganizationsByUserId).mockResolvedValue([mockOrganization]);
+    vi.mocked(getFirstEnvironmentIdByUserId).mockResolvedValue(mockEnvironmentId);
     vi.mocked(getMembershipByUserIdOrganizationId).mockResolvedValue(mockMembership);
-    vi.mocked(getUserProjectEnvironmentsByOrganizationIds).mockResolvedValue(mockUserProjects);
     vi.mocked(getAccessFlags).mockReturnValue({
       isManager: false,
       isOwner: false,
@@ -444,7 +385,7 @@ describe("Page", () => {
     const { container } = render(result);
 
     expect(container.querySelector('[data-testid="client-environment-redirect"]')).toHaveTextContent(
-      `User Environments: test-env-id, test-env-dev`
+      `Environment ID: ${mockEnvironmentId}`
     );
   });
 });

apps/web/app/page.tsx

@@ -1,9 +1,9 @@
 import ClientEnvironmentRedirect from "@/app/ClientEnvironmentRedirect";
+import { getFirstEnvironmentIdByUserId } from "@/lib/environment/service";
 import { getIsFreshInstance } from "@/lib/instance/service";
 import { getMembershipByUserIdOrganizationId } from "@/lib/membership/service";
 import { getAccessFlags } from "@/lib/membership/utils";
 import { getOrganizationsByUserId } from "@/lib/organization/service";
-import { getUserProjectEnvironmentsByOrganizationIds } from "@/lib/project/service";
 import { getUser } from "@/lib/user/service";
 import { authOptions } from "@/modules/auth/lib/authOptions";
 import { ClientLogout } from "@/modules/ui/components/client-logout";
@@ -34,37 +34,16 @@ const Page = async () => {
     return redirect("/setup/organization/create");
   }
 
-  const projectsByOrg = await getUserProjectEnvironmentsByOrganizationIds(
-    userOrganizations.map((org) => org.id),
-    user.id
-  );
-
-  // Flatten all environments from all projects across all organizations
-  const allEnvironments = projectsByOrg.flatMap((project) => project.environments);
-
-  // Find first production environment and collect all other environment IDs in one pass
-  const { firstProductionEnvironmentId, otherEnvironmentIds } = allEnvironments.reduce(
-    (acc, env) => {
-      if (env.type === "production" && !acc.firstProductionEnvironmentId) {
-        acc.firstProductionEnvironmentId = env.id;
-      } else {
-        acc.otherEnvironmentIds.add(env.id);
-      }
-      return acc;
-    },
-    { firstProductionEnvironmentId: null as string | null, otherEnvironmentIds: new Set<string>() }
-  );
-
-  const userEnvironments = [...otherEnvironmentIds];
+  let environmentId: string | null = null;
+  environmentId = await getFirstEnvironmentIdByUserId(session.user.id);
 
   const currentUserMembership = await getMembershipByUserIdOrganizationId(
     session.user.id,
     userOrganizations[0].id
   );
-
   const { isManager, isOwner } = getAccessFlags(currentUserMembership?.role);
 
-  if (!firstProductionEnvironmentId) {
+  if (!environmentId) {
     if (isOwner || isManager) {
       return redirect(`/organizations/${userOrganizations[0].id}/projects/new/mode`);
     } else {
@@ -72,10 +51,7 @@ const Page = async () => {
     }
   }
 
-  // Put the first production environment at the front of the array
-  const sortedUserEnvironments = [firstProductionEnvironmentId, ...userEnvironments];
-
-  return <ClientEnvironmentRedirect userEnvironments={sortedUserEnvironments} />;
+  return <ClientEnvironmentRedirect environmentId={environmentId} />;
 };
 
 export default Page;

apps/web/app/s/[surveyId]/not-found.tsx

@@ -1,5 +1,3 @@
 import { LinkSurveyNotFound } from "@/modules/survey/link/not-found";
 
-export default function NotFound() {
-  return <LinkSurveyNotFound />;
-}
+export default LinkSurveyNotFound;

apps/web/lib/airtable/service.ts

@@ -13,8 +13,7 @@ import {
   ZIntegrationAirtableTokenSchema,
 } from "@formbricks/types/integration/airtable";
 import { AIRTABLE_CLIENT_ID, AIRTABLE_MESSAGE_LIMIT } from "../constants";
-import { createOrUpdateIntegration, getIntegrationByType } from "../integration/service";
-import { delay } from "../utils/promises";
+import { createOrUpdateIntegration, deleteIntegration, getIntegrationByType } from "../integration/service";
 import { truncateText } from "../utils/strings";
 
 export const getBases = async (key: string) => {
@@ -100,11 +99,7 @@ export const getAirtableToken = async (environmentId: string) => {
       });
 
       if (!newToken) {
-        logger.error("Failed to fetch new Airtable token", {
-          environmentId,
-          airtableIntegration,
-        });
-        throw new Error("Failed to fetch new Airtable token");
+        throw new Error("Failed to create new token");
       }
 
       await createOrUpdateIntegration(environmentId, {
@@ -121,11 +116,9 @@ export const getAirtableToken = async (environmentId: string) => {
 
     return access_token;
   } catch (error) {
-    logger.error("Failed to get Airtable token", {
-      environmentId,
-      error,
-    });
-    throw new Error("Failed to get Airtable token");
+    await deleteIntegration(environmentId);
+
+    throw new Error("invalid token");
   }
 };
 
@@ -185,18 +178,6 @@ const addField = async (
   return await req.json();
 };
 
-const getExistingFields = async (key: TIntegrationAirtableCredential, baseId: string, tableId: string) => {
-  const req = await tableFetcher(key, baseId);
-  const tables = ZIntegrationAirtableTablesWithFields.parse(req).tables;
-  const currentTable = tables.find((t) => t.id === tableId);
-
-  if (!currentTable) {
-    throw new Error(`Table with ID ${tableId} not found`);
-  }
-
-  return new Set(currentTable.fields.map((f) => f.name));
-};
-
 export const writeData = async (
   key: TIntegrationAirtableCredential,
   configData: TIntegrationAirtableConfigData,
@@ -205,7 +186,6 @@ export const writeData = async (
   const responses = values[0];
   const questions = values[1];
 
-  // 1) Build the record payload
   const data: Record<string, string> = {};
   for (let i = 0; i < questions.length; i++) {
     data[questions[i]] =
@@ -214,73 +194,34 @@ export const writeData = async (
         : responses[i];
   }
 
-  // 2) Figure out which fields need creating
-  const existingFields = await getExistingFields(key, configData.baseId, configData.tableId);
-  const fieldsToCreate = questions.filter((q) => !existingFields.has(q));
+  const req = await tableFetcher(key, configData.baseId);
+  const tables = ZIntegrationAirtableTablesWithFields.parse(req).tables;
 
-  // 3) Create any missing fields with throttling to respect Airtable's 5 req/sec per base limit
-  if (fieldsToCreate.length > 0) {
-    // Sequential processing with delays
-    const DELAY_BETWEEN_REQUESTS = 250; // 250ms = 4 requests per second (staying under 5/sec limit)
+  const currentTable = tables.find((table) => table.id === configData.tableId);
+  if (currentTable) {
+    const currentFields = new Set(currentTable.fields.map((field) => field.name));
+    const fieldsToCreate = new Set<string>();
+    for (const field of questions) {
+      const hasField = currentFields.has(field);
+      if (!hasField) {
+        fieldsToCreate.add(field);
+      }
+    }
 
-    for (let i = 0; i < fieldsToCreate.length; i++) {
-      const fieldName = fieldsToCreate[i];
-
-      const createRes = await addField(key, configData.baseId, configData.tableId, {
-        name: fieldName,
-        type: "singleLineText",
+    if (fieldsToCreate.size > 0) {
+      const createFieldPromise: Promise<any>[] = [];
+      fieldsToCreate.forEach((fieldName) => {
+        createFieldPromise.push(
+          addField(key, configData.baseId, configData.tableId, {
+            name: fieldName,
+            type: "singleLineText",
+          })
+        );
       });
 
-      if (createRes?.error) {
-        throw new Error(`Failed to create field "${fieldName}": ${JSON.stringify(createRes)}`);
-      }
-
-      // Add delay between requests (except for the last one)
-      if (i < fieldsToCreate.length - 1) {
-        await delay(DELAY_BETWEEN_REQUESTS);
-      }
+      await Promise.all(createFieldPromise);
     }
-
-    // 4) Wait for the new fields to show up
-    await waitForFieldsToExist(key, configData, fieldsToCreate);
   }
 
-  // 5) Finally, add the records
   await addRecords(key, configData.baseId, configData.tableId, data);
 };
-
-async function waitForFieldsToExist(
-  key: TIntegrationAirtableCredential,
-  configData: TIntegrationAirtableConfigData,
-  fieldNames: string[],
-  maxRetries = 5,
-  intervalMs = 2000
-) {
-  let existingFields: Set<string> = new Set(),
-    missingFields: string[] = [];
-
-  for (let attempt = 1; attempt <= maxRetries; attempt++) {
-    existingFields = await getExistingFields(key, configData.baseId, configData.tableId);
-    missingFields = fieldNames.filter((f) => !existingFields.has(f));
-
-    if (missingFields.length === 0) {
-      return;
-    }
-
-    if (attempt < maxRetries) {
-      logger.error(
-        `Attempt ${attempt}/${maxRetries}: ${missingFields.length} field(s) still missing [${missingFields.join(
-          ", "
-        )}], retrying in ${intervalMs / 1000}s…`
-      );
-
-      await new Promise((r) => setTimeout(r, intervalMs));
-    }
-  }
-
-  throw new Error(
-    `Timed out waiting for ${missingFields.length} field(s) [${missingFields.join(
-      ", "
-    )}] to become available. Available fields: [${Array.from(existingFields).join(", ")}]`
-  );
-}

apps/web/lib/fileValidation.ts

@@ -65,8 +65,7 @@ export const validateSingleFile = (
   return !allowedFileExtensions || allowedFileExtensions.includes(extension as TAllowedFileExtension);
 };
 
-export const validateFileUploads = (data?: TResponseData, questions?: TSurveyQuestion[]): boolean => {
-  if (!data) return true;
+export const validateFileUploads = (data: TResponseData, questions?: TSurveyQuestion[]): boolean => {
   for (const key of Object.keys(data)) {
     const question = questions?.find((q) => q.id === key);
     if (!question || question.type !== TSurveyQuestionTypeEnum.FileUpload) continue;

apps/web/lib/organization/service.test.ts

@@ -1,16 +1,9 @@
 import { BILLING_LIMITS, PROJECT_FEATURE_KEYS } from "@/lib/constants";
-import { updateUser } from "@/lib/user/service";
 import { Prisma } from "@prisma/client";
 import { afterEach, describe, expect, test, vi } from "vitest";
 import { prisma } from "@formbricks/database";
 import { DatabaseError } from "@formbricks/types/errors";
-import {
-  createOrganization,
-  getOrganization,
-  getOrganizationsByUserId,
-  subscribeOrganizationMembersToSurveyResponses,
-  updateOrganization,
-} from "./service";
+import { createOrganization, getOrganization, getOrganizationsByUserId, updateOrganization } from "./service";
 
 vi.mock("@formbricks/database", () => ({
   prisma: {
@@ -20,16 +13,9 @@ vi.mock("@formbricks/database", () => ({
       create: vi.fn(),
       update: vi.fn(),
     },
-    user: {
-      findUnique: vi.fn(),
-    },
   },
 }));
 
-vi.mock("@/lib/user/service", () => ({
-  updateUser: vi.fn(),
-}));
-
 describe("Organization Service", () => {
   afterEach(() => {
     vi.clearAllMocks();
@@ -266,62 +252,4 @@ describe("Organization Service", () => {
       });
     });
   });
-
-  describe("subscribeOrganizationMembersToSurveyResponses", () => {
-    test("should subscribe user to survey responses when not unsubscribed", async () => {
-      const mockUser = {
-        id: "user-123",
-        notificationSettings: {
-          alert: { "existing-survey-id": true },
-          weeklySummary: {},
-          unsubscribedOrganizationIds: [], // User is subscribed to all organizations
-        },
-      } as any;
-
-      const surveyId = "survey-123";
-      const userId = "user-123";
-      const organizationId = "org-123";
-
-      vi.mocked(prisma.user.findUnique).mockResolvedValueOnce(mockUser);
-      vi.mocked(updateUser).mockResolvedValueOnce({} as any);
-
-      await subscribeOrganizationMembersToSurveyResponses(surveyId, userId, organizationId);
-
-      expect(prisma.user.findUnique).toHaveBeenCalledWith({
-        where: { id: userId },
-      });
-      expect(updateUser).toHaveBeenCalledWith(userId, {
-        notificationSettings: {
-          alert: {
-            "existing-survey-id": true,
-            "survey-123": true,
-          },
-          weeklySummary: {},
-          unsubscribedOrganizationIds: [],
-        },
-      });
-    });
-
-    test("should not subscribe user when unsubscribed from organization", async () => {
-      const mockUser = {
-        id: "user-123",
-        notificationSettings: {
-          alert: { "existing-survey-id": true },
-          weeklySummary: {},
-          unsubscribedOrganizationIds: ["org-123"], // User has unsubscribed from this organization
-        },
-      } as any;
-
-      const surveyId = "survey-123";
-      const userId = "user-123";
-      const organizationId = "org-123";
-
-      vi.mocked(prisma.user.findUnique).mockResolvedValueOnce(mockUser);
-
-      await subscribeOrganizationMembersToSurveyResponses(surveyId, userId, organizationId);
-
-      // Should not call updateUser because user is unsubscribed from this organization
-      expect(updateUser).not.toHaveBeenCalled();
-    });
-  });
 });

apps/web/lib/project/service.test.ts

@@ -1,16 +1,10 @@
 import { createId } from "@paralleldrive/cuid2";
-import { OrganizationRole, Prisma, WidgetPlacement } from "@prisma/client";
+import { Prisma } from "@prisma/client";
 import { afterEach, describe, expect, test, vi } from "vitest";
 import { prisma } from "@formbricks/database";
 import { DatabaseError, ValidationError } from "@formbricks/types/errors";
 import { ITEMS_PER_PAGE } from "../constants";
-import {
-  getProject,
-  getProjectByEnvironmentId,
-  getProjects,
-  getUserProjectEnvironmentsByOrganizationIds,
-  getUserProjects,
-} from "./service";
+import { getProject, getProjectByEnvironmentId, getProjects, getUserProjects } from "./service";
 
 vi.mock("@formbricks/database", () => ({
   prisma: {
@@ -21,7 +15,6 @@ vi.mock("@formbricks/database", () => ({
     },
     membership: {
       findFirst: vi.fn(),
-      findMany: vi.fn(),
     },
   },
 }));
@@ -42,20 +35,13 @@ describe("Project Service", () => {
       recontactDays: 0,
       linkSurveyBranding: true,
       inAppSurveyBranding: true,
-      config: {
-        channel: null,
-        industry: null,
-      },
-      placement: WidgetPlacement.bottomRight,
+      config: {},
+      placement: "bottomRight",
       clickOutsideClose: true,
       darkOverlay: false,
       environments: [],
-      styling: {
-        allowStyleOverwrite: true,
-      },
+      styling: {},
       logo: null,
-      brandColor: null,
-      highlightBorderColor: null,
     };
 
     vi.mocked(prisma.project.findUnique).mockResolvedValue(mockProject);
@@ -100,20 +86,13 @@ describe("Project Service", () => {
       recontactDays: 0,
       linkSurveyBranding: true,
       inAppSurveyBranding: true,
-      config: {
-        channel: null,
-        industry: null,
-      },
-      placement: WidgetPlacement.bottomRight,
+      config: {},
+      placement: "bottomRight",
       clickOutsideClose: true,
       darkOverlay: false,
       environments: [],
-      styling: {
-        allowStyleOverwrite: true,
-      },
+      styling: {},
       logo: null,
-      brandColor: null,
-      highlightBorderColor: null,
     };
 
     vi.mocked(prisma.project.findFirst).mockResolvedValue(mockProject);
@@ -165,20 +144,13 @@ describe("Project Service", () => {
         recontactDays: 0,
         linkSurveyBranding: true,
         inAppSurveyBranding: true,
-        config: {
-          channel: null,
-          industry: null,
-        },
-        placement: WidgetPlacement.bottomRight,
+        config: {},
+        placement: "bottomRight",
         clickOutsideClose: true,
         darkOverlay: false,
         environments: [],
-        styling: {
-          allowStyleOverwrite: true,
-        },
+        styling: {},
         logo: null,
-        brandColor: null,
-        highlightBorderColor: null,
       },
       {
         id: createId(),
@@ -190,29 +162,23 @@ describe("Project Service", () => {
         recontactDays: 0,
         linkSurveyBranding: true,
         inAppSurveyBranding: true,
-        config: {
-          channel: null,
-          industry: null,
-        },
-        placement: WidgetPlacement.bottomRight,
+        config: {},
+        placement: "bottomRight",
         clickOutsideClose: true,
         darkOverlay: false,
         environments: [],
-        styling: {
-          allowStyleOverwrite: true,
-        },
+        styling: {},
         logo: null,
-        brandColor: null,
-        highlightBorderColor: null,
       },
     ];
 
     vi.mocked(prisma.membership.findFirst).mockResolvedValue({
+      id: createId(),
       userId,
       organizationId,
-      role: OrganizationRole.owner,
-      accepted: true,
-      deprecatedRole: null,
+      role: "admin",
+      createdAt: new Date(),
+      updatedAt: new Date(),
     });
 
     vi.mocked(prisma.project.findMany).mockResolvedValue(mockProjects);
@@ -244,29 +210,23 @@ describe("Project Service", () => {
         recontactDays: 0,
         linkSurveyBranding: true,
         inAppSurveyBranding: true,
-        config: {
-          channel: null,
-          industry: null,
-        },
-        placement: WidgetPlacement.bottomRight,
+        config: {},
+        placement: "bottomRight",
         clickOutsideClose: true,
         darkOverlay: false,
         environments: [],
-        styling: {
-          allowStyleOverwrite: true,
-        },
+        styling: {},
         logo: null,
-        brandColor: null,
-        highlightBorderColor: null,
       },
     ];
 
     vi.mocked(prisma.membership.findFirst).mockResolvedValue({
+      id: createId(),
       userId,
       organizationId,
-      role: OrganizationRole.member,
-      accepted: true,
-      deprecatedRole: null,
+      role: "member",
+      createdAt: new Date(),
+      updatedAt: new Date(),
     });
 
     vi.mocked(prisma.project.findMany).mockResolvedValue(mockProjects);
@@ -318,29 +278,23 @@ describe("Project Service", () => {
         recontactDays: 0,
         linkSurveyBranding: true,
         inAppSurveyBranding: true,
-        config: {
-          channel: null,
-          industry: null,
-        },
-        placement: WidgetPlacement.bottomRight,
+        config: {},
+        placement: "bottomRight",
         clickOutsideClose: true,
         darkOverlay: false,
         environments: [],
-        styling: {
-          allowStyleOverwrite: true,
-        },
+        styling: {},
         logo: null,
-        brandColor: null,
-        highlightBorderColor: null,
       },
     ];
 
     vi.mocked(prisma.membership.findFirst).mockResolvedValue({
+      id: createId(),
       userId,
       organizationId,
-      role: OrganizationRole.owner,
-      accepted: true,
-      deprecatedRole: null,
+      role: "admin",
+      createdAt: new Date(),
+      updatedAt: new Date(),
     });
 
     vi.mocked(prisma.project.findMany).mockResolvedValue(mockProjects);
@@ -372,20 +326,13 @@ describe("Project Service", () => {
         recontactDays: 0,
         linkSurveyBranding: true,
         inAppSurveyBranding: true,
-        config: {
-          channel: null,
-          industry: null,
-        },
-        placement: WidgetPlacement.bottomRight,
+        config: {},
+        placement: "bottomRight",
         clickOutsideClose: true,
         darkOverlay: false,
         environments: [],
-        styling: {
-          allowStyleOverwrite: true,
-        },
+        styling: {},
         logo: null,
-        brandColor: null,
-        highlightBorderColor: null,
       },
       {
         id: createId(),
@@ -397,20 +344,13 @@ describe("Project Service", () => {
         recontactDays: 0,
         linkSurveyBranding: true,
         inAppSurveyBranding: true,
-        config: {
-          channel: null,
-          industry: null,
-        },
-        placement: WidgetPlacement.bottomRight,
+        config: {},
+        placement: "bottomRight",
         clickOutsideClose: true,
         darkOverlay: false,
         environments: [],
-        styling: {
-          allowStyleOverwrite: true,
-        },
+        styling: {},
         logo: null,
-        brandColor: null,
-        highlightBorderColor: null,
       },
     ];
 
@@ -442,20 +382,13 @@ describe("Project Service", () => {
         recontactDays: 0,
         linkSurveyBranding: true,
         inAppSurveyBranding: true,
-        config: {
-          channel: null,
-          industry: null,
-        },
-        placement: WidgetPlacement.bottomRight,
+        config: {},
+        placement: "bottomRight",
         clickOutsideClose: true,
         darkOverlay: false,
         environments: [],
-        styling: {
-          allowStyleOverwrite: true,
-        },
+        styling: {},
         logo: null,
-        brandColor: null,
-        highlightBorderColor: null,
       },
     ];
 
@@ -485,199 +418,4 @@ describe("Project Service", () => {
 
     await expect(getProjects(organizationId)).rejects.toThrow(DatabaseError);
   });
-
-  test("getProjectsByOrganizationIds should return projects for given organization IDs", async () => {
-    const organizationId1 = createId();
-    const organizationId2 = createId();
-    const userId = createId();
-    const mockProjects = [
-      {
-        environments: [],
-      },
-      {
-        environments: [],
-      },
-    ];
-
-    vi.mocked(prisma.membership.findMany).mockResolvedValue([
-      {
-        userId,
-        organizationId: organizationId1,
-        role: "owner" as any,
-        accepted: true,
-        deprecatedRole: null,
-      },
-      {
-        userId,
-        organizationId: organizationId2,
-        role: "owner" as any,
-        accepted: true,
-        deprecatedRole: null,
-      },
-    ]);
-
-    vi.mocked(prisma.project.findMany).mockResolvedValue(mockProjects as any);
-
-    const result = await getUserProjectEnvironmentsByOrganizationIds(
-      [organizationId1, organizationId2],
-      userId
-    );
-
-    expect(result).toEqual(mockProjects);
-    expect(prisma.project.findMany).toHaveBeenCalledWith({
-      where: {
-        OR: [{ organizationId: organizationId1 }, { organizationId: organizationId2 }],
-      },
-      select: { environments: true },
-    });
-  });
-
-  test("getProjectsByOrganizationIds should return empty array when no projects are found", async () => {
-    const organizationId1 = createId();
-    const organizationId2 = createId();
-    const userId = createId();
-
-    vi.mocked(prisma.membership.findMany).mockResolvedValue([
-      {
-        userId,
-        organizationId: organizationId1,
-        role: "owner" as any,
-        accepted: true,
-        deprecatedRole: null,
-      },
-      {
-        userId,
-        organizationId: organizationId2,
-        role: "owner" as any,
-        accepted: true,
-        deprecatedRole: null,
-      },
-    ]);
-
-    vi.mocked(prisma.project.findMany).mockResolvedValue([]);
-
-    const result = await getUserProjectEnvironmentsByOrganizationIds(
-      [organizationId1, organizationId2],
-      userId
-    );
-
-    expect(result).toEqual([]);
-    expect(prisma.project.findMany).toHaveBeenCalledWith({
-      where: {
-        OR: [{ organizationId: organizationId1 }, { organizationId: organizationId2 }],
-      },
-      select: { environments: true },
-    });
-  });
-
-  test("getProjectsByOrganizationIds should throw DatabaseError when prisma throws", async () => {
-    const organizationId1 = createId();
-    const organizationId2 = createId();
-    const userId = createId();
-    const prismaError = new Prisma.PrismaClientKnownRequestError("Database error", {
-      code: "P2002",
-      clientVersion: "5.0.0",
-    });
-
-    vi.mocked(prisma.membership.findMany).mockResolvedValue([
-      {
-        userId,
-        organizationId: organizationId1,
-        role: "owner" as any,
-        accepted: true,
-        deprecatedRole: null,
-      },
-    ]);
-
-    vi.mocked(prisma.project.findMany).mockRejectedValue(prismaError);
-
-    await expect(
-      getUserProjectEnvironmentsByOrganizationIds([organizationId1, organizationId2], userId)
-    ).rejects.toThrow(DatabaseError);
-  });
-
-  test("getProjectsByOrganizationIds should throw ValidationError with wrong input", async () => {
-    const userId = createId();
-    await expect(getUserProjectEnvironmentsByOrganizationIds(["wrong-id"], userId)).rejects.toThrow(
-      ValidationError
-    );
-  });
-
-  test("getProjectsByOrganizationIds should return empty array when user has no memberships", async () => {
-    const organizationId1 = createId();
-    const organizationId2 = createId();
-    const userId = createId();
-
-    // Mock no memberships found
-    vi.mocked(prisma.membership.findMany).mockResolvedValue([]);
-
-    const result = await getUserProjectEnvironmentsByOrganizationIds(
-      [organizationId1, organizationId2],
-      userId
-    );
-
-    expect(result).toEqual([]);
-    expect(prisma.membership.findMany).toHaveBeenCalledWith({
-      where: {
-        userId,
-        organizationId: {
-          in: [organizationId1, organizationId2],
-        },
-      },
-    });
-    // Should not call project.findMany when no memberships
-    expect(prisma.project.findMany).not.toHaveBeenCalled();
-  });
-
-  test("getProjectsByOrganizationIds should handle member role with team access", async () => {
-    const organizationId1 = createId();
-    const organizationId2 = createId();
-    const userId = createId();
-    const mockProjects = [
-      {
-        environments: [],
-      },
-    ];
-
-    // Mock membership where user is a member
-    vi.mocked(prisma.membership.findMany).mockResolvedValue([
-      {
-        userId,
-        organizationId: organizationId1,
-        role: "member" as any,
-        accepted: true,
-        deprecatedRole: null,
-      },
-    ]);
-
-    vi.mocked(prisma.project.findMany).mockResolvedValue(mockProjects as any);
-
-    const result = await getUserProjectEnvironmentsByOrganizationIds(
-      [organizationId1, organizationId2],
-      userId
-    );
-
-    expect(result).toEqual(mockProjects);
-    expect(prisma.project.findMany).toHaveBeenCalledWith({
-      where: {
-        OR: [
-          {
-            organizationId: organizationId1,
-            projectTeams: {
-              some: {
-                team: {
-                  teamUsers: {
-                    some: {
-                      userId,
-                    },
-                  },
-                },
-              },
-            },
-          },
-        ],
-      },
-      select: { environments: true },
-    });
-  });
 });

apps/web/lib/project/service.ts

@@ -170,67 +170,3 @@ export const getOrganizationProjectsCount = reactCache(async (organizationId: st
     throw error;
   }
 });
-
-export const getUserProjectEnvironmentsByOrganizationIds = reactCache(
-  async (organizationIds: string[], userId: string): Promise<Pick<TProject, "environments">[]> => {
-    validateInputs([organizationIds, ZId.array()], [userId, ZId]);
-    try {
-      if (organizationIds.length === 0) {
-        return [];
-      }
-
-      const memberships = await prisma.membership.findMany({
-        where: {
-          userId,
-          organizationId: {
-            in: organizationIds,
-          },
-        },
-      });
-
-      if (memberships.length === 0) {
-        return [];
-      }
-
-      const whereConditions: Prisma.ProjectWhereInput[] = memberships.map((membership) => {
-        let projectWhereClause: Prisma.ProjectWhereInput = {
-          organizationId: membership.organizationId,
-        };
-
-        if (membership.role === "member") {
-          projectWhereClause = {
-            ...projectWhereClause,
-            projectTeams: {
-              some: {
-                team: {
-                  teamUsers: {
-                    some: {
-                      userId,
-                    },
-                  },
-                },
-              },
-            },
-          };
-        }
-
-        return projectWhereClause;
-      });
-
-      const projects = await prisma.project.findMany({
-        where: {
-          OR: whereConditions,
-        },
-        select: { environments: true },
-      });
-
-      return projects;
-    } catch (err) {
-      if (err instanceof Prisma.PrismaClientKnownRequestError) {
-        throw new DatabaseError(err.message);
-      }
-
-      throw err;
-    }
-  }
-);

apps/web/locales/de-DE.json

@@ -34,8 +34,7 @@
           "text": "Du kannst Dich jetzt mit deinem neuen Passwort einloggen"
         }
       },
-      "reset_password": "Passwort zurücksetzen",
-      "reset_password_description": "Du wirst abgemeldet, um dein Passwort zurückzusetzen."
+      "reset_password": "Passwort zurücksetzen"
     },
     "invite": {
       "create_account": "Konto erstellen",
@@ -192,6 +191,7 @@
     "e_commerce": "E-Commerce",
     "edit": "Bearbeiten",
     "email": "E-Mail",
+    "embed": "Einbetten",
     "enterprise_license": "Enterprise Lizenz",
     "environment_not_found": "Umgebung nicht gefunden",
     "environment_notice": "Du befindest dich derzeit in der {environment}-Umgebung.",
@@ -309,6 +309,7 @@
     "project_not_found": "Projekt nicht gefunden",
     "project_permission_not_found": "Projekt-Berechtigung nicht gefunden",
     "projects": "Projekte",
+    "projects_limit_reached": "Projektlimit erreicht",
     "question": "Frage",
     "question_id": "Frage-ID",
     "questions": "Fragen",
@@ -1786,7 +1787,6 @@
         "setup_instructions": "Einrichtung",
         "setup_integrations": "Integrationen einrichten",
         "share_results": "Ergebnisse teilen",
-        "share_survey": "Umfrage teilen",
         "share_the_link": "Teile den Link",
         "share_the_link_to_get_responses": "Teile den Link, um Antworten einzusammeln",
         "show_all_responses_that_match": "Zeige alle Antworten, die übereinstimmen",

apps/web/locales/en-US.json

@@ -34,8 +34,7 @@
           "text": "You can now log in with your new password"
         }
       },
-      "reset_password": "Reset password",
-      "reset_password_description": "You will be logged out to reset your password."
+      "reset_password": "Reset password"
     },
     "invite": {
       "create_account": "Create an account",
@@ -192,6 +191,7 @@
     "e_commerce": "E-Commerce",
     "edit": "Edit",
     "email": "Email",
+    "embed": "Embed",
     "enterprise_license": "Enterprise License",
     "environment_not_found": "Environment not found",
     "environment_notice": "You're currently in the {environment} environment.",
@@ -309,6 +309,7 @@
     "project_not_found": "Project not found",
     "project_permission_not_found": "Project permission not found",
     "projects": "Projects",
+    "projects_limit_reached": "Projects limit reached",
     "question": "Question",
     "question_id": "Question ID",
     "questions": "Questions",
@@ -1786,7 +1787,6 @@
         "setup_instructions": "Setup instructions",
         "setup_integrations": "Setup integrations",
         "share_results": "Share results",
-        "share_survey": "Share survey",
         "share_the_link": "Share the link",
         "share_the_link_to_get_responses": "Share the link to get responses",
         "show_all_responses_that_match": "Show all responses that match",
@@ -1892,12 +1892,12 @@
   },
   "s": {
     "check_inbox_or_spam": "Please also check your spam folder if you don't see the email in your inbox.",
-    "completed": "This survey is closed.",
-    "create_your_own": "Create your own open-source survey",
+    "completed": "This free & open-source survey has been closed.",
+    "create_your_own": "Create your own",
     "enter_pin": "This survey is protected. Enter the PIN below",
     "just_curious": "Just curious?",
     "link_invalid": "This survey can only be taken by invitation.",
-    "paused": "This survey is temporarily paused.",
+    "paused": "This free & open-source survey is temporarily paused.",
     "please_try_again_with_the_original_link": "Please try again with the original link",
     "preview_survey_questions": "Preview survey questions.",
     "question_preview": "Question Preview",

apps/web/locales/fr-FR.json

@@ -34,8 +34,7 @@
           "text": "Vous pouvez maintenant vous connecter avec votre nouveau mot de passe."
         }
       },
-      "reset_password": "Réinitialiser le mot de passe",
-      "reset_password_description": "Vous serez déconnecté pour réinitialiser votre mot de passe."
+      "reset_password": "Réinitialiser le mot de passe"
     },
     "invite": {
       "create_account": "Créer un compte",
@@ -192,6 +191,7 @@
     "e_commerce": "E-commerce",
     "edit": "Modifier",
     "email": "Email",
+    "embed": "Intégrer",
     "enterprise_license": "Licence d'entreprise",
     "environment_not_found": "Environnement non trouvé",
     "environment_notice": "Vous êtes actuellement dans l'environnement {environment}.",
@@ -309,6 +309,7 @@
     "project_not_found": "Projet non trouvé",
     "project_permission_not_found": "Autorisation de projet non trouvée",
     "projects": "Projets",
+    "projects_limit_reached": "Limite de projets atteinte",
     "question": "Question",
     "question_id": "ID de la question",
     "questions": "Questions",
@@ -1786,7 +1787,6 @@
         "setup_instructions": "Instructions d'installation",
         "setup_integrations": "Configurer les intégrations",
         "share_results": "Partager les résultats",
-        "share_survey": "Partager l'enquête",
         "share_the_link": "Partager le lien",
         "share_the_link_to_get_responses": "Partagez le lien pour obtenir des réponses",
         "show_all_responses_that_match": "Afficher toutes les réponses correspondantes",

apps/web/locales/pt-BR.json

@@ -34,8 +34,7 @@
           "text": "Agora você pode fazer login com sua nova senha"
         }
       },
-      "reset_password": "Redefinir senha",
-      "reset_password_description": "Você será desconectado para redefinir sua senha."
+      "reset_password": "Redefinir senha"
     },
     "invite": {
       "create_account": "Cria uma conta",
@@ -192,6 +191,7 @@
     "e_commerce": "comércio eletrônico",
     "edit": "Editar",
     "email": "Email",
+    "embed": "incorporar",
     "enterprise_license": "Licença Empresarial",
     "environment_not_found": "Ambiente não encontrado",
     "environment_notice": "Você está atualmente no ambiente {environment}.",
@@ -309,6 +309,7 @@
     "project_not_found": "Projeto não encontrado",
     "project_permission_not_found": "Permissão do projeto não encontrada",
     "projects": "Projetos",
+    "projects_limit_reached": "Limites de projetos atingidos",
     "question": "Pergunta",
     "question_id": "ID da Pergunta",
     "questions": "Perguntas",
@@ -356,7 +357,7 @@
     "start_free_trial": "Iniciar Teste Grátis",
     "status": "status",
     "step_by_step_manual": "Manual passo a passo",
-    "styling": "Estilização",
+    "styling": "estilização",
     "submit": "Enviar",
     "summary": "Resumo",
     "survey": "Pesquisa",
@@ -368,7 +369,7 @@
     "survey_paused": "Pesquisa pausada.",
     "survey_scheduled": "Pesquisa agendada.",
     "survey_type": "Tipo de Pesquisa",
-    "surveys": "Pesquisas",
+    "surveys": "pesquisas",
     "switch_organization": "Mudar organização",
     "switch_to": "Mudar para {environment}",
     "table_items_deleted_successfully": "{type}s deletados com sucesso",
@@ -1786,7 +1787,6 @@
         "setup_instructions": "Instruções de configuração",
         "setup_integrations": "Configurar integrações",
         "share_results": "Compartilhar resultados",
-        "share_survey": "Compartilhar pesquisa",
         "share_the_link": "Compartilha o link",
         "share_the_link_to_get_responses": "Compartilha o link pra receber respostas",
         "show_all_responses_that_match": "Mostrar todas as respostas que correspondem",

apps/web/locales/pt-PT.json

@@ -34,8 +34,7 @@
           "text": "Pode agora iniciar sessão com a sua nova palavra-passe"
         }
       },
-      "reset_password": "Redefinir palavra-passe",
-      "reset_password_description": "Será desconectado para redefinir a sua palavra-passe."
+      "reset_password": "Redefinir palavra-passe"
     },
     "invite": {
       "create_account": "Criar uma conta",
@@ -192,6 +191,7 @@
     "e_commerce": "Comércio Eletrónico",
     "edit": "Editar",
     "email": "Email",
+    "embed": "Incorporar",
     "enterprise_license": "Licença Enterprise",
     "environment_not_found": "Ambiente não encontrado",
     "environment_notice": "Está atualmente no ambiente {environment}.",
@@ -309,6 +309,7 @@
     "project_not_found": "Projeto não encontrado",
     "project_permission_not_found": "Permissão do projeto não encontrada",
     "projects": "Projetos",
+    "projects_limit_reached": "Limite de projetos atingido",
     "question": "Pergunta",
     "question_id": "ID da pergunta",
     "questions": "Perguntas",
@@ -1786,7 +1787,6 @@
         "setup_instructions": "Instruções de configuração",
         "setup_integrations": "Configurar integrações",
         "share_results": "Partilhar resultados",
-        "share_survey": "Partilhar inquérito",
         "share_the_link": "Partilhar o link",
         "share_the_link_to_get_responses": "Partilhe o link para obter respostas",
         "show_all_responses_that_match": "Mostrar todas as respostas que correspondem",
@@ -1892,12 +1892,12 @@
   },
   "s": {
     "check_inbox_or_spam": "Por favor, verifique também a sua pasta de spam se não vir o email na sua caixa de entrada.",
-    "completed": "Este inquérito está encerrado.",
-    "create_your_own": "Crie o seu próprio inquérito de código aberto",
+    "completed": "Este inquérito gratuito e de código aberto foi encerrado.",
+    "create_your_own": "Crie o seu próprio",
     "enter_pin": "Este inquérito está protegido. Introduza o PIN abaixo",
     "just_curious": "Só por curiosidade?",
     "link_invalid": "Este inquérito só pode ser respondido por convite.",
-    "paused": "Este inquérito está temporariamente suspenso.",
+    "paused": "Este inquérito gratuito e de código aberto está temporariamente pausado.",
     "please_try_again_with_the_original_link": "Por favor, tente novamente com o link original",
     "preview_survey_questions": "Pré-visualizar perguntas do inquérito.",
     "question_preview": "Pré-visualização da Pergunta",

apps/web/locales/zh-Hant-TW.json

@@ -34,8 +34,7 @@
           "text": "您現在可以使用新密碼登入"
         }
       },
-      "reset_password": "重設密碼",
-      "reset_password_description": "您將被登出以重設您的密碼。"
+      "reset_password": "重設密碼"
     },
     "invite": {
       "create_account": "建立帳戶",
@@ -192,6 +191,7 @@
     "e_commerce": "電子商務",
     "edit": "編輯",
     "email": "電子郵件",
+    "embed": "嵌入",
     "enterprise_license": "企業授權",
     "environment_not_found": "找不到環境",
     "environment_notice": "您目前在 '{'environment'}' 環境中。",
@@ -309,6 +309,7 @@
     "project_not_found": "找不到專案",
     "project_permission_not_found": "找不到專案權限",
     "projects": "專案",
+    "projects_limit_reached": "已達到專案上限",
     "question": "問題",
     "question_id": "問題 ID",
     "questions": "問題",
@@ -1786,7 +1787,6 @@
         "setup_instructions": "設定說明",
         "setup_integrations": "設定整合",
         "share_results": "分享結果",
-        "share_survey": "分享問卷",
         "share_the_link": "分享連結",
         "share_the_link_to_get_responses": "分享連結以取得回應",
         "show_all_responses_that_match": "顯示所有相符的回應",

apps/web/modules/account/components/DeleteAccountModal/index.test.tsx

@@ -78,11 +78,6 @@ describe("DeleteAccountModal", () => {
       .spyOn(actions, "deleteUserAction")
       .mockResolvedValue("deleted-user-id" as any); // the return doesn't matter here
 
-    Object.defineProperty(window, "localStorage", {
-      writable: true,
-      value: { removeItem: vi.fn() },
-    });
-
     // Mock window.location.replace
     Object.defineProperty(window, "location", {
       writable: true,
@@ -110,7 +105,6 @@ describe("DeleteAccountModal", () => {
       expect(mockSignOut).toHaveBeenCalledWith({
         reason: "account_deletion",
         redirect: false, // Updated to match new implementation
-        clearEnvironmentId: true,
       });
       expect(window.location.replace).toHaveBeenCalledWith("/auth/login");
       expect(mockSetOpen).toHaveBeenCalledWith(false);
@@ -122,11 +116,6 @@ describe("DeleteAccountModal", () => {
       .spyOn(actions, "deleteUserAction")
       .mockResolvedValue("deleted-user-id" as any); // the return doesn't matter here
 
-    Object.defineProperty(window, "localStorage", {
-      writable: true,
-      value: { removeItem: vi.fn() },
-    });
-
     Object.defineProperty(window, "location", {
       writable: true,
       value: { replace: vi.fn() },
@@ -153,7 +142,6 @@ describe("DeleteAccountModal", () => {
       expect(mockSignOut).toHaveBeenCalledWith({
         reason: "account_deletion",
         redirect: false, // Updated to match new implementation
-        clearEnvironmentId: true,
       });
       expect(window.location.replace).toHaveBeenCalledWith(
         "https://app.formbricks.com/s/clri52y3z8f221225wjdhsoo2"

apps/web/modules/account/components/DeleteAccountModal/index.tsx

@@ -42,7 +42,6 @@ export const DeleteAccountModal = ({
       await signOutWithAudit({
         reason: "account_deletion",
         redirect: false, // Prevent NextAuth automatic redirect
-        clearEnvironmentId: true,
       });
 
       // Manual redirect after signOut completes

apps/web/modules/api/v2/lib/question.ts

@@ -33,11 +33,10 @@ export const validateOtherOptionLengthForMultipleChoice = ({
   surveyQuestions,
   responseLanguage,
 }: {
-  responseData?: TResponseData;
+  responseData: TResponseData;
   surveyQuestions: TSurveyQuestion[];
   responseLanguage?: string;
 }): string | undefined => {
-  if (!responseData) return undefined;
   for (const [questionId, answer] of Object.entries(responseData)) {
     const question = surveyQuestions.find((q) => q.id === questionId);
     if (!question) continue;

apps/web/modules/api/v2/management/contact-attribute-keys/[contactAttributeKeyId]/lib/contact-attribute-key.ts

@@ -1,6 +1,7 @@
 import { TContactAttributeKeyUpdateSchema } from "@/modules/api/v2/management/contact-attribute-keys/[contactAttributeKeyId]/types/contact-attribute-keys";
 import { ApiErrorResponseV2 } from "@/modules/api/v2/types/api-error";
-import { ContactAttributeKey, Prisma } from "@prisma/client";
+import { ContactAttributeKey } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { cache as reactCache } from "react";
 import { prisma } from "@formbricks/database";
 import { PrismaErrorType } from "@formbricks/database/types/error";
@@ -54,7 +55,7 @@ export const updateContactAttributeKey = async (
 
     return ok(updatedKey);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist
@@ -105,7 +106,7 @@ export const deleteContactAttributeKey = async (
 
     return ok(deletedKey);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist

apps/web/modules/api/v2/management/contact-attribute-keys/[contactAttributeKeyId]/lib/openapi.ts

@@ -16,7 +16,7 @@ export const getContactAttributeKeyEndpoint: ZodOpenApiOperationObject = {
       id: ZContactAttributeKeyIdSchema,
     }),
   },
-  tags: ["Management API - Contact Attribute Keys"],
+  tags: ["Management API > Contact Attribute Keys"],
   responses: {
     "200": {
       description: "Contact attribute key retrieved successfully.",
@@ -33,7 +33,7 @@ export const updateContactAttributeKeyEndpoint: ZodOpenApiOperationObject = {
   operationId: "updateContactAttributeKey",
   summary: "Update a contact attribute key",
   description: "Updates a contact attribute key in the database.",
-  tags: ["Management API - Contact Attribute Keys"],
+  tags: ["Management API > Contact Attribute Keys"],
   requestParams: {
     path: z.object({
       id: ZContactAttributeKeyIdSchema,
@@ -64,7 +64,7 @@ export const deleteContactAttributeKeyEndpoint: ZodOpenApiOperationObject = {
   operationId: "deleteContactAttributeKey",
   summary: "Delete a contact attribute key",
   description: "Deletes a contact attribute key from the database.",
-  tags: ["Management API - Contact Attribute Keys"],
+  tags: ["Management API > Contact Attribute Keys"],
   requestParams: {
     path: z.object({
       id: ZContactAttributeKeyIdSchema,

apps/web/modules/api/v2/management/contact-attribute-keys/[contactAttributeKeyId]/lib/tests/contact-attribute-key.test.ts

@@ -1,5 +1,6 @@
 import { TContactAttributeKeyUpdateSchema } from "@/modules/api/v2/management/contact-attribute-keys/[contactAttributeKeyId]/types/contact-attribute-keys";
-import { ContactAttributeKey, Prisma } from "@prisma/client";
+import { ContactAttributeKey } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { describe, expect, test, vi } from "vitest";
 import { prisma } from "@formbricks/database";
 import { PrismaErrorType } from "@formbricks/database/types/error";
@@ -43,12 +44,12 @@ const mockUpdateInput: TContactAttributeKeyUpdateSchema = {
   description: "User's verified email address",
 };
 
-const prismaNotFoundError = new Prisma.PrismaClientKnownRequestError("Mock error message", {
+const prismaNotFoundError = new PrismaClientKnownRequestError("Mock error message", {
   code: PrismaErrorType.RelatedRecordDoesNotExist,
   clientVersion: "0.0.1",
 });
 
-const prismaUniqueConstraintError = new Prisma.PrismaClientKnownRequestError("Mock error message", {
+const prismaUniqueConstraintError = new PrismaClientKnownRequestError("Mock error message", {
   code: PrismaErrorType.UniqueConstraintViolation,
   clientVersion: "0.0.1",
 });

apps/web/modules/api/v2/management/contact-attribute-keys/lib/contact-attribute-key.ts

@@ -5,6 +5,7 @@ import {
 } from "@/modules/api/v2/management/contact-attribute-keys/types/contact-attribute-keys";
 import { ApiErrorResponseV2 } from "@/modules/api/v2/types/api-error";
 import { ContactAttributeKey, Prisma } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { cache as reactCache } from "react";
 import { prisma } from "@formbricks/database";
 import { PrismaErrorType } from "@formbricks/database/types/error";
@@ -57,7 +58,7 @@ export const createContactAttributeKey = async (
 
     return ok(createdContactAttributeKey);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist

apps/web/modules/api/v2/management/contact-attribute-keys/lib/openapi.ts

@@ -16,7 +16,7 @@ export const getContactAttributeKeysEndpoint: ZodOpenApiOperationObject = {
   operationId: "getContactAttributeKeys",
   summary: "Get contact attribute keys",
   description: "Gets contact attribute keys from the database.",
-  tags: ["Management API - Contact Attribute Keys"],
+  tags: ["Management API > Contact Attribute Keys"],
   requestParams: {
     query: ZGetContactAttributeKeysFilter.sourceType(),
   },
@@ -36,7 +36,7 @@ export const createContactAttributeKeyEndpoint: ZodOpenApiOperationObject = {
   operationId: "createContactAttributeKey",
   summary: "Create a contact attribute key",
   description: "Creates a contact attribute key in the database.",
-  tags: ["Management API - Contact Attribute Keys"],
+  tags: ["Management API > Contact Attribute Keys"],
   requestBody: {
     required: true,
     description: "The contact attribute key to create",

apps/web/modules/api/v2/management/contact-attribute-keys/lib/tests/contact-attribute-key.test.ts

@@ -2,7 +2,8 @@ import {
   TContactAttributeKeyInput,
   TGetContactAttributeKeysFilter,
 } from "@/modules/api/v2/management/contact-attribute-keys/types/contact-attribute-keys";
-import { ContactAttributeKey, Prisma } from "@prisma/client";
+import { ContactAttributeKey } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { describe, expect, test, vi } from "vitest";
 import { prisma } from "@formbricks/database";
 import { PrismaErrorType } from "@formbricks/database/types/error";
@@ -105,7 +106,7 @@ describe("createContactAttributeKey", () => {
   });
 
   test("returns conflict error when key already exists", async () => {
-    const errToThrow = new Prisma.PrismaClientKnownRequestError("Mock error message", {
+    const errToThrow = new PrismaClientKnownRequestError("Mock error message", {
       code: PrismaErrorType.UniqueConstraintViolation,
       clientVersion: "0.0.1",
     });
@@ -128,7 +129,7 @@ describe("createContactAttributeKey", () => {
   });
 
   test("returns not found error when related record does not exist", async () => {
-    const errToThrow = new Prisma.PrismaClientKnownRequestError("Mock error message", {
+    const errToThrow = new PrismaClientKnownRequestError("Mock error message", {
       code: PrismaErrorType.RelatedRecordDoesNotExist,
       clientVersion: "0.0.1",
     });

apps/web/modules/api/v2/management/contact-attributes/[contactAttributeId]/lib/openapi.ts

@@ -12,7 +12,7 @@ export const getContactAttributeEndpoint: ZodOpenApiOperationObject = {
       contactAttributeId: z.string().cuid2(),
     }),
   },
-  tags: ["Management API - Contact Attributes"],
+  tags: ["Management API > Contact Attributes"],
   responses: {
     "200": {
       description: "Contact retrieved successfully.",
@@ -29,7 +29,7 @@ export const deleteContactAttributeEndpoint: ZodOpenApiOperationObject = {
   operationId: "deleteContactAttribute",
   summary: "Delete a contact attribute",
   description: "Deletes a contact attribute from the database.",
-  tags: ["Management API - Contact Attributes"],
+  tags: ["Management API > Contact Attributes"],
   requestParams: {
     path: z.object({
       contactAttributeId: z.string().cuid2(),
@@ -51,7 +51,7 @@ export const updateContactAttributeEndpoint: ZodOpenApiOperationObject = {
   operationId: "updateContactAttribute",
   summary: "Update a contact attribute",
   description: "Updates a contact attribute in the database.",
-  tags: ["Management API - Contact Attributes"],
+  tags: ["Management API > Contact Attributes"],
   requestParams: {
     path: z.object({
       contactAttributeId: z.string().cuid2(),

apps/web/modules/api/v2/management/contact-attributes/lib/openapi.ts

@@ -16,7 +16,7 @@ export const getContactAttributesEndpoint: ZodOpenApiOperationObject = {
   operationId: "getContactAttributes",
   summary: "Get contact attributes",
   description: "Gets contact attributes from the database.",
-  tags: ["Management API - Contact Attributes"],
+  tags: ["Management API > Contact Attributes"],
   requestParams: {
     query: ZGetContactAttributesFilter,
   },
@@ -36,7 +36,7 @@ export const createContactAttributeEndpoint: ZodOpenApiOperationObject = {
   operationId: "createContactAttribute",
   summary: "Create a contact attribute",
   description: "Creates a contact attribute in the database.",
-  tags: ["Management API - Contact Attributes"],
+  tags: ["Management API > Contact Attributes"],
   requestBody: {
     required: true,
     description: "The contact attribute to create",

apps/web/modules/api/v2/management/contacts/[contactId]/lib/openapi.ts

@@ -12,7 +12,7 @@ export const getContactEndpoint: ZodOpenApiOperationObject = {
       contactId: z.string().cuid2(),
     }),
   },
-  tags: ["Management API - Contacts"],
+  tags: ["Management API > Contacts"],
   responses: {
     "200": {
       description: "Contact retrieved successfully.",
@@ -29,7 +29,7 @@ export const deleteContactEndpoint: ZodOpenApiOperationObject = {
   operationId: "deleteContact",
   summary: "Delete a contact",
   description: "Deletes a contact from the database.",
-  tags: ["Management API - Contacts"],
+  tags: ["Management API > Contacts"],
   requestParams: {
     path: z.object({
       contactId: z.string().cuid2(),
@@ -51,7 +51,7 @@ export const updateContactEndpoint: ZodOpenApiOperationObject = {
   operationId: "updateContact",
   summary: "Update a contact",
   description: "Updates a contact in the database.",
-  tags: ["Management API - Contacts"],
+  tags: ["Management API > Contacts"],
   requestParams: {
     path: z.object({
       contactId: z.string().cuid2(),

apps/web/modules/api/v2/management/contacts/lib/openapi.ts

@@ -16,7 +16,7 @@ export const getContactsEndpoint: ZodOpenApiOperationObject = {
   requestParams: {
     query: ZGetContactsFilter,
   },
-  tags: ["Management API - Contacts"],
+  tags: ["Management API > Contacts"],
   responses: {
     "200": {
       description: "Contacts retrieved successfully.",
@@ -33,7 +33,7 @@ export const createContactEndpoint: ZodOpenApiOperationObject = {
   operationId: "createContact",
   summary: "Create a contact",
   description: "Creates a contact in the database.",
-  tags: ["Management API - Contacts"],
+  tags: ["Management API > Contacts"],
   requestBody: {
     required: true,
     description: "The contact to create",

apps/web/modules/api/v2/management/lib/helper.ts

@@ -25,9 +25,7 @@ export const getEnvironmentId = async (
  */
 export const getEnvironmentIdFromSurveyIds = async (
   surveyIds: string[]
-): Promise<Result<string | null, ApiErrorResponseV2>> => {
-  if (surveyIds.length === 0) return ok(null);
-
+): Promise<Result<string, ApiErrorResponseV2>> => {
   const result = await fetchEnvironmentIdFromSurveyIds(surveyIds);
 
   if (!result.ok) {

apps/web/modules/api/v2/management/responses/[responseId]/lib/display.ts

@@ -1,5 +1,5 @@
 import { ApiErrorResponseV2 } from "@/modules/api/v2/types/api-error";
-import { Prisma } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { prisma } from "@formbricks/database";
 import { PrismaErrorType } from "@formbricks/database/types/error";
 import { Result, err, ok } from "@formbricks/types/error-handlers";
@@ -19,7 +19,7 @@ export const deleteDisplay = async (displayId: string): Promise<Result<boolean,
 
     return ok(true);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist

apps/web/modules/api/v2/management/responses/[responseId]/lib/openapi.ts

@@ -14,7 +14,7 @@ export const getResponseEndpoint: ZodOpenApiOperationObject = {
       id: ZResponseIdSchema,
     }),
   },
-  tags: ["Management API - Responses"],
+  tags: ["Management API > Responses"],
   responses: {
     "200": {
       description: "Response retrieved successfully.",
@@ -31,7 +31,7 @@ export const deleteResponseEndpoint: ZodOpenApiOperationObject = {
   operationId: "deleteResponse",
   summary: "Delete a response",
   description: "Deletes a response from the database.",
-  tags: ["Management API - Responses"],
+  tags: ["Management API > Responses"],
   requestParams: {
     path: z.object({
       id: ZResponseIdSchema,
@@ -53,7 +53,7 @@ export const updateResponseEndpoint: ZodOpenApiOperationObject = {
   operationId: "updateResponse",
   summary: "Update a response",
   description: "Updates a response in the database.",
-  tags: ["Management API - Responses"],
+  tags: ["Management API > Responses"],
   requestParams: {
     path: z.object({
       id: ZResponseIdSchema,

apps/web/modules/api/v2/management/responses/[responseId]/lib/response.ts

@@ -3,7 +3,8 @@ import { getSurveyQuestions } from "@/modules/api/v2/management/responses/[respo
 import { findAndDeleteUploadedFilesInResponse } from "@/modules/api/v2/management/responses/[responseId]/lib/utils";
 import { ZResponseUpdateSchema } from "@/modules/api/v2/management/responses/[responseId]/types/responses";
 import { ApiErrorResponseV2 } from "@/modules/api/v2/types/api-error";
-import { Prisma, Response } from "@prisma/client";
+import { Response } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { cache as reactCache } from "react";
 import { z } from "zod";
 import { prisma } from "@formbricks/database";
@@ -55,7 +56,7 @@ export const deleteResponse = async (responseId: string): Promise<Result<Respons
 
     return ok(deletedResponse);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist
@@ -88,7 +89,7 @@ export const updateResponse = async (
 
     return ok(updatedResponse);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist

apps/web/modules/api/v2/management/responses/[responseId]/lib/tests/display.test.ts

@@ -1,5 +1,5 @@
 import { displayId, mockDisplay } from "./__mocks__/display.mock";
-import { Prisma } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { beforeEach, describe, expect, test, vi } from "vitest";
 import { prisma } from "@formbricks/database";
 import { PrismaErrorType } from "@formbricks/database/types/error";
@@ -39,7 +39,7 @@ describe("Display Lib", () => {
 
   test("return a not_found error when the display is not found", async () => {
     vi.mocked(prisma.display.delete).mockRejectedValue(
-      new Prisma.PrismaClientKnownRequestError("Display not found", {
+      new PrismaClientKnownRequestError("Display not found", {
         code: PrismaErrorType.RelatedRecordDoesNotExist,
         clientVersion: "1.0.0",
         meta: {

apps/web/modules/api/v2/management/responses/[responseId]/lib/tests/response.test.ts

@@ -1,5 +1,5 @@
 import { response, responseId, responseInput, survey } from "./__mocks__/response.mock";
-import { Prisma } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { beforeEach, describe, expect, test, vi } from "vitest";
 import { prisma } from "@formbricks/database";
 import { PrismaErrorType } from "@formbricks/database/types/error";
@@ -154,7 +154,7 @@ describe("Response Lib", () => {
 
     test("handle prisma client error code P2025", async () => {
       vi.mocked(prisma.response.delete).mockRejectedValue(
-        new Prisma.PrismaClientKnownRequestError("Response not found", {
+        new PrismaClientKnownRequestError("Response not found", {
           code: PrismaErrorType.RelatedRecordDoesNotExist,
           clientVersion: "1.0.0",
           meta: {
@@ -208,7 +208,7 @@ describe("Response Lib", () => {
 
     test("return a not_found error when the response is not found", async () => {
       vi.mocked(prisma.response.update).mockRejectedValue(
-        new Prisma.PrismaClientKnownRequestError("Response not found", {
+        new PrismaClientKnownRequestError("Response not found", {
           code: PrismaErrorType.RelatedRecordDoesNotExist,
           clientVersion: "1.0.0",
           meta: {

apps/web/modules/api/v2/management/responses/lib/openapi.ts

@@ -16,7 +16,7 @@ export const getResponsesEndpoint: ZodOpenApiOperationObject = {
   requestParams: {
     query: ZGetResponsesFilter.sourceType(),
   },
-  tags: ["Management API - Responses"],
+  tags: ["Management API > Responses"],
   responses: {
     "200": {
       description: "Responses retrieved successfully.",
@@ -33,7 +33,7 @@ export const createResponseEndpoint: ZodOpenApiOperationObject = {
   operationId: "createResponse",
   summary: "Create a response",
   description: "Creates a response in the database.",
-  tags: ["Management API - Responses"],
+  tags: ["Management API > Responses"],
   requestBody: {
     required: true,
     description: "The response to create",

apps/web/modules/api/v2/management/surveys/[surveyId]/contact-links/contacts/[contactId]/lib/openapi.ts

@@ -10,7 +10,7 @@ export const getPersonalizedSurveyLink: ZodOpenApiOperationObject = {
   requestParams: {
     path: ZContactLinkParams,
   },
-  tags: ["Management API - Surveys - Contact Links"],
+  tags: ["Management API > Surveys > Contact Links"],
   responses: {
     "200": {
       description: "Personalized survey link retrieved successfully.",

apps/web/modules/api/v2/management/surveys/[surveyId]/contact-links/segments/[segmentId]/lib/openapi.ts

@@ -10,7 +10,7 @@ export const getContactLinksBySegmentEndpoint: ZodOpenApiOperationObject = {
   operationId: "getContactLinksBySegment",
   summary: "Get survey links for contacts in a segment",
   description: "Generates personalized survey links for contacts in a segment.",
-  tags: ["Management API - Surveys - Contact Links"],
+  tags: ["Management API > Surveys > Contact Links"],
   requestParams: {
     path: ZContactLinksBySegmentParams,
     query: ZContactLinksBySegmentQuery,

apps/web/modules/api/v2/management/surveys/[surveyId]/lib/openapi.ts

@@ -13,7 +13,7 @@ export const getSurveyEndpoint: ZodOpenApiOperationObject = {
       id: surveyIdSchema,
     }),
   },
-  tags: ["Management API - Surveys"],
+  tags: ["Management API > Surveys"],
   responses: {
     "200": {
       description: "Response retrieved successfully.",
@@ -30,7 +30,7 @@ export const deleteSurveyEndpoint: ZodOpenApiOperationObject = {
   operationId: "deleteSurvey",
   summary: "Delete a survey",
   description: "Deletes a survey from the database.",
-  tags: ["Management API - Surveys"],
+  tags: ["Management API > Surveys"],
   requestParams: {
     path: z.object({
       id: surveyIdSchema,
@@ -52,7 +52,7 @@ export const updateSurveyEndpoint: ZodOpenApiOperationObject = {
   operationId: "updateSurvey",
   summary: "Update a survey",
   description: "Updates a survey in the database.",
-  tags: ["Management API - Surveys"],
+  tags: ["Management API >  Surveys"],
   requestParams: {
     path: z.object({
       id: surveyIdSchema,

apps/web/modules/api/v2/management/surveys/lib/openapi.ts

@@ -1,3 +1,8 @@
+// import {
+//   deleteSurveyEndpoint,
+//   getSurveyEndpoint,
+//   updateSurveyEndpoint,
+// } from "@/modules/api/v2/management/surveys/[surveyId]/lib/openapi";
 import { managementServer } from "@/modules/api/v2/management/lib/openapi";
 import { getPersonalizedSurveyLink } from "@/modules/api/v2/management/surveys/[surveyId]/contact-links/contacts/[contactId]/lib/openapi";
 import { ZGetSurveysFilter, ZSurveyInput } from "@/modules/api/v2/management/surveys/types/surveys";
@@ -12,7 +17,7 @@ export const getSurveysEndpoint: ZodOpenApiOperationObject = {
   requestParams: {
     query: ZGetSurveysFilter,
   },
-  tags: ["Management API - Surveys"],
+  tags: ["Management API > Surveys"],
   responses: {
     "200": {
       description: "Surveys retrieved successfully.",
@@ -29,7 +34,7 @@ export const createSurveyEndpoint: ZodOpenApiOperationObject = {
   operationId: "createSurvey",
   summary: "Create a survey",
   description: "Creates a survey in the database.",
-  tags: ["Management API - Surveys"],
+  tags: ["Management API > Surveys"],
   requestBody: {
     required: true,
     description: "The survey to create",

apps/web/modules/api/v2/management/webhooks/[webhookId]/lib/openapi.ts

@@ -14,7 +14,7 @@ export const getWebhookEndpoint: ZodOpenApiOperationObject = {
       id: ZWebhookIdSchema,
     }),
   },
-  tags: ["Management API - Webhooks"],
+  tags: ["Management API > Webhooks"],
   responses: {
     "200": {
       description: "Webhook retrieved successfully.",
@@ -31,7 +31,7 @@ export const deleteWebhookEndpoint: ZodOpenApiOperationObject = {
   operationId: "deleteWebhook",
   summary: "Delete a webhook",
   description: "Deletes a webhook from the database.",
-  tags: ["Management API - Webhooks"],
+  tags: ["Management API > Webhooks"],
   requestParams: {
     path: z.object({
       id: ZWebhookIdSchema,
@@ -53,7 +53,7 @@ export const updateWebhookEndpoint: ZodOpenApiOperationObject = {
   operationId: "updateWebhook",
   summary: "Update a webhook",
   description: "Updates a webhook in the database.",
-  tags: ["Management API - Webhooks"],
+  tags: ["Management API > Webhooks"],
   requestParams: {
     path: z.object({
       id: ZWebhookIdSchema,

apps/web/modules/api/v2/management/webhooks/[webhookId]/lib/tests/mocks/webhook.mock.ts

@@ -1,4 +1,5 @@
-import { Prisma, WebhookSource } from "@prisma/client";
+import { WebhookSource } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { PrismaErrorType } from "@formbricks/database/types/error";
 
 export const mockedPrismaWebhookUpdateReturn = {
@@ -13,7 +14,7 @@ export const mockedPrismaWebhookUpdateReturn = {
   surveyIds: [],
 };
 
-export const prismaNotFoundError = new Prisma.PrismaClientKnownRequestError("Record does not exist", {
+export const prismaNotFoundError = new PrismaClientKnownRequestError("Record does not exist", {
   code: PrismaErrorType.RecordDoesNotExist,
   clientVersion: "PrismaClient 4.0.0",
 });

apps/web/modules/api/v2/management/webhooks/[webhookId]/lib/webhook.ts

@@ -1,6 +1,7 @@
 import { ZWebhookUpdateSchema } from "@/modules/api/v2/management/webhooks/[webhookId]/types/webhooks";
 import { ApiErrorResponseV2 } from "@/modules/api/v2/types/api-error";
-import { Prisma, Webhook } from "@prisma/client";
+import { Webhook } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { z } from "zod";
 import { prisma } from "@formbricks/database";
 import { PrismaErrorType } from "@formbricks/database/types/error";
@@ -44,7 +45,7 @@ export const updateWebhook = async (
 
     return ok(updatedWebhook);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist
@@ -72,7 +73,7 @@ export const deleteWebhook = async (webhookId: string): Promise<Result<Webhook,
 
     return ok(deletedWebhook);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist

apps/web/modules/api/v2/management/webhooks/[webhookId]/route.ts

@@ -75,14 +75,13 @@ export const PUT = async (request: NextRequest, props: { params: Promise<{ webho
         );
       }
 
-      const surveysEnvironmentIdResult = await getEnvironmentIdFromSurveyIds(body.surveyIds);
+      // get surveys environment
+      const surveysEnvironmentId = await getEnvironmentIdFromSurveyIds(body.surveyIds);
 
-      if (!surveysEnvironmentIdResult.ok) {
-        return handleApiError(request, surveysEnvironmentIdResult.error, auditLog);
+      if (!surveysEnvironmentId.ok) {
+        return handleApiError(request, surveysEnvironmentId.error, auditLog);
       }
 
-      const surveysEnvironmentId = surveysEnvironmentIdResult.data;
-
       // get webhook environment
       const webhook = await getWebhook(params.webhookId);
 
@@ -102,7 +101,7 @@ export const PUT = async (request: NextRequest, props: { params: Promise<{ webho
       }
 
       // check if webhook environment matches the surveys environment
-      if (surveysEnvironmentId && webhook.data.environmentId !== surveysEnvironmentId) {
+      if (webhook.data.environmentId !== surveysEnvironmentId.data) {
         return handleApiError(
           request,
           {

apps/web/modules/api/v2/management/webhooks/lib/openapi.ts

@@ -16,7 +16,7 @@ export const getWebhooksEndpoint: ZodOpenApiOperationObject = {
   requestParams: {
     query: ZGetWebhooksFilter.sourceType(),
   },
-  tags: ["Management API - Webhooks"],
+  tags: ["Management API > Webhooks"],
   responses: {
     "200": {
       description: "Webhooks retrieved successfully.",
@@ -33,7 +33,7 @@ export const createWebhookEndpoint: ZodOpenApiOperationObject = {
   operationId: "createWebhook",
   summary: "Create a webhook",
   description: "Creates a webhook in the database.",
-  tags: ["Management API - Webhooks"],
+  tags: ["Management API > Webhooks"],
   requestBody: {
     required: true,
     description: "The webhook to create",

apps/web/modules/api/v2/management/webhooks/route.ts

@@ -57,12 +57,10 @@ export const POST = async (request: NextRequest) =>
         );
       }
 
-      if (body.surveyIds && body.surveyIds.length > 0) {
-        const environmentIdResult = await getEnvironmentIdFromSurveyIds(body.surveyIds);
+      const environmentIdResult = await getEnvironmentIdFromSurveyIds(body.surveyIds);
 
-        if (!environmentIdResult.ok) {
-          return handleApiError(request, environmentIdResult.error, auditLog);
-        }
+      if (!environmentIdResult.ok) {
+        return handleApiError(request, environmentIdResult.error, auditLog);
       }
 
       if (!hasPermission(authentication.environmentPermissions, body.environmentId, "POST")) {

apps/web/modules/api/v2/openapi-document.ts

@@ -66,43 +66,43 @@ const document = createDocument({
       description: "Operations for managing your API key.",
     },
     {
-      name: "Management API - Responses",
+      name: "Management API > Responses",
       description: "Operations for managing responses.",
     },
     {
-      name: "Management API - Contacts",
+      name: "Management API > Contacts",
       description: "Operations for managing contacts.",
     },
     {
-      name: "Management API - Contact Attributes",
+      name: "Management API > Contact Attributes",
       description: "Operations for managing contact attributes.",
     },
     {
-      name: "Management API - Contact Attribute Keys",
-      description: "Operations for managing contact attribute keys.",
+      name: "Management API > Contact Attributes Keys",
+      description: "Operations for managing contact attributes keys.",
     },
     {
-      name: "Management API - Surveys",
+      name: "Management API > Surveys",
       description: "Operations for managing surveys.",
     },
     {
-      name: "Management API - Surveys - Contact Links",
+      name: "Management API > Surveys > Contact Links",
       description: "Operations for generating personalized survey links for contacts.",
     },
     {
-      name: "Management API - Webhooks",
+      name: "Management API > Webhooks",
       description: "Operations for managing webhooks.",
     },
     {
-      name: "Organizations API - Teams",
+      name: "Organizations API > Teams",
       description: "Operations for managing teams.",
     },
     {
-      name: "Organizations API - Project Teams",
+      name: "Organizations API > Project Teams",
       description: "Operations for managing project teams.",
     },
     {
-      name: "Organizations API - Users",
+      name: "Organizations API > Users",
       description: "Operations for managing users.",
     },
   ],

apps/web/modules/api/v2/organizations/[organizationId]/project-teams/lib/openapi.ts

@@ -20,7 +20,7 @@ export const getProjectTeamsEndpoint: ZodOpenApiOperationObject = {
       organizationId: ZOrganizationIdSchema,
     }),
   },
-  tags: ["Organizations API - Project Teams"],
+  tags: ["Organizations API > Project Teams"],
   responses: {
     "200": {
       description: "Project teams retrieved successfully.",
@@ -42,7 +42,7 @@ export const createProjectTeamEndpoint: ZodOpenApiOperationObject = {
       organizationId: ZOrganizationIdSchema,
     }),
   },
-  tags: ["Organizations API - Project Teams"],
+  tags: ["Organizations API > Project Teams"],
   requestBody: {
     required: true,
     description: "The project team to create",
@@ -68,7 +68,7 @@ export const deleteProjectTeamEndpoint: ZodOpenApiOperationObject = {
   operationId: "deleteProjectTeam",
   summary: "Delete a project team",
   description: "Deletes a project team from the database.",
-  tags: ["Organizations API - Project Teams"],
+  tags: ["Organizations API > Project Teams"],
   requestParams: {
     query: ZGetProjectTeamUpdateFilter.required(),
     path: z.object({
@@ -91,7 +91,7 @@ export const updateProjectTeamEndpoint: ZodOpenApiOperationObject = {
   operationId: "updateProjectTeam",
   summary: "Update a project team",
   description: "Updates a project team in the database.",
-  tags: ["Organizations API - Project Teams"],
+  tags: ["Organizations API > Project Teams"],
   requestParams: {
     path: z.object({
       organizationId: ZOrganizationIdSchema,

apps/web/modules/api/v2/organizations/[organizationId]/teams/[teamId]/lib/openapi.ts

@@ -16,7 +16,7 @@ export const getTeamEndpoint: ZodOpenApiOperationObject = {
       organizationId: ZOrganizationIdSchema,
     }),
   },
-  tags: ["Organizations API - Teams"],
+  tags: ["Organizations API > Teams"],
   responses: {
     "200": {
       description: "Team retrieved successfully.",
@@ -33,7 +33,7 @@ export const deleteTeamEndpoint: ZodOpenApiOperationObject = {
   operationId: "deleteTeam",
   summary: "Delete a team",
   description: "Deletes a team from the database.",
-  tags: ["Organizations API - Teams"],
+  tags: ["Organizations API > Teams"],
   requestParams: {
     path: z.object({
       id: ZTeamIdSchema,
@@ -56,7 +56,7 @@ export const updateTeamEndpoint: ZodOpenApiOperationObject = {
   operationId: "updateTeam",
   summary: "Update a team",
   description: "Updates a team in the database.",
-  tags: ["Organizations API - Teams"],
+  tags: ["Organizations API > Teams"],
   requestParams: {
     path: z.object({
       id: ZTeamIdSchema,

apps/web/modules/api/v2/organizations/[organizationId]/teams/[teamId]/lib/teams.ts

@@ -1,6 +1,7 @@
 import { ZTeamUpdateSchema } from "@/modules/api/v2/organizations/[organizationId]/teams/[teamId]/types/teams";
 import { ApiErrorResponseV2 } from "@/modules/api/v2/types/api-error";
-import { Prisma, Team } from "@prisma/client";
+import { Team } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { cache as reactCache } from "react";
 import { z } from "zod";
 import { prisma } from "@formbricks/database";
@@ -50,7 +51,7 @@ export const deleteTeam = async (
 
     return ok(deletedTeam);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist
@@ -88,7 +89,7 @@ export const updateTeam = async (
 
     return ok(updatedTeam);
   } catch (error) {
-    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+    if (error instanceof PrismaClientKnownRequestError) {
       if (
         error.code === PrismaErrorType.RecordDoesNotExist ||
         error.code === PrismaErrorType.RelatedRecordDoesNotExist

apps/web/modules/api/v2/organizations/[organizationId]/teams/[teamId]/lib/tests/teams.test.ts

@@ -1,4 +1,4 @@
-import { Prisma } from "@prisma/client";
+import { PrismaClientKnownRequestError } from "@prisma/client/runtime/library";
 import { describe, expect, test, vi } from "vitest";
 import { prisma } from "@formbricks/database";
 import { PrismaErrorType } from "@formbricks/database/types/error";
@@ -74,7 +74,7 @@ describe("Teams Lib", () => {
 
     test("returns not_found error on known prisma error", async () => {
       (prisma.team.delete as any).mockRejectedValueOnce(
-        new Prisma.PrismaClientKnownRequestError("Not found", {
+        new PrismaClientKnownRequestError("Not found", {
           code: PrismaErrorType.RecordDoesNotExist,
           clientVersion: "1.0.0",
           meta: {},
@@ -120,7 +120,7 @@ describe("Teams Lib", () => {
 
     test("returns not_found error when update fails due to missing team", async () => {
       (prisma.team.update as any).mockRejectedValueOnce(
-        new Prisma.PrismaClientKnownRequestError("Not found", {
+        new PrismaClientKnownRequestError("Not found", {
           code: PrismaErrorType.RecordDoesNotExist,
           clientVersion: "1.0.0",
           meta: {},

apps/web/modules/api/v2/organizations/[organizationId]/teams/lib/openapi.ts

@@ -24,7 +24,7 @@ export const getTeamsEndpoint: ZodOpenApiOperationObject = {
     }),
     query: ZGetTeamsFilter.sourceType(),
   },
-  tags: ["Organizations API - Teams"],
+  tags: ["Organizations API > Teams"],
   responses: {
     "200": {
       description: "Teams retrieved successfully.",
@@ -46,7 +46,7 @@ export const createTeamEndpoint: ZodOpenApiOperationObject = {
       organizationId: ZOrganizationIdSchema,
     }),
   },
-  tags: ["Organizations API - Teams"],
+  tags: ["Organizations API > Teams"],
   requestBody: {
     required: true,
     description: "The team to create",

apps/web/modules/api/v2/organizations/[organizationId]/users/lib/openapi.ts

@@ -20,7 +20,7 @@ export const getUsersEndpoint: ZodOpenApiOperationObject = {
     }),
     query: ZGetUsersFilter.sourceType(),
   },
-  tags: ["Organizations API - Users"],
+  tags: ["Organizations API > Users"],
   responses: {
     "200": {
       description: "Users retrieved successfully.",
@@ -42,7 +42,7 @@ export const createUserEndpoint: ZodOpenApiOperationObject = {
       organizationId: ZOrganizationIdSchema,
     }),
   },
-  tags: ["Organizations API - Users"],
+  tags: ["Organizations API > Users"],
   requestBody: {
     required: true,
     description: "The user to create",
@@ -73,7 +73,7 @@ export const updateUserEndpoint: ZodOpenApiOperationObject = {
       organizationId: ZOrganizationIdSchema,
     }),
   },
-  tags: ["Organizations API - Users"],
+  tags: ["Organizations API > Users"],
   requestBody: {
     required: true,
     description: "The user to update",

apps/web/modules/auth/actions/sign-out.ts

@@ -13,13 +13,7 @@ export const logSignOutAction = async (
   userId: string,
   userEmail: string,
   context: {
-    reason?:
-      | "user_initiated"
-      | "account_deletion"
-      | "email_change"
-      | "session_timeout"
-      | "forced_logout"
-      | "password_reset";
+    reason?: "user_initiated" | "account_deletion" | "email_change" | "session_timeout" | "forced_logout";
     redirectUrl?: string;
     organizationId?: string;
   }

apps/web/modules/auth/forgot-password/actions.ts

@@ -1,11 +1,9 @@
 "use server";
 
-import { PASSWORD_RESET_DISABLED } from "@/lib/constants";
 import { actionClient } from "@/lib/utils/action-client";
 import { getUserByEmail } from "@/modules/auth/lib/user";
 import { sendForgotPasswordEmail } from "@/modules/email";
 import { z } from "zod";
-import { OperationNotAllowedError } from "@formbricks/types/errors";
 import { ZUserEmail } from "@formbricks/types/user";
 
 const ZForgotPasswordAction = z.object({
@@ -15,15 +13,9 @@ const ZForgotPasswordAction = z.object({
 export const forgotPasswordAction = actionClient
   .schema(ZForgotPasswordAction)
   .action(async ({ parsedInput }) => {
-    if (PASSWORD_RESET_DISABLED) {
-      throw new OperationNotAllowedError("Password reset is disabled");
-    }
-
     const user = await getUserByEmail(parsedInput.email);
-
-    if (user && user.identityProvider === "email") {
+    if (user) {
       await sendForgotPasswordEmail(user);
     }
-
     return { success: true };
   });

apps/web/modules/auth/hooks/use-sign-out.ts

@@ -1,21 +1,13 @@
-import { FORMBRICKS_ENVIRONMENT_ID_LS } from "@/lib/localStorage";
 import { logSignOutAction } from "@/modules/auth/actions/sign-out";
 import { signOut } from "next-auth/react";
 import { logger } from "@formbricks/logger";
 
 interface UseSignOutOptions {
-  reason?:
-    | "user_initiated"
-    | "account_deletion"
-    | "email_change"
-    | "session_timeout"
-    | "forced_logout"
-    | "password_reset";
+  reason?: "user_initiated" | "account_deletion" | "email_change" | "session_timeout" | "forced_logout";
   redirectUrl?: string;
   organizationId?: string;
   redirect?: boolean;
   callbackUrl?: string;
-  clearEnvironmentId?: boolean;
 }
 
 interface SessionUser {
@@ -44,10 +36,6 @@ export const useSignOut = (sessionUser?: SessionUser | null) => {
       }
     }
 
-    if (options?.clearEnvironmentId) {
-      localStorage.removeItem(FORMBRICKS_ENVIRONMENT_ID_LS);
-    }
-
     // Call NextAuth signOut
     return await signOut({
       redirect: options?.redirect,

apps/web/modules/auth/lib/user.ts

@@ -78,7 +78,6 @@ export const getUserByEmail = reactCache(async (email: string) => {
         email: true,
         emailVerified: true,
         isActive: true,
-        identityProvider: true,
       },
     });
 

apps/web/modules/auth/lib/utils.ts

@@ -283,13 +283,7 @@ export const logSignOut = (
   userId: string,
   userEmail: string,
   context?: {
-    reason?:
-      | "user_initiated"
-      | "account_deletion"
-      | "email_change"
-      | "session_timeout"
-      | "forced_logout"
-      | "password_reset";
+    reason?: "user_initiated" | "account_deletion" | "email_change" | "session_timeout" | "forced_logout";
     redirectUrl?: string;
     organizationId?: string;
   }

apps/web/modules/auth/verify-email-change/components/email-change-sign-in.test.tsx

@@ -48,9 +48,7 @@ describe("EmailChangeSignIn", () => {
       expect(screen.getByText("auth.email-change.email_change_success_description")).toBeInTheDocument();
     });
 
-    await waitFor(() => {
-      expect(signOut).toHaveBeenCalledWith({ redirect: false });
-    });
+    expect(signOut).toHaveBeenCalledWith({ redirect: false });
   });
 
   test("handles failed email change verification", async () => {

apps/web/modules/ee/audit-logs/types/audit-log.ts

@@ -50,7 +50,6 @@ export const ZAuditAction = z.enum([
   "twoFactorRequired",
   "emailVerificationAttempted",
   "userSignedOut",
-  "passwordReset",
 ]);
 export const ZActor = z.enum(["user", "api", "system"]);
 export const ZAuditStatus = z.enum(["success", "failure"]);

apps/web/modules/ee/contacts/api/v1/client/[environmentId]/user/lib/segments.ts

@@ -34,8 +34,8 @@ export const getSegments = reactCache((environmentId: string) =>
     },
     {
       key: createCacheKey.environment.segments(environmentId),
-      // This is a temporary fix for the invalidation issues, will be changed later with a proper solution
-      ttl: 5 * 60 * 1000, // 5 minutes in milliseconds
+      // 30 minutes TTL - segment definitions change infrequently
+      ttl: 60 * 30 * 1000, // 30 minutes in milliseconds
     }
   )()
 );

apps/web/modules/ee/contacts/api/v1/management/contact-attribute-keys/[contactAttributeKeyId]/lib/contact-attribute-key.test.ts

@@ -2,9 +2,10 @@ import { ContactAttributeKey, Prisma } from "@prisma/client";
 import { beforeEach, describe, expect, test, vi } from "vitest";
 import { prisma } from "@formbricks/database";
 import { TContactAttributeKey, TContactAttributeKeyType } from "@formbricks/types/contact-attribute-key";
-import { DatabaseError } from "@formbricks/types/errors";
+import { DatabaseError, OperationNotAllowedError } from "@formbricks/types/errors";
 import { TContactAttributeKeyUpdateInput } from "../types/contact-attribute-keys";
 import {
+  createContactAttributeKey,
   deleteContactAttributeKey,
   getContactAttributeKey,
   updateContactAttributeKey,
@@ -100,6 +101,79 @@ describe("getContactAttributeKey", () => {
   });
 });
 
+describe("createContactAttributeKey", () => {
+  const type: TContactAttributeKeyType = "custom";
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  test("should create and return a new contact attribute key", async () => {
+    const createdAttributeKey = { ...mockContactAttributeKey, id: "new_cak_id", key: mockKey, type };
+    vi.mocked(prisma.contactAttributeKey.count).mockResolvedValue(5); // Below limit
+    vi.mocked(prisma.contactAttributeKey.create).mockResolvedValue(createdAttributeKey);
+
+    const result = await createContactAttributeKey(mockEnvironmentId, mockKey, type);
+
+    expect(result).toEqual(createdAttributeKey);
+    expect(prisma.contactAttributeKey.count).toHaveBeenCalledWith({
+      where: { environmentId: mockEnvironmentId },
+    });
+    expect(prisma.contactAttributeKey.create).toHaveBeenCalledWith({
+      data: {
+        key: mockKey,
+        name: mockKey, // As per implementation
+        type,
+        environment: { connect: { id: mockEnvironmentId } },
+      },
+    });
+  });
+
+  test("should throw OperationNotAllowedError if max attribute classes reached", async () => {
+    // MAX_ATTRIBUTE_CLASSES_PER_ENVIRONMENT is mocked to 10
+    vi.mocked(prisma.contactAttributeKey.count).mockResolvedValue(10);
+
+    await expect(createContactAttributeKey(mockEnvironmentId, mockKey, type)).rejects.toThrow(
+      OperationNotAllowedError
+    );
+    expect(prisma.contactAttributeKey.count).toHaveBeenCalledWith({
+      where: { environmentId: mockEnvironmentId },
+    });
+    expect(prisma.contactAttributeKey.create).not.toHaveBeenCalled();
+  });
+
+  test("should throw Prisma error if prisma.contactAttributeKey.count fails", async () => {
+    const errorMessage = "Prisma count error";
+    const prismaError = new Prisma.PrismaClientKnownRequestError(errorMessage, {
+      code: "P1000",
+      clientVersion: "test",
+    });
+    vi.mocked(prisma.contactAttributeKey.count).mockRejectedValue(prismaError);
+
+    await expect(createContactAttributeKey(mockEnvironmentId, mockKey, type)).rejects.toThrow(prismaError);
+  });
+
+  test("should throw DatabaseError if Prisma create fails", async () => {
+    vi.mocked(prisma.contactAttributeKey.count).mockResolvedValue(5); // Below limit
+    const errorMessage = "Prisma create error";
+    vi.mocked(prisma.contactAttributeKey.create).mockRejectedValue(
+      new Prisma.PrismaClientKnownRequestError(errorMessage, { code: "P2000", clientVersion: "test" })
+    );
+
+    await expect(createContactAttributeKey(mockEnvironmentId, mockKey, type)).rejects.toThrow(DatabaseError);
+    await expect(createContactAttributeKey(mockEnvironmentId, mockKey, type)).rejects.toThrow(errorMessage);
+  });
+
+  test("should throw generic error if non-Prisma error occurs during create", async () => {
+    vi.mocked(prisma.contactAttributeKey.count).mockResolvedValue(5);
+    const errorMessage = "Some other error during create";
+    vi.mocked(prisma.contactAttributeKey.create).mockRejectedValue(new Error(errorMessage));
+
+    await expect(createContactAttributeKey(mockEnvironmentId, mockKey, type)).rejects.toThrow(Error);
+    await expect(createContactAttributeKey(mockEnvironmentId, mockKey, type)).rejects.toThrow(errorMessage);
+  });
+});
+
 describe("deleteContactAttributeKey", () => {
   beforeEach(() => {
     vi.clearAllMocks();

apps/web/modules/ee/contacts/api/v1/management/contact-attribute-keys/[contactAttributeKeyId]/lib/contact-attribute-key.ts

@@ -1,10 +1,15 @@
+import { MAX_ATTRIBUTE_CLASSES_PER_ENVIRONMENT } from "@/lib/constants";
 import { validateInputs } from "@/lib/utils/validate";
 import { Prisma } from "@prisma/client";
 import { cache as reactCache } from "react";
 import { prisma } from "@formbricks/database";
-import { ZId } from "@formbricks/types/common";
-import { TContactAttributeKey } from "@formbricks/types/contact-attribute-key";
-import { DatabaseError } from "@formbricks/types/errors";
+import { ZId, ZString } from "@formbricks/types/common";
+import {
+  TContactAttributeKey,
+  TContactAttributeKeyType,
+  ZContactAttributeKeyType,
+} from "@formbricks/types/contact-attribute-key";
+import { DatabaseError, OperationNotAllowedError } from "@formbricks/types/errors";
 import {
   TContactAttributeKeyUpdateInput,
   ZContactAttributeKeyUpdateInput,
@@ -29,6 +34,48 @@ export const getContactAttributeKey = reactCache(
   }
 );
 
+export const createContactAttributeKey = async (
+  environmentId: string,
+  key: string,
+  type: TContactAttributeKeyType
+): Promise<TContactAttributeKey | null> => {
+  validateInputs([environmentId, ZId], [key, ZString], [type, ZContactAttributeKeyType]);
+
+  const contactAttributeKeysCount = await prisma.contactAttributeKey.count({
+    where: {
+      environmentId,
+    },
+  });
+
+  if (contactAttributeKeysCount >= MAX_ATTRIBUTE_CLASSES_PER_ENVIRONMENT) {
+    throw new OperationNotAllowedError(
+      `Maximum number of attribute classes (${MAX_ATTRIBUTE_CLASSES_PER_ENVIRONMENT}) reached for environment ${environmentId}`
+    );
+  }
+
+  try {
+    const contactAttributeKey = await prisma.contactAttributeKey.create({
+      data: {
+        key,
+        name: key,
+        type,
+        environment: {
+          connect: {
+            id: environmentId,
+          },
+        },
+      },
+    });
+
+    return contactAttributeKey;
+  } catch (error) {
+    if (error instanceof Prisma.PrismaClientKnownRequestError) {
+      throw new DatabaseError(error.message);
+    }
+    throw error;
+  }
+};
+
 export const deleteContactAttributeKey = async (
   contactAttributeKeyId: string
 ): Promise<TContactAttributeKey> => {
@@ -63,8 +110,6 @@ export const updateContactAttributeKey = async (
       },
       data: {
         description: data.description,
-        name: data.name,
-        key: data.key,
       },
     });
 

apps/web/modules/ee/contacts/api/v1/management/contact-attribute-keys/[contactAttributeKeyId]/types/contact-attribute-keys.ts

@@ -5,14 +5,12 @@ export const ZContactAttributeKeyCreateInput = z.object({
   description: z.string().optional(),
   type: z.enum(["custom"]),
   environmentId: z.string(),
-  name: z.string().optional(),
 });
 export type TContactAttributeKeyCreateInput = z.infer<typeof ZContactAttributeKeyCreateInput>;
 
 export const ZContactAttributeKeyUpdateInput = z.object({
   description: z.string().optional(),
   name: z.string().optional(),
-  key: z.string().optional(),
 });
 
 export type TContactAttributeKeyUpdateInput = z.infer<typeof ZContactAttributeKeyUpdateInput>;

apps/web/modules/ee/contacts/api/v1/management/contact-attribute-keys/lib/contact-attribute-keys.test.ts

@@ -1,5 +1,4 @@
 import { MAX_ATTRIBUTE_CLASSES_PER_ENVIRONMENT } from "@/lib/constants";
-import { TContactAttributeKeyCreateInput } from "@/modules/ee/contacts/api/v1/management/contact-attribute-keys/[contactAttributeKeyId]/types/contact-attribute-keys";
 import { Prisma } from "@prisma/client";
 import { beforeEach, describe, expect, test, vi } from "vitest";
 import { prisma } from "@formbricks/database";
@@ -28,28 +27,8 @@ describe("getContactAttributeKeys", () => {
   test("should return contact attribute keys when found", async () => {
     const mockEnvironmentIds = ["env1", "env2"];
     const mockAttributeKeys = [
-      {
-        id: "key1",
-        environmentId: "env1",
-        name: "Key One",
-        key: "keyOne",
-        type: "custom" as TContactAttributeKeyType,
-        createdAt: new Date(),
-        updatedAt: new Date(),
-        description: null,
-        isUnique: false,
-      },
-      {
-        id: "key2",
-        environmentId: "env2",
-        name: "Key Two",
-        key: "keyTwo",
-        type: "custom" as TContactAttributeKeyType,
-        createdAt: new Date(),
-        updatedAt: new Date(),
-        description: null,
-        isUnique: false,
-      },
+      { id: "key1", environmentId: "env1", name: "Key One", key: "keyOne", type: "custom" },
+      { id: "key2", environmentId: "env2", name: "Key Two", key: "keyTwo", type: "custom" },
     ];
     vi.mocked(prisma.contactAttributeKey.findMany).mockResolvedValue(mockAttributeKeys);
 
@@ -100,31 +79,25 @@ describe("createContactAttributeKey", () => {
     description: null,
   };
 
-  const createInput: TContactAttributeKeyCreateInput = {
-    key,
-    type,
-    environmentId,
-    name: key,
-    description: "",
-  };
-
   beforeEach(() => {
     vi.clearAllMocks();
   });
 
   test("should create and return a new contact attribute key", async () => {
     vi.mocked(prisma.contactAttributeKey.count).mockResolvedValue(0);
-    vi.mocked(prisma.contactAttributeKey.create).mockResolvedValue(mockCreatedAttributeKey);
+    vi.mocked(prisma.contactAttributeKey.create).mockResolvedValue({
+      ...mockCreatedAttributeKey,
+      description: null, // ensure description is explicitly null if that's the case
+    });
 
-    const result = await createContactAttributeKey(environmentId, createInput);
+    const result = await createContactAttributeKey(environmentId, key, type);
 
     expect(prisma.contactAttributeKey.count).toHaveBeenCalledWith({ where: { environmentId } });
     expect(prisma.contactAttributeKey.create).toHaveBeenCalledWith({
       data: {
-        key: createInput.key,
-        name: createInput.name || createInput.key,
-        type: createInput.type,
-        description: createInput.description || "",
+        key,
+        name: key,
+        type,
         environment: { connect: { id: environmentId } },
       },
     });
@@ -134,7 +107,7 @@ describe("createContactAttributeKey", () => {
   test("should throw OperationNotAllowedError if max attribute classes reached", async () => {
     vi.mocked(prisma.contactAttributeKey.count).mockResolvedValue(MAX_ATTRIBUTE_CLASSES_PER_ENVIRONMENT);
 
-    await expect(createContactAttributeKey(environmentId, createInput)).rejects.toThrow(
+    await expect(createContactAttributeKey(environmentId, key, type)).rejects.toThrow(
       OperationNotAllowedError
     );
     expect(prisma.contactAttributeKey.count).toHaveBeenCalledWith({ where: { environmentId } });
@@ -148,8 +121,8 @@ describe("createContactAttributeKey", () => {
       new Prisma.PrismaClientKnownRequestError(errorMessage, { code: "P2000", clientVersion: "test" })
     );
 
-    await expect(createContactAttributeKey(environmentId, createInput)).rejects.toThrow(DatabaseError);
-    await expect(createContactAttributeKey(environmentId, createInput)).rejects.toThrow(errorMessage);
+    await expect(createContactAttributeKey(environmentId, key, type)).rejects.toThrow(DatabaseError);
+    await expect(createContactAttributeKey(environmentId, key, type)).rejects.toThrow(errorMessage);
   });
 
   test("should throw generic error if non-Prisma error occurs during create", async () => {
@@ -157,55 +130,7 @@ describe("createContactAttributeKey", () => {
     const errorMessage = "Some other create error";
     vi.mocked(prisma.contactAttributeKey.create).mockRejectedValue(new Error(errorMessage));
 
-    await expect(createContactAttributeKey(environmentId, createInput)).rejects.toThrow(Error);
-    await expect(createContactAttributeKey(environmentId, createInput)).rejects.toThrow(errorMessage);
-  });
-
-  test("should use key as name when name is not provided", async () => {
-    vi.mocked(prisma.contactAttributeKey.count).mockResolvedValue(0);
-    vi.mocked(prisma.contactAttributeKey.create).mockResolvedValue(mockCreatedAttributeKey);
-
-    const inputWithoutName: TContactAttributeKeyCreateInput = {
-      key,
-      type,
-      environmentId,
-      description: "",
-    };
-
-    await createContactAttributeKey(environmentId, inputWithoutName);
-
-    expect(prisma.contactAttributeKey.create).toHaveBeenCalledWith({
-      data: {
-        key: inputWithoutName.key,
-        name: inputWithoutName.key, // Should fall back to key when name is not provided
-        type: inputWithoutName.type,
-        description: inputWithoutName.description || "",
-        environment: { connect: { id: environmentId } },
-      },
-    });
-  });
-
-  test("should use empty string for description when description is not provided", async () => {
-    vi.mocked(prisma.contactAttributeKey.count).mockResolvedValue(0);
-    vi.mocked(prisma.contactAttributeKey.create).mockResolvedValue(mockCreatedAttributeKey);
-
-    const inputWithoutDescription: TContactAttributeKeyCreateInput = {
-      key,
-      type,
-      environmentId,
-      name: "Test Name",
-    };
-
-    await createContactAttributeKey(environmentId, inputWithoutDescription);
-
-    expect(prisma.contactAttributeKey.create).toHaveBeenCalledWith({
-      data: {
-        key: inputWithoutDescription.key,
-        name: inputWithoutDescription.name,
-        type: inputWithoutDescription.type,
-        description: "", // Should fall back to empty string when description is not provided
-        environment: { connect: { id: environmentId } },
-      },
-    });
+    await expect(createContactAttributeKey(environmentId, key, type)).rejects.toThrow(Error);
+    await expect(createContactAttributeKey(environmentId, key, type)).rejects.toThrow(errorMessage);
   });
 });

apps/web/modules/ee/contacts/api/v1/management/contact-attribute-keys/lib/contact-attribute-keys.ts

@@ -1,10 +1,14 @@
 import { MAX_ATTRIBUTE_CLASSES_PER_ENVIRONMENT } from "@/lib/constants";
-import { TContactAttributeKeyCreateInput } from "@/modules/ee/contacts/api/v1/management/contact-attribute-keys/[contactAttributeKeyId]/types/contact-attribute-keys";
+import { validateInputs } from "@/lib/utils/validate";
 import { Prisma } from "@prisma/client";
 import { cache as reactCache } from "react";
 import { prisma } from "@formbricks/database";
-import { PrismaErrorType } from "@formbricks/database/types/error";
-import { TContactAttributeKey } from "@formbricks/types/contact-attribute-key";
+import { ZId, ZString } from "@formbricks/types/common";
+import {
+  TContactAttributeKey,
+  TContactAttributeKeyType,
+  ZContactAttributeKeyType,
+} from "@formbricks/types/contact-attribute-key";
 import { DatabaseError, OperationNotAllowedError } from "@formbricks/types/errors";
 
 export const getContactAttributeKeys = reactCache(
@@ -26,8 +30,11 @@ export const getContactAttributeKeys = reactCache(
 
 export const createContactAttributeKey = async (
   environmentId: string,
-  data: TContactAttributeKeyCreateInput
+  key: string,
+  type: TContactAttributeKeyType
 ): Promise<TContactAttributeKey | null> => {
+  validateInputs([environmentId, ZId], [key, ZString], [type, ZContactAttributeKeyType]);
+
   const contactAttributeKeysCount = await prisma.contactAttributeKey.count({
     where: {
       environmentId,
@@ -43,10 +50,9 @@ export const createContactAttributeKey = async (
   try {
     const contactAttributeKey = await prisma.contactAttributeKey.create({
       data: {
-        key: data.key,
-        name: data.name ?? data.key,
-        type: data.type,
-        description: data.description ?? "",
+        key,
+        name: key,
+        type,
         environment: {
           connect: {
             id: environmentId,
@@ -58,10 +64,6 @@ export const createContactAttributeKey = async (
     return contactAttributeKey;
   } catch (error) {
     if (error instanceof Prisma.PrismaClientKnownRequestError) {
-      if (error.code === PrismaErrorType.UniqueConstraintViolation) {
-        throw new DatabaseError("Attribute key already exists");
-      }
-
       throw new DatabaseError(error.message);
     }
     throw error;