Skip to content

feat(entities-plugins): group datakit node types in node selection panel#3035

Merged
Justineo merged 8 commits intomainfrom
feat/datakit-grouping-nodes
Mar 25, 2026
Merged

feat(entities-plugins): group datakit node types in node selection panel#3035
Justineo merged 8 commits intomainfrom
feat/datakit-grouping-nodes

Conversation

@Justineo
Copy link
Member

Summary

Before

image

After

image

@Justineo Justineo requested review from a team as code owners March 23, 2026 07:01
Copilot AI review requested due to automatic review settings March 23, 2026 07:01
Copy link
Contributor

Copilot AI left a comment

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 Datakit flow editor node selection panel to group config node types into labeled sections, improving scanability and navigation in the side panel.

Changes:

  • Adds localized group labels for the node selection panel.
  • Introduces new typing for config node groups and group metadata, and exposes grouped node lists for rendering.
  • Updates the node panel UI to render grouped sections and adjusts spacing in the side panel.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/entities/entities-plugins/src/locales/en.json Adds localized titles for node panel grouping.
packages/entities/entities-plugins/src/components/free-form/Datakit/types.ts Introduces ConfigNodeGroup and related group/meta types; factors out NodeColors.
packages/entities/entities-plugins/src/components/free-form/Datakit/flow-editor/side-panel/SidePanel.vue Adds spacing for the node panel when displayed in the side panel.
packages/entities/entities-plugins/src/components/free-form/Datakit/flow-editor/side-panel/NodePanel.vue Renders node types grouped into sections with group headings.
packages/entities/entities-plugins/src/components/free-form/Datakit/flow-editor/node/node.ts Builds grouped catalog/meta structures and adds helpers for group meta + visuals.
packages/entities/entities-plugins/src/components/free-form/Datakit/flow-editor/node/node-visual.ts Refactors config node visuals into icon map + group color map.
packages/entities/entities-plugins/src/components/free-form/Datakit/flow-editor/node/NodePanelItem.vue Pulls item colors from the node’s group meta instead of per-node meta.
packages/entities/entities-plugins/src/components/free-form/Datakit/flow-editor/node/NodeBadge.vue Switches badge visuals to use getNodeVisual (so config nodes can use group colors).
packages/entities/entities-plugins/src/components/free-form/Datakit/flow-editor/node/FlowNode.vue Uses localized node type name for tooltip text.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Justineo Justineo force-pushed the feat/datakit-grouping-nodes branch from ef32075 to 5d24416 Compare March 25, 2026 01:14
@Justineo Justineo enabled auto-merge (squash) March 25, 2026 03:31
Copy link
Member

@kingyue737 kingyue737 left a comment

Choose a reason for hiding this comment

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

LGTM

@Justineo Justineo merged commit 8971857 into main Mar 25, 2026
16 of 19 checks passed
@Justineo Justineo deleted the feat/datakit-grouping-nodes branch March 25, 2026 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants