feat: location in nomissingexternal models linting error

[benfdking]

Depends on

• feat(vscode): add test recognition to vscode #5019

• Adds the range to missing external models
• Fixes a bug where non registered external models where not highlighting full thing for example in raw.demographics was only highlighting demographics

[Copilot]

Pull Request Overview

This PR enhances the location information for the nomissingexternalmodels linting rule by adding range information to violations and fixes a bug where unregistered external models with schema prefixes were not being highlighted correctly.

• Creates a new sqlmesh/utils/lineage.py module with extracted reference-finding functionality
• Updates the nomissingexternalmodels linter rule to provide precise location ranges for violations
• Fixes highlighting to include full table names (e.g., raw.demographics instead of just demographics)

Reviewed Changes

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

Show a summary per file

File	Description
sqlmesh/utils/lineage.py	New module containing extracted reference-finding logic and LSP reference classes
sqlmesh/core/linter/rules/builtin.py	Enhanced NoMissingExternalModels rule to provide location ranges for violations
sqlmesh/lsp/reference.py	Refactored to use extracted functionality from new lineage module
sqlmesh/core/linter/helpers.py	Added read_range_from_string helper function for reading text ranges
tests/lsp/test_reference_external_model.py	Added test for unregistered external models with schema prefixes
tests/core/linter/test_builtin.py	Updated test assertions for new violation format
sqlmesh/lsp/main.py	Updated import for LSPExternalModelReference
sqlmesh/lsp/completions.py	Updated import for generate_markdown_description
tests/utils/test_lineage_description.py	Updated import path
vscode/extension/src/lsp/lsp.ts	Fixed TypeScript type casting
sqlmesh/lsp/description.py	Removed file (functionality moved to lineage module)

Comments suppressed due to low confidence (2)

sqlmesh/utils/lineage.py:88

• The function read_range_from_string is duplicated between this file and sqlmesh/core/linter/helpers.py. Consider importing it from the helpers module instead of duplicating the implementation.

        root_scope = None

tests/lsp/test_reference_external_model.py:122

• The test verifies that the range covers the full qualified name 'raw.demographics', but it doesn't test the specific bug mentioned in the PR description where only 'demographics' was highlighted. Consider adding an assertion that verifies the range start position to ensure the catalog/schema prefix is included.

    assert read_range_from_file(path, reference.range) == "raw.demographics"