[Tweak] Улучшены портативные мониторы ОСЩ и Бригмедика 2.

[Armorkillerd]

Описание PR

Теперь портативные мониторы видят категорию неизвестных.
И локаль.

Почему / Баланс

Техническая информация

• Изменения были протестированы на локальном сервере, и всё работает отлично.
• PR закончен и требует просмотра изменений.

Медиа

Чейнджлог

🆑 Armorkillerd & WsWise

• tweak: Теперь портативные мониторы ОСЩ и бригмедика видят раздел неизвестных должностей.

[Armorkillerd]

терь без говна

[coderabbitai]

Walkthrough

Этот pull request содержит два основных изменения. Первое — модификация логики фильтрации датчиков в CrewMonitoringConsoleSystem.cs для включения датчиков без назначенных отделов, с потенциальной проблемой обработки null-значений. Второе — добавление локализованных строк интерфейса на русском языке для компонента crew-monitoring-component, включая заголовки, заполнители фильтров и текст ошибок.

Possibly related PRs

• [ADD] Монитор бригмедика  #1914: Другой PR также модифицирует логику фильтрации датчиков по отделам в CrewMonitoringConsoleSystem.cs, добавляя фильтрацию на основе пересечения JobDepartments с допустимыми отделами.

Suggested labels

Changes: Localization, Changes: UI, Status: Needs Review, S: Untriaged, size/S

Suggested reviewers

• Unlumy
• M1and1B
• Ratyyy

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description check	✅ Passed	Описание PR связано с набором изменений и объясняет, что портативные мониторы теперь видят категорию неизвестных должностей, а также была добавлена локализация.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	Заголовок точно отражает основное изменение: улучшение портативных мониторов для медицинских отделений путём добавления поддержки неизвестных ролей и локализации.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

📝 Coding Plan

• Generate coding plan for human review comments

---

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.}

Tip

You can generate walkthrough in a markdown collapsible section to save space.

Enable the reviews.collapse_walkthrough setting to generate walkthrough in a markdown collapsible section.

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

Content.Server/Medical/CrewMonitoring/CrewMonitoringConsoleSystem.cs (1)

121-121: ⚠️ Potential issue | 🟡 Minor

Для нового popup-ключа не хватает en-US записи.

Loc.GetString("crew-monitoring-component-upgrade-emag") вызывается на сервере, но в предоставленном Resources/Locale/en-US/medical/components/crew-monitoring-component.ftl этого ключа нет. На en-US сервере сообщение останется без нормальной локализации.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Server/Medical/CrewMonitoring/CrewMonitoringConsoleSystem.cs` at line
121, The popup key "crew-monitoring-component-upgrade-emag" used in
CrewMonitoringConsoleSystem.cs via Loc.GetString(...) is missing an en-US entry,
so add a corresponding translation line to
Resources/Locale/en-US/medical/components/crew-monitoring-component.ftl with the
same key and an appropriate English message (e.g., the EMAG upgrade popup text)
so the server will display a localized string instead of the raw key.

🧹 Nitpick comments (1)

Content.Server/Medical/CrewMonitoring/CrewMonitoringConsoleSystem.cs (1)

115-126: Добавьте /// <summary> к OnEmagged.

Это новый важный обработчик состояния консоли; короткая XML-документация упростит поддержку.

As per coding guidelines "и предлагай /// summary документацию к C# коду, к важным функциям или классам".

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Server/Medical/CrewMonitoring/CrewMonitoringConsoleSystem.cs` around
lines 115 - 126, Add an XML doc summary for the OnEmagged method: place a ///
<summary> block immediately above the private void OnEmagged(EntityUid uid,
CrewMonitoringConsoleComponent component, ref GotEmaggedEvent ev) declaration
that briefly describes its purpose (handles GotEmaggedEvent for the crew
monitoring console, plays spark sound, shows popup, sets component.IsEmagged,
updates the UI and marks the event as handled), and include any important
parameter notes if desired (e.g., that uid is the console entity and ev is
consumed). Ensure the summary uses standard C# XML comment format and is
concise.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@Content.Server/Medical/CrewMonitoring/CrewMonitoringConsoleSystem.cs`:
- Line 121: The popup key "crew-monitoring-component-upgrade-emag" used in
CrewMonitoringConsoleSystem.cs via Loc.GetString(...) is missing an en-US entry,
so add a corresponding translation line to
Resources/Locale/en-US/medical/components/crew-monitoring-component.ftl with the
same key and an appropriate English message (e.g., the EMAG upgrade popup text)
so the server will display a localized string instead of the raw key.

---

Nitpick comments:
In `@Content.Server/Medical/CrewMonitoring/CrewMonitoringConsoleSystem.cs`:
- Around line 115-126: Add an XML doc summary for the OnEmagged method: place a
/// <summary> block immediately above the private void OnEmagged(EntityUid uid,
CrewMonitoringConsoleComponent component, ref GotEmaggedEvent ev) declaration
that briefly describes its purpose (handles GotEmaggedEvent for the crew
monitoring console, plays spark sound, shows popup, sets component.IsEmagged,
updates the UI and marks the event as handled), and include any important
parameter notes if desired (e.g., that uid is the console entity and ev is
consumed). Ensure the summary uses standard C# XML comment format and is
concise.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ca6ee1d5-01d0-41d7-812d-7a2fe1b41112

📥 Commits

Reviewing files that changed from the base of the PR and between f44e20c and 4596345.

📒 Files selected for processing (2)

• Content.Server/Medical/CrewMonitoring/CrewMonitoringConsoleSystem.cs
• Resources/Locale/ru-RU/ADT/medical/components/crew-monitoring-component.ftl