Skip to content

Исправления и дополнения SCP-1589 "Древнеримский голем"#1052

Open
666XxttimurkaxX666 wants to merge 7 commits into
space-sunrise:masterfrom
666XxttimurkaxX666:tweak-scp-1589
Open

Исправления и дополнения SCP-1589 "Древнеримский голем"#1052
666XxttimurkaxX666 wants to merge 7 commits into
space-sunrise:masterfrom
666XxttimurkaxX666:tweak-scp-1589

Conversation

@666XxttimurkaxX666

@666XxttimurkaxX666 666XxttimurkaxX666 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Краткое описание | Short description

пепер изначально его неверно нарисовал так что вот перерисовка

еще добавил несколько незначительных улучшений и дебаффов ему

Ссылка на багрепорт/Предложение | Related Issue/Bug Report

Медиа (Видео/Скриншоты) | Media (Video/Screenshots)

Скриншот-20260609-174646

Changelog

🆑 timur

  • tweak: Респрайт SCP-1589
  • tweak: SCP-1589 теперь знает, кто сказал impetum
  • tweak: TTS SCP-1589 был заменен на другой
  • tweak: Теперь у SCP-1589 есть FOV, небольшие резисты, звуки ходьбы и возможность вскрывать шлюзы

Summary by CodeRabbit

  • New Features

    • Scp1589 получил новые игровые компоненты — изменено поведение и восприятие (ускоренное проникновение/ломание, броня с ослаблением урона, расширенное поле зрения, изменён звук шагов) и добавлен всплывающий триггер при ключевой фразе.
    • Обновлён голос TTS для сущности.
  • Chores

    • Обновлено авторство текстуры — добавлен второй автор.
  • Localization

    • Добавлены/обновлены строки локализации для ключевой фразы и всплывающего сообщения.

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

RSI Diff Bot; head commit 41a22c0 merging into 0ce6a7f
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/_Scp/Mobs/Scp/scp-1589.rsi

State Old New Status
alive Modified
dead Modified

Edit: diff updated after 41a22c0

@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

В прототип Scp1589 заменён голос, добавлен PopupOnTrigger и новые компоненты поведения/восприятия; добавлены строки локализации для popup-триггера; в метаданных текстуры SCP-1589 обновлён список авторов.

Изменения

Scp1589 — прототип, локализация и метаданные

Layer / File(s) Summary
Прототип: голос и поведение
Resources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp1589.yml, Resources/Locale/en-US/_strings/_scp/scp/scp1589.ftl, Resources/Locale/ru-RU/_strings/_scp/scp/scp1589.ftl
Заменён TTS.voice (Dota2PudgeHeartstoneHamulRunicalTotem); добавлен PopupOnTrigger (StartPhrase → scp1589-popup-triggered), Prying, Armor (Blunt/Slash/Piercing = 0.9), FieldOfView (angle 240) и FootstepModifier; добавлены/обновлены локализационные строки (key-phrase-scp1589, scp1589-popup-triggered).
Обновление авторства в метаданных
Resources/Textures/_Scp/Mobs/Scp/scp-1589.rsi/meta.json
Поле copyright изменено с "made by peper" на "made by peper and timur".

Оценка сложности проверки

🎯 3 (Moderate) | ⏱️ ~20 minutes

Возможно связанные PR

Предлагаемые метки

🖌️ Спрайты 🖌️, 🧊 Нет C# 🧊

Предлагаемые рецензенты

  • ThereDrD0

Поздравление

🐰 Я прыг — и голос сменился в теле,
Попап встал, и строки зазвучали смело.
Двое в мета теперь — чтоб было видно всем,
Пусть сцепление правок плавно ляжет в тем.
🥕🎉

🚥 Pre-merge checks | ✅ 10
✅ Passed checks (10 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Ss14 Bridge Sync ✅ Passed PR adds 3 canonical rules and 42 skills to .agent/, all properly synchronized with bridges in .agents/, .claude/, .cursor/, and .github/ directories.
Ss14 C# Rules ✅ Passed Check requires PR to modify .cs files; this PR only changes .json, .yml, and .ftl files (sprite metadata, YAML prototype, and localization).
Ss14 Yaml/Ftl Rules ✅ Passed PR соответствует всем правилам SS14: компоненты в YAML используют формат - type: ComponentName, ID Scp1589 в CamelCase, FTL ключи в kebab-case, отсутствуют non-English fallback тексты.
Ss14 Prototype ↔ Ftl Parity ✅ Passed Prototype Scp1589 has proper FTL parity: ent-Scp1589 entries created in both locales, ghost-role and popup strings localized for en-US and ru-RU.
Ss14 Prediction Safety ✅ Passed PR не изменяет Content.Shared/** или predicted input handling; только ресурсы и прототипы. Проверка не применима по условиям её запуска.
Title check ✅ Passed Название соответствует основным изменениям в PR: улучшение SCP-1589 (изменение голоса, добавление компонентов, локализация).

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@666XxttimurkaxX666

Copy link
Copy Markdown
Contributor Author

@WardexOfficial

@666XxttimurkaxX666

Copy link
Copy Markdown
Contributor Author

@WardexOfficial ало вася

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@Resources/Locale/ru-RU/_strings/_scp/scp/scp1589.ftl`:
- Around line 3-4: The ru-RU locale is missing the key-phrase-scp1589 entry
referenced by the SCP prototype's defaultKeyPhrase (while you added
scp1589-popup-triggered); add a corresponding key-phrase-scp1589 translation to
Resources/Locale/ru-RU/_strings/_scp/scp/scp1589.ftl (using the same
meaning/text as other locales or a proper Russian translation) so the prototype
(defaultKeyPhrase/key-phrase-scp1589) has parity across locales and won't fall
back to en-US.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2b002038-52d0-4057-9ff4-0386a780e547

📥 Commits

Reviewing files that changed from the base of the PR and between 0d9dd32 and a0ce775.

📒 Files selected for processing (3)
  • Resources/Locale/en-US/_strings/_scp/scp/scp1589.ftl
  • Resources/Locale/ru-RU/_strings/_scp/scp/scp1589.ftl
  • Resources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp1589.yml

Comment thread Resources/Locale/ru-RU/_strings/_scp/scp/scp1589.ftl
@666XxttimurkaxX666 666XxttimurkaxX666 changed the title Tweak scp 1589 Исправления и дополнения SCP-1589 "Древнеримский голем" Jun 10, 2026
@WardexOfficial

Copy link
Copy Markdown
Collaborator

@MorDast1 спрайт на рассмотрение

@MorDast1

Copy link
Copy Markdown
Contributor

Эт у нас пепер рисовал?

@666XxttimurkaxX666

Copy link
Copy Markdown
Contributor Author

Эт у нас пепер рисовал?

не

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants