Merged
Conversation
michaeljguarino
approved these changes
Feb 21, 2026
Contributor
Greptile SummaryDependency cleanup and UI improvements reducing lockfile by ~4000 lines and addressing Dependabot alerts. Key changes:
Confidence Score: 5/5
|
| Filename | Overview |
|---|---|
| assets/design-system/package.json | Updated emotion packages, removed unused dependencies (firebase-tools, babel, testing libraries), moved type-fest to devDependencies |
| assets/package.json | Removed content-disposition dependency, bumped graphql-codegen packages and vite, updated fast-xml-parser |
| assets/src/utils/contentDisposition.ts | Added custom content-disposition parser to replace removed npm dependency |
| assets/src/utils/contentDisposition.test.ts | Added tests for content-disposition parser covering quoted filenames, UTF-8 encoding, and error cases |
| assets/src/components/security/compliance/ComplianceReports.tsx | Replaced content-disposition npm package with custom parseContentDispositionFilename utility |
| assets/src/components/cd/cluster/Cluster.tsx | Changed default export to named export, updated ClusterContextType to allow nullable cluster, removed LoadingIndicator for cleaner loading state |
| assets/src/components/cd/cluster/ClusterNodes.tsx | Replaced LoadingIndicator with Table loading prop, changed to named export, switched from navigate to Link, uses useClusterContext |
| assets/src/components/cd/cluster/ClusterPods.tsx | Replaced LoadingIndicator with Table loading prop, changed to named export, removed manual empty state handling |
| assets/yarn.lock | Lockfile updated with trimmed dependencies, removing 4000+ lines of unused packages |
Last reviewed commit: 470120c
470120c to
f790645
Compare
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
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.
should fix majority of dependabot alerts. bumps or removes a handful of design system dependencies that in a lot of cases either aren't used or can be provided directly from the console's package.json. also removes all the old design system ci/cd that's not used anymore (particularly firebase who's deps were causing a handful of alerts). if we ever wanted to add it back it should just be done with a plural flow
also does another handful of loading spinner replacements in some of the cluster drill down views (part of PROD-4466)
Plural Flow: console