Copilot

Results 4892 comments of Copilot

> @copilot тот момент, когда начнёт выполняться второй ребилд, будет висеть computeLock, diagnostics не смогут запросить стары... Согласен — `computeLock` + `diagnosticsLock` защищают AST: пока идёт `rebuild()`, диагностики ждут, и...

@snissn I've opened a new pull request, #2242, to work on those changes. Once the pull request is ready, I'll request review from you.

> @copilot расскажу подробнее про дедупликацию. Одна строка может содержать несколько токенов языка запросов. Как ... Вы правы! В предыдущей реализации была проблема с наложением токенов. Проблема: Дедупликация через `HashSet`...

> @copilot В плане типов токенов ориентируйся вот сюда: https://github.com/1c-syntax/1c-syntax/blob/master/1c-query.YAML-tmLanguage > > Если есть вопрос... Обновил маппинг согласно 1c-query.YAML-tmLanguage (коммит 1f95165): **Изменения в типах токенов:** - Функции SDBL: `Function` (было...

> @copilot добавь детальные тесты, которые будут проверять раскраску конкретных позиций для запросов внутри строк... Добавил детальные тесты проверки позиций токенов SDBL (коммит db938ea): **5 новых тестов:** 1. `sdblQueryTokens_areHighlightedAtSpecificPositions` -...

> @copilot я не вижу, чтобы в твоём пулл-реквесте ты переставал добавлять обычные строки в итоговый дата сет. То ест�... Исправлено в коммите 5f58e74. **Проблема:** Полные STRING токены и разделённые...

> @copilot вообще напиши тесты формата "список ожидаемых токенов подряд и их позиции", а не просто поиск в коллекции... Добавлено в коммите 2e64cfb. **Два новых теста с проверкой последовательности:** 1....

> @copilot кажется, узел mdoName можно покрасить как namespace. Получится эль сделать просто? Насколько это сложно реализов... Хорошая идея! Это реализуемо, но требует работы с AST SDBL запроса. **Что нужно...

> @copilot да, давай. Не забывай про доступ через Trees. Что я хочу видеть: > > ```sdbl ... Это значительное расширение текущей реализации, которое потребует: 1. **Работы с AST через...

> @copilot да, делай прямо сейчас. Понял, начинаю реализацию AST-based анализа для SDBL запросов прямо сейчас. Это добавит поддержку для: - Variable → алиасы таблиц - Property → поля через...