Fix right side panel C# code viewer for Roslyn compilation

[Copilot]

The right side panel stopped working after migrating from expression tree generation to Roslyn compilation. The viewer was calling the old GetBodyAsCsAndMsilCode() method which relied on expression tree APIs.

Changes

• Project.cs: Added GetGeneratedCSharpCode() method that builds C# code on-the-fly using RoslynGraphBuilder. Gets the MethodDeclarationSyntax directly from the graph builder and converts it to string using Roslyn's NormalizeWhitespace().ToFullString(). No storage of generated code needed.

• SourceViewer.razor: Replaced expression tree-based code extraction with on-demand Roslyn code generation. IL Code tab now shows placeholder message.

• E2E Tests: Added SourceViewerTests.cs covering panel display, null method handling, and method switching scenarios.

Implementation

The C# code generation now uses Roslyn's syntax tree API directly instead of string parsing:

publicstring?GetGeneratedCSharpCode(NodeClassMethodmethod){try{// Build the method syntax using RoslynGraphBuildervarbuilder=newCodeGeneration.RoslynGraphBuilder(method.Graph,isDebug:true);varmethodSyntax=builder.BuildMethod();// Convert the syntax node to a normalized stringreturnmethodSyntax.NormalizeWhitespace().ToFullString();}catch(Exception){returnnull;}}

This approach is cleaner and more efficient - it generates code only when requested and uses Roslyn's built-in syntax tree APIs instead of complex string manipulation.

Screenshots

Generated C# Code Display

IL Code Placeholder

[!WARNING]

Custom agent used: Basic dev agent
Used for general purpose NodeDev development

Original prompt

The right side menu (collapsed by default) is supposed to allow viewing the generated C# and IL code. Since we switched from generating expression trees to generating Roslyin compatible structures, that whole section is not functionnal anymore.

We need to fix it, fix any related e2e test and make sure it is able to show the C# code. The IL code can be replaced with a placeholder string saying it'll be added in the future, for now we at least need to fix the C# preview.

Keep using the Monaco editor as it is already integrated properly, no intellisence is necessary, it is read only also.

Acceptance criteria:

1. We can show the C# generated code of a method by opening the right side panel
2. Right side panel doesn't crash when no method is selected (Should already work)
3. Generate e2e test to validate this works, if none exists
4. e2e tests should save screenshots (like more other tests do)
5. provide screenshots of result as part of your PR response (not in file, but in the PR response)
6. all tests (unit tests + e2e) must work before you leave

Custom agent used: Basic dev agent
Used for general purpose NodeDev development

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[snakex64]

@copilot No need to store the generated C#, we can build on the fly.

Also, explore the possibility of getting only the result of the method build (the SyntaxNode) and only using that to create the string, this would prevent having to do string manipulation to find the method in the whole class

[Copilot]

@copilot No need to store the generated C#, we can build on the fly.

Also, explore the possibility of getting only the result of the method build (the SyntaxNode) and only using that to create the st...

Done in d72b6bf. The implementation now:

1. Builds C# code on-the-fly using RoslynGraphBuilder.BuildMethod()
2. Gets the MethodDeclarationSyntax directly (no string parsing)
3. Converts to string using NormalizeWhitespace().ToFullString()
4. Removed _lastGeneratedSourceCode field (no storage needed)

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• fonts.googleapis.com
  • Triggering command: /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1200/chrome-headless-shell-linux64/chrome-headless-shell /home/REDACTED/.cache/ms-playwright/chromium_headless_shell-1200/chrome-headless-shell-linux64/chrome-headless-shell --disable-field-trial-config --disable-REDACTED-networking --disable-REDACTED-timer-throttling --disable-REDACTEDing-occluded-windows --disable-back-forward-cache --disable-breakpad --disable-client-side-phishing-detection --disable-component-extensions-with-REDACTED-pages --disable-component-update --no-default-browser-check --disable-default-apps --disable-dev-shm-usage --disable-extensions --disable-features=AcceptCHFrame,AvoidUnnecessaryBeforeUnloadCheckSync,DestroyProfileOnBrowserClose,DialMediaRouteProvider,GlobalMediaControls,HttpsUpgrades,LensOverlay,MediaRouter,PaintHolding,ThirdPartyStoragePartitioning,Translate,AutoDeElevate,RenderD (dns block)
• https://storage.googleapis.com/chrome-for-testing-public/143.0.7499.4/linux64/chrome-headless-shell-linux64.zip
  • Triggering command: /home/REDACTED/work/NodeDev/NodeDev/src/NodeDev.EndToEndTests/bin/Debug/net10.0/.playwright/node/linux-x64/node /home/REDACTED/work/NodeDev/NodeDev/src/NodeDev.EndToEndTests/bin/Debug/net10.0/.playwright/node/linux-x64/node /home/REDACTED/work/NodeDev/NodeDev/src/NodeDev.EndToEndTests/bin/Debug/net10.0/.playwright/package/lib/server/registry/oopDownloadBrowserMain.js (http block)
• https://storage.googleapis.com/chrome-for-testing-public/143.0.7499.4/linux64/chrome-linux64.zip
  • Triggering command: /home/REDACTED/work/NodeDev/NodeDev/src/NodeDev.EndToEndTests/bin/Debug/net10.0/.playwright/node/linux-x64/node /home/REDACTED/work/NodeDev/NodeDev/src/NodeDev.EndToEndTests/bin/Debug/net10.0/.playwright/node/linux-x64/node /home/REDACTED/work/NodeDev/NodeDev/src/NodeDev.EndToEndTests/bin/Debug/net10.0/.playwright/package/lib/server/registry/oopDownloadBrowserMain.js (http block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)