From 67722d0f87cabaeeceb7ed50d146d2bd3c32b2a6 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 29 Jun 2026 13:57:44 +0000
Subject: [PATCH 1/2] Initial plan


From f80c128989c09a63ab98a28bf4b44e44df02d493 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 29 Jun 2026 14:06:54 +0000
Subject: [PATCH 2/2] fix: remove unused buildProviderTargetEnv export

---
 src/services/api-proxy-env-config.ts         | 2 +-
 src/services/api-proxy-service-split.test.ts | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/services/api-proxy-env-config.ts b/src/services/api-proxy-env-config.ts
index f8d28593..5fd515e1 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.
  */
-export function buildProviderTargetEnv(config: WrapperConfig): Record<string, string> {
+function buildProviderTargetEnv(config: WrapperConfig): Record<string, string> {
   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 ceaa0fc3..ff7007f5 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, buildProviderTargetEnv } from './api-proxy-env-config';
+import { buildApiProxyBaseEnv } from './api-proxy-env-config';
 import { buildApiProxyLifecycleConfig } from './api-proxy-lifecycle-config';
 import { buildAgentCredentialEnv } from './api-proxy-credential-env';
 
@@ -68,12 +68,12 @@ describe('API proxy split builders', () => {
       .toThrow('buildApiProxyLifecycleConfig: networkConfig.proxyIp is required');
   });
 
-  it('buildProviderTargetEnv trims explicit provider session id', () => {
-    const env = buildProviderTargetEnv({
+  it('buildApiProxyBaseEnv trims explicit provider session id', () => {
+    const env = buildApiProxyBaseEnv({
       ...baseConfig,
       workDir: '/tmp/awf-test',
       copilotByokSessionId: ' session-123 ',
-    });
+    }, networkConfig);
 
     expect(env.AWF_PROVIDER_SESSION_ID).toBe('session-123');
   });