From 70383931bb113e8b7b504a947d494bc2aa142e2e Mon Sep 17 00:00:00 2001
From: Anish Sarkar <104695310+AnishSarkar22@users.noreply.github.com>
Date: Sat, 27 Dec 2025 15:27:29 +0530
Subject: [PATCH] feat: Apply minor UI styling adjustments to sidebar and
 progress components. Fixed linting for both frontend and backend

---
 .../app/tasks/chat/stream_new_chat.py           |  6 +++---
 .../components/sidebar/all-notes-sidebar.tsx    |  4 ++--
 surfsense_web/components/tool-ui/plan/plan.tsx  | 17 ++++++++++-------
 .../components/tool-ui/write-todos.tsx          |  3 ++-
 4 files changed, 17 insertions(+), 13 deletions(-)

diff --git a/surfsense_backend/app/tasks/chat/stream_new_chat.py b/surfsense_backend/app/tasks/chat/stream_new_chat.py
index 602dac0a5..52c27f554 100644
--- a/surfsense_backend/app/tasks/chat/stream_new_chat.py
+++ b/surfsense_backend/app/tasks/chat/stream_new_chat.py
@@ -72,10 +72,10 @@ def format_mentioned_documents_as_context(documents: list[Document]) -> str:
 def extract_todos_from_deepagents(command_output) -> dict:
     """
     Extract todos from deepagents' TodoListMiddleware Command output.
-    
+
     deepagents returns a Command object with:
     - Command.update['todos'] = [{'content': '...', 'status': '...'}]
-    
+
     Returns the todos directly (no transformation needed - UI matches deepagents format).
     """
     todos_data = []
@@ -89,7 +89,7 @@ def extract_todos_from_deepagents(command_output) -> dict:
             todos_data = command_output.get("todos", [])
         elif "update" in command_output and isinstance(command_output["update"], dict):
             todos_data = command_output["update"].get("todos", [])
-    
+
     return {"todos": todos_data}
 
 
diff --git a/surfsense_web/components/sidebar/all-notes-sidebar.tsx b/surfsense_web/components/sidebar/all-notes-sidebar.tsx
index d66a01780..53bfb6498 100644
--- a/surfsense_web/components/sidebar/all-notes-sidebar.tsx
+++ b/surfsense_web/components/sidebar/all-notes-sidebar.tsx
@@ -208,7 +208,7 @@ export function AllNotesSidebar({
 						aria-label={t("all_notes") || "All Notes"}
 					>
 						{/* Header */}
-						<div className="flex-shrink-0 p-4 pb-3 space-y-3 border-b">
+						<div className="flex-shrink-0 p-4 pb-3 space-y-3">
 							<div className="flex items-center justify-between">
 								<h2 className="text-lg font-semibold">{t("all_notes") || "All Notes"}</h2>
 								<Button
@@ -370,7 +370,7 @@ export function AllNotesSidebar({
 
 						{/* Footer with Add Note button */}
 						{onAddNote && notes.length > 0 && (
-							<div className="flex-shrink-0 p-3 border-t">
+							<div className="flex-shrink-0 p-3">
 								<Button
 									onClick={() => {
 										onAddNote();
diff --git a/surfsense_web/components/tool-ui/plan/plan.tsx b/surfsense_web/components/tool-ui/plan/plan.tsx
index 5c9335022..1980448c2 100644
--- a/surfsense_web/components/tool-ui/plan/plan.tsx
+++ b/surfsense_web/components/tool-ui/plan/plan.tsx
@@ -172,13 +172,16 @@ export const Plan: FC<PlanProps> = ({
 
 				{showProgress && (
 					<div className="mt-3 space-y-1.5">
-					<div className="flex items-center justify-between text-xs text-muted-foreground">
-						<span>
-							{progress.completed} of {progress.total} complete
-						</span>
-						<span>{Math.round(progress.percentage)}%</span>
-					</div>
-						<Progress value={progress.percentage} className="h-1.5 bg-muted [&>div]:bg-muted-foreground" />
+						<div className="flex items-center justify-between text-xs text-muted-foreground">
+							<span>
+								{progress.completed} of {progress.total} complete
+							</span>
+							<span>{Math.round(progress.percentage)}%</span>
+						</div>
+						<Progress
+							value={progress.percentage}
+							className="h-1.5 bg-muted [&>div]:bg-muted-foreground"
+						/>
 					</div>
 				)}
 			</CardHeader>
diff --git a/surfsense_web/components/tool-ui/write-todos.tsx b/surfsense_web/components/tool-ui/write-todos.tsx
index c3db8879a..a5da31e9e 100644
--- a/surfsense_web/components/tool-ui/write-todos.tsx
+++ b/surfsense_web/components/tool-ui/write-todos.tsx
@@ -139,7 +139,8 @@ export const WriteTodosToolUI = makeAssistantToolUI<WriteTodosData, WriteTodosDa
 		}
 
 		// Success - render the plan
-		const planToRender = currentPlanState || (hasTodos ? parseSerializablePlan({ todos: data.todos }) : null);
+		const planToRender =
+			currentPlanState || (hasTodos ? parseSerializablePlan({ todos: data.todos }) : null);
 		if (!planToRender) {
 			return <WriteTodosLoading />;
 		}