Изменение текстур стаков телекристаллов, БС и нормальности, а так же … #1078
Conversation
…фиксы. Изменение текстур кристаллов нормальности и телекристаллов. Добавление стаков БС кристаллам, кристаллам нормальности и телекристаллам.
WalkthroughВ данном pull request внесены изменения в локализацию, прототипы, стеки и метаданные для материалов, кристаллов и телекристаллов. Добавлены новые записи (например, Changes
Sequence Diagram(s)sequenceDiagram
participant User as Пользователь
participant ReactionSystem as СистемаРеакций
participant EntityFactory as ФабрикаСущностей
User->>ReactionSystem: Активация реакции CreateNormalityCrystal
ReactionSystem->>EntityFactory: Запрос создания CrystalNormality1
EntityFactory-->>ReactionSystem: Возврат созданного CrystalNormality1
ReactionSystem-->>User: Подтверждение создания сущности
Poem
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? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
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)
Other keywords and placeholders
CodeRabbit Configuration File (
|
github-actions
bot
added
S: Untriaged
size/M
Changes: Sprites
|
RSI Diff Bot; head commit b169b60 merging into 8576db1 Resources/Textures/Objects/Specific/Syndicate/telecrystal.rsi
Resources/Textures/Backmen/Objects/Materials/materials.rsi
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (11)
Resources/Prototypes/_Backmen/Reagents/Materials/materials.yml (1)
15-15: Отсутствует новая строка в конце файла.Отсутствует символ новой строки в конце файла, что выявлено статическим анализатором YAMLlint.
Добавьте пустую строку в конец файла для соответствия стандартам и предотвращения предупреждений линтера.
🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 15-15: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/_Backmen/Stacks/materials.yml (1)
13-13: Форматирование: символ новой строки
Обнаружено отсутствие символа новой строки в конце файла. Добавьте перенос строки для соответствия стандартам YAML.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 13-13: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/_Backmen/Entities/Objects/Materials/materials.yml (3)
45-45: Форматирование: лишние пробелы
Обнаружены лишние пробельные символы в конце строки (линия 45). Рекомендуется удалить их для соблюдения стандартов кодирования.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 45-45: trailing spaces
(trailing-spaces)
52-52: Проблема с отступами для компонента Stack
Нарушены правила отступов – ожидается 4 пробела, обнаружено 2. Исправьте отступы, чтобы обеспечить корректное форматирование YAML.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 52-52: wrong indentation: expected 4 but found 2
(indentation)
57-57: Неправильный отступ в списке тегов
Отступ для тегов должен составлять 6 пробелов, а найдено только 4. Проверьте и исправьте отступы для корректной структуры YAML.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 57-57: wrong indentation: expected 6 but found 4
(indentation)
Resources/Prototypes/Stacks/Materials/crystals.yml (2)
4-4: Форматирование свойства icon
Обратите внимание на избыточные пробелы после двоеточия в строке 4. Рекомендуется привести форматирование к общему стилю YAML.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 4-4: too many spaces after colon
(colons)
6-6: Новый символ новой строки
Добавьте символ новой строки в конце файла, чтобы удовлетворить требования YAMLlint по наличию переноса строки в конце файла.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 6-6: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/Entities/Objects/Specific/syndicate.yml (2)
11-26: Добавлена поддержка стеков для телекристаллов.Структура компонентов обновлена для поддержки различных состояний спрайтов в зависимости от размера стека. Изменения соответствуют цели PR по введению функциональности стекирования для телекристаллов.
Однако есть проблемы с форматированием YAML:
- Лишние пробелы в конце строк 15, 16 и 26
- Неправильные отступы в строках 13 и 22 (используется 6 пробелов вместо 4)
Рекомендую исправить форматирование для соответствия стандартам проекта:
- # Backmen-Stak-Start - layers: - - state: telecrystal_3 - map: ["base"] - - type: Item - size: Tiny +# Backmen-Stak-Start + layers: + - state: telecrystal_3 + map: ["base"] + - type: Item + size: TinyИ аналогично для других мест с неправильными отступами и пробелами.
🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 13-13: wrong indentation: expected 4 but found 6
(indentation)
[error] 15-15: trailing spaces
(trailing-spaces)
[error] 16-16: trailing spaces
(trailing-spaces)
[warning] 22-22: wrong indentation: expected 4 but found 6
(indentation)
[error] 26-26: trailing spaces
(trailing-spaces)
58-70: Добавлены явные указания stackType для дочерних сущностей телекристаллов.Явное указание типа стека в дочерних сущностях обеспечивает единообразие и предотвращает возможные ошибки при наследовании.
Необходимо удалить лишние пробелы в конце строк 61 и 69:
- count: 1 - + count: 1 +- count: 5 + count: 5🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 61-61: trailing spaces
(trailing-spaces)
[error] 69-69: trailing spaces
(trailing-spaces)
Resources/Prototypes/_Backmen/Entities/Objects/Specific/Research/crystals.yml (2)
2-29: Переработана структура кристалла нормальности для поддержки стеков.Сущность CrystalNormality теперь наследуется от MaterialBase вместо BaseItem, что позволяет использовать функционал материалов и стеков. Добавлены компоненты для поддержки различных размеров стеков и состояний спрайтов.
Есть проблемы с форматированием YAML:
- Лишние пробелы в конце строки 11
- Неправильные отступы в строках 10 и 21 (используется 6 пробелов вместо 4)
Рекомендую исправить форматирование:
- layers: - - state: normalitycrystal_3 - map: ["base"] + layers: + - state: normalitycrystal_3 + map: ["base"]И аналогично для других мест с неправильными отступами и пробелами.
🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 10-10: wrong indentation: expected 4 but found 6
(indentation)
[error] 11-11: trailing spaces
(trailing-spaces)
[warning] 21-21: wrong indentation: expected 4 but found 6
(indentation)
[warning] 27-27: wrong indentation: expected 4 but found 6
(indentation)
30-56: Добавлены новые варианты кристаллов нормальности с разными размерами стеков.Созданы сущности CrystalNormality1 и CrystalNormality10, наследуемые от основного CrystalNormality, с разными размерами стеков (1 и 10). Для CrystalNormality1 также добавлен тег RawMaterial, что позволяет использовать его как сырьевой материал.
Необходимо удалить лишние пробелы в конце строк 43, 55 и 56:
- - RawMaterial - + - RawMaterial +- - NormalityCrystal - + - NormalityCrystal +🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 43-43: trailing spaces
(trailing-spaces)
[error] 55-55: trailing spaces
(trailing-spaces)
[error] 56-56: trailing spaces
(trailing-spaces)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (6)
Resources/Textures/Backmen/Objects/Materials/materials.rsi/normalitycrystal.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Materials/materials.rsi/normalitycrystal_2.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Materials/materials.rsi/normalitycrystal_3.pngis excluded by!**/*.pngResources/Textures/Objects/Specific/Syndicate/telecrystal.rsi/telecrystal.pngis excluded by!**/*.pngResources/Textures/Objects/Specific/Syndicate/telecrystal.rsi/telecrystal_2.pngis excluded by!**/*.pngResources/Textures/Objects/Specific/Syndicate/telecrystal.rsi/telecrystal_3.pngis excluded by!**/*.png
📒 Files selected for processing (12)
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/materials/materials.ftl(1 hunks)Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/specific/research/crystals.ftl(1 hunks)Resources/Prototypes/Entities/Objects/Specific/syndicate.yml(2 hunks)Resources/Prototypes/Stacks/Materials/crystals.yml(1 hunks)Resources/Prototypes/_Backmen/Entities/Objects/Materials/materials.yml(2 hunks)Resources/Prototypes/_Backmen/Entities/Objects/Specific/Research/crystals.yml(1 hunks)Resources/Prototypes/_Backmen/Reactions/psionic.yml(1 hunks)Resources/Prototypes/_Backmen/Reagents/Materials/materials.yml(1 hunks)Resources/Prototypes/_Backmen/Recipes/Construction/Graphs/structures/glimmerdevices.yml(1 hunks)Resources/Prototypes/_Backmen/Stacks/materials.yml(1 hunks)Resources/Textures/Backmen/Objects/Materials/materials.rsi/meta.json(1 hunks)Resources/Textures/Objects/Specific/Syndicate/telecrystal.rsi/meta.json(1 hunks)
🧰 Additional context used
🪛 YAMLlint (1.35.1)
Resources/Prototypes/_Backmen/Entities/Objects/Materials/materials.yml
[error] 45-45: trailing spaces
(trailing-spaces)
[warning] 52-52: wrong indentation: expected 4 but found 2
(indentation)
[warning] 57-57: wrong indentation: expected 6 but found 4
(indentation)
Resources/Prototypes/_Backmen/Reagents/Materials/materials.yml
[error] 15-15: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/Stacks/Materials/crystals.yml
[warning] 4-4: too many spaces after colon
(colons)
[error] 6-6: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/_Backmen/Stacks/materials.yml
[error] 13-13: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/_Backmen/Entities/Objects/Specific/Research/crystals.yml
[warning] 10-10: wrong indentation: expected 4 but found 6
(indentation)
[error] 11-11: trailing spaces
(trailing-spaces)
[warning] 21-21: wrong indentation: expected 4 but found 6
(indentation)
[warning] 27-27: wrong indentation: expected 4 but found 6
(indentation)
[error] 43-43: trailing spaces
(trailing-spaces)
[error] 55-55: trailing spaces
(trailing-spaces)
[error] 56-56: trailing spaces
(trailing-spaces)
Resources/Prototypes/Entities/Objects/Specific/syndicate.yml
[warning] 13-13: wrong indentation: expected 4 but found 6
(indentation)
[error] 15-15: trailing spaces
(trailing-spaces)
[error] 16-16: trailing spaces
(trailing-spaces)
[warning] 22-22: wrong indentation: expected 4 but found 6
(indentation)
[error] 26-26: trailing spaces
(trailing-spaces)
[error] 61-61: trailing spaces
(trailing-spaces)
[error] 69-69: trailing spaces
(trailing-spaces)
🪛 GitHub Actions: YAML Linter
Resources/Prototypes/_Backmen/Reagents/Materials/materials.yml
[error] 14-16: No EntityPrototype found with id NormalityCrystal
⏰ Context from checks skipped due to timeout of 300000ms (2)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (14)
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/materials/materials.ftl (1)
7-9: Новая локализация добавлена корректно.Локализация для нового типа материала "блюспейс кристалл" с суффиксом 10 добавлена правильно, с соблюдением существующего формата и корректной ссылкой на общее описание.
Resources/Textures/Backmen/Objects/Materials/materials.rsi/meta.json (1)
9-28: Корректно добавлены новые состояния спрайтов.Новые состояния спрайтов для кристаллов нормальности ("normalitycrystal", "normalitycrystal_2", "normalitycrystal_3") добавлены с соблюдением существующего формата и структуры. Изменения соответствуют целям PR по модификации текстур.
Resources/Prototypes/_Backmen/Reactions/psionic.yml (1)
33-33: Обновлена ссылка на сущность с поддержкой стеков.Изменение корректно обновляет реакцию на создание нового типа кристалла нормальности с поддержкой стеков (CrystalNormality1) вместо старого типа. Это соответствует целям PR по внедрению функциональности стаков.
Resources/Prototypes/_Backmen/Recipes/Construction/Graphs/structures/glimmerdevices.yml (1)
34-36: Упрощение записи для NormalityCrystal
Объединение нескольких отдельных записей в одну с указанием параметровamount: 5иdoAfter: 5выглядит корректно и значительно упрощает структуру этапов строительства.Resources/Prototypes/_Backmen/Stacks/materials.yml (2)
6-7: Обновление параметра maxCount для Bluespace
УстановкаmaxCount: 30для материала Bluespace корректна и соответствует требованиям по управлению стековыми значениями.
8-13: Добавление нового материала NormalityCrystal
Новый материал с типом стека, заданием и максимальным количеством (maxCount: 30) соответствует общей стратегии унификации управления материалами.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 13-13: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/_Backmen/Entities/Objects/Materials/materials.yml (2)
28-28: Увеличение параметра count в Stack
Изменение значенияcountдо 30 для существующего компонента Stack выглядит оправданно и согласовано с изменениями в других файлах.
46-59: Добавление нового объекта MaterialBluespace10
Новая сущность MaterialBluespace10 добавлена корректно с компонентами Stack и Tag, что расширяет функциональность без нарушения структуры. Проверьте, чтобы обновления соответствовали ссылкам в локализации и другой логике системы.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 52-52: wrong indentation: expected 4 but found 2
(indentation)
[warning] 57-57: wrong indentation: expected 6 but found 4
(indentation)
Resources/Prototypes/Stacks/Materials/crystals.yml (1)
1-6: Обновление структуры Telecrystal
Изменения, включающие обновлённое свойствоiconс указаниемspriteиstate, а также добавлениеmaxCount: 30, способствуют унификации описания предметов.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 4-4: too many spaces after colon
(colons)
[error] 6-6: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Textures/Objects/Specific/Syndicate/telecrystal.rsi/meta.json (3)
13-18: Добавление состояний telecrystal_2 и telecrystal_3
Новые состоянияtelecrystal_2иtelecrystal_3добавлены корректно и позволят улучшить визуальное представление объекта.
19-26: Проверка состояния inhand-left и inhand-right
Убедитесь, что состоянияinhand-leftиinhand-rightс атрибутом"directions": 4по-прежнему необходимы. Если эти состояния больше не используются в функционале, рассмотрите возможность их удаления для упрощения структуры.
1-29: Общая структура meta.json
Файл структурирован корректно; внесённые изменения соответствуют обновлённой логике работы с состояниями объекта.Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/specific/research/crystals.ftl (1)
2-7: Локализация успешно обновлена для поддержки новых вариантов кристаллов.Добавлен суффикс "Полный" для основного кристалла нормальности и созданы новые локализации для вариантов с разным количеством (1 и 10). Локализация корректно соответствует изменениям в файлах прототипов сущностей.
Resources/Prototypes/_Backmen/Entities/Objects/Specific/Research/crystals.yml (1)
1-92: Необходима проверка совместимости с другими системами.Изменение родительского класса CrystalNormality с BaseItem на MaterialBase и добавление стековой функциональности могут повлиять на другие системы, которые взаимодействуют с кристаллами нормальности. Убедитесь, что все зависимые системы (крафт, исследования, хранение и т.д.) правильно работают с обновленными сущностями.
🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 10-10: wrong indentation: expected 4 but found 6
(indentation)
[error] 11-11: trailing spaces
(trailing-spaces)
[warning] 21-21: wrong indentation: expected 4 but found 6
(indentation)
[warning] 27-27: wrong indentation: expected 4 but found 6
(indentation)
[error] 43-43: trailing spaces
(trailing-spaces)
[error] 55-55: trailing spaces
(trailing-spaces)
[error] 56-56: trailing spaces
(trailing-spaces)
| id: NormalityCrystal | ||
| name: normality сrystal | ||
| icon: { sprite: Backmen/Objects/Materials/materials.rsi, state: normalitycrystal } | ||
| color: "#77e5f6" | ||
| stackEntity: NormalityCrystal | ||
| price: 20 |
There was a problem hiding this comment.
Отсутствует определение прототипа сущности NormalityCrystal.
Материал ссылается на стаковую сущность NormalityCrystal в строке 14, но прототип этой сущности не найден в системе. Это приведет к ошибкам при сборке и загрузке игры.
Необходимо добавить определение прототипа сущности NormalityCrystal в соответствующий файл (возможно, в директории Entities/Objects/Specific/Research/ или Materials/). Пример определения:
- type: entity
id: NormalityCrystal
name: normality crystal
parent: BaseItem
components:
- type: Sprite
sprite: Backmen/Objects/Materials/materials.rsi
state: normalitycrystal
- type: Item
size: 5
- type: Stack
stackType: NormalityCrystal🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 15-15: no new line character at the end of file
(new-line-at-end-of-file)
|
|
||
| - type: material | ||
| id: NormalityCrystal | ||
| name: normality сrystal |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Опечатка в названии материала.
В названии материала "normality сrystal" используется кириллическая буква "с" вместо латинской "c", что приведет к несоответствиям в отображении.
- name: normality сrystal
+ name: normality crystal📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| name: normality сrystal | |
| name: normality crystal |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
Resources/Prototypes/_Backmen/Reagents/Materials/materials.yml (1)
15-15: Отсутствует новая строка в конце файла.
Для соблюдения стандартов YAML и устранения предупреждения YAMLlint добавьте пустую строку в конце файла.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 15-15: no new line character at the end of file
(new-line-at-end-of-file)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/_Backmen/Reagents/Materials/materials.yml(1 hunks)
🧰 Additional context used
🪛 YAMLlint (1.35.1)
Resources/Prototypes/_Backmen/Reagents/Materials/materials.yml
[error] 15-15: no new line character at the end of file
(new-line-at-end-of-file)
⏰ Context from checks skipped due to timeout of 300000ms (2)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
🔇 Additional comments (1)
Resources/Prototypes/_Backmen/Reagents/Materials/materials.yml (1)
9-15: Новый материал "NormalityCrystal" корректно добавлен.
Проверьте, что значение поляstackEntity: CrystalNormalityсоответствует определённому прототипу в системе и что в остальных связанных файлах используется именно это имя. Также убедитесь, что цена (20) и остальные параметры соответствуют требованиям игрового баланса.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 15-15: no new line character at the end of file
(new-line-at-end-of-file)
…фиксы.
Изменение ТК, БС кристаллов и кристаллов нормальности.
Описание PR
Изменение текстур кристаллов нормальности и телекристаллов. Добавление стаков БС кристаллам, кристаллам нормальности и телекристаллам. Изменение создания дренажа сияния под стаки.
Медиа
Тип PR
Изменения
🆑 RaNlik
Summary by CodeRabbit
Новые возможности
Обновления