📄 openmetadata-ui/src/main/resources/ui/src/components/DataQuality/IncidentManager/DimensionalityTab/DimensionalityTab.tsx:285-299
The cell rendering logic (lines 286–323) uses a 4-level nested ternary expression to switch on col.id. This is hard to read and maintain. Consider extracting a helper function or using a switch statement / lookup map pattern, which is more consistent with how other core-ui Table consumers typically render cells.

📄 openmetadata-ui/src/main/resources/ui/src/components/DataQuality/IncidentManager/DimensionalityTab/DimensionalityTab.tsx:181-190
dimensionTableColumns is declared as a plain array inside the component body (lines 181–190). Since it depends on t() (which is stable across renders unless the language changes), wrapping it in useMemo with [t] as a dependency would avoid creating a new array reference on every render, preventing unnecessary reconciliation of Table.Header.

📄 openmetadata-ui/src/main/resources/ui/src/components/IncidentManager/IncidentManager.component.tsx:628
At line 628, key={record.id} passes undefined as the React key when record.id is absent (it's an optional field in TestCaseResolutionStatus). If multiple records lack an id, React cannot distinguish them during reconciliation, leading to duplicate-key warnings and potentially corrupted component state. The id prop also falls back to '', giving multiple rows the same DOM id.

📄 openmetadata-ui/src/main/resources/ui/src/components/IncidentManager/IncidentManager.component.tsx:615-629
renderRow is a plain function that closes over multiple reactive values (testCasePermissions, isPermissionLoading, handleStatusSubmit, etc.). Because it's recreated on every render, the Table.Body's dependencies array alone controls re-rendering — but having a new function reference every time defeats React's ability to bail out of rendering children. Wrapping it in useCallback with the same deps used in dependencies would be more consistent.

📄 openmetadata-ui/src/main/resources/ui/src/components/DataQuality/IncidentManager/FailedTestCaseSampleData/FailedTestCaseSampleData.component.tsx:292
The class failed-sample-data-column is applied at line 292 but has no corresponding style definition anywhere in the codebase (the .less file import was removed, and no global definition exists). Since Tailwind utility tw:bg-error-primary is already applied alongside it, the orphan class name is dead code that adds confusion.