Conversation
[dev] [Marfuen] mariano/vendor-research
PR SummaryMedium Risk Overview Risk assessment rendering is reorganized and restyled. Removes the legacy Written by Cursor Bugbot for commit 4c9d53c. This will update automatically on new commits. Configure here. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
🎉 This PR is included in version 3.7.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| certifications={certifications} | ||
| verifiedCount={verifiedCount} | ||
| /> | ||
| )} |
There was a problem hiding this comment.
Guard uses raw count instead of filtered count
Low Severity
The visibility guard for VendorRiskAssessmentCertificationsCard checks certifications.length > 0 (raw, unfiltered array) instead of filteredCerts.length > 0. Since filterCertifications only keeps specific types (ISO 27001, ISO 42001, SOC 2, HIPAA), a vendor with other certification types (e.g. GDPR, PCI DSS) would cause the card to render with a "No certifications found" message. The already-computed filteredCerts variable is available but unused for this guard.
Additional Locations (1)
2 participants
This is an automated pull request to release the candidate branch into production, which will trigger a deployment.
It was created by the [Production PR] action.