From 1bbb5a0f57058257da2e36a5f78c3c98c3648080 Mon Sep 17 00:00:00 2001
From: Harshith Mullapudi <harshithmullapudi@gmail.com>
Date: Fri, 11 Jul 2025 18:31:14 +0530
Subject: [PATCH] Fix: UI fixes for conversation

---
 .../components/conversation/conversation-list.tsx  |  2 +-
 .../conversation/conversation-textarea.client.tsx  |  4 ++--
 .../conversation/conversation.client.tsx           |  5 ++---
 .../components/conversation/editor-extensions.tsx  |  9 +--------
 apps/webapp/app/components/ui/header.tsx           |  2 +-
 apps/webapp/app/routes/api.v1.episode.delete.tsx   | 14 +++++++-------
 apps/webapp/package.json                           |  1 +
 7 files changed, 15 insertions(+), 22 deletions(-)

diff --git a/apps/webapp/app/components/conversation/conversation-list.tsx b/apps/webapp/app/components/conversation/conversation-list.tsx
index 5cc84a1..fcd666a 100644
--- a/apps/webapp/app/components/conversation/conversation-list.tsx
+++ b/apps/webapp/app/components/conversation/conversation-list.tsx
@@ -101,7 +101,7 @@ export const ConversationList = ({
           <div
             key={key}
             style={style}
-            className="-ml-1 flex items-center justify-start p-2 py-0 text-sm"
+            className="-mt-2 ml-1 flex items-center justify-start p-0 text-sm"
           >
             <Button
               variant="link"
diff --git a/apps/webapp/app/components/conversation/conversation-textarea.client.tsx b/apps/webapp/app/components/conversation/conversation-textarea.client.tsx
index a64a5b2..7b5747c 100644
--- a/apps/webapp/app/components/conversation/conversation-textarea.client.tsx
+++ b/apps/webapp/app/components/conversation/conversation-textarea.client.tsx
@@ -87,7 +87,7 @@ export function ConversationTextarea({
       onSubmit={(e) => submitForm(e)}
       className="pt-2"
     >
-      <div className="bg-background-3 rounded py-2">
+      <div className="bg-background-3 rounded-lg border-1 border-gray-300 py-2">
         <EditorRoot>
           <EditorContent
             // eslint-disable-next-line @typescript-eslint/no-explicit-any
@@ -146,7 +146,7 @@ export function ConversationTextarea({
             }}
             immediatelyRender={false}
             className={cn(
-              "editor-container text-md max-h-[400px] min-h-[40px] w-full min-w-full overflow-auto px-3 sm:rounded-lg",
+              "editor-container text-md max-h-[400px] min-h-[40px] w-full min-w-full overflow-auto rounded-lg px-3",
             )}
           />
         </EditorRoot>
diff --git a/apps/webapp/app/components/conversation/conversation.client.tsx b/apps/webapp/app/components/conversation/conversation.client.tsx
index 528595a..d5d9159 100644
--- a/apps/webapp/app/components/conversation/conversation.client.tsx
+++ b/apps/webapp/app/components/conversation/conversation.client.tsx
@@ -13,7 +13,6 @@ import {
   ResizablePanel,
   ResizablePanelGroup,
 } from "../ui/resizable";
-import { ConversationList } from "./conversation-list";
 
 export const ConversationNew = ({
   user,
@@ -71,7 +70,7 @@ export const ConversationNew = ({
                   <p className="text-muted-foreground mx-1 mb-4">
                     Demo UI: basic conversation to showcase memory integration.
                   </p>
-                  <div className="bg-background-3 border-border rounded-lg border-1 py-2">
+                  <div className="bg-background-3 rounded-lg border-1 border-gray-300 py-2">
                     <EditorRoot>
                       <EditorContent
                         ref={editorRef}
@@ -79,7 +78,7 @@ export const ConversationNew = ({
                         extensions={[
                           Placeholder.configure({
                             placeholder: () => {
-                              return "Ask CORE...";
+                              return "Ask CORE ...";
                             },
                             includeChildren: true,
                           }),
diff --git a/apps/webapp/app/components/conversation/editor-extensions.tsx b/apps/webapp/app/components/conversation/editor-extensions.tsx
index f89f502..2041de6 100644
--- a/apps/webapp/app/components/conversation/editor-extensions.tsx
+++ b/apps/webapp/app/components/conversation/editor-extensions.tsx
@@ -1,12 +1,5 @@
 import { cx } from "class-variance-authority";
-import {
-  StarterKit,
-  TiptapLink,
-  HorizontalRule,
-  Placeholder,
-  HighlightExtension,
-  AIHighlight,
-} from "novel";
+import { StarterKit, TiptapLink, HorizontalRule, Placeholder } from "novel";
 
 import CodeBlockLowlight from "@tiptap/extension-code-block-lowlight";
 import Heading from "@tiptap/extension-heading";
diff --git a/apps/webapp/app/components/ui/header.tsx b/apps/webapp/app/components/ui/header.tsx
index 933bfdb..a45252a 100644
--- a/apps/webapp/app/components/ui/header.tsx
+++ b/apps/webapp/app/components/ui/header.tsx
@@ -34,7 +34,7 @@ export function SiteHeader() {
   const showNewConversationButton = isConversationDetail(location.pathname);
 
   return (
-    <header className="border-border flex h-(--header-height) shrink-0 items-center gap-2 border-b transition-[width,height] ease-linear group-has-data-[collapsible=icon]/sidebar-wrapper:h-(--header-height)">
+    <header className="flex h-(--header-height) shrink-0 items-center gap-2 border-b border-gray-300 transition-[width,height] ease-linear group-has-data-[collapsible=icon]/sidebar-wrapper:h-(--header-height)">
       <div className="flex w-full items-center justify-between gap-1 px-4 pr-2 lg:gap-2">
         <div className="flex items-center gap-1">
           <SidebarTrigger className="-ml-1" />
diff --git a/apps/webapp/app/routes/api.v1.episode.delete.tsx b/apps/webapp/app/routes/api.v1.episode.delete.tsx
index 3ddf5a4..55dff0c 100644
--- a/apps/webapp/app/routes/api.v1.episode.delete.tsx
+++ b/apps/webapp/app/routes/api.v1.episode.delete.tsx
@@ -26,11 +26,11 @@ const { action, loader } = createActionApiRoute(
 
       if (!result.episodeDeleted) {
         return json(
-          { 
+          {
             error: "Episode not found or unauthorized",
-            code: "not_found"
+            code: "not_found",
           },
-          { status: 404 }
+          { status: 404 },
         );
       }
 
@@ -47,14 +47,14 @@ const { action, loader } = createActionApiRoute(
     } catch (error) {
       console.error("Error deleting episode:", error);
       return json(
-        { 
+        {
           error: "Failed to delete episode",
-          code: "internal_error"
+          code: "internal_error",
         },
-        { status: 500 }
+        { status: 500 },
       );
     }
   },
 );
 
-export { action, loader };
\ No newline at end of file
+export { action, loader };
diff --git a/apps/webapp/package.json b/apps/webapp/package.json
index 765eb37..be1500a 100644
--- a/apps/webapp/package.json
+++ b/apps/webapp/package.json
@@ -7,6 +7,7 @@
     "build": "remix vite:build",
     "dev": "node ./server.mjs",
     "lint": "eslint --ignore-path .gitignore --cache --cache-location ./node_modules/.cache/eslint .",
+    "lint:fix": "eslint --fix --ignore-path .gitignore --cache --cache-location ./node_modules/.cache/eslint .",
     "start": "remix-serve ./build/server/index.js",
     "typecheck": "tsc",
     "trigger:dev": "pnpm dlx trigger.dev@v4-beta dev"