Skip to content

Refactor CWE extraction logic to handle multiple descriptions - #35

Open
Darkiros wants to merge 1 commit into
mainfrom
Fix_missing_descriptions
Open

Refactor CWE extraction logic to handle multiple descriptions#35
Darkiros wants to merge 1 commit into
mainfrom
Fix_missing_descriptions

Conversation

@Darkiros

Copy link
Copy Markdown
Collaborator

Thank to @nal3x for the bug and the suggested code.
Issue #34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes CWE extraction from NVD CVE weakness blocks so that multiple CWE entries within a single weakness description array are no longer silently dropped (per Issue #34), improving downstream CWE→CAPEC/technique mapping completeness.

Changes:

  • Iterate over every description entry for Primary weakness blocks when extracting CWE IDs.
  • Apply the same multi-description iteration for Secondary weakness blocks when no Primary CWE is present.

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

Comment thread retrieve_cve.py Outdated
Comment thread retrieve_cve.py Outdated
@Darkiros
Darkiros force-pushed the Fix_missing_descriptions branch from d4a19d5 to e3b65f5 Compare July 21, 2026 13:37
@Darkiros
Darkiros force-pushed the Fix_missing_descriptions branch from e3b65f5 to 8a17d94 Compare July 21, 2026 13:40
@Darkiros Darkiros self-assigned this Jul 21, 2026
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