Skip to content

feat: add right-click context menu to lintfix format .jac files#57

Merged
kugesan1105 merged 8 commits intojaseci-labs:mainfrom
SandeepaHWP:context-menu-format
Mar 19, 2026
Merged

feat: add right-click context menu to lintfix format .jac files#57
kugesan1105 merged 8 commits intojaseci-labs:mainfrom
SandeepaHWP:context-menu-format

Conversation

@SandeepaHWP
Copy link
Copy Markdown
Contributor

@SandeepaHWP SandeepaHWP commented Mar 12, 2026

Summary

  • Adds jac.lintfixFormat command to the editor right-click context menu for .jac files
  • Command sends workspace/executeCommand to the Jac LSP server which applies auto-lint fixes and formats the document in place
  • Only visible when the active file is a .jac file (resourceLangId == jac)

Changes

  • package.json: Register jac.lintfixFormat command and add editor/context menu entry in group 1_modification
  • src/constants.ts: Add LINTFIX_FORMAT to the COMMANDS map
  • src/commands/index.ts: Register the command handler — validates active .jac editor, checks LSP is running, sends workspace/executeCommand with the file URI
Linfix.demo.mp4

And if the file has syntax error this popup will come

image

Adds a "Lintfix Format File" command to the editor context menu for .jac
files. The command sends workspace/executeCommand to the Jac LSP server
which applies auto-lint fixes and formats the document in place.
@SandeepaHWP SandeepaHWP force-pushed the context-menu-format branch from def837e to 1d4a87a Compare March 16, 2026 10:11
@RandiJay96 RandiJay96 moved this to In Progress PR in Core Stack Hacking Mar 17, 2026
@RandiJay96 RandiJay96 moved this from In Progress PR to Review in Core Stack Hacking Mar 17, 2026
@kugesan1105 kugesan1105 merged commit 7a27b6c into jaseci-labs:main Mar 19, 2026
2 checks passed
@RandiJay96 RandiJay96 moved this from Review to Completed in Core Stack Hacking Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants