Приведение маркеров комментариев Lust к единому формату#502
Open
Soragy-bot wants to merge 2 commits intospace-sunrise:masterfrom
Open
Приведение маркеров комментариев Lust к единому формату#502Soragy-bot wants to merge 2 commits intospace-sunrise:masterfrom
Soragy-bot wants to merge 2 commits intospace-sunrise:masterfrom
Conversation
Standardize all fork-specific comments to use: - `# Lust-Edit` for single-line changes - `# Lust-Start` / `# Lust-End` for blocks Affects .yml and .cs files across Resources and Content.
github-actions
bot
added
Несортированный
size/M
Changes: UI
Статус: Требует Рассмотрения
and removed
Несортированный
labels
Contributor
📝 WalkthroughWalkthroughМассовая унификация текста комментариев и маркеров редактирования в коде и YAML (в основном: "Lust-start" → "Lust-Start", "Lust-edit" → "Lust-Edit" и т.п.). Единственное функциональное изменение — удаление ряда public-прототипов (humanoidBaseSprite, bodyType и пр.) в Изменения
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Стихотворение
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 5
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
Content.Client/Stylesheets/StyleNano.cs (1)
1059-1071:⚠️ Potential issue | 🟡 MinorПроверьте полноту нормализации маркеров fork-специфичных изменений перед слиянием.
Маркеры типа
// Lust-Start,// Lust-End,// Lust-Editиспользуются для документирования отличий форка от upstream. В текущем состоянии репозитория существуют разные варианты написания:
- старые:
#Lust Edit,# Lust Edit,Sunrise-Lust edit,Sunrise-Lust Edit- новые:
// Lust-Start,// Lust-End,// Lust-Edit- альтернативные:
Sunrise-Lust start,Lust-Station-Start,Lust-Station-EndЕсли любые скрипты, CI-правила или инструменты синхронизации привязаны к определённым вариантам написания, то нормализация только части маркеров в этом PR оставит остальные помеченные блоки невидимыми для этих инструментов. Убедитесь, что все нормализации проведены одновременно и что внешние потребители маркеров (если они есть) поддерживают новый формат.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Content.Client/Stylesheets/StyleNano.cs` around lines 1059 - 1071, This change introduces new fork markers (// Lust-Start, // Lust-End, // Lust-Edit) but other variants remain in the codebase; search for and normalize all fork-specific markers (e.g., "#Lust Edit", "# Lust Edit", "Sunrise-Lust edit", "Sunrise-Lust Edit", "Sunrise-Lust start", "Lust-Station-Start", "Lust-Station-End", etc.) to the chosen canonical format (// Lust-Start / // Lust-End / // Lust-Edit) everywhere they appear (including comments around StyleRule / SelectorElement / ItemList blocks shown here) and update any CI/scripts or marker consumers to recognize the new format so no annotated blocks are missed.
🧹 Nitpick comments (2)
Resources/Prototypes/_Lust/Loadouts/Medical/chief_medical_officer.yml (1)
2-2: Уточните: комментарий удалён, а не переформатирован.В отличие от других файлов в этом PR, где маркеры
# Lust-...стандартизируются, здесь комментарий# Lust Stationбыл полностью удалён. Поскольку файл находится в каталоге_Lust/, возможно это намеренно — весь файл является fork-специфичным контентом и не требует маркировки отдельных строк.Если это осознанное решение — всё в порядке. Рекомендуется уточнить в описании PR, что для файлов в
_Lust/маркеры не нужны.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Resources/Prototypes/_Lust/Loadouts/Medical/chief_medical_officer.yml` at line 2, The top-line comment '# Lust Station' was removed from chief_medical_officer.yml; either restore that exact comment (consistent with other files using '# Lust-...' markers) or, if the removal was intentional because this file lives in the _Lust folder, add a note to the PR description stating that files in _Lust/ are fork-specific and do not require Lust markers; ensure the change is applied to chief_medical_officer.yml and any similar files so marker handling is consistent across the PR.Resources/Prototypes/_Lust/Loadouts/Command/head_of_personnal.yml (1)
1-1: Лучше сохранить маркер, а не удалять его полностью.Для консистентности с остальным PR и сохранения форк-трассировки рекомендую заменить удалённый комментарий на стандартный
# Lust-Edit.♻️ Предлагаемое изменение
-- type: loadout +- type: loadout # Lust-EditBased on learnings: комментарии, помечающие fork-специфичные изменения (например,
# Lust Edit), являются частью системы документирования и не должны удаляться.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Resources/Prototypes/_Lust/Loadouts/Command/head_of_personnal.yml` at line 1, Вместо полного удаления маркера в начале файла восстановите комментарий-флаг форка: добавьте строку "# Lust-Edit" (стандартный маркер) над существующей строкой "type: loadout", чтобы сохранить трассировку изменений и соответствовать остальным правкам.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@Content.Client/Lobby/UI/HumanoidProfileEditor.xaml.cs`:
- Around line 1369-1373: The Lust markers in the Sex.Futanari switch block (the
case containing Profile = Profile?.WithGender(Gender.Female);) use nonstandard
comments "// Lust-Start" and "// Lust-End"; replace these with the project's
canonical C# marker format (use "// Sunrise-Lust Edit" or the agreed single-line
marker pair used elsewhere) so the start and end of this forked block match
other C# edits and service scripts can detect them reliably; ensure both the
opening and closing markers are updated together and preserve the exact
placement around the case block.
In `@Resources/Prototypes/_Lust/Loadouts/Cargo/quartermaster.yml`:
- Line 1: Восстановите маркер fork-изменения в этом YAML-файле: добавьте строку
с комментарием "# Lust-Edit" в начало файла перед существующей записью "type:
loadout" (удалите/ниже учтите BOM/невидимые символы) и сохраните именно этот
формат (с дефисом и заглавной L как "# Lust-Edit") так, чтобы
форматтер/нормализация не удаляла комментарий; убедитесь, что файл
Resources/Prototypes/_Lust/Loadouts/Cargo/quartermaster.yml содержит этот
комментарий на первой строке и что CI/преттир не удаляет его при следующем
коммите.
In `@Resources/Prototypes/_Lust/Loadouts/Engineering/station_engineer.yml`:
- Line 2: Restore the system fork-marker comment "# Lust Edit" that was
removed—reinsert the exact comment line immediately above or next to the YAML
root declaration (the existing "type: loadout" entry) so the file again contains
the fork marker used by space-sunrise/lust-station; ensure the comment is
preserved verbatim and not normalized away by formatters.
In `@Resources/Prototypes/_Lust/Loadouts/Science/research_director.yml`:
- Line 1: Форк-маркер был удалён: восстановите в этом YAML-файле комментарий
ветки рядом с ключом "type: loadout" — добавьте строку-комментарий "# Lust Edit"
(или "# Lust-Edit") над или рядом с записью "type: loadout" чтобы сохранить
идентификатор ветки; не удаляйте этот маркер при правке и убедитесь, что его
формат совпадает с принятым в проекте (символ # и текст "Lust Edit"/"Lust-Edit")
для всех YAML-файлов.
In `@Resources/Prototypes/_Lust/Loadouts/Security/security_officer.yml`:
- Line 2: В этом файле удалили маркер ветки вместо того, чтобы заменить его на
«# Lust-Edit»: восстановите пометку ровно на строке с идентификатором записи
типа — на строке содержащей "type: loadout" оставьте комментарий "# Lust-Edit"
(например как конец той же строки или прямо перед ней), чтобы пометка о
fork-специфичности loadout (идентификатор "type: loadout") сохранилась и не
удалялась при нормализации/слиянии.
---
Outside diff comments:
In `@Content.Client/Stylesheets/StyleNano.cs`:
- Around line 1059-1071: This change introduces new fork markers (// Lust-Start,
// Lust-End, // Lust-Edit) but other variants remain in the codebase; search for
and normalize all fork-specific markers (e.g., "#Lust Edit", "# Lust Edit",
"Sunrise-Lust edit", "Sunrise-Lust Edit", "Sunrise-Lust start",
"Lust-Station-Start", "Lust-Station-End", etc.) to the chosen canonical format
(// Lust-Start / // Lust-End / // Lust-Edit) everywhere they appear (including
comments around StyleRule / SelectorElement / ItemList blocks shown here) and
update any CI/scripts or marker consumers to recognize the new format so no
annotated blocks are missed.
---
Nitpick comments:
In `@Resources/Prototypes/_Lust/Loadouts/Command/head_of_personnal.yml`:
- Line 1: Вместо полного удаления маркера в начале файла восстановите
комментарий-флаг форка: добавьте строку "# Lust-Edit" (стандартный маркер) над
существующей строкой "type: loadout", чтобы сохранить трассировку изменений и
соответствовать остальным правкам.
In `@Resources/Prototypes/_Lust/Loadouts/Medical/chief_medical_officer.yml`:
- Line 2: The top-line comment '# Lust Station' was removed from
chief_medical_officer.yml; either restore that exact comment (consistent with
other files using '# Lust-...' markers) or, if the removal was intentional
because this file lives in the _Lust folder, add a note to the PR description
stating that files in _Lust/ are fork-specific and do not require Lust markers;
ensure the change is applied to chief_medical_officer.yml and any similar files
so marker handling is consistent across the PR.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 8980ab75-43e6-427f-8219-78819f5cdf9e
📒 Files selected for processing (69)
Content.Client/Clothing/ClientClothingSystem.csContent.Client/Humanoid/HumanoidAppearanceSystem.csContent.Client/Lobby/UI/HumanoidProfileEditor.xaml.csContent.Client/Stylesheets/StyleNano.csContent.Server/Administration/Systems/AdminVerbSystem.Smites.csContent.Server/Speech/EntitySystems/StutteringSystem.csContent.Server/_Sunrise/Fugitive/FugitiveSystem.csContent.Shared/Humanoid/Sex.csContent.Shared/Humanoid/SharedHumanoidAppearanceSystem.csContent.Shared/Inventory/InventoryComponent.csContent.Shared/Silicons/Laws/Components/EmagSiliconLawComponent.csContent.Shared/Trigger/Components/Effects/ShockOnTriggerComponent.csResources/Prototypes/Body/Parts/silicon.ymlResources/Prototypes/Catalog/VendingMachines/Inventories/curadrobe.ymlResources/Prototypes/Catalog/VendingMachines/Inventories/medidrobe.ymlResources/Prototypes/Catalog/VendingMachines/Inventories/theater.ymlResources/Prototypes/Entities/Mobs/NPCs/regalrat.ymlResources/Prototypes/Entities/Mobs/Player/clone.ymlResources/Prototypes/Entities/Mobs/Species/dwarf.ymlResources/Prototypes/Entities/Mobs/Species/moth.ymlResources/Prototypes/Entities/Mobs/Species/reptilian.ymlResources/Prototypes/Entities/Mobs/Species/slime.ymlResources/Prototypes/Entities/Objects/Consumable/Food/produce.ymlResources/Prototypes/Entities/Objects/Weapons/Guns/Ammunition/explosives.ymlResources/Prototypes/Entities/Structures/Storage/Crates/crates.ymlResources/Prototypes/Loadouts/Jobs/Security/security_officer.ymlResources/Prototypes/Loadouts/loadout_groups.ymlResources/Prototypes/Loadouts/role_loadouts.ymlResources/Prototypes/Roles/Jobs/Civilian/assistant.ymlResources/Prototypes/Roles/Jobs/Security/detective.ymlResources/Prototypes/Roles/Jobs/Security/head_of_security.ymlResources/Prototypes/Roles/Jobs/Security/warden.ymlResources/Prototypes/Species/dwarf.ymlResources/Prototypes/Species/human.ymlResources/Prototypes/Species/moth.ymlResources/Prototypes/Species/reptilian.ymlResources/Prototypes/Species/slime.ymlResources/Prototypes/Species/vulpkanin.ymlResources/Prototypes/_Lust/Loadouts/Cargo/quartermaster.ymlResources/Prototypes/_Lust/Loadouts/Command/captain.ymlResources/Prototypes/_Lust/Loadouts/Command/head_of_personnal.ymlResources/Prototypes/_Lust/Loadouts/Engineering/station_engineer.ymlResources/Prototypes/_Lust/Loadouts/Medical/chief_medical_officer.ymlResources/Prototypes/_Lust/Loadouts/Science/research_director.ymlResources/Prototypes/_Lust/Loadouts/Security/security_officer.ymlResources/Prototypes/_Lust/loadout_groups.ymlResources/Prototypes/_Sunrise/Entities/Clothing/Neck/cloaks.ymlResources/Prototypes/_Sunrise/Entities/Mobs/Cyborgs/borg_chassis.ymlResources/Prototypes/_Sunrise/Entities/Mobs/Species/demon.ymlResources/Prototypes/_Sunrise/Entities/Mobs/Species/felinid.ymlResources/Prototypes/_Sunrise/Entities/Objects/Books/hyperlinkbooks.ymlResources/Prototypes/_Sunrise/Entities/Objects/Specific/suntick.ymlResources/Prototypes/_Sunrise/Loadouts/loadout_groups.ymlResources/Prototypes/_Sunrise/Loadouts/role_loadouts.ymlResources/Prototypes/_Sunrise/Roles/Jobs/Law/iaa.ymlResources/Prototypes/_Sunrise/Roles/Jobs/Security/security_pilot.ymlResources/Prototypes/_Sunrise/Roles/Jobs/Security/senior_officer.ymlResources/Prototypes/_Sunrise/Species/demon.ymlResources/Prototypes/_Sunrise/Species/felinid.ymlResources/Prototypes/_Sunrise/Species/human.ymlResources/Prototypes/_Sunrise/Species/humanoid_xeno.ymlResources/Prototypes/_Sunrise/Species/predator.ymlResources/Prototypes/_Sunrise/Species/tajaran.ymlResources/Prototypes/_Sunrise/Store/currency.ymlResources/Prototypes/_Sunrise/tts-voices.ymlResources/Prototypes/_Sunrise/tts_voices_sunrise.ymlResources/Prototypes/bar_signs.ymlResources/Prototypes/borg_types.ymlResources/migration.yml
Comment on lines
+1369
to
+1373
| // Lust-Start | ||
| case Sex.Futanari: | ||
| Profile = Profile?.WithGender(Gender.Female); | ||
| break; | ||
| // Lust-end | ||
| // Lust-End |
Contributor
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# 1) Ищем потенциальные зависимости от старого формата вне прототипов.
rg -n -e 'LUST STATION|Lust Edit|#Lust Edit|Lust-start|Lust-end' -g '!Resources/Prototypes/**' .
# 2) Проверяем, что новый формат действительно используется в прототипах.
rg -n -e '# Lust-(Start|End|Edit)\b' Resources/PrototypesRepository: space-sunrise/lust-station
Length of output: 50382
Несогласованность форматов маркеров Lust в кодовой базе.
В коде используются блочные комментарии // Lust-Start и // Lust-End, но в проекте одновременно присутствуют другие форматы: // Sunrise-Lust Edit в C# и # Lust-Edit / # Lust-Start в YAML. Это создаёт риск для служебных скриптов, которые ищут fork-специфичные изменения — они могут не распознать все вариации маркеров.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@Content.Client/Lobby/UI/HumanoidProfileEditor.xaml.cs` around lines 1369 -
1373, The Lust markers in the Sex.Futanari switch block (the case containing
Profile = Profile?.WithGender(Gender.Female);) use nonstandard comments "//
Lust-Start" and "// Lust-End"; replace these with the project's canonical C#
marker format (use "// Sunrise-Lust Edit" or the agreed single-line marker pair
used elsewhere) so the start and end of this forked block match other C# edits
and service scripts can detect them reliably; ensure both the opening and
closing markers are updated together and preserve the exact placement around the
case block.
| @@ -1,4 +1,4 @@ | |||
| - type: loadout # Lust Station | |||
| - type: loadout | |||
Contributor
There was a problem hiding this comment.
Верните маркер fork-изменения на Line 1 (в нормализованном формате).
Сейчас маркер удалён, из-за чего теряется обязательная пометка отличий от upstream. В рамках этого PR его нужно не удалять, а привести к единому виду (# Lust-Edit).
Предлагаемая правка
-- type: loadout
+- type: loadout # Lust-EditBased on learnings: "Сохраняйте комментарий '# Lust Edit' во всех YAML-файлах проекта-ответвления ... Эти комментарии ... не должны удаляться ...", а также "preserve any comments that mark fork-specific changes ... must not be deleted".
📝 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.
Suggested change
| - type: loadout | |
| - type: loadout # Lust-Edit |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@Resources/Prototypes/_Lust/Loadouts/Cargo/quartermaster.yml` at line 1,
Восстановите маркер fork-изменения в этом YAML-файле: добавьте строку с
комментарием "# Lust-Edit" в начало файла перед существующей записью "type:
loadout" (удалите/ниже учтите BOM/невидимые символы) и сохраните именно этот
формат (с дефисом и заглавной L как "# Lust-Edit") так, чтобы
форматтер/нормализация не удаляла комментарий; убедитесь, что файл
Resources/Prototypes/_Lust/Loadouts/Cargo/quartermaster.yml содержит этот
комментарий на первой строке и что CI/преттир не удаляет его при следующем
коммите.
Contributor
Author
There was a problem hiding this comment.
Это наш файл он находится в _Lust, то есть его не будет при upstream.
Contributor
There was a problem hiding this comment.
@Soragy-bot, понял — спасибо за пояснение. Если файл целиком находится в директории _Lust, то он по определению является fork-специфичным и отсутствует в upstream, поэтому маркер # Lust-Edit здесь избыточен. Мой комментарий был ошибочным — прошу прощения за шум.
✏️ Learnings added
Learnt from: Soragy-bot
Repo: space-sunrise/lust-station PR: 502
File: Resources/Prototypes/_Lust/Loadouts/Cargo/quartermaster.yml:1-1
Timestamp: 2026-03-18T16:12:01.337Z
Learning: В проекте space-sunrise/lust-station маркеры fork-изменений (`# Lust-Edit`, `# Lust-Start`, `# Lust-End`) необходимы только в файлах, которые присутствуют и в upstream (например, в директориях `_Sunrise`, `Prototypes/Species`, `Prototypes/Roles` и т.д.), чтобы отмечать отличия от оригинала. Файлы, находящиеся целиком в директории `_Lust`, являются fork-специфичными по самому своему расположению и не требуют таких маркеров — весь файл уже является изменением форка.
Note: Learnings are effective only in the context of similar code segments. To apply general coding guidelines, please configure review instructions. You can manage existing learnings in the app.
🧠 Learnings used
Learnt from: Orvex07
Repo: space-sunrise/lust-station PR: 442
File: Resources/Prototypes/_Lust/CP/Weapons/hl2_weapons.yml:56-105
Timestamp: 2025-12-31T16:02:40.555Z
Learning: В проекте space-sunrise/lust-station при анализе PR с контентом в директории _Lust/CP необходимо проверять существование прототипов не только в файлах PR, но и в upstream директории _Sunrise, так как _Lust может использовать прототипы оружия, магазинов и других предметов из _Sunrise (например, MagazineMP7 определён в Resources/Prototypes/_Sunrise/Entities/Objects/Weapons/Guns/Ammunition/Magazines/pistol.yml).
Learnt from: Esco-dev
Repo: space-sunrise/lust-station PR: 427
File: Resources/Prototypes/_Sunrise/Roles/Jobs/Command/nanotrasen_representative.yml:7-7
Timestamp: 2025-12-12T08:52:41.391Z
Learning: Сохраняйте комментарий '# Lust Edit' во всех YAML-файлах проекта-ответвления (fork) space-sunrise/lust-station. Эти комментарии помечают fork-специфические изменения по отношению к upstream sunrise-station и используются в системе документации изменений. Они не должны удаляться во время слияний/переносов изменений. При работе с файлами, где встречаются такие комментарии, сохраняйте их, чтобы сохранить контекст форка, и при необходимости добавляйте новые пометки вместо удаления.
Learnt from: Esco-dev
Repo: space-sunrise/lust-station PR: 427
File: Resources/Prototypes/_Sunrise/Roles/Jobs/Command/nanotrasen_representative.yml:7-7
Timestamp: 2025-12-12T08:52:27.300Z
Learning: In the repository space-sunrise/lust-station, preserve any comments that mark fork-specific changes, specifically the marker "#Lust Edit" in YAML files (e.g., Resources/Prototypes/_Sunrise/Roles/Jobs/Command/nanotrasen_representative.yml). These comments are part of the change-documented system relative to the upstream sunrise-station and must not be deleted, merged over, or removed during edits or merges.
Learnt from: daniil7383
Repo: space-sunrise/lust-station PR: 367
File: Tools/_lust/replace_nt.py:16-45
Timestamp: 2025-08-25T14:36:37.252Z
Learning: В проекте space-sunrise/lust-station пользователь daniil7383 предпочитает не добавлять аннотации типов возврата для функций в файле Tools/_lust/replace_nt.py, даже если на это указывает Ruff правило ANN201.
Learnt from: pxc1984
Repo: space-sunrise/lust-station PR: 272
File: Resources/Prototypes/_Lust/Entities/Clothing/Neck/collar.yml:8-8
Timestamp: 2025-06-07T21:58:43.603Z
Learning: В проекте space-sunrise/lust-station используется стандартное YAML форматирование, где элементы списка под ключом `components:` располагаются на том же уровне отступа, что и сам ключ. Это корректный YAML синтаксис, и YAMLlint может давать ложные срабатывания для такого форматирования.
Learnt from: darneya
Repo: space-sunrise/lust-station PR: 425
File: Resources/Prototypes/_Lust/CP/Clothing/jumpsuits.yml:197-208
Timestamp: 2025-12-27T09:48:52.949Z
Learning: В проекте space-sunrise/lust-station при добавлении контента Half-Life 2 (HL2) по техническому заданию пользователя Wein описания предметов должны строго следовать предоставленному ТЗ, даже если они могут показаться семантически несоответствующими названию предмета (например, ClothingUniformJumpsuitCPRefugee с описанием помощника на поле боя).
Learnt from: CR
Repo: space-sunrise/lust-station PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-01T18:35:30.075Z
Learning: Include changelog entries in PR descriptions for player-visible changes using the format with author as 'Copilot AI' and change types (add, remove, tweak, fix)
Learnt from: CR
Repo: space-sunrise/lust-station PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-01T18:35:30.075Z
Learning: Write changelog messages in Russian for consistency with existing entries
| @@ -1,5 +1,5 @@ | |||
| # Neck | |||
| - type: loadout # Lust Station | |||
| - type: loadout | |||
Contributor
There was a problem hiding this comment.
Восстановите fork-маркер на Line 2 (сейчас он удалён).
В этом PR маркеры нормализуются, но здесь маркер фактически исчез. Для поддержки синка с upstream его нужно сохранить в едином формате.
Предлагаемое исправление
-- type: loadout
+- type: loadout # Lust-Edit📝 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.
Suggested change
| - type: loadout | |
| - type: loadout # Lust-Edit |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@Resources/Prototypes/_Lust/Loadouts/Engineering/station_engineer.yml` at line
2, Restore the system fork-marker comment "# Lust Edit" that was
removed—reinsert the exact comment line immediately above or next to the YAML
root declaration (the existing "type: loadout" entry) so the file again contains
the fork marker used by space-sunrise/lust-station; ensure the comment is
preserved verbatim and not normalized away by formatters.
| @@ -1,4 +1,4 @@ | |||
| - type: loadout # Lust Station | |||
| - type: loadout | |||
Contributor
There was a problem hiding this comment.
Не удаляйте fork-маркер на type: loadout.
Здесь маркер Lust исчез совсем, хотя цель PR — унифицировать формат, а не убрать его. Верните комментарий в новом виде, например # Lust-Edit.
Предлагаемая правка
-- type: loadout
+- type: loadout # Lust-Edit📝 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.
Suggested change
| - type: loadout | |
| - type: loadout # Lust-Edit |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@Resources/Prototypes/_Lust/Loadouts/Science/research_director.yml` at line 1,
Форк-маркер был удалён: восстановите в этом YAML-файле комментарий ветки рядом с
ключом "type: loadout" — добавьте строку-комментарий "# Lust Edit" (или "#
Lust-Edit") над или рядом с записью "type: loadout" чтобы сохранить
идентификатор ветки; не удаляйте этот маркер при правке и убедитесь, что его
формат совпадает с принятым в проекте (символ # и текст "Lust Edit"/"Lust-Edit")
для всех YAML-файлов.
| @@ -1,5 +1,5 @@ | |||
| # Neck | |||
| - type: loadout # Lust Station | |||
| - type: loadout | |||
Contributor
There was a problem hiding this comment.
Здесь тоже нужен # Lust-Edit, а не полное удаление маркера.
Этот loadout остаётся fork-специфичным, поэтому после нормализации пометка должна сохраниться на строке type: loadout.
Предлагаемая правка
-- type: loadout
+- type: loadout # Lust-Edit📝 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.
Suggested change
| - type: loadout | |
| - type: loadout # Lust-Edit |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@Resources/Prototypes/_Lust/Loadouts/Security/security_officer.yml` at line 2,
В этом файле удалили маркер ветки вместо того, чтобы заменить его на «#
Lust-Edit»: восстановите пометку ровно на строке с идентификатором записи типа —
на строке содержащей "type: loadout" оставьте комментарий "# Lust-Edit"
(например как конец той же строки или прямо перед ней), чтобы пометка о
fork-специфичности loadout (идентификатор "type: loadout") сохранилась и не
удалялась при нормализации/слиянии.
Soragy-bot
force-pushed
the
chore/normalize-lust-markers
branch
from
e2da6b1 to
7fd1451
Compare
Contributor
There was a problem hiding this comment.
🧹 Nitpick comments (1)
Tools/_sunrise/Schemas/validate_yml.py (1)
95-95: Изменение комментария соответствует цели PR.Стандартизация маркера комментария выглядит корректно.
Однако стоит отметить, что параметр
dirзатеняет встроенную функцию Python, что может привести к путанице. Рассмотрите возможность переименования.🔧 Предлагаемое переименование параметра
-def check_yml_files(dir: str, ignore_list: List[str], ignore_files: List[str]): # Lust-Edit +def check_yml_files(directory: str, ignore_list: List[str], ignore_files: List[str]): # Lust-Edit key_pattern = re.compile(r'^(name|description|suffix|rules|desc):\s*(.+)') - for yml_rel in iglob("**/*.yml", root_dir=dir, recursive=True): + for yml_rel in iglob("**/*.yml", root_dir=directory, recursive=True): if any(ignored in yml_rel for ignored in ignore_files): # Lust-Edit continue # Lust-Edit - yml_path = os.path.join(dir, yml_rel) + yml_path = os.path.join(directory, yml_rel)И обновите вызов на строке 130:
- check_yml_files(args.yml_dir, ignore_list, ignore_files) # Lust-Edit + check_yml_files(args.yml_dir, ignore_list, ignore_files) # Lust-EditBased on learnings: В проекте space-sunrise/lust-station используется комментарий "# Lust Edit" для маркировки изменений в файлах относительно upstream проекта sunrise-station.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Tools/_sunrise/Schemas/validate_yml.py` at line 95, The function check_yml_files has a parameter named dir which shadows the built-in dir(); rename that parameter (for example to directory or dir_path) in the function definition and update all internal references inside check_yml_files (and its signature) as well as every call site (e.g., the invocation near the noted call on line ~130 in this file) to use the new name so no built-in is shadowed and behavior remains identical.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@Tools/_sunrise/Schemas/validate_yml.py`:
- Line 95: The function check_yml_files has a parameter named dir which shadows
the built-in dir(); rename that parameter (for example to directory or dir_path)
in the function definition and update all internal references inside
check_yml_files (and its signature) as well as every call site (e.g., the
invocation near the noted call on line ~130 in this file) to use the new name so
no built-in is shadowed and behavior remains identical.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 099ba5b8-a8e4-4145-846d-a370150af648
📒 Files selected for processing (7)
Content.Server/Database/ServerDbBase.csContent.Server/Station/Systems/StationJobsSystem.Roundstart.csContent.Shared/Roles/JobPrototype.csResources/Prototypes/Entities/Mobs/NPCs/regalrat.ymlResources/Prototypes/Roles/Jobs/departments.ymlTools/_sunrise/Schemas/ignore_list.ymlTools/_sunrise/Schemas/validate_yml.py
✅ Files skipped from review due to trivial changes (4)
- Content.Server/Database/ServerDbBase.cs
- Tools/_sunrise/Schemas/ignore_list.yml
- Content.Server/Station/Systems/StationJobsSystem.Roundstart.cs
- Content.Shared/Roles/JobPrototype.cs
🚧 Files skipped from review as they are similar to previous changes (1)
- Resources/Prototypes/Entities/Mobs/NPCs/regalrat.yml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Кратное описание
Приведение маркеров комментариев Lust к единому формату. Лучше дальше следить за тем, что комментарии указываются в едином формате. Так же удали комментарии в наших файлах.
По какой причине
Нужно было перевести это всё к единому виду для дальнейшего удобства.
Медиа(Видео/Скриншоты)
Changelog
🆑 Soragy
Summary by CodeRabbit
Примечания к выпуску
Chores
Refactor