Address various review feedback

Author: jsourcebot

packages/web/src/app/(app)/chat/[id]/components/chatThreadPanel.tsx

@@ -20,14 +20,6 @@ interface ChatThreadPanelProps {
     chatName?: string;
 }
 
-const normalizeDisabledMcpServerIds = (value: unknown): string[] => {
-    if (!Array.isArray(value)) {
-        return [];
-    }
-
-    return value.filter((id): id is string => typeof id === 'string');
-}
-
 export const ChatThreadPanel = ({
     languageModels,
     repos,
@@ -53,7 +45,7 @@ export const ChatThreadPanel = ({
     // Use the last user message to determine what repos, contexts, and MCP state we should select by default.
     const lastUserMessage = messages.findLast((message) => message.role === "user");
     const defaultSelectedSearchScopes = lastUserMessage?.metadata?.selectedSearchScopes ?? [];
-    const defaultDisabledMcpServerIds = normalizeDisabledMcpServerIds(lastUserMessage?.metadata?.disabledMcpServerIds);
+    const defaultDisabledMcpServerIds = lastUserMessage?.metadata?.disabledMcpServerIds ?? [];
     const [selectedSearchScopes, setSelectedSearchScopes] = useState<SearchScope[]>(defaultSelectedSearchScopes);
     const [disabledMcpServerIds, setDisabledMcpServerIds] = useState<string[]>(defaultDisabledMcpServerIds);
 
@@ -65,7 +57,7 @@ export const ChatThreadPanel = ({
         try {
             setInputMessage(chatState.inputMessage);
             setSelectedSearchScopes(chatState.selectedSearchScopes);
-            setDisabledMcpServerIds(normalizeDisabledMcpServerIds(chatState.disabledMcpServerIds));
+            setDisabledMcpServerIds(chatState.disabledMcpServerIds);
         } catch {
             console.error('Invalid chat state in session storage');
         } finally {

packages/web/src/app/(app)/settings/workspaceAskAgent/workspaceAskAgentPage.tsx

@@ -418,7 +418,7 @@ export function WorkspaceAskAgentPage({ callbackStatus, callbackServer, callback
             {/* Connectors section */}
             <div className="space-y-6">
                 <div>
-                    <h4 className="text-sm font-semibold text-foreground">Connectors</h4>
+                    <h4 className="text-md font-semibold text-foreground">Connectors</h4>
                     <p className="text-sm text-muted-foreground">
                         Connectors are MCP servers that let Ask Agent use approved external tools alongside your indexed code.
                     </p>

packages/web/src/features/chat/components/chatThread/chatThreadListItem.tsx

@@ -426,7 +426,7 @@ const ChatThreadListItemComponent = forwardRef<HTMLDivElement, ChatThreadListIte
                                 onHoveredReferenceChanged={setHoveredReference}
                                 style={rightPanelStyle}
                             />
-                        ) : isNetworkActive ? (
+                        ) : (isTurnInProgress) ? (
                             <div className="space-y-4">
                                 {Array.from({ length: 3 }).map((_, index) => (
                                     <Skeleton key={index} className="w-full h-48" />

packages/web/src/features/chat/components/chatThread/tools/toolSearchToolComponent.tsx

@@ -27,7 +27,6 @@ export const ToolSearchToolComponent = ({ query, results }: ToolSearchToolCompon
                     <span className="flex-shrink-0">Searched connector tools: <span className="italic">{query}</span></span>
                     <span className="flex-1" />
                     <span className="text-xs flex-shrink-0">{results.length} result{results.length === 1 ? '' : 's'}</span>
-                    <Separator orientation="vertical" className="h-3 flex-shrink-0" />
                 </div>
             </CollapsibleTrigger>
             <CollapsibleContent>