Brain Blip's back by vsixst · Pull Request #16 · Lua-Frontier/Monolith-DS

vsixst · 2026-07-03T01:33:09Z

О пулл-реквесте

Добавлена OutOfContainerSystem, добавляющая компоненты на сущность пока он вне контейнера (т.е. на полу, в космосе.. зачем я это уточняю в который раз?) и отбирающая их когда сущность в контейнере (в руках, в сумке, итд.).
С этой системой возвращён блип на мозге, но с двумя добавлениями:

Блип виден только вне грида и не в сумке/руках/теле
Блип виден только в радиусе 512 метров.

Из дополнительного:
Возвращение к логике визардов в ComponentTogglerSystem и исправление ошибки связанной с предиктом в ClothingGrantingSystem

Обоснование / Баланс

Хотели сделать блипы на мозге? В целом, ситуация с отображением в теле исправлена.

Технические детали

OutOfContainerSystem по сути просто перенос ClothingGrantingSystem (_Goobstation.Clothing.Systems) с использованием других ивентов (EntGotInsertedIntoContainerMessage и EntGotRemovedIntoContainerMessage) и проверки IsEntityInContainer. Комментарии к методам я думаю излишни, логика коротка и понятна.

Проблема с ComponentTogglerSystem достаточно тупая, но существует. Если компонент ComponentTogglerComponent висит на шлеме, который добавляется через скафандр - то при снятии скафандра с куклы другой куклы, компоненты не снимаются корректно. Т.к. я не знаю почему губы вынесли эту логику из OnToggled в ToggleComponent я, соотв., решил аналогично изменить логику в ToggleComponent, а не полностью вернуть систему к визденам. (У визденов нету метода ToggleComponent - вся логика находится в OnToggled методе).

Как протестировать

Ожидается, что вы уже скопировали репо, обновили движок, сбилдили и запустили.
Заспавнить (или достать из куклы) мозг
Открыть радар
Выкинуть мозг в космос

Медиа

Требования

Я прочитал CONTRIBUTING.md и соблюдаю Руководство по пулл-реквестам и изменению логов.
Я приложил медиа-материалы к этому PR или они не требуются.
Я ознакомился с Руководством по добавлению кораблей, если это уместно.
Я подтверждаю, что ИИ-инструменты не использовались для создания материалов в этом PR.

Критические изменения

Вроде я не насрал

Журнал изменений

🆑 vsixst

add: Возвращена метка мозга на радаре, если мозг находится в космосе а наблюдатель в радиусе 512м.

Примечание

Я кстати жду похвалу за такую роспись, чёт я много расписал для такого мелкого реквеста.

Основные: - Добавлена OutOfContainerSystem, добавляющая компоненты на ентити, если та находится вне контейнеров (т.е. на полу) - Возвращены блипы на мозг, с условием что мозг: а) Вне грида; б) Смотрящий в радиусе 512м. Чисто по приколу: - Обновлена/Возвращена логика в ComponentTogglerSystem на логику визденов. - Исправлена глупая ошибка с предиктом в ClothingGrantingSystem

Pcolik505

По комментам плюс-минус вот так.

Pcolik505 · 2026-07-03T11:42:37Z

+
+    // <summary>
+    // It holds the entity that the component gave the component to, so it can remove from it even if it changes parent.
+    // </summary>
+    [DataField]
+    public EntityUid? Target;


Suggested change

// <summary>

// It holds the entity that the component gave the component to, so it can remove from it even if it changes parent.

// </summary>

[DataField]

public EntityUid? Target;

// LuaM-start:

// <summary>

// It holds the entity that the component gave the component to, so it can remove from it even if it changes parent.

// </summary>

[DataField]

public EntityUid? Target;

// LuaM-end.

Pcolik505 · 2026-07-03T11:42:49Z

@@ -1,12 +1,15 @@
 using Content.Shared.Item.ItemToggle.Components;
+using Robust.Shared.Timing;


Suggested change

using Robust.Shared.Timing;

using Robust.Shared.Timing; // LuaM

Pcolik505 · 2026-07-03T11:43:09Z

 /// Handles <see cref="ComponentTogglerComponent"/> component manipulation.
 /// </summary>
-public sealed class ComponentTogglerSystem : EntitySystem
+public sealed partial class ComponentTogglerSystem : EntitySystem


Suggested change

public sealed partial class ComponentTogglerSystem : EntitySystem

public sealed partial class ComponentTogglerSystem : EntitySystem // LuaM: public sealed class > public sealed partial class

Pcolik505 · 2026-07-03T11:43:52Z

    public void ToggleComponent(EntityUid uid, bool activate)
    {
-        if (!TryComp<ComponentTogglerComponent>(uid, out var component))
+        if (!_timing.IsFirstTimePredicted) // LuaM: predict err fix


Suggested change

if (!_timing.IsFirstTimePredicted) // LuaM: predict err fix

if (!_timing.IsFirstTimePredicted) // LuaM: !TryComp<ComponentTogglerComponent>(uid, out var component) > _timing.IsFirstTimePredicted

Pcolik505 · 2026-07-03T11:44:20Z

            EntityManager.AddComponents(target, component.Components);
+        }
        else
-            EntityManager.RemoveComponents(target, component.RemoveComponents ?? component.Components);


Закомментить, а не удалить.

Pcolik505 · 2026-07-03T11:44:27Z

+        if (!_timing.IsFirstTimePredicted) // LuaM: predict err fix
            return;

-        var target = component.Parent ? Transform(uid).ParentUid : uid;


Закомментить, а не удалить.

Pcolik505 · 2026-07-03T11:44:37Z

            return;

-        var target = component.Parent ? Transform(uid).ParentUid : uid;
+        if (!TryComp<ComponentTogglerComponent>(uid, out var component))


Suggested change

if (!TryComp<ComponentTogglerComponent>(uid, out var component))

if (!TryComp<ComponentTogglerComponent>(uid, out var component)) // LuaM

Pcolik505 · 2026-07-03T11:46:05Z

+    components:
+    - type: RadarBlip
+      radarColor: "#E0FFFF"
+      scale: 3


Suggested change

scale: 3

scale: 3 // LuaM: 6 > 3

Pcolik505 · 2026-07-03T11:47:16Z

+      maxDistance: 512
+      requireNoGrid: true


Suggested change

maxDistance: 512

requireNoGrid: true

# LuaM-start:

maxDistance: 512

requireNoGrid: true

# LuaM-end.

vsixst requested review from Covral, HacksLua and Pcolik505 as code owners July 3, 2026 01:33

github-actions Bot added size/M S: Untriaged YML C# S: Needs Review and removed S: Untriaged labels Jul 3, 2026

Pcolik505 removed the request for review from HacksLua July 3, 2026 11:10

Pcolik505 requested changes Jul 3, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Brain Blip's back#16

Brain Blip's back#16
vsixst wants to merge 1 commit into
Lua-Frontier:masterfrom
vsixst:brainblip

vsixst commented Jul 3, 2026

Uh oh!

Pcolik505 left a comment

Uh oh!

Pcolik505 Jul 3, 2026

Uh oh!

Pcolik505 Jul 3, 2026

Uh oh!

Pcolik505 Jul 3, 2026

Uh oh!

Pcolik505 Jul 3, 2026

Uh oh!

Pcolik505 Jul 3, 2026

Uh oh!

Pcolik505 Jul 3, 2026

Uh oh!

Pcolik505 Jul 3, 2026

Uh oh!

Pcolik505 Jul 3, 2026

Uh oh!

Pcolik505 Jul 3, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		@@ -1,12 +1,15 @@
		using Content.Shared.Item.ItemToggle.Components;
		using Robust.Shared.Timing;

	using Robust.Shared.Timing;
	using Robust.Shared.Timing; // LuaM

	public sealed partial class ComponentTogglerSystem : EntitySystem
	public sealed partial class ComponentTogglerSystem : EntitySystem // LuaM: public sealed class > public sealed partial class

	if (!_timing.IsFirstTimePredicted) // LuaM: predict err fix
	if (!_timing.IsFirstTimePredicted) // LuaM: !TryComp<ComponentTogglerComponent>(uid, out var component) > _timing.IsFirstTimePredicted

	if (!TryComp<ComponentTogglerComponent>(uid, out var component))
	if (!TryComp<ComponentTogglerComponent>(uid, out var component)) // LuaM

Uh oh!

Conversation

vsixst commented Jul 3, 2026

О пулл-реквесте

Обоснование / Баланс

Технические детали

Как протестировать

Медиа

Требования

Критические изменения

Примечание

Uh oh!

Pcolik505 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants