pr-review-toolkit: drop AskUserQuestion preview field at posting approval - #96
Conversation
…oval The preview variant of AskUserQuestion is buggy, and the instruction also misdescribed the field (free-text notes are what the built-in "Other" option provides). The exact draft is already shown as prose before the question, so plain options lose nothing. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PqXbfR44bwfmPy4uNyf9ZU
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
🟢 Approval recommended
The focused documentation correction is complete and consistently versioned.
Pull request overview
Removes the buggy AskUserQuestion preview-field guidance while preserving approval behavior.
Changes:
- Uses plain approval options.
- Bumps plugin version to 2.0.1.
File summaries
| File | Description |
|---|---|
posting.md |
Updates posting approval guidance. |
plugin.json |
Applies the required patch bump. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Summary
skills/review-pr/references/posting.mdno longer instructs use of thepreviewfield on AskUserQuestion options — that variant of the UI is buggy, and the instruction also misdescribed the field (free-text notes are what the built-in "Other" option provides). The step now uses plain AskUserQuestion; the exact draft is already shown as prose before the question, and the four options and approval semantics are unchanged.pr-review-toolkitto 2.0.1 (patch).This was the only
preview-field reference in the repo; the other AskUserQuestion sites (address-pr-feedback,triage-prs) already use plain options.Validation
claude plugin validate .andclaude plugin validate ./pr-review-toolkitpassnpx markdownlint-cli2— 0 issuesuvx skillsaw --strict— all checks pass, grade A🤖 Generated with Claude Code
https://claude.ai/code/session_01PqXbfR44bwfmPy4uNyf9ZU
Generated by Claude Code