fix: add type declarations for fetch-error-handler

No need for tests for this module as it doesn't use default exports.

Author: rowanmanning

.gitignore

@@ -1,5 +1,4 @@
 .husky/_
-packages/fetch-error-handler/**/*.d.*
 packages/logger/**/*.d.*
 coverage
 node_modules/

jsconfig.build.json

@@ -8,7 +8,6 @@
 		"removeComments": true
 	},
 	"include": [
-		"packages/fetch-error-handler/**/*.js",
 		"packages/logger/**/*.js"
 	]
 }

packages/fetch-error-handler/.npmignore

@@ -1,5 +1,3 @@
-!*.d.ts
-!*.d.ts.map
 CHANGELOG.md
 docs
 test

packages/fetch-error-handler/package.json

@@ -14,7 +14,8 @@
     "node": "18.x || 20.x || 22.x",
     "npm": "8.x || 9.x || 10.x"
   },
-  "main": "lib",
+  "main": "lib/index.js",
+  "types": "types/index.d.ts",
   "dependencies": {
     "@dotcom-reliability-kit/errors": "^3.1.1"
   },

packages/fetch-error-handler/types/index.d.ts

@@ -0,0 +1,28 @@
+import { Writable } from 'stream';
+
+declare module '@dotcom-reliability-kit/fetch-error-handler' {
+	export type ErrorHandlerOptions = {
+		upstreamSystemCode?: string;
+	};
+
+	type NodeFetchResponseBody = {
+		pipe?: (stream: Writable) => void;
+	};
+
+	type FetchResponse = {
+		ok: boolean;
+		status: number;
+		url: string;
+		body: NodeFetchResponseBody | ReadableStream<Uint8Array> | null;
+	};
+
+	export type FetchErrorHandler = <Response extends FetchResponse>(
+		fetchPromise: Response | Promise<Response>
+	) => Promise<Response>;
+
+	export function createFetchErrorHandler(
+		options?: ErrorHandlerOptions
+	): FetchErrorHandler;
+
+	export const handleFetchErrors: FetchErrorHandler;
+}

scripts/clean-generated-types.sh

@@ -1,4 +1,3 @@
 #!/usr/bin/env bash
 
-find ./packages/fetch-error-handler -name "*.d.ts*" | xargs -r rm
 find ./packages/logger -name "*.d.ts*" | xargs -r rm