internal/app: copy parts of config to state

This is less error-prone than passing the address to the entire hst.Config struct, and reduces the likelihood of accidentally clobbering hst.Config. This also improves ease of testing.

Signed-off-by: Ophestra <cat@gensokyo.uk>

1 files changed, 1 insertions, 1 deletions

diff --git a/internal/app/finalise.go b/internal/app/finalise.go
index 62b380e4..0adfcb4e 100644
--- a/internal/app/finalise.go
+++ b/internal/app/finalise.go
@@ -81,7 +81,7 @@ func (k *outcome) finalise(ctx context.Context, msg message.Msg, id *state.ID, c
 	}
 
 	sys := system.New(k.ctx, msg, s.uid.unwrap())
-	if err := (&outcomeStateSys{config: config, sys: sys, outcomeState: &s}).toSystem(); err != nil {
+	if err := s.newSys(config, sys).toSystem(); err != nil {
 		return err
 	}