Fix renaming consistency between DB and runtime state

.gitignore

@@ -1 +1,2 @@
 dist/
+assets/artifacts/*.zip

internal/core/agents/agentlist.go

@@ -59,8 +59,8 @@ func (agents *Agents) Create(config *config.Config, session *yamux.Session) (*Ag
 }
 
 func (agents *Agents) Rename(oldAlias, newAlias string) error {
-	agents.mutex.RLock()
-	defer agents.mutex.RUnlock()
+	agents.mutex.Lock()
+	defer agents.mutex.Unlock()
 
 	agent := agents.active[oldAlias]
 	if agent == nil {

internal/core/tuns/tunlist.go

@@ -127,17 +127,18 @@ func (tuns *Tuns) Rename(oldAlias, newAlias string) error {
 	defer tuns.mutex.Unlock()
 
 	if tun, ok := tuns.active[oldAlias]; ok {
+		if err := tuns.store.RenameTun(oldAlias, newAlias); err != nil {
+			slog.Warn("Couldn't rename tun in storage",
+				slog.Any("reason", err),
+			)
+			return err
+		}
+
 		tun.Alias = newAlias
 
 		events.EventStream <- events.Event{Type: events.TunRenamed, Data: *tun}
 	}
 
-	if err := tuns.store.RenameTun(oldAlias, newAlias); err != nil {
-		slog.Warn("Couldn't rename tun from storage",
-			slog.Any("reason", err),
-		)
-	}
-
 	return nil
 }