Skip to content

SITE: update cmd_tools.py to handle drafting return info#3071

Merged
dwdougherty merged 3 commits intomainfrom
DOC-6501
Apr 22, 2026
Merged

SITE: update cmd_tools.py to handle drafting return info#3071
dwdougherty merged 3 commits intomainfrom
DOC-6501

Conversation

@dwdougherty
Copy link
Copy Markdown
Collaborator

@dwdougherty dwdougherty commented Apr 21, 2026

Note

Low Risk
Low risk: changes are limited to documentation generation logic and add tests, with no runtime impact beyond how command markdown is rendered.

Overview
Command page generation now populates the Return information section from command_data.reply_schema instead of always emitting TODO placeholders.

It adds RESP2/RESP3-specific schema-to-reply formatting (including oneOf nesting) with links to protocol spec anchors, fixes Hugo shortcode escaping in the multitabs block, and expands unit tests to cover the new mappings and fallback behavior.

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

@dwdougherty dwdougherty requested a review from a team April 21, 2026 20:23
@dwdougherty dwdougherty self-assigned this Apr 21, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2026

DOC-6501

@jit-ci
Copy link
Copy Markdown

jit-ci Bot commented Apr 21, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

Comment thread build/cmd_tools.py Outdated
Comment thread build/cmd_tools.py
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 86d5691. Configure here.

Comment thread build/cmd_tools.py
Comment thread build/cmd_tools.py
Copy link
Copy Markdown
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

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

Approved.

@dwdougherty
Copy link
Copy Markdown
Collaborator Author

Thank you, @andy-stark-redis!

@dwdougherty dwdougherty merged commit 8e43360 into main Apr 22, 2026
58 checks passed
@dwdougherty dwdougherty deleted the DOC-6501 branch April 22, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants