Skip to content

DOC-7171 Update TCE testing notes: Lettuce pins and captured stdout - #4245

Merged
andy-stark-redis merged 2 commits into
mainfrom
DOC-7171-tce-testing-notes
Oct 13, 2026
Merged

andy-stark-redis merged 2 commits into
mainfrom
DOC-7171-tce-testing-notes

Conversation

@andy-stark-redis

@andy-stark-redis andy-stark-redis commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Updates .claude/skills/tce-examples/reference/testing.md with two lessons from the HOTKEYS examples in #4241:

  • Java versions table: it said portable mode pins all Lettuce flavors at 6.5.5. In fact only pom-lettuce-sync.xml does; async and reactive pin 7.7.0. The table now shows the split and says to test lettuce-sync with --fidelity whenever an example needs a newer API.
  • Captured stdout: cargo test (Rust) and xunit (C#) capture printed output, so the harness log can't be compared with the >>> comments. For those clients, the REMOVE asserts are the only check on the comments.

This changes only AI-facing skill documentation. No site content changes.

🤖 Generated with Claude Code


Note

Low Risk
Documentation-only change to an internal Claude skill file; no runtime, CI, or published docs behavior.

Overview
Updates the TCE examples testing skill doc (.claude/skills/tce-examples/reference/testing.md) with two clarifications from recent example work—no harness or site changes.

Documents an unguarded trap: Rust cargo test and C# xunit capture stdout, so harness logs cannot be checked against >>> comment expectations; authors must put every printed value under REMOVE asserts for those clients.

Corrects the Java portable pins table: Lettuce is split into lettuce-sync (6.5.5) vs lettuce-async / reactive (7.7.0), replacing a single “lettuce-core” column that understated the async/reactive portable versions. Fidelity pins stay aligned at 7.4.0 across Lettuce flavors.

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

Promotes two lessons from the HOTKEYS examples (PR #4241). The Java divergence table
said portable Lettuce was 6.5.5, but only pom-lettuce-sync.xml is; async and reactive
are 7.7.0, so lettuce-sync can't compile 7.x APIs in portable mode. Also notes that
cargo test and xunit capture stdout, so REMOVE asserts are the only check on Rust and
C# >>> comments.

Ticket: DOC-7171
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

DOC-7171

@github-actions

github-actions Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

🧠 Redis Memory

Found 9 related items from repository history (4 new this commit):

Memory updated at eb9b3c0

@dwdougherty dwdougherty left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

… Known gap section

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@andy-stark-redis
andy-stark-redis merged commit 3124022 into main Oct 13, 2026
100 checks passed
@andy-stark-redis
andy-stark-redis deleted the DOC-7171-tce-testing-notes branch October 13, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clients Client library docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants