adapter: fix orphaned list items when DEL + recreate in same Lua script

[Copilot]

When a Lua script deleted a key and then recreated it as a list in the same transaction, the delta commit path (preserveExisting=true) skipped deleteLogicalKeyElems, leaving the original list items orphaned in storage.

Root cause: listCommitPlan always returned preserveExisting=true for non-materialized (delta) lists. But if the key was previously deleted in the same script, listState never loaded the persisted meta (because deleted[key]=true made keyType return redisTypeNone), so the delta path wrote only the new items while the old storage entries were never cleaned up.

-- before fix: [a, b, c] items from original list leaked in storage
redis.call("DEL", KEYS[1])
redis.call("RPUSH", KEYS[1], "d")
-- LRANGE returned ["d"] but [a,b,c] remained as orphaned storage entries

Changes

• luaScriptContext: add everDeleted map[string]bool field, populated by deleteLogical and never reset (unlike deleted, which is cleared when commands recreate a key).
• listCommitPlan: when everDeleted[key] is true, force the full-rewrite/materialized path (preserveExisting=false) so deleteLogicalKeyElems runs and cleans up stale storage before writing the new list.
• TestRedis_LuaDelAndRecreateListNoOrphan: covers the DEL → RPUSH sequence to guard against regression.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[bootjp]

@copilot fix golangci-lint

[Copilot]

@copilot fix golangci-lint

Fixed in commit b4ab53e:

• gci: fixed missing tab indentation in the test function body of TestRedis_LuaDelAndRecreateListNoOrphan in adapter/redis_lua_compat_test.go
• mnd: replaced literal 2 in len(args) != 2 (SELECT arity checks) with a named constant selectArgCount = 2 defined in proxy/pubsub.go, used in both proxy/pubsub.go and proxy/proxy.go