Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Note Other AI code review bot(s) detectedCodeRabbit 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 |
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
|
@copilot Изучи https://github.com/microsoft/lsif-java. Обнови исследование |
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Изучил lsif-java и обновил исследование (719f827). Добавлено:
Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Попробуй реализовать |
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Реализовал базовую поддержку LSIF (5a728f8). Добавлено:
Использование: java -jar bsl-language-server.jar lsif --srcDir ./src/cf --output ./dump.lsif |
|
@copilot Продолжай реализацию |
…mentSymbol Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Продолжил реализацию (1348ee3). Добавлены поставщики:
DTO-классы:
|
|
@copilot Прододдай |
Продолжил реализацию (a311fb5). Добавлено:
Все records теперь имеют полное описание:
Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Продолжай. Нужно два формата вывода, NDJSON и json,как в изначальном исследовании. Только сделай это красиво |
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Добавлена поддержка двух форматов вывода (ea67f1f):
Архитектура:
Использование: lsif --srcDir ./src --output ./dump.lsif # NDJSON (по умолчанию)
lsif --srcDir ./src --output ./dump.json --format json # JSON |
...main/java/com/github/_1c_syntax/bsl/languageserver/lsif/supplier/DefinitionLsifSupplier.java
Outdated
Show resolved
Hide resolved
…ferenceIndex Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
...main/java/com/github/_1c_syntax/bsl/languageserver/lsif/supplier/DefinitionLsifSupplier.java
Outdated
Show resolved
Hide resolved
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Изучил lsif-java и обновил исследование — архитектура адаптирована под стиль BSL LS с использованием паттерна Supplier.
Реализовано в нескольких коммитах — CLI-команда
Продолжена реализация — добавлены все основные поставщики LSIF-данных.
Добавлены DocumentLinkLsifSupplier и тесты.
Добавлена поддержка Moniker для кросс-проектной навигации.
Все @value+@builder классы заменены на Java records.
Выполнено.
Добавлен JavaDoc для всех records и публичных методов.
Добавлены форматы NDJSON и JSON через интерфейс LsifWriter. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
...main/java/com/github/_1c_syntax/bsl/languageserver/lsif/supplier/DefinitionLsifSupplier.java
Outdated
Show resolved
Hide resolved
...main/java/com/github/_1c_syntax/bsl/languageserver/lsif/supplier/DefinitionLsifSupplier.java
Show resolved
Hide resolved
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Реализация LSIF
CLI-команда
Архитектура LsifEmitter
emitRange()теперь принимаетURI documentUriнапрямую вместо определения документа по DocumentContext. Это обеспечивает:Глобальный кэш в LsifEmitter
symbolToResultSetCache— переиспользование resultSet для ссылок на один символ из разных документовdocumentUriToIdCache— получение ID документа для кросс-файловых ссылокДокументация
docs/features/LSIFSupport.md— исследование и архитектураOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.