fix: prevent binary download failures with compressed server responses

Request identity encoding and disable axios decompression for all
downloads to avoid failures when servers or proxies apply unexpected
content encoding. Fall back to x-original-content-length header for
progress tracking when content-length is absent.

Author: EhabY

src/core/cliManager.ts

@@ -355,7 +355,6 @@ export class CliManager {
 				binSource,
 				writeStream,
 				{
-					"Accept-Encoding": "gzip",
 					"If-None-Match": `"${etag}"`,
 				},
 				onProgress,
@@ -467,15 +466,19 @@ export class CliManager {
 			signal: controller.signal,
 			baseURL: baseUrl,
 			responseType: "stream",
-			headers,
-			decompress: true,
+			headers: {
+				"Accept-Encoding": "identity",
+				...headers,
+			},
+			decompress: false,
 			// Ignore all errors so we can catch a 404!
 			validateStatus: () => true,
 		});
 		this.output.info("Got status code", resp.status);
 
 		if (resp.status === 200) {
-			const rawContentLength = resp.headers["content-length"] as unknown;
+			const rawContentLength = (resp.headers["content-length"] ??
+				resp.headers["x-original-content-length"]) as unknown;
 			const contentLength = Number.parseInt(
 				typeof rawContentLength === "string" ? rawContentLength : "",
 			);

test/unit/core/cliManager.test.ts

@@ -275,7 +275,7 @@ describe("CliManager", () => {
 				expect.objectContaining({
 					responseType: "stream",
 					headers: expect.objectContaining({
-						"Accept-Encoding": "gzip",
+						"Accept-Encoding": "identity",
 						"If-None-Match": '""',
 					}),
 				}),
@@ -290,7 +290,7 @@ describe("CliManager", () => {
 				"/custom/path",
 				expect.objectContaining({
 					responseType: "stream",
-					decompress: true,
+					decompress: false,
 					validateStatus: expect.any(Function),
 				}),
 			);
@@ -425,10 +425,33 @@ describe("CliManager", () => {
 
 		it("handles missing content-length", async () => {
 			withSuccessfulDownload({ headers: {} });
+			mockProgress.clearProgressReports();
 			const result = await manager.fetchBinary(mockApi, "test");
 			expectPathsEqual(result, BINARY_PATH);
 			expect(memfs.existsSync(BINARY_PATH)).toBe(true);
-		});
+			// Without any content-length header, increment should be undefined.
+			const reports = mockProgress.getProgressReports();
+			expect(reports).not.toHaveLength(0);
+			for (const report of reports) {
+				expect(report).toMatchObject({ increment: undefined });
+			}
+		});
+
+		it.each(["content-length", "x-original-content-length"])(
+			"reports progress with %s header",
+			async (header) => {
+				withSuccessfulDownload({ headers: { [header]: "1024" } });
+				mockProgress.clearProgressReports();
+				const result = await manager.fetchBinary(mockApi, "test");
+				expectPathsEqual(result, BINARY_PATH);
+				expect(memfs.existsSync(BINARY_PATH)).toBe(true);
+				const reports = mockProgress.getProgressReports();
+				expect(reports).not.toHaveLength(0);
+				for (const report of reports) {
+					expect(report).toMatchObject({ increment: expect.any(Number) });
+				}
+			},
+		);
 	});
 
 	describe("Download Progress Tracking", () => {