Skip to content

Feature: Репродуктивная жидкость - #504

Open
Soragy-bot wants to merge 4 commits into
space-sunrise:masterfrom
Soragy-bot:feature/semen-dna-forensics
Open

Feature: Репродуктивная жидкость#504
Soragy-bot wants to merge 4 commits into
space-sunrise:masterfrom
Soragy-bot:feature/semen-dna-forensics

Conversation

@Soragy-bot

@Soragy-bot Soragy-bot commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

Кратное описание

Ну что заказать теперь в сперме будет содержать ДНК её владельца.

У меня по поводу файла есть сомнения, он вроде и санрайза, но там много наших изменений, но они не помечены, ну и я не стал их помечать.

По какой причине

Ну тут попросили #473

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

image

Changelog

🆑 Soragy

  • add: Репродуктивная жидкость теперь оставляет ДНК-следы для криминалистики.

Summary by CodeRabbit

  • Новые возможности

    • Биологические жидкости (сперма) теперь сохраняют информацию о ДНК источника.
    • Добавлено отслеживание происхождения в остатках жидкостей.
  • Изменения

    • Обновлено поведение создания и отображения жидкостей, чтобы учитывать источник ДНК.

@coderabbitai

coderabbitai Bot commented Mar 20, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough
📝 Walkthrough
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed Заголовок на русском языке описывает добавление репродуктивной жидкости, что соответствует основному изменению - добавлению ДНК в сперму для криминалистических целей.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ 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.

@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 the current code and only fix it if needed.

Inline comments:
In `@Content.Server/_Sunrise/InteractionsPanel/InteractionsPanel.Interactions.cs`:
- Around line 370-379: GetSemenDnaData currently creates a DnaData with a
localized "unknown" string when the source lacks a DnaComponent or DNA, causing
shared false-positive traces to be added later to ForensicsComponent.DNAs;
change GetSemenDnaData (in InteractionsPanel.Interactions.cs) to only construct
and return a List<ReagentData>/List<DnaData> containing a DnaData when
TryComp<DnaComponent>(source, out var dnaComp) &&
!string.IsNullOrEmpty(dnaComp.DNA) (assigning dnaData.DNA = dnaComp.DNA), and
otherwise return an empty list so no placeholder DnaData is appended to forensic
DNAs.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e6766b22-12df-4cce-a5c8-da0e930669c3

📥 Commits

Reviewing files that changed from the base of the PR and between e9e3df5 and 880d682.

📒 Files selected for processing (2)
  • Content.Server/_Sunrise/InteractionsPanel/InteractionsPanel.Interactions.cs
  • Resources/Prototypes/_Lust/ERP/Other/semen.yml

Comment thread Content.Server/_Sunrise/InteractionsPanel/InteractionsPanel.Interactions.cs Outdated
@ReWAFFlution

Copy link
Copy Markdown
Contributor

Нужно добавить комменты в стиле // Lust-Edit

@KaiserMaus

Copy link
Copy Markdown
Collaborator

Нужно добавить протеин

@Orvex07 Orvex07 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.

  1. Лучше всего сделать partial класс в папках _Lust, в котором будет логика ДНК и перенести туда изменения с ПРа #308 . Данный PR создаст в дальнейшем больше конфликтов из-за чего при мерджа sunrise -> lust могут удалится некоторые изменения.

  2. Перефразируй changelog. Изменю так же наименование ПРа на -> Feature: Репродуктивная жидкость. В последующем не указывай напрямую.

@Orvex07 Orvex07 changed the title feat: Добавлена ДНК для спермы Feature: Репродуктивная жидкость Mar 27, 2026
@Orvex07

Orvex07 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

ну да, лучше всего сразу почку заготовить. Так как панель будет улучшена в дальнейшем.

@Orvex07

Orvex07 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Или же я сделаю позже partial и только после доработак, приму ПР.

В любом случае, спасибо за контрибьют

Extract love/DNA/undress Lust code from _Sunrise to reduce upstream merge conflicts.
@github-actions github-actions Bot added size/M and removed size/S labels Jul 31, 2026
@Soragy-bot

Copy link
Copy Markdown
Contributor Author

@coderabbitai Сделай новое ревью PR

@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

@Soragy-bot Запускаю новое ревью PR #504.

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@Soragy-bot

Copy link
Copy Markdown
Contributor Author

@Soragy-bot Запускаю новое ревью PR #504.

✅ Action performed

Так проблемы есть или нет?

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants