Skip to content

Release 3.3.0: navigation, offline checks and colours - #5

Merged
yankikucuk merged 1 commit into
masterfrom
feature/navigation-lint-colors
Sep 8, 2026
Merged

yankikucuk merged 1 commit into
masterfrom
feature/navigation-lint-colors

Conversation

@yankikucuk

Copy link
Copy Markdown
Owner

Summary

Navigation. Go to Definition, Find All References, Document Highlight and Rename Symbol for functions, methods, types, enums, enum members, variables, parameters and import aliases. Resolution is scope aware: a global length is left alone inside a function that takes its own length parameter, and named arguments, import paths and type fields are never mistaken for references. Renaming a built-in is refused rather than half applied.

Offline checks. Nine rules run as you type with no network, so every user gets feedback whether or not they opt into the TradingView compiler: missing-version, old-version, legacy-name, local-scope-call, unknown-argument, duplicate-argument, unused-variable, unused-parameter, unused-import. Most carry a quick fix. Pine Script: Convert to v6 applies the migration fixes across a file and reports exactly what it changed.

Colours. Swatches with a picker for hex literals, the built-in constants, color.new() and color.rgb(). The constant values are read from the shipped reference data rather than hardcoded.

Semantic highlighting. The identifiers a document declares are classified, so parameters, enum members, type fields and import aliases are coloured for what they are. Both themes gained matching semanticTokenColors and now enable semantic highlighting.

Test plan

  • npm test (322 tests, 65 of them new)
  • npm run build and vsce package
  • Linter run over the 98 snippet and fixture sources: zero false errors; the three findings were verified against the compiler as real problems
  • Formatter corpus re-verified: 98 sources, none broken, all idempotent
  • A v4 script converted by the command, then formatted, compiles clean on the TradingView compiler
  • Rules local-scope-call and the colour model were derived from compiler probes rather than assumed

@yankikucuk
yankikucuk merged commit 40c2324 into master Sep 8, 2026
1 check passed
@yankikucuk
yankikucuk deleted the feature/navigation-lint-colors branch September 8, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant