Skip to content

DEV: add TCEs to XADD command page#2932

Open
dwdougherty wants to merge 2 commits intomainfrom
DOC-5640
Open

DEV: add TCEs to XADD command page#2932
dwdougherty wants to merge 2 commits intomainfrom
DOC-5640

Conversation

@dwdougherty
Copy link
Copy Markdown
Collaborator

@dwdougherty dwdougherty commented Mar 24, 2026

Note

Low Risk
Low risk documentation/sample-only change, but it adds many new multi-language example files that could fail doc-sample compilation/tests if formatting or client APIs are off.

Overview
Updates content/commands/xadd.md to use clients-example blocks for XADD, covering both basic stream appends and Redis 8.6+ idempotent production via IDMP/IDMPAUTO (plus XCFGSET configuration).

Adds a new cmds_stream example set under local_examples/ (Go, Node, Python, Java/Jedis, Java/Lettuce async+reactive, Rust async+sync, and .NET) implementing the xadd1 and xadd2 steps with assertions/cleanup to support docs-driven testing.

Written by Cursor Bugbot for commit 8d9fdeb. This will update automatically on new commits. Configure here.

@dwdougherty dwdougherty requested review from a team and andy-stark-redis March 24, 2026 19:00
@dwdougherty dwdougherty self-assigned this Mar 24, 2026
@dwdougherty dwdougherty added cmds oss Redis Open Source labels Mar 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 24, 2026

DOC-5640

@github-actions
Copy link
Copy Markdown
Contributor

@jit-ci
Copy link
Copy Markdown

jit-ci bot commented Mar 24, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

Copy link
Copy Markdown
Collaborator

@mich-elle-luna mich-elle-luna left a comment

Choose a reason for hiding this comment

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

one question, otherwise seems fine though I can't review for techincal

Corrected the description in the clients-example for clarity.
@dwdougherty
Copy link
Copy Markdown
Collaborator Author

Thank you for the review and spotting the error, @mich-elle-luna!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants