Skip to content

Replace refine_pr prompt with check_pr - a more assessment-focused approach #76

@Helmi

Description

@Helmi

Description

Transform the current refine_pr prompt into a new check_pr prompt that focuses on assessment and checking rather than automatically refining. The new prompt should be more open to the possibility that no changes might be needed.

Context

  • Project area: MCP Server implementation - prompt templates
  • Related prompts: Current refine_pr.yaml in /mcp-server/src/templates/prompts/
  • Complexity assessment: 5/10 (moderate - requires refactoring existing functionality)

Technical Details

Affected files:

  • /mcp-server/src/templates/prompts/refine_pr.yaml - Rename to check_pr.yaml and refactor content
  • /mcp-server/CHANGELOG.md - Update any references to refine_pr
  • Any documentation or code referencing the old prompt name

Requirements

  1. Rename prompt: Change refine_pr to check_pr to better reflect its purpose
  2. Assessment-first approach: The prompt should first check PR status before suggesting actions
  3. Support multiple PRs:
    • Accept optional PR number argument for checking specific PR
    • If no argument provided, check all open PRs in the repository
  4. Check CI/CD and reviews: Assess whether PRs have:
    • Passing CI/CD checks
    • Required reviews completed
    • Any blocking issues or feedback
  5. User confirmation required:
    • Only suggest work or merging after assessment
    • Require explicit user confirmation before:
      • Making any changes to address feedback
      • Merging PRs that are ready
  6. Flexible outcomes: Recognize that PRs might:
    • Be ready to merge as-is
    • Need minor adjustments
    • Require significant work
    • Be blocked by external factors

Implementation Steps

  • Create new check_pr.yaml based on existing refine_pr.yaml
  • Refactor prompt logic to be assessment-focused
  • Add logic to check all open PRs when no specific PR provided
  • Implement CI/CD status checking
  • Add review status assessment
  • Include user confirmation steps before actions
  • Update any references to old prompt name
  • Test the new prompt with various PR scenarios
  • Update documentation

Acceptance Criteria

  • New check_pr prompt exists and functions correctly
  • Old refine_pr references are removed or updated
  • Prompt can check single PR or all open PRs
  • Assessment happens before any action suggestions
  • User confirmation is required before changes or merging
  • Prompt handles various PR states gracefully

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions