1c-syntax / bsl-language-server

Реализация Language Server Protocol для языка 1C (BSL)

BSL Language Server

Реализация протокола language server protocol для языка 1C (BSL) - языка 1С:Предприятие 8 и OneScript.

Сайт проекта - https://1c-syntax.github.io/bsl-language-server

Сайт проекта (develop) - https://1c-syntax.github.io/bsl-language-server/dev

Исходники документации - GitHub

---

Language Server Protocol implementation for 1C (BSL) - 1C:Enterprise 8 and OneScript languages.

Project site - https://1c-syntax.github.io/bsl-language-server/en

Project site (develop) - https://1c-syntax.github.io/bsl-language-server/dev/en

Documentation sources - GitHub

bsl-language-server's Issues

(8.3.10+) Использование устаревшего метода клиентского приложения

Диагностика: Использован устаревший метод "%1" (актуальное - "%2").

Устаревший метод	Актуальный метод
УстановитьКраткийЗаголовокПриложения	КлиентскоеПриложение.УстановитьКраткийЗаголовок
ПолучитьКраткийЗаголовокПриложения	КлиентскоеПриложение.ПолучитьКраткийЗаголовок
УстановитьЗаголовокКлиентскогоПриложения	КлиентскоеПриложение.УстановитьЗаголовок
ПолучитьЗаголовокКлиентскогоПриложения	КлиентскоеПриложение.ПолучитьЗаголовок
ТекущийВариантОсновногоШрифтаКлиентскогоПриложения	КлиентскоеПриложение.ТекущийВариантОсновногоШрифта
ТекущийВариантИнтерфейсаКлиентскогоПриложения	КлиентскоеПриложение.ТекущийВариантИнтерфейса

Количество параметров метода

Настройки диагностик

Какие диагностики применять/исключать при анализе.

• Конкретный список
• По важности
• По области действия

Индивидуальные настройки, у каждой диагностики свои параметры

• Максимальная длинна строки
• Разрешить однотипные присваивания в одной строке
• и т.д.

[8.3.3+] НачалоВыбораИзСписка. Использование устаревшего события элемента управления

Совместимость: 8.3.3+
Диагностика: Событие НачалоВыбораИзСписка более не используется. Требуется расформирование обработчика этого события.

PS а я могу как нить сам теги добавлять?

Диагностика на длину метода

Значение по умолчанию - 200

Кейсы и прочие свичи должны содержать Иначе

Если x % 15 = 0 Тогда
	Результат = "FizzBuzz";
ИначеЕсли x % 3 = 0 Тогда
	Результат = "Fizz";
ИначеЕсли x % 5 = 0 Тогда
	Результат = "Buzz";
КонецЕсли;

Пустой блок

Неиспользуемый экспортный метод

Метод экспортный, но не вызывается из другим модулей. Лишнее слово "Экспорт".

Дополнительная сложность с обработчиками оповещений.

FP "Пустой блок кода" на пустых методах

Не реагировать на пустые методы.
На пустые методы отдельная диагностика?

Выражение должно заканчиваться ТочкойСЗапятой

Поддержка толстого клиента, управляемое приложение, клиент-сервер

Поддержка стандарта Поддержка толстого клиента, управляемое приложение, клиент-сервер

В модулях объектов, наборов записей и т.п.

1. в модулях [из списка по стандарту] есть области препроцессора
2. область #Иначе для коревой области препроцессора существует (нужно убедиться что проверка не дает фолс позитив на #Иначе в не корневой области)
3. в области #Иначе должен быть райз и только он, ничего больше ничего меньше

Ошибка

#Если Сервер Тогда
// ЧтоТо
#Иначе
ВызватьИсключение "Текст";
#КонецЕсли

Правильный модуль

#Если Сервер Или ТолстыйКлиентОбычноеПриложение Или ВнешнееСоединение Тогда
// ЧтоТо
#Иначе
ВызватьИсключение "Текст";
#КонецЕсли

Ошибка

Процедура МояПроцедура()
#Если Сервер Или ТолстыйКлиентОбычноеПриложение Или ВнешнееСоединение Тогда
// ЧтоТо
#Иначе
ВызватьИсключение "Текст";
#КонецЕсли
КонецПроцедуры

Ошибка

#Если Сервер Или ТолстыйКлиентОбычноеПриложение Или ВнешнееСоединение Тогда
// ЧтоТо
#Иначе
#КонецЕсли

Ошибка

#Если Сервер Или ТолстыйКлиентОбычноеПриложение Или ВнешнееСоединение Тогда
// ЧтоТо
#КонецЕсли

Модуль сеанса и все модули менеджеров заключены в конструкцию

#Если Сервер Или ТолстыйКлиентОбычноеПриложение Или ВнешнееСоединение Тогда
// ЧтоТо
#КонецЕсли

Приведение к числу в попытке

Например, неправильно:

Попытка
 КоличествоДнейРазрешения = Число(Значение);
Исключение
 КоличествоДнейРазрешения = 0; // значение по умолчанию
КонецПопытки;

Правильно:

ОписаниеТипа = Новый ОписаниеТипов("Число");
КоличествоДнейРазрешения = ОписаниеТипа.ПривестиЗначение(Значение);

FP "Один оператор в одной строке" на пустой оператор

Из проверки исключить срабатывание на пустой оператор.
т.к, пустой оператор в отдельной диагностике.

Позиционирование на многострочном блоке

LineLength на методах

Должно ли срабатывать на методах?

Процедура Подключаемый_СвойстваВыполнитьКоманду(ЭлементИлиКоманда, НавигационнаяСсылка = Неопределено, СтандартнаяОбработка = Неопределено)

Имхо да, но даже на SSL зашкаливает количество срабатываний.

может под настройку?

(8.3.6+) Найти -> СтрНайти

8.3.6+
Диагностика: Использование устаревшего метода глобального контекста

Использован устаревший глобальный метод "Найти" (актуальный - "СтрНайти").

FP Один оператор в одной строке для "однотипных" операторов присваивания

Одно выражение в одной строке

Неиспользуемый параметр

С ошибкой:

Функция СложитьДваЧисла(Знач ПервоеЧисло, Знач ВтороеЧисло, Знач ЛишнийПараметр)

    Возврат ПервоеЧисло+ВтороеЧисло;

КонецФункции

С ошибкой 2

Функция СложитьДваЧисла(Знач ПервоеЧисло, Знач ВтороеЧисло, Знач ЛишнийПараметр = Неопределено)

Возврат ПервоеЧисло+ВтороеЧисло;

КонецФункции

Без ошибки

Функция СложитьДваЧисла(Знач ПервоеЧисло, Знач ВтороеЧисло)

Возврат ПервоеЧисло+ВтороеЧисло;

КонецФункции

(8.3.12+) ОбластьПостроенияДиаграммы. Использование устаревшего свойства объекта

Совместимость 8.3.12+
Диагностика: Использование устаревшего свойства объекта ОбластьПостроенияДиаграммы: "%1" (актуальное - "%2").

Устаревший метод	Актуальный метод
ОтображатьШкалу	ОтображатьШкалы
ЛинииШкалы	ЛинииШкал
ЦветШкалы	ЦветШкал
ОтображатьПодписиШкалыСерии	ШкалаСерий.ПоложениеПодписейШкалы = ПоложениеПодписейШкалыДиаграммы.Снаружи
ОтображатьПодписиШкалыТочек	ШкалаТочек.ПоложениеПодписейШкалы = ПоложениеПодписейШкалыДиаграммы.Снаружи
ОтображатьПодписиШкалыЗначений	ШкалаТочек.ПоложениеПодписейШкалы = ПоложениеПодписейШкалыДиаграммы.Снаружи
ОтображатьЛинииЗначенийШкалы	ШкалаЗначений.ОтображениеЛинийСетки = ОтображениеЛинийСеткиДиаграммы.Отображать
ФорматШкалыЗначений	ШкалаЗначений.ФорматПодписей
ОриентацияМеток	ОриентацияПодписейДиаграммы.Горизонтально

FP Ограничение на длину строки в запросе

Репортер в сонар generic-issue

https://docs.sonarqube.org/latest/analysis/generic-issue/

Использование неопознанных символов в условии инструкции препроцессору

Если _ Тогда, Если Нечто Тогда должно баниться диагностикой.

AST-узел для анализа - preproc_unknownSymbol

Поддержка secondaryLocations для диагностик

Протащить в addDiagnostic. Возможно отрефакторить диагностику вложенного тернарного оператора.

Условие которое никогда не выполнится

Если что-то = Значение И чтото = другоеЗначение

Шильдик на телеграмм

https://t.me/bsl_language_server

Присвоение самой себе

A = A;
Структура.А = Структура.А;
Структура.Вставить("A", Структура.А) // Под вопросом;

Использование объектов не доступных в Linux-клиенте

Ругаться на: Новый COMОбъект
Диагностика: Проверить, что задействованы аналоги COM объектов при работе в Linux-клиенте.

Ругаться на: Новый Почта
Диагностика: Проверить, что пользователю доступны аналоги объекта Почта при работе в Linux-клиенте.

(8.3.10+) ВариантОткрытияОкна. Использование устаревшего системного перечисления

8.3.10+
Диагностика: Использование устаревшего системного перечисления ВариантОткрытияОкна
Системное перечисление ВариантОткрытияОкна более не доступно.
Требуется переработать код с отказом от использования перечисления.

Шильдик на скачавания

Область действия диагностики

Для начала os, bsl

Последовательность параметров метода

Необязательные после обязательных

Отчет в формате JUnit

Добавить отчёт в формате JUnit. Удобно для работы с Gitlab

Вложенный тернарный оператор

ЭтаФорма -> ЭтотОбъект

Диагностика: Использовано устаревшее имя свойства "ЭтаФорма" (актуальное - "ЭтотОбъект").
8.3.3+

Неиспользуемая локальная переменная

Неиспользуемый неэкспортный метод

На обработчики событий и на Подключаемый_ не должно ругаться

ограничение на букву ё

Ключ диагностики по умолчанию не должен содержать оконечное Diagnostic

Доработать функцию BSLDiagnostic::getCode так, чтобы конечное Diagnostic отрезалось.

Пустой оператор

(8.3.3+) КнопкаСпискаВыбора. Использование устаревшего свойства кнопки выбора из списка

Совместимость: 8.3.3+
Диагностика: Использовано устаревшее свойство "КнопкаСпискаВыбора". Возможно, требуется перейти на свойство "КнопкаВыпадающегоСписка".

VSC на ховере показывает Unsupported Markup Content

Проверить корректность реализации протокола:

CI: Настроить на трависе указание java.libraries для сонара

Есть проблемы с подхватом классов с lombok аннотациями. Документация рекомендует настраивать sonar.java.livraries и указывать там требуемые жарники. Как вариант - указание каталога .m2 текущего пользователя.

(8.3.12+) ОриентацияМетокДиаграммы. Использование устаревшего системного перечисления

Совместимость 8.3.12+
Диагностика: Использование устаревшего системного перечисления
Системное перечисление ОриентацияМетокДиаграммы более не доступно.
Актуальный вариант ОриентацияПодписейДиаграммы.

При ошибке разбора внутри функции Ложно срабатывает FunctionShouldHaveReturn

Проверка на модальность

Диагностика: Использование модальных окон

Модальный метод	Немодальный аналог
Вопрос	ПоказатьВопрос
Предупреждение	ПоказатьПредупреждение
ОткрытьЗначение	ПоказатьЗначение
ВвестиДату	ПоказатьВводДаты
ВвестиЗначение	ПоказатьВводЗначения
ВвестиСтроку	ПоказатьВводСтроки
ВвестиЧисло	ПоказатьВводЧисла
УстановитьВнешнююКомпоненту	НачатьУстановкуВнешнейКомпоненты
УстановитьРасширениеРаботыСФайлами	НачатьУстановкуРасширенияРаботыСФайлами
УстановитьРасширениеРаботыСКриптографией	НачатьУстановкуРасширенияРаботыСКриптографией
ПоместитьФайл	НачатьПомещениеФайла
ОткрытьФормуМодально	ОткрытьФорму
СписокЗначений.ОтметитьЭлементы	СписокЗначений.ПоказатьОтметкуЭлементов
СписокЗначений.ВыбратьЭлемент	СписокЗначений.ПоказатьВыборЭлемента
УправляемаяФорма.ВыбратьИзМеню	УправляемаяФорма.ПоказатьВыборИзМеню
УправляемаяФорма.ВыбратьИзСписка	УправляемаяФорма.ПоказатьВыборИзСписка
УправляемаяФорма.ОткрытьМодально	УправляемаяФорма.Открыть
ДиалогРасписанияРегламентногоЗадания.ОткрытьМодально	ДиалогРасписанияРегламентногоЗадания.Показать
КонструкторФорматнойСтроки.ОткрытьМодально	КонструкторФорматнойСтроки.Показать
ДиалогРедактированияСтандартногоПериода.Редактировать	ДиалогРедактированияСтандартногоПериода.Показать
ДиалогВыбораШрифта.Выбрать	ДиалогВыбораШрифта.Показать
ДиалогВыбораЦвета.Выбрать	ДиалогВыбораЦвета.Показать
ОбработкаРасшифровкиКомпоновкиДанных.ВыбратьДействие	ОбработкаРасшифровкиКомпоновкиДанных.ПоказатьВыборДействия

(8.3.12+) Диаграмма. Использование устаревшего свойства объекта

Совместимость 8.3.12+
Диагностика: Использование устаревшего свойства объекта Диаграмма: "%1" (актуальное - "%2").

Устаревший метод	Актуальный метод
ПалитраЦветов	ОписаниеПалитрыЦветов.ПалитраЦветов
ЦветНачалаГрадиентнойПалитры	ОписаниеПалитрыЦветов.ЦветНачалаГрадиентнойПалитры
ЦветКонцаГрадиентнойПалитры	ОписаниеПалитрыЦветов.ЦветКонцаГрадиентнойПалитры
МаксимальноеКоличествоЦветовГрадиентнойПалитры	ОписаниеПалитрыЦветов.МаксимальноеКоличествоЦветовГрадиентнойПалитры
ПолучитьПалитру	ОписаниеПалитрыЦветов.ПолучитьПалитру
УстановитьПалитру	ОписаниеПалитрыЦветов.УстановитьПалитру
ОтображатьЛегенду	Задайте отображаемую область с помощью одного из свойств ОбластьЛегендыДиаграммы, ОбластьЛегендыДиаграммыГанта или ОбластьЛегендыСводнойДиаграммы
ОтображатьЗаголовок	Задайте отображаемую область с помощью одного из свойств ОбластьЗаголовкаДиаграммы, ОбластьЗаголовкаДиаграммыГанта или ОбластьЗаголовкаСводнойДиаграммы

Fp LineLength на Нстр()

ТекстСообщения = СтроковыеФункцииКлиентСервер.ПодставитьПараметрыВСтроку(
НСтр("ru = 'Процедуре ЗаполнитьРеквизитСпособОтображенияПодсказки не удалось обработать некоторые вопросы шаблона анкеты (пропущены): %1'"),
ПроблемныхОбъектов);

Каноническое написание ключевых слов

https://its.1c.ru/db/v8std/content/2149184090/hdoc

Объект входит в несколько функциональных подсистем