Fix ESLint errors and README inconsistencies

- Replace `any` types with proper types in ipdata.ts (unknown, LookupResponse[])
- Fix no-use-before-define by moving keyBy() above its usage in tests
- Replace for...of with Array.reduce to satisfy no-restricted-syntax rule
- Type mockFetchResponse parameter as unknown instead of any
- Update README badge URL to match package.json repository (thomasconner)
- Add missing organisation field to README response fields list

https://claude.ai/code/session_01UabaLo3R3sXovyxaZo2yCF

Author: claude

README.md

@@ -1,6 +1,6 @@
 # IPData JavaScript Library
 
-[![](https://github.com/ConnerTechnology/ipdata-js-library/workflows/CI/badge.svg)](https://github.com/ConnerTechnology/ipdata-js-library/actions)
+[![](https://github.com/thomasconner/ipdata-js-library/workflows/CI/badge.svg)](https://github.com/thomasconner/ipdata-js-library/actions)
 
 JavaScript library that can be used in a web browser or Node.js application to gather information for an IP address using https://ipdata.co.
 
@@ -154,7 +154,7 @@ ipdata.bulkLookup(ips, fields)
 
 The following fields are available for use with `selectField` and `fields` parameters:
 
-`ip`, `is_eu`, `city`, `region`, `region_code`, `country_name`, `country_code`, `continent_name`, `continent_code`, `latitude`, `longitude`, `asn`, `company`, `postal`, `calling_code`, `flag`, `emoji_flag`, `emoji_unicode`, `carrier`, `languages`, `currency`, `time_zone`, `threat`, `count`
+`ip`, `is_eu`, `city`, `region`, `region_code`, `country_name`, `country_code`, `continent_name`, `continent_code`, `latitude`, `longitude`, `asn`, `company`, `organisation`, `postal`, `calling_code`, `flag`, `emoji_flag`, `emoji_unicode`, `carrier`, `languages`, `currency`, `time_zone`, `threat`, `count`
 
 The `company` field returns an object with `name`, `domain`, `network`, and `type` properties.
 

src/ipdata.test.ts

@@ -66,7 +66,7 @@ const MOCK_DEFAULT_IP_DATA = {
   ip: '203.0.113.1',
 };
 
-function mockFetchResponse(data: any, status = 200) {
+function mockFetchResponse(data: unknown, status = 200) {
   return Promise.resolve({
     ok: status >= 200 && status < 300,
     status,
@@ -77,6 +77,13 @@ function mockFetchResponse(data: any, status = 200) {
 const mockFetch = jest.fn() as jest.Mock;
 global.fetch = mockFetch;
 
+function keyBy(items: LookupResponse[]): Record<string, LookupResponse> {
+  return items.reduce<Record<string, LookupResponse>>((acc, item) => {
+    acc[item.ip] = item;
+    return acc;
+  }, {});
+}
+
 describe('constructor()', () => {
   it('should throw an error if an apiKey is not provided', () => {
     // eslint-disable-next-line @typescript-eslint/ban-ts-comment
@@ -359,11 +366,3 @@ describe('bulkLookup()', () => {
     });
   });
 });
-
-function keyBy(items: LookupResponse[]): Record<string, LookupResponse> {
-  const result: Record<string, LookupResponse> = {};
-  for (const item of items) {
-    result[item.ip] = item;
-  }
-  return result;
-}

src/ipdata.ts

@@ -190,13 +190,13 @@ export default class IPData {
       return { ...errorData, status: response.status } as LookupResponse;
     }
 
-    const responseData = (await response.json()) as any;
+    const responseData: unknown = await response.json();
     let data: LookupResponse;
 
     if (selectField) {
       data = { [selectField]: responseData, status: response.status } as unknown as LookupResponse;
     } else {
-      data = { ...responseData, status: response.status };
+      data = { ...(responseData as Record<string, unknown>), status: response.status } as LookupResponse;
     }
 
     this.cache.set(ip || DEFAULT_IP, data);
@@ -245,7 +245,7 @@ export default class IPData {
         return { ...errorData, status: response.status } as unknown as LookupResponse[];
       }
 
-      const responseData = (await response.json()) as any[];
+      const responseData = (await response.json()) as LookupResponse[];
       responseData.forEach((info) => {
         this.cache.set(info.ip, { ...info, status: response.status });
         responses.push(this.cache.get(info.ip)!);