Change Kill Person to "Teach person a lesson"

Content.Server/DeltaV/Objectives/Components/TeachLessonConditionComponent.cs

@@ -0,0 +1,10 @@
+using Content.Server.Objectives.Systems;
+
+namespace Content.Server.Objectives.Components;
+
+/// <summary>
+/// Requires that a target dies once and only once.
+/// Depends on <see cref="TargetObjectiveComponent"/> to function.
+/// </summary>
+[RegisterComponent, Access(typeof(TeachLessonConditionSystem))]
+public sealed partial class TeachLessonConditionComponent : Component;

Content.Server/DeltaV/Objectives/Systems/TeachLessonConditionSystem.cs

@@ -0,0 +1,48 @@
+using Content.Server.Objectives.Components;
+using Content.Shared.GameTicking;
+using Content.Shared.Mind;
+using Content.Shared.Objectives.Components;
+
+namespace Content.Server.Objectives.Systems;
+
+/// <summary>
+/// Handles teach a lesson condition logic, does not assign target.
+/// </summary>
+public sealed class TeachLessonConditionSystem : EntitySystem
+{
+    [Dependency] private readonly SharedMindSystem _mind = default!;
+    [Dependency] private readonly TargetObjectiveSystem _target = default!;
+
+    private readonly List<EntityUid> _wasKilled = [];
+
+    public override void Initialize()
+    {
+        base.Initialize();
+
+        SubscribeLocalEvent<TeachLessonConditionComponent, ObjectiveGetProgressEvent>(OnGetProgress);
+        SubscribeLocalEvent<RoundRestartCleanupEvent>(OnRoundEnd);
+    }
+
+    private void OnGetProgress(Entity<TeachLessonConditionComponent> ent, ref ObjectiveGetProgressEvent args)
+    {
+        if (!_target.GetTarget(ent, out var target))
+            return;
+
+        args.Progress = GetProgress(target.Value);
+    }
+
+    private float GetProgress(EntityUid target)
+    {
+        if (TryComp<MindComponent>(target, out var mind) && mind.OwnedEntity != null && !_mind.IsCharacterDeadIc(mind))
+            return _wasKilled.Contains(target) ? 1f : 0f;
+
+        _wasKilled.Add(target);
+        return 1f;
+    }
+
+    // Clear the wasKilled list on round end
+    private void OnRoundEnd(RoundRestartCleanupEvent  ev)
+    {
+        _wasKilled.Clear();
+    }
+}

Resources/Locale/en-US/deltav/objectives/conditions/teach-person.ftl

@@ -0,0 +1 @@
+objective-condition-teach-person-title = Teach {$targetName}, {CAPITALIZE($job)} a lesson

Resources/Prototypes/DeltaV/Objectives/traitor.yml

@@ -50,3 +50,32 @@
   - type: StealCondition
     stealGroup: AnimalSilvia
     owner: job-name-cmo
+
+# teach lesson
+- type: entity
+  abstract: true
+  parent: BaseTargetObjective
+  id: BaseTeachLessonObjective
+  components:
+  - type: Objective
+    unique: false
+    icon:
+      sprite: Objects/Weapons/Guns/Pistols/viper.rsi
+      state: icon
+  - type: ObjectiveBlacklistRequirement
+    blacklist:
+      components:
+      - SocialObjective
+
+- type: entity
+  parent: [BaseTraitorObjective, BaseTeachLessonObjective]
+  id: TeachLessonRandomPersonObjective
+  description: Kill them, and show everyone we mean business. They only need to die once.
+  components:
+  - type: Objective
+    difficulty: 1.75
+    unique: false
+  - type: TargetObjective
+    title: objective-condition-teach-person-title
+  - type: PickRandomPerson
+  - type: TeachLessonCondition

Resources/Prototypes/Objectives/objectiveGroups.yml

@@ -32,7 +32,8 @@
 - type: weightedRandom
   id: TraitorObjectiveGroupKill
   weights:
-    KillRandomPersonObjective: 1
+    # KillRandomPersonObjective: 1 # DeltaV Replaced for Teach Lesson
+    TeachLessonRandomPersonObjective: 1
     KillRandomHeadObjective: 0.25
 
 - type: weightedRandom