docs: expand Emacs setup with Eglot and lsp-mode configs#2972
Conversation
Replaces the minimal `eglot-server-programs` stub in editors.mdx with a complete walkthrough covering Eglot, lsp-mode, and a `likec4-mode` major mode definition for `.c4` / `.likec4` files. Based on the working setup posted by @vincent067 in likec4#2268, adapted to use the standalone `likec4-lsp` binary from `@likec4/lsp` (consistent with the rest of the page). Resolves likec4#2268
🦋 Changeset detectedLatest commit: b913de0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 22 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR expands Emacs editor support documentation for LikeC4. A new changeset records a patch to ChangesEmacs LSP Setup Documentation
🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/docs/src/content/docs/tooling/editors.mdx`:
- Line 96: The text says "minor wrapper mode" but the snippet uses
define-derived-mode ... prog-mode (i.e., a major mode); change the wording to
"major wrapper mode" or otherwise indicate it's a major mode derived from
prog-mode to match the code (referencing the define-derived-mode ... prog-mode
declaration).
🪄 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: b48d741a-10ff-458f-9ff8-a30d7839a64d
📒 Files selected for processing (2)
.changeset/docs-emacs-lsp-setup.mdapps/docs/src/content/docs/tooling/editors.mdx
Snippet defines likec4-mode via define-derived-mode ... prog-mode, which is a major mode. Updates the prose to match. Addresses CodeRabbit review feedback on likec4#2972.
Checklist
mainbefore creating this PR.pnpm typecheckandpnpm test. (no source changes; docs unaffected by either)Summary
Resolves #2268.
The existing Emacs section in
apps/docs/.../editors.mdxis a one-lineeglot-server-programsstub plus a "see #2268 for discussion" link. This PR replaces it with a complete walkthrough so new Emacs users can copy-paste a working setup.The content is based on @vincent067's post in #2268 — @davydkov asked for it to be added to the docs there. Adapted from his original to use the standalone
likec4-lspbinary from@likec4/lsp, which is what the rest of this page recommends (instead oflikec4 lspfrom the main CLI).What's added
likec4-modemajor-mode definition deriving fromprog-mode, wired to.c4/.likec4viaauto-mode-alist.Test plan
#standalone-language-server,#neovim) all resolve.likec4-lsp --stdiocommand matches the binary documented in the page's "Standalone Language Server" section.