Fixes several small styling issues, corrects ordering in the risk distribution trend, fixes several react and hydration errors, updates dependencies #613
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…tribution trend, fixes several react and hydration errors, updates dependencies Signed-off-by: Sebastian Kawelke <sebastian.kawelke@l3montree.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request addresses multiple small issues across the codebase, including styling improvements, bug fixes for React hydration errors, ordering corrections in visualizations, and dependency updates.
Changes:
- Fixed React hydration and controlled component issues in UI components (Input, Button, Dialog, etc.)
- Corrected risk distribution trend ordering from ascending to descending severity
- Added URL decoding support for package URLs and improved component styling
- Updated numerous dependencies to their latest versions
Reviewed changes
Copilot reviewed 21 out of 22 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/utils/common.ts | Added URL decoding to handle encoded characters in package URLs |
| src/pages/api/devguard-tunnel/[...path].ts | Increased response limit to 12MB for large dependency graphs |
| src/components/ui/input.tsx | Fixed controlled/uncontrolled component switching by ensuring value is never undefined |
| src/components/ui/input-with-button.tsx | Added check to make input readonly when no onChange handler is provided |
| src/components/ui/dialog.tsx | Properly destructured setOpen prop to avoid passing it in spread props |
| src/components/ui/button.tsx | Properly destructured isSubmitting prop to avoid passing it in spread props |
| src/components/risk-handling/LicenseRiskRow.tsx | Improved styling and added badge for unknown licenses |
| src/components/risk-assessment/RiskAssessmentFeedSkeleton.tsx | Fixed React key prop and replaced invalid p element with div |
| src/components/guides/risk-scanner-carousel-slides/ManualIntegrationSlide.tsx | Added upload state management with loading indicators |
| src/components/common/EcosystemImage.tsx | Fixed tooltip trigger by wrapping Image in span with asChild prop |
| src/components/VulnerableComponents.tsx | Made component items clickable links and added "See all" link |
| src/components/VulnEventItem.tsx | Improved spacing and sizing of ecosystem images |
| src/components/RiskHistoryDistributionDiagram.tsx | Fixed ordering to display risks from critical to low (descending) |
| src/components/DependencyGraphNode.tsx | Added asChild prop to DropdownMenuTrigger |
| src/app/.../license-risks/page.tsx | Updated section title for clarity |
| src/app/.../dependency-risks/page.tsx | Updated section title for clarity |
| src/app/.../dependency-risks/[vulnId]/page.tsx | Fixed invalid HTML structure and conditional rendering |
| src/app/.../dependencies/page.tsx | Improved ecosystem image styling |
| src/app/.../code-risks/page.tsx | Updated section title for clarity |
| src/app/.../assets/[assetSlug]/page.tsx | Fixed redirect logic by moving to useEffect hook |
| package.json | Updated multiple dependencies to latest versions |
| package-lock.json | Lockfile updates corresponding to package.json changes |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Sebastian Kawelke <sebastian.kawelke@l3montree.com>
Signed-off-by: Sebastian Kawelke <sebastian.kawelke@l3montree.com>
Signed-off-by: Sebastian Kawelke <sebastian.kawelke@l3montree.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.