Skip to content

Lsp integration#28

Merged
kugesan1105 merged 60 commits intojaseci-labs:mainfrom
ThiruvarankanM:LSP-Integration
Jan 11, 2026
Merged

Lsp integration#28
kugesan1105 merged 60 commits intojaseci-labs:mainfrom
ThiruvarankanM:LSP-Integration

Conversation

@ThiruvarankanM
Copy link
Copy Markdown
Contributor

@ThiruvarankanM ThiruvarankanM commented Jan 5, 2026 •
edited
Loading

SECTION A: Language Server (LSP) Integration

A1. LSP Initialization and Startup

  • Workspace is resolved correctly from VS Code workspace folders
  • Extension is already activated before LSP tests run
  • Environment Manager is exposed via extension exports
  • LSP Manager is exposed via extension exports
  • LSP Manager is available when a valid environment exists
  • LSP client is created and accessible through LSP Manager
  • LSP output channel is created during LSP startup
  • LSP output channel name includes "Jac Language Server"

A2. LSP Features (Code Intelligence)

  • LSP detects syntax errors in invalid .jac files via diagnostics
  • Diagnostics are reported after file is opened and analyzed
  • At least one diagnostic error is reported for invalid syntax
  • LSP provides hover information for valid JAC node definitions
  • Hover provider returns non-empty content
  • Hover content includes the node name (Bus)
  • LSP remains running when developerMode setting is toggled
  • LSP output channel remains available after developer mode change
2

pre-commit-ci Bot and others added 12 commits January 5, 2026 17:38
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
7d0f70a 
for more information, see https://pre-commit.ci
@ThiruvarankanM
Refactor fileExists helper function to improve readability and remove…
f700d0d 
… duplication
@ThiruvarankanM
Increase timeout for LSP diagnostics test to accommodate longer analy…
3fb590f 
…sis time
@ThiruvarankanM
Remove specific version from jaclang installation in integration tests
bf8ff30
@ThiruvarankanM
Increase timeouts for LSP initialization and diagnostics tests to imp…
fa030a2 
…rove reliability
@ThiruvarankanM
Specify jaclang version in environment integration tests to ensure co…
fa6e16c 
…nsistent installation
@ThiruvarankanM
Increase timeouts for LSP feature tests to enhance stability and acco…
70b9b40 
…mmodate longer processing times
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
ab0d603 
for more information, see https://pre-commit.ci
@ThiruvarankanM
Refactor test case for opening .jac files to improve clarity and accu…
ba30fda 
…racy
@ThiruvarankanM
Rename test JAC file for LSP feature tests to reflect syntax validation
ad8f34a
@ThiruvarankanM
docs: Update environment tests file header and Phase 1 description
7c0afe9 
- Change Phase 1 title to 'Extension Auto-Activation and Initialization'
- Add clarifying subtitle about auto-activation behavior
- Move envManager to top-level scope for Phase 1 and Phase 2 sharing
@ThiruvarankanM
delete package-lock.json
e851d3e
@ThiruvarankanM
Add shared test utilities for integration tests
f3c8c29
@ThiruvarankanM
Add missing test utility file reference in tsconfig
65c64a3
@ThiruvarankanM
Replace inline command utility functions with imports from testUtils
818f747
@ThiruvarankanM
Replace inline file existence check with utility import and reduce w…
b273228 
…ait time in LSP tests
@ThiruvarankanM
Remove redundant environment validation call in promptEnvironmentSele…
422c498 
…ction
@ThiruvarankanM
Add package-lock.json to .gitignore
9a9ce80
@ThiruvarankanM
Remove LSP integration test suite file
e762205
@ThiruvarankanM
Add initial LSP integration test suite
d78d38f
@ThiruvarankanM
Add LSP integration test suite documentation
044c802
@ThiruvarankanM
Refactor LSP integration test suite imports for better organization
559f3b5
@ThiruvarankanM
Fix formatting and ensure newline at end of lsp.integration.test.ts
4e78231
@ThiruvarankanM
Add missing newline at end of lsp.integration.test.ts and update test…
d3fa096 
… group documentation
@ThiruvarankanM
restructure test group for LSP Initialization
f8ee80a
@ThiruvarankanM
Add tests for LSP manager availability and output channel creation
fdc9498
@ThiruvarankanM
Add documentation for LSP Features test group in integration tests
9076ca6
@ThiruvarankanM
Add test setup for LSP Features code intelligence tests
2098132
@ThiruvarankanM
Add cleanup step to close all editors between tests
371b0f7
@ThiruvarankanM
Add cleanup step to remove test files after LSP Features tests
ce7ab60
@ThiruvarankanM
Add test for LSP diagnostics to detect syntax errors in JAC files
dfad186
@ThiruvarankanM
Add test for hover information retrieval for node definitions in LSP
c271fd9
@ThiruvarankanM
Add test to ensure LSP remains stable when toggling developer mode se…
74e8d1d 
…ttings
@ThiruvarankanM
Refactor LSP integration tests for improved readability and consistency
cd05980
@ThiruvarankanM
Remove cleanup integration test for virtual environment teardown
c03b495
@ThiruvarankanM
Add cleanup integration test for virtual environment teardown
d5df367
@ThiruvarankanM
Copy link
Copy Markdown
Contributor Author

ThiruvarankanM commented Jan 7, 2026

2

kugesan1105
kugesan1105 approved these changes Jan 11, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@kugesan1105 kugesan1105 left a comment

Choose a reason for hiding this comment

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

LGTM

@kugesan1105 kugesan1105 merged commit 153557b into jaseci-labs:main Jan 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kugesan1105 kugesan1105 kugesan1105 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ThiruvarankanM @kugesan1105