Refactor regex to use concise wildcard

[saichethana28]

Proposed change

Resolves #3013

This PR updates a regular expression in
backend/apps/owasp/models/common.py to make it easier to read and maintain.

The existing pattern used [\s\S] to match any character. This has been replaced with the equivalent wildcard . , which is clearer and follows common regex best practices.

Why this change?

• Fixes a low-severity code smell reported by SonarCloud
• Improves readability while keeping the behaviour unchanged
• Keeps the change small and focused on a single line

Checklist

• Required: I read and followed the contributing guidelines
• Required: I ran make check-test locally and all tests passed
• I used AI for code, documentation, or tests in this PR

[coderabbitai]

Summary by CodeRabbit

• Chores
  • Internal code optimization with no impact to user-facing functionality.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

A regex pattern in the metadata extraction function was simplified by replacing [\s\S] with . to improve readability while maintaining identical behavior under the existing re.DOTALL flag.

Changes

Cohort / File(s)	Summary
Regex simplification backend/apps/owasp/models/common.py	Refactored get_metadata regex pattern from r"^---\s*([\s\S]*?)\s*---" to r"^---\s*(.*?)\s*---", replacing the verbose character class [\s\S] with the concise . metacharacter

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

• arkid15r
• kasya

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: refactoring a regex pattern to use a more concise wildcard syntax, which aligns with the PR's primary objective.
Description check	✅ Passed	The description is directly related to the changeset, explaining the regex pattern change, the rationale (SonarCloud code smell fix), and confirming behavior remains unchanged.
Linked Issues check	✅ Passed	The PR fully implements the requirement from issue #3013: replacing [\\s\\S] with . in the regex pattern in backend/apps/owasp/models/common.py to fix the SonarCloud code smell.
Out of Scope Changes check	✅ Passed	All changes are directly related to the linked issue objective; only the regex pattern in the get_metadata function was modified with no unrelated alterations.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 36d4902 and c42fc94.

📒 Files selected for processing (1)

• backend/apps/owasp/models/common.py

🔇 Additional comments (1)

backend/apps/owasp/models/common.py (1)

239-239: Regex pattern simplification is correct and improves code readability.

The change from [\s\S]*? to .*? is functionally equivalent when re.DOTALL is used (line 241), as both patterns match any character including newlines. The new pattern is more idiomatic, follows Python regex best practices, and appropriately addresses the SonarCloud code smell. Tests in backend/tests/apps/owasp/models/common_test.py validate the method works correctly with multiline YAML content.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[saichethana28]

Hi @arkid15r @kasya
Just checking in to see if everything looks okay with this PR or if there’s anything I should update.