[Feature] New Lobby Music #155
Conversation
WalkthroughВ данном запросе на изменение были внесены изменения в класс Changes
Suggested labels
Suggested reviewers
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
Status: Needs Review
Changes: C#
Changes: YML
Changes: Audio
and removed
Status: Needs Review
labels
This reverts commit 08ecd6b.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (3)
Resources/Prototypes/_White/SoundCollections/lobby.yml (1)
1-20: Добавьте пустую строку в конец файлаДля соответствия стандартам YAML необходимо добавить пустую строку в конец файла.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 20-20: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/SoundCollections/lobby.yml (1)
23-23: Добавьте пустую строку в конец файлаДля соответствия стандартам YAML необходимо добавить пустую строку в конец файла.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 23-23: no new line character at the end of file
(new-line-at-end-of-file)
Content.Server/Audio/ContentAudioSystem.cs (1)
17-17: Используйте стандартный формат комментариевКомментарий
// WD EDITне соответствует стандартному формату документации. Рекомендуется либо удалить комментарий, либо добавить более подробное описание изменения.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (17)
Resources/Audio/_White/Lobby/all-that-I-can-see.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/atomicamnesiammx.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/circles-around-the-sun.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/comet_haley.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/endless_space.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/every_light_is_blinking_at_once.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/lasers_rip_apart_the_bulkhead.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/level!.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/marhaba.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/mod.flip-flap.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/monument.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/singuloose.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/space-oddity.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/space-shuttle.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/space_asshole.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/spaced.oggis excluded by!**/*.oggResources/Audio/_White/Lobby/title3.oggis excluded by!**/*.ogg
📒 Files selected for processing (4)
Content.Server/Audio/ContentAudioSystem.cs(1 hunks)Resources/Audio/_White/Lobby/attributions.yml(1 hunks)Resources/Prototypes/SoundCollections/lobby.yml(1 hunks)Resources/Prototypes/_White/SoundCollections/lobby.yml(1 hunks)
🧰 Additional context used
🪛 yamllint (1.35.1)
Resources/Prototypes/SoundCollections/lobby.yml
[error] 23-23: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/_White/SoundCollections/lobby.yml
[error] 20-20: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Audio/_White/Lobby/attributions.yml
[error] 79-79: no new line character at the end of file
(new-line-at-end-of-file)
🪛 GitHub Check: YAML RGA schema validator
Resources/Audio/_White/Lobby/attributions.yml
[failure] 1-1:
[/Resources/Audio/_White/Lobby/attributions.yml] 15.source: 'youtu.be/Hz6KO0gnrS8' is not a url.
🔇 Additional comments (5)
Resources/Prototypes/_White/SoundCollections/lobby.yml (1)
11-11: Проверьте специальные символы в именах файлов
Файлы level!.ogg и другие содержат специальные символы в именах. Хотя это технически допустимо, рекомендуется использовать только буквы, цифры и дефисы для обеспечения максимальной совместимости.
Also applies to: 12-12
Resources/Prototypes/SoundCollections/lobby.yml (1)
23-23:
Проверьте дублирование аудиофайла Monument.ogg
Файл Monument.ogg присутствует как в коллекции LobbyMusic, так и в новой коллекции WhiteLobbyMusic. Это может привести к путанице при воспроизведении.
Рекомендуется:
- Определить, какая коллекция должна содержать этот файл
- Удалить файл из одной из коллекций
🧰 Tools
🪛 yamllint (1.35.1)
[error] 23-23: no new line character at the end of file
(new-line-at-end-of-file)
Content.Server/Audio/ContentAudioSystem.cs (1)
17-17:
Проверьте все ссылки на старую коллекцию
Изменение имени коллекции с LobbyMusic на WhiteLobbyMusic может повлиять на другие части кодовой базы. Необходимо убедиться, что все ссылки обновлены.
✅ Verification successful
Изменение имени коллекции безопасно
Анализ кодовой базы показывает, что строка "WhiteLobbyMusic" используется только в одном месте - в константе LobbyMusicCollection. Все остальные упоминания "LobbyMusic" относятся к названиям переменных, методов и событий, а не к имени коллекции. Изменение имени коллекции не влияет на работу остального кода, так как все обращения к ней происходят через эту константу.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Search for references to old collection name
rg "LobbyMusic" --type-not yml
Length of output: 58
Script:
#!/bin/bash
# Search for references to old collection name with proper ripgrep syntax
rg "LobbyMusic" --type-not yaml
# Also search for WhiteLobbyMusic to see current usage
rg "WhiteLobbyMusic" --type-not yaml
# Use ast-grep to find string literals that might contain these collection names
ast-grep --pattern 'string $_ = "LobbyMusic"'
ast-grep --pattern 'string $_ = "WhiteLobbyMusic"'
Length of output: 5511
Resources/Audio/_White/Lobby/attributions.yml (2)
41-45: Проверьте пользовательские лицензии
Для треков с пользовательскими лицензиями необходимо убедиться в наличии соответствующих текстовых файлов с условиями лицензирования.
Also applies to: 56-60
✅ Verification successful
Let me check the content of the found license file to verify it matches the attribution.
Лицензионный файл найден и соответствует атрибуции
Файл flip-flap.txt присутствует в директории Resources/Audio/Lobby/ и содержит чёткие условия лицензирования от автора X-ceed, которые соответствуют информации, указанной в attributions.yml:
- Разрешено свободное распространение для некоммерческого использования
- Требуется разрешение автора для коммерческого использования
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify custom license files exist
# Check for flip-flap.txt mentioned in the attribution
echo "Checking for flip-flap.txt..."
fd -t f "flip-flap.txt" Resources/Audio/
# Check for any other license files
echo "Checking for other license files..."
fd -t f "license" Resources/Audio/_White/Lobby/
Length of output: 274
Script:
#!/bin/bash
# Check content of flip-flap.txt
cat Resources/Audio/Lobby/flip-flap.txt
Length of output: 1800
2-4: Проверьте актуальность источников музыки
Необходимо убедиться, что все указанные источники музыки доступны и активны.
Also applies to: 7-9, 12-14, 17-19, 22-24, 27-29, 32-34, 37-39, 47-49, 52-54, 62-64, 67-69, 72-74, 77-79
✅ Verification successful
Все источники музыки активны и доступны
Проверка показала, что все URL-адреса в файле attributions.yml возвращают код состояния HTTP 200, что означает их доступность. Единственное исключение - последняя ссылка youtu.be/Hz6KO0gnrS8, которая указана без протокола https://. Рекомендуется добавить протокол https:// для единообразия, но сама ссылка также ведет на действующее видео.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify music sources are accessible
# Extract and check all URLs from the YAML file
echo "Checking source URLs..."
grep "source:" Resources/Audio/_White/Lobby/attributions.yml | \
sed 's/.*source: "\(.*\)".*/\1/' | \
while read -r url; do
if [[ $url == http* ]]; then
echo "Checking: $url"
curl -sL -w "%{http_code}" "$url" -o /dev/null
fi
done
Length of output: 4875
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (2)
Resources/Audio/_White/Lobby/attributions.yml (2)
4-4: Очистите URL-адреса от лишних параметровСледующие URL содержат необязательные параметры отслеживания:
- Строка 4:
ysclid=m4k6j9v59l684837129- Строка 64:
?si=-UhlsL7qYvvV7KjvРекомендуется использовать чистые URL без параметров отслеживания.
- source: "https://www.youtube.com/watch?v=LFXo1XN6xc8&ysclid=m4k6j9v59l684837129" + source: "https://www.youtube.com/watch?v=LFXo1XN6xc8" - source: "https://youtu.be/rW86xYFmcVs?si=-UhlsL7qYvvV7Kjv" + source: "https://youtu.be/rW86xYFmcVs"Also applies to: 64-64, 74-74
1-1: Стандартизируйте именование файловВ именах файлов используются разные стили разделения слов:
- Дефисы:
all-that-I-can-see.ogg- Подчёркивания:
comet_haley.ogg- Специальные символы:
level!.oggРекомендуется выбрать единый стиль именования файлов для улучшения поддерживаемости.
Предлагаю использовать нижнее подчёркивание как стандартный разделитель:
- files: ["all-that-I-can-see.ogg"] + files: ["all_that_i_can_see.ogg"] - files: ["level!.ogg"] + files: ["level.ogg"] - files: ["mod.flip-flap.ogg"] + files: ["mod_flip_flap.ogg"]Also applies to: 6-6, 11-11, 16-16, 21-21, 26-26, 31-31, 36-36, 41-41, 46-46, 51-51, 56-56, 61-61, 66-66, 71-71, 76-76
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
Resources/Audio/_White/Lobby/attributions.yml(1 hunks)
🧰 Additional context used
🪛 yamllint (1.35.1)
Resources/Audio/_White/Lobby/attributions.yml
[error] 79-79: no new line character at the end of file
(new-line-at-end-of-file)
🔇 Additional comments (2)
Resources/Audio/_White/Lobby/attributions.yml (2)
1-79: Добавьте пустую строку в конец файла
Для соответствия стандартам YAML необходимо добавить пустую строку в конец файла.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 79-79: no new line character at the end of file
(new-line-at-end-of-file)
41-44: Проверьте наличие файлов с пользовательскими лицензиями
Для треков с пользовательскими лицензиями необходимо убедиться в наличии соответствующей документации:
- Для "Flip Flap" требуется файл
flip-flap.txtв директорииAudio/Lobby - Для "Space Asshole" указаны специальные условия использования
Also applies to: 56-60
✅ Verification successful
Let me check the content of the license file to verify it matches the attribution.
Файл лицензии найден и соответствует указанным условиям
Файл flip-flap.txt присутствует в директории Resources/Audio/Lobby и содержит корректную информацию о лицензии, которая соответствует описанию в attributions.yml. Лицензия явно разрешает бесплатное некоммерческое использование и распространение, что согласуется с указанным в атрибуции.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия файла лицензии
fd -t f "flip-flap.txt" Resources/Audio
Length of output: 75
Script:
#!/bin/bash
# Проверяем содержимое файла лицензии
cat Resources/Audio/Lobby/flip-flap.txt
Length of output: 1800
* feature: new lobby music * fix: attributions * fix: tupo * Revert "237.2.0" This reverts commit 08ecd6b. * fix: attributions
Описание PR
Я устал искать АП...
Музыка:
Изменения
🆑 Spatison