Skip to content

fix: Real support for .Net xUint test#67

Open
Yumitoya8569 wants to merge 4 commits intomicrosoft:mainfrom
Yumitoya8569:main
Open

fix: Real support for .Net xUint test#67
Yumitoya8569 wants to merge 4 commits intomicrosoft:mainfrom
Yumitoya8569:main

Conversation

@Yumitoya8569
Copy link
Copy Markdown

Current progress

The current version (1.1.3) does not work when running C# xUnit tests at all.
The reason is that the test process is not attached to the debugger.
This PR uses a workaround to attach it.

Follow-up

Currently, it uses a presumed name (testhost) to scan processes and obtain the PID.
There should be a better approach in the future.

@Yumitoya8569
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@Yumitoya8569
Copy link
Copy Markdown
Author

Yumitoya8569 commented May 8, 2026

Update

  1. Fixed an issue where the server would fail to start if the .csproj and .cs files were not in the same directory.
  2. Fixed an issue where continuous requests from the agent would cause the server to crash.

Follow-up

Continuous requests cause the server to crash due to the transport implementation.
Additionally, the singleton stateless mode described in the StreamableHTTPServerTransport doc does not actually work.
So for now, just fixed it with a workaround.
There should be a better approach in the future.

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.

1 participant