Fix CI

Author: Nigel Mukandi

.github/workflows/main.yaml

@@ -1,13 +1,18 @@
 name: main
 
+run-name: "${{ github.event.head_commit.message }} — SHA: ${{ github.sha }}"
+
 on:
   push: 
     branches:
       - main
   workflow_dispatch: 
 
+env: 
+  DOTNET_VERSION: '9.0'
+  
 jobs: 
-  test:
+  build:
     runs-on: ubuntu-latest
     steps:
       - name: checkout repository
@@ -16,7 +21,7 @@ jobs:
       - name: setup dotnet environment
         uses: actions/setup-dotnet@d4c94342e560b34958eacfc5d055d21461ed1c5d
         with:
-          dotnet-version: '9.0'
+          dotnet-version: ${{ env.DOTNET_VERSION }}
 
       - run: dotnet --info    
 
@@ -32,7 +37,8 @@ jobs:
   publish-artifacts:
     runs-on: ubuntu-latest
     name: publish artifacts
-    needs: test
+    needs: build
+    environment: production
     steps:
       - name: checkout code
         uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
@@ -42,7 +48,7 @@ jobs:
       - name: setup dotnet
         uses: actions/setup-dotnet@d4c94342e560b34958eacfc5d055d21461ed1c5d
         with:
-          dotnet-version: '9.0'
+          dotnet-version: ${{ env.DOTNET_VERSION }}
 
       - name: dotnet info
         run: dotnet --info

.github/workflows/pull-request.yaml

@@ -1,16 +1,21 @@
 name: pull request
+
+run-name: "PR #${{ github.event.pull_request.number }} — ${{ github.event.pull_request.title }}"
+
 on:
   pull_request: 
     branches:
       - main
 
 jobs: 
-  test:
+  build:
     runs-on: ubuntu-latest
     steps:
       - name: checkout repository
         uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8
-        
+        with: 
+          fetch-depth: '1'
+      
       - name: setup dotnet
         uses: actions/setup-dotnet@d4c94342e560b34958eacfc5d055d21461ed1c5d
         with:

.github/workflows/release.yaml

@@ -28,8 +28,8 @@ jobs:
         run: dotnet build --configuration Release
 
       - name: Extract version from tag
-          id: get_version
-          run: |
+        id: get_version
+        run: |
             # Remove the "refs/tags/" prefix from GITHUB_REF
             TAG=${GITHUB_REF#refs/tags/}
             # Remove leading 'v' if tag looks like "v1.2.3"

FiniteStateMachine/FiniteStateMachineBuilder.cs

@@ -10,7 +10,7 @@ public sealed class FiniteStateMachineBuilder<TState, TTrigger>
     where TState : notnull
 {
     private readonly TState _initialState;
-    private readonly Dictionary<TState, Dictionary<TTrigger, TState>> _transitions = new();
+    private readonly Dictionary<TState, Dictionary<TTrigger, TransitionOption<TTrigger, TState>>> _transitions = new();
 
     /// <summary>
     /// Initializes a new instance of the <see cref="FiniteStateMachineBuilder{TState, TTrigger}"/> class.
@@ -52,7 +52,7 @@ private Dictionary<RuleKey<TState, TTrigger>, Rule<TState, TTrigger>> GetTransit
         var rules = new Dictionary<RuleKey<TState, TTrigger>, Rule<TState, TTrigger>>();
         foreach (var (fromState, transitions) in _transitions)
         {
-            foreach (var (trigger, targetState) in transitions)
+            foreach (var (trigger, transitionOption) in transitions)
             {
                 var ruleKey = new RuleKey<TState, TTrigger>
                 {
@@ -62,8 +62,10 @@ private Dictionary<RuleKey<TState, TTrigger>, Rule<TState, TTrigger>> GetTransit
                 var rule = new Rule<TState, TTrigger>
                 {
                     From = fromState,
-                    To = targetState,
-                    Trigger = trigger
+                    To = transitionOption.TargetState,
+                    Trigger = trigger,
+                    EntryActions = transitionOption.EntryActions,
+                    ExitActions = transitionOption.ExitActions,
                 };
                 rules.Add(ruleKey, rule);
             }

FiniteStateMachine/Rule.cs

@@ -6,4 +6,8 @@ public sealed record Rule<TState, TTrigger>
     public required TState From { get; init; }
     public required TState To { get; init; }
     public required TTrigger Trigger { get; init; }
+
+    public required List<Action> EntryActions { get; init; }
+    
+    public required List<Action> ExitActions { get; init; }
 }

FiniteStateMachine/StateConfiguration.cs

@@ -8,11 +8,17 @@ namespace FiniteStateMachine;
 public sealed class StateConfiguration<TState, TTrigger> where TTrigger : Enum
 {
     private readonly TState _state;
+    
+    private readonly List<Action> _entryActions;
+    private readonly List<Action> _exitActions;
+    
+    internal IReadOnlyList<Action> EntryActions => _entryActions;
+    internal IReadOnlyList<Action> ExitActions => _exitActions;
 
     /// <summary>
     /// Gets the transitions defined for the current state.
     /// </summary>
-    internal Dictionary<TTrigger, TState> Transitions { get; } = new();
+    internal Dictionary<TTrigger, TransitionOption<TTrigger, TState>> Transitions { get; } = new();
 
     /// <summary>
     /// Initializes a new instance of the <see cref="StateConfiguration{TState, TTrigger}"/> class.
@@ -21,6 +27,14 @@ public sealed class StateConfiguration<TState, TTrigger> where TTrigger : Enum
     internal StateConfiguration(TState state)
     {
         _state = state;
+        _entryActions = new List<Action>();
+        _exitActions = new List<Action>();
+    }
+
+    internal void ClearActions()
+    {
+        _entryActions.Clear();
+        _exitActions.Clear();
     }
 
     /// <summary>
@@ -30,6 +44,19 @@ internal StateConfiguration(TState state)
     /// <returns>A configuration object to setup the transition.</returns>
     public TransitionConfiguration<TState, TTrigger> On(TTrigger trigger)
     {
+        
         return new TransitionConfiguration<TState, TTrigger>(this, trigger);
     }
+    
+    public StateConfiguration<TState, TTrigger> OnEnter(Action action)
+    {
+        _entryActions.Add(action);
+        return this;
+    }
+    
+    public StateConfiguration<TState, TTrigger> OnExit(Action action)
+    {
+        _exitActions.Add(action);
+        return this;
+    }
 }

FiniteStateMachine/StateMachine.cs

@@ -57,7 +57,13 @@ public bool Trigger(TTrigger trigger)
             Trigger = trigger
         };
         if (!_rules.TryGetValue(key, out var rule)) return false;
+        
+        rule.ExitActions.ForEach(action =>  action());
+        
         CurrentState = rule.To;
+        
+        rule.EntryActions.ForEach(action => action());
+        
         return true;
     }
 }

FiniteStateMachine/TransitionConfiguration.cs

@@ -13,7 +13,16 @@ internal TransitionConfiguration(StateConfiguration<TState, TTrigger> stateConfi
 
     public StateConfiguration<TState, TTrigger> GoTo(TState target)
     {
-        _stateConfiguration.Transitions[_trigger] = target;
+        _stateConfiguration.Transitions[_trigger] = new TransitionOption<TTrigger, TState>
+        {
+            Trigger = _trigger,
+            TargetState = target,
+            EntryActions = _stateConfiguration.EntryActions.ToList(),
+            ExitActions = _stateConfiguration.ExitActions.ToList()
+        };
+        
+        _stateConfiguration.ClearActions();
+        
         return _stateConfiguration;
     }
 }

FiniteStateMachine/TransitionOption.cs

@@ -0,0 +1,9 @@
+namespace FiniteStateMachine;
+
+internal sealed record TransitionOption<TTrigger, TState>
+{
+    public required TTrigger Trigger { get; init; }
+    public required TState TargetState { get; init; }
+    public required List<Action> EntryActions { get; init; }
+    public required List<Action> ExitActions { get; init; }
+}

FiniteStateMachineTest/EnumState.cs FiniteStateMachineTest/State.csFiniteStateMachineTest/EnumState.cs renamed to FiniteStateMachineTest/State.cs

@@ -1,6 +1,6 @@
 namespace FiniteStateMachineTest;
 
-internal enum EnumState
+internal enum State
 {
     Playing,
     Paused,