Skip to content

fix: remediate code scanning alerts#15

Merged
aallam merged 1 commit intomainfrom
fix/code-scanning-alerts
Apr 13, 2026
Merged

fix: remediate code scanning alerts#15
aallam merged 1 commit intomainfrom
fix/code-scanning-alerts

Conversation

@aallam
Copy link
Copy Markdown
Owner

@aallam aallam commented Apr 13, 2026

Summary

  • add explicit contents: read permissions to the CI workflow so the GITHUB_TOKEN uses least privilege
  • replace the edge-trimming regex in sanitizeIdentifier with a deterministic underscore scan to address the CodeQL polynomial ReDoS finding
  • add regression coverage for identifier sanitization behavior and for the CI workflow permissions block

@aallam aallam marked this pull request as ready for review April 13, 2026 09:28
@aallam aallam merged commit 4ccd3b2 into main Apr 13, 2026
8 checks passed
@aallam aallam deleted the fix/code-scanning-alerts branch April 13, 2026 09:28
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