diff --git a/open-lens/package.json b/open-lens/package.json index b59087e902bd..1e6b844e0dcc 100644 --- a/open-lens/package.json +++ b/open-lens/package.json @@ -179,7 +179,6 @@ "dependencies": { "@k8slens/application": "^6.5.0-alpha.8", "@k8slens/application-for-electron-main": "^6.5.0-alpha.7", - "@k8slens/cluster-sidebar": "^1.0.0-alpha.0", "@k8slens/core": "^6.5.0-alpha.13", "@k8slens/ensure-binaries": "^6.5.0-alpha.6", "@k8slens/event-emitter": "^1.0.0-alpha.5", diff --git a/open-lens/src/main/index.ts b/open-lens/src/main/index.ts index 6eaac3e1db82..190fbe7201ea 100644 --- a/open-lens/src/main/index.ts +++ b/open-lens/src/main/index.ts @@ -25,14 +25,13 @@ runInAction(() => { registerLensCore(di, environment); registerFeature(di, - loggerFeature, + loggerFeature, ); - registerFeature( - di, - applicationFeature, - applicationFeatureForElectronMain, - messagingFeatureForMain, + registerFeature(di, + applicationFeature, + applicationFeatureForElectronMain, + messagingFeatureForMain, ); try { diff --git a/open-lens/src/renderer/index.ts b/open-lens/src/renderer/index.ts index 8035c7c027f3..5981151da924 100644 --- a/open-lens/src/renderer/index.ts +++ b/open-lens/src/renderer/index.ts @@ -25,7 +25,6 @@ import { keyboardShortcutsFeature } from "@k8slens/keyboard-shortcuts"; import { reactApplicationFeature } from "@k8slens/react-application"; import { routingFeature } from "@k8slens/routing"; import { loggerFeature } from "@k8slens/logger"; -import { clusterSidebarFeature } from "@k8slens/cluster-sidebar"; const environment = "renderer"; @@ -50,8 +49,7 @@ runInAction(() => { keyboardShortcutsFeature, reactApplicationFeature, routingFeature, - metricsFeature, - clusterSidebarFeature, + metricsFeature ); autoRegister({ diff --git a/package-lock.json b/package-lock.json index 8fa3077115cc..f02b483e4aee 100644 --- a/package-lock.json +++ b/package-lock.json @@ -3959,10 +3959,6 @@ "resolved": "packages/cluster-settings", "link": true }, - "node_modules/@k8slens/cluster-sidebar": { - "resolved": "packages/cluster-sidebar", - "link": true - }, "node_modules/@k8slens/computed-channel": { "resolved": "packages/technical-features/messaging/computed-channel", "link": true @@ -34925,7 +34921,6 @@ "dependencies": { "@k8slens/application": "^6.5.0-alpha.8", "@k8slens/application-for-electron-main": "^6.5.0-alpha.7", - "@k8slens/cluster-sidebar": "^1.0.0-alpha.0", "@k8slens/core": "^6.5.0-alpha.13", "@k8slens/ensure-binaries": "^6.5.0-alpha.6", "@k8slens/event-emitter": "^1.0.0-alpha.5", @@ -35136,26 +35131,6 @@ "@ogre-tools/injectable": "^16.1.0" } }, - "packages/cluster-sidebar": { - "name": "@k8slens/cluster-sidebar", - "version": "1.0.0-alpha.0", - "license": "MIT", - "devDependencies": { - "@k8slens/eslint-config": "^6.5.0-alpha.3", - "@k8slens/jest": "^6.5.0-alpha.5", - "@k8slens/typescript": "^6.5.0-alpha.2", - "@k8slens/webpack": "^6.5.0-alpha.5" - }, - "peerDependencies": { - "@k8slens/feature-core": "^6.5.0-alpha.4", - "@k8slens/utilities": "^1.0.0-alpha.3", - "@ogre-tools/injectable": "^15.8.1", - "@ogre-tools/injectable-extension-for-auto-registration": "^15.8.1", - "@ogre-tools/injectable-extension-for-mobx": "^15.8.1", - "mobx": "^6.9.0", - "react": "^17.0.2" - } - }, "packages/core": { "name": "@k8slens/core", "version": "6.5.0-alpha.13", @@ -35351,7 +35326,6 @@ "@k8slens/application-for-electron-main": "^6.5.0-alpha.0", "@k8slens/button": "^1.0.0-alpha.5", "@k8slens/cluster-settings": "^6.5.0-alpha.1", - "@k8slens/cluster-sidebar": "^1.0.0-alpha.0", "@k8slens/error-boundary": "^1.0.0-alpha.5", "@k8slens/event-emitter": "^1.0.0-alpha.1", "@k8slens/kubectl-versions": "^1.0.0-alpha.0", diff --git a/packages/cluster-sidebar/.eslintrc.js b/packages/cluster-sidebar/.eslintrc.js deleted file mode 100644 index f404cf0ace57..000000000000 --- a/packages/cluster-sidebar/.eslintrc.js +++ /dev/null @@ -1,6 +0,0 @@ -module.exports = { - extends: "@k8slens/eslint-config/eslint", - parserOptions: { - project: "./tsconfig.json", - }, - }; diff --git a/packages/cluster-sidebar/.prettierrc b/packages/cluster-sidebar/.prettierrc deleted file mode 100644 index edd47b479e93..000000000000 --- a/packages/cluster-sidebar/.prettierrc +++ /dev/null @@ -1 +0,0 @@ -"@k8slens/eslint-config/prettier" diff --git a/packages/cluster-sidebar/index.ts b/packages/cluster-sidebar/index.ts deleted file mode 100644 index 4d072e91b2a9..000000000000 --- a/packages/cluster-sidebar/index.ts +++ /dev/null @@ -1,8 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ - -export * from "./src/tokens"; -export * from "./src/feature"; -export { default as sidebarItemsInjectable } from "./src/sidebar-items.injectable"; diff --git a/packages/cluster-sidebar/jest.config.js b/packages/cluster-sidebar/jest.config.js deleted file mode 100644 index 38d54ab7b6c3..000000000000 --- a/packages/cluster-sidebar/jest.config.js +++ /dev/null @@ -1 +0,0 @@ -module.exports = require("@k8slens/jest").monorepoPackageConfig(__dirname).configForReact; diff --git a/packages/cluster-sidebar/package.json b/packages/cluster-sidebar/package.json deleted file mode 100644 index 34a6a8a2128b..000000000000 --- a/packages/cluster-sidebar/package.json +++ /dev/null @@ -1,48 +0,0 @@ -{ - "name": "@k8slens/cluster-sidebar", - "private": false, - "version": "1.0.0-alpha.0", - "description": "Injection tokens for adding new sidebar items within the Cluster View", - "type": "commonjs", - "publishConfig": { - "access": "public", - "registry": "https://registry.npmjs.org/" - }, - "files": [ - "dist" - ], - "repository": { - "type": "git", - "url": "git+https://github.com/lensapp/lens.git" - }, - "main": "dist/index.js", - "types": "dist/index.d.ts", - "author": { - "name": "OpenLens Authors", - "email": "info@k8slens.dev" - }, - "license": "MIT", - "homepage": "https://github.com/lensapp/lens", - "scripts": { - "build": "lens-webpack-build", - "clean": "rimraf dist/", - "test": "jest --coverage --runInBand", - "lint": "lens-lint", - "lint:fix": "lens-lint --fix" - }, - "peerDependencies": { - "@k8slens/feature-core": "^6.5.0-alpha.4", - "@k8slens/utilities": "^1.0.0-alpha.3", - "@ogre-tools/injectable": "^15.8.1", - "@ogre-tools/injectable-extension-for-auto-registration": "^15.8.1", - "@ogre-tools/injectable-extension-for-mobx": "^15.8.1", - "mobx": "^6.9.0", - "react": "^17.0.2" - }, - "devDependencies": { - "@k8slens/eslint-config": "^6.5.0-alpha.3", - "@k8slens/jest": "^6.5.0-alpha.5", - "@k8slens/typescript": "^6.5.0-alpha.2", - "@k8slens/webpack": "^6.5.0-alpha.5" - } -} diff --git a/packages/cluster-sidebar/src/feature.ts b/packages/cluster-sidebar/src/feature.ts deleted file mode 100644 index d51e46c52422..000000000000 --- a/packages/cluster-sidebar/src/feature.ts +++ /dev/null @@ -1,14 +0,0 @@ -import { getFeature } from "@k8slens/feature-core"; -import { autoRegister } from "@ogre-tools/injectable-extension-for-auto-registration"; - -export const clusterSidebarFeature = getFeature({ - id: "cluster-side-feature", - - register: (di) => { - autoRegister({ - di, - targetModule: module, - getRequireContexts: () => [require.context("./", true, /\.injectable\.(ts|tsx)$/)], - }); - }, -}); diff --git a/packages/cluster-sidebar/src/sidebar-items.injectable.ts b/packages/cluster-sidebar/src/sidebar-items.injectable.ts deleted file mode 100644 index 6cef399fdd38..000000000000 --- a/packages/cluster-sidebar/src/sidebar-items.injectable.ts +++ /dev/null @@ -1,62 +0,0 @@ -/* eslint-disable prettier/prettier */ - -import { getInjectable } from "@ogre-tools/injectable"; -import { computedInjectManyInjectable } from "@ogre-tools/injectable-extension-for-mobx"; -import { HierarchicalSidebarItem, sidebarItemInjectionToken, SidebarItemRegistration } from "./tokens"; -import { computed } from "mobx"; -import { byOrderNumber } from "@k8slens/utilities"; - -const getSidebarItemsHierarchy = ( - registrations: SidebarItemRegistration[], - parentId: string | null, -): HierarchicalSidebarItem[] => ( - registrations - .filter((item) => item.parentId === parentId) - .map(({ - isActive, - isVisible, - ...registration - }) => { - const children = getSidebarItemsHierarchy(registrations, registration.id); - - return { - ...registration, - children, - isVisible: computed(() => { - if (children.length === 0) { - if (isVisible) { - return isVisible.get(); - } - - return true; - } - - return children.some((child) => child.isVisible.get()); - }), - isActive: computed(() => { - if (children.length === 0) { - if (isActive) { - return isActive.get(); - } - - return false; - } - - return children.some((child) => child.isActive.get()); - }), - }; - }) - .sort(byOrderNumber) -); - -const sidebarItemsInjectable = getInjectable({ - id: "sidebar-items", - instantiate: (di) => { - const computedInjectMany = di.inject(computedInjectManyInjectable); - const sidebarItemRegistrations = computedInjectMany(sidebarItemInjectionToken); - - return computed(() => getSidebarItemsHierarchy(sidebarItemRegistrations.get(), null)); - }, -}); - -export default sidebarItemsInjectable; diff --git a/packages/cluster-sidebar/src/tokens.ts b/packages/cluster-sidebar/src/tokens.ts deleted file mode 100644 index 3edbd0468eed..000000000000 --- a/packages/cluster-sidebar/src/tokens.ts +++ /dev/null @@ -1,37 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ - -import { getInjectionToken } from "@ogre-tools/injectable"; -import type { IComputedValue } from "mobx"; -import type { StrictReactNode } from "@k8slens/utilities"; - -export interface SidebarItemRegistration { - id: string; - parentId: string | null; - title: StrictReactNode; - onClick: () => void; - getIcon?: () => StrictReactNode; - isActive?: IComputedValue; - isVisible?: IComputedValue; - orderNumber: number; -} - -export interface SidebarItem { - id: string; - parentId: string | null; - title: StrictReactNode; - onClick: () => void; - getIcon?: () => StrictReactNode; - isActive: IComputedValue; - isVisible: IComputedValue; -} - -export interface HierarchicalSidebarItem extends SidebarItem { - children: HierarchicalSidebarItem[]; -} - -export const sidebarItemInjectionToken = getInjectionToken({ - id: "sidebar-item-injection-token", -}); diff --git a/packages/cluster-sidebar/tsconfig.json b/packages/cluster-sidebar/tsconfig.json deleted file mode 100644 index 2b0f0e56032a..000000000000 --- a/packages/cluster-sidebar/tsconfig.json +++ /dev/null @@ -1,4 +0,0 @@ -{ - "extends": "@k8slens/typescript/config/base.json", - "include": ["**/*.ts"] - } diff --git a/packages/cluster-sidebar/webpack.config.js b/packages/cluster-sidebar/webpack.config.js deleted file mode 100644 index 1cda407f5a9a..000000000000 --- a/packages/cluster-sidebar/webpack.config.js +++ /dev/null @@ -1 +0,0 @@ -module.exports = require("@k8slens/webpack").configForReact; diff --git a/packages/core/package.json b/packages/core/package.json index 2a6bf4716021..df9c8a27dbf8 100644 --- a/packages/core/package.json +++ b/packages/core/package.json @@ -290,7 +290,6 @@ "@k8slens/application-for-electron-main": "^6.5.0-alpha.0", "@k8slens/button": "^1.0.0-alpha.5", "@k8slens/cluster-settings": "^6.5.0-alpha.1", - "@k8slens/cluster-sidebar": "^1.0.0-alpha.0", "@k8slens/error-boundary": "^1.0.0-alpha.5", "@k8slens/event-emitter": "^1.0.0-alpha.1", "@k8slens/kubectl-versions": "^1.0.0-alpha.0", diff --git a/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/crd-list/crd-list-route.injectable.ts b/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/crd-list/crd-list-route.injectable.ts new file mode 100644 index 000000000000..854b3a8c3af6 --- /dev/null +++ b/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/crd-list/crd-list-route.injectable.ts @@ -0,0 +1,21 @@ +/** + * Copyright (c) OpenLens Authors. All rights reserved. + * Licensed under MIT License. See LICENSE in root directory for more information. + */ +import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; +import { frontEndRouteInjectionToken } from "../../../../front-end-route-injection-token"; + +const crdListRouteInjectable = getInjectable({ + id: "crd-list-route", + + instantiate: () => ({ + path: "/crd/definitions", + clusterFrame: true, + isEnabled: computed(() => true), + }), + + injectionToken: frontEndRouteInjectionToken, +}); + +export default crdListRouteInjectable; diff --git a/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/navigate-to-custom-resource-definitions.injectable.ts b/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/crd-list/navigate-to-crd-list.injectable.ts similarity index 75% rename from packages/core/src/common/front-end-routing/routes/cluster/custom-resources/navigate-to-custom-resource-definitions.injectable.ts rename to packages/core/src/common/front-end-routing/routes/cluster/custom-resources/crd-list/navigate-to-crd-list.injectable.ts index adf22eff34b7..bd78dd117441 100644 --- a/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/navigate-to-custom-resource-definitions.injectable.ts +++ b/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/crd-list/navigate-to-crd-list.injectable.ts @@ -3,8 +3,8 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { navigateToRouteInjectionToken } from "../../../navigate-to-route-injection-token"; -import crdListRouteInjectable from "./custom-resource-definitions.injectable"; +import crdListRouteInjectable from "./crd-list-route.injectable"; +import { navigateToRouteInjectionToken } from "../../../../navigate-to-route-injection-token"; const navigateToCrdListInjectable = getInjectable({ id: "navigate-to-crd-list", diff --git a/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resource-definitions.injectable.ts b/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resource-definitions.injectable.ts deleted file mode 100644 index 74251a64993d..000000000000 --- a/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resource-definitions.injectable.ts +++ /dev/null @@ -1,24 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { getInjectable } from "@ogre-tools/injectable"; -import { shouldShowResourceInjectionToken } from "../../../../../features/cluster/showing-kube-resources/common/allowed-resources-injection-token"; -import { frontEndRouteInjectionToken } from "../../../front-end-route-injection-token"; - -const customResourceDefinitionsRouteInjectable = getInjectable({ - id: "custom-resource-definitions-route", - - instantiate: (di) => ({ - path: "/crd/definitions", - clusterFrame: true, - isEnabled: di.inject(shouldShowResourceInjectionToken, { - group: "apiextensions.k8s.io", - apiName: "customresourcedefinitions", - }), - }), - - injectionToken: frontEndRouteInjectionToken, -}); - -export default customResourceDefinitionsRouteInjectable; diff --git a/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resources-route.injectable.ts b/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resources/custom-resources-route.injectable.ts similarity index 80% rename from packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resources-route.injectable.ts rename to packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resources/custom-resources-route.injectable.ts index 5f0c35c76d3f..cc7d9b40be53 100644 --- a/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resources-route.injectable.ts +++ b/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resources/custom-resources-route.injectable.ts @@ -4,8 +4,8 @@ */ import { getInjectable } from "@ogre-tools/injectable"; import { computed } from "mobx"; -import type { Route } from "../../../front-end-route-injection-token"; -import { frontEndRouteInjectionToken } from "../../../front-end-route-injection-token"; +import type { Route } from "../../../../front-end-route-injection-token"; +import { frontEndRouteInjectionToken } from "../../../../front-end-route-injection-token"; export interface CustomResourcesPathParameters { group?: string; diff --git a/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/navigate-to-custom-resources.injectable.ts b/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resources/navigate-to-custom-resources.injectable.ts similarity index 77% rename from packages/core/src/common/front-end-routing/routes/cluster/custom-resources/navigate-to-custom-resources.injectable.ts rename to packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resources/navigate-to-custom-resources.injectable.ts index d8206324e2bc..9df8cd6e2c1a 100644 --- a/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/navigate-to-custom-resources.injectable.ts +++ b/packages/core/src/common/front-end-routing/routes/cluster/custom-resources/custom-resources/navigate-to-custom-resources.injectable.ts @@ -3,9 +3,9 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { navigateToRouteInjectionToken } from "../../../navigate-to-route-injection-token"; import type { CustomResourcesPathParameters } from "./custom-resources-route.injectable"; import customResourcesRouteInjectable from "./custom-resources-route.injectable"; +import { navigateToRouteInjectionToken } from "../../../../navigate-to-route-injection-token"; const navigateToCustomResourcesInjectable = getInjectable({ id: "navigate-to-custom-resources", @@ -14,7 +14,8 @@ const navigateToCustomResourcesInjectable = getInjectable({ const navigateToRoute = di.inject(navigateToRouteInjectionToken); const route = di.inject(customResourcesRouteInjectable); - return (parameters?: CustomResourcesPathParameters) => navigateToRoute(route, { parameters }); + return (parameters?: CustomResourcesPathParameters) => + navigateToRoute(route, { parameters }); }, }); diff --git a/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/ingress-classes-route.injectable.ts b/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/ingress-classes-route.injectable.ts deleted file mode 100644 index 6ea8ffd73644..000000000000 --- a/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/ingress-classes-route.injectable.ts +++ /dev/null @@ -1,24 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { getInjectable } from "@ogre-tools/injectable"; -import { frontEndRouteInjectionToken } from "../../../../front-end-route-injection-token"; -import { shouldShowResourceInjectionToken } from "../../../../../../features/cluster/showing-kube-resources/common/allowed-resources-injection-token"; - -const ingressClassesRouteInjectable = getInjectable({ - id: "ingress-classes-route", - - instantiate: (di) => ({ - path: "/ingress-classes", - clusterFrame: true, - isEnabled: di.inject(shouldShowResourceInjectionToken, { - apiName: "ingressclasses", - group: "networking.k8s.io", - }), - }), - - injectionToken: frontEndRouteInjectionToken, -}); - -export default ingressClassesRouteInjectable; diff --git a/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/ingress-classeses-route.injectable.ts b/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/ingress-classeses-route.injectable.ts new file mode 100644 index 000000000000..efde1d8d7a0c --- /dev/null +++ b/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/ingress-classeses-route.injectable.ts @@ -0,0 +1,30 @@ +/** + * Copyright (c) OpenLens Authors. All rights reserved. + * Licensed under MIT License. See LICENSE in root directory for more information. + */ +import { getInjectable } from "@ogre-tools/injectable"; +import { frontEndRouteInjectionToken } from "../../../../front-end-route-injection-token"; +import { + shouldShowResourceInjectionToken, +} from "../../../../../../features/cluster/showing-kube-resources/common/allowed-resources-injection-token"; + +const ingressClassesesRouteInjectable = getInjectable({ + id: "ingress-classes-route", + + instantiate: (di) => { + const isEnabled = di.inject(shouldShowResourceInjectionToken, { + apiName: "ingressclasses", + group: "networking.k8s.io", + }); + + return { + path: "/ingress-classes", + clusterFrame: true, + isEnabled, + }; + }, + + injectionToken: frontEndRouteInjectionToken, +}); + +export default ingressClassesesRouteInjectable; diff --git a/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/navigate-to-ingress-classes.injectable.ts b/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/navigate-to-ingress-classes.injectable.ts index 1cc241ef7e04..22c666d5aee8 100644 --- a/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/navigate-to-ingress-classes.injectable.ts +++ b/packages/core/src/common/front-end-routing/routes/cluster/network/ingress-class/navigate-to-ingress-classes.injectable.ts @@ -4,14 +4,14 @@ */ import { getInjectable } from "@ogre-tools/injectable"; import { navigateToRouteInjectionToken } from "../../../../navigate-to-route-injection-token"; -import ingressClassesRouteInjectable from "./ingress-classes-route.injectable"; +import ingressClassesesRouteInjectable from "./ingress-classeses-route.injectable"; const navigateToIngressesInjectable = getInjectable({ id: "navigate-to-ingress-classes", instantiate: (di) => { const navigateToRoute = di.inject(navigateToRouteInjectionToken); - const route = di.inject(ingressClassesRouteInjectable); + const route = di.inject(ingressClassesesRouteInjectable); return () => navigateToRoute(route); }, diff --git a/packages/core/src/common/utils/composable-responsibilities/orderable/orderable.ts b/packages/core/src/common/utils/composable-responsibilities/orderable/orderable.ts new file mode 100644 index 000000000000..2b4d95b3f420 --- /dev/null +++ b/packages/core/src/common/utils/composable-responsibilities/orderable/orderable.ts @@ -0,0 +1,25 @@ +/** + * Copyright (c) OpenLens Authors. All rights reserved. + * Licensed under MIT License. See LICENSE in root directory for more information. + */ + +import { sortBy } from "lodash/fp"; + +export interface Orderable { + readonly orderNumber: number; +} + +export type MaybeOrderable = Orderable | object; + +export const orderByOrderNumber = (maybeOrderables: T[]) => + sortBy( + (orderable) => + "orderNumber" in orderable + ? orderable.orderNumber + : Number.MAX_SAFE_INTEGER, + maybeOrderables, + ); + +export const byOrderNumber = (left: T, right: T) => ( + left.orderNumber - right.orderNumber +); diff --git a/packages/core/src/features/application-menu/main/application-menu-item-composite.injectable.ts b/packages/core/src/features/application-menu/main/application-menu-item-composite.injectable.ts index 53a34338ba09..b9012fb67bdc 100644 --- a/packages/core/src/features/application-menu/main/application-menu-item-composite.injectable.ts +++ b/packages/core/src/features/application-menu/main/application-menu-item-composite.injectable.ts @@ -4,17 +4,18 @@ */ import { getInjectable } from "@ogre-tools/injectable"; import applicationMenuItemsInjectable from "./application-menu-items.injectable"; +import type { Composite } from "../../../common/utils/composite/get-composite/get-composite"; import { getCompositeFor } from "../../../common/utils/composite/get-composite/get-composite"; import { computed } from "mobx"; +import { pipeline } from "@ogre-tools/fp"; import type { ApplicationMenuItemTypes } from "./menu-items/application-menu-item-injection-token"; import type { RootComposite } from "../../../common/utils/composite/interfaces"; import type { Discriminable } from "../../../common/utils/composable-responsibilities/discriminable/discriminable"; +import { orderByOrderNumber } from "../../../common/utils/composable-responsibilities/orderable/orderable"; import logErrorInjectable from "../../../common/log-error.injectable"; import { isShown } from "../../../common/utils/composable-responsibilities/showable/showable"; -import type { Orderable } from "@k8slens/utilities"; -import { byOrderNumber } from "@k8slens/utilities"; -export type MenuItemRoot = Discriminable<"root"> & RootComposite<"root"> & Orderable; +export type MenuItemRoot = Discriminable<"root"> & RootComposite<"root">; const applicationMenuItemCompositeInjectable = getInjectable({ id: "application-menu-item-composite", @@ -23,30 +24,40 @@ const applicationMenuItemCompositeInjectable = getInjectable({ const menuItems = di.inject(applicationMenuItemsInjectable); const logError = di.inject(logErrorInjectable); - const getComposite = getCompositeFor({ - getId: (x) => x.id, - getParentId: (x) => x.parentId, - transformChildren: (children) => ( - [...children] - .sort(byOrderNumber) - .filter(isShown) - ), - handleMissingParentIds: ({ missingParentIds }) => { - const ids = missingParentIds.join('", "'); - - logError(`[MENU]: cannot render menu item for missing parentIds: "${ids}"`); - }, - }); + return computed((): Composite => { + const items = menuItems.get(); + + return pipeline( + [ + { + parentId: undefined, + id: "root", + kind: "root", + } as const, + + ...items, + ], - return computed(() => getComposite([ - { - parentId: undefined, - id: "root", - kind: "root", - orderNumber: -Infinity, - } as const, - ...menuItems.get(), - ])); + getCompositeFor({ + getId: (x) => x.id, + getParentId: (x) => x.parentId, + transformChildren: (children) => + pipeline( + children, + orderByOrderNumber, + (children) => children.filter(isShown), + ), + + handleMissingParentIds: ({ missingParentIds }) => { + logError( + `[MENU]: cannot render menu item for missing parentIds: "${missingParentIds.join( + '", "', + )}"`, + ); + }, + }), + ); + }); }, }); diff --git a/packages/core/src/features/application-menu/main/menu-items/application-menu-item-injection-token.ts b/packages/core/src/features/application-menu/main/menu-items/application-menu-item-injection-token.ts index 1da1bc00b793..b90e5e161290 100644 --- a/packages/core/src/features/application-menu/main/menu-items/application-menu-item-injection-token.ts +++ b/packages/core/src/features/application-menu/main/menu-items/application-menu-item-injection-token.ts @@ -8,7 +8,7 @@ import type { SetOptional } from "type-fest"; import type { ChildOfParentComposite, ParentOfChildComposite } from "../../../../common/utils/composite/interfaces"; import type { MaybeShowable } from "../../../../common/utils/composable-responsibilities/showable/showable"; import type { Discriminable } from "../../../../common/utils/composable-responsibilities/discriminable/discriminable"; -import type { Orderable } from "@k8slens/utilities"; +import type { Orderable } from "../../../../common/utils/composable-responsibilities/orderable/orderable"; export interface MayHaveKeyboardShortcut { keyboardShortcut?: string; diff --git a/packages/core/src/features/cluster/__snapshots__/custom-resources-in-sidebar.test.tsx.snap b/packages/core/src/features/cluster/__snapshots__/custom-resources-in-sidebar.test.tsx.snap deleted file mode 100644 index 7ec3ae37eef1..000000000000 --- a/packages/core/src/features/cluster/__snapshots__/custom-resources-in-sidebar.test.tsx.snap +++ /dev/null @@ -1,3847 +0,0 @@ -// Jest Snapshot v1, https://goo.gl/fbAQLP - -exports[`cluster - custom resources in sidebar renders 1`] = ` -
-
-
-
-
- -
- - - close - - -
- Close -
-
-
-
-
-
-
-
-
-
-
-
- ?? -
-
-
-
-
- - - - - - Workloads - - - - keyboard_arrow_down - - - -
-
- - - - device_hub - - - - Network - - - - keyboard_arrow_down - - - -
-
- - - - - - Helm - - - - keyboard_arrow_down - - - -
-
-
-
-
-
-
-
- -
-
-
-
-
- Overview -
-
-
- - -
-
-
- All namespaces -
-
- -
-
-
- - -
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
- -
-
-
-
- - - add - - -
- New tab -
-
- - - fullscreen - - -
- Fit to window -
- - - keyboard_arrow_up - - -
- Open -
-
-
-
-
-
-
-`; - -exports[`cluster - custom resources in sidebar when custom resource definitions are an allowed resource renders 1`] = ` -
-
-
-
-
- -
- - - close - - -
- Close -
-
-
-
-
-
-
-
-
-
-
-
- ?? -
-
-
-
-
- - - - - - Workloads - - - - keyboard_arrow_down - - - -
-
- - - - device_hub - - - - Network - - - - keyboard_arrow_down - - - -
-
- - - - - - Helm - - - - keyboard_arrow_down - - - -
-
- - - - extension - - - - Custom Resources - - - - keyboard_arrow_down - - - -
-
-
-
-
-
-
-
- -
-
-
-
-
- Overview -
-
-
- - -
-
-
- All namespaces -
-
- -
-
-
- - -
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
- -
-
-
-
- - - add - - -
- New tab -
-
- - - fullscreen - - -
- Fit to window -
- - - keyboard_arrow_up - - -
- Open -
-
-
-
-
-
-
-`; - -exports[`cluster - custom resources in sidebar when custom resource exists renders 1`] = ` -
-
-
-
-
- -
- - - close - - -
- Close -
-
-
-
-
-
-
-
-
-
-
-
- ?? -
-
-
-
-
- - - - - - Workloads - - - - keyboard_arrow_down - - - -
-
- - - - device_hub - - - - Network - - - - keyboard_arrow_down - - - -
-
- - - - - - Helm - - - - keyboard_arrow_down - - - -
-
-
-
-
-
-
-
- -
-
-
-
-
- Overview -
-
-
- - -
-
-
- All namespaces -
-
- -
-
-
- - -
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
- -
-
-
-
- - - add - - -
- New tab -
-
- - - fullscreen - - -
- Fit to window -
- - - keyboard_arrow_up - - -
- Open -
-
-
-
-
-
-
-`; - -exports[`cluster - custom resources in sidebar when custom resource exists when custom resource definitions are an allowed resource renders 1`] = ` -
-
-
-
-
- -
- - - close - - -
- Close -
-
-
-
-
-
-
-
-
-
-
-
- ?? -
-
-
-
-
- - - - - - Workloads - - - - keyboard_arrow_down - - - -
-
- - - - device_hub - - - - Network - - - - keyboard_arrow_down - - - -
-
- - - - - - Helm - - - - keyboard_arrow_down - - - -
-
- - - - extension - - - - Custom Resources - - - - keyboard_arrow_down - - - -
-
-
-
-
-
-
-
- -
-
-
-
-
- Overview -
-
-
- - -
-
-
- All namespaces -
-
- -
-
-
- - -
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
- -
-
-
-
- - - add - - -
- New tab -
-
- - - fullscreen - - -
- Fit to window -
- - - keyboard_arrow_up - - -
- Open -
-
-
-
-
-
-
-`; - -exports[`cluster - custom resources in sidebar when custom resource exists when custom resource definitions are an allowed resource when custom resources sidebar item is expanded renders 1`] = ` -
-
-
-
-
- -
- - - close - - -
- Close -
-
-
-
-
-
-
-
-
-
-
-
- ?? -
-
-
-
-
- - - - - - Workloads - - - - keyboard_arrow_down - - - -
-
- - - - device_hub - - - - Network - - - - keyboard_arrow_down - - - -
-
- - - - - - Helm - - - - keyboard_arrow_down - - - -
-
- - - - extension - - - - Custom Resources - - - - keyboard_arrow_up - - - -
-
- - - Definitions - - -
-
-
-
-
-
-
-
-
-
- -
-
-
-
-
- Overview -
-
-
- - -
-
-
- All namespaces -
-
- -
-
-
- - -
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
- -
-
-
-
- - - add - - -
- New tab -
-
- - - fullscreen - - -
- Fit to window -
- - - keyboard_arrow_up - - -
- Open -
-
-
-
-
-
-
-`; - -exports[`cluster - custom resources in sidebar when custom resource exists when specific custom resource is an allowed resource renders 1`] = ` -
-
-
-
-
- -
- - - close - - -
- Close -
-
-
-
-
-
-
-
-
-
-
-
- ?? -
-
-
-
-
- - - - - - Workloads - - - - keyboard_arrow_down - - - -
-
- - - - device_hub - - - - Network - - - - keyboard_arrow_down - - - -
-
- - - - - - Helm - - - - keyboard_arrow_down - - - -
-
- - - - extension - - - - Custom Resources - - - - keyboard_arrow_down - - - -
-
-
-
-
-
-
-
- -
-
-
-
-
- Overview -
-
-
- - -
-
-
- All namespaces -
-
- -
-
-
- - -
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
- -
-
-
-
- - - add - - -
- New tab -
-
- - - fullscreen - - -
- Fit to window -
- - - keyboard_arrow_up - - -
- Open -
-
-
-
-
-
-
-`; - -exports[`cluster - custom resources in sidebar when custom resource exists when specific custom resource is an allowed resource when custom resources sidebar item is expanded renders 1`] = ` -
-
-
-
-
- -
- - - close - - -
- Close -
-
-
-
-
-
-
-
-
-
-
-
- ?? -
-
-
-
-
- - - - - - Workloads - - - - keyboard_arrow_down - - - -
-
- - - - device_hub - - - - Network - - - - keyboard_arrow_down - - - -
-
- - - - - - Helm - - - - keyboard_arrow_down - - - -
-
- - - - extension - - - - Custom Resources - - - - keyboard_arrow_up - - - -
-
- - - some-group - - - - keyboard_arrow_down - - - -
-
-
-
-
-
-
-
-
-
- -
-
-
-
-
- Overview -
-
-
- - -
-
-
- All namespaces -
-
- -
-
-
- - -
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
- -
-
-
-
- - - add - - -
- New tab -
-
- - - fullscreen - - -
- Fit to window -
- - - keyboard_arrow_up - - -
- Open -
-
-
-
-
-
-
-`; - -exports[`cluster - custom resources in sidebar when custom resource exists when specific custom resource is an allowed resource when custom resources sidebar item is expanded when custom resources group sidebar item is expanded renders 1`] = ` -
-
-
-
-
- -
- - - close - - -
- Close -
-
-
-
-
-
-
-
-
-
-
-
- ?? -
-
-
-
-
- - - - - - Workloads - - - - keyboard_arrow_down - - - -
-
- - - - device_hub - - - - Network - - - - keyboard_arrow_down - - - -
-
- - - - - - Helm - - - - keyboard_arrow_down - - - -
-
- - - - extension - - - - Custom Resources - - - - keyboard_arrow_up - - - -
-
- - - some-group - - - - keyboard_arrow_up - - - -
-
- - - SomeResource - - -
-
-
-
-
-
-
-
-
-
-
-
- -
-
-
-
-
- Overview -
-
-
- - -
-
-
- All namespaces -
-
- -
-
-
- - -
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
- -
-
-
-
- - - add - - -
- New tab -
-
- - - fullscreen - - -
- Fit to window -
- - - keyboard_arrow_up - - -
- Open -
-
-
-
-
-
-
-`; diff --git a/packages/core/src/features/cluster/custom-resources-in-sidebar.test.tsx b/packages/core/src/features/cluster/custom-resources-in-sidebar.test.tsx deleted file mode 100644 index c9199aba425e..000000000000 --- a/packages/core/src/features/cluster/custom-resources-in-sidebar.test.tsx +++ /dev/null @@ -1,244 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ - -import type { RenderResult } from "@testing-library/react"; -import { CustomResourceDefinition } from "../../extensions/common-api/k8s-api"; -import type { CustomResourceDefinitionStore } from "../../renderer/components/custom-resources/definition.store"; -import customResourceDefinitionStoreInjectable from "../../renderer/components/custom-resources/definition.store.injectable"; -import type { ApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; -import { getApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; - -describe("cluster - custom resources in sidebar", () => { - let builder: ApplicationBuilder; - let result: RenderResult; - let customResourceDefinitionStore: CustomResourceDefinitionStore; - let customResourceDefinition: CustomResourceDefinition; - - beforeEach(async () => { - builder = getApplicationBuilder(); - builder.setEnvironmentToClusterFrame(); - - builder.afterWindowStart(({ windowDi }) => { - customResourceDefinitionStore = windowDi.inject(customResourceDefinitionStoreInjectable); - customResourceDefinition = new CustomResourceDefinition({ - apiVersion: "apiextensions.k8s.io/v1", - kind: "CustomResourceDefinition", - metadata: { - name: "some-crd", - selfLink: "/apis/apiextensions.k8s.io/v1/customresourcedefinitions/some-crd", - uid: "some-uid", - resourceVersion: "1", - }, - spec: { - group: "some-group", - scope: "Cluster", - names: { - kind: "SomeResource", - plural: "some-resources", - singular: "some-resource", - }, - versions: [ - { - storage: true, - name: "v1", - served: true, - additionalPrinterColumns: [ - { - name: "Some Column", - type: "string", - description: "Some description", - jsonPath: ".spec.someColumn", - }, - ], - }, - ], - }, - }); - }); - - result = await builder.render(); - }); - - it("renders", () => { - expect(result.container).toMatchSnapshot(); - }); - - it("shows the sidebar", () => { - expect(result.getByTestId("cluster-sidebar")).toBeInTheDocument(); - }); - - it("does not show Custom Resources section", () => { - expect(result.queryByTestId("sidebar-item-custom-resources")).not.toBeInTheDocument(); - }); - - describe("when custom resource exists", () => { - beforeEach(() => { - customResourceDefinitionStore.items.replace([ - customResourceDefinition, - ]); - }); - - it("renders", () => { - expect(result.container).toMatchSnapshot(); - }); - - it("still does not show Custom Resources sidebar", () => { - expect(result.queryByTestId("sidebar-item-custom-resources")).not.toBeInTheDocument(); - }); - - describe("when specific custom resource is an allowed resource", () => { - beforeEach(() => { - builder.allowKubeResource({ - apiName: "some-resources", - group: "some-group", - }); - }); - - it("renders", () => { - expect(result.container).toMatchSnapshot(); - }); - - it("shows Custom Resources sidebar", () => { - expect(result.getByTestId("sidebar-item-custom-resources")).toBeInTheDocument(); - }); - - it("shows Custom Resources sidebar as expandable", () => { - expect(result.getByTestId("sidebar-item-expand-icon-for-custom-resources")).toBeInTheDocument(); - }); - - it("does not show SomeResources sidebar", () => { - expect(result.queryByTestId("sidebar-item-custom-resource-group-some-group")).not.toBeInTheDocument(); - }); - - it("does not show Custom Resources Definitions sidebar", () => { - expect(result.queryByTestId("sidebar-item-custom-resource-definitions")).not.toBeInTheDocument(); - }); - - describe("when custom resources sidebar item is expanded", () => { - beforeEach(() => { - result.getByTestId("sidebar-item-expand-icon-for-custom-resources").click(); - }); - - it("renders", () => { - expect(result.container).toMatchSnapshot(); - }); - - it("shows Custom Resources sidebar", () => { - expect(result.getByTestId("sidebar-item-custom-resources")).toBeInTheDocument(); - }); - - it("shows Custom Resources sidebar as expandable", () => { - expect(result.getByTestId("sidebar-item-expand-icon-for-custom-resources")).toBeInTheDocument(); - }); - - it("shows some-group group sidebar item", () => { - expect(result.getByTestId("sidebar-item-custom-resource-group-some-group")).toBeInTheDocument(); - }); - - it("does not show Custom Resources Definitions sidebar", () => { - expect(result.queryByTestId("sidebar-item-custom-resource-definitions")).not.toBeInTheDocument(); - }); - - describe("when custom resources group sidebar item is expanded", () => { - beforeEach(() => { - result.getByTestId("sidebar-item-expand-icon-for-custom-resource-group-some-group").click(); - }); - - it("renders", () => { - expect(result.container).toMatchSnapshot(); - }); - - it("shows some-group group sidebar item", () => { - expect(result.getByTestId("sidebar-item-custom-resource-group-some-group")).toBeInTheDocument(); - }); - - it("shows some-resources group sidebar item", () => { - expect(result.getByTestId("sidebar-item-custom-resource-group-some-group/some-resources")).toBeInTheDocument(); - }); - }); - }); - }); - - describe("when custom resource definitions are an allowed resource", () => { - beforeEach(() => { - builder.allowKubeResource({ - apiName: "customresourcedefinitions", - group: "apiextensions.k8s.io", - }); - }); - - it("renders", () => { - expect(result.container).toMatchSnapshot(); - }); - - it("shows Custom Resources sidebar", () => { - expect(result.getByTestId("sidebar-item-custom-resources")).toBeInTheDocument(); - }); - - it("shows Custom Resources sidebar as expandable", () => { - expect(result.getByTestId("sidebar-item-expand-icon-for-custom-resources")).toBeInTheDocument(); - }); - - it("does not show SomeResources sidebar", () => { - expect(result.queryByTestId("sidebar-item-custom-resource-group-some-group")).not.toBeInTheDocument(); - }); - - it("does not show Custom Resources Definitions sidebar", () => { - expect(result.queryByTestId("sidebar-item-custom-resource-definitions")).not.toBeInTheDocument(); - }); - - describe("when custom resources sidebar item is expanded", () => { - beforeEach(() => { - result.getByTestId("sidebar-item-expand-icon-for-custom-resources").click(); - }); - - it("renders", () => { - expect(result.container).toMatchSnapshot(); - }); - - it("shows Custom Resources sidebar", () => { - expect(result.getByTestId("sidebar-item-custom-resources")).toBeInTheDocument(); - }); - - it("shows Custom Resources sidebar as expandable", () => { - expect(result.getByTestId("sidebar-item-expand-icon-for-custom-resources")).toBeInTheDocument(); - }); - - it("does not show SomeResources sidebar", () => { - expect(result.queryByTestId("sidebar-item-custom-resource-group-some-group")).not.toBeInTheDocument(); - }); - - it("shows Custom Resources Definitions sidebar", () => { - expect(result.getByTestId("sidebar-item-custom-resource-definitions")).toBeInTheDocument(); - }); - }); - }); - }); - - describe("when custom resource definitions are an allowed resource", () => { - beforeEach(() => { - builder.allowKubeResource({ - apiName: "customresourcedefinitions", - group: "apiextensions.k8s.io", - }); - }); - - it("renders", () => { - expect(result.container).toMatchSnapshot(); - }); - - it("shows Custom Resources sidebar", () => { - expect(result.getByTestId("sidebar-item-custom-resources")).toBeInTheDocument(); - }); - - it("shows Custom Resources sidebar as expandable", () => { - expect(result.getByTestId("sidebar-item-expand-icon-for-custom-resources")).toBeInTheDocument(); - }); - - it("does not show Custom Resources Definitions sidebar", () => { - expect(result.queryByTestId("sidebar-item-custom-resource-definitions")).not.toBeInTheDocument(); - }); - }); -}); diff --git a/packages/core/src/features/cluster/order-of-sidebar-items.test.tsx b/packages/core/src/features/cluster/order-of-sidebar-items.test.tsx index 244a2b6960ac..2b24c3a5b1b6 100644 --- a/packages/core/src/features/cluster/order-of-sidebar-items.test.tsx +++ b/packages/core/src/features/cluster/order-of-sidebar-items.test.tsx @@ -5,8 +5,9 @@ import { getInjectable } from "@ogre-tools/injectable"; import type { RenderResult } from "@testing-library/react"; import { fireEvent } from "@testing-library/react"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; -import { runInAction } from "mobx"; +import type { SidebarItemRegistration } from "../../renderer/components/layout/sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../../renderer/components/layout/sidebar-items.injectable"; +import { computed, runInAction } from "mobx"; import { noop } from "lodash/fp"; import type { ApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; import { getApplicationBuilder } from "../../renderer/components/test-utils/get-application-builder"; @@ -22,14 +23,7 @@ describe("cluster - order of sidebar items", () => { builder.beforeWindowStart(({ windowDi }) => { runInAction(() => { - windowDi.register( - someParentSidebarItemInjectable, - someOtherParentSidebarItemInjectable, - someAnotherParentSidebarItemInjectable, - someChildSidebarItemInjectable, - someOtherChildSidebarItemInjectable, - someAnotherChildSidebarItemInjectable, - ); + windowDi.register(testSidebarItemsInjectable); }); }); }); @@ -84,74 +78,54 @@ describe("cluster - order of sidebar items", () => { }); }); -const someParentSidebarItemInjectable = getInjectable({ - id: "some-parent-sidebar-item", - instantiate: () => ({ - id: "some-parent-id", - parentId: null, - title: "Some parent", - onClick: noop, - orderNumber: 42, - }), - injectionToken: sidebarItemInjectionToken, -}); - -const someOtherParentSidebarItemInjectable = getInjectable({ - id: "some-other-parent-sidebar-item", - instantiate: () => ({ - id: "some-other-parent-id", - parentId: null, - title: "Some other parent", - onClick: noop, - orderNumber: 126, - }), - injectionToken: sidebarItemInjectionToken, -}); - -const someAnotherParentSidebarItemInjectable = getInjectable({ - id: "some-another-parent-sidebar-item", - instantiate: () => ({ - id: "some-another-parent-id", - parentId: null, - title: "Some another parent", - onClick: noop, - orderNumber: 84, - }), - injectionToken: sidebarItemInjectionToken, -}); - -const someChildSidebarItemInjectable = getInjectable({ - id: "some-child-sidebar-item", - instantiate: (di) => ({ - id: "some-child-id", - parentId: di.inject(someParentSidebarItemInjectable).id, - title: "Some child", - onClick: noop, - orderNumber: 168, - }), - injectionToken: sidebarItemInjectionToken, -}); - -const someOtherChildSidebarItemInjectable = getInjectable({ - id: "some-other-child-sidebar-item", - instantiate: (di) => ({ - id: "some-other-child-id", - parentId: di.inject(someParentSidebarItemInjectable).id, - title: "Some other child", - onClick: noop, - orderNumber: 252, - }), - injectionToken: sidebarItemInjectionToken, -}); - -const someAnotherChildSidebarItemInjectable = getInjectable({ - id: "some-another-child-sidebar-item", - instantiate: (di) => ({ - id: "some-another-child-id", - parentId: di.inject(someParentSidebarItemInjectable).id, - title: "Some another child", - onClick: noop, - orderNumber: 210, - }), - injectionToken: sidebarItemInjectionToken, +const testSidebarItemsInjectable = getInjectable({ + id: "some-sidebar-item-injectable", + + instantiate: () => + computed((): SidebarItemRegistration[] => [ + { + id: "some-parent-id", + parentId: null, + title: "Some parent", + onClick: noop, + orderNumber: 42, + }, + { + id: "some-other-parent-id", + parentId: null, + title: "Some other parent", + onClick: noop, + orderNumber: 126, + }, + { + id: "some-another-parent-id", + parentId: null, + title: "Some another parent", + onClick: noop, + orderNumber: 84, + }, + { + id: "some-child-id", + parentId: "some-parent-id", + title: "Some child", + onClick: noop, + orderNumber: 168, + }, + { + id: "some-other-child-id", + parentId: "some-parent-id", + title: "Some other child", + onClick: noop, + orderNumber: 252, + }, + { + id: "some-another-child-id", + parentId: "some-parent-id", + title: "Some another child", + onClick: noop, + orderNumber: 210, + }, + ]), + + injectionToken: sidebarItemsInjectionToken, }); diff --git a/packages/core/src/features/cluster/sidebar-and-tab-navigation-for-core.test.tsx b/packages/core/src/features/cluster/sidebar-and-tab-navigation-for-core.test.tsx index d682e3ca164c..4bd3234d131b 100644 --- a/packages/core/src/features/cluster/sidebar-and-tab-navigation-for-core.test.tsx +++ b/packages/core/src/features/cluster/sidebar-and-tab-navigation-for-core.test.tsx @@ -8,7 +8,8 @@ import React from "react"; import type { RenderResult } from "@testing-library/react"; import { fireEvent } from "@testing-library/react"; import { routeSpecificComponentInjectionToken } from "../../renderer/routes/route-specific-component-injection-token"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import type { SidebarItemRegistration } from "../../renderer/components/layout/sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../../renderer/components/layout/sidebar-items.injectable"; import { computed, runInAction } from "mobx"; import { noop } from "lodash/fp"; import routeIsActiveInjectable from "../../renderer/routes/route-is-active.injectable"; @@ -43,12 +44,9 @@ describe("cluster - sidebar and tab navigation for core", () => { beforeEach(() => { builder.beforeWindowStart(({ windowDi }) => { runInAction(() => { - windowDi.register( - testRouteInjectable, - testRouteComponentInjectable, - someParentSidebarItemInjectable, - someChildSidebarItemInjectable, - ); + windowDi.register(testRouteInjectable); + windowDi.register(testRouteComponentInjectable); + windowDi.register(testSidebarItemsInjectable); }); }); }); @@ -294,36 +292,36 @@ describe("cluster - sidebar and tab navigation for core", () => { }); }); -const someParentSidebarItemInjectable = getInjectable({ - id: "some-parent-sidebar-item", - instantiate: () => ({ - id: "some-parent-id", - parentId: null, - title: "Some parent", - onClick: noop, - getIcon: () =>
, - orderNumber: 42, - }), - injectionToken: sidebarItemInjectionToken, -}); +const testSidebarItemsInjectable = getInjectable({ + id: "some-sidebar-items-injectable", -const someChildSidebarItemInjectable = getInjectable({ - id: "some-child-sidebar-item", instantiate: (di) => { const route = di.inject(testRouteInjectable); const navigateToRoute = di.inject(navigateToRouteInjectionToken); const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "some-child-id", - parentId: di.inject(someParentSidebarItemInjectable).id, - title: "Some child", - onClick: () => navigateToRoute(route), - isActive: routeIsActive, - orderNumber: 168, - }; + return computed((): SidebarItemRegistration[] => [ + { + id: "some-parent-id", + parentId: null, + title: "Some parent", + onClick: noop, + getIcon: () =>
, + orderNumber: 42, + }, + + { + id: "some-child-id", + parentId: "some-parent-id", + title: "Some child", + onClick: () => navigateToRoute(route), + isActive: routeIsActive, + orderNumber: 42, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + + injectionToken: sidebarItemsInjectionToken, }); const testRouteInjectable = getInjectable({ diff --git a/packages/core/src/features/cluster/visibility-of-sidebar-items.test.tsx b/packages/core/src/features/cluster/visibility-of-sidebar-items.test.tsx index 4146e25ed78d..46a891624116 100644 --- a/packages/core/src/features/cluster/visibility-of-sidebar-items.test.tsx +++ b/packages/core/src/features/cluster/visibility-of-sidebar-items.test.tsx @@ -4,8 +4,9 @@ */ import { getInjectable } from "@ogre-tools/injectable"; import type { RenderResult } from "@testing-library/react"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; -import { runInAction } from "mobx"; +import type { SidebarItemRegistration } from "../../renderer/components/layout/sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../../renderer/components/layout/sidebar-items.injectable"; +import { computed, runInAction } from "mobx"; import { routeSpecificComponentInjectionToken } from "../../renderer/routes/route-specific-component-injection-token"; import React from "react"; import { frontEndRouteInjectionToken } from "../../common/front-end-routing/front-end-route-injection-token"; @@ -101,16 +102,18 @@ const testSidebarItemsInjectable = getInjectable({ const testRoute = di.inject(testRouteInjectable); const navigateToRoute = di.inject(navigateToRouteInjectionToken); - return { - id: "some-item-id", - parentId: null, - title: "Some item", - onClick: () => navigateToRoute(testRoute), - isVisible: testRoute.isEnabled, - orderNumber: 42, - }; + return computed((): SidebarItemRegistration[] => [ + { + id: "some-item-id", + parentId: null, + title: "Some item", + onClick: () => navigateToRoute(testRoute), + isVisible: testRoute.isEnabled, + orderNumber: 42, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); diff --git a/packages/core/src/features/preferences/renderer/preference-items/preference-item-injection-token.ts b/packages/core/src/features/preferences/renderer/preference-items/preference-item-injection-token.ts index 71787b3fdfe9..38eb14aad80a 100644 --- a/packages/core/src/features/preferences/renderer/preference-items/preference-item-injection-token.ts +++ b/packages/core/src/features/preferences/renderer/preference-items/preference-item-injection-token.ts @@ -8,7 +8,7 @@ import type { ChildOfParentComposite, ParentOfChildComposite } from "../../../.. import type { Discriminable } from "../../../../common/utils/composable-responsibilities/discriminable/discriminable"; import type { Labelable } from "../../../../common/utils/composable-responsibilities/labelable/labelable"; import type { MaybeShowable } from "../../../../common/utils/composable-responsibilities/showable/showable"; -import type { Orderable, MaybeOrderable } from "@k8slens/utilities"; +import type { Orderable } from "../../../../common/utils/composable-responsibilities/orderable/orderable"; import type { GetSeparator } from "../../../../common/utils/add-separator/add-separator"; import type { Composite } from "../../../../common/utils/composite/get-composite/get-composite"; @@ -50,7 +50,6 @@ export type PreferencePage = & Discriminable<"page"> & ParentOfChildComposite & ChildOfParentComposite - & MaybeOrderable & MaybeShowable & RenderableWithSiblings; @@ -58,7 +57,6 @@ export type PreferenceBlock = & Discriminable<"block"> & ParentOfChildComposite & ChildOfParentComposite - & MaybeOrderable & MaybeShowable & RenderableWithSiblings; diff --git a/packages/core/src/features/preferences/renderer/preference-items/preference-tab-root.tsx b/packages/core/src/features/preferences/renderer/preference-items/preference-tab-root.tsx index e95e8680ebb0..2847c07515a8 100644 --- a/packages/core/src/features/preferences/renderer/preference-items/preference-tab-root.tsx +++ b/packages/core/src/features/preferences/renderer/preference-items/preference-tab-root.tsx @@ -8,19 +8,16 @@ import type { RootComposite } from "../../../../common/utils/composite/interface import type { Discriminable } from "../../../../common/utils/composable-responsibilities/discriminable/discriminable"; import type { ChildrenAreSeparated } from "./preference-item-injection-token"; import styles from "./preference-tab-root.module.scss"; -import type { Orderable } from "@k8slens/utilities"; export type PreferenceTabsRoot = & Discriminable<"preference-tabs-root"> & RootComposite - & ChildrenAreSeparated - & Orderable; + & ChildrenAreSeparated; export const preferenceTabsRoot: PreferenceTabsRoot = { kind: "preference-tabs-root" as const, id: "preference-tabs", parentId: undefined, - orderNumber: Infinity, childSeparator: () => (
diff --git a/packages/core/src/features/preferences/renderer/preference-items/preferences-composite.injectable.ts b/packages/core/src/features/preferences/renderer/preference-items/preferences-composite.injectable.ts index 576dfd1cb7ed..dd1ffd1ed8de 100644 --- a/packages/core/src/features/preferences/renderer/preference-items/preferences-composite.injectable.ts +++ b/packages/core/src/features/preferences/renderer/preference-items/preferences-composite.injectable.ts @@ -7,12 +7,13 @@ import { computedInjectManyInjectable } from "@ogre-tools/injectable-extension-f import { computed } from "mobx"; import type { PreferenceItemTypes } from "./preference-item-injection-token"; import { preferenceItemInjectionToken } from "./preference-item-injection-token"; +import { pipeline } from "@ogre-tools/fp"; import type { PreferenceTabsRoot } from "./preference-tab-root"; import { preferenceTabsRoot } from "./preference-tab-root"; import logErrorInjectable from "../../../../common/log-error.injectable"; import { isShown } from "../../../../common/utils/composable-responsibilities/showable/showable"; +import { orderByOrderNumber } from "../../../../common/utils/composable-responsibilities/orderable/orderable"; import { getCompositeFor } from "../../../../common/utils/composite/get-composite/get-composite"; -import { byOrderNumber } from "@k8slens/utilities"; const preferencesCompositeInjectable = getInjectable({ id: "preferences-composite", @@ -26,25 +27,27 @@ const preferencesCompositeInjectable = getInjectable({ getId: (x) => x.id, getParentId: (x) => x.parentId, - handleMissingParentIds: ({ missingParentIds, availableParentIds }) => { - const missingIds = missingParentIds.join('", "'); - const availableIds = availableParentIds.join("\n"); - - logError([ - `Tried to create preferences, but encountered references to unknown ids: "${missingIds}".`, - "Available ids are:", - availableIds, - ].join("\n\n")); + handleMissingParentIds: (ids) => { + logError( + `Tried to create preferences, but encountered references to unknown ids: "${ids.missingParentIds.join( + '", "', + )}".\n\nAvailable ids are:\n\n${ids.availableParentIds.join("\n")}`, + ); }, - transformChildren: (children) => ( - children - .filter(isShown) - .sort(byOrderNumber) - ), + transformChildren: (children) => + pipeline( + children.filter(isShown), + orderByOrderNumber, + ), }); - return computed(() => getComposite([preferenceTabsRoot, ...preferenceItems.get()])); + return computed(() => + pipeline( + [preferenceTabsRoot, ...preferenceItems.get()], + getComposite, + ), + ); }, }); diff --git a/packages/core/src/main/cluster/request-api-resources.injectable.ts b/packages/core/src/main/cluster/request-api-resources.injectable.ts index 1de88b5f607c..3d8180b7a638 100644 --- a/packages/core/src/main/cluster/request-api-resources.injectable.ts +++ b/packages/core/src/main/cluster/request-api-resources.injectable.ts @@ -12,7 +12,7 @@ import { backoffCaller, withConcurrencyLimit } from "@k8slens/utilities"; import requestKubeApiResourcesForInjectable from "./request-kube-api-resources-for.injectable"; import type { AsyncResult } from "@k8slens/utilities"; import broadcastConnectionUpdateInjectable from "./broadcast-connection-update.injectable"; -import { byOrderNumber } from "@k8slens/utilities"; +import { byOrderNumber } from "../../common/utils/composable-responsibilities/orderable/orderable"; export type RequestApiResources = (cluster: Cluster) => AsyncResult; diff --git a/packages/core/src/renderer/components/catalog/entity-details/detail-items.injectable.ts b/packages/core/src/renderer/components/catalog/entity-details/detail-items.injectable.ts index ee0630f571c0..7ef414d98805 100644 --- a/packages/core/src/renderer/components/catalog/entity-details/detail-items.injectable.ts +++ b/packages/core/src/renderer/components/catalog/entity-details/detail-items.injectable.ts @@ -5,7 +5,7 @@ import { getInjectable, lifecycleEnum } from "@ogre-tools/injectable"; import { computedInjectManyInjectable } from "@ogre-tools/injectable-extension-for-mobx"; import { computed } from "mobx"; -import { byOrderNumber } from "@k8slens/utilities"; +import { byOrderNumber } from "../../../../common/utils/composable-responsibilities/orderable/orderable"; import type { CatalogEntity } from "../../../api/catalog-entity"; import { catalogEntityDetailItemInjectionToken } from "./token"; diff --git a/packages/core/src/renderer/components/cluster/cluster-overview-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/cluster/cluster-overview-sidebar-items.injectable.tsx index 4645439bf26e..d3cf4e681461 100644 --- a/packages/core/src/renderer/components/cluster/cluster-overview-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/cluster/cluster-overview-sidebar-items.injectable.tsx @@ -5,30 +5,39 @@ import { getInjectable } from "@ogre-tools/injectable"; import { Icon } from "../icon"; import React from "react"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; +import { computed } from "mobx"; import clusterOverviewRouteInjectable from "../../../common/front-end-routing/routes/cluster/overview/cluster-overview-route.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToClusterOverviewInjectable from "../../../common/front-end-routing/routes/cluster/overview/navigate-to-cluster-overview.injectable"; -const clusterOverviewSidebarItemInjectable = getInjectable({ - id: "cluster-overview-sidebar-item", +const clusterOverviewSidebarItemsInjectable = getInjectable({ + id: "cluster-overview-sidebar-items", instantiate: (di) => { const route = di.inject(clusterOverviewRouteInjectable); + const navigateToClusterOverview = di.inject(navigateToClusterOverviewInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "cluster-overview", - parentId: null, - title: "Cluster", - getIcon: () => , - onClick: di.inject(navigateToClusterOverviewInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 10, - }; + return computed((): SidebarItemRegistration[] => [ + { + id: "cluster-overview", + parentId: null, + title: "Cluster", + getIcon: () => , + onClick: navigateToClusterOverview, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 10, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default clusterOverviewSidebarItemInjectable; +export default clusterOverviewSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/cluster/cluster-overview.tsx b/packages/core/src/renderer/components/cluster/cluster-overview.tsx index f2c5c1dace97..f14c8fea2ef8 100644 --- a/packages/core/src/renderer/components/cluster/cluster-overview.tsx +++ b/packages/core/src/renderer/components/cluster/cluster-overview.tsx @@ -10,7 +10,6 @@ import type { IComputedValue } from "mobx"; import { disposeOnUnmount, observer } from "mobx-react"; import type { NodeStore } from "../nodes/store"; import type { PodStore } from "../workloads-pods/store"; -import { byOrderNumber } from "@k8slens/utilities"; import { TabLayout } from "../layout/tab-layout"; import { Spinner } from "../spinner"; import { ClusterIssues } from "./cluster-issues"; @@ -26,6 +25,7 @@ import nodeStoreInjectable from "../nodes/store.injectable"; import enabledMetricsInjectable from "../../api/catalog/entity/metrics-enabled.injectable"; import type { ClusterOverviewUIBlock } from "@k8slens/metrics"; import { clusterOverviewUIBlockInjectionToken } from "@k8slens/metrics"; +import { orderByOrderNumber } from "../../../common/utils/composable-responsibilities/orderable/orderable"; import { computedInjectManyInjectable } from "@ogre-tools/injectable-extension-for-mobx"; import type { ClusterMetricData } from "../../../common/k8s-api/endpoints/metrics.api/request-cluster-metrics-by-node-names.injectable"; import clusterOverviewMetricsInjectable from "./cluster-metrics.injectable"; @@ -56,9 +56,10 @@ class NonInjectedClusterOverview extends React.Component { return ( <> { - [...this.props.uiBlocks.get()] - .sort(byOrderNumber) - .map(block => ) + orderByOrderNumber(this.props.uiBlocks.get()) + .map((block) => ( + + )) } diff --git a/packages/core/src/renderer/components/command-palette/registered-commands/internal-commands.injectable.tsx b/packages/core/src/renderer/components/command-palette/registered-commands/internal-commands.injectable.tsx index e160e1b49ccb..578b5fc867af 100644 --- a/packages/core/src/renderer/components/command-palette/registered-commands/internal-commands.injectable.tsx +++ b/packages/core/src/renderer/components/command-palette/registered-commands/internal-commands.injectable.tsx @@ -34,7 +34,7 @@ import navigateToDaemonsetsInjectable from "../../../../common/front-end-routing import navigateToStatefulsetsInjectable from "../../../../common/front-end-routing/routes/cluster/workloads/statefulsets/navigate-to-statefulsets.injectable"; import navigateToJobsInjectable from "../../../../common/front-end-routing/routes/cluster/workloads/jobs/navigate-to-jobs.injectable"; import navigateToCronJobsInjectable from "../../../../common/front-end-routing/routes/cluster/workloads/cron-jobs/navigate-to-cron-jobs.injectable"; -import navigateToCustomResourcesInjectable from "../../../../common/front-end-routing/routes/cluster/custom-resources/navigate-to-custom-resources.injectable"; +import navigateToCustomResourcesInjectable from "../../../../common/front-end-routing/routes/cluster/custom-resources/custom-resources/navigate-to-custom-resources.injectable"; import navigateToEntitySettingsInjectable from "../../../../common/front-end-routing/routes/entity-settings/navigate-to-entity-settings.injectable"; // TODO: Importing from features is not OK. Make commands to comply with Open Closed Principle to allow moving implementation under a feature diff --git a/packages/core/src/renderer/components/config-horizontal-pod-autoscalers/sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config-horizontal-pod-autoscalers/sidebar-items.injectable.tsx index 0415fb7d5cdf..f8aaf999847f 100644 --- a/packages/core/src/renderer/components/config-horizontal-pod-autoscalers/sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/config-horizontal-pod-autoscalers/sidebar-items.injectable.tsx @@ -3,30 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import horizontalPodAutoscalersRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/horizontal-pod-autoscalers/horizontal-pod-autoscalers-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import type { SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToHorizontalPodAutoscalersInjectable from "../../../common/front-end-routing/routes/cluster/config/horizontal-pod-autoscalers/navigate-to-horizontal-pod-autoscalers.injectable"; -const horizontalPodAutoScalersSidebarItemInjectable = getInjectable({ - id: "horizontal-pod-auto-scalers-sidebar-item", +const horizontalPodAutoScalersSidebarItemsInjectable = getInjectable({ + id: "horizontal-pod-auto-scalers-sidebar-items", instantiate: (di) => { const route = di.inject(horizontalPodAutoscalersRouteInjectable); + const navigateToHorizontalPodAutoscalers = di.inject(navigateToHorizontalPodAutoscalersInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "horizontal-pod-auto-scalers", - parentId: di.inject(configSidebarItemInjectable).id, - title: "HPA", - onClick: di.inject(navigateToHorizontalPodAutoscalersInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 50, - }; + return computed((): SidebarItemRegistration[] => [ + { + id: "horizontal-pod-auto-scalers", + parentId: configSidebarItemId, + title: "HPA", + onClick: navigateToHorizontalPodAutoscalers, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 50, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default horizontalPodAutoScalersSidebarItemInjectable; +export default horizontalPodAutoScalersSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-leases/leases-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config-leases/leases-sidebar-items.injectable.tsx index c2031ff11b5f..806749664a28 100644 --- a/packages/core/src/renderer/components/config-leases/leases-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/config-leases/leases-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import leasesRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/leases/leases-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToLeasesInjectable from "../../../common/front-end-routing/routes/cluster/config/leases/navigate-to-leases.injectable"; -const leasesSidebarItemInjectable = getInjectable({ - id: "leases-sidebar-item", +const leasesSidebarItemsInjectable = getInjectable({ + id: "leases-sidebar-items", instantiate: (di) => { const route = di.inject(leasesRouteInjectable); + const navigateToLeases = di.inject(navigateToLeasesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "leases", - parentId: di.inject(configSidebarItemInjectable).id, - title: "Leases", - onClick: di.inject(navigateToLeasesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 80, - }; + return computed(() => [ + { + id: "leases", + parentId: configSidebarItemId, + title: "Leases", + onClick: navigateToLeases, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 80, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default leasesSidebarItemInjectable; +export default leasesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-limit-ranges/limit-ranges-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config-limit-ranges/limit-ranges-sidebar-items.injectable.tsx index d9e627a0a18a..55b629737074 100644 --- a/packages/core/src/renderer/components/config-limit-ranges/limit-ranges-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/config-limit-ranges/limit-ranges-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import limitRangesRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/limit-ranges/limit-ranges-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToLimitRangesInjectable from "../../../common/front-end-routing/routes/cluster/config/limit-ranges/navigate-to-limit-ranges.injectable"; -const limitRangesSidebarItemInjectable = getInjectable({ - id: "limit-ranges-sidebar-item", +const limitRangesSidebarItemsInjectable = getInjectable({ + id: "limit-ranges-sidebar-items", instantiate: (di) => { const route = di.inject(limitRangesRouteInjectable); + const navigateToLimitRanges = di.inject(navigateToLimitRangesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "limit-ranges", - parentId: di.inject(configSidebarItemInjectable).id, - title: "Limit Ranges", - onClick: di.inject(navigateToLimitRangesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 40, - }; + return computed(() => [ + { + id: "limit-ranges", + parentId: configSidebarItemId, + title: "Limit Ranges", + onClick: navigateToLimitRanges, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 40, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default limitRangesSidebarItemInjectable; +export default limitRangesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-maps/config-maps-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config-maps/config-maps-sidebar-items.injectable.tsx index 53ff516c538b..90f776f1c36a 100644 --- a/packages/core/src/renderer/components/config-maps/config-maps-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/config-maps/config-maps-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import configMapsRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/config-maps/config-maps-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToConfigMapsInjectable from "../../../common/front-end-routing/routes/cluster/config/config-maps/navigate-to-config-maps.injectable"; -const configMapsSidebarItemInjectable = getInjectable({ - id: "config-maps-sidebar-item", +const configMapsSidebarItemsInjectable = getInjectable({ + id: "config-maps-sidebar-items", instantiate: (di) => { const route = di.inject(configMapsRouteInjectable); + const navigateToConfigMaps = di.inject(navigateToConfigMapsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "config-maps", - parentId: di.inject(configSidebarItemInjectable).id, - title: "ConfigMaps", - onClick: di.inject(navigateToConfigMapsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 10, - }; + return computed(() => [ + { + id: "config-maps", + parentId: configSidebarItemId, + title: "ConfigMaps", + onClick: navigateToConfigMaps, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 10, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default configMapsSidebarItemInjectable; +export default configMapsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-mutating-webhook-configurations/mutating-webhook-configurations-sidebar-items.injectable.ts b/packages/core/src/renderer/components/config-mutating-webhook-configurations/mutating-webhook-configurations-sidebar-items.injectable.ts index 8dc30d90a4b2..a897adbfa3ec 100644 --- a/packages/core/src/renderer/components/config-mutating-webhook-configurations/mutating-webhook-configurations-sidebar-items.injectable.ts +++ b/packages/core/src/renderer/components/config-mutating-webhook-configurations/mutating-webhook-configurations-sidebar-items.injectable.ts @@ -3,30 +3,37 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { computed } from "mobx"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; -import mutatingWebhookConfigurationsRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/mutating-webhook-configurations/mutating-webhook-configurations-route.injectable"; -import navigateToMutatingWebhookConfigurationsInjectable from "../../../common/front-end-routing/routes/cluster/config/mutating-webhook-configurations/navigate-to-mutating-webhook-configurations.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; +import mutatingWebhookConfigurationsRouteInjectable + from "../../../common/front-end-routing/routes/cluster/config/mutating-webhook-configurations/mutating-webhook-configurations-route.injectable"; +import navigateToMutatingWebhookConfigurationsInjectable + from "../../../common/front-end-routing/routes/cluster/config/mutating-webhook-configurations/navigate-to-mutating-webhook-configurations.injectable"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; -const mutatingWebhookConfigurationsSidebarItemInjectable = getInjectable({ - id: "mutating-webhook-configurations-sidebar-item", +const mutatingWebhookConfigurationsSidebarItemsInjectable = getInjectable({ + id: "mutating-webhook-configurations-sidebar-items", instantiate: (di) => { const route = di.inject(mutatingWebhookConfigurationsRouteInjectable); + const navigateToPage = di.inject(navigateToMutatingWebhookConfigurationsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "mutating-webhook-configurations", - parentId: di.inject(configSidebarItemInjectable).id, - title: "Mutating Webhook Configs", - onClick: di.inject(navigateToMutatingWebhookConfigurationsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 100, - }; + return computed(() => [ + { + id: "mutating-webhook-configurations", + parentId: configSidebarItemId, + title: "Mutating Webhook Configs", + onClick: navigateToPage, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 100, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default mutatingWebhookConfigurationsSidebarItemInjectable; +export default mutatingWebhookConfigurationsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-pod-disruption-budgets/pod-disruption-budgets-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config-pod-disruption-budgets/pod-disruption-budgets-sidebar-items.injectable.tsx index a44ce1b9b9a2..51f6fd70a9c1 100644 --- a/packages/core/src/renderer/components/config-pod-disruption-budgets/pod-disruption-budgets-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/config-pod-disruption-budgets/pod-disruption-budgets-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import podDisruptionBudgetsRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/pod-disruption-budgets/pod-disruption-budgets-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToPodDisruptionBudgetsInjectable from "../../../common/front-end-routing/routes/cluster/config/pod-disruption-budgets/navigate-to-pod-disruption-budgets.injectable"; -const podDisruptionBudgetsSidebarItemInjectable = getInjectable({ - id: "pod-disruption-budgets-sidebar-item", +const podDisruptionBudgetsSidebarItemsInjectable = getInjectable({ + id: "pod-disruption-budgets-sidebar-items", instantiate: (di) => { const route = di.inject(podDisruptionBudgetsRouteInjectable); + const navigateToPodDisruptionBudgets = di.inject(navigateToPodDisruptionBudgetsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "pod-disruption-budgets", - parentId: di.inject(configSidebarItemInjectable).id, - title: "Pod Disruption Budgets", - onClick: di.inject(navigateToPodDisruptionBudgetsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 60, - }; + return computed(() => [ + { + id: "pod-disruption-budgets", + parentId: configSidebarItemId, + title: "Pod Disruption Budgets", + onClick: navigateToPodDisruptionBudgets, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 60, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default podDisruptionBudgetsSidebarItemInjectable; +export default podDisruptionBudgetsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-priority-classes/priority-classes-items.injectable.tsx b/packages/core/src/renderer/components/config-priority-classes/priority-classes-items.injectable.tsx index 9ba31243ea15..be4ed383a95c 100644 --- a/packages/core/src/renderer/components/config-priority-classes/priority-classes-items.injectable.tsx +++ b/packages/core/src/renderer/components/config-priority-classes/priority-classes-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import priorityClassesRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/priority-classes/priority-classes-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToPriorityClassesInjectable from "../../../common/front-end-routing/routes/cluster/config/priority-classes/navigate-to-priority-classes.injectable"; -const priorityClassesSidebarItemInjectable = getInjectable({ - id: "priority-classes-sidebar-item", +const priorityClassesSidebarItemsInjectable = getInjectable({ + id: "priority-classes-sidebar-items", instantiate: (di) => { const route = di.inject(priorityClassesRouteInjectable); + const navigateToPriorityClasses = di.inject(navigateToPriorityClassesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "priority-classes", - parentId: di.inject(configSidebarItemInjectable).id, - title: "Priority Classes", - onClick: di.inject(navigateToPriorityClassesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 70, - }; + return computed(() => [ + { + id: "priority-classes", + parentId: configSidebarItemId, + title: "Priority Classes", + onClick: navigateToPriorityClasses, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 70, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default priorityClassesSidebarItemInjectable; +export default priorityClassesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-resource-quotas/resource-quotas-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config-resource-quotas/resource-quotas-sidebar-items.injectable.tsx index be84cdf18511..51eddb15a48f 100644 --- a/packages/core/src/renderer/components/config-resource-quotas/resource-quotas-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/config-resource-quotas/resource-quotas-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import resourceQuotasRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/resource-quotas/resource-quotas-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToResourceQuotasInjectable from "../../../common/front-end-routing/routes/cluster/config/resource-quotas/navigate-to-resource-quotas.injectable"; -const resourceQuotasSidebarItemInjectable = getInjectable({ - id: "resource-quotas-sidebar-item", +const resourceQuotasSidebarItemsInjectable = getInjectable({ + id: "resource-quotas-sidebar-items", instantiate: (di) => { const route = di.inject(resourceQuotasRouteInjectable); + const navigateToResourceQuotas = di.inject(navigateToResourceQuotasInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "resource-quotas", - parentId: di.inject(configSidebarItemInjectable).id, - title: "Resource Quotas", - onClick: di.inject(navigateToResourceQuotasInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 30, - }; + return computed(() => [ + { + id: "resource-quotas", + parentId: configSidebarItemId, + title: "Resource Quotas", + onClick: navigateToResourceQuotas, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 30, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default resourceQuotasSidebarItemInjectable; +export default resourceQuotasSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-runtime-classes/runtime-classes-items.injectable.tsx b/packages/core/src/renderer/components/config-runtime-classes/runtime-classes-items.injectable.tsx new file mode 100644 index 000000000000..7cd7c796636e --- /dev/null +++ b/packages/core/src/renderer/components/config-runtime-classes/runtime-classes-items.injectable.tsx @@ -0,0 +1,38 @@ +/** + * Copyright (c) OpenLens Authors. All rights reserved. + * Licensed under MIT License. See LICENSE in root directory for more information. + */ +import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; + +import runtimeClassesRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/runtime-classes/runtime-classes-route.injectable"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; +import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; +import navigateToRuntimeClassesInjectable from "../../../common/front-end-routing/routes/cluster/config/runtime-classes/navigate-to-runtime-classes.injectable"; + +const runtimeClassesSidebarItemsInjectable = getInjectable({ + id: "runtime-classes-sidebar-items", + + instantiate: (di) => { + const route = di.inject(runtimeClassesRouteInjectable); + const navigateToRuntimeClasses = di.inject(navigateToRuntimeClassesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); + + return computed(() => [ + { + id: "runtime-classes", + parentId: configSidebarItemId, + title: "Runtime Classes", + onClick: navigateToRuntimeClasses, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 70, + }, + ]); + }, + + injectionToken: sidebarItemsInjectionToken, +}); + +export default runtimeClassesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-runtime-classes/runtime-classes-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config-runtime-classes/runtime-classes-sidebar-items.injectable.tsx deleted file mode 100644 index b059712b7904..000000000000 --- a/packages/core/src/renderer/components/config-runtime-classes/runtime-classes-sidebar-items.injectable.tsx +++ /dev/null @@ -1,33 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { getInjectable } from "@ogre-tools/injectable"; - -import runtimeClassesRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/runtime-classes/runtime-classes-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; -import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; -import navigateToRuntimeClassesInjectable from "../../../common/front-end-routing/routes/cluster/config/runtime-classes/navigate-to-runtime-classes.injectable"; - -const runtimeClassesSidebarItemInjectable = getInjectable({ - id: "runtime-classes-sidebar-item", - - instantiate: (di) => { - const route = di.inject(runtimeClassesRouteInjectable); - - return { - id: "runtime-classes", - parentId: di.inject(configSidebarItemInjectable).id, - title: "Runtime Classes", - onClick: di.inject(navigateToRuntimeClassesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 70, - }; - }, - - injectionToken: sidebarItemInjectionToken, -}); - -export default runtimeClassesSidebarItemInjectable; diff --git a/packages/core/src/renderer/components/config-secrets/secrets-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config-secrets/secrets-sidebar-items.injectable.tsx index 51cd1a2f9b8d..834f9f756c43 100644 --- a/packages/core/src/renderer/components/config-secrets/secrets-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/config-secrets/secrets-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import secretsRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/secrets/secrets-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToSecretsInjectable from "../../../common/front-end-routing/routes/cluster/config/secrets/navigate-to-secrets.injectable"; -const secretsSidebarItemInjectable = getInjectable({ - id: "secrets-sidebar-item", +const secretsSidebarItemsInjectable = getInjectable({ + id: "secrets-sidebar-items", instantiate: (di) => { const route = di.inject(secretsRouteInjectable); + const navigateToSecrets = di.inject(navigateToSecretsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "secrets", - parentId: di.inject(configSidebarItemInjectable).id, - title: "Secrets", - onClick: di.inject(navigateToSecretsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 20, - }; + return computed(() => [ + { + id: "secrets", + parentId: configSidebarItemId, + title: "Secrets", + onClick: navigateToSecrets, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 20, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default secretsSidebarItemInjectable; +export default secretsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-validating-webhook-configurations/validating-webhook-configurations-sidebar-items.injectable.ts b/packages/core/src/renderer/components/config-validating-webhook-configurations/validating-webhook-configurations-sidebar-items.injectable.ts index 875fa3efdf08..5717be3dd4d7 100644 --- a/packages/core/src/renderer/components/config-validating-webhook-configurations/validating-webhook-configurations-sidebar-items.injectable.ts +++ b/packages/core/src/renderer/components/config-validating-webhook-configurations/validating-webhook-configurations-sidebar-items.injectable.ts @@ -3,30 +3,37 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { computed } from "mobx"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; -import validatingWebhookConfigurationsRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/validating-webhook-configurations/validating-webhook-configurations-route.injectable"; -import navigateToValidatingWebhookConfigurationsInjectable from "../../../common/front-end-routing/routes/cluster/config/validating-webhook-configurations/navigate-to-validating-webhook-configurations.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; +import validatingWebhookConfigurationsRouteInjectable + from "../../../common/front-end-routing/routes/cluster/config/validating-webhook-configurations/validating-webhook-configurations-route.injectable"; +import navigateToValidatingWebhookConfigurationsInjectable + from "../../../common/front-end-routing/routes/cluster/config/validating-webhook-configurations/navigate-to-validating-webhook-configurations.injectable"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; -const validatingWebhookConfigurationsSidebarItemInjectable = getInjectable({ - id: "validating-webhook-configurations-sidebar-item", +const validatingWebhookConfigurationsSidebarItemsInjectable = getInjectable({ + id: "validating-webhook-configurations-sidebar-items", instantiate: (di) => { const route = di.inject(validatingWebhookConfigurationsRouteInjectable); + const navigateToPage = di.inject(navigateToValidatingWebhookConfigurationsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "validating-webhook-configurations", - parentId: di.inject(configSidebarItemInjectable).id, - title: "Validating Webhook Configs", - onClick: di.inject(navigateToValidatingWebhookConfigurationsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 100, - }; + return computed(() => [ + { + id: "validating-webhook-configurations", + parentId: configSidebarItemId, + title: "Validating Webhook Configs", + onClick: navigateToPage, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 100, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default validatingWebhookConfigurationsSidebarItemInjectable; +export default validatingWebhookConfigurationsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config-vertical-pod-autoscalers/vertical-pod-auto-scalers-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config-vertical-pod-autoscalers/vertical-pod-auto-scalers-sidebar-items.injectable.tsx index ead9ccfe0753..0c9835a33c5f 100644 --- a/packages/core/src/renderer/components/config-vertical-pod-autoscalers/vertical-pod-auto-scalers-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/config-vertical-pod-autoscalers/vertical-pod-auto-scalers-sidebar-items.injectable.tsx @@ -3,30 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import verticalPodAutoscalersRouteInjectable from "../../../common/front-end-routing/routes/cluster/config/vertical-pod-autoscalers/vertical-pod-autoscalers-route.injectable"; -import configSidebarItemInjectable from "../config/config-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { configSidebarItemId } from "../config/config-sidebar-items.injectable"; +import type { SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToVerticalPodAutoscalersInjectable from "../../../common/front-end-routing/routes/cluster/config/vertical-pod-autoscalers/navigate-to-vertical-pod-autoscalers.injectable"; -const verticalPodAutoScalersSidebarItemInjectable = getInjectable({ - id: "vertical-pod-auto-scalers-sidebar-item", +const verticalPodAutoScalersSidebarItemsInjectable = getInjectable({ + id: "vertical-pod-auto-scalers-sidebar-items", instantiate: (di) => { const route = di.inject(verticalPodAutoscalersRouteInjectable); + const navigateToVerticalPodAutoscalers = di.inject(navigateToVerticalPodAutoscalersInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "vertical-pod-auto-scalers", - parentId: di.inject(configSidebarItemInjectable).id, - title: "VPA", - onClick: di.inject(navigateToVerticalPodAutoscalersInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 50, - }; + return computed((): SidebarItemRegistration[] => [ + { + id: "vertical-pod-auto-scalers", + parentId: configSidebarItemId, + title: "VPA", + onClick: navigateToVerticalPodAutoscalers, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 50, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default verticalPodAutoScalersSidebarItemInjectable; +export default verticalPodAutoScalersSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/config/config-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/config/config-sidebar-items.injectable.tsx index 1adaa623a5ed..0730954aff67 100644 --- a/packages/core/src/renderer/components/config/config-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/config/config-sidebar-items.injectable.tsx @@ -5,22 +5,32 @@ import { getInjectable } from "@ogre-tools/injectable"; import { Icon } from "../icon"; import React from "react"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; +import { computed } from "mobx"; import { noop } from "lodash/fp"; -const configSidebarItemInjectable = getInjectable({ - id: "config-sidebar-item", +export const configSidebarItemId = "config"; - instantiate: () => ({ - id: "config", - parentId: null, - title: "Config", - getIcon: () => , - onClick: noop, - orderNumber: 40, - }), +const configSidebarItemsInjectable = getInjectable({ + id: "config-sidebar-items", - injectionToken: sidebarItemInjectionToken, + instantiate: () => + computed((): SidebarItemRegistration[] => [ + { + id: configSidebarItemId, + parentId: null, + title: "Config", + getIcon: () => , + onClick: noop, + orderNumber: 40, + }, + ]), + + injectionToken: sidebarItemsInjectionToken, }); -export default configSidebarItemInjectable; +export default configSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/custom-resources/crd-list-route-component.injectable.ts b/packages/core/src/renderer/components/custom-resources/crd-list-route-component.injectable.ts index be1823e24763..55e4a2100671 100644 --- a/packages/core/src/renderer/components/custom-resources/crd-list-route-component.injectable.ts +++ b/packages/core/src/renderer/components/custom-resources/crd-list-route-component.injectable.ts @@ -4,7 +4,7 @@ */ import { getInjectable } from "@ogre-tools/injectable"; import { routeSpecificComponentInjectionToken } from "../../routes/route-specific-component-injection-token"; -import crdListRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/custom-resource-definitions.injectable"; +import crdListRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/crd-list/crd-list-route.injectable"; import { CustomResourceDefinitions } from "./crd-list"; const crdListRouteComponentInjectable = getInjectable({ diff --git a/packages/core/src/renderer/components/custom-resources/custom-resource-definition-groups-sidebar-items-computed.injectable.ts b/packages/core/src/renderer/components/custom-resources/custom-resource-definition-groups-sidebar-items-computed.injectable.ts deleted file mode 100644 index 4c68b2dc17ec..000000000000 --- a/packages/core/src/renderer/components/custom-resources/custom-resource-definition-groups-sidebar-items-computed.injectable.ts +++ /dev/null @@ -1,86 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import type { SidebarItemRegistration } from "@k8slens/cluster-sidebar"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; -import type { CustomResourceDefinition } from "@k8slens/kube-object"; -import { iter, noop, computedAnd } from "@k8slens/utilities"; -import { getInjectable } from "@ogre-tools/injectable"; -import { matches } from "lodash"; -import { computed } from "mobx"; -import customResourcesRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/custom-resources-route.injectable"; -import navigateToCustomResourcesInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/navigate-to-custom-resources.injectable"; -import { shouldShowResourceInjectionToken } from "../../../features/cluster/showing-kube-resources/common/allowed-resources-injection-token"; -import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; -import routePathParametersInjectable from "../../routes/route-path-parameters.injectable"; -import customResourcesSidebarItemInjectable from "./custom-resources-sidebar-items.injectable"; -import customResourceDefinitionsInjectable from "./custom-resources.injectable"; - -const customResourceDefinitionGroupsSidebarItemsComputedInjectable = getInjectable({ - id: "custom-resource-definition-groups-sidebar-items-computed", - instantiate: (di) => { - const customResourceDefinitions = di.inject(customResourceDefinitionsInjectable); - const navigateToCustomResources = di.inject(navigateToCustomResourcesInjectable); - const customResourcesRoute = di.inject(customResourcesRouteInjectable); - const pathParameters = di.inject(routePathParametersInjectable, customResourcesRoute); - - const toCustomResourceGroupToSidebarItems = ([group, definitions]: [string, CustomResourceDefinition[]], index: number) => { - const customResourceGroupSidebarItem = getInjectable({ - id: `custom-resource-group-${group}-sidebar-item`, - instantiate: (di): SidebarItemRegistration => ({ - id: `custom-resource-group-${group}`, - parentId: di.inject(customResourcesSidebarItemInjectable).id, - onClick: noop, - title: group, - orderNumber: 10 * index, - }), - injectionToken: sidebarItemInjectionToken, - }); - const customResourceSidebarItems = definitions.map((definition, index) => { - const parameters = { - group: definition.getGroup(), - name: definition.getPluralName(), - }; - - return getInjectable({ - id: `custom-resource-group-${group}/${definition.getPluralName()}-sidebar-item`, - instantiate: (di): SidebarItemRegistration => ({ - id: `custom-resource-group-${group}/${definition.getPluralName()}`, - parentId: di.inject(customResourceGroupSidebarItem).id, - onClick: () => navigateToCustomResources(parameters), - title: definition.getResourceKind(), - isActive: computedAnd( - di.inject(routeIsActiveInjectable, customResourcesRoute), - computed(() => matches(parameters)(pathParameters.get())), - ), - isVisible: di.inject(shouldShowResourceInjectionToken, { - group: definition.getGroup(), - apiName: definition.getPluralName(), - }), - orderNumber: 10 * index, - }), - injectionToken: sidebarItemInjectionToken, - }); - }); - - return [ - customResourceGroupSidebarItem, - ...customResourceSidebarItems, - ]; - }; - - return computed(() => { - const customResourceDefinitionGroups = ( - iter.chain(customResourceDefinitions.get().values()) - .map((crd) => [crd.getGroup(), crd] as const) - .toMap() - ); - - return Array.from(customResourceDefinitionGroups.entries(), toCustomResourceGroupToSidebarItems) - .flat(); - }); - }, -}); - -export default customResourceDefinitionGroupsSidebarItemsComputedInjectable; diff --git a/packages/core/src/renderer/components/custom-resources/custom-resource-definition-groups-sidebar-items-registrator.injectable.ts b/packages/core/src/renderer/components/custom-resources/custom-resource-definition-groups-sidebar-items-registrator.injectable.ts deleted file mode 100644 index 979eb8882d6f..000000000000 --- a/packages/core/src/renderer/components/custom-resources/custom-resource-definition-groups-sidebar-items-registrator.injectable.ts +++ /dev/null @@ -1,29 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { getInjectable } from "@ogre-tools/injectable"; -import { reaction } from "mobx"; -import { injectableDifferencingRegistratorWith } from "../../../common/utils/registrator-helper"; -import { beforeClusterFrameStartsSecondInjectionToken } from "../../before-frame-starts/tokens"; -import customResourceDefinitionGroupsSidebarItemsComputedInjectable from "./custom-resource-definition-groups-sidebar-items-computed.injectable"; - -const customResourceDefinitionGroupsSidebarItemsRegistratorInjectable = getInjectable({ - id: "custom-resource-definition-groups-sidebar-items-registrator", - instantiate: (di) => ({ - run: () => { - const sidebarItems = di.inject(customResourceDefinitionGroupsSidebarItemsComputedInjectable); - const injectableDifferencingRegistrator = injectableDifferencingRegistratorWith(di); - - reaction( - () => sidebarItems.get(), - injectableDifferencingRegistrator, - { fireImmediately: true }, - ); - }, - }), - injectionToken: beforeClusterFrameStartsSecondInjectionToken, -}); - -export default customResourceDefinitionGroupsSidebarItemsRegistratorInjectable; - diff --git a/packages/core/src/renderer/components/custom-resources/custom-resource-definitions-sidebar-item.injectable.ts b/packages/core/src/renderer/components/custom-resources/custom-resource-definitions-sidebar-item.injectable.ts deleted file mode 100644 index d32c6026b683..000000000000 --- a/packages/core/src/renderer/components/custom-resources/custom-resource-definitions-sidebar-item.injectable.ts +++ /dev/null @@ -1,30 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; -import { getInjectable } from "@ogre-tools/injectable"; -import customResourceDefinitionsRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/custom-resource-definitions.injectable"; -import navigateToCustomResourcesInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/navigate-to-custom-resources.injectable"; -import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; -import customResourcesSidebarItemInjectable from "./custom-resources-sidebar-items.injectable"; - -const customResourceDefinitionsSidebarItemInjectable = getInjectable({ - id: "custom-resource-definitions-sidebar-item", - instantiate: (di) => { - const customResourceDefinitionsRoute = di.inject(customResourceDefinitionsRouteInjectable); - - return { - id: "custom-resource-definitions", - parentId: di.inject(customResourcesSidebarItemInjectable).id, - title: "Definitions", - onClick: di.inject(navigateToCustomResourcesInjectable), - isActive: di.inject(routeIsActiveInjectable, customResourceDefinitionsRoute), - isVisible: customResourceDefinitionsRoute.isEnabled, - orderNumber: 10, - }; - }, - injectionToken: sidebarItemInjectionToken, -}); - -export default customResourceDefinitionsSidebarItemInjectable; diff --git a/packages/core/src/renderer/components/custom-resources/custom-resource-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/custom-resources/custom-resource-sidebar-items.injectable.tsx new file mode 100644 index 000000000000..973a0756e69f --- /dev/null +++ b/packages/core/src/renderer/components/custom-resources/custom-resource-sidebar-items.injectable.tsx @@ -0,0 +1,59 @@ +/** + * Copyright (c) OpenLens Authors. All rights reserved. + * Licensed under MIT License. See LICENSE in root directory for more information. + */ +import { getInjectable } from "@ogre-tools/injectable"; +import { noop } from "lodash/fp"; +import { computed } from "mobx"; +import type { SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; +import { Icon } from "../icon"; +import React from "react"; +import crdListRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/crd-list/crd-list-route.injectable"; +import sidebarItemsForDefinitionGroupsInjectable from "./sidebar-items-for-definition-groups.injectable"; +import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; +import navigateToCrdListInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/crd-list/navigate-to-crd-list.injectable"; + +const customResourceSidebarItemsInjectable = getInjectable({ + id: "custom-resource-sidebar-items", + + instantiate: (di) => { + const navigateToCrdList = di.inject(navigateToCrdListInjectable); + const crdListRoute = di.inject(crdListRouteInjectable); + const crdListRouteIsActive = di.inject(routeIsActiveInjectable, crdListRoute); + const definitionGroupSidebarItems = di.inject(sidebarItemsForDefinitionGroupsInjectable); + + return computed((): SidebarItemRegistration[] => { + const definitionsItem = { + id: "definitions", + parentId: "custom-resources", + title: "Definitions", + onClick: navigateToCrdList, + isActive: crdListRouteIsActive, + isVisible: crdListRoute.isEnabled, + orderNumber: 10, + }; + + const childrenAndGrandChildren = computed(() => [ + definitionsItem, + ...definitionGroupSidebarItems.get(), + ]); + + const parentItem: SidebarItemRegistration = { + id: "custom-resources", + parentId: null, + title: "Custom Resources", + getIcon: () => , + onClick: noop, + isVisible: computed(() => childrenAndGrandChildren.get().some(item => item.isVisible?.get())), + orderNumber: 110, + }; + + return [parentItem, definitionsItem, ...definitionGroupSidebarItems.get()]; + }); + }, + + injectionToken: sidebarItemsInjectionToken, +}); + +export default customResourceSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/custom-resources/custom-resources-route-component.injectable.ts b/packages/core/src/renderer/components/custom-resources/custom-resources-route-component.injectable.ts index 9d7d165b7408..b31c5577d44b 100644 --- a/packages/core/src/renderer/components/custom-resources/custom-resources-route-component.injectable.ts +++ b/packages/core/src/renderer/components/custom-resources/custom-resources-route-component.injectable.ts @@ -4,7 +4,7 @@ */ import { getInjectable } from "@ogre-tools/injectable"; import { CustomResources } from "./crd-resources"; -import customResourcesRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/custom-resources-route.injectable"; +import customResourcesRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/custom-resources/custom-resources-route.injectable"; import { routeSpecificComponentInjectionToken } from "../../routes/route-specific-component-injection-token"; const customResourcesRouteComponentInjectable = getInjectable({ diff --git a/packages/core/src/renderer/components/custom-resources/custom-resources-route-parameters.injectable.ts b/packages/core/src/renderer/components/custom-resources/custom-resources-route-parameters.injectable.ts index 69368218fd97..2debbe85bd84 100644 --- a/packages/core/src/renderer/components/custom-resources/custom-resources-route-parameters.injectable.ts +++ b/packages/core/src/renderer/components/custom-resources/custom-resources-route-parameters.injectable.ts @@ -5,7 +5,7 @@ import { getInjectable } from "@ogre-tools/injectable"; import { computed } from "mobx"; import routePathParametersInjectable from "../../routes/route-path-parameters.injectable"; -import customResourcesRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/custom-resources-route.injectable"; +import customResourcesRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/custom-resources/custom-resources-route.injectable"; const customResourcesRouteParametersInjectable = getInjectable({ id: "custom-resources-route-parameters", diff --git a/packages/core/src/renderer/components/custom-resources/custom-resources-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/custom-resources/custom-resources-sidebar-items.injectable.tsx deleted file mode 100644 index 1487d829271f..000000000000 --- a/packages/core/src/renderer/components/custom-resources/custom-resources-sidebar-items.injectable.tsx +++ /dev/null @@ -1,24 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; -import { noop } from "@k8slens/utilities"; -import { getInjectable } from "@ogre-tools/injectable"; -import React from "react"; -import { Icon } from "../icon"; - -const customResourcesSidebarItemInjectable = getInjectable({ - id: "custom-resources-sidebar-item", - instantiate: () => ({ - id: "custom-resources", - parentId: null, - title: "Custom Resources", - getIcon: () => , - onClick: noop, - orderNumber: 110, - }), - injectionToken: sidebarItemInjectionToken, -}); - -export default customResourcesSidebarItemInjectable; diff --git a/packages/core/src/renderer/components/custom-resources/sidebar-items-for-definition-groups.injectable.ts b/packages/core/src/renderer/components/custom-resources/sidebar-items-for-definition-groups.injectable.ts new file mode 100644 index 000000000000..895df08dc4be --- /dev/null +++ b/packages/core/src/renderer/components/custom-resources/sidebar-items-for-definition-groups.injectable.ts @@ -0,0 +1,81 @@ +/** + * Copyright (c) OpenLens Authors. All rights reserved. + * Licensed under MIT License. See LICENSE in root directory for more information. + */ +import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; +import crdListRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/crd-list/crd-list-route.injectable"; +import customResourceDefinitionsInjectable from "./custom-resources.injectable"; +import { groupBy, matches, noop, some, toPairs } from "lodash/fp"; +import customResourcesRouteInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/custom-resources/custom-resources-route.injectable"; +import type { SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; +import navigateToCustomResourcesInjectable from "../../../common/front-end-routing/routes/cluster/custom-resources/custom-resources/navigate-to-custom-resources.injectable"; +import routePathParametersInjectable from "../../routes/route-path-parameters.injectable"; + +const sidebarItemsForDefinitionGroupsInjectable = getInjectable({ + id: "sidebar-items-for-definition-groups", + + instantiate: (di) => { + const customResourceDefinitions = di.inject( + customResourceDefinitionsInjectable, + ); + + const crdRoute = di.inject(customResourcesRouteInjectable); + const crdRouteIsActive = di.inject(routeIsActiveInjectable, crdRoute); + const crdListRoute = di.inject(crdListRouteInjectable); + const pathParameters = di.inject(routePathParametersInjectable, crdRoute); + const navigateToCustomResources = di.inject(navigateToCustomResourcesInjectable); + + return computed((): SidebarItemRegistration[] => { + const definitions = customResourceDefinitions.get(); + + const groupedCrds = toPairs( + groupBy((crd) => crd.getGroup(), definitions), + ); + + return groupedCrds.flatMap(([group, definitions]) => { + const childItems = definitions.map((crd) => { + const title = crd.getResourceKind(); + + const crdPathParameters = { + group: crd.getGroup(), + name: crd.getPluralName(), + }; + + return { + id: `custom-resource-definition-group-${group}-crd-${crd.getId()}`, + parentId: `custom-resource-definition-group-${group}`, + title, + + onClick: () => navigateToCustomResources(crdPathParameters), + + isActive: computed( + () => + crdRouteIsActive.get() && + matches(crdPathParameters, pathParameters.get()), + ), + + isVisible: crdListRoute.isEnabled, + orderNumber: 10, + }; + }); + + return [ + { + id: `custom-resource-definition-group-${group}`, + parentId: "custom-resources", + title: group, + onClick: noop, + isVisible: computed(() => some(item => item.isVisible.get(), childItems)), + orderNumber: 10, + }, + + ...childItems, + ]; + }); + }); + }, +}); + +export default sidebarItemsForDefinitionGroupsInjectable; diff --git a/packages/core/src/renderer/components/entity-settings/active-tabs.injectable.ts b/packages/core/src/renderer/components/entity-settings/active-tabs.injectable.ts index 1dabd579f277..ae488f234ecf 100644 --- a/packages/core/src/renderer/components/entity-settings/active-tabs.injectable.ts +++ b/packages/core/src/renderer/components/entity-settings/active-tabs.injectable.ts @@ -4,7 +4,7 @@ */ import { getInjectable, lifecycleEnum } from "@ogre-tools/injectable"; import { action } from "mobx"; -import { byOrderNumber } from "@k8slens/utilities"; +import { byOrderNumber } from "../../../common/utils/composable-responsibilities/orderable/orderable"; import type { CatalogEntity } from "../../api/catalog-entity"; import { observableHistoryInjectionToken } from "@k8slens/routing"; import type { RegisteredEntitySetting } from "./extension-registrator.injectable"; diff --git a/packages/core/src/renderer/components/entity-settings/settings.injectable.ts b/packages/core/src/renderer/components/entity-settings/settings.injectable.ts index dd519351fdd3..c70063f2b620 100644 --- a/packages/core/src/renderer/components/entity-settings/settings.injectable.ts +++ b/packages/core/src/renderer/components/entity-settings/settings.injectable.ts @@ -5,7 +5,7 @@ import { getInjectable, lifecycleEnum } from "@ogre-tools/injectable"; import { computedInjectManyInjectable } from "@ogre-tools/injectable-extension-for-mobx"; import { computed } from "mobx"; -import { byOrderNumber } from "@k8slens/utilities"; +import { byOrderNumber } from "../../../common/utils/composable-responsibilities/orderable/orderable"; import type { CatalogEntity } from "../../api/catalog-entity"; import { entitySettingInjectionToken } from "./token"; diff --git a/packages/core/src/renderer/components/events/events-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/events/events-sidebar-items.injectable.tsx index bec30a1e0803..02d965c85438 100644 --- a/packages/core/src/renderer/components/events/events-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/events/events-sidebar-items.injectable.tsx @@ -3,34 +3,42 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import React from "react"; - -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; import { Icon } from "../icon"; import eventsRouteInjectable from "../../../common/front-end-routing/routes/cluster/events/events-route.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToEventsInjectable from "../../../common/front-end-routing/routes/cluster/events/navigate-to-events.injectable"; -const eventsSidebarItemInjectable = getInjectable({ - id: "events-sidebar-item", +const eventsSidebarItemsInjectable = getInjectable({ + id: "events-sidebar-items", instantiate: (di) => { const route = di.inject(eventsRouteInjectable); + const navigateToEvents = di.inject(navigateToEventsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "events", - parentId: null, - getIcon: () => , - title: "Events", - onClick: di.inject(navigateToEventsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 80, - }; + return computed((): SidebarItemRegistration[] => [ + { + id: "events", + parentId: null, + getIcon: () => , + title: "Events", + onClick: navigateToEvents, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 80, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default eventsSidebarItemInjectable; +export default eventsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/helm-charts/helm-charts-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/helm-charts/helm-charts-sidebar-items.injectable.tsx index 9798bc5e32a4..271826aea77e 100644 --- a/packages/core/src/renderer/components/helm-charts/helm-charts-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/helm-charts/helm-charts-sidebar-items.injectable.tsx @@ -3,30 +3,35 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import helmChartsRouteInjectable from "../../../common/front-end-routing/routes/cluster/helm/charts/helm-charts-route.injectable"; -import helmSidebarItemInjectable from "../helm/helm-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { helmSidebarItemId } from "../helm/helm-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToHelmChartsInjectable from "../../../common/front-end-routing/routes/cluster/helm/charts/navigate-to-helm-charts.injectable"; -const helmChartsSidebarItemInjectable = getInjectable({ - id: "helm-charts-sidebar-item", +const helmChartsSidebarItemsInjectable = getInjectable({ + id: "helm-charts-sidebar-items", instantiate: (di) => { const route = di.inject(helmChartsRouteInjectable); + const navigateToHelmCharts = di.inject(navigateToHelmChartsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "charts", - parentId: di.inject(helmSidebarItemInjectable).id, - title: "Charts", - onClick: di.inject(navigateToHelmChartsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 10, - }; + return computed(() => [ + { + id: "charts", + parentId: helmSidebarItemId, + title: "Charts", + onClick: navigateToHelmCharts, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 10, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default helmChartsSidebarItemInjectable; +export default helmChartsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/helm-releases/helm-releases-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/helm-releases/helm-releases-sidebar-items.injectable.tsx index 0164a04988ed..0a47da38adb5 100644 --- a/packages/core/src/renderer/components/helm-releases/helm-releases-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/helm-releases/helm-releases-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import helmReleasesRouteInjectable from "../../../common/front-end-routing/routes/cluster/helm/releases/helm-releases-route.injectable"; -import helmSidebarItemInjectable from "../helm/helm-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { helmSidebarItemId } from "../helm/helm-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToHelmReleasesInjectable from "../../../common/front-end-routing/routes/cluster/helm/releases/navigate-to-helm-releases.injectable"; -const helmReleasesSidebarItemInjectable = getInjectable({ - id: "helm-releases-sidebar-item", +const helmReleasesSidebarItemsInjectable = getInjectable({ + id: "helm-releases-sidebar-items", instantiate: (di) => { const route = di.inject(helmReleasesRouteInjectable); + const navigateToHelmReleases = di.inject(navigateToHelmReleasesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "releases", - parentId: di.inject(helmSidebarItemInjectable).id, - title: "Releases", - onClick: di.inject(navigateToHelmReleasesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 20, - }; + return computed(() => [ + { + id: "releases", + parentId: helmSidebarItemId, + title: "Releases", + onClick: navigateToHelmReleases, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 20, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default helmReleasesSidebarItemInjectable; +export default helmReleasesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/helm/helm-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/helm/helm-sidebar-items.injectable.tsx index 0bad314b2230..c7085b382a03 100644 --- a/packages/core/src/renderer/components/helm/helm-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/helm/helm-sidebar-items.injectable.tsx @@ -3,24 +3,34 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import React from "react"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; import { Icon } from "../icon"; import { noop } from "lodash/fp"; -const helmSidebarItemInjectable = getInjectable({ - id: "helm-sidebar-item", +export const helmSidebarItemId = "helm"; - instantiate: () => ({ - id: "helm", - parentId: null, - getIcon: () => , - title: "Helm", - onClick: noop, - orderNumber: 90, - }), +const helmSidebarItemsInjectable = getInjectable({ + id: "helm-sidebar-items", - injectionToken: sidebarItemInjectionToken, + instantiate: () => + computed((): SidebarItemRegistration[] => [ + { + id: helmSidebarItemId, + parentId: null, + getIcon: () => , + title: "Helm", + onClick: noop, + orderNumber: 90, + }, + ]), + + injectionToken: sidebarItemsInjectionToken, }); -export default helmSidebarItemInjectable; +export default helmSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/layout/extension-sidebar-item-registrator.injectable.tsx b/packages/core/src/renderer/components/layout/extension-sidebar-item-registrator.injectable.tsx index ad00dba3f5ba..ea49cdbb5fc7 100644 --- a/packages/core/src/renderer/components/layout/extension-sidebar-item-registrator.injectable.tsx +++ b/packages/core/src/renderer/components/layout/extension-sidebar-item-registrator.injectable.tsx @@ -5,7 +5,8 @@ import { getInjectable } from "@ogre-tools/injectable"; import React from "react"; import { extensionRegistratorInjectionToken } from "../../../extensions/extension-loader/extension-registrator-injection-token"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import type { SidebarItemRegistration } from "./sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "./sidebar-items.injectable"; import { computed } from "mobx"; import routesInjectable from "../../routes/routes.injectable"; import { matches, noop } from "lodash/fp"; @@ -24,66 +25,71 @@ const extensionSidebarItemRegistratorInjectable = getInjectable({ const routes = di.inject(routesInjectable); const extensionShouldBeEnabledForClusterFrame = di.inject(extensionShouldBeEnabledForClusterFrameInjectable, extension); - return computed(() => { - const extensionRoutes = routes.get().filter(matches({ extension })); + const sidebarItemsForExtensionInjectable = getInjectable({ + id: `sidebar-items-for-extension-${extension.sanitizedExtensionId}`, + injectionToken: sidebarItemsInjectionToken, - return extension.clusterPageMenus.map((registration) => { - const targetRoutePath = getExtensionRoutePath( - extension, - registration.target?.pageId, - ); + instantiate: (di) => { + return computed(() => { + const extensionRoutes = routes.get().filter(matches({ extension })); - const targetRoute = extensionRoutes.find( - matches({ path: targetRoutePath }), - ); + return extension.clusterPageMenus.map((registration) => { + const targetRoutePath = getExtensionRoutePath( + extension, + registration.target?.pageId, + ); - const isVisible = computed(() => { - if (!extensionShouldBeEnabledForClusterFrame.value.get()) { - return false; - } + const targetRoute = extensionRoutes.find( + matches({ path: targetRoutePath }), + ); - if (!registration.visible) { - return true; - } - - return registration.visible.get(); - }); + const isVisible = computed(() => { + if (!extensionShouldBeEnabledForClusterFrame.value.get()) { + return false; + } - const id = registration.id ? - `${extension.sanitizedExtensionId}-${registration.id}` - : extension.sanitizedExtensionId; - - return getInjectable({ - id: `${id}-sidebar-item`, - instantiate: () => ({ - id, - orderNumber: registration.orderNumber ?? 9999, - - parentId: registration.parentId - ? `${extension.sanitizedExtensionId}-${registration.parentId}` - : null, - - isVisible, - - title: registration.title, - getIcon: registration.components.Icon - ? () => - : undefined, - ...(targetRoute - ? { - onClick: () => navigateToRoute(targetRoute), - - isActive: di.inject( - routeIsActiveInjectable, - targetRoute, - ), + if (!registration.visible) { + return true; } - : { onClick: noop }), - }), - injectionToken: sidebarItemInjectionToken, + + return registration.visible.get(); + }); + + const res: SidebarItemRegistration = { + id: `${extension.sanitizedExtensionId}-${registration.id}`, + orderNumber: registration.orderNumber ?? 9999, + + parentId: registration.parentId + ? `${extension.sanitizedExtensionId}-${registration.parentId}` + : null, + + isVisible, + + title: registration.title, + getIcon: registration.components.Icon + ? () => + : undefined, + ...(targetRoute + ? { + onClick: () => navigateToRoute(targetRoute), + + isActive: di.inject( + routeIsActiveInjectable, + targetRoute, + ), + } + : { onClick: noop }), + }; + + return res; + }); }); - }); + }, }); + + return [ + sidebarItemsForExtensionInjectable, + ]; }, injectionToken: extensionRegistratorInjectionToken, diff --git a/packages/core/src/renderer/components/layout/siblings-in-tab-layout.tsx b/packages/core/src/renderer/components/layout/siblings-in-tab-layout.tsx index a69de81c3202..ff32e8279435 100644 --- a/packages/core/src/renderer/components/layout/siblings-in-tab-layout.tsx +++ b/packages/core/src/renderer/components/layout/siblings-in-tab-layout.tsx @@ -8,7 +8,7 @@ import { observer } from "mobx-react"; import React from "react"; import siblingTabsInjectable from "../../routes/sibling-tabs.injectable"; import { TabLayout } from "./tab-layout-2"; -import type { HierarchicalSidebarItem } from "@k8slens/cluster-sidebar"; +import type { HierarchicalSidebarItem } from "./sidebar-items.injectable"; import type { StrictReactNode } from "@k8slens/utilities"; interface SiblingTabLayoutProps { diff --git a/packages/core/src/renderer/components/layout/sidebar-item.tsx b/packages/core/src/renderer/components/layout/sidebar-item.tsx index 9464a3574f79..e28332dcb74b 100644 --- a/packages/core/src/renderer/components/layout/sidebar-item.tsx +++ b/packages/core/src/renderer/components/layout/sidebar-item.tsx @@ -6,13 +6,15 @@ import styles from "./sidebar-items.module.scss"; import React from "react"; +import { computed, makeObservable } from "mobx"; +import { cssNames } from "@k8slens/utilities"; import { observer } from "mobx-react"; import { NavLink } from "react-router-dom"; import { Icon } from "../icon"; import { withInjectables } from "@ogre-tools/injectable-react"; import type { SidebarStorageState } from "./sidebar-storage/sidebar-storage.injectable"; import sidebarStorageInjectable from "./sidebar-storage/sidebar-storage.injectable"; -import type { HierarchicalSidebarItem } from "@k8slens/cluster-sidebar"; +import type { HierarchicalSidebarItem } from "./sidebar-items.injectable"; import type { StorageLayer } from "../../utils/storage-helper"; interface Dependencies { @@ -23,73 +25,97 @@ export interface SidebarItemProps { item: HierarchicalSidebarItem; } -const NonInjectedSidebarItem = observer((props: SidebarItemProps & Dependencies) => { - const { item, sidebarStorage } = props; - const id = item.id; - const expanded = sidebarStorage.get().expanded[id] ?? false; - const isExpandable = item.children.length > 0 && item.children.some(item => item.isVisible.get()); - const isActive = item.isActive.get(); +@observer +class NonInjectedSidebarItem extends React.Component< + SidebarItemProps & Dependencies +> { + static displayName = "SidebarItem"; - const toggleExpand = () => { - sidebarStorage.merge((draft) => { - draft.expanded[id] = !draft.expanded[id]; + constructor(props: SidebarItemProps & Dependencies) { + super(props); + makeObservable(this); + } + + get id(): string { + return this.item.id; + } + + @computed get expanded(): boolean { + return Boolean(this.props.sidebarStorage.get().expanded[this.id]); + } + + @computed get isExpandable(): boolean { + return this.props.item.children.length > 0; + } + + @computed get isActive(): boolean { + return this.props.item.isActive.get(); + } + + get item() { + return this.props.item; + } + + toggleExpand = () => { + this.props.sidebarStorage.merge((draft) => { + draft.expanded[this.id] = !draft.expanded[this.id]; }); }; - const renderSubMenu = () => { + + renderSubMenu() { + const { isExpandable, expanded } = this; + if (!isExpandable || !expanded) { return null; } return ( -
- {item.children.map((child) => ( - - ))} -
+
    + {this.props.item.children.map(item => )} +
); - }; - - if (!item.isVisible.get()) { - return null; } - return ( -
- isActive} - className={styles.navItem} - onClick={(event) => { - event.preventDefault(); - event.stopPropagation(); - - if (isExpandable) { - toggleExpand(); - } else { - item.onClick(); - } - }} - data-testid={`sidebar-item-link-for-${id}`} + render() { + return ( +
- {item.getIcon?.()} - {item.title} - {isExpandable && ( - - )} - - {renderSubMenu()} -
- ); -}); + this.isActive} + className={styles.navItem} + onClick={(event) => { + event.preventDefault(); + event.stopPropagation(); + + if (this.isExpandable) { + this.toggleExpand(); + } else { + this.item.onClick(); + } + }} + data-testid={`sidebar-item-link-for-${this.id}`} + > + {this.item.getIcon?.()} + {this.item.title} + {this.isExpandable && ( + + )} + + {this.renderSubMenu()} +
+ ); + } +} export const SidebarItem = withInjectables(NonInjectedSidebarItem, { getProps: (di, props) => ({ @@ -97,5 +123,3 @@ export const SidebarItem = withInjectables(NonIn sidebarStorage: di.inject(sidebarStorageInjectable), }), }); - -SidebarItem.displayName = "SidebarItem"; diff --git a/packages/core/src/renderer/components/layout/sidebar-items.injectable.ts b/packages/core/src/renderer/components/layout/sidebar-items.injectable.ts new file mode 100644 index 000000000000..fdcf98545960 --- /dev/null +++ b/packages/core/src/renderer/components/layout/sidebar-items.injectable.ts @@ -0,0 +1,80 @@ +/** + * Copyright (c) OpenLens Authors. All rights reserved. + * Licensed under MIT License. See LICENSE in root directory for more information. + */ +import { getInjectable, getInjectionToken } from "@ogre-tools/injectable"; +import type { IComputedValue } from "mobx"; +import { computed } from "mobx"; +import { computedInjectManyInjectable } from "@ogre-tools/injectable-extension-for-mobx"; +import { byOrderNumber } from "../../../common/utils/composable-responsibilities/orderable/orderable"; +import type { SetRequired } from "type-fest"; +import type { StrictReactNode } from "@k8slens/utilities"; + +export interface SidebarItemRegistration { + id: string; + parentId: string | null; + title: StrictReactNode; + onClick: () => void; + getIcon?: () => StrictReactNode; + isActive?: IComputedValue; + isVisible?: IComputedValue; + orderNumber: number; +} + +export const sidebarItemsInjectionToken = getInjectionToken< + IComputedValue +>({ id: "sidebar-items-injection-token" }); + +export interface HierarchicalSidebarItem extends SetRequired { + children: HierarchicalSidebarItem[]; +} + +const sidebarItemsInjectable = getInjectable({ + id: "sidebar-items", + + instantiate: (di) => { + const computedInjectMany = di.inject(computedInjectManyInjectable); + const sidebarItemRegistrations = computedInjectMany(sidebarItemsInjectionToken); + + return computed((): HierarchicalSidebarItem[] => { + const registrations = sidebarItemRegistrations + .get() + .flatMap(reg => reg.get()); + + const getSidebarItemsHierarchy = (registrations: SidebarItemRegistration[]) => { + const impl = (parentId: string | null): HierarchicalSidebarItem[] => ( + registrations + .filter((item) => item.parentId === parentId) + .map(({ + isActive = computed(() => false), + isVisible = computed(() => true), + ...registration + }) => { + const children = impl(registration.id); + + return { + ...registration, + children, + isVisible, + isActive: computed(() => { + if (children.length === 0) { + return isActive.get(); + } + + return children.some(child => child.isActive.get()); + }), + }; + }) + .filter(({ isVisible }) => isVisible.get()) + .sort(byOrderNumber) + ); + + return impl(null); + }; + + return getSidebarItemsHierarchy(registrations); + }); + }, +}); + +export default sidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/layout/sidebar.tsx b/packages/core/src/renderer/components/layout/sidebar.tsx index c13be0296179..f2171095157a 100644 --- a/packages/core/src/renderer/components/layout/sidebar.tsx +++ b/packages/core/src/renderer/components/layout/sidebar.tsx @@ -12,8 +12,8 @@ import { SidebarItem } from "./sidebar-item"; import type { CatalogEntityRegistry } from "../../api/catalog/entity/registry"; import { SidebarCluster } from "./sidebar-cluster"; import { withInjectables } from "@ogre-tools/injectable-react"; -import type { HierarchicalSidebarItem } from "@k8slens/cluster-sidebar"; -import { sidebarItemsInjectable } from "@k8slens/cluster-sidebar"; +import type { HierarchicalSidebarItem } from "./sidebar-items.injectable"; +import sidebarItemsInjectable from "./sidebar-items.injectable"; import type { IComputedValue } from "mobx"; import catalogEntityRegistryInjectable from "../../api/catalog/entity/registry.injectable"; diff --git a/packages/core/src/renderer/components/layout/tab-layout-2.tsx b/packages/core/src/renderer/components/layout/tab-layout-2.tsx index 537226b9db6c..eab234ed07ac 100644 --- a/packages/core/src/renderer/components/layout/tab-layout-2.tsx +++ b/packages/core/src/renderer/components/layout/tab-layout-2.tsx @@ -11,7 +11,7 @@ import type { StrictReactNode } from "@k8slens/utilities"; import { cssNames } from "@k8slens/utilities"; import { Tab, Tabs } from "../tabs"; import { ErrorBoundary } from "@k8slens/error-boundary"; -import type { HierarchicalSidebarItem } from "@k8slens/cluster-sidebar"; +import type { HierarchicalSidebarItem } from "./sidebar-items.injectable"; export interface TabLayoutProps { tabs?: HierarchicalSidebarItem[]; diff --git a/packages/core/src/renderer/components/namespaces/namespaces-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/namespaces/namespaces-sidebar-items.injectable.tsx index d2cee9e50aac..0a17b16fbd57 100644 --- a/packages/core/src/renderer/components/namespaces/namespaces-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/namespaces/namespaces-sidebar-items.injectable.tsx @@ -3,32 +3,41 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { computed } from "mobx"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; import { Icon } from "../icon"; import React from "react"; import namespacesRouteInjectable from "../../../common/front-end-routing/routes/cluster/namespaces/namespaces-route.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToNamespacesInjectable from "../../../common/front-end-routing/routes/cluster/namespaces/navigate-to-namespaces.injectable"; -const namespacesSidebarItemInjectable = getInjectable({ - id: "namespaces-sidebar-item", +const namespacesSidebarItemsInjectable = getInjectable({ + id: "namespaces", instantiate: (di) => { const route = di.inject(namespacesRouteInjectable); + const navigateToNamespaces = di.inject(navigateToNamespacesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "namespaces", - parentId: null, - getIcon: () => , - title: "Namespaces", - onClick: di.inject(navigateToNamespacesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 70, - }; + return computed((): SidebarItemRegistration[] => [ + { + id: "namespaces", + parentId: null, + getIcon: () => , + title: "Namespaces", + onClick: navigateToNamespaces, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 70, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default namespacesSidebarItemInjectable; +export default namespacesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/network-endpoints/endpoints-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/network-endpoints/endpoints-sidebar-items.injectable.tsx index 623e419ffb58..43df72d293d5 100644 --- a/packages/core/src/renderer/components/network-endpoints/endpoints-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/network-endpoints/endpoints-sidebar-items.injectable.tsx @@ -3,30 +3,35 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import endpointsRouteInjectable from "../../../common/front-end-routing/routes/cluster/network/endpoints/endpoints-route.injectable"; -import networkSidebarItemInjectable from "../network/network-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { networkSidebarItemId } from "../network/network-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToEndpointsInjectable from "../../../common/front-end-routing/routes/cluster/network/endpoints/navigate-to-endpoints.injectable"; -const endpointsSidebarItemInjectable = getInjectable({ - id: "endpoints-sidebar-item", +const endpointsSidebarItemsInjectable = getInjectable({ + id: "endpoints-sidebar-items", instantiate: (di) => { const route = di.inject(endpointsRouteInjectable); + const navigateToEndpoints = di.inject(navigateToEndpointsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "endpoints", - parentId: di.inject(networkSidebarItemInjectable).id, - title: "Endpoints", - onClick: di.inject(navigateToEndpointsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 20, - }; + return computed(() => [ + { + id: "endpoints", + parentId: networkSidebarItemId, + title: "Endpoints", + onClick: navigateToEndpoints, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 20, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default endpointsSidebarItemInjectable; +export default endpointsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/network-ingresses/ingress-class-route-component.injectable.ts b/packages/core/src/renderer/components/network-ingresses/ingress-class-route-component.injectable.ts index eda5dd666b44..fd7a5fbc9cdb 100644 --- a/packages/core/src/renderer/components/network-ingresses/ingress-class-route-component.injectable.ts +++ b/packages/core/src/renderer/components/network-ingresses/ingress-class-route-component.injectable.ts @@ -3,15 +3,18 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { routeSpecificComponentInjectionToken } from "../../routes/route-specific-component-injection-token"; -import ingressClassesRouteInjectable from "../../../common/front-end-routing/routes/cluster/network/ingress-class/ingress-classes-route.injectable"; +import { + routeSpecificComponentInjectionToken, +} from "../../routes/route-specific-component-injection-token"; +import ingressClassesesRouteInjectable + from "../../../common/front-end-routing/routes/cluster/network/ingress-class/ingress-classeses-route.injectable"; import { IngressClasses } from "./ingress-classes"; const ingressClassesRouteComponentInjectable = getInjectable({ id: "ingress-classes-route-component", instantiate: (di) => ({ - route: di.inject(ingressClassesRouteInjectable), + route: di.inject(ingressClassesesRouteInjectable), Component: IngressClasses, }), diff --git a/packages/core/src/renderer/components/network-ingresses/ingress-classes-sidebar-item.injectable.ts b/packages/core/src/renderer/components/network-ingresses/ingress-classes-sidebar-item.injectable.ts deleted file mode 100644 index 75c40dfb47a4..000000000000 --- a/packages/core/src/renderer/components/network-ingresses/ingress-classes-sidebar-item.injectable.ts +++ /dev/null @@ -1,30 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; -import { getInjectable } from "@ogre-tools/injectable"; -import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; -import networkSidebarItemInjectable from "../network/network-sidebar-items.injectable"; -import navigateToIngressClassesInjectable from "../../../common/front-end-routing/routes/cluster/network/ingress-class/navigate-to-ingress-classes.injectable"; -import ingressClassesRouteInjectable from "../../../common/front-end-routing/routes/cluster/network/ingress-class/ingress-classes-route.injectable"; - -const ingressClassesSidebarItemInjectable = getInjectable({ - id: "ingress-classes-sidebar-item", - instantiate: (di) => { - const ingressClassRoute = di.inject(ingressClassesRouteInjectable); - - return { - id: "ingressclasses", - parentId: di.inject(networkSidebarItemInjectable).id, - title: "Ingress Classes", - onClick: di.inject(navigateToIngressClassesInjectable), - isActive: di.inject(routeIsActiveInjectable, ingressClassRoute), - isVisible: ingressClassRoute.isEnabled, - orderNumber: 31, - }; - }, - injectionToken: sidebarItemInjectionToken, -}); - -export default ingressClassesSidebarItemInjectable; diff --git a/packages/core/src/renderer/components/network-ingresses/ingresses-sidebar-items.injectable.ts b/packages/core/src/renderer/components/network-ingresses/ingresses-sidebar-items.injectable.ts deleted file mode 100644 index c658ef69d8eb..000000000000 --- a/packages/core/src/renderer/components/network-ingresses/ingresses-sidebar-items.injectable.ts +++ /dev/null @@ -1,30 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; -import { getInjectable } from "@ogre-tools/injectable"; -import navigateToIngressesInjectable from "../../../common/front-end-routing/routes/cluster/network/ingress-class/navigate-to-ingress-classes.injectable"; -import ingressesRouteInjectable from "../../../common/front-end-routing/routes/cluster/network/ingresses/ingresses-route.injectable"; -import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; -import networkSidebarItemInjectable from "../network/network-sidebar-items.injectable"; - -const ingressesSidebarItemInjectable = getInjectable({ - id: "ingresses-sidebar-item", - instantiate: (di) => { - const ingressRoute = di.inject(ingressesRouteInjectable); - - return { - id: "ingresses", - parentId: di.inject(networkSidebarItemInjectable).id, - title: "Ingresses", - onClick: di.inject(navigateToIngressesInjectable), - isActive: di.inject(routeIsActiveInjectable, ingressRoute), - isVisible: ingressRoute.isEnabled, - orderNumber: 30, - }; - }, - injectionToken: sidebarItemInjectionToken, -}); - -export default ingressesSidebarItemInjectable; diff --git a/packages/core/src/renderer/components/network-ingresses/ingresses-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/network-ingresses/ingresses-sidebar-items.injectable.tsx new file mode 100644 index 000000000000..b8a90c39604f --- /dev/null +++ b/packages/core/src/renderer/components/network-ingresses/ingresses-sidebar-items.injectable.tsx @@ -0,0 +1,56 @@ +/** + * Copyright (c) OpenLens Authors. All rights reserved. + * Licensed under MIT License. See LICENSE in root directory for more information. + */ +import { getInjectable } from "@ogre-tools/injectable"; +import type { IComputedValue } from "mobx"; +import { computed } from "mobx"; +import ingressesRouteInjectable + from "../../../common/front-end-routing/routes/cluster/network/ingresses/ingresses-route.injectable"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; +import { networkSidebarItemId } from "../network/network-sidebar-items.injectable"; +import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; +import navigateToIngressesInjectable + from "../../../common/front-end-routing/routes/cluster/network/ingresses/navigate-to-ingresses.injectable"; +import ingressClassesesRouteInjectable + from "../../../common/front-end-routing/routes/cluster/network/ingress-class/ingress-classeses-route.injectable"; +import navigateToIngressClassesInjectable + from "../../../common/front-end-routing/routes/cluster/network/ingress-class/navigate-to-ingress-classes.injectable"; + +const ingressesSidebarItemsInjectable = getInjectable({ + id: "ingresses-sidebar-items", + + instantiate: (di): IComputedValue => { + const ingressRoute = di.inject(ingressesRouteInjectable); + const ingressClassRoute = di.inject(ingressClassesesRouteInjectable); + + return computed(() => [ + { + id: "ingresses", + parentId: networkSidebarItemId, + title: "Ingresses", + onClick: di.inject(navigateToIngressesInjectable), + isActive: di.inject(routeIsActiveInjectable, ingressRoute), + isVisible: ingressRoute.isEnabled, + orderNumber: 30, + }, + { + id: "ingressclasses", + parentId: networkSidebarItemId, + title: "Ingress Classes", + onClick: di.inject(navigateToIngressClassesInjectable), + isActive: di.inject(routeIsActiveInjectable, ingressClassRoute), + isVisible: ingressClassRoute.isEnabled, + orderNumber: 31, + }, + ]); + }, + + injectionToken: sidebarItemsInjectionToken, +}); + +export default ingressesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/network-policies/network-policies-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/network-policies/network-policies-sidebar-items.injectable.tsx index 365f32b6182f..3326594ba4e5 100644 --- a/packages/core/src/renderer/components/network-policies/network-policies-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/network-policies/network-policies-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import networkPoliciesRouteInjectable from "../../../common/front-end-routing/routes/cluster/network/network-policies/network-policies-route.injectable"; -import networkSidebarItemInjectable from "../network/network-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { networkSidebarItemId } from "../network/network-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToNetworkPoliciesInjectable from "../../../common/front-end-routing/routes/cluster/network/network-policies/navigate-to-network-policies.injectable"; -const networkPoliciesSidebarItemInjectable = getInjectable({ - id: "network-policies-sidebar-item", +const networkPoliciesSidebarItemsInjectable = getInjectable({ + id: "network-policies-sidebar-items", instantiate: (di) => { const route = di.inject(networkPoliciesRouteInjectable); + const navigateToNetworkPolicies = di.inject(navigateToNetworkPoliciesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "network-policies", - parentId: di.inject(networkSidebarItemInjectable).id, - title: "Network Policies", - onClick: di.inject(navigateToNetworkPoliciesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 40, - }; + return computed(() => [ + { + id: "network-policies", + parentId: networkSidebarItemId, + title: "Network Policies", + onClick: navigateToNetworkPolicies, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 40, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default networkPoliciesSidebarItemInjectable; +export default networkPoliciesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/network-port-forwards/port-forwards-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/network-port-forwards/port-forwards-sidebar-items.injectable.tsx index 6fe529c0c31c..eafcf07ce25d 100644 --- a/packages/core/src/renderer/components/network-port-forwards/port-forwards-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/network-port-forwards/port-forwards-sidebar-items.injectable.tsx @@ -3,31 +3,37 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import portForwardsRouteInjectable from "../../../common/front-end-routing/routes/cluster/network/port-forwards/port-forwards-route.injectable"; -import networkSidebarItemInjectable from "../network/network-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { networkSidebarItemId } from "../network/network-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToPortForwardsInjectable from "../../../common/front-end-routing/routes/cluster/network/port-forwards/navigate-to-port-forwards.injectable"; -const portForwardsSidebarItemInjectable = getInjectable({ - id: "port-forwards-sidebar-item", +const portForwardsSidebarItemsInjectable = getInjectable({ + id: "port-forwards-sidebar-items", instantiate: (di) => { const route = di.inject(portForwardsRouteInjectable); + const navigateToPortForwards = di.inject(navigateToPortForwardsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "port-forwards", - parentId: di.inject(networkSidebarItemInjectable).id, - title: "Port Forwarding", - onClick: di.inject(navigateToPortForwardsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 50, - }; + return computed(() => [ + { + id: "port-forwards", + parentId: networkSidebarItemId, + + title: "Port Forwarding", + onClick: navigateToPortForwards, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 50, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default portForwardsSidebarItemInjectable; +export default portForwardsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/network-services/services-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/network-services/services-sidebar-items.injectable.tsx index 98e54280d22f..f25bcaddab17 100644 --- a/packages/core/src/renderer/components/network-services/services-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/network-services/services-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import servicesRouteInjectable from "../../../common/front-end-routing/routes/cluster/network/services/services-route.injectable"; -import networkSidebarItemInjectable from "../network/network-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { networkSidebarItemId } from "../network/network-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToServicesInjectable from "../../../common/front-end-routing/routes/cluster/network/services/navigate-to-services.injectable"; -const servicesSidebarItemInjectable = getInjectable({ - id: "services-sidebar-item", +const servicesSidebarItemsInjectable = getInjectable({ + id: "services-sidebar-items", instantiate: (di) => { const route = di.inject(servicesRouteInjectable); + const navigateToServices = di.inject(navigateToServicesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "services", - parentId: di.inject(networkSidebarItemInjectable).id, - title: "Services", - onClick: di.inject(navigateToServicesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 10, - }; + return computed(() => [ + { + id: "services", + parentId: networkSidebarItemId, + title: "Services", + onClick: navigateToServices, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 10, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default servicesSidebarItemInjectable; +export default servicesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/network/network-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/network/network-sidebar-items.injectable.tsx index bcc8fa6560bc..bb6f107d9de1 100644 --- a/packages/core/src/renderer/components/network/network-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/network/network-sidebar-items.injectable.tsx @@ -3,24 +3,34 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { computed } from "mobx"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; import { Icon } from "../icon"; import React from "react"; import { noop } from "lodash/fp"; -const networkSidebarItemInjectable = getInjectable({ - id: "network-sidebar-item", +export const networkSidebarItemId = "network"; - instantiate: () => ({ - id: "network", - parentId: null, - getIcon: () => , - title: "Network", - onClick: noop, - orderNumber: 50, - }), +const networkSidebarItemsInjectable = getInjectable({ + id: "network-sidebar-items", - injectionToken: sidebarItemInjectionToken, + instantiate: () => + computed((): SidebarItemRegistration[] => [ + { + id: networkSidebarItemId, + parentId: null, + getIcon: () => , + title: "Network", + onClick: noop, + orderNumber: 50, + }, + ]), + + injectionToken: sidebarItemsInjectionToken, }); -export default networkSidebarItemInjectable; +export default networkSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/nodes/nodes-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/nodes/nodes-sidebar-items.injectable.tsx index 0718711634f7..580ebd3c90fb 100644 --- a/packages/core/src/renderer/components/nodes/nodes-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/nodes/nodes-sidebar-items.injectable.tsx @@ -3,10 +3,12 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; - +import { computed } from "mobx"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; import { - sidebarItemInjectionToken, -} from "@k8slens/cluster-sidebar"; + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; import { Icon } from "../icon"; import React from "react"; @@ -14,25 +16,29 @@ import nodesRouteInjectable from "../../../common/front-end-routing/routes/clust import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToNodesInjectable from "../../../common/front-end-routing/routes/cluster/nodes/navigate-to-nodes.injectable"; -const nodesSidebarItemInjectable = getInjectable({ - id: "nodes-sidebar-item", +const nodesSidebarItemsInjectable = getInjectable({ + id: "nodes-sidebar-items", instantiate: (di) => { const route = di.inject(nodesRouteInjectable); + const navigateToNodes = di.inject(navigateToNodesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "nodes", - parentId: null, - getIcon: () => , - title: "Nodes", - onClick: di.inject(navigateToNodesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 20, - }; + return computed((): SidebarItemRegistration[] => [ + { + id: "nodes", + parentId: null, + getIcon: () => , + title: "Nodes", + onClick: navigateToNodes, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 20, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default nodesSidebarItemInjectable; +export default nodesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/pod-security-policies/pod-security-policies-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/pod-security-policies/pod-security-policies-sidebar-items.injectable.tsx index 0c5583fc26a2..c743a7650ce7 100644 --- a/packages/core/src/renderer/components/pod-security-policies/pod-security-policies-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/pod-security-policies/pod-security-policies-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import podSecurityPoliciesRouteInjectable from "../../../common/front-end-routing/routes/cluster/user-management/pod-security-policies/pod-security-policies-route.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { userManagementSidebarItemId } from "../user-management/user-management-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToPodSecurityPoliciesInjectable from "../../../common/front-end-routing/routes/cluster/user-management/pod-security-policies/navigate-to-pod-security-policies.injectable"; -import userManagementSidebarItemInjectable from "../user-management/user-management-sidebar-items.injectable"; -const podSecurityPoliciesSidebarItemInjectable = getInjectable({ - id: "pod-security-policies-sidebar-item", +const podSecurityPoliciesSidebarItemsInjectable = getInjectable({ + id: "pod-security-policies-sidebar-items", instantiate: (di) => { const route = di.inject(podSecurityPoliciesRouteInjectable); + const navigateToPodSecurityPolicies = di.inject(navigateToPodSecurityPoliciesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "pod-security-policies", - parentId: di.inject(userManagementSidebarItemInjectable).id, - title: "Pod Security Policies", - onClick: di.inject(navigateToPodSecurityPoliciesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 60, - }; + return computed(() => [ + { + id: "pod-security-policies", + parentId: userManagementSidebarItemId, + title: "Pod Security Policies", + onClick: navigateToPodSecurityPolicies, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 60, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default podSecurityPoliciesSidebarItemInjectable; +export default podSecurityPoliciesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/storage-classes/storage-classes-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/storage-classes/storage-classes-sidebar-items.injectable.tsx index b553652e51c1..256c913b7311 100644 --- a/packages/core/src/renderer/components/storage-classes/storage-classes-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/storage-classes/storage-classes-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import storageClassesRouteInjectable from "../../../common/front-end-routing/routes/cluster/storage/storage-classes/storage-classes-route.injectable"; -import storageSidebarItemInjectable from "../storage/storage-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { storageSidebarItemId } from "../storage/storage-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToStorageClassesInjectable from "../../../common/front-end-routing/routes/cluster/storage/storage-classes/navigate-to-storage-classes.injectable"; -const storageClassesSidebarItemInjectable = getInjectable({ - id: "storage-classes-sidebar-item", +const storageClassesSidebarItemsInjectable = getInjectable({ + id: "storage-classes-sidebar-items", instantiate: (di) => { const route = di.inject(storageClassesRouteInjectable); + const navigateToStorageClasses = di.inject(navigateToStorageClassesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "storage-classes", - parentId: di.inject(storageSidebarItemInjectable).id, - title: "Storage Classes", - onClick: di.inject(navigateToStorageClassesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 30, - }; + return computed(() => [ + { + id: "storage-classes", + parentId: storageSidebarItemId, + title: "Storage Classes", + onClick: navigateToStorageClasses, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 30, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default storageClassesSidebarItemInjectable; +export default storageClassesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/storage-volume-claims/persistent-volume-claims-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/storage-volume-claims/persistent-volume-claims-sidebar-items.injectable.tsx index d4562003a9ea..9270ade0ccd7 100644 --- a/packages/core/src/renderer/components/storage-volume-claims/persistent-volume-claims-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/storage-volume-claims/persistent-volume-claims-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import persistentVolumeClaimsRouteInjectable from "../../../common/front-end-routing/routes/cluster/storage/persistent-volume-claims/persistent-volume-claims-route.injectable"; -import storageSidebarItemInjectable from "../storage/storage-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { storageSidebarItemId } from "../storage/storage-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToPersistentVolumeClaimsInjectable from "../../../common/front-end-routing/routes/cluster/storage/persistent-volume-claims/navigate-to-persistent-volume-claims.injectable"; -const persistentVolumeClaimsSidebarItemInjectable = getInjectable({ - id: "persistent-volume-claims-sidebar-item", +const persistentVolumeClaimsSidebarItemsInjectable = getInjectable({ + id: "persistent-volume-claims-sidebar-items", instantiate: (di) => { const route = di.inject(persistentVolumeClaimsRouteInjectable); + const navigateToPersistentVolumeClaims = di.inject(navigateToPersistentVolumeClaimsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "persistent-volume-claims", - parentId: di.inject(storageSidebarItemInjectable).id, - title: "Persistent Volume Claims", - onClick: di.inject(navigateToPersistentVolumeClaimsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 10, - }; + return computed(() => [ + { + id: "persistent-volume-claims", + parentId: storageSidebarItemId, + title: "Persistent Volume Claims", + onClick: navigateToPersistentVolumeClaims, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 10, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default persistentVolumeClaimsSidebarItemInjectable; +export default persistentVolumeClaimsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/storage-volumes/persistent-volumes-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/storage-volumes/persistent-volumes-sidebar-items.injectable.tsx index ad4058ebdb93..0e6da86ebaaf 100644 --- a/packages/core/src/renderer/components/storage-volumes/persistent-volumes-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/storage-volumes/persistent-volumes-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import persistentVolumesRouteInjectable from "../../../common/front-end-routing/routes/cluster/storage/persistent-volumes/persistent-volumes-route.injectable"; -import storageSidebarItemInjectable from "../storage/storage-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { storageSidebarItemId } from "../storage/storage-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToPersistentVolumesInjectable from "../../../common/front-end-routing/routes/cluster/storage/persistent-volumes/navigate-to-persistent-volumes.injectable"; -const persistentVolumesSidebarItemInjectable = getInjectable({ - id: "persistent-volumes-sidebar-item", +const persistentVolumesSidebarItemsInjectable = getInjectable({ + id: "persistent-volumes-sidebar-items", instantiate: (di) => { const route = di.inject(persistentVolumesRouteInjectable); + const navigateToPersistentVolumes = di.inject(navigateToPersistentVolumesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "persistent-volumes", - parentId: di.inject(storageSidebarItemInjectable).id, - title: "Persistent Volumes", - onClick: di.inject(navigateToPersistentVolumesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 20, - }; + return computed(() => [ + { + id: "persistent-volumes", + parentId: storageSidebarItemId, + title: "Persistent Volumes", + onClick: navigateToPersistentVolumes, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 20, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default persistentVolumesSidebarItemInjectable; +export default persistentVolumesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/storage/storage-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/storage/storage-sidebar-items.injectable.tsx index fbbd6f0ce1aa..fe243ba729e6 100644 --- a/packages/core/src/renderer/components/storage/storage-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/storage/storage-sidebar-items.injectable.tsx @@ -3,24 +3,34 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { computed } from "mobx"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; import { Icon } from "../icon"; import React from "react"; import { noop } from "lodash/fp"; -const storageSidebarItemInjectable = getInjectable({ - id: "storage-sidebar-item", +export const storageSidebarItemId = "storage"; - instantiate: () => ({ - id: "storage", - parentId: null, - getIcon: () => , - title: "Storage", - onClick: noop, - orderNumber: 60, - }), +const storageSidebarItemsInjectable = getInjectable({ + id: "storage-sidebar-items", - injectionToken: sidebarItemInjectionToken, + instantiate: () => + computed((): SidebarItemRegistration[] => [ + { + id: storageSidebarItemId, + parentId: null, + getIcon: () => , + title: "Storage", + onClick: noop, + orderNumber: 60, + }, + ]), + + injectionToken: sidebarItemsInjectionToken, }); -export default storageSidebarItemInjectable; +export default storageSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/user-management/cluster-role-bindings/cluster-role-bindings-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/user-management/cluster-role-bindings/cluster-role-bindings-sidebar-items.injectable.tsx index 049a9e4d1dd1..bb4ed2a928f4 100644 --- a/packages/core/src/renderer/components/user-management/cluster-role-bindings/cluster-role-bindings-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/user-management/cluster-role-bindings/cluster-role-bindings-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import clusterRoleBindingsRouteInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/cluster-role-bindings/cluster-role-bindings-route.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { userManagementSidebarItemId } from "../user-management-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../../routes/route-is-active.injectable"; import navigateToClusterRoleBindingsInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/cluster-role-bindings/navigate-to-cluster-role-bindings.injectable"; -import userManagementSidebarItemInjectable from "../user-management-sidebar-items.injectable"; -const clusterRoleBindingsSidebarItemInjectable = getInjectable({ - id: "cluster-role-bindings-sidebar-item", +const clusterRoleBindingsSidebarItemsInjectable = getInjectable({ + id: "cluster-role-bindings-sidebar-items", instantiate: (di) => { const route = di.inject(clusterRoleBindingsRouteInjectable); + const navigateToClusterRoleBindings = di.inject(navigateToClusterRoleBindingsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "cluster-role-bindings", - parentId: di.inject(userManagementSidebarItemInjectable).id, - title: "Cluster Role Bindings", - onClick: di.inject(navigateToClusterRoleBindingsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 40, - }; + return computed(() => [ + { + id: "cluster-role-bindings", + parentId: userManagementSidebarItemId, + title: "Cluster Role Bindings", + onClick: navigateToClusterRoleBindings, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 40, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default clusterRoleBindingsSidebarItemInjectable; +export default clusterRoleBindingsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/user-management/cluster-roles/cluster-roles-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/user-management/cluster-roles/cluster-roles-sidebar-items.injectable.tsx index a9de072a7a65..54ae68ef4485 100644 --- a/packages/core/src/renderer/components/user-management/cluster-roles/cluster-roles-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/user-management/cluster-roles/cluster-roles-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import clusterRolesRouteInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/cluster-roles/cluster-roles-route.injectable"; -import userManagementSidebarItemInjectable from "../user-management-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { userManagementSidebarItemId } from "../user-management-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../../routes/route-is-active.injectable"; import navigateToClusterRolesInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/cluster-roles/navigate-to-cluster-roles.injectable"; -const clusterRolesSidebarItemInjectable = getInjectable({ - id: "cluster-roles-sidebar-item", +const clusterRolesSidebarItemsInjectable = getInjectable({ + id: "cluster-roles-sidebar-items", instantiate: (di) => { const route = di.inject(clusterRolesRouteInjectable); + const navigateToClusterRoles = di.inject(navigateToClusterRolesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "cluster-roles", - parentId: di.inject(userManagementSidebarItemInjectable).id, - title: "Cluster Roles", - onClick: di.inject(navigateToClusterRolesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 20, - }; + return computed(() => [ + { + id: "cluster-roles", + parentId: userManagementSidebarItemId, + title: "Cluster Roles", + onClick: navigateToClusterRoles, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 20, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default clusterRolesSidebarItemInjectable; +export default clusterRolesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/user-management/role-bindings/role-bindings-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/user-management/role-bindings/role-bindings-sidebar-items.injectable.tsx index 18ce36249d98..f2eb240839e8 100644 --- a/packages/core/src/renderer/components/user-management/role-bindings/role-bindings-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/user-management/role-bindings/role-bindings-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import roleBindingsRouteInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/role-bindings/role-bindings-route.injectable"; -import userManagementSidebarItemInjectable from "../user-management-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { userManagementSidebarItemId } from "../user-management-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../../routes/route-is-active.injectable"; import navigateToRoleBindingsInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/role-bindings/navigate-to-role-bindings.injectable"; -const roleBindingsSidebarItemInjectable = getInjectable({ - id: "role-bindings-sidebar-item", +const roleBindingsSidebarItemsInjectable = getInjectable({ + id: "role-bindings-sidebar-items", instantiate: (di) => { const route = di.inject(roleBindingsRouteInjectable); + const navigateToRoleBindings = di.inject(navigateToRoleBindingsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "role-bindings", - parentId: di.inject(userManagementSidebarItemInjectable).id, - title: "Role Bindings", - onClick: di.inject(navigateToRoleBindingsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 50, - }; + return computed(() => [ + { + id: "role-bindings", + parentId: userManagementSidebarItemId, + title: "Role Bindings", + onClick: navigateToRoleBindings, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 50, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default roleBindingsSidebarItemInjectable; +export default roleBindingsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/user-management/roles/roles-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/user-management/roles/roles-sidebar-items.injectable.tsx index 753fd43f65e5..97ee9715a2d0 100644 --- a/packages/core/src/renderer/components/user-management/roles/roles-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/user-management/roles/roles-sidebar-items.injectable.tsx @@ -3,30 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; + import rolesRouteInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/roles/roles-route.injectable"; -import userManagementSidebarItemInjectable from "../user-management-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { userManagementSidebarItemId } from "../user-management-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../../routes/route-is-active.injectable"; import navigateToRolesInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/roles/navigate-to-roles.injectable"; -const rolesSidebarItemInjectable = getInjectable({ - id: "roles-sidebar-item", +const rolesSidebarItemsInjectable = getInjectable({ + id: "roles-sidebar-items", instantiate: (di) => { const route = di.inject(rolesRouteInjectable); + const navigateToRoles = di.inject(navigateToRolesInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "roles", - parentId: di.inject(userManagementSidebarItemInjectable).id, - title: "Roles", - onClick: di.inject(navigateToRolesInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 30, - }; + return computed(() => [ + { + id: "roles", + parentId: userManagementSidebarItemId, + title: "Roles", + onClick: navigateToRoles, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 30, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default rolesSidebarItemInjectable; +export default rolesSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/user-management/service-accounts/service-accounts-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/user-management/service-accounts/service-accounts-sidebar-items.injectable.tsx index 2315cc48f297..d01b8d6480f9 100644 --- a/packages/core/src/renderer/components/user-management/service-accounts/service-accounts-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/user-management/service-accounts/service-accounts-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import serviceAccountsRouteInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/service-accounts/service-accounts-route.injectable"; -import userManagementSidebarItemInjectable from "../user-management-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { userManagementSidebarItemId } from "../user-management-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../../routes/route-is-active.injectable"; import navigateToServiceAccountsInjectable from "../../../../common/front-end-routing/routes/cluster/user-management/service-accounts/navigate-to-service-accounts.injectable"; -const serviceAccountsSidebarItemInjectable = getInjectable({ - id: "service-accounts-sidebar-item", +const serviceAccountsSidebarItemsInjectable = getInjectable({ + id: "service-accounts-sidebar-items", instantiate: (di) => { const route = di.inject(serviceAccountsRouteInjectable); + const navigateToServiceAccounts = di.inject(navigateToServiceAccountsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "service-accounts", - parentId: di.inject(userManagementSidebarItemInjectable).id, - title: "Service Accounts", - onClick: di.inject(navigateToServiceAccountsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 10, - }; + return computed(() => [ + { + id: "service-accounts", + parentId: userManagementSidebarItemId, + title: "Service Accounts", + onClick: navigateToServiceAccounts, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 10, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default serviceAccountsSidebarItemInjectable; +export default serviceAccountsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/user-management/user-management-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/user-management/user-management-sidebar-items.injectable.tsx index 7bcdeb21e26d..5a14bd083bd3 100644 --- a/packages/core/src/renderer/components/user-management/user-management-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/user-management/user-management-sidebar-items.injectable.tsx @@ -3,24 +3,34 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { computed } from "mobx"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; import { Icon } from "../icon"; import React from "react"; import { noop } from "lodash/fp"; -const userManagementSidebarItemInjectable = getInjectable({ - id: "user-management-sidebar-item", +export const userManagementSidebarItemId = "user-management"; - instantiate: () => ({ - id: "user-management", - parentId: null, - getIcon: () => , - title: "Access Control", - onClick: noop, - orderNumber: 100, - }), +const userManagementSidebarItemsInjectable = getInjectable({ + id: "user-management-sidebar-items", - injectionToken: sidebarItemInjectionToken, + instantiate: () => + computed((): SidebarItemRegistration[] => [ + { + id: userManagementSidebarItemId, + parentId: null, + getIcon: () => , + title: "Access Control", + onClick: noop, + orderNumber: 100, + }, + ]), + + injectionToken: sidebarItemsInjectionToken, }); -export default userManagementSidebarItemInjectable; +export default userManagementSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads-cronjobs/cron-jobs-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/workloads-cronjobs/cron-jobs-sidebar-items.injectable.tsx index d3262e9af515..290fe5a4d692 100644 --- a/packages/core/src/renderer/components/workloads-cronjobs/cron-jobs-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/workloads-cronjobs/cron-jobs-sidebar-items.injectable.tsx @@ -3,30 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; + import cronJobsRouteInjectable from "../../../common/front-end-routing/routes/cluster/workloads/cron-jobs/cron-jobs-route.injectable"; -import workloadsSidebarItemInjectable from "../workloads/workloads-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { workloadsSidebarItemId } from "../workloads/workloads-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToCronJobsInjectable from "../../../common/front-end-routing/routes/cluster/workloads/cron-jobs/navigate-to-cron-jobs.injectable"; -const cronJobsSidebarItemInjectable = getInjectable({ - id: "cron-jobs-sidebar-item", +const cronJobsSidebarItemsInjectable = getInjectable({ + id: "cron-jobs-sidebar-items", instantiate: (di) => { const route = di.inject(cronJobsRouteInjectable); + const navigateToCronJobs = di.inject(navigateToCronJobsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "cron-jobs", - parentId: di.inject(workloadsSidebarItemInjectable).id, - title: "CronJobs", - onClick: di.inject(navigateToCronJobsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 80, - }; + return computed(() => [ + { + id: "cron-jobs", + parentId: workloadsSidebarItemId, + title: "CronJobs", + onClick: navigateToCronJobs, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 80, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default cronJobsSidebarItemInjectable; +export default cronJobsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads-daemonsets/daemonsets-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/workloads-daemonsets/daemonsets-sidebar-items.injectable.tsx index 9ef6a9a434d4..c5442e3642c7 100644 --- a/packages/core/src/renderer/components/workloads-daemonsets/daemonsets-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/workloads-daemonsets/daemonsets-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import daemonsetsRouteInjectable from "../../../common/front-end-routing/routes/cluster/workloads/daemonsets/daemonsets-route.injectable"; -import workloadsSidebarItemInjectable from "../workloads/workloads-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { workloadsSidebarItemId } from "../workloads/workloads-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToDaemonsetsInjectable from "../../../common/front-end-routing/routes/cluster/workloads/daemonsets/navigate-to-daemonsets.injectable"; -const daemonsetsSidebarItemInjectable = getInjectable({ - id: "daemonsets-sidebar-item", +const daemonsetsSidebarItemsInjectable = getInjectable({ + id: "daemonsets-sidebar-items", instantiate: (di) => { const route = di.inject(daemonsetsRouteInjectable); + const navigateToDaemonsets = di.inject(navigateToDaemonsetsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "daemon-sets", - parentId: di.inject(workloadsSidebarItemInjectable).id, - title: "DaemonSets", - onClick: di.inject(navigateToDaemonsetsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 40, - }; + return computed(() => [ + { + id: "daemon-sets", + parentId: workloadsSidebarItemId, + title: "DaemonSets", + onClick: () => navigateToDaemonsets(), + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 40, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default daemonsetsSidebarItemInjectable; +export default daemonsetsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads-deployments/deployments-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/workloads-deployments/deployments-sidebar-items.injectable.tsx index 5d354bbe25fd..85aed486dcf7 100644 --- a/packages/core/src/renderer/components/workloads-deployments/deployments-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/workloads-deployments/deployments-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import deploymentsRouteInjectable from "../../../common/front-end-routing/routes/cluster/workloads/deployments/deployments-route.injectable"; -import workloadsSidebarItemInjectable from "../workloads/workloads-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { workloadsSidebarItemId } from "../workloads/workloads-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToDeploymentsInjectable from "../../../common/front-end-routing/routes/cluster/workloads/deployments/navigate-to-deployments.injectable"; -const deploymentsSidebarItemInjectable = getInjectable({ - id: "deployments-sidebar-item", +const deploymentsSidebarItemsInjectable = getInjectable({ + id: "deployments-sidebar-items", instantiate: (di) => { const route = di.inject(deploymentsRouteInjectable); + const navigateToDeployments = di.inject(navigateToDeploymentsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "deployments", - parentId: di.inject(workloadsSidebarItemInjectable).id, - title: "Deployments", - onClick: di.inject(navigateToDeploymentsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 30, - }; + return computed(() => [ + { + id: "deployments", + parentId: workloadsSidebarItemId, + title: "Deployments", + onClick: navigateToDeployments, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 30, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default deploymentsSidebarItemInjectable; +export default deploymentsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads-jobs/jobs-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/workloads-jobs/jobs-sidebar-items.injectable.tsx index 57e0edc75a69..acc788dd2cce 100644 --- a/packages/core/src/renderer/components/workloads-jobs/jobs-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/workloads-jobs/jobs-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import jobsRouteInjectable from "../../../common/front-end-routing/routes/cluster/workloads/jobs/jobs-route.injectable"; -import workloadsSidebarItemInjectable from "../workloads/workloads-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { workloadsSidebarItemId } from "../workloads/workloads-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToJobsInjectable from "../../../common/front-end-routing/routes/cluster/workloads/jobs/navigate-to-jobs.injectable"; -const jobsSidebarItemInjectable = getInjectable({ - id: "jobs-sidebar-item", +const jobsSidebarItemsInjectable = getInjectable({ + id: "jobs-sidebar-items", instantiate: (di) => { const route = di.inject(jobsRouteInjectable); + const navigateToJobs = di.inject(navigateToJobsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "jobs", - parentId: di.inject(workloadsSidebarItemInjectable).id, - title: "Jobs", - onClick: di.inject(navigateToJobsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 70, - }; + return computed(() => [ + { + id: "jobs", + parentId: workloadsSidebarItemId, + title: "Jobs", + onClick: navigateToJobs, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 70, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default jobsSidebarItemInjectable; +export default jobsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads-overview/workloads-overview-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/workloads-overview/workloads-overview-sidebar-items.injectable.tsx index fda9f9eb02ee..98efb01c7bee 100644 --- a/packages/core/src/renderer/components/workloads-overview/workloads-overview-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/workloads-overview/workloads-overview-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import workloadsOverviewRouteInjectable from "../../../common/front-end-routing/routes/cluster/workloads/overview/workloads-overview-route.injectable"; -import workloadsSidebarItemInjectable from "../workloads/workloads-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { workloadsSidebarItemId } from "../workloads/workloads-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToWorkloadsOverviewInjectable from "../../../common/front-end-routing/routes/cluster/workloads/overview/navigate-to-workloads-overview.injectable"; -const workloadsOverviewSidebarItemInjectable = getInjectable({ - id: "workloads-overview-sidebar-item", +const workloadsOverviewSidebarItemsInjectable = getInjectable({ + id: "workloads-overview-sidebar-items", instantiate: (di) => { const route = di.inject(workloadsOverviewRouteInjectable); + const navigateToWorkloadsOverview = di.inject(navigateToWorkloadsOverviewInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "overview", - parentId: di.inject(workloadsSidebarItemInjectable).id, - title: "Overview", - onClick: di.inject(navigateToWorkloadsOverviewInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 10, - }; + return computed(() => [ + { + id: "overview", + parentId: workloadsSidebarItemId, + title: "Overview", + onClick: navigateToWorkloadsOverview, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 10, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default workloadsOverviewSidebarItemInjectable; +export default workloadsOverviewSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads-overview/workloads/workloads.injectable.ts b/packages/core/src/renderer/components/workloads-overview/workloads/workloads.injectable.ts index cdd72d9c1b0a..a90e3d63d910 100644 --- a/packages/core/src/renderer/components/workloads-overview/workloads/workloads.injectable.ts +++ b/packages/core/src/renderer/components/workloads-overview/workloads/workloads.injectable.ts @@ -5,7 +5,7 @@ import { getInjectable } from "@ogre-tools/injectable"; import { computed } from "mobx"; import { shouldShowResourceInjectionToken } from "../../../../features/cluster/showing-kube-resources/common/allowed-resources-injection-token"; -import { byOrderNumber } from "@k8slens/utilities"; +import { byOrderNumber } from "../../../../common/utils/composable-responsibilities/orderable/orderable"; import { workloadInjectionToken } from "./workload-injection-token"; const workloadsInjectable = getInjectable({ diff --git a/packages/core/src/renderer/components/workloads-pods/pods-sidebar-items.injectable.ts b/packages/core/src/renderer/components/workloads-pods/pods-sidebar-items.injectable.ts index d509f5ae6723..74d8d18538b9 100644 --- a/packages/core/src/renderer/components/workloads-pods/pods-sidebar-items.injectable.ts +++ b/packages/core/src/renderer/components/workloads-pods/pods-sidebar-items.injectable.ts @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import podsRouteInjectable from "../../../common/front-end-routing/routes/cluster/workloads/pods/pods-route.injectable"; -import workloadsSidebarItemInjectable from "../workloads/workloads-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { workloadsSidebarItemId } from "../workloads/workloads-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToPodsInjectable from "../../../common/front-end-routing/routes/cluster/workloads/pods/navigate-to-pods.injectable"; -const podsSidebarItemInjectable = getInjectable({ - id: "pods-sidebar-item", +const podsSidebarItemsInjectable = getInjectable({ + id: "pods-sidebar-items", instantiate: (di) => { const route = di.inject(podsRouteInjectable); + const navigateToPods = di.inject(navigateToPodsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "pods", - parentId: di.inject(workloadsSidebarItemInjectable).id, - title: "Pods", - onClick: di.inject(navigateToPodsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 20, - }; + return computed(() => [ + { + id: "pods", + parentId: workloadsSidebarItemId, + title: "Pods", + onClick: navigateToPods, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 20, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default podsSidebarItemInjectable; +export default podsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads-replicasets/replicasets-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/workloads-replicasets/replicasets-sidebar-items.injectable.tsx index b94b09b2ce21..a6745b3759c7 100644 --- a/packages/core/src/renderer/components/workloads-replicasets/replicasets-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/workloads-replicasets/replicasets-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import replicasetsRouteInjectable from "../../../common/front-end-routing/routes/cluster/workloads/replicasets/replicasets-route.injectable"; -import workloadsSidebarItemInjectable from "../workloads/workloads-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { workloadsSidebarItemId } from "../workloads/workloads-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToReplicasetsInjectable from "../../../common/front-end-routing/routes/cluster/workloads/replicasets/navigate-to-replicasets.injectable"; -const replicasetsSidebarItemInjectable = getInjectable({ - id: "replicasets-sidebar-item", +const replicasetsSidebarItemsInjectable = getInjectable({ + id: "replicasets-sidebar-items", instantiate: (di) => { const route = di.inject(replicasetsRouteInjectable); + const navigateToReplicasets = di.inject(navigateToReplicasetsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "replica-sets", - parentId: di.inject(workloadsSidebarItemInjectable).id, - title: "ReplicaSets", - onClick: di.inject(navigateToReplicasetsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 60, - }; + return computed(() => [ + { + id: "replica-sets", + parentId: workloadsSidebarItemId, + title: "ReplicaSets", + onClick: navigateToReplicasets, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 60, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default replicasetsSidebarItemInjectable; +export default replicasetsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads-replication-controllers/replication-controller-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/workloads-replication-controllers/replication-controller-sidebar-items.injectable.tsx index 6e7005541c3a..9108faaabc23 100644 --- a/packages/core/src/renderer/components/workloads-replication-controllers/replication-controller-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/workloads-replication-controllers/replication-controller-sidebar-items.injectable.tsx @@ -3,30 +3,35 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import workloadsSidebarItemInjectable from "../workloads/workloads-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { computed } from "mobx"; +import { workloadsSidebarItemId } from "../workloads/workloads-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import replicationControllersRouteInjectable from "../../../common/front-end-routing/routes/cluster/workloads/replication-controllers/route.injectable"; import navigateToReplicationControllersInjectable from "../../../common/front-end-routing/routes/cluster/workloads/replication-controllers/navigate-to.injectable"; -const replicationControllerSidebarItemInjectable = getInjectable({ - id: "replication-controller-sidebar-item", +const replicationControllerSidebarItemsInjectable = getInjectable({ + id: "replication-controller-sidebar-items", instantiate: (di) => { const route = di.inject(replicationControllersRouteInjectable); + const navigateToPage = di.inject(navigateToReplicationControllersInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "replication-controllers", - parentId: di.inject(workloadsSidebarItemInjectable).id, - title: "Replication Controllers", - onClick: di.inject(navigateToReplicationControllersInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 61, - }; + return computed(() => [ + { + id: "replication-controllers", + parentId: workloadsSidebarItemId, + title: "Replication Controllers", + onClick: navigateToPage, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 61, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default replicationControllerSidebarItemInjectable; +export default replicationControllerSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads-statefulsets/statefulsets-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/workloads-statefulsets/statefulsets-sidebar-items.injectable.tsx index 6797efe17c3d..2d73309c9d4d 100644 --- a/packages/core/src/renderer/components/workloads-statefulsets/statefulsets-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/workloads-statefulsets/statefulsets-sidebar-items.injectable.tsx @@ -3,31 +3,36 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; +import { computed } from "mobx"; import statefulsetsRouteInjectable from "../../../common/front-end-routing/routes/cluster/workloads/statefulsets/statefulsets-route.injectable"; -import workloadsSidebarItemInjectable from "../workloads/workloads-sidebar-items.injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { workloadsSidebarItemId } from "../workloads/workloads-sidebar-items.injectable"; +import { sidebarItemsInjectionToken } from "../layout/sidebar-items.injectable"; import routeIsActiveInjectable from "../../routes/route-is-active.injectable"; import navigateToStatefulsetsInjectable from "../../../common/front-end-routing/routes/cluster/workloads/statefulsets/navigate-to-statefulsets.injectable"; -const statefulsetsSidebarItemInjectable = getInjectable({ - id: "statefulsets-sidebar-item", +const statefulsetsSidebarItemsInjectable = getInjectable({ + id: "statefulsets-sidebar-items", instantiate: (di) => { const route = di.inject(statefulsetsRouteInjectable); + const navigateToStatefulsets = di.inject(navigateToStatefulsetsInjectable); + const routeIsActive = di.inject(routeIsActiveInjectable, route); - return { - id: "stateful-sets", - parentId: di.inject(workloadsSidebarItemInjectable).id, - title: "StatefulSets", - onClick: di.inject(navigateToStatefulsetsInjectable), - isActive: di.inject(routeIsActiveInjectable, route), - isVisible: route.isEnabled, - orderNumber: 50, - }; + return computed(() => [ + { + id: "stateful-sets", + parentId: workloadsSidebarItemId, + title: "StatefulSets", + onClick: navigateToStatefulsets, + isActive: routeIsActive, + isVisible: route.isEnabled, + orderNumber: 50, + }, + ]); }, - injectionToken: sidebarItemInjectionToken, + injectionToken: sidebarItemsInjectionToken, }); -export default statefulsetsSidebarItemInjectable; +export default statefulsetsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/components/workloads/workloads-sidebar-items.injectable.tsx b/packages/core/src/renderer/components/workloads/workloads-sidebar-items.injectable.tsx index 7783d756286a..933dced46ab3 100644 --- a/packages/core/src/renderer/components/workloads/workloads-sidebar-items.injectable.tsx +++ b/packages/core/src/renderer/components/workloads/workloads-sidebar-items.injectable.tsx @@ -3,24 +3,34 @@ * Licensed under MIT License. See LICENSE in root directory for more information. */ import { getInjectable } from "@ogre-tools/injectable"; -import { sidebarItemInjectionToken } from "@k8slens/cluster-sidebar"; +import { computed } from "mobx"; +import type { + SidebarItemRegistration } from "../layout/sidebar-items.injectable"; +import { + sidebarItemsInjectionToken, +} from "../layout/sidebar-items.injectable"; import { Icon } from "../icon"; import React from "react"; import { noop } from "lodash/fp"; -const workloadsSidebarItemInjectable = getInjectable({ - id: "workloads-sidebar-item", +export const workloadsSidebarItemId = "workloads"; - instantiate: () => ({ - id: "workloads", - parentId: null, - title: "Workloads", - getIcon: () => , - onClick: noop, - orderNumber: 20, - }), +const workloadsSidebarItemsInjectable = getInjectable({ + id: "workloads-sidebar-items", - injectionToken: sidebarItemInjectionToken, + instantiate: () => + computed((): SidebarItemRegistration[] => [ + { + id: workloadsSidebarItemId, + parentId: null, + title: "Workloads", + getIcon: () => , + onClick: noop, + orderNumber: 20, + }, + ]), + + injectionToken: sidebarItemsInjectionToken, }); -export default workloadsSidebarItemInjectable; +export default workloadsSidebarItemsInjectable; diff --git a/packages/core/src/renderer/getDiForUnitTesting.tsx b/packages/core/src/renderer/getDiForUnitTesting.tsx index 548e55f669c8..1a1023a30c67 100644 --- a/packages/core/src/renderer/getDiForUnitTesting.tsx +++ b/packages/core/src/renderer/getDiForUnitTesting.tsx @@ -21,7 +21,6 @@ import { registerFeature } from "@k8slens/feature-core"; import { messagingFeature, testUtils as messagingTestUtils } from "@k8slens/messaging"; import { routingFeature } from "@k8slens/routing"; import { loggerFeature } from "@k8slens/logger"; -import { clusterSidebarFeature } from "@k8slens/cluster-sidebar"; export const getDiForUnitTesting = () => { const environment = "renderer"; @@ -39,7 +38,6 @@ export const getDiForUnitTesting = () => { messagingTestUtils.messagingFeatureForUnitTesting, routingFeature, loggerFeature, - clusterSidebarFeature, ); }); diff --git a/packages/core/src/renderer/routes/sibling-tabs.injectable.ts b/packages/core/src/renderer/routes/sibling-tabs.injectable.ts index e35848e217ab..78c3ba6f58b7 100644 --- a/packages/core/src/renderer/routes/sibling-tabs.injectable.ts +++ b/packages/core/src/renderer/routes/sibling-tabs.injectable.ts @@ -4,7 +4,7 @@ */ import { computed } from "mobx"; import { getInjectable } from "@ogre-tools/injectable"; -import { sidebarItemsInjectable } from "@k8slens/cluster-sidebar"; +import sidebarItemsInjectable from "../components/layout/sidebar-items.injectable"; const siblingTabsInjectable = getInjectable({ id: "sibling-tabs", @@ -12,14 +12,11 @@ const siblingTabsInjectable = getInjectable({ instantiate: (di) => { const sidebarItems = di.inject(sidebarItemsInjectable); - return computed(() => ( - sidebarItems - .get() - .find(({ isActive }) => isActive.get()) - ?.children - .filter(({ isVisible }) => isVisible.get()) - ?? [] - )); + return computed(() => { + const item = sidebarItems.get().find(({ isActive }) => isActive.get()); + + return item?.children ?? []; + }); }, }); diff --git a/packages/utility-features/utilities/index.ts b/packages/utility-features/utilities/index.ts index 8cb46adbebf1..59f90d3da5b3 100644 --- a/packages/utility-features/utilities/index.ts +++ b/packages/utility-features/utilities/index.ts @@ -5,7 +5,7 @@ export * from "./src/base64"; export * from "./src/buildUrl"; export * from "./src/camelCase"; export * from "./src/collection-functions"; -export * from "./src/computed-helpers"; +export * from "./src/computed-or"; export * from "./src/convertCpu"; export * from "./src/convertMemory"; export * from "./src/cssNames"; @@ -28,7 +28,6 @@ export * from "./src/noop"; export * from "./src/object"; export * from "./src/observable-crate"; export * from "./src/on-keyboard-shortcut"; -export * from "./src/orderable"; export * from "./src/prevDefault"; export * from "./src/readableStream"; export * from "./src/readonly"; diff --git a/packages/utility-features/utilities/src/computed-helpers.ts b/packages/utility-features/utilities/src/computed-or.ts similarity index 73% rename from packages/utility-features/utilities/src/computed-helpers.ts rename to packages/utility-features/utilities/src/computed-or.ts index 5524ce5fe84e..4e933949240b 100644 --- a/packages/utility-features/utilities/src/computed-helpers.ts +++ b/packages/utility-features/utilities/src/computed-or.ts @@ -9,7 +9,3 @@ import { computed } from "mobx"; export const computedOr = (...values: IComputedValue[]) => computed(( () => values.some(value => value.get()) )); - -export const computedAnd = (...values: IComputedValue[]) => computed(( - () => values.every(value => value.get()) -)); diff --git a/packages/utility-features/utilities/src/iter.ts b/packages/utility-features/utilities/src/iter.ts index 3d5f8530f5d8..b20f48a67aa5 100644 --- a/packages/utility-features/utilities/src/iter.ts +++ b/packages/utility-features/utilities/src/iter.ts @@ -11,7 +11,7 @@ interface Iterator extends Iterable { find(fn: (val: T) => unknown): T | undefined; collect(fn: (values: Iterable) => U): U; toArray(): T[]; - toMap(): T extends readonly [infer K, infer V] ? Map : never; + toMap(): T extends [infer K, infer V] ? Map : never; toSet(): Set; map(fn: (val: T) => U): Iterator; flatMap(fn: (val: T) => U[]): Iterator; @@ -30,22 +30,7 @@ function chain(src: IterableIterator): Iterator { join: (sep) => join(src, sep), collect: (fn) => fn(src), toArray: () => [...src], - toMap: () => { - const res = new Map(); - - for (const item of src[Symbol.iterator]()) { - const [key, val] = item as [unknown, unknown]; - const existing = res.get(key); - - if (existing) { - existing.push(val); - } else { - res.set(key, [val]); - } - } - - return res as T extends readonly [infer K, infer V] ? Map : never; - }, + toMap: () => new Map(src as IterableIterator<[any, any]>) as any, toSet: () => new Set(src), concat: (src2) => chain(concat(src, src2)), take: (count) => chain(take(src, count)), diff --git a/packages/utility-features/utilities/src/orderable.ts b/packages/utility-features/utilities/src/orderable.ts deleted file mode 100644 index 738808a0e2f6..000000000000 --- a/packages/utility-features/utilities/src/orderable.ts +++ /dev/null @@ -1,14 +0,0 @@ -/** - * The Orderable interface is used to define an object that has an order number. - */ -export interface Orderable { - readonly orderNumber: number; -} - -export interface MaybeOrderable { - readonly orderNumber?: number; -} - -export const byOrderNumber = (left: T, right: T) => ( - (left.orderNumber ?? Number.MAX_SAFE_INTEGER) - (right.orderNumber ?? Number.MAX_SAFE_INTEGER) -);