Skip to content

Commit

Permalink
fix(biome): send correct language to lsp proxy (#4773)
Browse files Browse the repository at this point in the history
Since Biome understands `typescriptreact` and `javascriptreact` as
languages, we can send the `filetype` to the LSP, rather than only
sending `typescript` for both `ts` and `tsx` files, or `javascript` for
`js` and `jsx` files.

fixes: #4752
  • Loading branch information
redbmk authored Jun 24, 2024
1 parent f2aef2f commit 5606606
Show file tree
Hide file tree
Showing 4 changed files with 23 additions and 0 deletions.
1 change: 1 addition & 0 deletions ale_linters/javascript/biome.vim
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
call ale#linter#Define('javascript', {
\ 'name': 'biome',
\ 'lsp': 'stdio',
\ 'language': function('ale#handlers#biome#GetLanguage'),
\ 'executable': function('ale#handlers#biome#GetExecutable'),
\ 'command': function('ale#handlers#biome#GetCommand'),
\ 'project_root': function('ale#handlers#biome#GetProjectRoot'),
Expand Down
1 change: 1 addition & 0 deletions ale_linters/typescript/biome.vim
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
call ale#linter#Define('typescript', {
\ 'name': 'biome',
\ 'lsp': 'stdio',
\ 'language': function('ale#handlers#biome#GetLanguage'),
\ 'executable': function('ale#handlers#biome#GetExecutable'),
\ 'command': function('ale#handlers#biome#GetCommand'),
\ 'project_root': function('ale#handlers#biome#GetProjectRoot'),
Expand Down
4 changes: 4 additions & 0 deletions autoload/ale/handlers/biome.vim
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,10 @@ function! ale#handlers#biome#GetCommand(buffer) abort
\ . (!empty(l:options) ? ' ' . l:options : '')
endfunction

function! ale#handlers#biome#GetLanguage(buffer) abort
return getbufvar(a:buffer, '&filetype')
endfunction

function! ale#handlers#biome#GetProjectRoot(buffer) abort
let l:biome_file = ale#path#FindNearestFile(a:buffer, 'biome.json')

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,20 @@ Execute(The biome command should accept options):
let g:ale_biome_options = '--foobar'

AssertLinter 'biome', ale#Escape('biome') . ' lsp-proxy --foobar'

Execute(Uses the filetype as the language):
call ale#test#SetFilename('test.ts')
set filetype=typescript
AssertLSPLanguage 'typescript'

call ale#test#SetFilename('test.tsx')
set filetype=typescriptreact
AssertLSPLanguage 'typescriptreact'

call ale#test#SetFilename('test.js')
set filetype=javascript
AssertLSPLanguage 'javascript'

call ale#test#SetFilename('test.jsx')
set filetype=javascriptreact
AssertLSPLanguage 'javascriptreact'

0 comments on commit 5606606

Please sign in to comment.