add: фальшивый имплант МЩ триторам#8544
add: фальшивый имплант МЩ триторам#8544SCPfollower wants to merge 15 commits intoss220-space:master220from
Conversation
zalupa экшены оболочки для мода оболочка для мода перенесена в мехфаб маркер. удаление устаревшего переопределённого прока get_equipped_items, замена передаваемых аргументов на битфлаги, где этого ещё не было. компоненту лазера добавлен кд на использование 1 секунда
|
/gemini review |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces changes related to a new 'fake mindshield' implant. However, a critical security vulnerability has been identified: the real mindshield in the Security Officer starter kit was erroneously replaced with this fake version. This leaves the station's security force vulnerable to antagonist conversion. Beyond this critical issue, the review also highlights several areas for improvement, including duplicated code, inefficient loop usage, and naming conventions for uplink items.
|
/gemini review |
There was a problem hiding this comment.
Code Review
Аналитический отчёт от Большого Брата. Обработка запроса завершена.
Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит требующей доработок.
Сводка анализа:
- 🔴 Критических замечаний: 1
- 🟡 Рекомендаций: 2
- 💡 Предложений: 0
🔴 Критические замечания (требуют обязательного исправления):
code/game/objects/items/weapons/storage/boxes.dm:1123: Стартовый набор офицера службы безопасности теперь содержит имплантер с фальшивым имплантом защиты разума вместо настоящего. Это может привести к непреднамеренным последствиям в игровом процессе, так как офицеры будут получать предмет, предназначенный для антагонистов.
🟡 Рекомендации по улучшению:
code/game/data_huds.dm:355-361: Логика проверки типа импланта может быть упрощена для улучшения читаемости и устранения избыточной проверкиistype().code/game/objects/items/weapons/implants/implant_mindshield.dm:103-108: Блок кода для переключения состоянияhud_visibleиicon_stateможно сделать более лаконичным, что улучшит читаемость.
Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.
Спасибо за ваш вклад в проект!
There was a problem hiding this comment.
Code Review
Аналитический отчёт от Большого Брата. Обработка запроса завершена.
Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит требующей доработок.
Сводка анализа:
- 🔴 Критических замечаний: 1
- 🟡 Рекомендаций: 2
- 💡 Предложений: 0
🔴 Критические замечания (требуют обязательного исправления):
code/game/objects/items/weapons/storage/boxes.dm:1123: Стартовый набор офицера СБ содержит фальшивый имплант, предназначенный для антагонистов. Это может быть ошибкой, которая повлияет на игровой баланс.
🟡 Рекомендации по улучшению:
code/game/data_huds.dm:355-361: Обнаружена избыточная проверка типа, которую можно упростить для улучшения читаемости и производительности.code/game/objects/items/weapons/implants/implant_mindshield.dm:103-108: Логика переключения состояния импланта может быть сделана более лаконичной.
Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.
Спасибо за ваш вклад в проект!
dageavtobusnick
added
the
📜 Локализация
pilygun
left a comment
pilygun
left a comment
There was a problem hiding this comment.
Локализация норм,
по поводу предлоги я бы на твоем месте поставил импланту цену 8-10 ТК.
Стоит обычный МЩ в бандле офицера заменить на этот.
Опционально добавить в очень редкий лут техов / контрабандные заказы карго
pilygun
added
the
☑️ Локализация одобрена
|
Не уверен на счет повышения цены, ибо за 7 тк можно взять станглавсы, за 6 тк джаммер. |
4 participants
Что этот ПР делает
Добавляет новый имплант - фальшивый МЩ, что имитирует МЩ на ХУДах СБ + может включаться/выключаться. Не дает защиту от порабощения/стана культей и пр
Почему это хорошо для игры
Больше возможностей для стелса!
Демонстрация изменений
Демонстрации изменений
Тестирование
Проверял на локалке, с виду все работает