Skip to content

fix(tools): symbol search includes projects in virtual folder - #121

Open
wd357dui wants to merge 1 commit into
CodingWithCalvin:mainfrom
wd357dui:fix/tools/symbol-search-solution-folders
Open

wd357dui wants to merge 1 commit into
CodingWithCalvin:mainfrom
wd357dui:fix/tools/symbol-search-solution-folders

Conversation

@wd357dui

@wd357dui wd357dui commented Nov 9, 2026

Copy link
Copy Markdown

Description

Projects located inside a solution folder (Solution Explorer's virtual folder) weren't included in a symbol search; as a result, symbol_workspace wasn't able to find any symbols located in those projects; consequently, test_run couldn't match unit test projects located in solution folders either. This fix addresses this problem by adding an "is this a solution folder?" check in the solution items enumeration logic in the collect symbol path.

Type of Change

  • feat - New feature
  • fix - Bug fix
  • docs - Documentation only
  • refactor - Code change that neither fixes a bug nor adds a feature
  • test - Adding or updating tests
  • chore - Maintenance tasks
  • ci - CI/CD changes

Related Issues

Checklist

  • My code follows the project's code style
  • I have tested my changes locally
  • I have updated documentation if needed
  • My commits follow the conventional commit format
  • This PR has a descriptive title using conventional commit format

Screenshots (if applicable)

Additional Notes


Projects located inside a solution folder (solution explorer's virtual folder) weren't included in a symbol search; as a result, `symbol_workspace` wasn't able to find any symbols located in those projects; consequently, `test_run` couldn't match unit test projects located in solution folders either. This fix addresses this problem by adding an "is this a solution folder?" check in the solution items enumeration logic in the collect symbol path.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant