-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Labels
enhancementNew feature or requestNew feature or request
Description
Problem
CodeQL workflow was temporarily removed in PR #23 due to persistent extension pack failures. The pr-diff-range extension pack was generating "undefined" values for the restrictAlertsTo predicate, causing analysis to fail.
Error Details
ERROR: In extension for codeql/util:restrictAlertsTo, row X is invalid.
Found '"undefined", "undefined", "undefined"', which does not match the signature
'restrictAlertsTo(string filePath, int lineStart, int lineEnd)'.
Attempted Fixes
- ✅ Replaced Autobuild with manual
cargo build --all-features - ✅ Added explicit
queries: +security-and-quality - ✅ Added
fetch-depth: 0for full git history - ❌ None resolved the extension pack issue
Current Status
- CodeQL workflow removed to unblock CI
- Other security measures remain: cargo-deny, clippy, tests
- Need to investigate and re-implement CodeQL analysis
Solution Options
- Disable PR filtering entirely - Run CodeQL on full codebase without diff filtering
- Use older CodeQL action version - Try version without pr-diff-range extension
- Custom CodeQL configuration - Manual setup without problematic extensions
- Wait for upstream fix - Monitor GitHub's codeql-action for extension pack fixes
Acceptance Criteria
- CodeQL analysis runs successfully on PRs and main branch
- No "undefined" values in restrictAlertsTo predicate
- Security analysis covers Rust code comprehensively
- CI pipeline remains fast and reliable
Priority
Medium - Security analysis is valuable but not blocking development
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request