default rootUri

[mattn]

castwide/solargraph#151

rootUr should be able to set null for default behavior.

[prabirshrestha]

Read the spec seems like all version v1,v2 and v3 supports null. We should fix it.

[prabirshrestha]

Now that I think of it this case was already supported when I implemented it.

vim-lsp/autoload/lsp.vim

Lines 380 to 392 in e090d84

	let l:server_info = l:server['server_info']
	if has_key(l:server_info, 'root_uri')
	let l:root_uri = l:server_info['root_uri'](l:server_info)
	else
	let l:root_uri = lsp#utils#get_default_root_uri()
	endif
	if empty(l:root_uri)
	let l:msg = s:new_rpc_error('ignore initialization lsp server due to empty root_uri', { 'server_name': a:server_name, 'lsp_id': l:server['lsp_id'] })
	call lsp#log(l:msg)
	call a:cb(l:msg)
	return
	endif

If you provide root_uri as function.

function! s:find_root_uri(server_name) abort
    " add your custom logic here
    " return empty string if you want to vim-lsp to ignore initialization
endfunction

 au User lsp_setup call lsp#register_server({
        \ 'name': 'solargraph',
        \ 'cmd': {server_info->['solargraph']},
        \ 'root_uri': function('s:find_root_uri'),
        \ 'whitelist': ['python'],
        \ })

Does it mean let l:root_uri = lsp#utils#get_default_root_uri() is not a good default instead?

If there was a plugin specific to solargraph, then it is easier for the users then users don't need to do to much. I actually have created org for these sort of plugins. https://github.com/vimlsp. Unforutnately vim-lsp was taken already.

Plug 'vimlsp/solargraph'

[mattn]

Cool. Now I sent e-mail to GitHub to ask that we can use vim-lsp if they are not active.

[prabirshrestha]

That would be great.

[mattn]

Unfortunately, vim-lsp seems to be active on private repos.

[prabirshrestha]

@mattn Did 'root_uri': function('s:find_root_uri'), solve the issue?

[mattn]

I am convinced that it will work, but solargraph still does not go well. Hmm. I tried {x->0}. But i don't try other yet.

[prabirshrestha]

Closing for now. I think it should be possible.