Repository navigation
Issue Navigator - #2006
Issue Navigator#2006FastestMolasses wants to merge 17 commits into
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR introduces the Issue Navigator sidebar, which displays code errors and warnings. Key changes include:
- Adding a new IssueNavigatorViewModel and associated data structures for managing diagnostics.
- Integrating new Issue Navigator UI components into the NavigatorArea.
- Updating the LSP service and WorkspaceDocument to support the new issue tracking.
Reviewed Changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated no comments.
Show a summary per file
Comments suppressed due to low confidence (1)
CodeEdit/Features/Documents/WorkspaceDocument/WorkspaceDocument.swift:37
- The IssueNavigatorViewModel is initialized with a hardcoded project name "Test". Consider retrieving the actual project name from the root folder to avoid hardcoding.
var issueNavigatorViewModel: IssueNavigatorViewModel? = IssueNavigatorViewModel(projectName: "Test")
thecoolwinter
left a comment
thecoolwinter
left a comment
There was a problem hiding this comment.
A few thoughts, love the progress here. Hard to test without some of the more recent changes on main due to some crashes happening.
thecoolwinter
left a comment
thecoolwinter
left a comment
There was a problem hiding this comment.
Looks good! I am struggling to test it consistently with a real language server, but not because of the components you've added here. I think we may need to add a call to the pull diagnostics server capability to round out the diagnostics fetching, but I say lets add an issue for that and come back to it.
Description
Adds the Issue Navigator side menu to allow users to see the errors and warnings in their codebase.
Related Issues
Checklist
Screenshots