fix: resolve all linting issues

- Replace any types with proper type annotations
- Convert forEach loops to for...of loops
- Add interface for API response types

Author: Mearman

src/cli.test.ts

@@ -11,8 +11,8 @@ vi.mock('./tools/search.js');
 vi.mock('./tools/status.js');
 
 describe('CLI', () => {
-	let consoleLogSpy: any;
-	let consoleErrorSpy: any;
+	let consoleLogSpy: ReturnType<typeof vi.spyOn>;
+	let consoleErrorSpy: ReturnType<typeof vi.spyOn>;
 
 	beforeEach(() => {
 		vi.clearAllMocks();

src/cli.ts

@@ -81,13 +81,13 @@ export function createCLI() {
 				if (result.success && result.results && result.results.length > 0) {
 					spinner.succeed(chalk.green(`Found ${result.totalResults} archives`));
 					console.log(`\n${chalk.bold('Archive snapshots:')}`);
-					result.results.forEach((snapshot) => {
+					for (const snapshot of result.results) {
 						console.log(chalk.gray('─'.repeat(60)));
 						console.log(chalk.blue('Date:'), snapshot.date);
 						console.log(chalk.blue('URL:'), snapshot.archivedUrl);
 						console.log(chalk.blue('Status:'), snapshot.statusCode);
 						console.log(chalk.blue('Type:'), snapshot.mimeType);
-					});
+					}
 				} else {
 					spinner.fail(chalk.yellow(result.message || 'No archives found'));
 				}
@@ -113,9 +113,9 @@ export function createCLI() {
 						console.log(chalk.blue('Last capture:'), result.lastCapture);
 						if (result.yearlyCaptures) {
 							console.log(`\n${chalk.bold('Yearly captures:')}`);
-							Object.entries(result.yearlyCaptures).forEach(([year, count]) => {
+							for (const [year, count] of Object.entries(result.yearlyCaptures)) {
 								console.log(chalk.blue(`${year}:`), count);
-							});
+							}
 						}
 					} else {
 						spinner.warn(chalk.yellow('URL has not been archived'));

src/tools/retrieve.test.ts

@@ -16,9 +16,7 @@ vi.mock('../utils/rate-limit.js');
 describe('getArchivedUrl', () => {
 	beforeEach(() => {
 		vi.clearAllMocks();
-		vi.spyOn(rateLimitModule.waybackRateLimiter, 'waitForSlot').mockResolvedValue(
-			undefined as any,
-		);
+		vi.spyOn(rateLimitModule.waybackRateLimiter, 'waitForSlot').mockResolvedValue(undefined);
 		vi.spyOn(rateLimitModule.waybackRateLimiter, 'recordRequest').mockImplementation(() => {});
 	});
 

src/tools/save.test.ts

@@ -16,9 +16,7 @@ vi.mock('../utils/rate-limit.js');
 describe('saveUrl', () => {
 	beforeEach(() => {
 		vi.clearAllMocks();
-		vi.spyOn(rateLimitModule.waybackRateLimiter, 'waitForSlot').mockResolvedValue(
-			undefined as any,
-		);
+		vi.spyOn(rateLimitModule.waybackRateLimiter, 'waitForSlot').mockResolvedValue(undefined);
 		vi.spyOn(rateLimitModule.waybackRateLimiter, 'recordRequest').mockImplementation(() => {});
 	});
 

src/tools/search.test.ts

@@ -16,9 +16,7 @@ vi.mock('../utils/rate-limit.js');
 describe('searchArchives', () => {
 	beforeEach(() => {
 		vi.clearAllMocks();
-		vi.spyOn(rateLimitModule.waybackRateLimiter, 'waitForSlot').mockResolvedValue(
-			undefined as any,
-		);
+		vi.spyOn(rateLimitModule.waybackRateLimiter, 'waitForSlot').mockResolvedValue(undefined);
 		vi.spyOn(rateLimitModule.waybackRateLimiter, 'recordRequest').mockImplementation(() => {});
 	});
 

src/tools/status.test.ts

@@ -16,9 +16,7 @@ vi.mock('../utils/rate-limit.js');
 describe('checkArchiveStatus', () => {
 	beforeEach(() => {
 		vi.clearAllMocks();
-		vi.spyOn(rateLimitModule.waybackRateLimiter, 'waitForSlot').mockResolvedValue(
-			undefined as any,
-		);
+		vi.spyOn(rateLimitModule.waybackRateLimiter, 'waitForSlot').mockResolvedValue(undefined);
 		vi.spyOn(rateLimitModule.waybackRateLimiter, 'recordRequest').mockImplementation(() => {});
 	});
 

src/tools/status.ts

@@ -7,6 +7,23 @@ import { HttpError, fetchWithTimeout, parseJsonResponse } from '../utils/http.js
 import { waybackRateLimiter } from '../utils/rate-limit.js';
 import { validateUrl } from '../utils/validation.js';
 
+interface AvailabilityResponse {
+	archivedSnapshots?: {
+		closest?: {
+			available: boolean;
+			timestamp: string;
+			url?: string;
+		};
+	};
+	archived_snapshots?: {
+		closest?: {
+			available: boolean;
+			timestamp: string;
+			url?: string;
+		};
+	};
+}
+
 export const CheckArchiveStatusSchema = z.object({
 	url: z.string().url().describe('The URL to check'),
 });
@@ -98,7 +115,7 @@ export async function checkArchiveStatus(input: CheckArchiveStatusInput): Promis
 			},
 		});
 
-		const availData = await parseJsonResponse<any>(availResponse);
+		const availData = await parseJsonResponse<AvailabilityResponse>(availResponse);
 
 		if (availData.archived_snapshots?.closest?.available) {
 			return {