fix: import uuid - #6208
Merged
Merged
Conversation
NguyenHoangSon96
force-pushed
the
fix/uuid-import
branch
from
April 29, 2026 03:37
2b491c8 to
92e21aa
Compare
There was a problem hiding this comment.
Pull request overview
This PR updates the UI dependency stack to use modern uuid (with built-in TypeScript types) and upgrades Parcel, then updates all UI call sites to use the new uuid import style.
Changes:
- Remove
@types/uuidand upgradeuuidto^14.0.0, updating imports/usages across the UI. - Upgrade Parcel to
^2.16.4(plus related resolver config) and refreshyarn.lock. - Adjust Jest config so
uuidcan be transformed during tests.
Reviewed changes
Copilot reviewed 37 out of 38 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
yarn.lock |
Lockfile refresh reflecting uuid and Parcel upgrades and transitive dependency changes. |
ui/src/worker/utils/index.ts |
Switches UUID generation to v4 named import. |
ui/src/worker/JobManager.ts |
Switches UUID generation to v4 named import for worker job IDs. |
ui/src/utils/defaultQueryConfig.ts |
Uses uuidv4() for default query IDs. |
ui/src/tempVars/constants/index.ts |
Uses uuidv4() for default template IDs. |
ui/src/shared/utils/timeMachine.ts |
Uses uuidv4() when creating query drafts. |
ui/src/shared/reducers/notifications.ts |
Uses uuidv4() for notification IDs. |
ui/src/shared/parsing/flux/response.ts |
Uses uuidv4() for parsed Flux table IDs. |
ui/src/shared/constants/graphColorPalettes.ts |
Uses uuidv4() for color palette item IDs. |
ui/src/shared/components/time_series/TimeSeries.tsx |
Uses uuidv4() for request correlation IDs. |
ui/src/shared/components/threesizer/Threesizer.tsx |
Uses uuidv4() for generated division IDs. |
ui/src/shared/components/ThresholdsList.tsx |
Uses uuidv4() for new threshold IDs. |
ui/src/shared/components/TableGraphTransform.tsx |
Uses uuidv4() for per-render UUID passed to children. |
ui/src/shared/components/OptIn.tsx |
Uses uuidv4() for component instance ID. |
ui/src/shared/components/NewAnnotation.tsx |
Uses uuidv4() when creating new annotation IDs. |
ui/src/shared/components/ColorScaleDropdown.tsx |
Uses uuidv4() for React keys in dropdown rendering. |
ui/src/shared/components/AutoRefresh.tsx |
Uses uuidv4() for refresh IDs. |
ui/src/shared/components/AnnotationTagsDropdown.tsx |
Uses uuidv4() for new tag-filter IDs. |
ui/src/shared/components/AnnotationEditorForm.tsx |
Uses uuidv4() for tag IDs in the editor state. |
ui/src/shared/apis/metaQuery.js |
Migrates UUID import to uuidv4 (but currently contains a functional bug in the empty-response payload). |
ui/src/shared/annotations/helpers.ts |
Uses uuidv4() for new tag filter IDs. |
ui/src/logs/utils/search.ts |
Uses uuidv4() for filter IDs. |
ui/src/logs/utils/matchSections.ts |
Uses uuidv4() for match section IDs. |
ui/src/logs/utils/index.ts |
Uses uuidv4() for generated query config IDs. |
ui/src/logs/containers/LogsPage.tsx |
Uses uuidv4() when adding a filter. |
ui/src/kapacitor/containers/TickscriptPage.tsx |
Uses uuidv4() for log IDs/keys. |
ui/src/kapacitor/components/RuleGraph.tsx |
Uses uuidv4() for the uuid prop. |
ui/src/kapacitor/actions/view/index.js |
Uses uuidv4() for default rule query IDs. |
ui/src/flux/components/TimeMachineTables.tsx |
Uses uuidv4() for the uuid prop passed to transforms. |
ui/src/data_explorer/actions/view/index.ts |
Uses uuidv4() as the default query ID. |
ui/src/dashboards/utils/protoboardToDashboard.ts |
Uses uuidv4() for protoboard template IDs. |
ui/src/dashboards/components/GaugeOptions.tsx |
Uses uuidv4() for new threshold IDs. |
ui/src/admin/reducers/influxdb.js |
Uses uuidv4() for temporary database link IDs. |
ui/src/admin/actions/chronograf.js |
Uses uuidv4() for temporary IDs when creating entities. |
ui/package.json |
Bumps uuid to ^14.0.0, bumps Parcel deps, removes @types/uuid. |
ui/jest.config.js |
Allows Jest to transform uuid from node_modules. |
package.json |
Bumps Parcel dev deps; adds Parcel resolver config and supporting deps. |
CHANGELOG.md |
Adds changelog entry for dependency upgrades and type package removal. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 37 out of 38 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Closes PR
Changes