From 0ac51b2002fa8e6733ac07c97c338ffa141b3716 Mon Sep 17 00:00:00 2001
From: Asriel Camora <asriel@camora.dev>
Date: Wed, 14 Jun 2023 10:35:43 -0700
Subject: [PATCH] Standardize more action code

---
 Craftimizer/Simulator/Actions/AdvancedTouch.cs   |  4 +---
 Craftimizer/Simulator/Actions/BaseAction.cs      | 16 ++++++++++++++--
 Craftimizer/Simulator/Actions/BasicSynthesis.cs  |  4 +---
 Craftimizer/Simulator/Actions/BasicTouch.cs      |  4 +---
 .../Simulator/Actions/ByregotsBlessing.cs        |  5 +++--
 .../Simulator/Actions/CarefulSynthesis.cs        |  4 +---
 .../Simulator/Actions/DelicateSynthesis.cs       |  8 ++------
 Craftimizer/Simulator/Actions/FinalAppraisal.cs  |  2 +-
 .../Simulator/Actions/FocusedSynthesis.cs        |  4 +---
 Craftimizer/Simulator/Actions/FocusedTouch.cs    |  4 +---
 Craftimizer/Simulator/Actions/GreatStrides.cs    |  1 -
 Craftimizer/Simulator/Actions/Groundwork.cs      | 13 +++++++++----
 Craftimizer/Simulator/Actions/HastyTouch.cs      |  4 +---
 Craftimizer/Simulator/Actions/Innovation.cs      |  1 -
 .../Simulator/Actions/IntensiveSynthesis.cs      |  4 +---
 Craftimizer/Simulator/Actions/Manipulation.cs    |  1 -
 Craftimizer/Simulator/Actions/MastersMend.cs     |  1 -
 Craftimizer/Simulator/Actions/MuscleMemory.cs    |  3 ++-
 Craftimizer/Simulator/Actions/Observe.cs         |  3 ---
 Craftimizer/Simulator/Actions/PreciseTouch.cs    |  3 ++-
 .../Simulator/Actions/PreparatoryTouch.cs        |  3 ++-
 .../Simulator/Actions/PrudentSynthesis.cs        |  4 +---
 Craftimizer/Simulator/Actions/PrudentTouch.cs    |  4 +---
 Craftimizer/Simulator/Actions/RapidSynthesis.cs  |  4 +---
 Craftimizer/Simulator/Actions/Reflect.cs         |  3 ++-
 Craftimizer/Simulator/Actions/StandardTouch.cs   |  4 +---
 Craftimizer/Simulator/Actions/TrainedEye.cs      |  4 ++--
 Craftimizer/Simulator/Actions/TrainedFinesse.cs  |  4 +---
 .../Simulator/Actions/TricksOfTheTrade.cs        |  1 -
 Craftimizer/Simulator/Actions/Veneration.cs      |  1 -
 Craftimizer/Simulator/Actions/WasteNot.cs        |  1 -
 Craftimizer/Simulator/Actions/WasteNot2.cs       |  1 -
 32 files changed, 52 insertions(+), 71 deletions(-)

diff --git a/Craftimizer/Simulator/Actions/AdvancedTouch.cs b/Craftimizer/Simulator/Actions/AdvancedTouch.cs
index 775ac5e..305f5bd 100644
--- a/Craftimizer/Simulator/Actions/AdvancedTouch.cs
+++ b/Craftimizer/Simulator/Actions/AdvancedTouch.cs
@@ -10,7 +10,5 @@ internal class AdvancedTouch : BaseAction
 
     public override int CPCost => Simulation.GetPreviousAction() is StandardTouch && Simulation.GetPreviousAction(2) is BasicTouch ? 18 : 46;
     public override float Efficiency => 1.50f;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseQuality(Efficiency);
+    public override bool IncreasesQuality => true;
 }
diff --git a/Craftimizer/Simulator/Actions/BaseAction.cs b/Craftimizer/Simulator/Actions/BaseAction.cs
index 384dc98..b4257ab 100644
--- a/Craftimizer/Simulator/Actions/BaseAction.cs
+++ b/Craftimizer/Simulator/Actions/BaseAction.cs
@@ -25,9 +25,12 @@ public abstract class BaseAction
     public abstract int ActionId { get; }
 
     public abstract int CPCost { get; }
-    public abstract float Efficiency { get; }
+    public virtual float Efficiency => 0f;
+    public virtual bool IncreasesProgress => false;
+    public virtual bool IncreasesQuality => false;
     public virtual float SuccessRate => 1f;
     public virtual int DurabilityCost => 10;
+    public virtual bool IncreasesStepCount => true;
 
     private (CraftAction? CraftAction, Action? Action) GetActionRow(ClassJob classJob)
     {
@@ -95,5 +98,14 @@ public abstract class BaseAction
             UseSuccess();
     }
 
-    public abstract void UseSuccess();
+    public virtual void UseSuccess()
+    {
+        if (Efficiency != 0f)
+        {
+            if (IncreasesProgress)
+                Simulation.IncreaseProgress(Efficiency);
+            if (IncreasesQuality)
+                Simulation.IncreaseQuality(Efficiency);
+        }
+    }
 }
diff --git a/Craftimizer/Simulator/Actions/BasicSynthesis.cs b/Craftimizer/Simulator/Actions/BasicSynthesis.cs
index 56ab3e3..9de266e 100644
--- a/Craftimizer/Simulator/Actions/BasicSynthesis.cs
+++ b/Craftimizer/Simulator/Actions/BasicSynthesis.cs
@@ -11,7 +11,5 @@ internal class BasicSynthesis : BaseAction
     public override int CPCost => 0;
     // Basic Synthesis Mastery Trait
     public override float Efficiency => Simulation.Stats.Level >= 31 ? 1.20f : 1.00f;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseProgress(Efficiency);
+    public override bool IncreasesProgress => true;
 }
diff --git a/Craftimizer/Simulator/Actions/BasicTouch.cs b/Craftimizer/Simulator/Actions/BasicTouch.cs
index f6fdfc1..3fd7951 100644
--- a/Craftimizer/Simulator/Actions/BasicTouch.cs
+++ b/Craftimizer/Simulator/Actions/BasicTouch.cs
@@ -10,7 +10,5 @@ internal class BasicTouch : BaseAction
 
     public override int CPCost => 18;
     public override float Efficiency => 1.00f;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseQuality(Efficiency);
+    public override bool IncreasesQuality => true;
 }
diff --git a/Craftimizer/Simulator/Actions/ByregotsBlessing.cs b/Craftimizer/Simulator/Actions/ByregotsBlessing.cs
index 75ecc0d..b0c3ff2 100644
--- a/Craftimizer/Simulator/Actions/ByregotsBlessing.cs
+++ b/Craftimizer/Simulator/Actions/ByregotsBlessing.cs
@@ -9,13 +9,14 @@ internal class ByregotsBlessing : BaseAction
     public override int ActionId => 100339;
 
     public override int CPCost => 24;
-    public override float Efficiency => 1.00f + 0.20f * (Simulation.GetEffect(Effect.InnerQuiet)?.Strength ?? 0);
+    public override float Efficiency => 1.00f + (0.20f * (Simulation.GetEffect(Effect.InnerQuiet)?.Strength ?? 0));
+    public override bool IncreasesQuality => true;
 
     public override bool CanUse => Simulation.HasEffect(Effect.InnerQuiet) && base.CanUse;
 
     public override void UseSuccess()
     {
-        Simulation.IncreaseQuality(Efficiency);
+        base.UseSuccess();
         Simulation.RemoveEffect(Effect.InnerQuiet);
     }
 }
diff --git a/Craftimizer/Simulator/Actions/CarefulSynthesis.cs b/Craftimizer/Simulator/Actions/CarefulSynthesis.cs
index ac7420f..8320d2c 100644
--- a/Craftimizer/Simulator/Actions/CarefulSynthesis.cs
+++ b/Craftimizer/Simulator/Actions/CarefulSynthesis.cs
@@ -11,7 +11,5 @@ internal class CarefulSynthesis : BaseAction
     public override int CPCost => 7;
     // Careful Synthesis Mastery Trait
     public override float Efficiency => Simulation.Stats.Level >= 82 ? 1.80f : 1.50f;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseProgress(Efficiency);
+    public override bool IncreasesProgress => true;
 }
diff --git a/Craftimizer/Simulator/Actions/DelicateSynthesis.cs b/Craftimizer/Simulator/Actions/DelicateSynthesis.cs
index 53d3da1..c8b6ba1 100644
--- a/Craftimizer/Simulator/Actions/DelicateSynthesis.cs
+++ b/Craftimizer/Simulator/Actions/DelicateSynthesis.cs
@@ -10,10 +10,6 @@ internal class DelicateSynthesis : BaseAction
 
     public override int CPCost => 32;
     public override float Efficiency => 1.00f;
-
-    public override void UseSuccess()
-    {
-        Simulation.IncreaseQuality(Efficiency);
-        Simulation.IncreaseProgress(Efficiency);
-    }
+    public override bool IncreasesProgress => true;
+    public override bool IncreasesQuality => true;
 }
diff --git a/Craftimizer/Simulator/Actions/FinalAppraisal.cs b/Craftimizer/Simulator/Actions/FinalAppraisal.cs
index 5e1fe5a..a5935ff 100644
--- a/Craftimizer/Simulator/Actions/FinalAppraisal.cs
+++ b/Craftimizer/Simulator/Actions/FinalAppraisal.cs
@@ -9,8 +9,8 @@ internal class FinalAppraisal : BaseAction
     public override int ActionId => 19012;
 
     public override int CPCost => 1;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
+    public override bool IncreasesStepCount => false;
 
     public override void UseSuccess() =>
         Simulation.AddEffect(Effect.FinalAppraisal, 5);
diff --git a/Craftimizer/Simulator/Actions/FocusedSynthesis.cs b/Craftimizer/Simulator/Actions/FocusedSynthesis.cs
index 123aeb5..82753ce 100644
--- a/Craftimizer/Simulator/Actions/FocusedSynthesis.cs
+++ b/Craftimizer/Simulator/Actions/FocusedSynthesis.cs
@@ -10,8 +10,6 @@ internal class FocusedSynthesis : BaseAction
 
     public override int CPCost => 5;
     public override float Efficiency => 2.00f;
+    public override bool IncreasesProgress => true;
     public override float SuccessRate => Simulation.GetPreviousAction() is Observe ? 1.00f : 0.50f;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseProgress(Efficiency);
 }
diff --git a/Craftimizer/Simulator/Actions/FocusedTouch.cs b/Craftimizer/Simulator/Actions/FocusedTouch.cs
index af6474f..5d13b6b 100644
--- a/Craftimizer/Simulator/Actions/FocusedTouch.cs
+++ b/Craftimizer/Simulator/Actions/FocusedTouch.cs
@@ -10,8 +10,6 @@ internal class FocusedTouch : BaseAction
 
     public override int CPCost => 18;
     public override float Efficiency => 1.50f;
+    public override bool IncreasesQuality => true;
     public override float SuccessRate => Simulation.GetPreviousAction() is Observe ? 1.00f : 0.50f;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseQuality(Efficiency);
 }
diff --git a/Craftimizer/Simulator/Actions/GreatStrides.cs b/Craftimizer/Simulator/Actions/GreatStrides.cs
index 0225b53..76d8eaf 100644
--- a/Craftimizer/Simulator/Actions/GreatStrides.cs
+++ b/Craftimizer/Simulator/Actions/GreatStrides.cs
@@ -9,7 +9,6 @@ internal class GreatStrides : BaseAction
     public override int ActionId => 260;
 
     public override int CPCost => 32;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
 
     public override void UseSuccess() =>
diff --git a/Craftimizer/Simulator/Actions/Groundwork.cs b/Craftimizer/Simulator/Actions/Groundwork.cs
index 446386f..0a01727 100644
--- a/Craftimizer/Simulator/Actions/Groundwork.cs
+++ b/Craftimizer/Simulator/Actions/Groundwork.cs
@@ -10,9 +10,14 @@ internal class Groundwork : BaseAction
 
     public override int CPCost => 18;
     // Groundwork Mastery Trait
-    public override float Efficiency => Simulation.Stats.Level >= 86 ? 3.60f : 3.00f;
+    public override float Efficiency
+    {
+        get
+        {
+            var ret = Simulation.Stats.Level >= 86 ? 3.60f : 3.00f;
+            return Simulation.Durability < DurabilityCost ? ret / 2 : ret;
+        }
+    }
+    public override bool IncreasesProgress => true;
     public override int DurabilityCost => 20;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseProgress(Efficiency);
 }
diff --git a/Craftimizer/Simulator/Actions/HastyTouch.cs b/Craftimizer/Simulator/Actions/HastyTouch.cs
index eac9550..2a920ba 100644
--- a/Craftimizer/Simulator/Actions/HastyTouch.cs
+++ b/Craftimizer/Simulator/Actions/HastyTouch.cs
@@ -10,8 +10,6 @@ internal class HastyTouch : BaseAction
 
     public override int CPCost => 0;
     public override float Efficiency => 1.00f;
+    public override bool IncreasesQuality => true;
     public override float SuccessRate => 0.60f;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseQuality(Efficiency);
 }
diff --git a/Craftimizer/Simulator/Actions/Innovation.cs b/Craftimizer/Simulator/Actions/Innovation.cs
index d3126f2..90b67ae 100644
--- a/Craftimizer/Simulator/Actions/Innovation.cs
+++ b/Craftimizer/Simulator/Actions/Innovation.cs
@@ -9,7 +9,6 @@ internal class Innovation : BaseAction
     public override int ActionId => 19004;
 
     public override int CPCost => 18;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
 
     public override void UseSuccess() =>
diff --git a/Craftimizer/Simulator/Actions/IntensiveSynthesis.cs b/Craftimizer/Simulator/Actions/IntensiveSynthesis.cs
index 311edd0..4bef0cb 100644
--- a/Craftimizer/Simulator/Actions/IntensiveSynthesis.cs
+++ b/Craftimizer/Simulator/Actions/IntensiveSynthesis.cs
@@ -10,11 +10,9 @@ internal class IntensiveSynthesis : BaseAction
 
     public override int CPCost => 6;
     public override float Efficiency => 4.00f;
+    public override bool IncreasesProgress => true;
 
     public override bool CanUse =>
         (Simulation.Condition == Condition.Good || Simulation.Condition == Condition.Excellent)
         && base.CanUse;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseProgress(Efficiency);
 }
diff --git a/Craftimizer/Simulator/Actions/Manipulation.cs b/Craftimizer/Simulator/Actions/Manipulation.cs
index 3941f75..ab88d94 100644
--- a/Craftimizer/Simulator/Actions/Manipulation.cs
+++ b/Craftimizer/Simulator/Actions/Manipulation.cs
@@ -9,7 +9,6 @@ internal class Manipulation : BaseAction
     public override int ActionId => 4574;
 
     public override int CPCost => 96;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
 
     public override void UseSuccess() =>
diff --git a/Craftimizer/Simulator/Actions/MastersMend.cs b/Craftimizer/Simulator/Actions/MastersMend.cs
index b999b90..7ce47c1 100644
--- a/Craftimizer/Simulator/Actions/MastersMend.cs
+++ b/Craftimizer/Simulator/Actions/MastersMend.cs
@@ -9,7 +9,6 @@ internal class MastersMend : BaseAction
     public override int ActionId => 100003;
 
     public override int CPCost => 88;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
 
     public override void UseSuccess() =>
diff --git a/Craftimizer/Simulator/Actions/MuscleMemory.cs b/Craftimizer/Simulator/Actions/MuscleMemory.cs
index 158a952..f8f28d8 100644
--- a/Craftimizer/Simulator/Actions/MuscleMemory.cs
+++ b/Craftimizer/Simulator/Actions/MuscleMemory.cs
@@ -10,12 +10,13 @@ internal class MuscleMemory : BaseAction
 
     public override int CPCost => 6;
     public override float Efficiency => 3.00f;
+    public override bool IncreasesProgress => true;
 
     public override bool CanUse => Simulation.IsFirstStep && base.CanUse;
 
     public override void UseSuccess()
     {
-        Simulation.IncreaseProgress(Efficiency);
+        base.UseSuccess();
         Simulation.AddEffect(Effect.MuscleMemory, 5);
     }
 }
diff --git a/Craftimizer/Simulator/Actions/Observe.cs b/Craftimizer/Simulator/Actions/Observe.cs
index f88cdde..a75d391 100644
--- a/Craftimizer/Simulator/Actions/Observe.cs
+++ b/Craftimizer/Simulator/Actions/Observe.cs
@@ -9,8 +9,5 @@ internal class Observe : BaseAction
     public override int ActionId => 100010;
 
     public override int CPCost => 7;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
-
-    public override void UseSuccess() { }
 }
diff --git a/Craftimizer/Simulator/Actions/PreciseTouch.cs b/Craftimizer/Simulator/Actions/PreciseTouch.cs
index a96f5ed..0e3d967 100644
--- a/Craftimizer/Simulator/Actions/PreciseTouch.cs
+++ b/Craftimizer/Simulator/Actions/PreciseTouch.cs
@@ -10,6 +10,7 @@ internal class PreciseTouch : BaseAction
 
     public override int CPCost => 18;
     public override float Efficiency => 1.50f;
+    public override bool IncreasesQuality => true;
 
     public override bool CanUse =>
         (Simulation.Condition == Condition.Good || Simulation.Condition == Condition.Excellent)
@@ -17,7 +18,7 @@ internal class PreciseTouch : BaseAction
 
     public override void UseSuccess()
     {
-        Simulation.IncreaseQuality(Efficiency);
+        base.UseSuccess();
         Simulation.StrengthenEffect(Effect.InnerQuiet);
     }
 }
diff --git a/Craftimizer/Simulator/Actions/PreparatoryTouch.cs b/Craftimizer/Simulator/Actions/PreparatoryTouch.cs
index d9e9126..c513ec9 100644
--- a/Craftimizer/Simulator/Actions/PreparatoryTouch.cs
+++ b/Craftimizer/Simulator/Actions/PreparatoryTouch.cs
@@ -10,11 +10,12 @@ internal class PreparatoryTouch : BaseAction
 
     public override int CPCost => 40;
     public override float Efficiency => 2.00f;
+    public override bool IncreasesQuality => true;
     public override int DurabilityCost => 20;
 
     public override void UseSuccess()
     {
-        Simulation.IncreaseQuality(Efficiency);
+        base.UseSuccess();
         Simulation.StrengthenEffect(Effect.InnerQuiet);
     }
 }
diff --git a/Craftimizer/Simulator/Actions/PrudentSynthesis.cs b/Craftimizer/Simulator/Actions/PrudentSynthesis.cs
index 5979470..5bbb1c5 100644
--- a/Craftimizer/Simulator/Actions/PrudentSynthesis.cs
+++ b/Craftimizer/Simulator/Actions/PrudentSynthesis.cs
@@ -10,12 +10,10 @@ internal class PrudentSynthesis : BaseAction
 
     public override int CPCost => 18;
     public override float Efficiency => 1.80f;
+    public override bool IncreasesProgress => true;
     public override int DurabilityCost => base.DurabilityCost / 2;
 
     public override bool CanUse =>
         !(Simulation.HasEffect(Effect.WasteNot) || Simulation.HasEffect(Effect.WasteNot2))
         && base.CanUse;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseProgress(Efficiency);
 }
diff --git a/Craftimizer/Simulator/Actions/PrudentTouch.cs b/Craftimizer/Simulator/Actions/PrudentTouch.cs
index fe5828b..f601894 100644
--- a/Craftimizer/Simulator/Actions/PrudentTouch.cs
+++ b/Craftimizer/Simulator/Actions/PrudentTouch.cs
@@ -10,12 +10,10 @@ internal class PrudentTouch : BaseAction
 
     public override int CPCost => 25;
     public override float Efficiency => 1.00f;
+    public override bool IncreasesQuality => true;
     public override int DurabilityCost => base.DurabilityCost / 2;
 
     public override bool CanUse =>
         !(Simulation.HasEffect(Effect.WasteNot) || Simulation.HasEffect(Effect.WasteNot2))
         && base.CanUse;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseQuality(Efficiency);
 }
diff --git a/Craftimizer/Simulator/Actions/RapidSynthesis.cs b/Craftimizer/Simulator/Actions/RapidSynthesis.cs
index 25bf241..0200858 100644
--- a/Craftimizer/Simulator/Actions/RapidSynthesis.cs
+++ b/Craftimizer/Simulator/Actions/RapidSynthesis.cs
@@ -11,8 +11,6 @@ internal class RapidSynthesis : BaseAction
     public override int CPCost => 0;
     // Rapid Synthesis Mastery Trait
     public override float Efficiency => Simulation.Stats.Level >= 63 ? 5.00f : 2.50f;
+    public override bool IncreasesProgress => true;
     public override float SuccessRate => 0.50f;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseProgress(Efficiency);
 }
diff --git a/Craftimizer/Simulator/Actions/Reflect.cs b/Craftimizer/Simulator/Actions/Reflect.cs
index 275802f..1228c37 100644
--- a/Craftimizer/Simulator/Actions/Reflect.cs
+++ b/Craftimizer/Simulator/Actions/Reflect.cs
@@ -10,12 +10,13 @@ internal class Reflect : BaseAction
 
     public override int CPCost => 6;
     public override float Efficiency => 1.00f;
+    public override bool IncreasesQuality => true;
 
     public override bool CanUse => Simulation.IsFirstStep && base.CanUse;
 
     public override void UseSuccess()
     {
-        Simulation.IncreaseQuality(Efficiency);
+        base.UseSuccess();
         Simulation.StrengthenEffect(Effect.InnerQuiet);
     }
 }
diff --git a/Craftimizer/Simulator/Actions/StandardTouch.cs b/Craftimizer/Simulator/Actions/StandardTouch.cs
index c22b922..bcd887d 100644
--- a/Craftimizer/Simulator/Actions/StandardTouch.cs
+++ b/Craftimizer/Simulator/Actions/StandardTouch.cs
@@ -10,7 +10,5 @@ internal class StandardTouch : BaseAction
 
     public override int CPCost => Simulation.GetPreviousAction() is BasicTouch ? 18 : 32;
     public override float Efficiency => 1.25f;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseQuality(Efficiency);
+    public override bool IncreasesQuality => true;
 }
diff --git a/Craftimizer/Simulator/Actions/TrainedEye.cs b/Craftimizer/Simulator/Actions/TrainedEye.cs
index d571c20..8e2207e 100644
--- a/Craftimizer/Simulator/Actions/TrainedEye.cs
+++ b/Craftimizer/Simulator/Actions/TrainedEye.cs
@@ -9,10 +9,10 @@ internal class TrainedEye : BaseAction
     public override int ActionId => 100283;
 
     public override int CPCost => 250;
-    public override float Efficiency => 0f;
+    public override bool IncreasesQuality => true;
 
     public override bool CanUse => Simulation.IsFirstStep && base.CanUse;
 
     public override void UseSuccess() =>
-        Simulation.IncreaseQuality(float.PositiveInfinity);
+        Simulation.IncreaseQualityRaw(Simulation.MaxQuality - Simulation.Quality);
 }
diff --git a/Craftimizer/Simulator/Actions/TrainedFinesse.cs b/Craftimizer/Simulator/Actions/TrainedFinesse.cs
index 15a0f4d..4640942 100644
--- a/Craftimizer/Simulator/Actions/TrainedFinesse.cs
+++ b/Craftimizer/Simulator/Actions/TrainedFinesse.cs
@@ -10,12 +10,10 @@ internal class TrainedFinesse : BaseAction
 
     public override int CPCost => 32;
     public override float Efficiency => 1.00f;
+    public override bool IncreasesQuality => true;
     public override int DurabilityCost => 0;
 
     public override bool CanUse =>
         (Simulation.GetEffect(Effect.InnerQuiet)?.Strength ?? 0) == 10
         && base.CanUse;
-
-    public override void UseSuccess() =>
-        Simulation.IncreaseProgress(Efficiency);
 }
diff --git a/Craftimizer/Simulator/Actions/TricksOfTheTrade.cs b/Craftimizer/Simulator/Actions/TricksOfTheTrade.cs
index e775e19..5085a94 100644
--- a/Craftimizer/Simulator/Actions/TricksOfTheTrade.cs
+++ b/Craftimizer/Simulator/Actions/TricksOfTheTrade.cs
@@ -9,7 +9,6 @@ internal class TricksOfTheTrade : BaseAction
     public override int ActionId => 100371;
 
     public override int CPCost => 0;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
 
     public override bool CanUse =>
diff --git a/Craftimizer/Simulator/Actions/Veneration.cs b/Craftimizer/Simulator/Actions/Veneration.cs
index a428c16..41fd1c9 100644
--- a/Craftimizer/Simulator/Actions/Veneration.cs
+++ b/Craftimizer/Simulator/Actions/Veneration.cs
@@ -9,7 +9,6 @@ internal class Veneration : BaseAction
     public override int ActionId => 19297;
 
     public override int CPCost => 18;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
 
     public override void UseSuccess() =>
diff --git a/Craftimizer/Simulator/Actions/WasteNot.cs b/Craftimizer/Simulator/Actions/WasteNot.cs
index c2bc238..45af866 100644
--- a/Craftimizer/Simulator/Actions/WasteNot.cs
+++ b/Craftimizer/Simulator/Actions/WasteNot.cs
@@ -9,7 +9,6 @@ internal class WasteNot : BaseAction
     public override int ActionId => 4631;
 
     public override int CPCost => 56;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
 
     public override void UseSuccess()
diff --git a/Craftimizer/Simulator/Actions/WasteNot2.cs b/Craftimizer/Simulator/Actions/WasteNot2.cs
index e9374be..78c2c64 100644
--- a/Craftimizer/Simulator/Actions/WasteNot2.cs
+++ b/Craftimizer/Simulator/Actions/WasteNot2.cs
@@ -9,7 +9,6 @@ internal class WasteNot2 : BaseAction
     public override int ActionId => 4639;
 
     public override int CPCost => 98;
-    public override float Efficiency => 0f;
     public override int DurabilityCost => 0;
 
     public override void UseSuccess()