Skip to content

fix: SCA scan permissions + dependency override bumps - #330

Merged
netrajpatel merged 3 commits into
developmentfrom
fix/sca-permissions-dep-overrides
Aug 4, 2026
Merged

fix: SCA scan permissions + dependency override bumps#330
netrajpatel merged 3 commits into
developmentfrom
fix/sca-permissions-dep-overrides

Conversation

@cs-raj

@cs-raj cs-raj commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • SCA scan workflow: Added permissions: contents: read and pull-requests: write to the security-sca job in .github/workflows/sca-scan.yml — required for Snyk to annotate PRs with vulnerability findings
  • Dependency overrides: Bumped four pinned overrides in pnpm-workspace.yaml to latest patch:
    • brace-expansion 5.0.8 → 5.0.9
    • js-yaml 5.2.2 → 5.2.3
    • fast-uri 4.1.1 → 4.1.2
    • ws 8.21.1 → 8.21.2
  • Lockfile: Updated pnpm-lock.yaml to reflect override changes

Test plan

  • SCA scan workflow triggers correctly and can write PR comments
  • pnpm install resolves cleanly with updated overrides
  • No regressions in unit tests

🤖 Generated with Claude Code

Aravind-Kumar-cstk and others added 3 commits August 2, 2026 17:39
Bumps pinned overrides in pnpm-workspace.yaml:
- brace-expansion 5.0.8 → 5.0.9
- js-yaml 5.2.2 → 5.2.3
- fast-uri 4.1.1 → 4.1.2
- ws 8.21.1 → 8.21.2

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cs-raj
cs-raj requested review from a team as code owners August 4, 2026 12:28
@snyk-io

snyk-io Bot commented Aug 4, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 1 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

⚠️ Warning: The following vulnerabilities have exceeded their SLA thresholds (days since publication).

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 1 90 / 365 days ⚠️ Warning
🔵 Low 0 0 180 / 365 days ✅ Passed

ℹ️ Vulnerabilities Without Available Fixes (Informational Only)

The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:

  • Critical without fixes: 0
  • High without fixes: 0
  • Medium without fixes: 1
  • Low without fixes: 0

⚠️ BUILD PASSED WITH WARNINGS - SLA breaches detected for issues without available fixes

Consider reviewing these vulnerabilities when fixes become available.

@netrajpatel
netrajpatel merged commit e2a6756 into development Aug 4, 2026
11 checks passed
@netrajpatel
netrajpatel deleted the fix/sca-permissions-dep-overrides branch August 4, 2026 12:31
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.

6 participants