Skip to content

Conversation

@edmundmiller
Copy link

@edmundmiller edmundmiller commented Sep 15, 2025

Summary

  • Add built-in Nextflow Language Server support for .nf and .config files
  • Auto-download language server JAR from GitHub releases
  • Increase LSP initialization timeout for slower Java-based servers

Changes

  • Add Nextflow file extensions to language mapping
  • Implement NextflowLS server with automatic JAR download
  • Enhanced error handling and logging for LSP troubleshooting
  • Increased LSP client timeout from 5s to 15s

Testing

  • LSP server initializes successfully
  • Nextflow files are recognized and opened
  • JAR download works automatically
  • Integration follows existing LSP patterns

edmundmiller and others added 2 commits September 15, 2025 13:06
- Add .nf and .config file extensions to language mapping
- Implement NextflowLS server with automatic JAR download from GitHub releases
- Support root directory detection prioritizing nextflow.config files
- Require Java 17+ for running the language server
- Auto-download language-server-all.jar from nextflow-io/language-server releases

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Add comprehensive logging throughout Nextflow LSP spawn function
- Improve JAR download with proper arrayBuffer handling and validation
- Add file size validation and cleanup on download failure
- Increase LSP client initialization timeout from 5s to 15s for slower servers
- Add detailed error logging for troubleshooting initialization issues

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@edmundmiller edmundmiller marked this pull request as ready for review September 15, 2025 21:48
@drewbitt
Copy link

.config being mapped to nextflow feels wrong

@edmundmiller
Copy link
Author

.config being mapped to nextflow feels wrong

I could switch it to nextflow.config, but typically we add .config after other config files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants