Skip to content

feat: лайтовая зависимость от кофе (попапы ломки без симптомов) - #3217

Open
ultradyper wants to merge 3 commits into
AdventureTimeSS14:masterfrom
ultradyper:feat/coffee-addiction
Open

feat: лайтовая зависимость от кофе (попапы ломки без симптомов)#3217
ultradyper wants to merge 3 commits into
AdventureTimeSS14:masterfrom
ultradyper:feat/coffee-addiction

Conversation

@ultradyper

@ultradyper ultradyper commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Описание PR

Лайтовая зависимость от кофе для экипажа: "Я не я, если не выпью кофейку". Новый канал Coffee в существующей системе аддикций ADT.

Почему / Баланс

Без трайта, только получаемая зависимость (как никотин/алкоголь). Никаких сильных последствий - только попапы: через 10 минут без кофе каждые 60 секунд напоминание о ломке. Ни дрожи, ни заикания, ни слабости. Лечится воздержанием или детоксином, видна в анализаторе здоровья.

Техническая информация

  • AddictionKind.Coffee - шестой канал, добавлен в конец enum (безопасно для сети)

  • Определение по id реагента Coffee (DataField CoffeeReagent), как у никотина; латте/айс-кофе/кофейные ликёры не подсаживают

  • Кофе исключён из всей симптомной логики (Update-лупа, RefreshSymptoms, RaiseSymptomsChanged), чтобы кофейная ломка не снимала чужие симптомы вроде дрожи от электрошока

  • Локализация в Resources/Locale/ru-RU/ADT/addictions.ftl

  • Изменения были протестированы на локальном сервере, и всё работает отлично.

  • PR закончен и требует просмотра изменений.

Медиа

Чейнджлог

🆑 ultradyper

  • add: Добавлена зависимость от кофе.

@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

@ultradyper, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 29 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 04a64ffe-b248-4f43-83c4-8b6a9ac98fcc

📥 Commits

Reviewing files that changed from the base of the PR and between 8917ddc and 81b81a2.

📒 Files selected for processing (5)
  • Content.Client/HealthAnalyzer/UI/HealthAnalyzerControl.xaml.cs
  • Content.Server/ADT/Addiction/AddictionSymptomsSystem.cs
  • Content.Server/ADT/Addiction/AddictionSystem.cs
  • Content.Shared/ADT/Addiction/AddictionComponent.cs
  • Resources/Locale/ru-RU/ADT/addictions.ftl
✨ 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.

@github-actions github-actions Bot added size/S Status: Needs Review Changes: UI Изменение интерфейса Changes: Localization Изменение локализации and removed Status: Needs Review size/S labels Aug 18, 2026

@Inconnu1337 Inconnu1337 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ты вообще свой код смотришь перед тем, как открывать ПР? Что сложного в том, чтобы осмотреть этот нейрослоп и убрать комментарии агента, которые он расставил на каждое изменение, подробно разжёвывая даже самые очевидные изменения в коде?

@Inconnu1337

Copy link
Copy Markdown
Collaborator

Это касается всех твоих ПРов.

@ultradyper

Copy link
Copy Markdown
Contributor Author

Ты вообще свой код смотришь перед тем, как открывать ПР? Что сложного в том, чтобы осмотреть этот нейрослоп и убрать комментарии агента, которые он расставил на каждое изменение, подробно разжёвывая даже самые очевидные изменения в коде?
Ок если тебя так не устраивают обычные комментарии, то их не будет учту.

Removed comment about coffee addiction symptoms.
Remove commented code regarding coffee addiction symptoms.
@Inconnu1337

Copy link
Copy Markdown
Collaborator

Помимо комментариев, меня не устраивает ещё одна вещь. Вся эта система зависимостей представляет собой сплошной хардкод. Каждая новая зависимость требует редактирования кода, каждый новый реагент, который должен стать аллергией, требует добавления нового кода. Никакой гибкости и расширяемости.

@Inconnu1337

Copy link
Copy Markdown
Collaborator

По хорошему, все эффекты зависимостей должны быть вынесены в отдельные классы или что то в этом роде. Тогда при добавлении новой зависимости достаточно будет описать её в прототипе, указать строки локализации, эффекты и прочие необходимые параметры. При этом сам код системы зависимостей каждый раз менять не придётся

@ultradyper

Copy link
Copy Markdown
Contributor Author

По хорошему, все эффекты зависимостей должны быть вынесены в отдельные классы или что то в этом роде. Тогда при добавлении новой зависимости достаточно будет описать её в прототипе, указать строки локализации, эффекты и прочие необходимые параметры. При этом сам код системы зависимостей каждый раз менять не придётся

Железобетонно прав. Сделаю полный рефактор в скором времени, но на самом деле врядли будет добавляться еще какая-то зависимость от чего либо, и это не имеет смысла именно сейчас.

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

Labels

Changes: Localization Изменение локализации Changes: UI Изменение интерфейса size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants