Skip to content

pr review - #1126

Merged
umesh-more-cstk merged 1 commit into
feat/skillfrom
feat/skill-v1
Jul 30, 2026
Merged

pr review#1126
umesh-more-cstk merged 1 commit into
feat/skillfrom
feat/skill-v1

Conversation

@umesh-more-cstk

Copy link
Copy Markdown
Contributor

🔗 Jira Ticket

Replace with your ticket link — required before requesting review.

MIGRATION-XXXX


📋 PR Type

  • ✨ Feature
  • 🐛 Bug Fix
  • 🔥 Hotfix
  • ♻️ Refactor
  • 🧹 Chore / Dependency Update
  • 📝 Documentation

📝 Description

What changed?

Why?


🧩 Affected Areas

  • api — Node.js backend
  • ui — React frontend
  • upload-api — Upload API server
  • docker / docker-compose
  • CI / GitHub Actions workflows
  • Environment variables / config
  • Other:

🧪 How to Test

Expected result:


📸 Screenshots / Recordings

Before After

🔗 Related PRs / Dependencies


✅ Author Checklist

Complete this before moving the PR out of Draft.

  • Branch follows naming convention: feature/, bugfix/, or hotfix/ + 5–30 lowercase chars
  • Jira ticket linked above
  • Self-reviewed the diff — no debug logs, commented-out code, or TODOs left in
  • .env / example.env updated if new environment variables were added
  • No sensitive credentials or secrets committed
  • Existing tests pass locally (npm test)
  • New tests written (or not applicable — explain why)
  • README.md / docs updated if behaviour changed
  • Talisman pre-push scan passes (no secrets flagged)

👀 Reviewer Notes


Migration v2 · Docs · Issues

@umesh-more-cstk
umesh-more-cstk requested a review from a team as a code owner July 30, 2026 11:02
@umesh-more-cstk
umesh-more-cstk merged commit 701c670 into feat/skill Jul 30, 2026
4 of 9 checks passed
@umesh-more-cstk
umesh-more-cstk deleted the feat/skill-v1 branch July 30, 2026 11:02
@github-actions

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 4 0 10 ✅ Passed
🟠 High Severity 67 345 25 ❌ Failed
🟡 Medium Severity 174 1 500 ✅ Passed
🔵 Low Severity 7 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 4 0 15 / 30 days ❌ Failed
🟠 High 44 0 30 / 120 days ❌ Failed
🟡 Medium 7 0 90 / 365 days ❌ Failed
🔵 Low 0 0 180 / 365 days ✅ Passed

🔴 Critical Severity - SLA Breached Issues (with fixes)

Showing 4 issue(s) that have exceeded the 15-day SLA threshold:

  1. Deserialization of Untrusted Data

    • ID: SNYK-JS-REACTROUTER-17137394
    • Package: react-router@7.12.0
    • Published: 56 days ago (SLA: 15 days)
    • CVSS Score: 9.2
    • CVE: CVE-2026-42211
  2. Deserialization of Untrusted Data

    • ID: SNYK-JS-REACTROUTER-17137394
    • Package: react-router@7.12.0
    • Published: 56 days ago (SLA: 15 days)
    • CVSS Score: 9.2
    • CVE: CVE-2026-42211
  3. CRLF Injection

    • ID: SNYK-JS-UNDICI-17372658
    • Package: undici@7.24.6
    • Published: 42 days ago (SLA: 15 days)
    • CVSS Score: 9.2
    • CVE: CVE-2026-9679
  4. Resources Downloaded over Insecure Protocol

    • ID: SNYK-JS-ESBUILD-17750822
    • Package: esbuild@0.27.3
    • Published: 28 days ago (SLA: 15 days)
    • CVSS Score: 9.2

🟠 High Severity - SLA Breached Issues (with fixes)

Showing 20 issue(s) that have exceeded the 30-day SLA threshold:

  1. Directory Traversal

    • ID: SNYK-JS-TAR-15307072
    • Package: tar@6.2.1
    • Published: 162 days ago (SLA: 30 days)
    • CVSS Score: 8.4
    • CVE: CVE-2026-26960
  2. Symlink Attack

    • ID: SNYK-JS-TAR-15416075
    • Package: tar@6.2.1
    • Published: 147 days ago (SLA: 30 days)
    • CVSS Score: 8.2
    • CVE: CVE-2026-29786
  3. Symlink Attack

    • ID: SNYK-JS-TAR-15456201
    • Package: tar@6.2.1
    • Published: 141 days ago (SLA: 30 days)
    • CVSS Score: 8.2
    • CVE: CVE-2026-31802
  4. Interpretation Conflict

    • ID: SNYK-JS-FASTURI-16642394
    • Package: fast-uri@3.1.0
    • Published: 78 days ago (SLA: 30 days)
    • CVSS Score: 8.7
    • CVE: CVE-2026-6322
  5. Directory Traversal

    • ID: SNYK-JS-FASTURI-16642399
    • Package: fast-uri@3.1.0
    • Published: 78 days ago (SLA: 30 days)
    • CVSS Score: 8.7
    • CVE: CVE-2026-6321
  6. Interpretation Conflict

    • ID: SNYK-JS-FASTURI-16642394
    • Package: fast-uri@3.1.0
    • Published: 78 days ago (SLA: 30 days)
    • CVSS Score: 8.7
    • CVE: CVE-2026-6322
  7. Directory Traversal

    • ID: SNYK-JS-FASTURI-16642399
    • Package: fast-uri@3.1.0
    • Published: 78 days ago (SLA: 30 days)
    • CVSS Score: 8.7
    • CVE: CVE-2026-6321
  8. Allocation of Resources Without Limits or Throttling

    • ID: SNYK-JS-BRACEEXPANSION-16755174
    • Package: brace-expansion@5.0.5
    • Published: 72 days ago (SLA: 30 days)
    • CVSS Score: 8.7
    • CVE: CVE-2026-45149
  9. Allocation of Resources Without Limits or Throttling

    • ID: SNYK-JS-BRACEEXPANSION-16755174
    • Package: brace-expansion@5.0.5
    • Published: 72 days ago (SLA: 30 days)
    • CVSS Score: 8.7
    • CVE: CVE-2026-45149
  10. Server-side Request Forgery (SSRF)

  • ID: SNYK-JS-AXIOS-17111062
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7.7
  • CVE: CVE-2026-44492
  1. Prototype Pollution
  • ID: SNYK-JS-AXIOS-17111079
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7
  • CVE: CVE-2026-44494
  1. Server-side Request Forgery (SSRF)
  • ID: SNYK-JS-AXIOS-17111062
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7.7
  • CVE: CVE-2026-44492
  1. Prototype Pollution
  • ID: SNYK-JS-AXIOS-17111079
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7
  • CVE: CVE-2026-44494
  1. Server-side Request Forgery (SSRF)
  • ID: SNYK-JS-AXIOS-17111062
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7.7
  • CVE: CVE-2026-44492
  1. Prototype Pollution
  • ID: SNYK-JS-AXIOS-17111079
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7
  • CVE: CVE-2026-44494
  1. Server-side Request Forgery (SSRF)
  • ID: SNYK-JS-AXIOS-17111062
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7.7
  • CVE: CVE-2026-44492
  1. Prototype Pollution
  • ID: SNYK-JS-AXIOS-17111079
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7
  • CVE: CVE-2026-44494
  1. Server-side Request Forgery (SSRF)
  • ID: SNYK-JS-AXIOS-17111062
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7.7
  • CVE: CVE-2026-44492
  1. Prototype Pollution
  • ID: SNYK-JS-AXIOS-17111079
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7
  • CVE: CVE-2026-44494
  1. Server-side Request Forgery (SSRF)
  • ID: SNYK-JS-AXIOS-17111062
  • Package: axios@1.15.2
  • Published: 60 days ago (SLA: 30 days)
  • CVSS Score: 7.7
  • CVE: CVE-2026-44492

🟡 Medium Severity - SLA Breached Issues (with fixes)

Showing 7 issue(s) that have exceeded the 90-day SLA threshold:

  1. Regular Expression Denial of Service (ReDoS)

    • ID: SNYK-JS-SHOWDOWN-8685130
    • Package: showdown@1.9.1
    • Published: 542 days ago (SLA: 90 days)
    • CVSS Score: 6.9
    • CVE: CVE-2024-1899
  2. Regular Expression Denial of Service (ReDoS)

    • ID: SNYK-JS-SHOWDOWN-8685130
    • Package: showdown@1.9.1
    • Published: 542 days ago (SLA: 90 days)
    • CVSS Score: 6.9
    • CVE: CVE-2024-1899
  3. Improper Handling of Unicode Encoding

    • ID: SNYK-JS-TAR-15038581
    • Package: tar@6.2.1
    • Published: 191 days ago (SLA: 90 days)
    • CVSS Score: 6.4
    • CVE: CVE-2026-23950
  4. Directory Traversal

    • ID: SNYK-JS-TAR-15032660
    • Package: tar@6.2.1
    • Published: 185 days ago (SLA: 90 days)
    • CVSS Score: 6
    • CVE: CVE-2026-23745
  5. Directory Traversal

    • ID: SNYK-JS-TAR-15127355
    • Package: tar@6.2.1
    • Published: 182 days ago (SLA: 90 days)
    • CVSS Score: 6.2
    • CVE: CVE-2026-24842
  6. Improper Validation of Specified Index, Position, or Offset in Input

    • ID: SNYK-JS-UUID-16133035
    • Package: uuid@8.3.2
    • Published: 98 days ago (SLA: 90 days)
    • CVSS Score: 6.3
    • CVE: CVE-2026-41907
  7. Improper Validation of Specified Index, Position, or Offset in Input

    • ID: SNYK-JS-UUID-16133035
    • Package: uuid@8.3.2
    • Published: 98 days ago (SLA: 90 days)
    • CVSS Score: 6.3
    • CVE: CVE-2026-41907

ℹ️ 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: 345
  • Medium without fixes: 1
  • Low without fixes: 0

❌ BUILD FAILED - Security checks failed

Please review and fix the security vulnerabilities before merging.

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