♻️ Refactor: update left navigation menu#3282
Open
xuyaqist wants to merge 13 commits into
Open
Conversation
…f-verification" field
xuyaqist
changed the title
Contributor
There was a problem hiding this comment.
Pull request overview
Refactors the left navigation/menu authorization model to support hierarchical menus (parent/child), updates seeded role permissions accordingly, and introduces new frontend routes/pages (e.g., MCP Space, Skill Space, Agent Space, Resource/Owner management) with updated i18n strings.
Changes:
- Add
parent_keytorole_permission_tand reseed/migrate LEFT_NAV_MENU permissions into a parent/child structure. - Update SideNavigation to render nested submenus based on accessible routes.
- Add/rename several UI pages (MCP Space, Skill Space, Agent Space, Resource/Owner manage) and update locale strings.
Reviewed changes
Copilot reviewed 10 out of 48 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| k8s/helm/nexent/charts/nexent-common/files/init.sql | Adds parent_key column and reseeds hierarchical LEFT_NAV_MENU permissions for Helm installs. |
| docker/init.sql | Adds parent_key column and reseeds hierarchical LEFT_NAV_MENU permissions for docker init. |
| docker/sql/v2.2.2_0622_update_left_nav_menu.sql | Migration script to clear/reinsert hierarchical LEFT_NAV_MENU permissions and add parent_key. |
| frontend/components/navigation/SideNavigation.tsx | Builds nested AntD menu structure using parentKey relationships. |
| frontend/public/locales/en/common.json | Adds sidebar/menu labels and Skill Space strings; updates MCP title. |
| frontend/public/locales/zh/common.json | Adds sidebar/menu labels and Skill Space strings; updates MCP title and space title. |
| frontend/app/[locale]/skill-space/page.tsx | New Skill Space “coming soon” page. |
| frontend/app/[locale]/mcp-space/page.tsx | New MCP Space page (imported/published tabs, filtering, detail modals). |
| frontend/app/[locale]/mcp-space/components/shared/TransportIcon.tsx | Shared icon mapping for MCP transport types. |
| frontend/app/[locale]/mcp-space/components/shared/TagEditor.tsx | Shared tag editing UI for MCP flows. |
| frontend/app/[locale]/mcp-space/components/shared/StatusBadge.tsx | Shared status badge component for MCP cards. |
| frontend/app/[locale]/mcp-space/components/shared/JsonPreviewModal.tsx | JSON preview modal for registry/config payloads. |
| frontend/app/[locale]/mcp-space/components/shared/ContainerPortField.tsx | Container port input + availability/suggestion UI. |
| frontend/app/[locale]/mcp-space/components/McpServicesFilterBar.tsx | Shared filter bar (source/transport/tag) for MCP lists. |
| frontend/app/[locale]/mcp-space/components/McpServiceCard.tsx | Card UI for imported MCP services. |
| frontend/app/[locale]/mcp-space/components/PublishedServiceCard.tsx | Card UI for “my published” MCP services. |
| frontend/app/[locale]/mcp-space/components/PublishedServiceDetailModal.tsx | Editable detail modal for published MCP services. |
| frontend/app/[locale]/mcp-space/components/PublishConfirmModal.tsx | Publish confirmation modal with override fields. |
| frontend/app/[locale]/mcp-space/components/add/AddMcpServiceModal.tsx | Add-service modal with Local/Registry/Community tabs. |
| frontend/app/[locale]/mcp-space/components/add/local/AddMcpServiceLocalSection.tsx | Local add flow (URL/container) with validation. |
| frontend/app/[locale]/mcp-space/components/add/registry/AddMcpServiceRegistrySection.tsx | Registry browser + quick-add picker modal. |
| frontend/app/[locale]/mcp-space/components/add/registry/McpRegistryToolbar.tsx | Registry toolbar (search/filters/updatedSince/includeDeleted). |
| frontend/app/[locale]/mcp-space/components/add/registry/McpRegistryCardList.tsx | Registry card list with paging controls. |
| frontend/app/[locale]/mcp-space/components/add/registry/McpRegistryCard.tsx | Registry service card UI. |
| frontend/app/[locale]/mcp-space/components/add/community/AddMcpServiceCommunitySection.tsx | Community browser + quick-add flow. |
| frontend/app/[locale]/mcp-space/components/add/community/McpCommunityToolbar.tsx | Community toolbar (search + transport/tag filters). |
| frontend/app/[locale]/mcp-space/components/add/community/McpCommunityCardList.tsx | Community card list with paging controls. |
| frontend/app/[locale]/mcp-space/components/add/community/McpCommunityCard.tsx | Community service card UI. |
| frontend/app/[locale]/mcp-space/components/add/community/McpCommunityDetailModal.tsx | Community detail modal + JSON previews. |
| frontend/app/[locale]/resource-manage/page.tsx | New entry page for Resource Manage route. |
| frontend/app/[locale]/resource-manage/components/resources/UserList.tsx | User list CRUD UI (edit/delete) + create-group modal. |
| frontend/app/[locale]/resource-manage/components/resources/GroupList.tsx | Group list CRUD + member management modal scaffolding. |
| frontend/app/[locale]/resource-manage/components/resources/KnowledgeList.tsx | Knowledge base list with edit + summary viewer modal. |
| frontend/app/[locale]/resource-manage/components/resources/ModelList.tsx | Tenant model list merged with monitoring metrics + dialogs. |
| frontend/app/[locale]/resource-manage/components/AssetOwnerResourcesComp.tsx | Asset owner management UI using resource-manage components. |
| frontend/app/[locale]/owner-manage/page.tsx | Points Owner Manage page to new AssetOwnerResourcesComp location. |
| frontend/app/[locale]/agents/components/agentInfo/AgentGenerateDetail.tsx | Adjusts layout spans in agent detail form section. |
| frontend/app/[locale]/agent-space/page.tsx | New Agent Space page (card grid + import wizard). |
| frontend/app/[locale]/agent-space/components/AgentCard.tsx | Agent card UI with delete/export/chat/detail actions. |
| frontend/app/[locale]/agent-space/components/AgentDetailModal.tsx | Agent detail modal with tabbed sections. |
Comments suppressed due to low confidence (1)
frontend/app/[locale]/skill-space/page.tsx:10
- The component signature uses an empty destructuring pattern (
({})), which will fail lint (no-empty-pattern). Remove the unused parameter.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+10
to
+11
| ALTER TABLE role_permission_t | ||
| ADD COLUMN IF NOT EXISTS parent_key VARCHAR(50); |
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.
Uh oh!
There was an error while loading. Please reload this page.