Skip to content

use likec4-lsp as the LSP command#4

Open
spaasis wants to merge 1 commit into
likec4:mainfrom
spaasis:patch-1
Open

use likec4-lsp as the LSP command#4
spaasis wants to merge 1 commit into
likec4:mainfrom
spaasis:patch-1

Conversation

@spaasis
Copy link
Copy Markdown

@spaasis spaasis commented Apr 22, 2026

By the documentation likec4-lsp is the command to use. Tested on my machine - running
npm install -g @likec4/lsp added the likec4-lsp command, and using that in this plugin made the LSP attach properly

Signed-off-by: spaasis <17126867+spaasis@users.noreply.github.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 22, 2026

📝 Walkthrough

Walkthrough

Updated the LSP language server invocation in lsp/likec4.lua to use a standalone binary likec4-lsp instead of the subcommand approach. Added documentation reference for the standalone language server.

Changes

Cohort / File(s) Summary
LSP Configuration
lsp/likec4.lua
Updated cmd field from { "likec4", "lsp", "--stdio" } to { "likec4-lsp", "--stdio" }. Added documentation link comment referencing the standalone language server.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided by the author, making it impossible to assess relevance to the changeset. Add a pull request description explaining the motivation and impact of switching to the standalone likec4-lsp binary.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: updating the LSP command from 'likec4' to 'likec4-lsp'.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@lsp/likec4.lua`:
- Around line 3-5: The change to use the standalone binary in the LSP config
(cmd = { "likec4-lsp", "--stdio" }) is a breaking change for users who only have
the main likec4 CLI; update the project’s release notes and this file’s top
comment to document the migration path: state that users can either install the
new `@likec4/lsp` package to get the likec4-lsp binary or continue using the main
CLI by configuring the command as likec4 lsp --stdio (e.g., cmd = { "likec4",
"lsp", "--stdio" }) and include exact install instructions (npm install -g
`@likec4/lsp`) and a short sentence about versions (since v1.54.0) so users know
when the standalone binary became available.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f3a2dd65-0dbb-4ac0-a0ee-a9a88552d248

📥 Commits

Reviewing files that changed from the base of the PR and between cf307c7 and 5dfc5ea.

📒 Files selected for processing (1)
  • lsp/likec4.lua

Comment thread lsp/likec4.lua
Comment on lines +3 to +5
-- This assumes 'likec4-lsp' is in your system's PATH.
-- https://likec4.dev/tooling/editors/#standalone-language-server
cmd = { "likec4-lsp", "--stdio" },
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🌐 Web query:

Does likec4 provide a standalone likec4-lsp binary, and what is the correct command syntax to start it?

💡 Result:

Yes, LikeC4 provides a standalone likec4-lsp binary via the @likec4/lsp npm package (new in v1.54.0, April 2026) or previously via the main likec4 CLI and @likec4/language-server package. To install and start: 1. Global install: npm install -g @likec4/lsp (provides likec4-lsp binary, self-contained with zero dependencies). 2. Or install main CLI: npm install -g likec4 (provides likec4 lsp command). Correct command syntax (one transport option required): - likec4-lsp --stdio - likec4 lsp --stdio - likec4 lsp --node-ipc - likec4 lsp --socket=3000 - likec4 lsp --pipe /tmp/likec4.pipe Additional options: --watch (enable watcher), --no-manual-layouts, --use-dot. Note: Older @likec4/language-server provided likec4-language-server --stdio but is now internal/deprecated in favor of main likec4 package or new @likec4/lsp.

Citations:


Document migration path for the breaking change to standalone binary.

The change to likec4-lsp --stdio is technically correct (the standalone binary exists via the @likec4/lsp npm package as of v1.54.0) and the command syntax is valid per the official documentation.

However, this remains a breaking change for existing users: those with the likec4 CLI installed but not the new @likec4/lsp package will lose LSP functionality after upgrading. Ensure release notes clearly document that users must either:

  • Install the new @likec4/lsp package for the standalone binary, or
  • Switch to likec4 lsp --stdio if they have the main likec4 CLI installed
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@lsp/likec4.lua` around lines 3 - 5, The change to use the standalone binary
in the LSP config (cmd = { "likec4-lsp", "--stdio" }) is a breaking change for
users who only have the main likec4 CLI; update the project’s release notes and
this file’s top comment to document the migration path: state that users can
either install the new `@likec4/lsp` package to get the likec4-lsp binary or
continue using the main CLI by configuring the command as likec4 lsp --stdio
(e.g., cmd = { "likec4", "lsp", "--stdio" }) and include exact install
instructions (npm install -g `@likec4/lsp`) and a short sentence about versions
(since v1.54.0) so users know when the standalone binary became available.

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