feat(lsp): references function now detects non registered external models

[benfdking]

No description provided.

[Copilot]

Pull Request Overview

This PR enhances the LSP references function to detect and handle unregistered external models that are not defined in the configuration files. Previously, the system would ignore references to external models that weren't registered, but now it creates specific reference objects for these unregistered models.

• Added support for detecting unregistered external models as LSP references
• Made the uri field optional in LSPExternalModelReference to accommodate unregistered models
• Updated LSP handlers to handle references with null URIs appropriately

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
sqlmesh/lsp/reference.py	Made uri field optional in LSPExternalModelReference and added logic to create references for unregistered external models
sqlmesh/lsp/main.py	Updated goto_definition and find_references handlers to check for null URIs before processing
tests/lsp/test_reference_external_model.py	Added test for unregistered external model detection and updated existing test to handle optional URI
tests/lsp/test_reference.py	Updated test assertions to handle optional URI field

Comments suppressed due to low confidence (1)

tests/lsp/test_reference_external_model.py:50

• The function name 'test' is too generic and doesn't describe what it tests. Consider renaming it to 'test_reference_unregistered_external_model' to be more descriptive.

def test(tmp_path: Path):