Skip to content

[bug_fix] Update prompt emphasize on generating patch with tests#6001

Open
suhaibmujahid wants to merge 1 commit intomozilla:masterfrom
suhaibmujahid:fix-and-test
Open

[bug_fix] Update prompt emphasize on generating patch with tests#6001
suhaibmujahid wants to merge 1 commit intomozilla:masterfrom
suhaibmujahid:fix-and-test

Conversation

@suhaibmujahid
Copy link
Copy Markdown
Member

@suhaibmujahid suhaibmujahid commented May 7, 2026

Copy link
Copy Markdown

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

Updates the bug-fix agent system prompt to shift from pure triage toward producing fix patches and validating them with tests, aligning the prompt with recent generated examples.

Changes:

  • Renames the agent role from “bug-triage” to “bug-fix” and updates the job statement to include fixing.
  • Adds guidance allowing source-repo edits to prepare/inspect a candidate patch.
  • Adds guidance to update/write tests and verify failure-before / pass-after.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread bugbug/tools/bug_fix/prompts/system.md Outdated
Comment thread bugbug/tools/bug_fix/prompts/system.md
Comment thread bugbug/tools/bug_fix/prompts/system.md Outdated

Source-repo edits (Write/Edit) are allowed so you can prepare and inspect a candidate patch.

Test your changes by updating existing tests or writing a new test if needed. If an existing test already covers the issue, you can just run it to verify that it fails before the fix and passes after.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Might want to add instructions to avoid "stupid" tests that Claude often generates, @padenot do you have any tips from your usage on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants