From 8b5543ec19bde3733bf058be67f2a2afc067fec8 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 6 Jan 2026 18:03:26 +0000
Subject: [PATCH] Simplify constructor by handling undefined in helper method

Co-authored-by: TingluoHuang <1750815+TingluoHuang@users.noreply.github.com>
---
 src/index.ts | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/src/index.ts b/src/index.ts
index e2e21f5..352fa10 100644
--- a/src/index.ts
+++ b/src/index.ts
@@ -147,9 +147,7 @@ export class HttpClient {
     handlers?: ifm.RequestHandler[],
     requestOptions?: ifm.RequestOptions
   ) {
-    this.userAgent = userAgent
-      ? this._getUserAgentWithOrchestrationId(userAgent)
-      : undefined
+    this.userAgent = this._getUserAgentWithOrchestrationId(userAgent)
     this.handlers = handlers || []
     this.requestOptions = requestOptions
     if (requestOptions) {
@@ -818,7 +816,12 @@ export class HttpClient {
     return proxyAgent
   }
 
-  private _getUserAgentWithOrchestrationId(userAgent: string): string {
+  private _getUserAgentWithOrchestrationId(
+    userAgent?: string
+  ): string | undefined {
+    if (!userAgent) {
+      return undefined
+    }
     const orchId = process.env['ACTIONS_ORCHESTRATION_ID']
     if (orchId) {
       // Sanitize the orchestration ID to ensure it contains only valid characters