Fix and optimize web search implementation

- Switch to DuckDuckGo as primary search method (Google was being blocked)
- Add DuckDuckGo redirect URL extraction to get real URLs from redirect links
- Improve parsing with multiple strategies: markdown links and direct URL extraction
- Add better error diagnostics with content preview
- Optimize performance: early exits, smart retries, reduced wait times
- Fix React duplicate key warnings in chat UI (use toolId instead of index)
- Add defensive check for message editing to prevent errors

Author: Tajudeen

extensions/open-remote-ssh/package.json

@@ -26,6 +26,7 @@
     "onCommand:openremotessh.openEmptyWindowInCurrentWindow",
     "onCommand:openremotessh.openConfigFile",
     "onCommand:openremotessh.showLog",
+    "onCommand:openremotessh.explorer.add",
     "onResolveRemoteAuthority:ssh-remote",
     "onView:sshHosts"
   ],

src/vs/platform/webContentExtractor/electron-main/webContentExtractorService.ts

@@ -95,12 +95,79 @@ export class NativeWebContentExtractorService implements IWebContentExtractorSer
 
 	private async extractAX(win: BrowserWindow, uri: URI): Promise<WebContentExtractResult> {
 		await win.loadURL(uri.toString(true));
-		win.webContents.debugger.attach('1.1');
-		const result: { nodes: AXNode[] } = await win.webContents.debugger.sendCommand('Accessibility.getFullAXTree');
-		const str = convertAXTreeToMarkdown(uri, result.nodes);
-		this._logger.info(`[NativeWebContentExtractorService] Content extracted from ${uri}`);
-		this._logger.trace(`[NativeWebContentExtractorService] Extracted content: ${str}`);
-		return { status: 'ok', result: str };
+
+		// Check if this is a JSON API endpoint
+		const isJsonApi = uri.toString(true).includes('format=json') || uri.toString(true).includes('/api/') || uri.path.includes('.json');
+
+		// Wait for page to load (with timeout)
+		await Promise.race([
+			new Promise<void>((resolve) => {
+				win.webContents.once('dom-ready', () => {
+					setTimeout(resolve, 500);
+				});
+			}),
+			new Promise<void>((resolve) => {
+				win.webContents.once('did-finish-load', () => {
+					setTimeout(resolve, 500);
+				});
+			}),
+			new Promise<void>((resolve) => setTimeout(resolve, 8000)) // 8 second timeout
+		]);
+
+		// For JSON APIs, try to extract raw JSON using JavaScript before accessibility tree
+		if (isJsonApi) {
+			try {
+				const rawText = await Promise.race([
+					win.webContents.executeJavaScript(`
+						(() => {
+							// JSON APIs often render in <pre> tags or as plain text
+							const preTag = document.querySelector('pre');
+							if (preTag && preTag.textContent) {
+								return preTag.textContent.trim();
+							}
+							if (document.body) {
+								const bodyText = document.body.innerText || document.body.textContent || '';
+								if (bodyText.trim()) {
+									return bodyText.trim();
+								}
+							}
+							return document.documentElement.innerText || document.documentElement.textContent || '';
+						})()
+					`),
+					new Promise<string>((_, reject) => setTimeout(() => reject(new Error('Timeout')), 3000))
+				]) as string;
+
+				if (rawText && rawText.trim()) {
+					const trimmed = rawText.trim();
+					// Validate it looks like JSON
+					if (trimmed.startsWith('{') || trimmed.startsWith('[')) {
+						this._logger.info(`[NativeWebContentExtractorService] Raw JSON content extracted from ${uri}`);
+						return { status: 'ok', result: trimmed };
+					}
+				}
+			} catch (error) {
+				this._logger.warn(`[NativeWebContentExtractorService] Failed to extract JSON via JavaScript from ${uri}: ${error}`);
+			}
+		}
+
+		// Fallback to accessibility tree extraction (for HTML pages or if JSON extraction failed)
+		try {
+			win.webContents.debugger.attach('1.1');
+			const result: { nodes: AXNode[] } = await win.webContents.debugger.sendCommand('Accessibility.getFullAXTree');
+			const str = convertAXTreeToMarkdown(uri, result.nodes);
+			this._logger.info(`[NativeWebContentExtractorService] Content extracted from ${uri}`);
+			this._logger.trace(`[NativeWebContentExtractorService] Extracted content: ${str.substring(0, 200)}...`);
+			return { status: 'ok', result: str };
+		} finally {
+			// Detach debugger when done
+			try {
+				if (win.webContents.debugger.isAttached()) {
+					win.webContents.debugger.detach();
+				}
+			} catch (e) {
+				// Ignore detach errors
+			}
+		}
 	}
 
 	private interceptRedirects(win: BrowserWindow, uri: URI, store: DisposableStore) {

src/vs/workbench/contrib/comments/browser/commentsController.ts

@@ -1084,7 +1084,12 @@ export class CommentController implements IEditorContribution {
 	}
 
 	private onEditorMouseDown(e: IEditorMouseEvent): void {
-		this.mouseDownInfo = (e.target.element?.className.indexOf('comment-range-glyph') ?? -1) >= 0 ? parseMouseDownInfoFromEvent(e) : null;
+		const element = e.target.element;
+		const hasCommentGlyph = element && (
+			(typeof element.className === 'string' && element.className.indexOf('comment-range-glyph') >= 0) ||
+			(element.classList && element.classList.contains('comment-range-glyph'))
+		);
+		this.mouseDownInfo = hasCommentGlyph ? parseMouseDownInfoFromEvent(e) : null;
 	}
 
 	private onEditorMouseUp(e: IEditorMouseEvent): void {

src/vs/workbench/contrib/cortexide/browser/media/cortexide.css

@@ -575,7 +575,8 @@ body.monaco-workbench {
 	color: var(--cortex-text-muted) !important;
 	border-radius: 14px !important;
 	border: 1px solid transparent !important;
-	padding: 6px 10px !important;
+	padding: 9px 12px !important;
+	min-height: 40px !important;
 }
 
 .monaco-workbench .quick-input-widget .quick-input-list .quick-input-list-entry .quick-input-list-label {

src/vs/workbench/contrib/cortexide/browser/react/src/sidebar-tsx/SidebarChat.tsx

@@ -1340,6 +1340,15 @@ const UserMessageComponent = ({ chatMessage, messageIdx, isCheckpointGhost, curr
 			// cancel any streams on this thread - use subscribed state
 			const threadId = currentThreadId
 
+			// Defensive check: verify the message is still a user message before editing
+			const thread = chatThreadsState.allThreads[threadId]
+			if (!thread || !thread.messages || thread.messages[messageIdx]?.role !== 'user') {
+				console.error('Error while editing message: Message is not a user message or no longer exists')
+				setIsBeingEdited(false)
+				chatThreadsService.setCurrentlyFocusedMessageIdx(undefined)
+				return
+			}
+
 			await chatThreadsService.abortRunning(threadId)
 
 			// update state
@@ -3270,7 +3279,7 @@ const PlanComponent = React.memo(({ message, isCheckpointGhost, threadId, messag
 														<div className="text-void-fg-3 text-xs mb-2 font-medium">Expected Tools:</div>
 														<div className="flex flex-wrap gap-1.5">
 															{step.tools.map((tool, i) => (
-																<span key={i} className="px-2 py-0.5 bg-blue-500/10 text-blue-400 text-xs rounded border border-blue-500/20">
+																<span key={`${step.stepNumber}-tool-${tool}-${i}`} className="px-2 py-0.5 bg-blue-500/10 text-blue-400 text-xs rounded border border-blue-500/20">
 																	{tool}
 																</span>
 															))}
@@ -3290,7 +3299,7 @@ const PlanComponent = React.memo(({ message, isCheckpointGhost, threadId, messag
 																const isError = toolMsg.type === 'tool_error'
 
 																return (
-																	<div key={i} className={`p-2 rounded border text-xs ${
+																	<div key={toolId} className={`p-2 rounded border text-xs ${
 																		isSuccess ? 'bg-green-500/10 border-green-500/20' :
 																		isError ? 'bg-red-500/10 border-red-500/20' :
 																		'bg-blue-500/10 border-blue-500/20'