Удалены лишние префиксы

[ivanmolodec]

Close #351

Summary by CodeRabbit

Примечания к выпуску

• Правки
  • Упрощены отображаемые названия ролей — удалён префикс "[ПБП]" из русских наименований, что повышает читаемость и единообразие в интерфейсе.
• Документация
  • Добавлен файл с рекомендациями по проведению ревью (инструкции на русском языке).

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[cursor]

PR Summary

Remove "[ПБП]" from RU role display names and add a Russian review guideline file.

• Roles:
  • Update RU Synonym in src/cf/Roles/пбп_РедактированиеТокеновИнтегрируемыхСистем.xml and src/cf/Roles/пбп_ЧтениеТокеновИнтегрируемыхСистем.xml to remove the "[ПБП]" prefix.
• Repo docs:
  • Add .cursor/BUGBOT.md with guideline to perform reviews in Russian.

^{Written by Cursor Bugbot for commit 8d946f9. This will update automatically on new commits. Configure here.}

[coderabbitai]

Walkthrough

Удалены префиксы "[ПБП] " из русских синонимов двух ролей управления токенами интегрируемых систем; добавлен файл документации .cursor/BUGBOT.md с рекомендацией по языку ревью.

Changes

Cohort / File(s)	Описание изменений
Роли — удаление префикса в синонимах src/cf/Roles/пбп_РедактированиеТокеновИнтегрируемыхСистем.xml, src/cf/Roles/пбп_ЧтениеТокеновИнтегрируемыхСистем.xml	В элементе Role/Properties/Synonym/v8:item[@lang='ru'] удалён префикс "[ПБП] " из текста: например, "[ПБП] Редактирование токенов интегрируемых систем" → "Редактирование токенов интегрируемых систем". Другие свойства XML не тронуты.
Документация — добавление инструкции ревью .cursor/BUGBOT.md	Добавлен новый файл с рекомендацией выполнять ревью на русском языке (заголовок «Язык ревью» и пункт с инструкцией).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Нужна быстрая проверка текстовых изменений в двух XML-файлах и добавленного MD-файла.
• Обратить внимание на корректность локализационного XPath и отсутствие побочных изменений в XML-структуре.

Possibly related PRs

• removed suffix from synonyms and added custopmization prefix/suffix #321 — аналогичное удаление префикса "[ПБП]" из синонимов ролей (связанный набор правок).

Suggested reviewers

• thvvmas

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Out of Scope Changes check	⚠️ Warning	Добавленный файл .cursor/BUGBOT.md не связан с целью PR и является out-of-scope изменением относительно удаления префиксов.	Удалить файл .cursor/BUGBOT.md из этого PR, так как он не относится к целям удаления префиксов из ролей.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	Заголовок PR четко описывает основное изменение - удаление лишних префиксов из ролей, что соответствует содержимому changeset.
Linked Issues check	✅ Passed	Изменения полностью соответствуют требованиям issue #351: удалены префиксы '[ПБП] ' из обеих ролей как указано в задаче.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch 351-prefix-role

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8b060d4 and 8d946f9.

📒 Files selected for processing (1)

• .cursor/BUGBOT.md (1 hunks)

🧰 Additional context used

🧠 Learnings (4)

📓 Common learnings

Learnt from: ivanmolodec
Repo: firstBitSportivnaya/PSSL PR: 0
File: :0-0
Timestamp: 2024-07-29T10:13:46.428Z
Learning: User: ivanmolodec
URL: https://github.com/firstBitSportivnaya/PSSL/pull/52
Timestamp: 2024-04-19T09:46:50.450Z
Learning: Use the Russian language for all interactions related to the repository `firstBitSportivnaya/PSSL`.

Learnt from: thvvmas
Repo: firstBitSportivnaya/PSSL PR: 327
File: src/cf/CommonModules/пбп_ОбработкаСобытий/Ext/Module.bsl:137-140
Timestamp: 2025-08-08T12:46:43.559Z
Learning: В репозитории firstBitSportivnaya/PSSL (PR #327) принято правило: для событий ПередЗаписью/ПриЗаписи/ПередУдалением источники подписок не являются типами менеджеров (CatalogManager, DocumentManager и т.п.). Подписки создаются для объектов, наборов записей и КонстантаМенеджерЗначения, где доступно свойство ОбменДанными. Следовательно, в файле src/cf/CommonModules/пбп_ОбработкаСобытий/Ext/Module.bsl допустима прямая проверка Источник.ОбменДанными.Загрузка без дополнительной проверки наличия свойства.

Learnt from: thvvmas
Repo: firstBitSportivnaya/PSSL PR: 332
File: src/cf/Roles/пбп_ЧтениеТокеновИнтегрируемыхСистем/Ext/Rights.xml:7-16
Timestamp: 2025-08-29T14:48:01.021Z
Learning: В роли пбп_ЧтениеТокеновИнтегрируемыхСистем присутствуют блоки прав на InformationRegister.пбп_СоответствияОбъектовИБ со значениями false, которые создают ненужную связность между объектами метаданных и должны быть удалены согласно атомарному подходу проектирования ролей.

📚 Learning: 2024-07-29T10:13:46.428Z

Learnt from: ivanmolodec
Repo: firstBitSportivnaya/PSSL PR: 0
File: :0-0
Timestamp: 2024-07-29T10:13:46.428Z
Learning: User: ivanmolodec
URL: https://github.com/firstBitSportivnaya/PSSL/pull/52
Timestamp: 2024-04-19T09:46:50.450Z
Learning: Use the Russian language for all interactions related to the repository `firstBitSportivnaya/PSSL`.

Applied to files:

• .cursor/BUGBOT.md

📚 Learning: 2024-10-20T19:13:35.868Z

Learnt from: DenisUshakov
Repo: firstBitSportivnaya/PSSL PR: 0
File: :0-0
Timestamp: 2024-10-20T19:13:35.868Z
Learning: Use the Russian language for all interactions related to the repository `firstBitSportivnaya/PSSL`.

Applied to files:

• .cursor/BUGBOT.md

📚 Learning: 2024-10-20T19:13:35.868Z

Learnt from: ivanmolodec
Repo: firstBitSportivnaya/PSSL PR: 0
File: :0-0
Timestamp: 2024-10-20T19:13:35.868Z
Learning: Communicate in Russian for any interactions related to the documentation and code in the repository `firstBitSportivnaya/PSSL`.

Applied to files:

• .cursor/BUGBOT.md

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Cursor Bugbot

🔇 Additional comments (1)

.cursor/BUGBOT.md (1)

1-5: Рассмотрите размещение этого файла в отдельном PR.

Добавленный файл с рекомендацией по языку ревью логичен и соответствует практике репозитория (как подтверждают извлеченные рекомендации). Однако этот файл не связан с основной целью PR #351 — удалением префиксов из названий ролей. Рекомендую либо:

1. Разместить этот файл в отдельном PR для лучшей организации истории коммитов
2. Если этот файл критичен для текущего PR, уточнить в описании PR, почему он включен

Технически содержание файла корректно, но его логическая связь с PR неясна.

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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