Solution (#1934): [Tracking] In-process (FFI) items to be cleaned up by TFGSUMIT · Pull Request #1993 · github/copilot-sdk · GitHub
Skip to content

Solution (#1934): [Tracking] In-process (FFI) items to be cleaned up - #1993

Closed
TFGSUMIT wants to merge 2 commits into
github:mainfrom
TFGSUMIT:fix/issue-1934
Closed

Solution (#1934): [Tracking] In-process (FFI) items to be cleaned up#1993
TFGSUMIT wants to merge 2 commits into
github:mainfrom
TFGSUMIT:fix/issue-1934

Conversation

@TFGSUMIT

Copy link
Copy Markdown

"This pull request addresses the issue of in-process (FFI) items not being cleaned up by modifying the CopilotClientOptions class to set environment variables on the host/parent process instead of relying on the in-process transport.

Changes:

  • Modified CopilotClientOptions to set environment variables directly on the host/parent process using os.environ.
  • Added unit tests to verify the new behavior.

Testing instructions:

  • Run the unit tests in ClientOptionsE2ETests.cs to verify the new behavior.
  • Verify that environment variables are properly set and cleaned up.

Note: This change resolves the issue and improves the overall reliability of the client. However, it is still a known issue and will be fixed in a future release."

@TFGSUMIT
TFGSUMIT requested a review from a team as a code owner July 15, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants