Merge branch 'main' into feat/show-resource-errors

Author: lucasgoral

.github/workflows/build.yaml

@@ -15,7 +15,7 @@ jobs:
       - name: Set up Node.js
         uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
         with:
-          node-version: 22
+          node-version: 24
           cache: 'npm' # Added caching for faster installs
 
       - name: Install dependencies

.github/workflows/main.yaml

@@ -32,7 +32,7 @@ jobs:
 
       - name: Docker meta
         id: meta
-        uses: docker/metadata-action@902fa8ec7d6ecbf8d84d538b9b233a880e428804 # v5
+        uses: docker/metadata-action@c1e51972afc2121e065aed6d45c65596fe445f3f # v5
         with:
           images: ${{env.REGISTRY}}/${{env.IMAGE_NAME}}
           tags: |

Dockerfile

@@ -18,7 +18,7 @@ RUN npm prune --omit=dev
 
 
 # PRODUCTION STAGE
-FROM gcr.io/distroless/nodejs24-debian12@sha256:20a51c926c0bb68a9b1f7059c81516da002655f8a896a2cb7bc56b56974782b3 AS production
+FROM gcr.io/distroless/nodejs24-debian12@sha256:ca801d691284502b3b433d9fbd13f85b5c222f62b3716e2fa8838f72a8f5c17c AS production
 WORKDIR /usr/src/app
 
 # Copy built files

cypress/support/commands.ts

@@ -1,2 +1,2 @@
 import '@ui5/webcomponents-cypress-commands';
-import "../../i18n";
+import "../../src/utils/i18n/i18n";

i18n.ts

@@ -1,21 +1,25 @@
-import React, { useState, useRef } from 'react';
+import React, { useState, useRef, useMemo } from 'react';
 import { CreateProjectWorkspaceDialog, OnCreatePayload } from './CreateProjectWorkspaceDialog';
 import { MemberRoles } from '../../lib/api/types/shared/members';
 import { ErrorDialogHandle } from '../Shared/ErrorMessageBox';
 
 import { useForm } from 'react-hook-form';
 import { zodResolver } from '@hookform/resolvers/zod';
-import { validationSchemaProjectWorkspace } from '../../lib/api/validations/schemas.ts';
+import { createProjectWorkspaceSchema } from '../../lib/api/validations/schemas.ts';
 import { CreateDialogProps } from './CreateWorkspaceDialogContainer.tsx';
+import { useTranslation } from 'react-i18next';
 
 export const CreateProjectWorkspaceDialogWrapper: React.FC<{
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   spyFormBody?: (data: any) => object;
 }> = ({ spyFormBody }) => {
   const [isOpen, setIsOpen] = useState(true);
+  const { t } = useTranslation();
 
   const errorDialogRef = useRef<ErrorDialogHandle>(null);
 
+  const validationSchemaProjectWorkspace = useMemo(() => createProjectWorkspaceSchema(t), [t]);
+
   const {
     register,
     handleSubmit,

src/components/Dialogs/CreateProjectDialog.cy.tsx

@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useRef } from 'react';
+import { useCallback, useEffect, useMemo, useRef } from 'react';
 import { useApiResourceMutation } from '../../lib/api/useApiResource';
 import { ErrorDialogHandle } from '../Shared/ErrorMessageBox.tsx';
 import { APIError } from '../../lib/api/error';
@@ -12,7 +12,7 @@ import { useTranslation } from 'react-i18next';
 import { zodResolver } from '@hookform/resolvers/zod';
 import { useForm } from 'react-hook-form';
 import { CreateProject, CreateProjectResource, CreateProjectType } from '../../lib/api/types/crate/createProject.ts';
-import { validationSchemaProjectWorkspace } from '../../lib/api/validations/schemas.ts';
+import { createProjectWorkspaceSchema } from '../../lib/api/validations/schemas.ts';
 import { CreateDialogProps } from './CreateWorkspaceDialogContainer.tsx';
 
 export function CreateProjectDialogContainer({
@@ -22,6 +22,8 @@ export function CreateProjectDialogContainer({
   isOpen: boolean;
   setIsOpen: (isOpen: boolean) => void;
 }) {
+  const { t } = useTranslation();
+  const validationSchemaProjectWorkspace = useMemo(() => createProjectWorkspaceSchema(t), [t]);
   const {
     watch,
     register,
@@ -39,7 +41,6 @@ export function CreateProjectDialogContainer({
       members: [],
     },
   });
-  const { t } = useTranslation();
   const { user } = useAuthOnboarding();
 
   const username = user?.email;

src/components/Dialogs/CreateProjectDialogContainer.tsx

@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useRef } from 'react';
+import { useCallback, useEffect, useMemo, useRef } from 'react';
 import { useApiResourceMutation, useRevalidateApiResource } from '../../lib/api/useApiResource';
 import { ErrorDialogHandle } from '../Shared/ErrorMessageBox.tsx';
 import { APIError } from '../../lib/api/error';
@@ -16,7 +16,7 @@ import { Member, MemberRoles } from '../../lib/api/types/shared/members.ts';
 import { useTranslation } from 'react-i18next';
 import { zodResolver } from '@hookform/resolvers/zod';
 import { useForm } from 'react-hook-form';
-import { validationSchemaProjectWorkspace } from '../../lib/api/validations/schemas.ts';
+import { createProjectWorkspaceSchema } from '../../lib/api/validations/schemas.ts';
 import { ComponentsListItem } from '../../lib/api/types/crate/createManagedControlPlane.ts';
 
 export type CreateDialogProps = {
@@ -37,6 +37,8 @@ export function CreateWorkspaceDialogContainer({
   setIsOpen: (isOpen: boolean) => void;
   project?: string;
 }) {
+  const { t } = useTranslation();
+  const validationSchemaProjectWorkspace = useMemo(() => createProjectWorkspaceSchema(t), [t]);
   const {
     register,
     handleSubmit,
@@ -54,7 +56,6 @@ export function CreateWorkspaceDialogContainer({
       chargingTargetType: '',
     },
   });
-  const { t } = useTranslation();
   const { user } = useAuthOnboarding();
 
   const username = user?.email;

src/components/Dialogs/CreateWorkspaceDialogContainer.tsx

@@ -23,7 +23,7 @@ import { useTranslation } from 'react-i18next';
 import { useAuthOnboarding } from '../../../spaces/onboarding/auth/AuthContextOnboarding.tsx';
 import { ErrorDialog, ErrorDialogHandle } from '../../Shared/ErrorMessageBox.tsx';
 import { CreateDialogProps } from '../../Dialogs/CreateWorkspaceDialogContainer.tsx';
-import { validationSchemaCreateManagedControlPlane } from '../../../lib/api/validations/schemas.ts';
+import { createManagedControlPlaneSchema } from '../../../lib/api/validations/schemas.ts';
 import { Member, MemberRoles } from '../../../lib/api/types/shared/members.ts';
 import { useApiResourceMutation } from '../../../lib/api/useApiResource.ts';
 import {
@@ -62,6 +62,7 @@ export const CreateManagedControlPlaneWizardContainer: FC<CreateManagedControlPl
   const errorDialogRef = useRef<ErrorDialogHandle>(null);
 
   const [selectedStep, setSelectedStep] = useState<WizardStepType>('metadata');
+  const validationSchemaCreateManagedControlPlane = useMemo(() => createManagedControlPlaneSchema(t), [t]);
 
   const {
     register,

src/components/Wizards/CreateManagedControlPlane/CreateManagedControlPlaneWizardContainer.tsx

@@ -1,56 +1,59 @@
 import { z } from 'zod';
 import { Member } from '../types/shared/members.ts';
-import i18n from '../../../../i18n.ts';
+import { TFunction } from 'i18next';
 import { btpChargingTargetRegex, managedControlPlaneNameRegex, projectWorkspaceNameRegex } from './regex.ts';
 
-const { t } = i18n;
-
 const member = z.custom<Member>();
 
 // Shared superRefine helper for charging target validation
-function validateChargingTarget<T extends { chargingTargetType?: string; chargingTarget?: string }>(
-  data: T,
-  ctx: z.RefinementCtx,
+function createValidateChargingTarget<T extends { chargingTargetType?: string; chargingTarget?: string }>(
+  t: TFunction,
 ) {
-  if (data.chargingTargetType && data.chargingTarget && !btpChargingTargetRegex.test(data.chargingTarget ?? '')) {
-    ctx.addIssue({
-      path: ['chargingTarget'],
-      code: z.ZodIssueCode.custom,
-      message: t('validationErrors.notValidChargingTargetFormat'),
-    });
-  } else if (data.chargingTargetType && !data.chargingTarget) {
-    ctx.addIssue({
-      path: ['chargingTarget'],
-      code: z.ZodIssueCode.custom,
-      message: t('validationErrors.required'),
-    });
-  }
+  return (data: T, ctx: z.RefinementCtx) => {
+    if (data.chargingTargetType && data.chargingTarget && !btpChargingTargetRegex.test(data.chargingTarget ?? '')) {
+      ctx.addIssue({
+        path: ['chargingTarget'],
+        code: z.ZodIssueCode.custom,
+        message: t('validationErrors.notValidChargingTargetFormat'),
+      });
+    } else if (data.chargingTargetType && !data.chargingTarget) {
+      ctx.addIssue({
+        path: ['chargingTarget'],
+        code: z.ZodIssueCode.custom,
+        message: t('validationErrors.required'),
+      });
+    }
+  };
 }
 
-export const validationSchemaProjectWorkspace = z
-  .object({
-    name: z
-      .string()
-      .min(1, t('validationErrors.required'))
-      .regex(projectWorkspaceNameRegex, t('validationErrors.properFormatting'))
-      .max(25, t('validationErrors.maxChars', { maxLength: 25 })),
-    displayName: z.string().optional(),
-    chargingTarget: z.string().optional(),
-    chargingTargetType: z.string().optional(),
-    members: z.array(member).refine((members) => members?.length > 0),
-  })
-  .superRefine(validateChargingTarget);
+export function createProjectWorkspaceSchema(t: TFunction) {
+  return z
+    .object({
+      name: z
+        .string()
+        .min(1, t('validationErrors.required'))
+        .regex(projectWorkspaceNameRegex, t('validationErrors.properFormatting'))
+        .max(25, t('validationErrors.maxChars', { maxLength: 25 })),
+      displayName: z.string().optional(),
+      chargingTarget: z.string().optional(),
+      chargingTargetType: z.string().optional(),
+      members: z.array(member).refine((members) => members?.length > 0),
+    })
+    .superRefine(createValidateChargingTarget(t));
+}
 
-export const validationSchemaCreateManagedControlPlane = z
-  .object({
-    name: z
-      .string()
-      .min(1, t('validationErrors.required'))
-      .regex(managedControlPlaneNameRegex, t('validationErrors.properFormattingLowercase'))
-      .max(36, t('validationErrors.maxChars', { maxLength: 36 })),
-    displayName: z.string().optional(),
-    chargingTarget: z.string().optional(),
-    chargingTargetType: z.string().optional(),
-    members: z.array(member),
-  })
-  .superRefine(validateChargingTarget);
+export function createManagedControlPlaneSchema(t: TFunction) {
+  return z
+    .object({
+      name: z
+        .string()
+        .min(1, t('validationErrors.required'))
+        .regex(managedControlPlaneNameRegex, t('validationErrors.properFormattingLowercase'))
+        .max(36, t('validationErrors.maxChars', { maxLength: 36 })),
+      displayName: z.string().optional(),
+      chargingTarget: z.string().optional(),
+      chargingTargetType: z.string().optional(),
+      members: z.array(member),
+    })
+    .superRefine(createValidateChargingTarget(t));
+}