fix: avoid crash on invalid URIs

[radeksimko]

The language server isn't able to do anything with invalid/unsupported URIs, such as the ones listed in hashicorp/vscode-terraform#1159

There's a broader question of whether these URIs should even be send to LS. Given that the known majority of such URIs come from VS Code, I created a separate ticket hashicorp/vscode-terraform#1159 to discuss the problem there.

Prior to this patch it would crash when encountered such a URI during initialize, textDocument/didOpen or workspace/didChangeWorkspaceFolders, as reported in #959

We already handle this gracefully in custom commands:

terraform-ls/internal/langserver/handlers/command/module_calls.go

Lines 42 to 44 in 000cb53

	if !uri.IsURIValid(modUri) {
	return response, fmt.Errorf("URI %q is not valid", modUri)
	}

terraform-ls/internal/langserver/handlers/command/module_callers.go

Lines 30 to 32 in 000cb53

	if !uri.IsURIValid(modUri) {
	return nil, fmt.Errorf("URI %q is not valid", modUri)
	}

terraform-ls/internal/langserver/handlers/command/module_providers.go

Lines 39 to 41 in 000cb53

	if !uri.IsURIValid(modUri) {
	return response, fmt.Errorf("URI %q is not valid", modUri)
	}

terraform-ls/internal/langserver/handlers/command/init.go

Lines 23 to 25 in 000cb53

	if !uri.IsURIValid(dirUri) {
	return nil, fmt.Errorf("URI %q is not valid", dirUri)
	}

terraform-ls/internal/langserver/handlers/command/validate.go

Lines 25 to 27 in 000cb53

	if !uri.IsURIValid(dirUri) {
	return nil, fmt.Errorf("URI %q is not valid", dirUri)
	}

[jpogran]

Until we figure out why we're getting so many non file URIs I think this is a great stop-gap for improving UX

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.