Skip to content

refactor: split server flix.ts and handlers.ts into domain-focused modules#541

Merged
magnus-madsen merged 1 commit into
masterfrom
refactor/split-server-modules
Mar 30, 2026
Merged

refactor: split server flix.ts and handlers.ts into domain-focused modules#541
magnus-madsen merged 1 commit into
masterfrom
refactor/split-server-modules

Conversation

@magnus-madsen
Copy link
Copy Markdown
Member

@magnus-madsen magnus-madsen commented Mar 30, 2026

Summary

  • Split engine/flix.ts (268 lines) into config.ts (type definitions), process.ts (Flix process lifecycle), and workspace.ts (file/pkg/jar operations)
  • Split handlers/handlers.ts (388 lines) into lifecycle.ts, navigation.ts, editing.ts, symbols.ts, semantic.ts, and resources.ts by domain concern
  • Updated barrel re-exports in engine/index.ts and handlers/index.ts; no changes to server.ts or other consumers

Test plan

  • tsc --noEmit passes with no errors
  • Verify extension activates and connects to Flix compiler
  • Verify LSP features work (hover, go-to-definition, completion, diagnostics)

🤖 Generated with Claude Code

…dules

Split engine/flix.ts (268 lines) into config.ts (types), process.ts
(lifecycle), and workspace.ts (file/pkg/jar ops). Split
handlers/handlers.ts (388 lines) into lifecycle.ts, navigation.ts,
editing.ts, symbols.ts, semantic.ts, and resources.ts by concern.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@magnus-madsen magnus-madsen merged commit 33a6ab3 into master Mar 30, 2026
9 checks passed
@magnus-madsen magnus-madsen deleted the refactor/split-server-modules branch March 30, 2026 05:30
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.

1 participant