Skip to content

add: магнитное улучшение для рудных сумок#8552

Open
SCPfollower wants to merge 18 commits intoss220-space:master220from
SCPfollower:ore_satchel_upgrade
Open

add: магнитное улучшение для рудных сумок#8552
SCPfollower wants to merge 18 commits intoss220-space:master220from
SCPfollower:ore_satchel_upgrade

Conversation

@SCPfollower
Copy link
Contributor

@SCPfollower SCPfollower commented Mar 2, 2026

Что этот ПР делает

Добавляет 2 новых предмета: улучшение для сумок для руды, который позволяет собирать руду в области 3 на 3 (продается за 600 очков в шахтерском вендомате и идет в базовом наборе со взрывчаткой с ваучера); улучшалку для боргов с тем же функционалом, но собственно для боргов

Почему это хорошо для игры

QoL для реально копающих шахтеров. И боргов.

Демонстрация изменений

Демонстрации изменений

Тестирование

Проверял на локалке, с виду все работает

@github-actions github-actions bot added the 🎨 Спрайты PR содержит изменения в файлах DMI. label Mar 2, 2026
@SCPfollower
Copy link
Contributor Author

/gemini review

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

Аналитический отчёт от Большого Брата. Обработка запроса завершена.

Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит удовлетворительно.

Сводка анализа:

  • 🔴 Критических замечаний: 0
  • 🟡 Рекомендаций: 3
  • 💡 Предложений: 0

🟡 Рекомендации по улучшению:

  • code/game/objects/items/robot/robot_upgrades.dm:898: В цикле используется несуществующий тип /obj/item/storage/bag/ore/mining_satchel. Вероятно, имелся в виду базовый тип /obj/item/storage/bag/ore.
  • code/modules/mining/ores_coins.dm:72: Логика сбора руды дублируется для обычного и улучшенного режима. Это можно упростить, вынеся повторяющийся код в общую часть и изменяя только радиус поиска.
  • code/game/objects/items/weapons/mining_satchel_upgrade.dm:36: Процедура update_desc изменяет глобальную переменную desc, что может привести к неожиданному поведению, если несколько объектов этого типа будут существовать одновременно. Рекомендуется использовать SEND_SIGNAL для обновления описания, чтобы избежать подобных проблем.

Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.

Спасибо за ваш вклад в проект!

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

Аналитический отчёт от Большого Брата. Обработка запроса завершена.

Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит удовлетворительно.

Сводка анализа:

  • 🔴 Критических замечаний: 0
  • 🟡 Рекомендаций: 4
  • 💡 Предложений: 0

🟡 Рекомендации по улучшению:

  • code/game/objects/items/robot/robot_upgrades.dm:273: Логика сохранения улучшения aoe при апгрейде до сумки хранения некорректна при наличии у борга нескольких сумок.
  • code/game/objects/items/robot/robot_upgrades.dm:894-914: Дублирование кода и потенциальный баг в процедурах action() и deactivate() для улучшения borg_mining_sat_upgr.
  • code/modules/mining/ores_coins.dm:72-87: Дублирование кода в логике подбора руды, что нарушает стайлгайд.
  • code/modules/research/designs/mechfabricator_designs.dm:1426: ID дизайна borg_upgrade_soh_magnet вводит в заблуждение, так как улучшение применимо ко всем шахтёрским сумкам, а не только к сумке хранения (SoH).

Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.

Спасибо за ваш вклад в проект!

@shahribek
Copy link
Contributor

была фича экслюзивом для мехов...
теперь не эксклюзив

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎨 Спрайты PR содержит изменения в файлах DMI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants