bufbuild
Add an allowlist of which clients can use the LSP server
This PR uses the client's reported identity to allowlist which clients we support, with a breakout flag called --break-warranty-seal.
Initially allowlisted are several VSCode variants and nvim, along with comments for where those names came from.
The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).
| Build | Format | Lint | Breaking | Updated (UTC) |
|---|---|---|---|---|
✅ passed | ✅ passed | ✅ passed | ✅ passed | Nov 25, 2024, 6:37 PM |
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Currently blocked on a decision on the set of clients to include. We intend to merge this but not before we make a decision on this front.
I don't think we should touch this PR until we are able to have a convo with the right folks about what we are officially supporting, but when we do come back to this, I think a better name for this flag would be --allow-unsupported-clients.
Should we close this now?
I think we should close this, I don't think we're intending on restricting the client.
