fix: Missing input validation in GitLab CI

[mrwind-up-bird]

AutoFix: Missing input validation in GitLab CI

Category: security
Severity: medium

Issue

GitLab CI configuration uses shell commands with variables like CI_MERGE_REQUEST_DIFF_BASE_SHA without validation. These could potentially contain malicious input leading to command injection.

Fix

The fix sanitizes GitLab CI variables (CI_MERGE_REQUEST_DIFF_BASE_SHA and CI_COMMIT_SHA) by filtering out any non-hexadecimal characters using printf and tr commands. This prevents command injection attacks by ensuring only valid Git SHA hashes (containing only a-f and 0-9 characters) are used in shell commands. The fix is applied consistently across all instances where these variables are used in git diff and ckb impact commands.

---

Generated by nyxCore AutoFix

[codecov]

Codecov Report

❌ Patch coverage is 53.79630% with 998 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
cmd/ckb/setup.go	0.0%	161 Missing ⚠️
internal/suggest/analyzer.go	53.6%	142 Missing and 12 partials ⚠️
internal/testgap/analyzer.go	12.3%	140 Missing and 2 partials ⚠️
internal/query/compound_refactor.go	37.1%	105 Missing and 15 partials ⚠️
internal/query/prepare_rename.go	0.0%	70 Missing ⚠️
internal/mcp/server.go	19.1%	51 Missing and 4 partials ⚠️
internal/mcp/tool_impls_compound.go	46.3%	41 Missing and 10 partials ⚠️
internal/query/prepare_move.go	63.1%	32 Missing and 6 partials ⚠️
internal/audit/analyzer.go	21.0%	30 Missing ⚠️
internal/query/compound.go	25.0%	24 Missing and 3 partials ⚠️
... and 13 more

Additional details and impacted files

@@            Coverage Diff            @@
##           develop    #149     +/-   ##
=========================================
+ Coverage     45.1%   45.6%   +0.5%     
=========================================
  Files          350     367     +17     
  Lines        59747   61934   +2187     
=========================================
+ Hits         26961   28285   +1324     
- Misses       30960   31736    +776     
- Partials      1826    1913     +87     

Flag	Coverage Δ
unit	45.6% <53.7%> (+0.5%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.