From 109aaee6593ac6e72d23b817e5b75292c2ed7bed Mon Sep 17 00:00:00 2001 From: Ophestra Date: Fri, 10 Oct 2025 03:19:09 +0900 Subject: 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 --- internal/app/finalise.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'internal/app/finalise.go') 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 } -- cgit v1.3.1