-
Notifications
You must be signed in to change notification settings - Fork 1.8k
feat(lsp): add tsgo language server support #1083
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
|
All contributors have signed the CLA. Thank you! ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 7 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
|
@sisyphus-dev-ai is this supported also on opencode by native? ulw |
|
No, tsgo is NOT natively supported by opencode. I read opencode's actual LSP source code ( Built-in TypeScript LSP in opencodeThe export const Typescript: Info = {
id: "typescript",
// ...
async spawn(root) {
const tsserver = await Bun.resolve("typescript/lib/tsserver.js", Instance.directory).catch(() => {})
// ...
const proc = spawn(BunProc.which(), ["x", "typescript-language-server", "--stdio"], { cwd: root, ... })
return { process: proc, initialization: { tsserver: { path: tsserver } } }
},
}No mention of Community interest exists but no actionIssue #67 ("[docs] how to use typescript-go lsp") was opened by a community member with a manual config workaround, but:
ConclusionThis PR adds value that opencode doesn't provide natively. Users can currently only configure tsgo manually via opencode's |
|
@code-yeongyu actually, I think the install hint is misleading. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 7 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Summary by cubic
Add tsgo as a built-in TypeScript LSP and prefer it when installed, with automatic fallback to typescript-language-server.
New Features
Dependencies
Written for commit 8175e88. Summary will update on new commits.