feat(diagnostics): параметры-списки принимают строку или массив (тип List)#4268
feat(diagnostics): параметры-списки принимают строку или массив (тип List)#4268nixel2007 wants to merge 5 commits into
Conversation
Параметр `userWordsToIgnore` диагностики Typo получил тип `Either<String, List<String>>`: значение принимается как строка через запятую (в том числе из SonarQube UI — там параметр остаётся скаляром) так и как JSON-массив слов в `.bsl-language-server.json`. В `DiagnosticHelper` добавлены переиспользуемые помощники для остальных списочных параметров: - `castParameterValue` — оборачивает значение из конфигурации в `Either` по типу целевого поля (строка → left, список → right); - `asStringList` — разворачивает `Either` в список строк. Схема параметров (`parameters-schema.json`) намеренно не правится руками — она генерируется задачей `precommit`. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W86iNs3Sj9wTS6NFFHvVbv
…ring>> `@DiagnosticParameter(type = Either.class)` объявляет параметр-список, который принимает строку через запятую (в т.ч. из SonarQube UI) либо JSON-массив в `.bsl-language-server.json`. `DiagnosticParameterInfo` теперь понимает `Either.class` (внешнее представление по умолчанию — строка), а `DiagnosticHelper` включает его в набор применяемых типов. Тип аннотации у `Typo.userWordsToIgnore` переведён на `Either.class` — это единая точка, из которой генератор JSON-схемы (bslls-dev-tools) и sonar-плагин выводят своё представление параметра. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W86iNs3Sj9wTS6NFFHvVbv
Параметр-список объявляется как `@DiagnosticParameter(type = List.class)` с полем `List<String>` (вместо `Either<String, List<String>>`): «строка через запятую ИЛИ массив» — это про входное представление, а не про тип параметра, поэтому Either избыточен. Значение из конфигурации приводится к `List<String>` в `DiagnosticHelper.castParameterValue` (строка → split по запятой, JSON-массив → список). `asStringList(String)` разбирает строку и переиспользуется для инициализации полей значением по умолчанию. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W86iNs3Sj9wTS6NFFHvVbv
|
Warning Review limit reached
Next review available in: 31 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughDiagnostic configuration now supports ChangesDiagnostic list parameter support
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/DiagnosticHelper.java`:
- Around line 157-165: Update castParameterValue to return null immediately when
value is null, before the List-type handling and any String.valueOf conversion.
Preserve the existing list conversion behavior for non-null values and return
the original value for non-list fields.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 78a5d3c1-0152-47f6-b6fd-3eddd48d0bce
📒 Files selected for processing (4)
src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/DiagnosticHelper.javasrc/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/TypoDiagnostic.javasrc/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/info/DiagnosticParameterInfo.javasrc/test/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/TypoDiagnosticTest.java
…ll"] Если в конфигурации параметр-список задан как `null`, прежний код доходил до `String.valueOf(value)` и получал строку "null" → список из одного элемента "null". Добавлен ранний возврат `null` до обработки типа List (поведение для null-значения — как раньше: присваивается полю как есть). Замечание из ревью CodeRabbit на PR #4268. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W86iNs3Sj9wTS6NFFHvVbv
@nullable Пакет помечен @NullMarked (JSpecify), поэтому возврат null из castParameterValue нарушал контракт non-null (SonarCloud java:S2637, Reliability). Возвращаемый тип помечен @nullable — метод легально возвращает null для null-значения параметра. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W86iNs3Sj9wTS6NFFHvVbv
|
2 participants
Описание
Параметры-списки диагностик (значения через запятую) получили первоклассный тип
Listи теперь принимают два входных представления:.bsl-language-server.json("userWordsToIgnore": ["Слово1", "Слово2"]).Реализация:
@DiagnosticParameter(type = List.class)+ полеList<String>вместо строки.DiagnosticHelper.castParameterValueприводит значение из конфигурации кList<String>: строка →splitпо запятой (с trim/отбросом пустых), JSON-массив (Jackson →ArrayList) → список.DiagnosticHelper.asStringList(String)— разбор CSV, переиспользуется для инициализации полей значением по умолчанию.DiagnosticParameterInfoиDiagnosticHelper.configureDiagnosticзнают проList.class; внешнее представление значения по умолчанию остаётся строкой (для SonarQube UI и JSON-схемы).Typo(userWordsToIgnore).Обратная совместимость полная: старые CSV-строки в конфигах и любые значения из SonarQube продолжают работать; массив — новая опциональная возможность JSON-конфига.
Схема параметров
parameters-schema.jsonне правился руками — он генерируетсяgradlew precommitчерез плагинbslls-dev-tools. Поддержка"type": ["string","array"]добавлена в 1c-syntax/bslls-dev-tools#PR (см. связанные PR ниже). После релиза плагина и бампа его версии здесь схема перегенерится автоматически. В рантайме массив уже принимается независимо от схемы.Связанные PR
List(веткаclaude/diagnostic-params-string-list-28j3v4)claude/diagnostic-params-string-list-28j3v4)Связанные задачи
Closes
Чеклист
Общие
TypoDiagnosticTest, в т.ч. новый кейс с массивом)gradlew precommit) — регенерация схемы зависит от релизаbslls-dev-tools, см. вышеДля диагностик
Дополнительно
Дальнейший план — раскатать тип
Listна остальные списочные параметры (LatinAndCyrillicSymbolInWord,MagicNumber/MagicDate,CommentedCodeи др.;UsingServiceTag— отдельно, там|-разделитель/regex).🤖 Generated with Claude Code
Generated by Claude Code
Summary by CodeRabbit
New Features
Bug Fixes