kubev2v
diff --git a/‎.cursor/rules/react-components.mdc‎
Lines changed: 38 additions & 1 deletion b/‎.cursor/rules/react-components.mdc‎
Lines changed: 38 additions & 1 deletion
diff --git a/‎.cursor/rules/selectors.mdc‎
Lines changed: 36 additions & 0 deletions b/‎.cursor/rules/selectors.mdc‎
Lines changed: 36 additions & 0 deletions
diff --git a/‎cspell.wordlist.txt‎
Lines changed: 3 additions & 0 deletions b/‎cspell.wordlist.txt‎
Lines changed: 3 additions & 0 deletions
diff --git a/‎i18n-scripts.config.json‎
Lines changed: 3 additions & 1 deletion b/‎i18n-scripts.config.json‎
Lines changed: 3 additions & 1 deletion
diff --git a/‎locales/en/plugin__forklift-console-plugin.json‎
Lines changed: 42 additions & 1 deletion b/‎locales/en/plugin__forklift-console-plugin.json‎
Lines changed: 42 additions & 1 deletion
diff --git a/‎locales/es/plugin__forklift-console-plugin.json‎
Lines changed: 44 additions & 1 deletion b/‎locales/es/plugin__forklift-console-plugin.json‎
Lines changed: 44 additions & 1 deletion
diff --git a/‎locales/fr/plugin__forklift-console-plugin.json‎
Lines changed: 44 additions & 1 deletion b/‎locales/fr/plugin__forklift-console-plugin.json‎
Lines changed: 44 additions & 1 deletion
diff --git a/‎locales/ja/plugin__forklift-console-plugin.json‎
Lines changed: 40 additions & 1 deletion b/‎locales/ja/plugin__forklift-console-plugin.json‎
Lines changed: 40 additions & 1 deletion
diff --git a/‎locales/ko/plugin__forklift-console-plugin.json‎
Lines changed: 40 additions & 1 deletion b/‎locales/ko/plugin__forklift-console-plugin.json‎
Lines changed: 40 additions & 1 deletion
diff --git a/‎locales/zh/plugin__forklift-console-plugin.json‎
Lines changed: 40 additions & 1 deletion b/‎locales/zh/plugin__forklift-console-plugin.json‎
Lines changed: 40 additions & 1 deletion
@@ -117,7 +117,44 @@ const { control, handleSubmit, formState: { errors } } = methods;
 
 ---
 
-## 6. Test Identifiers
+## 6. Modals
+
+Always use the shared `ModalForm` component (`@components/ModalForm/ModalForm`) when creating new modals. It handles:
+- Modal shell (open/close, variant, position)
+- Loading state on confirm button
+- Error display (catches thrown errors automatically)
+- Confirm/Cancel buttons with proper variants
+- `closeModal` on success
+
+```typescript
+import ModalForm from '@components/ModalForm/ModalForm';
+
+const MyModal: ModalComponent<MyModalProps> = ({ myData, ...rest }) => {
+  const handleConfirm = useCallback(async () => {
+    await doSomething(myData);
+    // throw new Error('message') on failure -- ModalForm catches and displays it
+  }, [myData]);
+
+  return (
+    <ModalForm
+      title={t('My title')}
+      onConfirm={handleConfirm}
+      confirmLabel={t('Confirm')}
+      isDisabled={!isValid}
+      variant={ModalVariant.large}
+      {...rest}
+    >
+      {/* Modal body content */}
+    </ModalForm>
+  );
+};
+```
+
+Do NOT use raw `Modal` + `ModalHeader` + `ModalBody` + `ModalFooter` + manual error/loading state.
+
+---
+
+## 7. Test Identifiers
 
 Use `testId` prop for test selectors:
 ```typescript
 
@@ -0,0 +1,36 @@
+---
+description: Use selectors for K8s resource property access
+globs: "**/*.{ts,tsx}"
+alwaysApply: false
+---
+
+# Selector Usage Rule
+
+When accessing properties of Kubernetes resources (CRDs, Pods, Secrets, etc.), always use a selector function instead of direct property chains.
+
+## Common selectors (`@utils/crds/common/selectors`)
+
+Available for all `K8sResourceCommon` resources:
+- `getName(resource)` instead of `resource?.metadata?.name`
+- `getNamespace(resource)` instead of `resource?.metadata?.namespace`
+- `getUID(resource)` instead of `resource?.metadata?.uid`
+- `getCreatedAt(resource)` instead of `resource?.metadata?.creationTimestamp`
+- `getLabels(resource)` instead of `resource?.metadata?.labels`
+- `getAnnotations(resource)` instead of `resource?.metadata?.annotations`
+- `getApiVersion(resource)` instead of `resource?.apiVersion`
+- `getKind(resource)` instead of `resource?.kind`
+
+Provider-specific:
+- `getType(provider)` instead of `provider?.spec?.type`
+- `getProviderSecretRef(provider)` instead of `provider?.spec?.secret`
+- `getVddkInitImage(provider)` instead of `provider?.spec?.settings?.vddkInitImage`
+
+## CRD-specific selectors
+
+If a property access is specific to a CRD (e.g., Conversion, Plan), check for existing selectors under `@utils/crds/<crd>/selectors.ts`.
+
+## When no selector exists
+
+1. If the property is common to all K8s resources, add it to `src/utils/crds/common/selectors.ts`
+2. If the property is CRD-specific, add it to the CRD's selectors file under `src/utils/crds/<crd>/selectors.ts`
+3. Never access `metadata.*` or `spec.*` or `status.*` directly in component or hook code
@@ -55,6 +55,9 @@ snapshotting
 hotplug
 RHVM
 metav
+moref
+mountpoint
+mountpoints
 tolerations
 Virtualmachine
 domainname
 
@@ -5,6 +5,8 @@
   "languages": ["ja", "zh-cn", "ko", "fr", "es"],
   "languageAliases": { "zh-cn": "zh" },
   "pluralOverrides": {
-    "{{count}} virtual machine selected_other": "{{count}} virtual machines selected"
+    "{{count}} virtual machine selected_other": "{{count}} virtual machines selected",
+    "Failed to create inspection for {{count}} VM_other": "Failed to create inspections for {{count}} VMs",
+    "Inspect {{count}} VM_other": "Inspect {{count}} VMs"
   }
 }
Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,8 @@`
`5`	`5`	`"languages": ["ja", "zh-cn", "ko", "fr", "es"],`
`6`	`6`	`"languageAliases": { "zh-cn": "zh" },`
`7`	`7`	`"pluralOverrides": {`
`8`		`- "{{count}} virtual machine selected_other": "{{count}} virtual machines selected"`
	`8`	`+ "{{count}} virtual machine selected_other": "{{count}} virtual machines selected",`
	`9`	`+ "Failed to create inspection for {{count}} VM_other": "Failed to create inspections for {{count}} VMs",`
	`10`	`+ "Inspect {{count}} VM_other": "Inspect {{count}} VMs"`
`9`	`11`	`}`
`10`	`12`	`}`