Bug - explainshell services has no /api path, so arg tips doesn't work

[palansher]

Hello!

Thanks you for nice tool!

bash-language-server/server/src/analyser.ts

Line 636 in afd0d82

const url = `${endpoint}/api/explain?${searchParams}`

If replace

const url = `${endpoint}/api/explain?${searchParams}`

to

const url = `${endpoint}/explain?${searchParams}`

then will be working requests like this:
https://explainshell.com/api/explain?cmd=grep+-v+%22current%22

to https://explainshell.com or custom buit https://github.com/idank/explainshell

The reason of that: there is no /api path supported on known explainshell services.

Currently all requests to /api return 404 error

[skovhus]

Thanks, they must have changed their URL. Are you up for doing a PR?

There is some documentation and a security warning here:
https://github.com/bash-lsp/bash-language-server/tree/main/vscode-client#configuration

[palansher]

Are you up for doing a PR

Hello! Maybe I can. If you kindly explain what is 'PR' , you mentioned :)

The Pull Request? I am a junior coder, But I will try.

[skovhus]

Yes, I meant a pull request. :)