fix: improve handling of net::ERR_ABORTED errors in URL fetching

src/core/mentions/index.ts

@@ -35,6 +35,9 @@ function getUrlErrorMessage(error: unknown): string {
 	if (errorMessage.includes("net::ERR_INTERNET_DISCONNECTED")) {
 		return t("common:errors.no_internet")
 	}
+	if (errorMessage.includes("net::ERR_ABORTED")) {
+		return t("common:errors.url_request_aborted")
+	}
 	if (errorMessage.includes("403") || errorMessage.includes("Forbidden")) {
 		return t("common:errors.url_forbidden")
 	}

src/i18n/locales/en/common.json

@@ -61,6 +61,7 @@
 		"no_internet": "No internet connection. Please check your network connection and try again.",
 		"url_forbidden": "Access to this website is forbidden. The site may block automated access or require authentication.",
 		"url_page_not_found": "The page was not found. Please check if the URL is correct.",
+		"url_request_aborted": "The request to fetch the URL was aborted. This may happen if the site blocks automated access, requires authentication, or if there's a network issue. Please try again or check if the URL is accessible in a regular browser.",
 		"url_fetch_failed": "Failed to fetch URL content: {{error}}",
 		"url_fetch_error_with_url": "Error fetching content for {{url}}: {{error}}",
 		"command_timeout": "Command execution timed out after {{seconds}} seconds",

src/services/browser/__tests__/UrlContentFetcher.spec.ts

@@ -273,6 +273,36 @@ describe("UrlContentFetcher", () => {
 			await expect(urlContentFetcher.urlToMarkdown("https://example.com")).rejects.toThrow("Simple string error")
 			expect(mockPage.goto).toHaveBeenCalledTimes(1)
 		})
+
+		it("should retry net::ERR_ABORTED like other network errors", async () => {
+			const abortedError = new Error("net::ERR_ABORTED at https://example.com")
+			mockPage.goto.mockRejectedValueOnce(abortedError).mockResolvedValueOnce(undefined)
+
+			const result = await urlContentFetcher.urlToMarkdown("https://example.com")
+
+			expect(mockPage.goto).toHaveBeenCalledTimes(2)
+			expect(mockPage.goto).toHaveBeenNthCalledWith(1, "https://example.com", {
+				timeout: 30000,
+				waitUntil: ["domcontentloaded", "networkidle2"],
+			})
+			expect(mockPage.goto).toHaveBeenNthCalledWith(2, "https://example.com", {
+				timeout: 20000,
+				waitUntil: ["domcontentloaded"],
+			})
+			expect(result).toBe("# Test content")
+		})
+
+		it("should throw error when ERR_ABORTED retry also fails", async () => {
+			const abortedError = new Error("net::ERR_ABORTED at https://example.com")
+			const retryError = new Error("net::ERR_CONNECTION_REFUSED")
+			mockPage.goto.mockRejectedValueOnce(abortedError).mockRejectedValueOnce(retryError)
+
+			await expect(urlContentFetcher.urlToMarkdown("https://example.com")).rejects.toThrow(
+				"net::ERR_CONNECTION_REFUSED",
+			)
+
+			expect(mockPage.goto).toHaveBeenCalledTimes(2)
+		})
 	})
 
 	describe("closeBrowser", () => {