Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Бафф Инжей и не только #310

Merged
merged 17 commits into from
Oct 31, 2024

Conversation

Farrellka-dev
Copy link
Contributor

@Farrellka-dev Farrellka-dev commented Oct 29, 2024

image

Описание обновления:

Я пробафал Инженерный Отдел, Полевого Врача, Главу Персонала и Офицера "Синий Щит".
Добавил трусики и ручки главам.
Задебафал перк на бионическую руку.

Медиа

та лень делать скрин

Проверки

  • PR завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.
  • Я не добавлял контент нарушающий чужие авторские права.
  • Я добавил свой контент только в папку _LostParadise(только если вы не меняете оригинальный код).
  • Я не добавлял переводы в прототипы, а сделал их с помощью скрипта translations.bat в Tools/SS14_RU.

Изменения

🆑 Farrellka

  • add: Добавлен газоанализатор в КПК всем инженерам!
  • add: Добавлена возможность Суперматерии говорить в рацию Инженерам о её нестабильности!
  • add: Добавлены магнитбутсы в шкафы атмосам.
  • add: В лодаут СИ добавлены его одежда, а так же каски.
  • add: Добавлен боевой гипо в шкаф Полевого Врача!
  • add: Добавлен боевой гипо за 4 очка, а так же боевая аптечка за 2 очка в лодаут Офицера "Синий Щит"!
  • add: Добавлены трусики главам отделов в их шкафчики, а так же ручки в их КПК!
  • tweak: Изменены скафандры инженеров. Теперь их шлема защищают от сварки.
  • tweak: Изменён перк "Бионическая рука". Теперь она работает как лом, открывая только выключенные шлюзы. Так же стоимость уменьшена с 9 до 4.
  • tweak: Изменены гарнитуры Главы Персонала. Теперь они имеют все ключи шифрования, как это было раньше.
  • fix: Исправлены КПК Полевого Инженера и Пилота СБ! Теперь они имеют программы Службы Безопасности.
  • fix: Исправлен лодаут Офицера "Синий Щит". Теперь можно выбрать лишь один пистолет .45 и обойму к нему.

Copy link
Contributor

coderabbitai bot commented Oct 29, 2024

Walkthrough

В данном пулл-реквесте представлено множество изменений в различных файлах конфигурации и локализации. Основные изменения касаются добавления новых зависимостей, методов и сущностей, а также обновления существующих настроек для улучшения функциональности системы. Включены новые элементы инвентаря, обновлены описания и исправлены ошибки в локализации. Также были внесены изменения в определения ролей и загрузок, что повлияло на механики игры и структуру иерархий.

Changes

Файл Изменения
Content.Server/Supermatter/Systems/SupermatterSystem.Processing.cs Добавлены зависимости IPrototypeManager и RadioSystem, новый метод Report, обновлен метод SendSupermatterAnnouncement.
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/devices/pda.ftl Изменена капитализация идентификаторов для "Adjutant", "Expeditor", "SecPilot".
Resources/Locale/ru-RU/ss14-ru/prototypes/deltav/entities/clothing/ears/headsets.ftl Исправлены опечатки и капитализация в описаниях для нескольких гарнитур.
Resources/Locale/ru-RU/ss14-ru/prototypes/entities/structures/power/generation/supermatter/supermatter.ftl Исправлена орфография термина "Supermatter".
Resources/Prototypes/Catalog/Fills/Lockers/engineer.yml Добавлены новые элементы и вероятности для нескольких объектов хранения.
Resources/Prototypes/Catalog/Fills/Lockers/heads.yml Добавлены новые сущности для шкафчиков, обновлены содержимое и вероятности.
Resources/Prototypes/Catalog/Fills/Lockers/security.yml Удалена сущность LockerWardenFilledHardsuit, обновлены другие шкафчики и добавлены новые элементы оружия.
Resources/Prototypes/Catalog/Fills/Lockers/suit_storage.yml Добавлены новые сущности для хранения костюмов, обновлены существующие.
Resources/Prototypes/DeltaV/Entities/Clothing/Ears/headsets.yml Обновлены ключи для гарнитур, заменены некоторые ключи на EncryptionKeyStationMaster.
Resources/Prototypes/Entities/Clothing/Head/hardsuit-helmets.yml Добавлены компоненты защиты для нескольких шлемов.
Resources/Prototypes/Entities/Objects/Devices/pda.yml Добавлена новая сущность BaseEngineerPDA, обновлены родительские классы для нескольких PDA.
Resources/Prototypes/Traits/physical.yml Изменены очки и компоненты для различных черт, некоторые черты закомментированы.
Resources/Prototypes/_LostParadise/CharacterItemGroups/bsGroups.yml Добавлены новые группы предметов для персонажей.
Resources/Prototypes/_LostParadise/Entities/Clothing/Eyes/Hud/hud.yml Добавлены новые сущности для HUD, обновлены существующие.
Resources/Prototypes/_LostParadise/Entities/Objects/Devices/pda.yml Обновлена родительская сущность для LPPSecurityEngineerPDA, добавлены компоненты.
Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml Добавлены новые загрузки и обновлены требования для существующих загрузок.
Resources/Prototypes/Roles/Jobs/Engineering/atmospheric_technician.yml Добавлено свойство canBeAntag: false для роли Atmospheric Technician.
Resources/Prototypes/Roles/Jobs/Engineering/senior_engineer.yml Добавлено свойство canBeAntag: false для роли Senior Engineer.
Resources/Prototypes/Roles/Jobs/Medical/senior_physician.yml Добавлено свойство canBeAntag: false для роли Senior Physician.
Resources/Prototypes/Roles/Jobs/Science/senior_researcher.yml Добавлено свойство canBeAntag: false, обновлены требования к отделу.
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/misc/papers/pen.ftl Изменено название для ручки квартирмейстера.
Resources/Prototypes/Loadouts/Jobs/Heads/chiefEngineer.yml Добавлены новые загрузки для роли Chief Engineer.
Resources/Prototypes/Loadouts/Jobs/engineering.yml Добавлены новые загрузки, обновлены требования для существующих загрузок.

Sequence Diagram(s)

sequenceDiagram
    participant SupermatterSystem
    participant IPrototypeManager
    participant RadioSystem

    SupermatterSystem->>IPrototypeManager: Получение канала
    SupermatterSystem->>RadioSystem: Отправка сообщения
    SupermatterSystem->>SupermatterSystem: Отправка объявления
Loading

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: ASSERTIVE

📥 Commits

Reviewing files that changed from the base of the PR and between 4b87fb5 and b241544.

📒 Files selected for processing (1)
  • Resources/Prototypes/Entities/Clothing/Head/hardsuit-helmets.yml (4 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
Resources/Prototypes/Entities/Clothing/Head/hardsuit-helmets.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

🪛 yamllint
Resources/Prototypes/Entities/Clothing/Head/hardsuit-helmets.yml

[error] 598-598: trailing spaces

(trailing-spaces)

🔇 Additional comments (3)
Resources/Prototypes/Entities/Clothing/Head/hardsuit-helmets.yml (3)

43-43: Улучшена защита глаз для инженерных скафандров!

Добавление компонента EyeProtection для инженерных скафандров является логичным улучшением, так как инженеры часто работают со сваркой и другими опасными для глаз операциями.

Also applies to: 62-62, 254-254


Line range hint 171-172: Обновление названия и описания скафандра медика

Переименование с "brigmedic" на "corpsman" в названии и описании улучшает согласованность терминологии.

🧰 Tools
🪛 yamllint

[warning] 45-45: missing starting space in comment

(comments)


596-598: 🧹 Nitpick (assertive)

Значительное улучшение защиты для скафандра инженера ERT

Добавление компонентов FlashImmunity и EyeProtection усиливает защитные свойства скафандра, что соответствует его роли в элитном спасательном подразделении.

Обнаружен лишний пробел в конце строки 598. Предлагаю исправить:

  - type: EyeProtection
-  
+

Likely invalid or redundant comment.

🧰 Tools
🪛 yamllint

[error] 598-598: trailing spaces

(trailing-spaces)


Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@github-actions github-actions bot added Статус: Требуется одобрение Ожидание разработчиков для принятия решений Изменения: C# Изменяет какие-либо файлы .cs Изменения: Локализация Изменяет какие-либо файлы .ftl Изменения: YML Изменяет какие-либо файлы .yml labels Oct 29, 2024
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 4

🧹 Outside diff range comments (9)
Resources/Prototypes/_LostParadise/Entities/Clothing/Eyes/Hud/hud.yml (2)

Line range hint 8-51: Рекомендуется рефакторинг для уменьшения дублирования кода

Три сущности очков (Black, Red, Blue) имеют идентичную структуру, отличаясь только цветом и именем. Предлагаю создать базовую сущность для уменьшения дублирования.

- type: entity
  id: LPPBaseHudGlasses
  parent: [ClothingEyesBase, LPPShowSecurityIcons]
  abstract: true
  components:
  - type: FlashImmunity
  - type: EyeProtection
    protectionTime: 5

- type: entity
  parent: LPPBaseHudGlasses
  id: LPPBlackHudGlasses
  name: Sunglasses with HUD
  description: Useful for Security and Cargo.
  components:
  - type: Sprite
    sprite: _LostParadise/Clothing/Eyes/Glasses/blackhudglasses.rsi
  - type: Clothing
    sprite: _LostParadise/Clothing/Eyes/Glasses/blackhudglasses.rsi
🧰 Tools
🪛 yamllint

[error] 81-81: wrong indentation: expected 6 but found 4

(indentation)


Line range hint 83-97: Необходимо переработать описание NPZ HUD

Описание содержит специфические детали лора о NPZ и NT, которые лучше вынести в файлы локализации. Рекомендуется использовать более общее описание в прототипе.

Предлагаю изменить описание на более общее:

  id: LPPNPZEyehud
  name: NPZ HUD
- description: The HUD of the NPZ, with built-in additional elements such as protection from bright flashes, as well as, in connection with the latest cooperation measures between the NPZ and NT, the built-in database of NanoTraisen to identify ranks and criminals at the corporation's facilities.
+ description: Advanced HUD system with integrated flash protection and security identification features.
🧰 Tools
🪛 yamllint

[error] 81-81: wrong indentation: expected 6 but found 4

(indentation)

Resources/Prototypes/Catalog/Fills/Lockers/engineer.yml (1)

Line range hint 14-19: Оптимизируйте дублирование CableApcStack

Вместо трех отдельных записей CableApcStack с вероятностью 0.3, лучше использовать одну запись с параметром amount.

-      - id: CableApcStack
-        prob: 0.3
-      - id: CableApcStack
-        prob: 0.3
-      - id: CableApcStack
-        prob: 0.3
+      - id: CableApcStack
+        prob: 0.3
+        amount: 3
🧰 Tools
🪛 yamllint

[warning] 96-96: too few spaces before comment

(comments)

Resources/Prototypes/Catalog/Fills/Lockers/suit_storage.yml (1)

Line range hint 279-293: Требуется удаление русского комментария

В прототипе SuitStorageParamedic обнаружен комментарий на русском языке "Изменение от LPP". Согласно правилам кодирования, в прототипах запрещены любые переводы на русский язык.

Примените следующие изменения:

- - id: LPPHardsuitAssaultParamedic #Изменение от LPP
+ - id: LPPHardsuitAssaultParamedic #LPP modification
🧰 Tools
🪛 yamllint

[error] 125-125: wrong indentation: expected 6 but found 2

(indentation)

Resources/Prototypes/Catalog/Fills/Lockers/security.yml (3)

Line range hint 89-90: Необходимо удалить русскоязычный комментарий

Согласно правилам, в прототипах запрещены комментарии на русском языке. Пожалуйста, удалите или переведите комментарий "Кибербуллинг метаюзеров :D".

      - id: LPPMagazinePistolPF20Rubber
-        prob: 0.5 #Кибербуллинг метаюзеров :D
+        prob: 0.5
🧰 Tools
🪛 yamllint

[warning] 171-171: comment not indented like content

(comments-indentation)


[warning] 174-174: too few spaces before comment

(comments)


Line range hint 35-36: Необходимо удалить русскоязычный комментарий

Согласно правилам, в прототипах запрещены комментарии на русском языке. Пожалуйста, удалите или переведите комментарий "На будущее, будет хайриском".

      - id: LPPHOSUplinkShop
-      #На будущее, будет хайриском
🧰 Tools
🪛 yamllint

[warning] 171-171: comment not indented like content

(comments-indentation)


[warning] 174-174: too few spaces before comment

(comments)


Обнаружены зависимости LockerWardenFilledHardsuit в файлах карт

Найдены следующие файлы, требующие обновления после удаления LockerWardenFilledHardsuit:

  • Resources/Maps/asterisk.yml
  • Resources/Maps/edge.yml
  • Resources/Maps/arena.yml
  • Resources/Maps/pebble.yml
  • Resources/Maps/saltern.yml
  • Resources/Maps/lighthouse.yml
  • Resources/Maps/tortuga.yml
  • Resources/Maps/gaxstation.yml
  • Resources/Maps/hive.yml
  • Resources/Maps/hammurabi.yml
  • Resources/Maps/_LostParadise/LPPPebble.yml
  • Resources/Maps/_LostParadise/LPPNormandy.yml
  • Resources/Maps/_LostParadise/LPPAsterisk.yml

Необходимо заменить все упоминания LockerWardenFilledHardsuit в этих файлах на соответствующую альтернативу.

🔗 Analysis chain

Line range hint 1-37: Проверьте влияние удаления LockerWardenFilledHardsuit

Убедитесь, что все ссылки на LockerWardenFilledHardsuit были обновлены в других файлах.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Search for references to the removed entity
rg "LockerWardenFilledHardsuit"

Length of output: 1256

🧰 Tools
🪛 yamllint

[warning] 171-171: comment not indented like content

(comments-indentation)


[warning] 174-174: too few spaces before comment

(comments)

Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (1)

Line range hint 1-400: Уберите русский текст из комментариев

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

🧰 Tools
🪛 yamllint

[warning] 181-181: too few spaces before comment

(comments)


[warning] 186-186: too few spaces before comment

(comments)


[warning] 187-187: too few spaces before comment

(comments)

Resources/Prototypes/_LostParadise/Entities/Clothing/OuterClothing/hardsuits.yml (1)

Line range hint 16-17: Путь к спрайту должен быть в директории _LostParadise

Текущий путь к спрайту Clothing/OuterClothing/Hardsuits/SecEng.rsi должен находиться в директории _LostParadise:

-    sprite: Clothing/OuterClothing/Hardsuits/SecEng.rsi
+    sprite: _LostParadise/Clothing/OuterClothing/Hardsuits/SecEng.rsi
🧰 Tools
🪛 yamllint

[warning] 227-227: too few spaces before comment

(comments)


[warning] 228-228: too few spaces before comment

(comments)

📜 Review details

Configuration used: CodeRabbit UI
Review profile: ASSERTIVE

📥 Commits

Files that changed from the base of the PR and between 54a328d and 826064f.

📒 Files selected for processing (19)
  • Content.Server/Supermatter/Systems/SupermatterSystem.Processing.cs (3 hunks)
  • Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/devices/pda.ftl (1 hunks)
  • Resources/Locale/ru-RU/ss14-ru/prototypes/deltav/entities/clothing/ears/headsets.ftl (1 hunks)
  • Resources/Locale/ru-RU/ss14-ru/prototypes/entities/structures/power/generation/supermatter/supermatter.ftl (1 hunks)
  • Resources/Prototypes/Catalog/Fills/Lockers/engineer.yml (1 hunks)
  • Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (2 hunks)
  • Resources/Prototypes/Catalog/Fills/Lockers/security.yml (2 hunks)
  • Resources/Prototypes/Catalog/Fills/Lockers/suit_storage.yml (1 hunks)
  • Resources/Prototypes/DeltaV/Entities/Clothing/Ears/headsets.yml (2 hunks)
  • Resources/Prototypes/Entities/Clothing/Eyes/glasses.yml (1 hunks)
  • Resources/Prototypes/Entities/Clothing/Head/hardsuit-helmets.yml (4 hunks)
  • Resources/Prototypes/Entities/Clothing/OuterClothing/hardsuits.yml (4 hunks)
  • Resources/Prototypes/Entities/Objects/Devices/pda.yml (6 hunks)
  • Resources/Prototypes/Traits/physical.yml (2 hunks)
  • Resources/Prototypes/_LostParadise/CharacterItemGroups/bsGroups.yml (1 hunks)
  • Resources/Prototypes/_LostParadise/Entities/Clothing/Eyes/Hud/hud.yml (1 hunks)
  • Resources/Prototypes/_LostParadise/Entities/Clothing/OuterClothing/hardsuits.yml (1 hunks)
  • Resources/Prototypes/_LostParadise/Entities/Objects/Devices/pda.yml (3 hunks)
  • Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml (4 hunks)
🧰 Additional context used
📓 Path-based instructions (18)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/devices/pda.ftl (1)

Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }

А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст

Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix

Resources/Locale/ru-RU/ss14-ru/prototypes/deltav/entities/clothing/ears/headsets.ftl (1)

Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }

А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст

Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix

Resources/Locale/ru-RU/ss14-ru/prototypes/entities/structures/power/generation/supermatter/supermatter.ftl (1)

Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }

А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст

Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix

Resources/Prototypes/Catalog/Fills/Lockers/engineer.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Catalog/Fills/Lockers/security.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Catalog/Fills/Lockers/suit_storage.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/DeltaV/Entities/Clothing/Ears/headsets.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Entities/Clothing/Eyes/glasses.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Entities/Clothing/Head/hardsuit-helmets.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Entities/Clothing/OuterClothing/hardsuits.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Entities/Objects/Devices/pda.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Traits/physical.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/_LostParadise/CharacterItemGroups/bsGroups.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/_LostParadise/Entities/Clothing/Eyes/Hud/hud.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/_LostParadise/Entities/Clothing/OuterClothing/hardsuits.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/_LostParadise/Entities/Objects/Devices/pda.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

🪛 yamllint
Resources/Prototypes/DeltaV/Entities/Clothing/Ears/headsets.yml

[error] 10-10: wrong indentation: expected 8 but found 6

(indentation)


[error] 20-20: wrong indentation: expected 8 but found 6

(indentation)

Resources/Prototypes/Entities/Clothing/Head/hardsuit-helmets.yml

[error] 601-601: trailing spaces

(trailing-spaces)

Resources/Prototypes/Entities/Clothing/OuterClothing/hardsuits.yml

[warning] 70-70: too few spaces before comment

(comments)


[warning] 71-71: too few spaces before comment

(comments)


[warning] 128-128: too few spaces before comment

(comments)


[warning] 129-129: too few spaces before comment

(comments)


[warning] 437-437: too few spaces before comment

(comments)


[warning] 438-438: too few spaces before comment

(comments)


[warning] 954-954: too few spaces before comment

(comments)


[warning] 955-955: too few spaces before comment

(comments)

Resources/Prototypes/Entities/Objects/Devices/pda.yml

[error] 125-125: wrong indentation: expected 4 but found 2

(indentation)

Resources/Prototypes/_LostParadise/Entities/Clothing/OuterClothing/hardsuits.yml

[warning] 227-227: too few spaces before comment

(comments)


[warning] 228-228: too few spaces before comment

(comments)

Resources/Prototypes/_LostParadise/Entities/Objects/Devices/pda.yml

[error] 86-86: no new line character at the end of file

(new-line-at-end-of-file)

Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml

[error] 18-18: trailing spaces

(trailing-spaces)


[error] 30-30: trailing spaces

(trailing-spaces)

🔇 Additional comments (37)
Resources/Locale/ru-RU/ss14-ru/prototypes/entities/structures/power/generation/supermatter/supermatter.ftl (1)

1-2: Правильное исправление орфографии!

Исправление написания слова "Кристалл" (с двумя "л") соответствует правилам русского языка. Форматирование локализации также соответствует всем требованиям:

  • Начинается с заглавной буквы после "="
  • Описание корректно оформлено как под-свойство
Resources/Prototypes/_LostParadise/CharacterItemGroups/bsGroups.yml (2)

1-8: Группа LoadoutWeaponBS корректно определена.

Структура и определение группы оружия соответствует требованиям. Идентификаторы следуют принятым соглашениям об именовании.


9-15: Группа LoadoutMagazineBS корректно определена.

Структура и определение группы магазинов соответствует требованиям. Идентификаторы следуют принятым соглашениям об именовании.

✅ Verification successful

Группа LoadoutMagazineBS полностью верифицирована

Проверка показала, что:

  • Все прототипы магазинов (MagazineMagnum и MagazineUniversalMagnum) корректно определены в базовых прототипах игры
  • Группа LoadoutMagazineBS правильно используется в loadout'ах офицера BlueShield
  • Связи между группой магазинов, прототипами оружия и требованиями к профессии настроены корректно
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Проверка существования связанных прототипов оружия и магазинов

# Поиск определений прототипов оружия и магазинов
rg -l "id: LoadoutCommandBSWeaponPistolN1984|id: LoadoutCommandBSWeaponPistolUniversal|id: LoadoutCommandBSMagazineUniversalMagnum|id: LoadoutCommandBSMagazineMagnum" Resources/Prototypes/

Length of output: 322


Script:

#!/bin/bash
# Проверим содержимое файлов для анализа связей между прототипами
echo "=== Содержимое bsGroups.yml ==="
cat Resources/Prototypes/_LostParadise/CharacterItemGroups/bsGroups.yml

echo -e "\n=== Содержимое blueshield.yml ==="
cat Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml

Length of output: 2578


Script:

#!/bin/bash
# Проверим существование предметов, на которые ссылаются loadout'ы
echo "=== Поиск определений предметов оружия и магазинов ==="
rg "id: (WeaponPistolN1984|WeaponPistolUniversal|MagazineUniversalMagnum|MagazineMagnum)" Resources/Prototypes/

# Проверим использование группы LoadoutMagazineBS в других местах
echo -e "\n=== Поиск использования группы LoadoutMagazineBS ==="
rg "LoadoutMagazineBS" Resources/Prototypes/ -C 2

Length of output: 5196

Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/devices/pda.ftl (2)

7-9: Изменения корректны

Остальные переводы соответствуют правилам локализации и имеют правильное форматирование.


5-5: ⚠️ Potential issue

Исправьте заглавную букву после знака равенства

Согласно правилам локализации, все переводы, начинающиеся с 'ent-', должны начинаться с заглавной буквы после знака равенства.

Примените следующие изменения:

-ent-LPPAdjutantPDA = КПК адъютанта
+ent-LPPAdjutantPDA = КПК Адъютанта

Likely invalid or redundant comment.

Resources/Locale/ru-RU/ss14-ru/prototypes/deltav/entities/clothing/ears/headsets.ftl (1)

7-8: 🧹 Nitpick (assertive)

Проверьте написание с заглавной буквы

Согласно правилам локализации, все переводы, начинающиеся с "ent-", должны начинаться с заглавной буквы после знака равенства.

-ent-ClothingHeadsetJustice = Гарнитура юридического отдела
+ent-ClothingHeadsetJustice = Гарнитура Юридического отдела
Resources/Prototypes/_LostParadise/Entities/Objects/Devices/pda.yml (3)

Line range hint 1-20: Корректное обновление PDA инженера службы безопасности!

Изменение родительского класса на BaseEngineerPDA и добавление CartridgeLoader с необходимыми картриджами соответствует целям PR по улучшению инженерного отдела.


80-86: Правильное добавление картриджей для PDA пилота службы безопасности!

Добавление CartridgeLoader с необходимыми картриджами безопасности соответствует целям PR по исправлению устройств связи для пилота службы безопасности.

🧰 Tools
🪛 yamllint

[error] 86-86: no new line character at the end of file

(new-line-at-end-of-file)


86-86: ⚠️ Potential issue

Добавьте пустую строку в конец файла

Для соответствия стандартам YAML необходимо добавить символ новой строки в конец файла.

Примените следующее исправление:

      - SecWatchCartridge
+

Likely invalid or redundant comment.

🧰 Tools
🪛 yamllint

[error] 86-86: no new line character at the end of file

(new-line-at-end-of-file)

Resources/Prototypes/_LostParadise/Loadouts/Jobs/blueshield.yml (3)

42-43: Добавление групповых требований для оружия и магазинов выглядит логичным.

Использование CharacterItemGroupRequirement с группами LoadoutWeaponBS и LoadoutMagazineBS улучшает организацию и управление доступными предметами.

Also applies to: 55-56, 68-69, 81-82


25-36: 🧹 Nitpick (assertive)

Конфигурация загрузки боевого гипоспрея выглядит корректно.

Структура и параметры соответствуют требованиям. Обратите внимание на пробел в конце строки 30.

  requirements: 
-    - !type:CharacterJobRequirement
+  - !type:CharacterJobRequirement

Likely invalid or redundant comment.

🧰 Tools
🪛 yamllint

[error] 30-30: trailing spaces

(trailing-spaces)


13-24: 🧹 Nitpick (assertive)

Конфигурация загрузки боевой аптечки выглядит корректно.

Структура и параметры соответствуют требованиям. Обратите внимание на пробел в конце строки 18.

  requirements: 
-    - !type:CharacterJobRequirement
+  - !type:CharacterJobRequirement

Likely invalid or redundant comment.

🧰 Tools
🪛 yamllint

[error] 18-18: trailing spaces

(trailing-spaces)

Resources/Prototypes/_LostParadise/Entities/Clothing/Eyes/Hud/hud.yml (3)

Line range hint 1-6: Базовая сущность реализована корректно!

Абстрактная сущность определена правильно и содержит только необходимый компонент.

🧰 Tools
🪛 yamllint

[error] 81-81: wrong indentation: expected 6 but found 4

(indentation)


Line range hint 53-67: Реализация химических очков корректна!

Все компоненты определены правильно, описание на английском языке соответствует требованиям.

🧰 Tools
🪛 yamllint

[error] 81-81: wrong indentation: expected 6 but found 4

(indentation)


Line range hint 69-81: Реализация HUD инженера безопасности корректна!

Компоненты определены правильно, включая теги безопасности и защиту от вспышек.

🧰 Tools
🪛 yamllint

[error] 81-81: wrong indentation: expected 6 but found 4

(indentation)

Resources/Prototypes/DeltaV/Entities/Clothing/Ears/headsets.yml (3)

10-10: Подтверждаю изменение ключей шифрования для HoP!

Изменение соответствует целям PR по восстановлению всех ключей шифрования для Head of Personnel через единый EncryptionKeyStationMaster.

🧰 Tools
🪛 yamllint

[error] 10-10: wrong indentation: expected 8 but found 6

(indentation)


20-20: Подтверждаю изменение ключей шифрования для сервисного наушника!

Унификация с наушником HoP логична, так как это альтернативная версия того же устройства.

🧰 Tools
🪛 yamllint

[error] 20-20: wrong indentation: expected 8 but found 6

(indentation)


Line range hint 31-35: Проверьте безопасность добавления EncryptionKeyStationMaster к синдикатскому наушнику!

Добавление мастер-ключа к устройству прослушивания синдиката может иметь серьезные последствия для игрового баланса. Прошу подтвердить, что это намеренное изменение.

✅ Verification successful

Обнаружено намеренное добавление мастер-ключа к наушнику синдиката

Анализ кода показывает, что это намеренное изменение - наушник ClothingHeadsetAltSyndicateListening специально спроектирован как устройство прослушивания с двумя слотами для ключей шифрования:

  • EncryptionKeySyndie
  • EncryptionKeyStationMaster

Это соответствует его описанию как "продвинутого модульного перехватывающего устройства синдиката" и его назначению для прослушивания каналов связи станции.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Поиск других прототипов синдиката с доступом к EncryptionKeyStationMaster
rg -l "EncryptionKeyStationMaster.*EncryptionKeySyndie|EncryptionKeySyndie.*EncryptionKeyStationMaster" Resources/Prototypes

Length of output: 126


Script:

#!/bin/bash
# Проверяем текущее содержимое файла
cat Resources/Prototypes/DeltaV/Entities/Clothing/Ears/headsets.yml

# Поиск всех упоминаний EncryptionKeyStationMaster
rg "EncryptionKeyStationMaster" -A 5 -B 5

# Поиск всех упоминаний EncryptionKeySyndie
rg "EncryptionKeySyndie" -A 5 -B 5

Length of output: 35004

🧰 Tools
🪛 yamllint

[error] 7-7: wrong indentation: expected at least 3

(indentation)


[error] 10-10: wrong indentation: expected 8 but found 6

(indentation)

Resources/Prototypes/Catalog/Fills/Lockers/engineer.yml (2)

91-93: Добавление нового снаряжения выглядит логичным!

Магнитные ботинки и боеприпасы для RCD являются полезным дополнением к снаряжению атмосферного техника.


Line range hint 1-180: Структура файла соответствует стандартам!

  • Правильное именование сущностей с использованием суффиксов
  • Согласованное распределение предметов между связанными шкафчиками
  • Комментарии на английском языке
🧰 Tools
🪛 yamllint

[warning] 96-96: too few spaces before comment

(comments)

Resources/Prototypes/Entities/Clothing/Eyes/glasses.yml (1)

65-68: Корректное добавление защитных свойств к инженерным очкам!

Добавление компонентов FlashImmunity и EyeProtection полностью соответствует целям PR по улучшению защитных свойств инженерного снаряжения. Реализация согласуется с другими защитными очками в файле.

Resources/Prototypes/Catalog/Fills/Lockers/suit_storage.yml (2)

128-128: LGTM: Добавление магнитных ботинок

Добавление магнитных ботинок в комплект атмосферного техника логично и соответствует другим инженерным костюмам.


Line range hint 137-141: LGTM: Обновление боевых скафандров

Замена стандартных скафандров на боевые варианты для офицеров безопасности, главы службы безопасности и смотрителя соответствует целям PR по улучшению различных ролей.

Also applies to: 199-203, 214-218

🧰 Tools
🪛 yamllint

[error] 125-125: wrong indentation: expected 6 but found 2

(indentation)

Resources/Prototypes/Catalog/Fills/Lockers/security.yml (2)

Line range hint 173-209: Изменения для Corpsman выглядят корректно

Структура изменений логична и хорошо документирована. Добавление LPPCombatHypospray соответствует целям PR по улучшению роли.

🧰 Tools
🪛 yamllint

[warning] 208-208: too few spaces before comment

(comments)


Line range hint 284-385: Структура оружейных сейфов реализована корректно

Все новые сущности GunSafe имеют правильную структуру и последовательное определение содержимого. Количество оружия и боеприпасов сбалансировано для каждого типа.

🧰 Tools
🪛 yamllint

[warning] 171-171: comment not indented like content

(comments-indentation)


[warning] 174-174: too few spaces before comment

(comments)

Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (1)

184-185: Проверьте расположение новых прототипов

Новые предметы LPPRCDFAPDevice и LPPRCDFAPAmmo должны находиться в нашей папке прототипов. Пожалуйста, убедитесь, что они расположены правильно.

Also applies to: 211-212

✅ Verification successful

Прототипы находятся в правильном месте

Прототипы LPPRCDFAPDevice и LPPRCDFAPAmmo корректно определены в папке Resources/Prototypes/_LostParadise/. Они являются частью модификации Lost Paradise и следуют установленной структуре проекта, что подтверждается наличием других файлов с префиксом LPP в соответствующей директории.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Check if the new prototypes are in the correct location
fd -g "*LPPRCDFAPDevice*" -g "*LPPRCDFAPAmmo*"

Length of output: 151


Script:

#!/bin/bash
# Search for files containing LPPRCDFAPDevice or LPPRCDFAPAmmo
rg -l "LPPRCDFAPDevice|LPPRCDFAPAmmo" --type yaml

# Also search for any files with LPP prefix to understand the pattern
fd "LPP.*\.yml$"

Length of output: 2202

Resources/Prototypes/Traits/physical.yml (1)

417-417: Изменения баланса BionicArm подтверждены

Уменьшение стоимости с -9 до -4 очков и ограничение функции взлома только для обесточенных дверей корректно отражает заявленный нерф бионической руки.

Для проверки корректности изменений выполните следующий скрипт:

Also applies to: 426-426

✅ Verification successful

Изменение баланса BionicArm подтверждено и обосновано

Анализ показывает, что изменение параметра pryPowered: false для бионической руки соответствует балансу игры:

  • Большинство стандартных инструментов (например, обычный лом) не имеют возможности взламывать работающие двери
  • Только специальные инструменты (jaws of life) и особые сущности (терминатор, ксеноморф) имеют pryPowered: true
  • Снижение стоимости с -9 до -4 очков компенсирует ограничение функциональности
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Описание: Проверка использования компонента Prying в других прототипах
# Ожидаем: Найти другие прототипы с похожей функциональностью для сравнения баланса

rg -l "type: Prying" --type yaml

Length of output: 890


Script:

#!/bin/bash
# Проверим содержимое файлов с компонентом Prying для сравнения параметров
rg "type: Prying" -A 5 -B 2 --type yaml

Length of output: 9005

Content.Server/Supermatter/Systems/SupermatterSystem.Processing.cs (2)

9-11: Корректное добавление зависимостей для радио-функционала!

Добавление RadioSystem и IPrototypeManager логично дополняет функционал системы для отправки инженерных оповещений.

Also applies to: 17-18


324-328: Отличная интеграция радио-оповещений!

Добавление радио-канала для инженеров логично дополняет существующую систему оповещений, сохраняя обратную совместимость.

Resources/Prototypes/_LostParadise/Entities/Clothing/OuterClothing/hardsuits.yml (2)

227-228: ⚠️ Potential issue

Исправьте форматирование комментариев

Добавьте пробел перед комментариями в определении состава воздуха:

-        - 0.307706858 # oxygen
-        - 0.307706858 # nitrogen
+        - 0.307706858  # oxygen
+        - 0.307706858  # nitrogen

Likely invalid or redundant comment.

🧰 Tools
🪛 yamllint

[warning] 227-227: too few spaces before comment

(comments)


[warning] 228-228: too few spaces before comment

(comments)


221-242: Проверьте безопасность компонента Explosive

Компонент Explosive с maxIntensity: 20 может быть опасным. Убедитесь, что:

  1. Взрыв активируется только при критических повреждениях костюма
  2. Интенсивность взрыва соответствует игровому балансу
  3. Существует механизм предупреждения игрока о критическом состоянии костюма
✅ Verification successful

Интенсивность взрыва костюма соответствует игровому балансу

Анализ компонента Explosive в кодовой базе показывает, что:

  1. Значение maxIntensity: 20 для скафандра соответствует стандартным значениям для подобного снаряжения (такое же значение у других скафандров и газовых баллонов)
  2. Это значительно меньше, чем у боевых взрывных устройств (гранаты: 200-20000, C4: 60-360)
  3. Находится на уровне базовых устройств (микроволновка: 10, турели: 10)
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Поиск других сущностей с компонентом Explosive для сравнения интенсивности
ast-grep --pattern 'type: Explosive
  $$$
  maxIntensity: $_'

Length of output: 63


Script:

#!/bin/bash
# Search for Explosive components in yaml files
rg "type: Explosive" -A 3 --type yaml

# Also search for any documentation about explosions
fd "explosion|explosive" --type md

Length of output: 22824

🧰 Tools
🪛 yamllint

[warning] 227-227: too few spaces before comment

(comments)


[warning] 228-228: too few spaces before comment

(comments)

Resources/Prototypes/Entities/Clothing/Head/hardsuit-helmets.yml (2)

43-44: Улучшение защиты инженерных шлемов одобрено!

Добавление компонентов FlashImmunity и EyeProtection логично дополняет функциональность инженерных шлемов, обеспечивая защиту от яркого света при сварочных работах.

Also applies to: 63-64, 256-257, 599-601


601-601: ⚠️ Potential issue

Необходимо удалить пробелы в конце строки

Примените следующее исправление:

-  
+

Likely invalid or redundant comment.

🧰 Tools
🪛 yamllint

[error] 601-601: trailing spaces

(trailing-spaces)

Resources/Prototypes/Entities/Objects/Devices/pda.yml (3)

102-103: Добавление интерфейса газоанализатора в BasePDA выглядит корректно!

Интерфейс GasAnalyzerBoundUserInterface добавлен в базовый PDA, что позволяет всем наследникам использовать функционал газоанализатора при необходимости.


120-126: Создание базового инженерного PDA реализовано правильно!

Новая сущность BaseEngineerPDA наследуется от BasePDA и включает компонент GasAnalyzer, что логично для инженерного оборудования.

🧰 Tools
🪛 yamllint

[error] 125-125: wrong indentation: expected 4 but found 2

(indentation)


140-140: Корректное изменение иерархии инженерных PDA!

Следующие PDA теперь правильно наследуются от BaseEngineerPDA:

  • TechnicalAssistantPDA
  • CEPDA
  • EngineerPDA
  • AtmosPDA

Это изменение улучшает организацию кода и обеспечивает всех инженеров доступом к газоанализатору.

Also applies to: 529-529, 544-544, 842-842

Resources/Prototypes/Entities/Clothing/OuterClothing/hardsuits.yml (1)

64-85: Проверьте конфигурацию газовых баллонов для баланса.

Реализация газовых баллонов и реактивных ранцев добавлена к инженерным скафандрам, что соответствует целям PR. Однако есть несколько моментов для рассмотрения:

  1. Объем газа различается между обычным инженерным скафандром (1.5) и скафандром главного инженера (3.0).
  2. Все остальные параметры (давление, расход топлива) идентичны.

Запустите следующий скрипт для проверки баланса:

Also applies to: 122-143, 431-452, 948-969

✅ Verification successful

Конфигурация газовых баллонов соответствует игровому балансу

Анализ показывает, что конфигурации газовых баллонов в скафандрах соответствуют существующим стандартам:

  • Объем 1.5 для базовых скафандров (4 минуты работы) соответствует Extended Emergency Tank
  • Объем 3.0 для улучшенных скафандров (8 минут работы) логично увеличен в 2 раза
  • Давление на выходе 42.6 и расход топлива 0.00085 соответствуют стандартным значениям для реактивных ранцев

Разница в объемах между базовыми и улучшенными скафандрами обоснована их назначением и соответствует общей системе баланса, где более продвинутое снаряжение имеет увеличенный запас воздуха.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Поиск других прототипов с компонентом GasTank для сравнения конфигурации
rg -l "type: GasTank" | while read -r file; do
  echo "=== $file ==="
  rg -A 10 "type: GasTank" "$file"
done

Length of output: 13573

🧰 Tools
🪛 yamllint

[warning] 70-70: too few spaces before comment

(comments)


[warning] 71-71: too few spaces before comment

(comments)

@BL02DL
Copy link
Collaborator

BL02DL commented Oct 29, 2024

@Farrellka-dev Уберай джетпак в скафах инжей

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI
Review profile: ASSERTIVE

📥 Commits

Files that changed from the base of the PR and between 6e26c5b and 4ca876d.

📒 Files selected for processing (1)
  • Content.Server/Supermatter/Systems/SupermatterSystem.Processing.cs (3 hunks)
🔇 Additional comments (2)
Content.Server/Supermatter/Systems/SupermatterSystem.Processing.cs (2)

9-11: Корректное добавление зависимостей!

Зависимости правильно внедрены и необходимы для реализации радио-функционала.

Also applies to: 17-18


301-310: Необходимо улучшить обработку ошибок в методе Report!

Предыдущий комментарий по улучшению обработки ошибок всё ещё актуален.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 5

📜 Review details

Configuration used: CodeRabbit UI
Review profile: ASSERTIVE

📥 Commits

Reviewing files that changed from the base of the PR and between 224316a and 0d91f78.

📒 Files selected for processing (8)
  • Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/misc/papers/pen.ftl (1 hunks)
  • Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (13 hunks)
  • Resources/Prototypes/Entities/Objects/Devices/pda.yml (8 hunks)
  • Resources/Prototypes/Loadouts/Jobs/Heads/chiefEngineer.yml (1 hunks)
  • Resources/Prototypes/Loadouts/Jobs/engineering.yml (5 hunks)
  • Resources/Prototypes/Loadouts/backpacks.yml (1 hunks)
  • Resources/Prototypes/Loadouts/duffelbags.yml (1 hunks)
  • Resources/Prototypes/Loadouts/satchels.yml (1 hunks)
🧰 Additional context used
📓 Path-based instructions (8)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/misc/papers/pen.ftl (1)

Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }

А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст

Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix

Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Entities/Objects/Devices/pda.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Loadouts/Jobs/Heads/chiefEngineer.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Loadouts/Jobs/engineering.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Loadouts/backpacks.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Loadouts/duffelbags.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

Resources/Prototypes/Loadouts/satchels.yml (1)

Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.

Все добавляемые прототипы должны быть в нашей папке.

Никогда не предлагать ничего по отступам

При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.

🪛 yamllint
Resources/Prototypes/Catalog/Fills/Lockers/heads.yml

[warning] 26-26: too few spaces before comment

(comments)


[warning] 26-26: missing starting space in comment

(comments)


[warning] 60-60: too few spaces before comment

(comments)


[warning] 60-60: missing starting space in comment

(comments)


[warning] 94-94: too few spaces before comment

(comments)


[warning] 94-94: missing starting space in comment

(comments)


[warning] 127-127: too few spaces before comment

(comments)


[warning] 127-127: missing starting space in comment

(comments)


[warning] 163-163: too few spaces before comment

(comments)


[warning] 163-163: missing starting space in comment

(comments)


[warning] 191-191: too few spaces before comment

(comments)


[warning] 192-192: too few spaces before comment

(comments)


[warning] 194-194: too few spaces before comment

(comments)


[warning] 194-194: missing starting space in comment

(comments)


[warning] 219-219: too few spaces before comment

(comments)


[warning] 221-221: too few spaces before comment

(comments)


[warning] 221-221: missing starting space in comment

(comments)


[warning] 250-250: too few spaces before comment

(comments)


[warning] 250-250: missing starting space in comment

(comments)


[warning] 275-275: too few spaces before comment

(comments)


[warning] 275-275: missing starting space in comment

(comments)


[warning] 299-299: too few spaces before comment

(comments)


[warning] 299-299: missing starting space in comment

(comments)


[warning] 322-322: too few spaces before comment

(comments)


[warning] 322-322: missing starting space in comment

(comments)

Resources/Prototypes/Entities/Objects/Devices/pda.yml

[error] 125-125: wrong indentation: expected 4 but found 2

(indentation)


[error] 370-370: wrong indentation: expected 10 but found 8

(indentation)


[error] 540-540: wrong indentation: expected 4 but found 2

(indentation)


[error] 548-548: wrong indentation: expected 10 but found 8

(indentation)


[error] 672-672: wrong indentation: expected 10 but found 8

(indentation)


[error] 736-736: wrong indentation: expected 10 but found 8

(indentation)

Resources/Prototypes/Loadouts/Jobs/Heads/chiefEngineer.yml

[error] 106-106: no new line character at the end of file

(new-line-at-end-of-file)

Resources/Prototypes/Loadouts/backpacks.yml

[error] 296-296: no new line character at the end of file

(new-line-at-end-of-file)

Resources/Prototypes/Loadouts/satchels.yml

[error] 275-275: no new line character at the end of file

(new-line-at-end-of-file)

🔇 Additional comments (15)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/misc/papers/pen.ftl (1)

5-6: Улучшение перевода названия должности!

Изменение корректно. Использование слитного написания "Квартирмейстер" вместо раздельного "Квартир Мейстер" соответствует правильному написанию этой должности в русском языке.

Resources/Prototypes/Loadouts/Jobs/Heads/chiefEngineer.yml (1)

106-106: ⚠️ Potential issue

Отсутствует символ новой строки в конце файла.

Добавьте пустую строку в конец файла для соответствия стандартам YAML.

Примените следующие изменения:

     - ClothingUniformJumpsuitChiefEngineerTurtle
+

Likely invalid or redundant comment.

🧰 Tools
🪛 yamllint

[error] 106-106: no new line character at the end of file

(new-line-at-end-of-file)

Resources/Prototypes/Loadouts/Jobs/engineering.yml (3)

Line range hint 1-52: Новые комплекты экипировки выглядят корректно!

Добавление защитных очков (meson glasses) для инженеров и атмосферных техников является важным дополнением к системе безопасности. Костюм цыплёнка добавляет немного веселья в игру, что тоже хорошо.


87-87: Расширение доступа к каскам для Главного Инженера логично!

Добавление должности ChiefEngineer к существующим каскам разных цветов обеспечивает правильную иерархию доступа к защитному снаряжению.

Also applies to: 103-103, 119-119, 149-149


133-133: Ограничение белой каски только для Главного Инженера - отличное решение!

Эксклюзивный доступ к белой каске для ChiefEngineer создаёт чёткую визуальную иерархию и помогает быстро идентифицировать руководителя инженерного отдела.

Resources/Prototypes/Loadouts/duffelbags.yml (1)

250-262: Проверьте расположение прототипа.

Структура и содержание нового прототипа корректны и соответствуют существующим конвенциям. Однако необходимо убедиться, что файл находится в правильной папке согласно требованиям.

Resources/Prototypes/Loadouts/satchels.yml (1)

263-275: Конфигурация загрузки для Главного Инженера корректна.

Структура нового загрузочного набора соответствует существующим конфигурациям и правильно определяет требования для роли Главного Инженера.

🧰 Tools
🪛 yamllint

[error] 275-275: no new line character at the end of file

(new-line-at-end-of-file)

Resources/Prototypes/Loadouts/backpacks.yml (1)

284-296: Структура конфигурации загрузки корректна!

Новая конфигурация загрузки для сумки-планшета главного инженера следует установленному шаблону и правильно определяет все необходимые параметры.

🧰 Tools
🪛 yamllint

[error] 296-296: no new line character at the end of file

(new-line-at-end-of-file)

Resources/Prototypes/Catalog/Fills/Lockers/heads.yml (3)

Line range hint 1-400: Проверьте расположение прототипов

Согласно руководству по кодированию, все новые прототипы должны находиться в "нашей" папке. Убедитесь, что все добавленные прототипы (LPPPanties, RCDFAP) расположены правильно.

🧰 Tools
🪛 yamllint

[warning] 186-186: too few spaces before comment

(comments)


[warning] 191-191: too few spaces before comment

(comments)


[warning] 192-192: too few spaces before comment

(comments)


[warning] 194-194: too few spaces before comment

(comments)


[warning] 194-194: missing starting space in comment

(comments)


26-26: Обнаружены повторяющиеся предметы нижнего белья во всех шкафчиках глав

Добавление предметов LPPPanties во все шкафчики глав кажется избыточным и может не соответствовать игровому балансу. Рекомендуется пересмотреть необходимость этих предметов или ограничить их распространение.

Also applies to: 60-60, 94-94, 127-127, 163-163, 194-194, 221-221, 250-250, 275-275, 299-299, 322-322, 356-356, 388-388

🧰 Tools
🪛 yamllint

[warning] 26-26: too few spaces before comment

(comments)


[warning] 26-26: missing starting space in comment

(comments)


189-190: Проверьте баланс новых устройств RCDFAP

Добавление LPPRCDFAPDevice и LPPRCDFAPAmmo в шкафчики CE может значительно повлиять на игровой баланс. Убедитесь, что эти предметы правильно сбалансированы.

Also applies to: 217-218

✅ Verification successful

RCDFAP сбалансирован и готов к использованию

На основе анализа прототипов и рецептов:

  • Устройство имеет ограничение в 130 зарядов
  • Стоимость создания сбалансирована (1000 стали + 300 пластика для устройства, 500 стали + 250 пластика для боеприпасов)
  • Функционал ограничен созданием атмосферных труб и устройств
  • Доступно через исследования вместе с обычным RCD

Размещение в шкафчиках CE является обоснованным, так как устройство предназначено для инженерных работ с атмосферными системами.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Поиск прототипов RCDFAP для проверки их характеристик
ast-grep --pattern 'id: LPPRCDFAPDevice'
ast-grep --pattern 'id: LPPRCDFAPAmmo'

Length of output: 15263


Script:

#!/bin/bash
# Поиск определения прототипов RCDFAP для проверки их характеристик
rg -A 20 'id: LPPRCDFAPDevice' Resources/Prototypes/_LostParadise/Entities/Objects/Tools/RCDFAP.yml
rg -A 20 'id: LPPRCDFAPAmmo' Resources/Prototypes/_LostParadise/Entities/Objects/Tools/RCDFAP.yml

# Проверка рецептов крафта
rg -A 10 'id: LPPRCDFAPDevice' Resources/Prototypes/_LostParadise/Recipes/Lathes/Industrial.yml
rg -A 10 'id: LPPRCDFAPAmmo' Resources/Prototypes/_LostParadise/Recipes/Lathes/Industrial.yml

# Проверка исследований
rg -A 10 'LPPRCDFAPDevice' Resources/Prototypes/Research/industrial.yml

Length of output: 2771

Resources/Prototypes/Entities/Objects/Devices/pda.yml (4)

102-103: Добавление интерфейса газоанализатора в базовый PDA

Корректное добавление интерфейса газоанализатора в список доступных интерфейсов PDA.


Line range hint 140-152: Правильное наследование инженерных PDA

Корректное переназначение родительского класса для инженерных PDA (TechnicalAssistant, CE, Engineer, Atmos) на BaseEngineerPDA, что обеспечивает им функциональность газоанализатора.

Also applies to: 535-548, 556-566, 866-877


365-370: ⚠️ Potential issue

Исправьте отступы в определении penSlot для QuartermasterPDA

В определении penSlot для QuartermasterPDA используются неправильные отступы.

Примените следующие изменения:

      penSlot:
        startingItem: LPPQMPen
        priority: -1
        whitelist:
-        tags:
-        - Write
+          tags:
+          - Write

Likely invalid or redundant comment.

🧰 Tools
🪛 yamllint

[error] 370-370: wrong indentation: expected 10 but found 8

(indentation)


120-126: ⚠️ Potential issue

Исправьте отступы в определении BaseEngineerPDA

В определении компонента GasAnalyzer используется неправильный отступ (2 пробела вместо 4).

Примените следующие изменения:

  - type: entity
    parent: BasePDA
    id: BaseEngineerPDA
    abstract: true
    components:
-  - type: GasAnalyzer
+    - type: GasAnalyzer

Likely invalid or redundant comment.

🧰 Tools
🪛 yamllint

[error] 125-125: wrong indentation: expected 4 but found 2

(indentation)

Resources/Prototypes/Loadouts/Jobs/Heads/chiefEngineer.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Loadouts/satchels.yml Outdated Show resolved Hide resolved
Resources/Prototypes/Loadouts/backpacks.yml Outdated Show resolved Hide resolved
@Farrellka-dev
Copy link
Contributor Author

Farrellka-dev commented Oct 31, 2024

Тут в тестах ошибка не моя, она гуляет по ПРам по приколу...

@Evgencheg Evgencheg merged commit e881af1 into Lost-Paradise-Project:master Oct 31, 2024
11 checks passed
@Farrellka-dev Farrellka-dev deleted the engibuff branch November 1, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Изменения: C# Изменяет какие-либо файлы .cs Изменения: YML Изменяет какие-либо файлы .yml Изменения: Локализация Изменяет какие-либо файлы .ftl Статус: Требуется одобрение Ожидание разработчиков для принятия решений
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants