refactor: remove dead exports from preflight, network-setup, domain-acl, and logs barrel (#3051)

- Unexport AllowedDomainsResult interface in preflight.ts (never imported)
- Unexport NetworkSetupResult interface in network-setup.ts (never imported)
- Unexport DomainsByProtocol, PatternsByProtocol interfaces in domain-acl.ts
  (only used internally)
- Unexport groupDomainsByProtocol, groupPatternsByProtocol functions in
  domain-acl.ts (only called within the same file)
- Trim logs/index.ts barrel to the 3 symbols actually consumed via the
  barrel (listLogSources, LogFormatter, streamLogs); remove 9 dead
  re-exports (parseLogLine, extractDomain, discoverLogSources,
  selectMostRecent, isContainerRunning, validateSource, aggregateLogs,
  loadAllLogs, loadAndAggregate, AggregatedStats, DomainStats, formatStats)

Closes #3044
Closes #3041
Closes #3047
Closes #3048
Closes #3050
Closes #3025

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

Author: lpcox

src/commands/network-setup.ts

@@ -7,7 +7,7 @@ import { UpstreamProxyConfig } from '../types';
 /**
  * The resolved network configuration produced by {@link resolveNetworkConfig}.
  */
-export interface NetworkSetupResult {
+interface NetworkSetupResult {
   upstreamProxy: UpstreamProxyConfig | undefined;
   dnsServers: string[];
   dnsOverHttps: string | undefined;

src/commands/preflight.ts

@@ -20,7 +20,7 @@ type OptionSourceResolver = (optionName: string) => string | undefined;
 /**
  * The result produced by {@link resolveAllowedDomains}.
  */
-export interface AllowedDomainsResult {
+interface AllowedDomainsResult {
   allowedDomains: string[];
   localhostResult: ReturnType<typeof processLocalhostKeyword>;
   resolvedCopilotApiTarget: string | undefined;

src/logs/index.ts

@@ -2,23 +2,8 @@
  * Log viewing utilities for Squid proxy logs
  */
 
-export { parseLogLine, extractDomain } from './log-parser';
 export { LogFormatter } from './log-formatter';
 export {
-  discoverLogSources,
-  selectMostRecent,
-  isContainerRunning,
-  validateSource,
   listLogSources,
 } from './log-discovery';
 export { streamLogs } from './log-streamer';
-export {
-  aggregateLogs,
-  loadAllLogs,
-  loadAndAggregate,
-  AggregatedStats,
-  DomainStats,
-} from './log-aggregator';
-export {
-  formatStats,
-} from './stats-formatter';

src/squid/domain-acl.ts

@@ -9,7 +9,7 @@ import {
 /**
  * Groups domains by their protocol restriction
  */
-export interface DomainsByProtocol {
+interface DomainsByProtocol {
   http: string[];
   https: string[];
   both: string[];
@@ -18,7 +18,7 @@ export interface DomainsByProtocol {
 /**
  * Groups patterns by their protocol restriction
  */
-export interface PatternsByProtocol {
+interface PatternsByProtocol {
   http: DomainPattern[];
   https: DomainPattern[];
   both: DomainPattern[];
@@ -50,7 +50,7 @@ export function formatDomainForSquid(domain: string): string {
 /**
  * Group plain domains by protocol
  */
-export function groupDomainsByProtocol(domains: PlainDomainEntry[]): DomainsByProtocol {
+function groupDomainsByProtocol(domains: PlainDomainEntry[]): DomainsByProtocol {
   const result: DomainsByProtocol = { http: [], https: [], both: [] };
   for (const entry of domains) {
     result[entry.protocol].push(entry.domain);
@@ -61,7 +61,7 @@ export function groupDomainsByProtocol(domains: PlainDomainEntry[]): DomainsByPr
 /**
  * Group patterns by protocol
  */
-export function groupPatternsByProtocol(patterns: DomainPattern[]): PatternsByProtocol {
+function groupPatternsByProtocol(patterns: DomainPattern[]): PatternsByProtocol {
   const result: PatternsByProtocol = { http: [], https: [], both: [] };
   for (const pattern of patterns) {
     result[pattern.protocol].push(pattern);