From cbee1422b42b7dcad85c400784a60a949cfd40b0 Mon Sep 17 00:00:00 2001
From: Colin Dawson <me@cjdawson.com>
Date: Fri, 13 Feb 2026 01:04:22 +0000
Subject: [PATCH] Split the non-essential parts out of the ITask, into ITags
 and IBudget

---
 e-suite.Workflow.Core/Interfaces/IBudget.cs | 36 +++++++++++++++++++
 e-suite.Workflow.Core/Interfaces/ITags.cs   |  6 ++++
 e-suite.Workflow.Core/Interfaces/ITask.cs   | 38 ++-------------------
 e-suite.Workflow.Core/TaskBase.cs           |  4 +--
 4 files changed, 46 insertions(+), 38 deletions(-)
 create mode 100644 e-suite.Workflow.Core/Interfaces/IBudget.cs
 create mode 100644 e-suite.Workflow.Core/Interfaces/ITags.cs

diff --git a/e-suite.Workflow.Core/Interfaces/IBudget.cs b/e-suite.Workflow.Core/Interfaces/IBudget.cs
new file mode 100644
index 0000000..effd483
--- /dev/null
+++ b/e-suite.Workflow.Core/Interfaces/IBudget.cs
@@ -0,0 +1,36 @@
+using e_suite.Workflow.Core.Enums;
+
+namespace e_suite.Workflow.Core.Interfaces;
+
+public interface IBudget
+{
+    /// <summary>
+    /// units of work involved in this item (UoW is a business term in this context)
+    /// </summary>
+    public decimal UnitsOfWork { get; set; }
+
+    /// <summary>
+    /// Cost of this work item
+    /// </summary>
+    public Cost Cost { get; set; }
+
+    /// <summary>
+    /// The option to show in budget.
+    /// </summary>
+    public BudgetOption BudgetOption { get; set; }
+
+    /// <summary>
+    /// Time tracking of this item
+    /// </summary>
+    public bool AllowTimeTracking { get; set; }
+
+    /// <summary>
+    /// Priority of this item
+    /// </summary>
+    public Priority Priority { get; set; }
+
+    /// <summary>
+    /// alloted duration of this item.
+    /// </summary>
+    public TimeSpan Duration { get; set; }
+}
\ No newline at end of file
diff --git a/e-suite.Workflow.Core/Interfaces/ITags.cs b/e-suite.Workflow.Core/Interfaces/ITags.cs
new file mode 100644
index 0000000..5e0bba5
--- /dev/null
+++ b/e-suite.Workflow.Core/Interfaces/ITags.cs
@@ -0,0 +1,6 @@
+namespace e_suite.Workflow.Core.Interfaces;
+
+public interface ITags
+{
+    public IList<string> Tags { get; set; }
+}
\ No newline at end of file
diff --git a/e-suite.Workflow.Core/Interfaces/ITask.cs b/e-suite.Workflow.Core/Interfaces/ITask.cs
index 61d5459..e65406a 100644
--- a/e-suite.Workflow.Core/Interfaces/ITask.cs
+++ b/e-suite.Workflow.Core/Interfaces/ITask.cs
@@ -1,6 +1,4 @@
-using e_suite.Workflow.Core.Enums;
-
-namespace e_suite.Workflow.Core.Interfaces;
+namespace e_suite.Workflow.Core.Interfaces;
 
 public interface ITask
 { 
@@ -27,40 +25,8 @@ public interface ITask
     /// <summary>
     /// List of tasks that need to be completed before this on can start.  (If empty, will start when the workflow starts)
     /// </summary>
-    IList<ITask> Predecessors { get; }
-
-    /// <summary>
-    /// units of work involved in this item (UoW is a business term in this context)
-    /// </summary>
-    public decimal UnitsOfWork { get; set; }
-
-    /// <summary>
-    /// Cost of this work item
-    /// </summary>
-    public Cost Cost { get; set; }
-
-    /// <summary>
-    /// The option to show in budget.
-    /// </summary>
-    public BudgetOption BudgetOption { get; set; }
-
-    /// <summary>
-    /// Time tracking of this item
-    /// </summary>
-    public bool AllowTimeTracking { get; set; }
-
-    /// <summary>
-    /// Priority of this item
-    /// </summary>
-    public Priority Priority { get; set; }
+    ITask? Predecessor { get; set;  }
     
-    /// <summary>
-    /// alloted duration of this item.
-    /// </summary>
-    public TimeSpan Duration { get; set; }
-    
-    public IList<string> Tags { get; set; }
-
     //Todo can only be on a run-time instance
     ///// <summary>
     ///// Called when the task status has been progressed from Pending to Active.
diff --git a/e-suite.Workflow.Core/TaskBase.cs b/e-suite.Workflow.Core/TaskBase.cs
index c7915b4..0e75863 100644
--- a/e-suite.Workflow.Core/TaskBase.cs
+++ b/e-suite.Workflow.Core/TaskBase.cs
@@ -3,13 +3,13 @@ using e_suite.Workflow.Core.Interfaces;
 
 namespace e_suite.Workflow.Core;
 
-public abstract class TaskBase : ITask
+public abstract class TaskBase : ITask, IBudget, ITags
 {
     public Guid Guid { get; set; } = Guid.CreateVersion7();
     public required ITask Parent { get; set; }
     public required string Name { get; set; }
     public string Description { get; set; } = string.Empty;
-    public IList<ITask> Predecessors { get; set; } = new List<ITask>();
+    public ITask? Predecessor { get; set; }
     public decimal UnitsOfWork { get; set; } = 1;
     public required Cost Cost { get; set; }
     public BudgetOption BudgetOption { get; set; } = BudgetOption.DoNotShow;