refactor: remove dead exports from export audit (#2960)

- Remove unused `OptionSourceResolver` type export from main-action.ts
  (not imported anywhere) — closes #2927
- Remove dead `extractPort` function from log-parser.ts (never called
  internally or externally) and its barrel re-export — closes #2914
- Remove redundant `PidTrackResult` re-export from pid-tracker.ts
  (already exported from types/index.ts) — closes #2915

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: lpcox

src/commands/main-action.ts

@@ -62,7 +62,7 @@ import {
  * Injected to decouple the action handler from the global program instance,
  * enabling independent unit testing.
  */
-export type OptionSourceResolver = (optionName: string) => string | undefined;
+type OptionSourceResolver = (optionName: string) => string | undefined;
 
 /**
  * Creates the main `awf` action handler bound to a specific option-source

src/logs/index.ts

@@ -2,7 +2,7 @@
  * Log viewing utilities for Squid proxy logs
  */
 
-export { parseLogLine, extractDomain, extractPort } from './log-parser';
+export { parseLogLine, extractDomain } from './log-parser';
 export { LogFormatter } from './log-formatter';
 export {
   discoverLogSources,

src/logs/log-parser.test.ts

@@ -2,7 +2,7 @@
  * Unit tests for log-parser.ts
  */
 
-import { parseLogLine, extractDomain, extractPort, parseAuditJsonlLine } from './log-parser';
+import { parseLogLine, extractDomain, parseAuditJsonlLine } from './log-parser';
 
 describe('log-parser', () => {
   describe('parseLogLine', () => {
@@ -224,26 +224,6 @@ describe('log-parser', () => {
     });
   });
 
-  describe('extractPort', () => {
-    it('should extract port from CONNECT URL', () => {
-      expect(extractPort('api.github.com:443', 'CONNECT')).toBe('443');
-      expect(extractPort('example.com:8080', 'CONNECT')).toBe('8080');
-    });
-
-    it('should return undefined for CONNECT URL without port', () => {
-      expect(extractPort('api.github.com', 'CONNECT')).toBeUndefined();
-    });
-
-    it('should return undefined for non-CONNECT methods', () => {
-      expect(extractPort('http://example.com:80/', 'GET')).toBeUndefined();
-      expect(extractPort('http://example.com/', 'POST')).toBeUndefined();
-    });
-
-    it('should not extract non-numeric port', () => {
-      expect(extractPort('api.github.com:abc', 'CONNECT')).toBeUndefined();
-    });
-  });
-
   describe('parseAuditJsonlLine', () => {
     it('should parse a valid JSONL CONNECT entry', () => {
       const line = '{"ts":1761074374.646,"client":"172.30.0.20","host":"api.github.com:443","dest":"140.82.114.22:443","method":"CONNECT","status":200,"decision":"TCP_TUNNEL","url":"api.github.com:443"}';

src/logs/log-parser.ts

@@ -142,26 +142,6 @@ export function extractDomain(url: string, host: string, method: string): string
   }
 }
 
-/**
- * Extracts the port from a domain:port string or URL
- *
- * @param url - URL field or domain:port string
- * @param method - HTTP method
- * @returns Port number string or undefined
- */
-export function extractPort(url: string, method: string): string | undefined {
-  if (method === 'CONNECT') {
-    const colonIndex = url.lastIndexOf(':');
-    if (colonIndex !== -1) {
-      const possiblePort = url.substring(colonIndex + 1);
-      if (/^\d+$/.test(possiblePort)) {
-        return possiblePort;
-      }
-    }
-  }
-  return undefined;
-}
-
 /**
  * Parses a single line from the JSONL audit log (audit.jsonl).
  *

src/pid-tracker.ts

@@ -24,9 +24,6 @@ import * as fs from 'fs';
 import * as path from 'path';
 import { PidTrackResult } from './types';
 
-// Re-export PidTrackResult for convenience
-export { PidTrackResult } from './types';
-
 /**
  * Parsed entry from /proc/net/tcp
  */