Skip to content

Add skill for cluster and replication#26

Merged
valkirilov merged 1 commit into
mainfrom
feature/redis-clustering
May 26, 2026
Merged

Add skill for cluster and replication#26
valkirilov merged 1 commit into
mainfrom
feature/redis-clustering

Conversation

@valkirilov
Copy link
Copy Markdown
Member

@valkirilov valkirilov commented May 22, 2026

Why

Continues the agentskills.io migration of redis-development's rules into focused spec-compliant skills (see #20#25).

This PR carves out the clustering and replication topic into its own skill so an agent has sharp triggers for "Redis Cluster", "hash tag", "CROSSSLOT", "read replica" instead of fishing through a 33-rule grab-bag.

Spec reference

What's in this PR

A single commit, additive only — no files deleted from skills/redis-development/rules/. The legacy redis-development skill continues to include the two cluster-* rules unchanged, so downstream plugin consumers keep working through the migration. Duplicates are removed in the final cleanup PR.

Two rules folded into `redis-clustering`:

SKILL.md has two sections, framed around the two failure modes that bite most new cluster users:

  • Hash tags for multi-key operations — only the {...} portion hashes to a slot; MGET / SDIFF / pipelines / transactions need all keys on the same slot or they error out with CROSSSLOT. Includes the guidance to scope tags to the meaningful entity ({user:1001}) rather than bare identifiers ({1001}) that collide across namespaces.
  • Read replicas for read-heavy workloadsread_from_replicas=True for Redis Cluster and standalone replica setups, with the eventually-consistent caveat made concrete: fine for caches / analytics / dashboards, not for balances or idempotency state.

Validation

  • skill-validator check skills/redis-clusteringpassed, 0 warnings.
  • npm run validate → rules, claude-plugins, cursor-plugins all green.
  • Branched from main, independent of the other migration PRs. Can merge in parallel.

Out of scope (follow-ups)

  • Removing the duplicate cluster-*.md source rules — happens in the final cleanup PR.
  • Authoring evals for clustering topics — separate work after the structural migration completes.

🤖 Generated with Claude Code


Note

Low Risk
Low risk because this PR is additive documentation/content only and does not change runtime code paths or behavior.

Overview
Introduces a new spec-compliant redis-clustering skill (SKILL.md + references/) that consolidates Redis Cluster key-design guidance around hash tags for multi-key operations (avoiding CROSSSLOT errors in pipelines/transactions/scripts) and routing reads to replicas for read-heavy workloads, including the eventual-consistency caveats.

Adds two standalone reference docs (references/hash-tags.md, references/read-replicas.md) with concrete Python/Java examples and links to official Redis docs.

Reviewed by Cursor Bugbot for commit d6e6f4b. Bugbot is set up for automated code reviews on this repo. Configure here.

…ication

Introduces skills/redis-clustering/ covering the two cluster-* rules
from skills/redis-development/rules/ in agentskills.io spec layout:

  cluster-hash-tags     → references/hash-tags.md
  cluster-read-replicas → references/read-replicas.md

SKILL.md frames the skill around the two failure modes that bite most
new cluster users: CROSSSLOT errors on multi-key operations (solved by
hash tags) and overloading primaries with read traffic (solved by
replica reads). Reference files carry the full Python and Java code
samples.

Additive only: the source cluster-*.md rules under skills/redis-development/
remain in place so the legacy compiled AGENTS.md continues to serve
existing plugin consumers unchanged. They are removed in the final
cleanup PR alongside the rest of the rules/ tree.

Validation:
- skill-validator check skills/redis-clustering → passed (0 warnings)
- npm run validate → rules + plugin validators green

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@valkirilov valkirilov changed the title feat(redis-clustering): add spec-compliant skill for cluster and replication Add skill for cluster and replication May 22, 2026
@jit-ci
Copy link
Copy Markdown

jit-ci Bot commented May 22, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@valkirilov valkirilov merged commit 9d76cea into main May 26, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants