Update FICClientId retrieval to use connection-specific environment variable

[rido-min]

This pull request updates the handling of the FICClientId environment variable in the AuthConfiguration module to make it connection-specific, aligning it with other configuration properties. Corresponding test cases have been updated to reflect this change.

Updates to AuthConfiguration:

• packages/agents-hosting/src/auth/authConfiguration.ts: Modified the FICClientId property to use a connection-specific environment variable (${cnxName}_FICClientId) instead of a global one.

Updates to tests:

• packages/agents-hosting/test/hosting/authConfiguration.test.ts: Updated test cases to expect the FICClientId property to be undefined when the connection-specific environment variable is not set, instead of falling back to the global FICClientId. [1] [2]

[Copilot]

Pull Request Overview

Update retrieval of FICClientId in AuthConfiguration to use connection-specific environment variable instead of the global one

• Changed source loader to fetch FICClientId from ${cnxName}_FICClientId
• Updated tests to expect undefined when no connection-specific FICClientId is set

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/agents-hosting/src/auth/authConfiguration.ts	Use connection-specific env var for FICClientId instead of global
packages/agents-hosting/test/hosting/authConfiguration.test.ts	Updated expected FICClientId to undefined, reflecting removal of global fallback

Comments suppressed due to low confidence (3)

packages/agents-hosting/test/hosting/authConfiguration.test.ts:100

• The inline comment is now inaccurate: FICClientId no longer falls back to the global variable. Update it to describe the new behavior, e.g., "// No connection-specific FICClientId set".

      assert.strictEqual(config.FICClientId, undefined) // Falls back to global FICClientId

packages/agents-hosting/test/hosting/authConfiguration.test.ts:122

• Add or update an inline comment here to explain why FICClientId is undefined instead of falling back to the global variable.

      assert.strictEqual(config.FICClientId, undefined)

packages/agents-hosting/src/auth/authConfiguration.ts:99

• Add a unit test that sets a connection-specific FICClientId environment variable and verifies that loadAuthConfigFromEnv correctly retrieves it.

      FICClientId: process.env[`${cnxName}_FICClientId`],