Skip to content

fix: import uuid - #6208

Merged
NguyenHoangSon96 merged 3 commits into
masterfrom
fix/uuid-import
Apr 29, 2026
Merged

fix: import uuid#6208
NguyenHoangSon96 merged 3 commits into
masterfrom
fix/uuid-import

Conversation

@NguyenHoangSon96

@NguyenHoangSon96 NguyenHoangSon96 commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Closes PR

Changes

  1. Remove "@types/uuid" because newer "uuid" versions are included types in the package.
  2. Upgrade "uuid" package to "^14.0.0".
  3. Upgrade "@parcel/core" version to "^2.16.4".
  • CHANGELOG.md updated with a link to the PR (not the Issue)
  • Well-formatted commit messages
  • Rebased/mergeable
  • Tests pass
  • swagger.json updated (if modified Go structs or API)
  • Sign CLA (if not already signed)

@NguyenHoangSon96 NguyenHoangSon96 self-assigned this Apr 28, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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/uuid and upgrade uuid to ^14.0.0, updating imports/usages across the UI.
  • Upgrade Parcel to ^2.16.4 (plus related resolver config) and refresh yarn.lock.
  • Adjust Jest config so uuid can 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.

Comment thread ui/src/shared/apis/metaQuery.js Outdated

@alespour alespour left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread ui/package.json
Comment thread package.json
@NguyenHoangSon96
NguyenHoangSon96 merged commit 8e2157a into master Apr 29, 2026
7 checks passed
@NguyenHoangSon96
NguyenHoangSon96 deleted the fix/uuid-import branch April 29, 2026 07:08
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.

3 participants