Skip to content

Added Indication on the device if AI can control it#370

Open
Yuoko wants to merge 3 commits into
masterfrom
ai-slop
Open

Added Indication on the device if AI can control it#370
Yuoko wants to merge 3 commits into
masterfrom
ai-slop

Conversation

@Yuoko

@Yuoko Yuoko commented Mar 16, 2026

Copy link
Copy Markdown
Collaborator

Описание PR

При осмотре устройств теперь будет появляться приписка "Это устройство может управляться ИИ", если оно и правда может.

Частичный порт непринятого PRа губстейшенов - Goob-Station/Goob-Station#5726

Считаю что это неплохое изменение, ведь поможет простому экипажу ориентироваться в том, где они могут применить ИИ и исключит всякие глупые вопросы.

Технические детали

Устройство имеющее компонент StationAIWhitelistComponent при ивенте OnExamine (осмотр) имеет приписку "Это устройство может управляться ИИ".

Это было необязательно, на приписка есть на двух локализациях (en-ES и ru-RU)

Медиа

image image image image

Чек-лист

  • PR полностью завершён и мне не нужна помощь, чтобы его закончить.
  • Я запускал локальный сервер со своими изменениями, всё протестировал, и всё работает как должно.

Список изменений

🆑 Yuoko

  • add: При осмотре устройств к которым станционный ИИ имеет доступ, теперь можно увидеть соответствующее сообщение обозначающее этот факт.

При осмотре машинерии и не только теперь указывается, если ИИ имеет доступ и может управлять этим устройством.
@Yuoko Yuoko requested review from CrimeMoot and HyperB1 as code owners March 16, 2026 05:48
@github-actions github-actions Bot added S: Needs Review Status: Требуется рассмотрение Changes: Localization Изменения затрагивают локализацию. size/S labels Mar 16, 2026
@coderabbitai

coderabbitai Bot commented Mar 16, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Добавлена поддержка осмотра для StationAiWhitelistComponent: подписка на событие ExaminedEvent и обработчик, который при осмотре отправляет локализованную строку station-ai-whitelist-examine. Добавлены соответствующие переводы на английском и русском.

Changes

Cohort / File(s) Summary
Система осмотра StationAi
Content.Shared/Silicons/StationAi/Systems/SharedStationAiSystem.cs
Добавлена подписка на ExaminedEvent и метод-обработчик, который при осмотре отправляет локализованную разметку station-ai-whitelist-examine.
Локализация
Resources/Locale/en-US/_Goobstation/silicons/station-ai.ftl, Resources/Locale/ru-RU/_Ganimed/silicons/station-ai.ftl
Добавлен ключ station-ai-whitelist-examine с переводами: английский — "This device can be controlled by an AI.", русский — "Это устройство может управляться ИИ.".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

size/XS, S: Needs Review, CL: Valid, A: Command

Suggested reviewers

  • CrimeMoot
  • HyperB1
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 40.00% which is insufficient. The required threshold is 50.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description check ✅ Passed Описание полностью соответствует изменениям: объясняет функциональность, причину, технические детали и включает скриншоты.
Title check ✅ Passed The title accurately describes the main change: adding visual indication on devices showing whether the AI can control them.

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

📝 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

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

@Yuoko Yuoko added the P3: Standard Стандартный приоритет рассмотрения label Mar 16, 2026
@Yuoko Yuoko added this to the AI overhaul & new features milestone Mar 16, 2026
@Yuoko Yuoko self-assigned this Mar 16, 2026
@Yuoko Yuoko added the A: Silicon Area: Silicons, AI, cyborgs. label Mar 16, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
Resources/Locale/ru-RU/_Ganimed/silicons/station-ai.ftl (1)

1-1: Несоответствие путей директорий между локализациями.

Русский перевод находится в _Ganimed/, а английский — в _Goobstation/. Для удобства поддержки рекомендуется использовать единую структуру директорий для всех локализаций.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 0e5c0dd2-fdd3-4177-aaa3-b0039e5f7752

📥 Commits

Reviewing files that changed from the base of the PR and between 2013dac and b58ef91.

📒 Files selected for processing (3)
  • Content.Shared/Silicons/StationAi/Systems/SharedStationAiSystem.cs
  • Resources/Locale/en-US/_Goobstation/silicons/station-ai.ftl
  • Resources/Locale/ru-RU/_Ganimed/silicons/station-ai.ftl
📜 Review details
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2026-01-28T21:02:49.548Z
Learnt from: HyperB1
Repo: ss14-ganimed/Ganimed14 PR: 350
File: Resources/Locale/en-US/_Harmony/game-ticking/game-presets/preset-conspirator.ftl:14-14
Timestamp: 2026-01-28T21:02:49.548Z
Learning: Verify that locale FTl files under Resources/Locale maintain consistency with control keycodes. If a fork modifies a keycode (e.g., 'o' → 'г' in radio_channels.yml for a specific channel), the English (en-US) locale should be updated to reflect the intended behavior or explicitly document the exception. Ensure the English instruction uses the same keycode as the default/original behavior unless the fork’s logic requires a deliberate change, to avoid localization discrepancies.

Applied to files:

  • Resources/Locale/en-US/_Goobstation/silicons/station-ai.ftl
  • Resources/Locale/ru-RU/_Ganimed/silicons/station-ai.ftl
🔇 Additional comments (2)
Resources/Locale/en-US/_Goobstation/silicons/station-ai.ftl (1)

1-1: LGTM!

Текст перевода корректен и ключ локализации соответствует русскому файлу.

Content.Shared/Silicons/StationAi/Systems/SharedStationAiSystem.cs (1)

37-37: LGTM!

Реализация корректна:

  • Подписка на ExaminedEvent соответствует существующим паттернам в системе.
  • Проверка !component.Enabled корректно предотвращает отображение сообщения, когда управление ИИ отключено (например, перерезан провод).
  • Использование PushMarkup соответствует практике добавления текста при осмотре.

Also applies to: 118-119, 395-401

@HyperB1 HyperB1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Всё добавленное и изменённое помечай комментарием.
  2. В конце каждого нового файла должен быть один LF.
  3. Требуется тест, поменяется ли описание, если на устройстве (шлюзе или вендомате) перерезать AI access провод.
  4. А зачем оно вообще? У нас почему/баланс в описание PR включён если что.

Comment thread Content.Shared/Silicons/StationAi/Systems/SharedStationAiSystem.cs Outdated
Comment thread Content.Shared/Silicons/StationAi/Systems/SharedStationAiSystem.cs Outdated
Comment thread Content.Shared/Silicons/StationAi/Systems/SharedStationAiSystem.cs
Comment thread Resources/Locale/en-US/_Goobstation/silicons/station-ai.ftl Outdated
Comment thread Resources/Locale/ru-RU/_Ganimed/silicons/station-ai.ftl Outdated
@github-actions github-actions Bot added S: Awaiting Changes Status: В ожидании запрошенных изменений and removed S: Needs Review Status: Требуется рассмотрение labels Mar 16, 2026
@Yuoko

Yuoko commented Mar 16, 2026

Copy link
Copy Markdown
Collaborator Author
  1. Всё добавленное и изменённое помечай комментарием.
  2. В конце каждого нового файла должен быть один LF.
  3. Требуется тест, поменяется ли описание, если на устройстве (шлюзе или вендомате) перерезать AI access провод.
  4. А зачем оно вообще? У нас почему/баланс в описание PR включён если что.

Пометили.
Сделал.
Провёл тест, надпись пропадает. Делаю фикс что-бы избежать меты
Это добавит удобства для экипажа, будет намного проще понять с чем ИИ может тебе помочь и исключит глупые вопросы связанные с "ИИ А ТЫ МОЖЕШЬ ВЫДАТЬ МНЕ ПОПИТЬ" или в этом духе.

@Yuoko Yuoko requested a review from HyperB1 March 16, 2026 06:54
@github-actions github-actions Bot added the S: Needs Review Status: Требуется рассмотрение label Mar 16, 2026
@Yuoko

Yuoko commented Mar 16, 2026

Copy link
Copy Markdown
Collaborator Author
image Исправил

@github-actions github-actions Bot removed the S: Awaiting Changes Status: В ожидании запрошенных изменений label Mar 16, 2026
@Yuoko Yuoko changed the title AI small enhancement Added Indication on the device if AI can control it Mar 16, 2026
@HyperB1 HyperB1 added the T: Enhancement Type: Новый контент, QoL и улучшения label Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: Silicon Area: Silicons, AI, cyborgs. Changes: Localization Изменения затрагивают локализацию. P3: Standard Стандартный приоритет рассмотрения S: Needs Review Status: Требуется рассмотрение size/S T: Enhancement Type: Новый контент, QoL и улучшения

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants