Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Incompatible with any sublime-workspace? #109

Closed
Sevans711 opened this issue May 20, 2023 · 3 comments · Fixed by sublimelsp/LSP#2276
Closed

Incompatible with any sublime-workspace? #109

Sevans711 opened this issue May 20, 2023 · 3 comments · Fixed by sublimelsp/LSP#2276
Labels
upstream Upstream issue

Comments

@Sevans711
Copy link

When I open a sublime workspace (e.g., file.sublime-workspace), I encounter the error below. It occurs repeatedly as LSP-copilot continues trying and failing to initialize.

Other notes:

  • This error occurs for workspaces I created before installing LSP-copilot & also for workspaces I created after intalling LSP-copilot.
  • If I restart sublime & open a file without using a sublime-workspace, then LSP-copilot seems to be working properly.

The error:

failed to register session LSP-copilot to listener ViewListener(201)
Traceback (most recent call last):
  File "/Users/Sevans/Library/Application Support/Sublime Text 3/Installed Packages/LSP.sublime-package/plugin/core/windows.py", line 183, in _publish_sessions_to_listener_async
    listener.on_session_initialized_async(session)
  File "/Users/Sevans/Library/Application Support/Sublime Text 3/Installed Packages/LSP.sublime-package/plugin/documents.py", line 211, in on_session_initialized_async
    self._session_views[session.config.name] = SessionView(self, session, self._uri)
  File "/Users/Sevans/Library/Application Support/Sublime Text 3/Installed Packages/LSP.sublime-package/plugin/session_view.py", line 54, in __init__
    session_buffer = SessionBuffer(self, buffer_id, uri)
  File "/Users/Sevans/Library/Application Support/Sublime Text 3/Installed Packages/LSP.sublime-package/plugin/session_buffer.py", line 149, in __init__
    self._check_did_open(view)
  File "/Users/Sevans/Library/Application Support/Sublime Text 3/Installed Packages/LSP.sublime-package/plugin/session_buffer.py", line 174, in _check_did_open
    self.do_document_diagnostic_async(view, version)
  File "/Users/Sevans/Library/Application Support/Sublime Text 3/Installed Packages/LSP.sublime-package/plugin/session_buffer.py", line 460, in do_document_diagnostic_async
    if mgr.should_ignore_diagnostics(self.last_known_uri, self.session.config):
  File "/Users/Sevans/Library/Application Support/Sublime Text 3/Installed Packages/LSP.sublime-package/plugin/core/windows.py", line 364, in should_ignore_diagnostics
    if self._workspace.includes_excluded_path(path):
  File "/Users/Sevans/Library/Application Support/Sublime Text 3/Installed Packages/LSP.sublime-package/plugin/core/workspace.py", line 107, in includes_excluded_path
    exclude_patterns = self._folders_exclude_patterns[i]
IndexError: list index out of range
@jfcherng
Copy link
Collaborator

jfcherng commented May 20, 2023 •
edited
Loading

I use ST's "project" feature (project + workspace) on daily basis and have never encountered this...

Not sure why LSP-copilot has so many "weird" issues since it's just a normal LSP client like other Node.js-based ones...

It's a LSP's issue. You may try to reinstall LSP and if it's in vain, I suggest report it in LSP's repo.

@jfcherng jfcherng added the upstream Upstream issue label May 20, 2023
@rchl rchl mentioned this issue May 20, 2023
Merged
@rchl
Copy link

rchl commented May 20, 2023

Think sublimelsp/LSP#2276 should fix it.

@jfcherng
Copy link
Collaborator

jfcherng commented May 23, 2023 •
edited
Loading

Closed since upstream's fix has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
upstream Upstream issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix crash on checking excluded folders with missing project data sublimelsp/LSP
3 participants
@rchl @jfcherng @Sevans711