Skip to content

Conversation

@nstrayer
Copy link
Contributor

Summary

Replaces the older .claude/commands/draft-pr-body.md command with a new positron-pr-helper Claude Code skill that provides a richer, more interactive workflow for creating PR bodies.

Key improvements:

  • Dynamic test tag fetching: Always pulls the latest e2e test tags from test-tags.ts instead of relying on a static, manually-maintained list
  • Interactive workflow: Guided question-and-answer flow to gather PR context (issue number, type, summary, screenshots)
  • Better categorization: Automatically categorizes test tags by type (feature, platform, performance, special)
  • Multiple output options: Copy to clipboard, update existing PR via gh, save to file, or display
  • Comprehensive templates: Separate templates for bug fixes, new features, UI changes, and maintenance PRs
  • Helper script: Includes fetch-test-tags.sh for standalone tag extraction

This command writing this PR body:

image

Release Notes

New Features

  • N/A

Bug Fixes

  • N/A

QA Notes

This is a development tooling improvement. To test:

  1. Invoke the skill: Please use the positron-pr-helper skill
  2. Answer the interactive questions about your PR
  3. Verify the generated PR body includes:
    • Proper formatting (Summary, Release Notes, QA Notes sections)
    • Current e2e test tags from test-tags.ts
    • Appropriate template based on PR type
  4. Test one of the output options (clipboard, file, or gh pr edit)

The skill should handle various PR types: bug fixes, new features, UI changes, and maintenance work.

@nstrayer nstrayer requested review from seeM, wch and wesm December 19, 2025 15:44
@nstrayer nstrayer marked this pull request as ready for review December 19, 2025 15:44
@github-actions
Copy link

github-actions bot commented Dec 19, 2025

E2E Tests 🚀
This PR will run tests tagged with: @:critical

readme  valid tags

seeM
seeM previously approved these changes Jan 5, 2026
Copy link
Contributor

@seeM seeM left a comment

Choose a reason for hiding this comment

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

LGTM, one minor change

Co-authored-by: Wasim Lorgat <mwlorgat@gmail.com>
Signed-off-by: Nick Strayer <nick.strayer@posit.co>
@nstrayer nstrayer merged commit 3bcd38b into main Jan 6, 2026
5 of 7 checks passed
@nstrayer nstrayer deleted the nstrayer/update-pr-body-tags branch January 6, 2026 15:02
@github-actions github-actions bot locked and limited conversation to collaborators Jan 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants