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

[Soragy-bot]

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

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

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

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

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

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

Changelog

🆑 Soragy

• add: Сперма теперь содержит ДНК источника, так что будьте аккуратны, детектив может вас найти.

Summary by CodeRabbit

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

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

• Изменения

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

[coderabbitai]

📝 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
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	Заголовок на русском языке описывает добавление репродуктивной жидкости, что соответствует основному изменению - добавлению ДНК в сперму для криминалистических целей.

_{✏️ 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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

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

[ReWAFFlution]

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

[KaiserMaus]

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

[Orvex07]

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

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

[Orvex07]

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

[Orvex07]

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

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