refactor: standardize Redis key and index naming to dash convention

[bsbodden]

Summary

• Rename all Redis key prefixes and index names from underscores to dashes (memory_idx → memory-idx, working_memory → working-memory, auth_token → auth-token, etc.) to align with RedisVL conventions
• Add migrate_redis_key_naming_4() migration function that uses SCAN+RENAME in batched pipelines and drops old indexes without data loss
• Add agent-memory migrate-redis-naming CLI command with --dry-run and --batch-size flags, plus automatic index rebuild after migration
• Update all tests, documentation, and notebook references to use the new naming

Closes #39

Status 🔔

‼️‼️ Pending upstream decision on API

Test plan

• uv run ruff check — all clean
• uv run ruff format — all clean
• uv run pre-commit run --all-files — all hooks pass
• uv run pytest — 703 tests pass, 0 failures
• agent-memory migrate-redis-naming --help — CLI command works
• Run agent-memory migrate-redis-naming --dry-run against a Redis instance with old-format keys
• Run agent-memory migrate-redis-naming against a Redis instance with old-format keys and verify keys are renamed
• Start server and verify index names are memory-records and working-memory-idx

[github-actions]

Review completed. Found one critical issue with the migration logic that could cause data loss.

[jit-ci]

🛡️ Jit Security Scan Results

✅ No security findings were detected in this PR

---

^{Security scan by Jit}

[Copilot]

Pull request overview

This PR standardizes Redis key prefixes and RediSearch/RedisVL index names from underscore (_) to dash (-) conventions across the server, migrations, CLI tooling, tests, and documentation—adding a dedicated migration path to rename existing keys and rebuild indexes.

Changes:

• Update Redis key/index naming conventions to dash-based forms (e.g., memory-idx:*, working-memory:*, memory-records, working-memory-idx).
• Add migration 4 to SCAN+RENAME existing keys and drop legacy index names, plus a CLI command to run it (with dry-run).
• Update tests and docs/notebook references to match the new naming.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
agent_memory_server/config.py	Updates default RedisVL and working-memory index naming to dash convention.
agent_memory_server/utils/keys.py	Updates key construction helpers to emit dash-based key prefixes.
agent_memory_server/working_memory.py	Updates migration status key names and scan pattern to new working-memory:* prefix.
agent_memory_server/working_memory_index.py	Updates working memory index docstring to reflect new prefix.
agent_memory_server/migrations.py	Adds migrate_redis_key_naming_4() to rename keys and drop legacy index names.
agent_memory_server/cli.py	Adds migrate-redis-naming CLI command to run migration 4 and rebuild indexes.
agent_memory_server/memory_vector_db.py	Updates comment to reflect new RedisVL index key prefix example.
docs/configuration.md	Updates documented env var defaults for new index name/prefix.
examples/agent_memory_server_interactive_guide.ipynb	Updates Redis key naming examples to dash convention (but includes one incorrect tool name).
tests/test_working_memory.py	Updates test cleanup scan patterns to working-memory:*.
tests/test_token_auth.py	Updates auth token key expectations to dash convention.
tests/test_long_term_memory.py	Updates memory index prefix expectation in assertions.
tests/benchmarks/test_migration_benchmark.py	Updates benchmark cleanup scan patterns to working-memory:*.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.