Skip to content

docs: add SonarQube/Codacy linter compliance rules#102

Merged
JE-Chen merged 3 commits intomainfrom
dev
Apr 18, 2026
Merged

docs: add SonarQube/Codacy linter compliance rules#102
JE-Chen merged 3 commits intomainfrom
dev

Conversation

@JE-Chen
Copy link
Copy Markdown
Member

@JE-Chen JE-Chen commented Apr 18, 2026

Summary

  • Add Linter Compliance section to CLAUDE.md covering SonarQube, Codacy, Pylint, Flake8, Bandit, Radon
  • Six rule groups: Complexity & Structure, Naming & Style, Bug-Prone Patterns, Type Safety, Security, Testing Hygiene
  • Codebase scanned against new rules — no violations except two framework-imposed exemptions (Locust request_hook signature, inline CSS in HTML template)

Test plan

  • Review added rules for accuracy against current Sonar/Codacy defaults
  • Confirm the two exemptions (request_hook params, embedded CSS lines) are acceptable

JE-Chen added 3 commits April 18, 2026 00:39
- Escape test record fields in HTML report to prevent injection
- Whitelist builtins registered on Executor; drop eval/exec/compile/__import__/open/input/breakpoint
- Re-raise invalid action_list instead of swallowing the error
- Promote per-call Lock() to module-level in json_file and json/html reports
- Drop unused server variable in load_density driver
@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@sonarqubecloud
Copy link
Copy Markdown

@JE-Chen JE-Chen merged commit 9e33088 into main Apr 18, 2026
14 checks passed
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.

1 participant