From 07fe8d0944ddd0beea09e61acff4380af477ca43 Mon Sep 17 00:00:00 2001 From: Landon Cox Date: Mon, 29 Jun 2026 09:20:13 -0700 Subject: [PATCH 1/2] fix: export missing symbols required by tests - Export secureWipeFile from ssl-key-storage.ts (test imports it) - Export buildProviderTargetEnv from api-proxy-env-config.ts (test uses it) - Add missing import of buildProviderTargetEnv in api-proxy-service-split.test.ts Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- src/services/api-proxy-env-config.ts | 2 +- src/services/api-proxy-service-split.test.ts | 2 +- src/ssl-key-storage.ts | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/services/api-proxy-env-config.ts b/src/services/api-proxy-env-config.ts index 2d2c59bb9..6b09133b1 100644 --- a/src/services/api-proxy-env-config.ts +++ b/src/services/api-proxy-env-config.ts @@ -9,7 +9,7 @@ import { NetworkConfig } from './squid-service'; * Builds provider API target/basePath environment variables for the api-proxy container. * Centralizes the repetitive per-provider target/basePath conditional env generation. */ -function buildProviderTargetEnv(config: WrapperConfig): Record { +export function buildProviderTargetEnv(config: WrapperConfig): Record { const copilotProviderType = config.copilotProviderType || getConfigEnvValue(config, COPILOT_ENV.PROVIDER_TYPE); const copilotProviderBaseUrl = config.copilotProviderBaseUrl || getConfigEnvValue(config, COPILOT_ENV.PROVIDER_BASE_URL); const copilotProviderApiKey = config.copilotProviderApiKey; diff --git a/src/services/api-proxy-service-split.test.ts b/src/services/api-proxy-service-split.test.ts index 0648b1188..e518d561e 100644 --- a/src/services/api-proxy-service-split.test.ts +++ b/src/services/api-proxy-service-split.test.ts @@ -3,7 +3,7 @@ import { parseImageTag } from '../image-tag'; import { COPILOT_PLACEHOLDER_TOKEN } from '../constants/placeholders'; import { baseConfig } from '../test-helpers/docker-test-fixtures.test-utils'; import { buildApiProxyServiceConfig } from './api-proxy-service-config'; -import { buildApiProxyBaseEnv } from './api-proxy-env-config'; +import { buildApiProxyBaseEnv, buildProviderTargetEnv } from './api-proxy-env-config'; import { buildApiProxyLifecycleConfig } from './api-proxy-lifecycle-config'; import { buildAgentCredentialEnv } from './api-proxy-credential-env'; diff --git a/src/ssl-key-storage.ts b/src/ssl-key-storage.ts index 85d159777..327349b27 100644 --- a/src/ssl-key-storage.ts +++ b/src/ssl-key-storage.ts @@ -78,7 +78,7 @@ export async function unmountSslTmpfs(sslDir: string): Promise { * * @param filePath - Path to the file to securely wipe */ -function secureWipeFile(filePath: string): void { +export function secureWipeFile(filePath: string): void { let fd: number | undefined; try { From 56fb1bdd0e41a5fdae172e39278ab0c3a81105ef Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 29 Jun 2026 16:44:46 +0000 Subject: [PATCH 2/2] chore: mark test-only exports as internal --- src/services/api-proxy-env-config.ts | 1 + src/ssl-key-storage.ts | 1 + 2 files changed, 2 insertions(+) diff --git a/src/services/api-proxy-env-config.ts b/src/services/api-proxy-env-config.ts index 6b09133b1..35033305b 100644 --- a/src/services/api-proxy-env-config.ts +++ b/src/services/api-proxy-env-config.ts @@ -8,6 +8,7 @@ import { NetworkConfig } from './squid-service'; /** * Builds provider API target/basePath environment variables for the api-proxy container. * Centralizes the repetitive per-provider target/basePath conditional env generation. + * @internal Exported for testing */ export function buildProviderTargetEnv(config: WrapperConfig): Record { const copilotProviderType = config.copilotProviderType || getConfigEnvValue(config, COPILOT_ENV.PROVIDER_TYPE); diff --git a/src/ssl-key-storage.ts b/src/ssl-key-storage.ts index 327349b27..10b5d27ce 100644 --- a/src/ssl-key-storage.ts +++ b/src/ssl-key-storage.ts @@ -77,6 +77,7 @@ export async function unmountSslTmpfs(sslDir: string): Promise { * cannot be guaranteed on all filesystems (for example, journaling/COW). * * @param filePath - Path to the file to securely wipe + * @internal Exported for testing */ export function secureWipeFile(filePath: string): void { let fd: number | undefined;