refactor(worker): improve AbuseIPDB component quality

- Renamed files from abusedb to abuseipdb for clarity
- Extracted port definitions to avoid duplication between metadata and resolvePorts
- Added parameters array for maxAgeInDays and verbose in metadata
- Improved type safety by removing 'as any' assertions
- Added AbuseIPDBOutput interface in tests
- Added test cases for validation errors and verbose parameter
- Fixed readonly array type mismatch with SDK

Signed-off-by: betterclever <[email protected]>

Author: betterclever

worker/src/components/index.ts

@@ -55,7 +55,7 @@ import './security/atlassian-offboarding';
 import './security/trufflehog';
 import './security/terminal-demo';
 import './security/virustotal';
-import './security/abusedb';
+import './security/abuseipdb';
 
 // GitHub components
 import './github/connection-provider';

worker/src/components/security/__tests__/abusedb-integration.test.ts

@@ -0,0 +1,83 @@
+/**
+ * Optional integration test for AbuseIPDB component.
+ * Requires a valid AbuseIPDB API key.
+ * Enable by setting RUN_ABUSEDB_TESTS=1 and providing ABUSEIPDB_API_KEY.
+ */
+import { describe, expect, test, beforeEach } from 'bun:test';
+import { componentRegistry, createExecutionContext, type ExecutionContext } from '@shipsec/component-sdk';
+import '../../index'; // Ensure registry is populated
+
+interface AbuseIPDBOutput {
+  ipAddress: string;
+  isPublic?: boolean;
+  ipVersion?: number;
+  isWhitelisted?: boolean;
+  abuseConfidenceScore: number;
+  countryCode?: string;
+  usageType?: string;
+  isp?: string;
+  domain?: string;
+  hostnames?: string[];
+  totalReports?: number;
+  numDistinctUsers?: number;
+  lastReportedAt?: string;
+  reports?: Record<string, unknown>[];
+  full_report: Record<string, unknown>;
+}
+
+const shouldRunIntegration =
+  process.env.RUN_ABUSEDB_TESTS === '1' && !!process.env.ABUSEIPDB_API_KEY;
+
+(shouldRunIntegration ? describe : describe.skip)('AbuseIPDB Integration', () => {
+    let context: ExecutionContext;
+
+    beforeEach(async () => {
+        context = createExecutionContext({
+            runId: 'test-run',
+            componentRef: 'abuseipdb-integration-test',
+        });
+    });
+
+    test('checks a known IP address', async () => {
+        const component = componentRegistry.get('security.abuseipdb.check');
+        expect(component).toBeDefined();
+        
+        // 1.1.1.1 is Cloudflare DNS and should exist in AbuseIPDB
+        const ipToCheck = '1.1.1.1'; 
+        
+        const params = {
+            ipAddress: ipToCheck,
+            apiKey: process.env.ABUSEIPDB_API_KEY!,
+            maxAgeInDays: 90,
+            verbose: true
+        };
+
+        const result = await component!.execute(params, context) as AbuseIPDBOutput;
+
+        expect(result.ipAddress).toBe(ipToCheck);
+        expect(typeof result.abuseConfidenceScore).toBe('number');
+        expect(result.full_report).toBeDefined();
+    });
+
+    test('checks a known malicious IP address', async () => {
+        const component = componentRegistry.get('security.abuseipdb.check');
+        expect(component).toBeDefined();
+        
+        // Using Google DNS as a safe, known IP
+        const ipToCheck = '8.8.8.8'; 
+        
+        const params = {
+            ipAddress: ipToCheck,
+            apiKey: process.env.ABUSEIPDB_API_KEY!,
+            maxAgeInDays: 90,
+            verbose: false
+        };
+
+        const result = await component!.execute(params, context) as AbuseIPDBOutput;
+
+        expect(result.ipAddress).toBe(ipToCheck);
+        expect(typeof result.abuseConfidenceScore).toBe('number');
+        expect(result.abuseConfidenceScore).toBeGreaterThanOrEqual(0);
+        expect(result.abuseConfidenceScore).toBeLessThanOrEqual(100);
+    });
+});