tidy nits

Author: connor4312

src/vs/workbench/contrib/mcp/common/mcpResourceFilesystem.ts

@@ -275,35 +275,27 @@ export class McpResourceFilesystem extends Disposable implements IWorkbenchContr
 
 	private async _readURIInner(uri: URI, token?: CancellationToken): Promise<IReadData> {
 		const { resourceURI, server } = this._decodeURI(uri);
-		let fallbackToMCPServer = false;
-		let returnVal: { contents: (MCP.TextResourceContents | MCP.BlobResourceContents)[]; resourceURI: URL; forSameURI: (MCP.TextResourceContents | MCP.BlobResourceContents)[] } | undefined;
 		const matchedServer = this._mcpService.servers.get().find(s => s.definition.id === server.definition.id);
+
 		//check for http/https resources and use web content extractor service to fetch the contents.
 		if (canLoadMcpNetworkResourceDirectly(resourceURI, matchedServer)) {
 			const extractURI = URI.parse(resourceURI.toString());
-			const result = await this._webContentExtractorService.extract([extractURI], { followRedirects: false });
-			returnVal = {
-				contents: result.map(r => {
-					if (r.status === 'ok') {
-						return { uri: resourceURI.toString(), text: r.result };
-					} else {
-						fallbackToMCPServer = true;
-						return { uri: resourceURI.toString(), text: '' };
-					}
-				}),
-				resourceURI,
-				forSameURI: result.filter(r => r.status === 'ok').map(r => ({ uri: resourceURI.toString(), text: r.result }))
-			};
-		}
-		if (fallbackToMCPServer || !returnVal) {
-			const res = await McpServer.callOn(server, r => r.readResource({ uri: resourceURI.toString() }, token), token);
-			returnVal = {
-				contents: res.contents,
-				resourceURI,
-				forSameURI: res.contents.filter(c => equalsUrlPath(c.uri, resourceURI))
-			};
+			const result = (await this._webContentExtractorService.extract([extractURI], { followRedirects: false })).at(0);
+			if (result?.status === 'ok') {
+				return {
+					contents: [{ uri: resourceURI.toString(), text: result.result }],
+					resourceURI,
+					forSameURI: [{ uri: resourceURI.toString(), text: result.result }]
+				};
+			}
 		}
-		return returnVal;
+
+		const res = await McpServer.callOn(server, r => r.readResource({ uri: resourceURI.toString() }, token), token);
+		return {
+			contents: res.contents,
+			resourceURI,
+			forSameURI: res.contents.filter(c => equalsUrlPath(c.uri, resourceURI))
+		};
 	}
 }