fix: distinguish between hooks and channels in RENAME error message

The RENAME command was incorrectly returning "key has hooks set" when
a channel (not a hook) was associated with the key. This fix:

- Adds a new error errKeyHasChannelsSet for channel-specific errors
- Updates RENAME logic to check for hooks and channels separately
- Returns the appropriate error message based on what's blocking the rename
- Updates tests to expect "key has channels set" when channels are present

Fixes #764

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: majiayu000 <1835304752@qq.com>

Author: 2 people

internal/server/crud.go

@@ -479,17 +479,23 @@ func (s *Server) cmdRENAME(msg *Message) (resp.Value, commandDetails, error) {
 	if col == nil {
 		return retwerr(errKeyNotFound)
 	}
-	var ierr error
+	var hasHook, hasChannel bool
 	s.hooks.Ascend(nil, func(v interface{}) bool {
 		h := v.(*Hook)
 		if h.Key == key || h.Key == newKey {
-			ierr = errKeyHasHooksSet
-			return false
+			if h.channel {
+				hasChannel = true
+			} else {
+				hasHook = true
+			}
 		}
 		return true
 	})
-	if ierr != nil {
-		return retwerr(ierr)
+	if hasHook {
+		return retwerr(errKeyHasHooksSet)
+	}
+	if hasChannel {
+		return retwerr(errKeyHasChannelsSet)
 	}
 	var updated bool
 	newCol, _ := s.cols.Get(newKey)

internal/server/token.go

@@ -21,6 +21,7 @@ var errIDNotFound = errors.New("id not found")
 var errIDAlreadyExists = errors.New("id already exists")
 var errPathNotFound = errors.New("path not found")
 var errKeyHasHooksSet = errors.New("key has hooks set")
+var errKeyHasChannelsSet = errors.New("key has channels set")
 var errNotRectangle = errors.New("not a rectangle")
 
 func errInvalidArgument(arg string) error {

tests/keys_test.go

@@ -113,8 +113,8 @@ func keys_RENAME_test(mc *mockServer) error {
 		Do("RENAME", "foo", "mynewkey").Err("key not found"),
 		Do("SCAN", "mynewkey", "COUNT").Str("1"),
 		Do("SETCHAN", "mychan", "INTERSECTS", "mynewkey", "BOUNDS", 10, 10, 20, 20).Str("1"),
-		Do("RENAME", "mynewkey", "foo2").Err("key has hooks set"),
-		Do("RENAMENX", "mynewkey", "foo2").Err("key has hooks set"),
+		Do("RENAME", "mynewkey", "foo2").Err("key has channels set"),
+		Do("RENAMENX", "mynewkey", "foo2").Err("key has channels set"),
 		Do("SET", "mykey", "myid1", "HASH", "9my5xp7").OK(),
 		Do("RENAME", "mykey", "foo2").OK(),
 		Do("RENAMENX", "foo2", "foo3").Str("1"),