Remove feedback module imports and router inclusion

frontend/__tests__/components/feedback-actions.test.tsx

@@ -1,76 +0,0 @@
-import { screen, within } from "@testing-library/react";
-import userEvent from "@testing-library/user-event";
-import { afterEach, describe, expect, it, vi } from "vitest";
-import { renderWithProviders } from "test-utils";
-import { TrajectoryActions } from "#/components/features/trajectory/trajectory-actions";
-
-describe("TrajectoryActions", () => {
-  const user = userEvent.setup();
-  const onPositiveFeedback = vi.fn();
-  const onNegativeFeedback = vi.fn();
-  const onExportTrajectory = vi.fn();
-
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it("should render correctly", () => {
-    renderWithProviders(
-      <TrajectoryActions
-        onPositiveFeedback={onPositiveFeedback}
-        onNegativeFeedback={onNegativeFeedback}
-        onExportTrajectory={onExportTrajectory}
-      />,
-    );
-
-    const actions = screen.getByTestId("feedback-actions");
-    within(actions).getByTestId("positive-feedback");
-    within(actions).getByTestId("negative-feedback");
-    within(actions).getByTestId("export-trajectory");
-  });
-
-  it("should call onPositiveFeedback when positive feedback is clicked", async () => {
-    renderWithProviders(
-      <TrajectoryActions
-        onPositiveFeedback={onPositiveFeedback}
-        onNegativeFeedback={onNegativeFeedback}
-        onExportTrajectory={onExportTrajectory}
-      />,
-    );
-
-    const positiveFeedback = screen.getByTestId("positive-feedback");
-    await user.click(positiveFeedback);
-
-    expect(onPositiveFeedback).toHaveBeenCalled();
-  });
-
-  it("should call onNegativeFeedback when negative feedback is clicked", async () => {
-    renderWithProviders(
-      <TrajectoryActions
-        onPositiveFeedback={onPositiveFeedback}
-        onNegativeFeedback={onNegativeFeedback}
-        onExportTrajectory={onExportTrajectory}
-      />,
-    );
-
-    const negativeFeedback = screen.getByTestId("negative-feedback");
-    await user.click(negativeFeedback);
-
-    expect(onNegativeFeedback).toHaveBeenCalled();
-  });
-
-  it("should call onExportTrajectory when export button is clicked", async () => {
-    renderWithProviders(
-      <TrajectoryActions
-        onPositiveFeedback={onPositiveFeedback}
-        onNegativeFeedback={onNegativeFeedback}
-        onExportTrajectory={onExportTrajectory}
-      />,
-    );
-
-    const exportButton = screen.getByTestId("export-trajectory");
-    await user.click(exportButton);
-
-    expect(onExportTrajectory).toHaveBeenCalled();
-  });
-});

frontend/__tests__/components/feedback-form.test.tsx

@@ -1,68 +0,0 @@
-import { afterEach, describe, expect, it, vi } from "vitest";
-
-// Mock useParams before importing components
-vi.mock("react-router", async () => {
-  const actual = await vi.importActual("react-router");
-  return {
-    ...(actual as object),
-    useParams: () => ({ conversationId: "test-conversation-id" }),
-  };
-});
-
-import { screen } from "@testing-library/react";
-import userEvent from "@testing-library/user-event";
-import { renderWithProviders } from "test-utils";
-import { FeedbackForm } from "#/components/features/feedback/feedback-form";
-import { I18nKey } from "#/i18n/declaration";
-
-describe("FeedbackForm", () => {
-  const user = userEvent.setup();
-  const onCloseMock = vi.fn();
-
-  afterEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it("should render correctly", () => {
-    renderWithProviders(
-      <FeedbackForm polarity="positive" onClose={onCloseMock} />,
-    );
-
-    screen.getByLabelText(I18nKey.FEEDBACK$EMAIL_LABEL);
-    screen.getByLabelText(I18nKey.FEEDBACK$PRIVATE_LABEL);
-    screen.getByLabelText(I18nKey.FEEDBACK$PUBLIC_LABEL);
-
-    screen.getByRole("button", { name: I18nKey.FEEDBACK$SHARE_LABEL });
-    screen.getByRole("button", { name: I18nKey.FEEDBACK$CANCEL_LABEL });
-  });
-
-  it("should switch between private and public permissions", async () => {
-    renderWithProviders(
-      <FeedbackForm polarity="positive" onClose={onCloseMock} />,
-    );
-    const privateRadio = screen.getByLabelText(I18nKey.FEEDBACK$PRIVATE_LABEL);
-    const publicRadio = screen.getByLabelText(I18nKey.FEEDBACK$PUBLIC_LABEL);
-
-    expect(privateRadio).toBeChecked(); // private is the default value
-    expect(publicRadio).not.toBeChecked();
-
-    await user.click(publicRadio);
-    expect(publicRadio).toBeChecked();
-    expect(privateRadio).not.toBeChecked();
-
-    await user.click(privateRadio);
-    expect(privateRadio).toBeChecked();
-    expect(publicRadio).not.toBeChecked();
-  });
-
-  it("should call onClose when the close button is clicked", async () => {
-    renderWithProviders(
-      <FeedbackForm polarity="positive" onClose={onCloseMock} />,
-    );
-    await user.click(
-      screen.getByRole("button", { name: I18nKey.FEEDBACK$CANCEL_LABEL }),
-    );
-
-    expect(onCloseMock).toHaveBeenCalled();
-  });
-});

frontend/src/api/open-hands.ts

@@ -1,7 +1,5 @@
 import { AxiosHeaders } from "axios";
 import {
-  Feedback,
-  FeedbackResponse,
   GitHubAccessTokenResponse,
   GetConfigResponse,
   GetVSCodeUrlResponse,
@@ -95,20 +93,6 @@ class OpenHands {
     return headers;
   }
 
-  /**
-   * Send feedback to the server
-   * @param data Feedback data
-   * @returns The stored feedback data
-   */
-  static async submitFeedback(
-    conversationId: string,
-    feedback: Feedback,
-  ): Promise<FeedbackResponse> {
-    const url = `/api/conversations/${conversationId}/submit-feedback`;
-    const { data } = await openHands.post<FeedbackResponse>(url, feedback);
-    return data;
-  }
-
   /**
    * Authenticate with GitHub token
    * @returns Response with authentication status and user info if successful

frontend/src/api/open-hands.types.ts

@@ -14,17 +14,6 @@ export interface FileUploadSuccessResponse {
   skipped_files: { name: string; reason: string }[];
 }
 
-export interface FeedbackBodyResponse {
-  message: string;
-  feedback_id: string;
-  password: string;
-}
-
-export interface FeedbackResponse {
-  statusCode: number;
-  body: FeedbackBodyResponse;
-}
-
 export interface GitHubAccessTokenResponse {
   access_token: string;
 }
@@ -34,15 +23,6 @@ export interface AuthenticationResponse {
   login?: string; // Only present when allow list is enabled
 }
 
-export interface Feedback {
-  version: string;
-  email: string;
-  token: string;
-  polarity: "positive" | "negative";
-  permissions: "public" | "private";
-  trajectory: unknown[];
-}
-
 export interface GetConfigResponse {
   APP_MODE: "saas" | "oss";
   APP_SLUG?: string;

frontend/src/components/features/chat/chat-interface.tsx

@@ -11,7 +11,6 @@ import { InteractiveChatBox } from "./interactive-chat-box";
 import { RootState } from "#/store";
 import { AgentState } from "#/types/agent-state";
 import { generateAgentStateChangeEvent } from "#/services/agent-state-service";
-import { FeedbackModal } from "../feedback/feedback-modal";
 import { useScrollToBottom } from "#/hooks/use-scroll-to-bottom";
 import { TypingIndicator } from "./typing-indicator";
 import { useWsClient } from "#/context/ws-client-provider";
@@ -50,10 +49,6 @@ export function ChatInterface() {
 
   const { curAgentState } = useSelector((state: RootState) => state.agent);
 
-  const [feedbackPolarity, setFeedbackPolarity] = React.useState<
-    "positive" | "negative"
-  >("positive");
-  const [feedbackModalIsOpen, setFeedbackModalIsOpen] = React.useState(false);
   const [messageToSend, setMessageToSend] = React.useState<string | null>(null);
   const { selectedRepository, replayJson } = useSelector(
     (state: RootState) => state.initialQuery,
@@ -96,13 +91,6 @@ export function ChatInterface() {
     send(generateAgentStateChangeEvent(AgentState.STOPPED));
   };
 
-  const onClickShareFeedbackActionButton = async (
-    polarity: "positive" | "negative",
-  ) => {
-    setFeedbackModalIsOpen(true);
-    setFeedbackPolarity(polarity);
-  };
-
   const onClickExportTrajectoryButton = () => {
     if (!params.conversationId) {
       displayErrorToast(t(I18nKey.CONVERSATION$DOWNLOAD_ERROR));
@@ -164,12 +152,6 @@ export function ChatInterface() {
       <div className="flex flex-col gap-[6px] px-4 pb-4">
         <div className="flex justify-between relative">
           <TrajectoryActions
-            onPositiveFeedback={() =>
-              onClickShareFeedbackActionButton("positive")
-            }
-            onNegativeFeedback={() =>
-              onClickShareFeedbackActionButton("negative")
-            }
             onExportTrajectory={() => onClickExportTrajectoryButton()}
           />
 
@@ -194,12 +176,6 @@ export function ChatInterface() {
           onChange={setMessageToSend}
         />
       </div>
-
-      <FeedbackModal
-        isOpen={feedbackModalIsOpen}
-        onClose={() => setFeedbackModalIsOpen(false)}
-        polarity={feedbackPolarity}
-      />
     </div>
   );
 }

frontend/src/components/features/feedback/feedback-form.tsx

@@ -1,152 +0,0 @@
-import React from "react";
-import hotToast from "react-hot-toast";
-import { useTranslation } from "react-i18next";
-import { I18nKey } from "#/i18n/declaration";
-import { Feedback } from "#/api/open-hands.types";
-import { useSubmitFeedback } from "#/hooks/mutation/use-submit-feedback";
-import { BrandButton } from "../settings/brand-button";
-
-const FEEDBACK_VERSION = "1.0";
-const VIEWER_PAGE = "https://www.all-hands.dev/share";
-
-interface FeedbackFormProps {
-  onClose: () => void;
-  polarity: "positive" | "negative";
-}
-
-export function FeedbackForm({ onClose, polarity }: FeedbackFormProps) {
-  const { t } = useTranslation();
-
-  const copiedToClipboardToast = () => {
-    hotToast(t(I18nKey.FEEDBACK$PASSWORD_COPIED_MESSAGE), {
-      icon: "📋",
-      position: "bottom-right",
-    });
-  };
-
-  const onPressToast = (password: string) => {
-    navigator.clipboard.writeText(password);
-    copiedToClipboardToast();
-  };
-
-  const shareFeedbackToast = (
-    message: string,
-    link: string,
-    password: string,
-  ) => {
-    hotToast(
-      <div className="flex flex-col gap-1">
-        <span>{message}</span>
-        <a
-          data-testid="toast-share-url"
-          className="text-blue-500 underline"
-          onClick={() => onPressToast(password)}
-          href={link}
-          target="_blank"
-          rel="noreferrer"
-        >
-          {t(I18nKey.FEEDBACK$GO_TO_FEEDBACK)}
-        </a>
-        <span onClick={() => onPressToast(password)} className="cursor-pointer">
-          {t(I18nKey.FEEDBACK$PASSWORD)}: {password}{" "}
-          <span className="text-gray-500">
-            ({t(I18nKey.FEEDBACK$COPY_LABEL)})
-          </span>
-        </span>
-      </div>,
-      { duration: 10000 },
-    );
-  };
-
-  const { mutate: submitFeedback, isPending } = useSubmitFeedback();
-
-  const handleSubmit = async (event: React.FormEvent<HTMLFormElement>) => {
-    event?.preventDefault();
-    const formData = new FormData(event.currentTarget);
-
-    const email = formData.get("email")?.toString() || "";
-    const permissions = (formData.get("permissions")?.toString() ||
-      "private") as "private" | "public";
-
-    const feedback: Feedback = {
-      version: FEEDBACK_VERSION,
-      email,
-      polarity,
-      permissions,
-      trajectory: [],
-      token: "",
-    };
-
-    submitFeedback(
-      { feedback },
-      {
-        onSuccess: (data) => {
-          const { message, feedback_id, password } = data.body; // eslint-disable-line
-          const link = `${VIEWER_PAGE}?share_id=${feedback_id}`;
-          shareFeedbackToast(message, link, password);
-          onClose();
-        },
-      },
-    );
-  };
-
-  return (
-    <form onSubmit={handleSubmit} className="flex flex-col gap-6 w-full">
-      <label className="flex flex-col gap-2">
-        <span className="text-xs text-neutral-400">
-          {t(I18nKey.FEEDBACK$EMAIL_LABEL)}
-        </span>
-        <input
-          required
-          name="email"
-          type="email"
-          placeholder={t(I18nKey.FEEDBACK$EMAIL_PLACEHOLDER)}
-          className="bg-[#27272A] px-3 py-[10px] rounded"
-        />
-      </label>
-
-      <div className="flex gap-4 text-neutral-400">
-        <label className="flex gap-2 cursor-pointer">
-          <input
-            name="permissions"
-            value="private"
-            type="radio"
-            defaultChecked
-          />
-          {t(I18nKey.FEEDBACK$PRIVATE_LABEL)}
-        </label>
-        <label className="flex gap-2 cursor-pointer">
-          <input name="permissions" value="public" type="radio" />
-          {t(I18nKey.FEEDBACK$PUBLIC_LABEL)}
-        </label>
-      </div>
-
-      <div className="flex gap-2">
-        <BrandButton
-          type="submit"
-          variant="primary"
-          className="grow"
-          isDisabled={isPending}
-        >
-          {isPending
-            ? t(I18nKey.FEEDBACK$SUBMITTING_LABEL)
-            : t(I18nKey.FEEDBACK$SHARE_LABEL)}
-        </BrandButton>
-        <BrandButton
-          type="button"
-          variant="secondary"
-          className="grow"
-          onClick={onClose}
-          isDisabled={isPending}
-        >
-          {t(I18nKey.FEEDBACK$CANCEL_LABEL)}
-        </BrandButton>
-      </div>
-      {isPending && (
-        <p className="text-sm text-center text-neutral-400">
-          {t(I18nKey.FEEDBACK$SUBMITTING_MESSAGE)}
-        </p>
-      )}
-    </form>
-  );
-}

frontend/src/components/features/feedback/feedback-modal.tsx

@@ -1,34 +0,0 @@
-import { useTranslation } from "react-i18next";
-import { I18nKey } from "#/i18n/declaration";
-import {
-  BaseModalTitle,
-  BaseModalDescription,
-} from "#/components/shared/modals/confirmation-modals/base-modal";
-import { ModalBackdrop } from "#/components/shared/modals/modal-backdrop";
-import { ModalBody } from "#/components/shared/modals/modal-body";
-import { FeedbackForm } from "./feedback-form";
-
-interface FeedbackModalProps {
-  onClose: () => void;
-  isOpen: boolean;
-  polarity: "positive" | "negative";
-}
-
-export function FeedbackModal({
-  onClose,
-  isOpen,
-  polarity,
-}: FeedbackModalProps) {
-  const { t } = useTranslation();
-  if (!isOpen) return null;
-
-  return (
-    <ModalBackdrop onClose={onClose}>
-      <ModalBody className="border border-tertiary">
-        <BaseModalTitle title={t(I18nKey.FEEDBACK$TITLE)} />
-        <BaseModalDescription description={t(I18nKey.FEEDBACK$DESCRIPTION)} />
-        <FeedbackForm onClose={onClose} polarity={polarity} />
-      </ModalBody>
-    </ModalBackdrop>
-  );
-}

frontend/src/components/features/trajectory/trajectory-actions.tsx

@@ -1,37 +1,19 @@
 import { useTranslation } from "react-i18next";
 import { I18nKey } from "#/i18n/declaration";
-import ThumbsUpIcon from "#/icons/thumbs-up.svg?react";
-import ThumbDownIcon from "#/icons/thumbs-down.svg?react";
 import ExportIcon from "#/icons/export.svg?react";
 import { TrajectoryActionButton } from "#/components/shared/buttons/trajectory-action-button";
 
 interface TrajectoryActionsProps {
-  onPositiveFeedback: () => void;
-  onNegativeFeedback: () => void;
   onExportTrajectory: () => void;
 }
 
 export function TrajectoryActions({
-  onPositiveFeedback,
-  onNegativeFeedback,
   onExportTrajectory,
 }: TrajectoryActionsProps) {
   const { t } = useTranslation();
 
   return (
-    <div data-testid="feedback-actions" className="flex gap-1">
-      <TrajectoryActionButton
-        testId="positive-feedback"
-        onClick={onPositiveFeedback}
-        icon={<ThumbsUpIcon width={15} height={15} />}
-        tooltip={t(I18nKey.BUTTON$MARK_HELPFUL)}
-      />
-      <TrajectoryActionButton
-        testId="negative-feedback"
-        onClick={onNegativeFeedback}
-        icon={<ThumbDownIcon width={15} height={15} />}
-        tooltip={t(I18nKey.BUTTON$MARK_NOT_HELPFUL)}
-      />
+    <div data-testid="trajectory-actions" className="flex gap-1">
       <TrajectoryActionButton
         testId="export-trajectory"
         onClick={onExportTrajectory}

frontend/src/hooks/mutation/use-submit-feedback.ts

@@ -1,22 +0,0 @@
-import { useMutation } from "@tanstack/react-query";
-import { Feedback } from "#/api/open-hands.types";
-import OpenHands from "#/api/open-hands";
-import { useConversationId } from "#/hooks/use-conversation-id";
-import { displayErrorToast } from "#/utils/custom-toast-handlers";
-
-type SubmitFeedbackArgs = {
-  feedback: Feedback;
-};
-
-export const useSubmitFeedback = () => {
-  const { conversationId } = useConversationId();
-  return useMutation({
-    mutationFn: ({ feedback }: SubmitFeedbackArgs) =>
-      OpenHands.submitFeedback(conversationId, feedback),
-    onError: (error) => {
-      displayErrorToast(error.message);
-    },
-    retry: 2,
-    retryDelay: 500,
-  });
-};

frontend/src/icons/thumbs-down.svg

@@ -1,5 +0,0 @@
-<svg width="15" height="16" viewBox="0 0 15 16" fill="none" xmlns="http://www.w3.org/2000/svg">
-  <path
-    d="M11.8749 1.75H3.91861C3.47998 1.75015 3.05528 1.90407 2.71841 2.18499C2.38154 2.4659 2.15382 2.85603 2.07486 3.2875L1.28111 7.6625C1.23166 7.93277 1.2422 8.21062 1.31201 8.47636C1.38182 8.74211 1.50917 8.98927 1.68507 9.20035C1.86097 9.41142 2.08111 9.58126 2.32991 9.69785C2.57872 9.81443 2.8501 9.87491 3.12486 9.875H5.97486L5.62486 10.7688C5.47928 11.1601 5.4308 11.5809 5.48357 11.995C5.53635 12.4092 5.68881 12.8044 5.92787 13.1467C6.16694 13.489 6.48547 13.7683 6.85615 13.9604C7.22683 14.1526 7.63859 14.2519 8.05611 14.25C8.17634 14.2497 8.29394 14.2148 8.39482 14.1494C8.4957 14.084 8.57557 13.9909 8.62486 13.8813L10.4061 9.875H11.8749C12.3721 9.875 12.8491 9.67746 13.2007 9.32583C13.5523 8.97419 13.7499 8.49728 13.7499 8V3.625C13.7499 3.12772 13.5523 2.65081 13.2007 2.29917C12.8491 1.94754 12.3721 1.75 11.8749 1.75ZM9.37486 9.11875L7.67486 12.9438C7.50092 12.8911 7.3396 12.8034 7.20083 12.6861C7.06206 12.5688 6.94878 12.4242 6.86798 12.2615C6.78717 12.0987 6.74055 11.9211 6.73099 11.7396C6.72143 11.5581 6.74912 11.3766 6.81236 11.2062L7.14361 10.3125C7.2142 10.1236 7.23803 9.92041 7.21307 9.72029C7.18811 9.52018 7.1151 9.32907 7.00028 9.16329C6.88546 8.9975 6.73223 8.86196 6.55367 8.76823C6.37511 8.67449 6.17653 8.62535 5.97486 8.625H3.12486C3.03304 8.62515 2.94232 8.60507 2.85914 8.56618C2.77597 8.52729 2.70238 8.47055 2.64361 8.4C2.58341 8.33042 2.5393 8.24841 2.51445 8.15982C2.4896 8.07123 2.48462 7.97824 2.49986 7.8875L3.29361 3.5125C3.32024 3.3669 3.39767 3.23548 3.51212 3.14162C3.62657 3.04777 3.77062 2.99759 3.91861 3H9.37486V9.11875ZM12.4999 8C12.4999 8.16576 12.434 8.32473 12.3168 8.44194C12.1996 8.55915 12.0406 8.625 11.8749 8.625H10.6249V3H11.8749C12.0406 3 12.1996 3.06585 12.3168 3.18306C12.434 3.30027 12.4999 3.45924 12.4999 3.625V8Z"
-    fill="white" />
-</svg>

frontend/src/icons/thumbs-up.svg

@@ -1,5 +0,0 @@
-<svg width="15" height="16" viewBox="0 0 15 16" fill="none" xmlns="http://www.w3.org/2000/svg">
-  <path
-    d="M13.3125 6.80003C13.1369 6.58918 12.9171 6.41945 12.6687 6.30282C12.4204 6.18619 12.1494 6.1255 11.875 6.12503H9.025L9.375 5.23128C9.52058 4.83995 9.56907 4.41916 9.51629 4.00498C9.46351 3.5908 9.31106 3.19561 9.07199 2.8533C8.83293 2.51099 8.51439 2.23178 8.14371 2.03962C7.77303 1.84746 7.36127 1.74809 6.94375 1.75003C6.82352 1.75028 6.70592 1.7852 6.60504 1.8506C6.50417 1.91601 6.42429 2.00912 6.375 2.11878L4.59375 6.12503H3.125C2.62772 6.12503 2.15081 6.32257 1.79917 6.6742C1.44754 7.02583 1.25 7.50275 1.25 8.00003V12.375C1.25 12.8723 1.44754 13.3492 1.79917 13.7009C2.15081 14.0525 2.62772 14.25 3.125 14.25H11.0812C11.5199 14.2499 11.9446 14.096 12.2815 13.815C12.6183 13.5341 12.846 13.144 12.925 12.7125L13.7188 8.33753C13.7678 8.06714 13.7569 7.78927 13.6867 7.52358C13.6165 7.25788 13.4887 7.01087 13.3125 6.80003ZM4.375 13H3.125C2.95924 13 2.80027 12.9342 2.68306 12.817C2.56585 12.6998 2.5 12.5408 2.5 12.375V8.00003C2.5 7.83427 2.56585 7.6753 2.68306 7.55809C2.80027 7.44088 2.95924 7.37503 3.125 7.37503H4.375V13ZM12.5 8.11253L11.7062 12.4875C11.6796 12.6331 11.6022 12.7646 11.4877 12.8584C11.3733 12.9523 11.2292 13.0024 11.0812 13H5.625V6.88128L7.325 3.05628C7.49999 3.10729 7.6625 3.19403 7.80229 3.31102C7.94207 3.428 8.05608 3.57269 8.13712 3.73596C8.21817 3.89923 8.26449 4.07752 8.27316 4.25959C8.28183 4.44166 8.25266 4.62355 8.1875 4.79378L7.85625 5.68753C7.78567 5.87644 7.76184 6.07962 7.7868 6.27973C7.81176 6.47985 7.88476 6.67095 7.99958 6.83674C8.11441 7.00253 8.26763 7.13807 8.44619 7.2318C8.62475 7.32554 8.82333 7.37468 9.025 7.37503H11.875C11.9668 7.37488 12.0575 7.39496 12.1407 7.43385C12.2239 7.47274 12.2975 7.52948 12.3563 7.60003C12.4165 7.66961 12.4606 7.75162 12.4854 7.84021C12.5103 7.9288 12.5152 8.02179 12.5 8.11253Z"
-    fill="white" />
-</svg>

openhands/server/app.py

@@ -15,7 +15,6 @@ from fastapi import (
 import openhands.agenthub  # noqa F401 (we import this to get the agents registered)
 from openhands import __version__
 from openhands.server.routes.conversation import app as conversation_api_router
-from openhands.server.routes.feedback import app as feedback_api_router
 from openhands.server.routes.files import app as files_api_router
 from openhands.server.routes.git import app as git_api_router
 from openhands.server.routes.health import add_health_endpoints
@@ -63,7 +62,6 @@ app = FastAPI(
 app.include_router(public_api_router)
 app.include_router(files_api_router)
 app.include_router(security_api_router)
-app.include_router(feedback_api_router)
 app.include_router(conversation_api_router)
 app.include_router(manage_conversation_api_router)
 app.include_router(settings_router)

openhands/server/data_models/feedback.py

@@ -1,45 +0,0 @@
-import json
-from typing import Any, Literal
-
-import httpx
-from pydantic import BaseModel
-
-from openhands.core.logger import openhands_logger as logger
-
-
-class FeedbackDataModel(BaseModel):
-    version: str
-    email: str
-    polarity: Literal['positive', 'negative']
-    feedback: Literal[
-        'positive', 'negative'
-    ]  # TODO: remove this, its here for backward compatibility
-    permissions: Literal['public', 'private']
-    trajectory: list[dict[str, Any]] | None
-
-
-FEEDBACK_URL = 'https://share-od-trajectory-3u9bw9tx.uc.gateway.dev/share_od_trajectory'
-
-
-def store_feedback(feedback: FeedbackDataModel) -> dict[str, str]:
-    # Start logging
-    feedback.feedback = feedback.polarity
-    display_feedback = feedback.model_dump()
-    if 'trajectory' in display_feedback:
-        display_feedback['trajectory'] = (
-            f'elided [length: {len(display_feedback["trajectory"])}'
-        )
-    if 'token' in display_feedback:
-        display_feedback['token'] = 'elided'
-    logger.debug(f'Got feedback: {display_feedback}')
-    # Start actual request
-    response = httpx.post(
-        FEEDBACK_URL,
-        headers={'Content-Type': 'application/json'},
-        json=feedback.model_dump(),
-    )
-    if response.status_code != 200:
-        raise ValueError(f'Failed to store feedback: {response.text}')
-    response_data: dict[str, str] = json.loads(response.text)
-    logger.debug(f'Stored feedback: {response.text}')
-    return response_data

openhands/server/routes/feedback.py

@@ -1,62 +0,0 @@
-from fastapi import APIRouter, Depends, Request, status
-from fastapi.responses import JSONResponse
-
-from openhands.core.logger import openhands_logger as logger
-from openhands.events.async_event_store_wrapper import AsyncEventStoreWrapper
-from openhands.events.serialization import event_to_dict
-from openhands.server.data_models.feedback import FeedbackDataModel, store_feedback
-from openhands.server.dependencies import get_dependencies
-from openhands.server.utils import get_conversation
-from openhands.utils.async_utils import call_sync_from_async
-from openhands.server.session.conversation import ServerConversation
-
-app = APIRouter(prefix='/api/conversations/{conversation_id}', dependencies=get_dependencies())
-
-
-@app.post('/submit-feedback')
-async def submit_feedback(request: Request, conversation: ServerConversation = Depends(get_conversation)) -> JSONResponse:
-    """Submit user feedback.
-
-    This function stores the provided feedback data.
-
-    To submit feedback:
-    ```sh
-    curl -X POST -d '{"email": "test@example.com"}' -H "Authorization:"
-    ```
-
-    Args:
-        request (Request): The incoming request object.
-        feedback (FeedbackDataModel): The feedback data to be stored.
-
-    Returns:
-        dict: The stored feedback data.
-
-    Raises:
-        HTTPException: If there's an error submitting the feedback.
-    """
-    # Assuming the storage service is already configured in the backend
-    # and there is a function to handle the storage.
-    body = await request.json()
-    async_store = AsyncEventStoreWrapper(
-        conversation.event_stream, filter_hidden=True
-    )
-    trajectory = []
-    async for event in async_store:
-        trajectory.append(event_to_dict(event))
-    feedback = FeedbackDataModel(
-        email=body.get('email', ''),
-        version=body.get('version', ''),
-        permissions=body.get('permissions', 'private'),
-        polarity=body.get('polarity', ''),
-        feedback=body.get('polarity', ''),
-        trajectory=trajectory,
-    )
-    try:
-        feedback_data = await call_sync_from_async(store_feedback, feedback)
-        return JSONResponse(status_code=status.HTTP_200_OK, content=feedback_data)
-    except Exception as e:
-        logger.error(f'Error submitting feedback: {e}')
-        return JSONResponse(
-            status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
-            content={'error': 'Failed to submit feedback'},
-        )