Added 10 salvage wrecks to the magnet#4011
Merged
KaiserMaus merged 4 commits intoMar 18, 2026
Merged
Conversation
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughДобавлены девять новых файлов карт Salvage под Changes
Оценка сложности рецензирования кода🎯 4 (Сложная) | ⏱️ ~45 минут Suggested labels
Стихотворение
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment |
coderabbitai
Bot
added
🧊 Нет C# 🧊
🔎 На Рассмотрение 🔎
🗺️ Карты 🗺️
📦 YAML | Прототип 📦
labels
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 4
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@Resources/Maps/Salvage/medium-brigmed.yml`:
- Around line 524-533: В файле карты используется прототип DebugAPCRecharging,
который берёт компонент BatterySelfRecharger из
Resources/Prototypes/Entities/Structures/Power/debug_power.yml и даёт
бесконечную зарядку; замените этот прототип в записи сущности (proto:
DebugAPCRecharging) на подходящий боевой/игровой прототип APC (или другой
нормальный прототип питания без BatterySelfRecharger) либо удалите/исправьте
прототип так, чтобы не применять BatterySelfRecharger с 50000W, чтобы карта не
имела отладочного вечного источника энергии.
In `@Resources/Maps/Salvage/medium-diamondroid.yml`:
- Around line 434-454: ShelfWood (proto: ShelfWood, entity uid: 36) is being
placed at the same coordinates (pos: 3.5,2.5) as another entity (uid: 67 /
WallMining) so the shelf with storedItems (storedItems: 37) spawns inside the
wall and blocks PKAUpgradeRange; to fix, move the ShelfWood entity (uid: 36) to
a free adjacent tile or change its parent so it is not inside WallMining, or
alternatively shift uid: 67 away from 3.5,2.5; apply the same correction for the
duplicate case referenced at lines 483-488 (ensure no two entities share pos:
3.5,2.5 and that ShelfWood isn’t enclosed by WallMining).
In `@Resources/Maps/Salvage/small-mercsand.yml`:
- Around line 285-293: В файле найдена сущность с proto: DebugAPCRecharging (uid
86) — это отладочная APC с авто-зарядом 50000W; замените или удалите её:
откройте запись сущности с uid 86 и либо поменяйте proto на боевой/обычный
прототип APC (например стандартный APC/PowerAPC в проекте), либо полностью
удалите поле proto/вставьте отсутствие APC, чтобы этот wreck не имел бесплатного
питания; убедитесь, что больше нет ссылок на DebugAPCRecharging в той же
сущности.
- Around line 121-130: В файле есть дублирующие сущности CableApcExtension с uid
95 и uid 96, обе с pos: 3.5,-0.5; удалите одну из этих записей (оставьте только
одну сущность на позиции 3.5,-0.5) и убедитесь, что оставшаяся запись сохраняет
корректный uid и parent, чтобы не создавать наложенных объектов и лишних узлов
сети.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: b325dd8e-6f68-4148-885d-30be28ae8fbd
📒 Files selected for processing (12)
Resources/Maps/Salvage/large-syndieoutpost.ymlResources/Maps/Salvage/large-xenoroid.ymlResources/Maps/Salvage/medium-brigmed.ymlResources/Maps/Salvage/medium-diamondroid.ymlResources/Maps/Salvage/medium-freelancershuttle.ymlResources/Maps/Salvage/small-defibwreck.ymlResources/Maps/Salvage/small-johnspiderclan.ymlResources/Maps/Salvage/small-mercsand.ymlResources/Maps/Salvage/small-scichunk.ymlResources/Maps/Salvage/small-xenochunk.ymlResources/Prototypes/Maps/salvage.ymlResources/Prototypes/_Sunrise/Maps/salvage.yml
Contributor
There was a problem hiding this comment.
Actionable comments posted: 4
♻️ Duplicate comments (3)
Resources/Maps/_Sunrise/Salvage/medium-brigmed.yml (1)
510-519:⚠️ Potential issue | 🟠 MajorЗдесь по-прежнему используется
DebugAPCRecharging.В
Resources/Prototypes/Entities/Structures/Power/debug_power.yml:156-164этот прототип помечен какDebugи получаетBatterySelfRechargerна50000. На карте обломка это даёт бесконечное питание и ломает ожидаемый дефицит энергии. Здесь нужен обычный APC или другой игровой источник питания без отладочной автозарядки.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Resources/Maps/_Sunrise/Salvage/medium-brigmed.yml` around lines 510 - 519, В карте используется прототип DebugAPCRecharging, который содержит Debug-качества (BatterySelfRecharger = 50000) и даёт бесконечную энергию; замените строку proto: DebugAPCRecharging в этой сущности на обычный игровой источник питания (например стандартный APC или любой другой не-Debug прототип питания), чтобы убрать автозарядку; убедитесь, что новая сущность не содержит компонента BatterySelfRecharger с циклом 50000 или иного debug-реклаймера.Resources/Maps/_Sunrise/Salvage/small-mercsand.yml (2)
285-293:⚠️ Potential issue | 🟠 Major
DebugAPCRechargingне должен использоваться в продакшен-картах.
DebugAPCRecharging— это отладочная сущность с авто-зарядом 50000W. Из-за неё этот wreck всегда будет иметь бесконечное бесплатное питание, что ломает баланс сальважа.Замените на обычный
APCBasicили удалите, если питание не требуется.,
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Resources/Maps/_Sunrise/Salvage/small-mercsand.yml` around lines 285 - 293, В блоке сущности с uid: 86 замените прототип DebugAPCRecharging (отладочная единица с авто‑зарядом 50000W) на боевой прототип APCBasic или полностью удалите этот объект из списка entities если питание не требуется; найдите запись proto: DebugAPCRecharging и обновите её на proto: APCBasic или удалите весь объект с uid: 86 (включая его компоненты Transform и Fixtures), чтобы убрать бесконечный бесплатный питер и восстановить баланс сальважа.
121-130:⚠️ Potential issue | 🟡 MinorДубликат
CableApcExtensionна одной клетке.Два кабеля (uid: 95 и uid: 96) созданы на одной позиции
3.5,-0.5. Это приводит к наложению сущностей и лишнему узлу сети.🔧 Предлагаемое исправление
- uid: 95 components: - type: Transform pos: 3.5,-0.5 parent: 1 - - uid: 96 - components: - - type: Transform - pos: 3.5,-0.5 - parent: 1 - uid: 97,
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Resources/Maps/_Sunrise/Salvage/small-mercsand.yml` around lines 121 - 130, There are two CableApcExtension entities overlapping at pos 3.5,-0.5 (uid: 95 and uid: 96); remove or de-duplicate one of them (e.g., delete the block with uid: 96) or move it to a unique position so there’s only one entity at 3.5,-0.5; ensure the remaining component keeps the type: Transform with pos: 3.5,-0.5 and parent: 1 and that no other uid duplicates persist.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@Resources/Maps/_Sunrise/Salvage/large-xenoroid.yml`:
- Around line 415-432: В файле фигурируют отладочные APC `DebugAPCRecharging`
(uid: 55 и uid: 56) с бесконечной подачей энергии — замените эти сущности на
боевые `APCBasic` или удалите их; если оставляете, подключите их кабелями к
существующему `GeneratorBasic` (uid: 61) и уберите/измените параметры
авто‑заряда, чтобы убрать 50000W. Найдите определения сущностей с uid 55 и 56 и
либо замените их proto на `APCBasic` и добавьте/настройте проводку к
`GeneratorBasic`, либо удалите эти блоки из списка entities, чтобы устранить
дебаг-APC из продакшен-карты.
In `@Resources/Maps/_Sunrise/Salvage/medium-freelancershuttle.yml`:
- Around line 247-256: Замените прототип DebugAPCRecharging в блоке сущности
(uid: 19) на подходящий игровой APC без отладочной автозарядки (например просто
APC или StandardAPC), чтобы убрать привязку к Debug прототипу с
BatterySelfRecharger=50000; откройте запись с proto: DebugAPCRecharging в
medium-freelancershuttle.yml и подмените значение proto на стандартный игровой
источник питания, убедившись, что новый прототип не содержит компонента
BatterySelfRecharger.
In `@Resources/Maps/_Sunrise/Salvage/small-johnspiderclan.yml`:
- Around line 175-187: В карте используется прототип DebugAPCRecharging, который
помечен как Debug и даёт компонент BatterySelfRecharger с 50000 (в
debug_power.yml), что обеспечивает бесконечную энергию; замените proto:
DebugAPCRecharging на обычный игровой источник питания (например, стандартный
APC/не-debug прототип вместо DebugAPCRecharging) в записи с uid: 19 либо
подберите другой не-debug прототип питания без BatterySelfRecharger, чтобы
восстановить ожидаемый дефицит энергии.
In `@Resources/Maps/_Sunrise/Salvage/small-scichunk.yml`:
- Around line 180-188: В блоке сущности замените использование отладочного
прототипа DebugAPCRecharging (который даёт BatterySelfRecharger=50000) на
обычный игровой источник питания (например стандартный APC/PowerAPC прототип),
чтобы убрать автозарядку и вернуть нормальный дефицит энергии; откройте запись с
proto: DebugAPCRecharging и поменяйте имя прототипа на соответствующий
боевой/игровой APC (без Debug/DebugAPCRecharging) и убедитесь, что связанные
компоненты (например BatterySelfRecharger) удалены или не заданы.
---
Duplicate comments:
In `@Resources/Maps/_Sunrise/Salvage/medium-brigmed.yml`:
- Around line 510-519: В карте используется прототип DebugAPCRecharging, который
содержит Debug-качества (BatterySelfRecharger = 50000) и даёт бесконечную
энергию; замените строку proto: DebugAPCRecharging в этой сущности на обычный
игровой источник питания (например стандартный APC или любой другой не-Debug
прототип питания), чтобы убрать автозарядку; убедитесь, что новая сущность не
содержит компонента BatterySelfRecharger с циклом 50000 или иного
debug-реклаймера.
In `@Resources/Maps/_Sunrise/Salvage/small-mercsand.yml`:
- Around line 285-293: В блоке сущности с uid: 86 замените прототип
DebugAPCRecharging (отладочная единица с авто‑зарядом 50000W) на боевой прототип
APCBasic или полностью удалите этот объект из списка entities если питание не
требуется; найдите запись proto: DebugAPCRecharging и обновите её на proto:
APCBasic или удалите весь объект с uid: 86 (включая его компоненты Transform и
Fixtures), чтобы убрать бесконечный бесплатный питер и восстановить баланс
сальважа.
- Around line 121-130: There are two CableApcExtension entities overlapping at
pos 3.5,-0.5 (uid: 95 and uid: 96); remove or de-duplicate one of them (e.g.,
delete the block with uid: 96) or move it to a unique position so there’s only
one entity at 3.5,-0.5; ensure the remaining component keeps the type: Transform
with pos: 3.5,-0.5 and parent: 1 and that no other uid duplicates persist.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 0e5b4980-e015-46a5-9659-47d9cba2ba5a
📒 Files selected for processing (12)
Resources/Maps/_Sunrise/Salvage/large-syndieoutpost.ymlResources/Maps/_Sunrise/Salvage/large-xenoroid.ymlResources/Maps/_Sunrise/Salvage/medium-brigmed.ymlResources/Maps/_Sunrise/Salvage/medium-diamondroid.ymlResources/Maps/_Sunrise/Salvage/medium-freelancershuttle.ymlResources/Maps/_Sunrise/Salvage/small-defibwreck.ymlResources/Maps/_Sunrise/Salvage/small-johnspiderclan.ymlResources/Maps/_Sunrise/Salvage/small-mercsand.ymlResources/Maps/_Sunrise/Salvage/small-scichunk.ymlResources/Maps/_Sunrise/Salvage/small-xenochunk.ymlResources/Prototypes/Maps/salvage.ymlResources/Prototypes/_Sunrise/Maps/salvage.yml
💤 Files with no reviewable changes (1)
- Resources/Prototypes/Maps/salvage.yml
coderabbitai
Bot
added
🛠️ Есть C# 🛠️
github-project-automation
Bot
moved this from НЕ РАЗОБРАНО
to СДЕЛАНО
in Разбор PR и Issues
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Кратное описание
Ранний порт 41904
По какой причине
Медиа(Видео/Скриншоты)
space-wizards/space-station-14#41904
Смотрите сами скриншоты
Changelog
🆑 Bandirpvp
Summary by CodeRabbit