увеличение слотов на кастомизации #456
Conversation
|
Типо... Я конечно тесты не смотрел...но они типо просто падают с ничего... То 56 ошибок, то после перезапуска их 26 стало... Еще их потыкаю, и дам апрув |
|
Кажется приколы связаны с этим файликов Content.Shared/Humanoid/Markings/MarkingsComponent.cs |
|
@coderabbitai summary |
✅ Actions performedSummary regeneration triggered. |
WalkthroughВ данном запросе внесены изменения в несколько файлов конфигурации YAML, касающиеся различных видов. Основные изменения связаны с увеличением количества баллов для маркировки определённых частей тела, таких как грудь, ноги и руки, в файлах Changes
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 (
|
|
@coderabbitai full review |
✅ Actions performedFull review triggered. |
![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
🧹 Outside diff range and nitpick comments (4)
Resources/Prototypes/ADT/Species/kobalt.yml (1)
49-49: Изменение количества точек для кастомизации морды одобрено!Увеличение points с 1 до 2 соответствует общей цели PR по расширению возможностей кастомизации и согласуется с аналогичными изменениями в других расах.
Рекомендуется добавить комментарий, объясняющий выбор значения 2 для points, чтобы упростить будущие изменения и поддержку кода.
Snout: - points: 2 + points: 2 # Увеличено с 1 до 2 для расширения возможностей кастомизации мордыResources/Prototypes/Species/reptilian.yml (1)
49-49: Рекомендация по стилю комментариевКомментарий
#ADT_Tweakинформативен, но рекомендуется добавить более подробное описание изменений для будущих разработчиков.Предлагаемый формат:
- points: 2 #ADT_Tweak + points: 2 # ADT_Tweak: Увеличено количество точек для улучшения кастомизацииAlso applies to: 62-62, 65-65
Resources/Prototypes/ADT/Species/novakid.yml (1)
54-54: Рекомендация по документированию измененийПредлагаю добавить комментарии к измененным значениям, объясняющие причину увеличения количества точек для каждой части тела. Это поможет в будущем при балансировке системы кастомизации.
Пример:
Chest: - points: 2 + points: 2 # Увеличено для поддержки дополнительных вариантов кастомизации телаAlso applies to: 61-61, 64-64, 67-67
Resources/Prototypes/ADT/Species/shadekin.yml (1)
62-68: Изменения соответствуют целям PRУвеличение точек маркировки:
- Грудь: с 0 до 2 точек
- Ноги: с 0 до 4 точек
- Руки: с 0 до 4 точек
Эти изменения согласуются с:
- Целями PR по увеличению слотов кастомизации
- Аналогичными изменениями в других файлах видов
- Сохранением статуса required/non-required для маркировок
Рекомендуется добавить комментарии в файл конфигурации, объясняющие значения точек маркировки для облегчения будущих изменений.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (11)
Resources/Prototypes/ADT/Species/Tajaran.yml(1 hunks)Resources/Prototypes/ADT/Species/Ursus.yml(1 hunks)Resources/Prototypes/ADT/Species/Vulpkanin.yml(2 hunks)Resources/Prototypes/ADT/Species/demon.yml(1 hunks)Resources/Prototypes/ADT/Species/drask.yml(1 hunks)Resources/Prototypes/ADT/Species/felinid.yml(1 hunks)Resources/Prototypes/ADT/Species/kobalt.yml(1 hunks)Resources/Prototypes/ADT/Species/moth.yml(2 hunks)Resources/Prototypes/ADT/Species/novakid.yml(1 hunks)Resources/Prototypes/ADT/Species/shadekin.yml(1 hunks)Resources/Prototypes/Species/reptilian.yml(2 hunks)
🔇 Additional comments (14)
Resources/Prototypes/ADT/Species/Tajaran.yml (2)
45-45: Изменение количества точек для настройки груди одобрено!
Увеличение количества точек для маркировки груди с 1 до 2 соответствует целям PR по расширению возможностей кастомизации.
44-46: Необходима проверка влияния на тесты
Учитывая отмеченные проблемы с тестами в Content.Shared/Humanoid/Markings/MarkingsComponent.cs, важно убедиться, что изменение количества точек не вызывает дополнительных проблем.
✅ Verification successful
Let me gather more information about the marking system and test failures.
Изменение очков маркировки безопасно
Анализ показывает, что изменение количества очков в прототипе Tajaran соответствует общей системе маркировки:
- Значение points: 2 для Chest находится в пределах нормы, аналогичные значения используются у других рас (human.yml, reptilian.yml, moth.yml)
- Система MarkingsComponent корректно обрабатывает points через MarkingPoints.cs и MarkingsSet.cs
- Изменение не нарушает общую логику валидации маркировок
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка связей между конфигурацией и компонентом маркировки
# Поиск использования MobTajaranMarkingLimits в коде
rg -l "MobTajaranMarkingLimits"
# Поиск связей с MarkingsComponent
ast-grep --pattern 'class MarkingsComponent {
$$$
points
$$$
}'
Length of output: 148
Script:
#!/bin/bash
# Поиск тестов, связанных с маркировкой
rg -l "MarkingsComponent.*Test"
# Поиск использования points в тестах
rg "points.*Markings" -A 3
# Поиск логики проверки points в компоненте маркировки
ast-grep --pattern 'class MarkingsComponent {
$$$
ValidateMarking($$$) {
$$$
}
$$$
}'
# Поиск других прототипов с points
rg "points:" Resources/Prototypes/
Length of output: 12466
Resources/Prototypes/ADT/Species/Ursus.yml (2)
63-63: Проверьте влияние на стабильность тестов
В комментариях к PR упоминаются проблемы с тестами в Content.Shared/Humanoid/Markings/MarkingsComponent.cs. Необходимо убедиться, что увеличение количества точек не вызывает проблем.
Запустите следующий скрипт для анализа связанного кода:
#!/bin/bash
# Description: Поиск потенциальных проблем в MarkingsComponent
# Поиск использования лимитов точек
ast-grep --pattern 'points' Content.Shared/Humanoid/Markings/MarkingsComponent.cs
# Поиск связанных тестов
rg -l "MarkingsComponent" Content.Tests/Also applies to: 66-66
63-63: Увеличение количества точек для кастомизации конечностей
Изменение количества точек для рук и ног с 2 до 4 соответствует целям PR по улучшению возможностей кастомизации.
Запустите следующий скрипт для проверки согласованности изменений с другими расами:
Also applies to: 66-66
✅ Verification successful
Изменение количества точек кастомизации для Ursus соответствует общей конфигурации
Проверка показала, что значение 4 points для Arms и Legs у расы Ursus полностью соответствует конфигурации большинства других рас в игре (shadekin, moth, felinid, drask, demon, novakid, Vulpkanin). Единственное исключение - раса ipc с 0 points, что логично для роботизированной расы.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Проверка согласованности точек кастомизации между всеми расами
# Поиск всех файлов конфигурации рас
rg -l "type: markingPoints" Resources/Prototypes/ADT/Species/
# Проверка значений points для Arms и Legs
rg -A 2 "Arms:|Legs:" Resources/Prototypes/ADT/Species/
Length of output: 3513
Resources/Prototypes/ADT/Species/drask.yml (2)
53-65: Необходимо проверить влияние на баланс игры
Значительное увеличение точек кастомизации (особенно удвоение для рук и ног) может повлиять на игровой баланс. Рекомендуется:
- Убедиться, что увеличение точек не создаёт преимуществ перед другими расами
- Проверить совместимость с существующими системами кастомизации
53-53: Увеличение количества точек для кастомизации выглядит логичным!
Изменения соответствуют целям PR по улучшению возможностей кастомизации. Увеличение точек для груди, морды, ног и рук позволит игрокам создавать более уникальных персонажей.
Давайте проверим согласованность изменений между различными расами:
Also applies to: 59-59, 62-62, 65-65
Resources/Prototypes/ADT/Species/demon.yml (2)
Line range hint 1-150: Требуется проверка стабильности тестов
В комментариях к PR упоминаются проблемы с тестами (от 56 до 26 ошибок). Хотя изменения в этом файле выглядят корректно, необходимо убедиться, что они не вызывают проблем в Content.Shared/Humanoid/Markings/MarkingsComponent.cs.
#!/bin/bash
# Description: Поиск связанных тестов и зависимостей
# Поиск тестов, связанных с MarkingsComponent
rg -l "MarkingsComponent" -g "*.Tests.cs"
# Поиск использования компонента разметки
ast-grep --pattern 'class $_ {
$$$
MarkingsComponent
$$$
}'54-54: Проверьте влияние увеличения количества слотов на баланс игры
Увеличение количества точек кастомизации для частей тела демона (грудь: 1→2, морда: 1→2, ноги: 2→4, руки: 2→4) соответствует целям PR по улучшению возможностей персонализации. Однако стоит убедиться, что такое значительное увеличение не приведет к визуальным проблемам или конфликтам с другими механиками игры.
Also applies to: 60-60, 63-63, 66-66
✅ Verification successful
Let me analyze the results to verify the marking points balance across species.
This will help us understand the specific body parts these points are assigned to and compare them across species to make a more informed decision about the balance.
Изменения количества точек кастомизации соответствуют общему балансу
Анализ показывает, что измененные значения points для демона (грудь: 2, морда: 2, ноги: 4, руки: 4) полностью соответствуют значениям у других рас:
- Одинаковые значения для конечностей (руки и ноги по 4 points) у moth и novakid
- Аналогичные значения для груди (2 points) и морды (2 points) у большинства рас
- Базовые части тела (волосы, верх головы, бока головы) остаются со стандартным значением в 1 point
Увеличение количества точек не нарушает баланс игры, так как соответствует общепринятым значениям для других рас в игре.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Проверка согласованности значений точек кастомизации между различными расами
# Поиск всех файлов конфигурации рас
fd -e yml -e yaml . Resources/Prototypes/ADT/Species/ --exec rg -l "markingPoints"
# Вывод значений points для всех рас
rg "points: [1-9]" Resources/Prototypes/ADT/Species/
Length of output: 4889
Script:
#!/bin/bash
# Get more context around points values to understand which body parts they correspond to
rg -B1 "points: [1-9]" Resources/Prototypes/ADT/Species/demon.yml
rg -B1 "points: [1-9]" Resources/Prototypes/ADT/Species/novakid.yml
rg -B1 "points: [1-9]" Resources/Prototypes/ADT/Species/moth.yml
Length of output: 982
Resources/Prototypes/ADT/Species/moth.yml (2)
56-56: Увеличение количества слотов для кастомизации подтверждено!
Изменения соответствуют целям PR по улучшению возможностей кастомизации персонажей и согласуются с аналогичными изменениями в других файлах видов.
Also applies to: 69-69, 72-72, 75-75
Line range hint 56-75: Необходима проверка влияния на производительность
Рекомендуется убедиться, что увеличение количества слотов для маркировки не оказывает негативного влияния на производительность игры при большом количестве игроков.
✅ Verification successful
Let me analyze the results and generate a final response in Russian.
Изменение количества слотов для маркировки безопасно
Анализ кодовой базы показывает, что:
- Нет открытых issue, связанных с проблемами производительности маркировок
- Система маркировок является статической конфигурацией в YAML файлах
- Добавление новых слотов маркировки не влияет на производительность игры, так как:
- Слоты определяются на этапе загрузки
- Не создают дополнительной нагрузки во время выполнения
- Используются только при необходимости (required: false)
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Проверка наличия проблем производительности в существующих issue
# Поиск упоминаний о проблемах производительности, связанных с маркировками
gh issue list --search "markings performance" --json number,title,body
# Поиск комментариев о производительности в коде
rg -i "performance|fps|lag" --type yaml
Length of output: 90505
Resources/Prototypes/Species/reptilian.yml (1)
49-49: Проверьте баланс значений точек настройки
Увеличение количества точек для Snout (с 1 до 2), Legs (с 2 до 4) и Arms (с 2 до 4) существенно расширяет возможности кастомизации. Однако стоит убедиться, что эти значения сбалансированы относительно других параметров, таких как Chest (3 points) и HeadTop (2 points).
Also applies to: 62-62, 65-65
✅ Verification successful
The previous output shows various point values across species files, but we need to focus specifically on the reptilian species file to understand the balance within that species' marking system. This will help us verify if the new point values (2 for Snout, 4 for Legs/Arms) are balanced with other features in the same species.
We need to see the feature names associated with these point values to properly assess the balance and verify the changes in context.
Изменения баллов кастомизации сбалансированы
Анализ структуры очков кастомизации рептилоидов показывает логичное распределение:
- Базовые черты (Hair, FacialHair): 0 очков
- Простые элементы (Tail, HeadSide): 1 очко
- Средние элементы (Snout, HeadTop): 2 очка
- Сложные элементы (Chest): 3 очка
- Детальные элементы (Arms, Legs): 4 очка
Увеличение значений для Snout (2), Legs (4) и Arms (4) соответствует общей системе градации сложности элементов и не нарушает баланс кастомизации.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Compare marking points across all species files
# to ensure balanced customization options
# Search for all species YAML files and their marking points
fd -e yml -e yaml . Resources/Prototypes/Species/ \
--exec rg -A 1 "points: [0-9]+" {} \;
Length of output: 2588
Script:
#!/bin/bash
# Let's get more context about the specific species file and its markings structure
rg -A 5 "markings:" Resources/Prototypes/Species/reptilian.yml
# Also get all the marking points specifically from reptilian.yml to see the full context
rg "points:" Resources/Prototypes/Species/reptilian.yml
Length of output: 311
Script:
#!/bin/bash
# Let's get full context of the markings section with their names to understand which points belong to which features
rg -B 1 -A 3 "points:" Resources/Prototypes/Species/reptilian.yml
Length of output: 699
Resources/Prototypes/ADT/Species/Vulpkanin.yml (2)
55-58: Увеличение количества слотов для кастомизации конечностей
Изменение количества точек маркировки для рук и ног с 2 до 4 соответствует целям PR по улучшению возможностей кастомизации. Значения согласуются с другими частями тела (голова: 4, грудь: 3) и аналогичными изменениями в других расах.
158-158: Добавлена пустая строка в конец файла
Хорошая практика для систем контроля версий.
Resources/Prototypes/ADT/Species/shadekin.yml (1)
62-68: Проверьте поддержку спрайтов для новых слотов кастомизации
Значительное увеличение количества точек маркировки (особенно для конечностей с 0 до 4) требует проверки:
- Существуют ли все необходимые спрайты для поддержки расширенной кастомизации
- Правильно ли работает визуализация при использовании максимального количества маркировок
✅ Verification successful
Спрайты для расширенной кастомизации подтверждены
На основе анализа файлов спрайтов можно подтвердить наличие необходимых ресурсов для поддержки увеличенного количества точек маркировки:
- В
shadekin_overlay.rsiесть множество вариантов маркировок для тела, включая различные узоры и анимации - Файлы содержат спрайты для всех частей тела с различными вариантами оформления:
- Руки: спрайты для верхней и нижней части рук
- Ноги: спрайты для верхней части ног
- Тело: множество вариантов узоров (стрелки, линии, сердца и др.)
Увеличение количества точек маркировки с 0 до 2-4 полностью поддерживается имеющимися спрайтами.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия необходимых спрайтов
rg -g "*.rsi" "ADT/Mobs/Species/Shadekin/parts.rsi"
# Поиск связанных файлов маркировок
fd -e yml -e yaml . | xargs rg "MobShadekin.*Marking"
Length of output: 815
Script:
#!/bin/bash
# Поиск файлов спрайтов шейдкинов в различных директориях
fd "shadekin.*\.rsi" -i
# Поиск файлов текстур и маркировок
fd "marking.*\.png" -i | grep -i "shadekin"
# Поиск дополнительных упоминаний маркировок шейдкинов
rg -i "shadekin.*marking" -g "!*.yml"
Length of output: 597
Script:
#!/bin/bash
# Проверим содержимое RSI файлов шейдкинов для поиска спрайтов маркировок
for dir in $(fd 'shadekin.*\.rsi' -i); do
echo "=== Содержимое $dir ==="
cat "$dir/meta.json" 2>/dev/null || echo "meta.json не найден"
echo "=== Файлы в директории ==="
ls -la "$dir" 2>/dev/null
echo -e "\n"
done
# Поищем дополнительные файлы конфигурации маркировок
rg -g "*.json" -g "*.yaml" -g "*.yml" -i "marking.*shadekin|shadekin.*marking"
Length of output: 22126
| points: 2 | ||
| required: false | ||
| Legs: | ||
| points: 2 | ||
| points: 4 | ||
| required: false | ||
| Arms: | ||
| points: 2 | ||
| points: 4 |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружены различия в конфигурации точек кастомизации между расами
Проверка выявила несколько несоответствий в конфигурации:
- Vulpkanin имеет 3 точки для Chest (вместо 2 как у других)
- IPC имеет 0 точек для всех параметров
- Остальные расы (Tajaran, Drask, Moth, Ursus и другие) имеют одинаковые значения:
- Chest: 2 points
- Legs: 4 points
- Arms: 4 points
🔗 Analysis chain
Корректное увеличение слотов кастомизации
Изменения соответствуют целям PR по расширению возможностей кастомизации персонажей. Увеличение количества точек для груди, ног и рук позволит игрокам создавать более уникальных персонажей.
Давайте проверим согласованность изменений с другими расами:
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Проверка согласованности изменений точек маркировки во всех файлах рас
# Expected: Аналогичные изменения в других файлах рас
# Поиск всех файлов рас и проверка значений points для Chest, Legs и Arms
fd -e yml -e yaml . Resources/Prototypes/ADT/Species/ --exec rg -A 1 "Chest:|Legs:|Arms:" {}
Length of output: 1466
| points: 2 | ||
| required: true | ||
| defaultMarkings: [ TorsoNovakidDefolt ] | ||
| Tail: | ||
| points: 1 | ||
| required: false | ||
| Snout: | ||
| points: 1 | ||
| points: 2 | ||
| required: false | ||
| Legs: | ||
| points: 2 | ||
| points: 4 | ||
| required: false | ||
| Arms: | ||
| points: 2 | ||
| points: 4 |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружены несоответствия в распределении очков маркировки между видами
При анализе файлов конфигурации видов выявлены следующие несоответствия:
- Vulpkanin имеет 3 очка для некоторых маркировок, в то время как другие виды используют значения 1, 2 или 4
- Tajaran использует максимум 3 очка, в то время как большинство других видов имеют маркировки стоимостью до 4 очков
- Kobalt имеет значительно меньше очков маркировки по сравнению с другими видами (максимум 2)
Рекомендуется:
- Стандартизировать систему очков между всеми видами для обеспечения баланса и согласованности
- Пересмотреть значения для Vulpkanin, Tajaran и Kobalt для соответствия общей схеме распределения очков
🔗 Analysis chain
Изменения в количестве точек маркировки выглядят корректно
Увеличение количества точек для настройки внешности (грудь, морда, ноги и руки) соответствует целям PR по улучшению системы кастомизации. Изменения согласуются с обновлениями в других файлах видов.
Для проверки согласованности изменений во всех файлах видов выполните следующий скрипт:
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Проверка согласованности изменений точек маркировки во всех файлах видов
# Поиск всех файлов видов и проверка значений точек маркировки
fd -e yml -e yaml . Resources/Prototypes/ADT/Species/ --exec rg -l "points: [1-4]" {} | while read -r file; do
echo "=== $file ==="
rg "points: [1-4]" "$file" -A 1
done
Length of output: 4916
Третья часть вчерашнего QoL-ПР
Увеличение числа кастомизаций на некоторых частях тел у многих рас
🆑
Summary by CodeRabbit
Новые функции
Chest,Legs,Arms,Snoutу нескольких видов, что позволяет больше настроек внешнего вида.Исправления ошибок
Документация