Skip to content

Conversation

@hannesrudolph
Copy link
Collaborator

@hannesrudolph hannesrudolph commented Jul 8, 2025

Important

Removes tool access for issue-fixer-orchestrator and pr-fixer-orchestrator modes in .roomodes.

  • Behavior:
    • Removes tool access for issue-fixer-orchestrator and pr-fixer-orchestrator modes by setting groups to an empty list in .roomodes.
  • Misc:
    • Minor whitespace change in .roomodes.

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

Moved the 'issue-fixer-orchestrator' mode below 'pr-fixer-orchestrator' and updated their 'groups' fields to empty lists for consistency. No functional changes to mode logic.
Copilot AI review requested due to automatic review settings July 8, 2025 23:47
@hannesrudolph hannesrudolph requested review from cte, jr and mrubens as code owners July 8, 2025 23:47
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 8, 2025
@delve-auditor
Copy link

delve-auditor bot commented Jul 8, 2025

No security or compliance issues detected. Reviewed everything up to a7fd789.

Security Overview
  • 🔎 Scanned files: 1 changed file(s)
Detected Code Changes
Change Type Relevant files
Refactor ► .roomodes
    Reordered orchestrator mode definitions
    Updated groups field to empty lists for consistency

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.

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 updates internal orchestrator mode definitions to remove tool access by clearing the groups field and adjusting the ordering.

  • Removed the original issue-fixer-orchestrator block and re-added it with empty groups.
  • Updated pr-fixer-orchestrator to disable tool-related groups and added source: project.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jul 9, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 9, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Draft / In Progress] in Roo Code Roadmap Jul 9, 2025
@hannesrudolph hannesrudolph added PR - Draft / In Progress and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Jul 9, 2025
@mrubens mrubens merged commit 2a3e40e into main Jul 9, 2025
34 of 42 checks passed
@github-project-automation github-project-automation bot moved this from PR [Draft / In Progress] to Done in Roo Code Roadmap Jul 9, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jul 9, 2025
@mrubens mrubens deleted the fix-orchestrators branch July 9, 2025 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer PR - Draft / In Progress size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants