Skip to content

Fix stream-specific suffixes in CVE titles - #844

Merged
nforro merged 1 commit into
packit:mainfrom
nforro:titles
Sep 23, 2026
Merged

nforro merged 1 commit into
packit:mainfrom
nforro:titles

Conversation

@nforro

@nforro nforro commented Sep 23, 2026

Copy link
Copy Markdown
Member

No description provided.

@qodo-for-packit

Copy link
Copy Markdown

PR Summary by Qodo

Normalize stream-specific suffixes in CVE titles

🐞 Bug fix 🧪 Tests 🕐 10-20 Minutes

Grey Divider

AI Description

• Strip trailing [rhel-*] stream suffixes from canonical CVE merge request titles.
• Normalize legacy cached CVE titles without affecting embedded bracketed text.
• Cover live Jira and cached-title resolution paths with unit tests.
Diagram

graph TD
  A["Jira CVE summary"] --> B["Title resolver"] --> C["Suffix normalizer"] --> E["Canonical MR title"]
  D[("Redis cache")] --> B
  C --> D
Loading
High-Level Assessment

The targeted normalization helper is the appropriate approach because it handles both new Jira summaries and legacy cached titles without requiring cache migration or broadly stripping bracketed text elsewhere in a title.

Files changed (2) +49 / -3

Bug fix (1) +11 / -2
tasks.pyRemove trailing RHEL stream suffixes from CVE titles +11/-2

Remove trailing RHEL stream suffixes from CVE titles

• Adds targeted normalization for trailing '[rhel-*]' suffixes. Applies it to new CVE titles and cached CVE metadata while leaving non-CVE title generation unchanged.

ymir/agents/tasks.py

Tests (1) +38 / -1
test_tasks.pyTest CVE suffix normalization across title sources +38/-1

Test CVE suffix normalization across title sources

• Verifies trailing suffix removal, preservation of embedded bracketed text, live Jira title resolution, and cleanup of existing cached CVE titles.

ymir/agents/tests/unit/test_tasks.py

@qodo-for-packit

Copy link
Copy Markdown

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

Grey Divider

Tip of the day
💡 Did you know, you can keep summaries lean with Finding overflow, which tucks the rest behind 'View more'

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@lbarcziova lbarcziova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks!

Signed-off-by: Nikola Forró <nforro@redhat.com>
Assisted-by: GPT-5.6 Luna via OpenCode
@nforro
nforro merged commit 77e7d41 into packit:main Sep 23, 2026
15 checks passed
@nforro
nforro deleted the titles branch September 23, 2026 15:43
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.

2 participants