Skip to content

fix: correct /help project identity - #3367

Merged
IsmaelMartinez merged 1 commit into
The-PR-Agent:mainfrom
oleksii-tumanov:fix/help-project-identity
Sep 14, 2026
Merged

IsmaelMartinez merged 1 commit into
The-PR-Agent:mainfrom
oleksii-tumanov:fix/help-project-identity

Conversation

@oleksii-tumanov

Copy link
Copy Markdown
Contributor

Summary

The /help system prompt still says the open-source project was recently renamed to Qodo Merge, while the repository now identifies it as PR-Agent.

  • remove the stale rename claim from the runtime prompt
  • add regression coverage for the rendered project identity

Result

You are Doc-helper, a language model designed to answer questions about the open-source PR-Agent project.

Testing

  • PYTHONPATH=. uv run pytest -q tests/unittest/test_pr_help_message_rendering.py (17 passed)
  • PYTHONPATH=. uv run pytest -q tests/unittest (7890 passed, 33 skipped, 1 xfailed)
  • uv run pre-commit run --files pr_agent/settings/pr_help_prompts.toml tests/unittest/test_pr_help_message_rendering.py (passed)

@github-actions github-actions Bot added the bug label Sep 14, 2026
@qodo-free-for-open-source-projects

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review
  • Author self-review: I have reviewed the code review findings, and addressed the relevant ones.

Grey Divider

Tip of the day
💡 Did you know, you can ask Qodo to dismiss a finding you disagree with, with your reason on record

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

Copy link
Copy Markdown
Contributor

PR Summary by Qodo

Correct /help prompt project identity

🐞 Bug fix 🧪 Tests 🕐 Less than 10 minutes

Grey Divider

AI Description

• Correct /help to identify the open-source project as PR-Agent.
• Add regression coverage preventing stale rename language from returning.
Diagram

graph TD
  A["Help prompt config"] -->|loaded by| B["Prompt renderer"] -->|produces| C["System prompt"]
  D["Regression test"] -.->|validates identity| C
Loading
High-Level Assessment

The direct prompt correction plus a rendered-output regression test is the appropriate approach. A full prompt snapshot was considered but would be more brittle and less focused than asserting the required identity and forbidden stale wording.

Files changed (2) +8 / -1

Bug fix (1) +1 / -1
pr_help_prompts.tomlCorrect the '/help' system prompt identity +1/-1

Correct the '/help' system prompt identity

• Replaces the obsolete Qodo Merge rename statement with a concise description identifying PR-Agent as the open-source project.

pr_agent/settings/pr_help_prompts.toml

Tests (1) +7 / -0
test_pr_help_message_rendering.pyAdd project identity regression coverage +7/-0

Add project identity regression coverage

• Adds a rendered-prompt test requiring the PR-Agent identity and rejecting the stale “recently renamed” wording.

tests/unittest/test_pr_help_message_rendering.py

@IsmaelMartinez IsmaelMartinez left a comment

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.

Thanks for pulling the 2024 rename claim out of the /help prompt and pinning the rendered identity with a test. Approving.

The README now describes PR-Agent as a community-maintained project distinct from Qodo's offering, so that sentence was steering Doc-helper wrong, and it was the only such line left in the runtime prompts. Full suite green.

@IsmaelMartinez
IsmaelMartinez merged commit c1a4398 into The-PR-Agent:main Sep 14, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants