Skip to content

Conversation

@hannesrudolph
Copy link
Collaborator

@hannesrudolph hannesrudolph commented Jul 1, 2025

Description

This PR updates the issue-fixer mode to use GitHub CLI (gh) for all GitHub operations instead of the GitHub MCP server. This simplifies the setup and removes external dependencies.

Changes Made

Mode Configuration Updates

  • Removed mcp from the groups list in .roomodes
  • Updated roleDefinition to mention GitHub CLI usage
  • Made the mode work with any GitHub repository (not just RooCodeInc/Roo-Code)
  • Simplified whenToUse description to focus only on issue fixing

Workflow Updates

  • All GitHub operations now use gh commands:
    • gh issue view for retrieving issues
    • gh issue view --comments for getting comments
    • gh pr create for creating pull requests
    • gh pr checks --watch for monitoring CI/CD
    • gh repo fork for forking when needed
    • gh api for advanced operations

File Changes

  • Renamed 4_github_mcp_tool_usage.xml to 4_github_cli_usage.xml with comprehensive CLI documentation
  • Updated 5_pull_request_workflow.xml to use dynamic repository references
  • Removed pr_comments section from 8_github_communication_guidelines.xml
  • Removed 9_pr_review_workflow.xml (functionality moved to pr-fixer mode)

Benefits

  1. No External Dependencies: Only requires standard GitHub CLI tool
  2. Simpler Setup: Users authenticate once with gh auth login
  3. Works with Any Repository: Dynamically extracts repo info from issue URLs
  4. Clearer Scope: Mode focuses exclusively on issue fixing

Testing

  • Verified all XML files are well-formed
  • Checked that all GitHub CLI commands are properly documented
  • Ensured mode configuration is valid
  • All lint checks pass

Checklist

  • Code follows project style guidelines
  • Self-review completed
  • Documentation updated
  • No breaking changes for existing users

Important

Refactor issue-fixer mode to use GitHub CLI instead of MCP, updating workflows and documentation for flexibility and simplicity.

  • Mode Configuration:
    • Removed mcp from .roomodes groups list.
    • Updated roleDefinition to use GitHub CLI.
    • Mode now works with any GitHub repository.
  • Workflow Updates:
    • Replaced MCP operations with GitHub CLI commands in 1_Workflow.xml and 5_pull_request_workflow.xml.
    • Commands include gh issue view, gh pr create, gh pr checks, and more.
  • File Changes:
    • Renamed 4_github_mcp_tool_usage.xml to 4_github_cli_usage.xml.
    • Updated 5_pull_request_workflow.xml for dynamic repo references.
    • Removed pr_comments from 8_github_communication_guidelines.xml.
    • Deleted 9_pr_review_workflow.xml (moved to pr-fixer mode).

This description was created by Ellipsis for 29eed2c. You can customize this summary. It will automatically update as commits are pushed.

- Remove MCP dependency from issue-fixer mode configuration
- Update all GitHub operations to use gh CLI commands
- Make mode work with any GitHub repository (dynamic owner/repo)
- Remove PR review functionality (moved to pr-fixer mode)
- Update documentation to reflect GitHub CLI usage
- Simplify authentication to use standard gh auth login
Copilot AI review requested due to automatic review settings July 1, 2025 20:48
@hannesrudolph hannesrudolph requested review from cte, jr and mrubens as code owners July 1, 2025 20:48
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. documentation Improvements or additions to documentation labels Jul 1, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the issue-fixer mode to replace all GitHub MCP operations with the GitHub CLI, simplifies configuration, and broadens repository support.

  • Removed MCP-based workflows and references
  • Updated mode settings to use gh for issue retrieval, commenting, and PRs
  • Added comprehensive CLI usage documentation and dynamic repo parsing

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.roomodes Switched mode description to use GitHub CLI, removed mcp group
.roo/rules-issue-fixer/9_pr_review_workflow.xml Deleted old MCP-based PR review workflow
.roo/rules-issue-fixer/8_github_communication_guidelines.xml Removed PR comment guidelines; retained issue comments
.roo/rules-issue-fixer/5_pull_request_workflow.xml Added steps and commands to create and monitor PRs via gh
.roo/rules-issue-fixer/4_github_mcp_tool_usage.xml Removed deprecated MCP tool usage documentation
.roo/rules-issue-fixer/4_github_cli_usage.xml Added full GitHub CLI command reference and examples
.roo/rules-issue-fixer/1_Workflow.xml Streamlined initial workflow to fetch issues via gh
Comments suppressed due to low confidence (1)

.roo/rules-issue-fixer/4_github_cli_usage.xml:68

  • The syntax declaration omits the --limit flag shown in examples. Update it to gh search code "[search-query]" --repo [owner]/[repo] --limit [number] for consistency with usage examples.
      <syntax>gh search code "[search-query]" --repo [owner]/[repo]</syntax>

@delve-auditor
Copy link

delve-auditor bot commented Jul 1, 2025

No security or compliance issues detected. Reviewed everything up to 29eed2c.

Security Overview
  • 🔎 Scanned files: 7 changed file(s)
Detected Code Changes
Change Type Relevant files
Refactor ► 1_Workflow.xml
    Update GitHub operations to use gh CLI instead of MCP tools
► 4_github_cli_usage.xml
    Add new GitHub CLI usage documentation
► 4_github_mcp_tool_usage.xml
    Remove MCP tool usage documentation
► 5_pull_request_workflow.xml
    Update PR workflow for GitHub CLI
► 8_github_communication_guidelines.xml
    Remove PR comments section
► 9_pr_review_workflow.xml
    Remove PR review workflow
► .roomodes
    Update mode description and capabilities

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Jul 1, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jul 1, 2025
@mrubens mrubens merged commit 1acbd45 into main Jul 1, 2025
23 checks passed
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 1, 2025
@mrubens mrubens deleted the refactor/issue-fixer-github-cli branch July 1, 2025 21:04
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jul 1, 2025
@github-project-automation github-project-automation bot moved this from PR [Needs Prelim Review] to Done in Roo Code Roadmap Jul 1, 2025
utarn pushed a commit to modelharbor/ModelHarbor-Agent that referenced this pull request Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants