Spell the sync-rename guard as one raise
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013VNdx5qiRSWADBoVfDRcrd
change
commit 672609e255e24e0be7d6234effa16c97d1ace57a
author Claude <noreply@anthropic.com>
date
parent d4bcaeb8
diff --git a/lib/pro_tacts/store.rb b/lib/pro_tacts/store.rb
index ddd11ac..f6524fb 100644
--- a/lib/pro_tacts/store.rb
+++ b/lib/pro_tacts/store.rb
@@ -964,9 +964,8 @@ module ProTacts
     #: (String id, String? name) -> void
     def refuse_sync_rename(id, name)
       was = groups.where(id:).sole.fetch(:name)&.to_s
-      return if was == group_name(name) || !Group.sync_name?(was)
-
-      raise SyncGroupRename, "#{was} is a book's group, named by the book"
+      renamed = was != group_name(name)
+      raise SyncGroupRename, "#{was} is a book's group, named by the book" if renamed && Group.sync_name?(was)
     end
 
     # The rename itself, with no refusal: #name_book's way of renaming