What claude did to upgrade

Author: jamo

package.json

@@ -13,34 +13,38 @@
     "git-pre-commit": "find . -path './**/*.ts' -not -path '*/node_modules/*' -not -path '*/dist/*' -exec sh -c 'for n; do tsfmt --verify \"$n\" || exit 1; done' sh {} +",
     "test": "jest --coverage",
     "build": "rm -rf ./dist && ncc build src/index.ts -m -s -o dist",
-    "prepublish": "npm run build"
+    "prepublish": "npm run build",
+    "prepare": "husky"
   },
   "peerDependencies": {
     "node-fetch": "2"
   },
   "devDependencies": {
-    "@types/debug": "4.1.5",
-    "@types/jest": "25.1.3",
-    "@types/lru-cache": "5.1.0",
-    "@zeit/git-hooks": "0.1.4",
-    "@vercel/ncc": "0.33.2",
-    "agentkeepalive": "4.2.0",
+    "@types/debug": "4.1.12",
+    "@types/jest": "29.5.14",
+    "husky": "9.1.7",
+    "@vercel/ncc": "0.38.3",
+    "agentkeepalive": "4.5.0",
     "async-retry-ng": "2.0.1",
-    "debug": "4.1.1",
-    "jest": "25.1.0",
-    "lint-staged": "10.0.7",
-    "lru-cache": "5.1.1",
-    "node-fetch": "2.6.7",
-    "raw-body": "2.4.2",
-    "ts-jest": "25.2.1",
-    "typescript": "4.5.5",
+    "debug": "4.4.0",
+    "jest": "29.7.0",
+    "lint-staged": "15.2.11",
+    "lru-cache": "11.0.2",
+    "node-fetch": "2.7.0",
+    "raw-body": "2.5.2",
+    "ts-jest": "29.2.5",
+    "typescript": "5.7.2",
     "typescript-formatter": "7.2.2"
   },
   "dependencies": {
     "@types/node-fetch": "2"
   },
+  "overrides": {
+    "js-yaml": "^4.1.1"
+  },
   "jest": {
     "preset": "ts-jest",
-    "verbose": true
+    "verbose": true,
+    "testEnvironment": "node"
   }
 }

src/dns-resolve.ts

@@ -1,12 +1,12 @@
 import dns from 'dns';
-import LRU from 'lru-cache';
+import { LRUCache } from 'lru-cache';
 import retry, { Options as RetryOptions } from 'async-retry-ng';
 import resolve4 from './resolve4';
 import resolve6 from './resolve6';
 
 const lruOptions = { max: 500 };
-let cache4: LRU<string, string | Promise<string>>;
-let cache6: LRU<string, string | Promise<string>>;
+let cache4: LRUCache<string, string | Promise<string>>;
+let cache6: LRUCache<string, string | Promise<string>>;
 
 export const localhostRegex = /(?:\.|^)localhost\.?$/
 
@@ -49,7 +49,7 @@ export default async function dnsResolve(host: string, options: Options = {}) {
 	}
 
 	if (refreshCache) {
-		cache.del(host);
+		cache.delete(host);
 	} else {
 		const ip = cache.get(host);
 		if (ip) return await ip;
@@ -62,15 +62,15 @@ export default async function dnsResolve(host: string, options: Options = {}) {
 			throw new DNSError('ENOTFOUND', host);
 		}
 		const ttl = Math.max(rec.ttl, minimumCacheTime);
-		cache.set(host, rec.address, ttl * 1000);
+		cache.set(host, rec.address, { ttl: ttl * 1000 });
 		return rec.address;
 	})();
 
-	cache.set(host, p, 5000);
+	cache.set(host, p, { ttl: 5000 });
 	return p;
 }
 
 export function setupCache() {
-	cache4 = new LRU(lruOptions);
-	cache6 = new LRU(lruOptions);
+	cache4 = new LRUCache(lruOptions);
+	cache6 = new LRUCache(lruOptions);
 }

src/index.ts

@@ -1,4 +1,4 @@
-import { resolve as resolveUrl } from 'url';
+import { URL } from 'url';
 import { Headers, Response } from 'node-fetch';
 import { Readable } from 'stream';
 import createDebug from 'debug';
@@ -140,7 +140,7 @@ function setupFetch(fetch: Fetch, agentOpts: AgentOptions = {}): any {
 			if (fetchOpts.redirect === 'manual') {
 				const location = res.headers.get('location');
 				if (location) {
-					res.headers.set('Location', resolveUrl(url, location));
+					res.headers.set('Location', new URL(location, url).href);
 				}
 
 				return res;

src/parse-host.ts

@@ -1,10 +1,10 @@
 import { isIP } from 'net';
-import { parse as parseUrl } from 'url';
+import { URL } from 'url';
 import { Headers } from 'node-fetch';
 import resolve from './dns-resolve';
 
 export default async function parseHost(url: string, headers: Headers) {
-	const parsedUrl = parseUrl(url);
+	const parsedUrl = new URL(url);
 	const host = headers.get('host') || parsedUrl.host;
 
 	if (!host) {
@@ -19,14 +19,8 @@ export default async function parseHost(url: string, headers: Headers) {
 			throw new Error('Unable to determine hostname');
 		}
 
-		// We need to create a new URL object here because url.parse() doesn't
-		// return a functional WHATWG URL object but something that only looks
-		// similar and has the same properties.
-		// TS doesn't know about the existence global WHATWG URL.
-		// @ts-ignore
-		const newUrl = new URL(parsedUrl.href);
-		newUrl.hostname = await resolve(parsedUrl.hostname);
-		url = newUrl.href;
+		parsedUrl.hostname = await resolve(parsedUrl.hostname);
+		url = parsedUrl.href;
 	}
 
 	return [url, host];

src/redirect.ts

@@ -1,7 +1,4 @@
-import {
-	parse as parseUrl,
-	resolve as resolveUrl
-} from 'url';
+import { URL } from 'url';
 import { Headers, Response } from 'node-fetch';
 import AgentWrapper from './agent-wrapper';
 import { FetchOptions } from './types';
@@ -30,9 +27,11 @@ export function makeRedirectOpts(res: Response, opts: FetchOptions, agentWrapper
 	if (!location) {
 		throw new Error('"Location" header is missing');
 	}
-	const locationUrl = resolveUrl(res.url, location);
+	// Handle both absolute and relative URLs
+	// If location is absolute, use it directly; if relative, resolve against res.url
+	const locationUrl = new URL(location, res.url || 'http://placeholder').href;
 
-	const host = parseUrl(locationUrl).host;
+	const host = new URL(locationUrl).host;
 	if (!host) {
 		throw new Error('Cannot determine Host');
 	}