refactor(daemon): simplify target membership check

commit: 4441ea376e21a61a2a6adb3335b1efa33eb8ac0d
author: brice · 2mo ago · 2026-07-28 16:20 UTC
parent: 52e1724
signature: GPG VerifiedCD88F9E564E0102646FB6077A1A07C7C7C3E349D · arch-gpg Verified for the committer email

1 file · +3 -8

M internal/usecase/daemon/layout_mutation.go +3 -8

@@ -1,6 +1,8 @@
1	1		package daemon
2	2
3	3		import (
	4	+	"slices"
	5	+
4	6		"github.com/bnema/vev/internal/domain"
5	7		"github.com/bnema/vev/internal/usecase/layout"
6	8		)

@@ -31,14 +33,7 @@ func (d *Daemon) mutateTargetLayoutChanged(target daemonActionTarget, requirePan
31	33		return false, layout.ErrNotFound
32	34		}
33	35		target.session.mu.Lock()
34		-	foundTab := false
35		-	for _, tb := range target.session.tabs {
36		-	if tb == target.tab {
37		-	foundTab = true
38		-	break
39		-	}
40		-	}
41		-	if !foundTab {
	36	+	if !slices.Contains(target.session.tabs, target.tab) {
42	37		target.session.mu.Unlock()
43	38		d.mu.Unlock()
44	39		return false, layout.ErrNotFound

 package daemon
 import (
+	"slices"
 	"github.com/bnema/vev/internal/domain"
 	"github.com/bnema/vev/internal/usecase/layout"
 )
 		return false, layout.ErrNotFound
 	}
 	target.session.mu.Lock()
-	foundTab := false
+	if !slices.Contains(target.session.tabs, target.tab) {
-	for _, tb := range target.session.tabs {
-		if tb == target.tab {
-			foundTab = true
-			break
-		}
-	}
-	if !foundTab {
 		target.session.mu.Unlock()
 		d.mu.Unlock()
 		return false, layout.ErrNotFound