Skip to content

Show internal symbols#148

Merged
jjonescz merged 11 commits into
mainfrom
internal-symbols
May 30, 2026
Merged

Show internal symbols#148
jjonescz merged 11 commits into
mainfrom
internal-symbols

Conversation

@jjonescz

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings May 30, 2026 14:03
@jjonescz jjonescz added the enhancement New feature or request label May 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for selecting which symbol representations are shown in tree output, including Roslyn internal symbols, and updates related build/test configuration.

Changes:

  • Replaces the tree “Symbols” boolean with SymbolDisplayKinds.
  • Adds Roslyn internal symbol accessors and TreeFormatter handling for internal/public symbol display.
  • Updates snapshots, package/build settings, launch settings, and VS Code watch tasks.

Reviewed changes

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

Show a summary per file
File Description
test/UnitTests/TreeFormatterTests.cs Adds symbol display test coverage and updates formatter options.
test/UnitTests/Snapshots/TreeFormatterTests.BoundBody.verified.txt Updates snapshot output for newer Roslyn line numbers.
src/WebAssembly/Properties/launchSettings.json Disables automatic browser launch for local profiles.
src/Shared/ICompiler.cs Introduces SymbolDisplayKinds and updates preferences.
src/RoslynCodeStyleAccess/RoslynCodeStyleAccess.csproj Removes code-style analyzers after asset resolution.
src/RoslynAccess/RoslynAccessors.cs Adds helpers for internal Roslyn symbol access/equality.
src/Compiler/TreeFormatter.cs Adds public/internal symbol display filtering and underlying symbol traversal.
src/Compiler/Compiler.cs Passes symbol display kind preferences into tree formatting.
src/App/Lab/SavedState.cs Persists symbol display kind in saved state/preferences.
src/App/Lab/Page.razor Replaces the symbol checkbox with a select control.
eng/BuildTools/BuildTools.csproj Adjusts Roslyn package version override behavior.
eng/AppEntryPoint.targets Removes code-style analyzers after asset resolution.
Directory.Packages.props Updates Roslyn/Razor version properties.
Directory.Build.props Adds compiler-toolset/caching compiler switches.
.vscode/tasks.json Updates server watch args and adds a wasm watch task.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/App/Lab/SavedState.cs
Comment thread src/App/Lab/Page.razor
@jjonescz jjonescz merged commit 5262f34 into main May 30, 2026
5 checks passed
@jjonescz jjonescz deleted the internal-symbols branch May 30, 2026 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants