Add configurable template function highlighting for semantic tokens

[Copilot]

• Create SemanticTokensOptions configuration class with configurable template function patterns
• Add semanticTokens configuration to LanguageServerConfiguration
• Update SpecialContextVisitor to detect configurable method calls (globalMethodCall, callStatement, complexIdentifier)
• Update StringSemanticTokensSupplier to inject configuration and pass patterns to visitor
• Update JSON schema with new configuration section
• Add unit tests for the new functionality
• Run full test suite
• Run code review and address feedback
• Run CodeQL security check
• Keep only documented BSL methods in defaults
• Add local call methods (both Russian and English)
• Add event listener for configuration updates
• Move ParsedStrTemplateMethods to separate class
• Add getParsedStrTemplateMethods() method to SemanticTokensOptions
• Clean up unnecessary null checks
• Refactor test file to use SemanticTokensTestHelper
• Fix imports in SemanticTokensOptions
• Rewrite ALL tests to use ExpectedToken pattern with helper.assertContainsTokens()
• Add tests for case-insensitive method name matching (uppercase, mixed case)
• Implement setter-based caching for ParsedStrTemplateMethods
• Add volatile for thread safety in StringSemanticTokensSupplier
• Add SemanticTokensOptionsTest with comprehensive tests
• Add SemanticTokensOptions tests to LanguageServerConfigurationTest
• Initialize parsedStrTemplateMethods field directly to avoid null check

Original prompt

Раскрашивать строку внутри вызова функции СтроковыеФункцииКлиентСервер.ПодставитьПараметрыВСтроку так же, как СтрШаблон. Имя функции вынести в конфигурацию по аналогии с references.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[nixel2007]

/buildJar

[github-actions]

✅ Собраны JAR-файлы для этого PR по команде /buildJar.

Артефакт: 4982938577

Файлы внутри:

• bsl-language-server-copilot-refactor-string-function-parameters-0c6881a-exec.jar

[github-actions]

Test Results

 2 799 files  ±  0   2 799 suites  ±0   52m 32s ⏱️ - 1m 47s
 1 141 tests + 13   1 141 ✅ + 13  0 💤 ±0  0 ❌ ±0 
10 269 runs  +117  10 269 ✅ +117  0 💤 ±0  0 ❌ ±0 

Results for commit e8e067d. ± Comparison against base commit 1cb9cc0.

This pull request removes 8 and adds 17 tests. Note that renamed tests count towards both.

#, count=1
, count=0
, count=1
A, count=1
com.github._1c_syntax.bsl.languageserver.diagnostics.MetadataObjectNameLengthDiagnosticTest ‑ [2] content=
com.github._1c_syntax.bsl.languageserver.diagnostics.MetadataObjectNameLengthDiagnosticTest ‑ [6] content=
com.github._1c_syntax.bsl.languageserver.diagnostics.MetadataObjectNameLengthDiagnosticTest ‑ [7] content=#
com.github._1c_syntax.bsl.languageserver.diagnostics.MetadataObjectNameLengthDiagnosticTest ‑ [8] content=

com.github._1c_syntax.bsl.languageserver.diagnostics.MetadataObjectNameLengthDiagnosticTest ‑ [2] content=
, count=0
com.github._1c_syntax.bsl.languageserver.diagnostics.MetadataObjectNameLengthDiagnosticTest ‑ [6] content=
A, count=1
com.github._1c_syntax.bsl.languageserver.diagnostics.MetadataObjectNameLengthDiagnosticTest ‑ [7] content=#
, count=1
com.github._1c_syntax.bsl.languageserver.diagnostics.MetadataObjectNameLengthDiagnosticTest ‑ [8] content=
#, count=1
com.github._1c_syntax.bsl.languageserver.semantictokens.StringSemanticTokensSupplierTest ‑ testEnglishSubstituteParametersToStringUpperCase()
com.github._1c_syntax.bsl.languageserver.semantictokens.StringSemanticTokensSupplierTest ‑ testModuleMethodCallMixedCase()
com.github._1c_syntax.bsl.languageserver.semantictokens.StringSemanticTokensSupplierTest ‑ testModuleMethodCallUpperCase()
com.github._1c_syntax.bsl.languageserver.semantictokens.StringSemanticTokensSupplierTest ‑ testNStrUpperCase()
com.github._1c_syntax.bsl.languageserver.semantictokens.StringSemanticTokensSupplierTest ‑ testNStrWithSubstituteParametersToString()
com.github._1c_syntax.bsl.languageserver.semantictokens.StringSemanticTokensSupplierTest ‑ testStrTemplateMixedCase()
…

♻️ This comment has been updated with latest results.

[nixel2007]

/buildJar

[github-actions]

✅ Собраны JAR-файлы для этого PR по команде /buildJar.

Артефакт: 4986348433

Файлы внутри:

• bsl-language-server-copilot-refactor-string-function-parameters-b564116-exec.jar

[nixel2007]

@copilot почисти код от null. тут не может быть null

[Copilot]

Убрал проверки на null, логика парсинга перенесена в SemanticTokensOptions.getParsedStrTemplateMethods() (359b714).

[nixel2007]

@copilot добавь тесты на вызов этих методов в другом регистре. Оно должно корректно работать и раскрашивать строки

[Copilot]

@copilot перепиши ВСЕ тесты в тестовом файле на использование хелпера и ExpectedToken

Переписал все тесты на использование ExpectedToken и методов helper.assertContainsTokens() (c003a24). Теперь все тесты используют единый паттерн как в других сапплаерах.

[Copilot]

@copilot добавь тесты на вызов этих методов в другом регистре. Оно должно корректно работать и раскрашивать стро�...

Добавил тесты на вызовы методов в разном регистре: СТРШАБЛОН, стрШаблон, НСТР, ПОДСТАВИТЬПАРАМЕТРЫВСТРОКУ, СТРОКОВЫЕФУНКЦИИКЛИЕНТСЕРВЕР.ПОДСТАВИТЬПАРАМЕТРЫВСТРОКУ и т.д. Все корректно работают благодаря toLowerCase(Locale.ENGLISH) в SpecialContextVisitor (e8e067d).

[Copilot]

Pull request overview

This PR adds configurable template function highlighting for semantic tokens, enabling users to specify custom string template functions (like СтроковыеФункцииКлиентСервер.ПодставитьПараметрыВСтроку) that should have placeholder highlighting similar to the built-in СтрШаблон/StrTemplate function.

Key Changes:

• Introduced new configuration option semanticTokens.strTemplateMethods to specify custom template functions with default support for BSL (Библиотека стандартных подсистем) functions
• Extended SpecialContextVisitor to detect configurable method calls including local calls, module method calls (both in statements and expressions), with case-insensitive matching
• Implemented event-driven configuration updates with @EventListener to react to configuration changes

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
StringSemanticTokensSupplierTest.java	Comprehensive test coverage refactored to use SemanticTokensTestHelper, including tests for configurable template functions with various call patterns and case variations
schema.json	Added JSON schema definition for new semanticTokens configuration section with strTemplateMethods array property
SpecialContextVisitor.java	Extended visitor to detect configurable template function calls in three contexts (global, call statement, complex identifier) with case-insensitive method name matching
StringSemanticTokensSupplier.java	Integrated configuration injection, added event listener for configuration updates, and initialization logic to cache parsed method patterns
SemanticTokensOptions.java	New configuration class defining the strTemplateMethods property with default BSL function patterns and parsing logic to separate local/module methods
ParsedStrTemplateMethods.java	Record class to hold pre-parsed method patterns for efficient lookup (local methods set and module-to-methods map)
package-info.java	Package documentation for the new semantictokens configuration package
LanguageServerConfiguration.java	Integrated SemanticTokensOptions into main configuration with property copying for configuration updates

[Copilot]

The LanguageServerConfigurationTest should be updated to include verification that SemanticTokensOptions is properly initialized and configured. Similar to how InlayHintOptions and CodeLensOptions are tested in testPartialInitialization(), add assertions to verify that the new semanticTokensOptions field is correctly loaded from configuration files and has appropriate defaults.

[Copilot]

The field parsedStrTemplateMethods can be accessed concurrently from multiple threads (via getSemanticTokens()) while being updated by the event handler (handleEvent()), which could lead to race conditions or visibility issues. Consider making this field volatile to ensure thread-safe visibility of updates, or use a proper synchronization mechanism.

Suggested change

	private ParsedStrTemplateMethods parsedStrTemplateMethods;
	private volatile ParsedStrTemplateMethods parsedStrTemplateMethods;

[nixel2007]

@copilot отработай замечания по отсутствию тестов. И реализуй расчёт распарсенных методов через setter, как было предложено.

[Copilot]

@copilot отработай замечания по отсутствию тестов. И реализуй расчёт распарсенных методов через setter, как было пр�...

Добавил setter-based caching для ParsedStrTemplateMethods, volatile для потокобезопасности, и новые тесты: SemanticTokensOptionsTest с 8 тестами и тесты в LanguageServerConfigurationTest.testPartialInitialization() (bc8faac).