fix: Address code review feedback - improve type safety and use DOM elements for fallback renderers

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/plugin-aggrid/src/ObjectAgGridImpl.tsx

@@ -466,15 +466,30 @@ function applyFieldTypeFormatting(colDef: ColDef, field: FieldMetadata): void {
         break;
 
       case 'object':
-        colDef.cellRenderer = () => '<span class="text-gray-500 italic">[Object]</span>';
+        colDef.cellRenderer = () => {
+          const span = document.createElement('span');
+          span.className = 'text-gray-500 italic';
+          span.textContent = '[Object]';
+          return span;
+        };
         break;
 
       case 'vector':
-        colDef.cellRenderer = () => '<span class="text-gray-500 italic">[Vector]</span>';
+        colDef.cellRenderer = () => {
+          const span = document.createElement('span');
+          span.className = 'text-gray-500 italic';
+          span.textContent = '[Vector]';
+          return span;
+        };
         break;
 
       case 'grid':
-        colDef.cellRenderer = () => '<span class="text-gray-500 italic">[Grid]</span>';
+        colDef.cellRenderer = () => {
+          const span = document.createElement('span');
+          span.className = 'text-gray-500 italic';
+          span.textContent = '[Grid]';
+          return span;
+        };
         break;
 
       default:

packages/plugin-aggrid/src/field-renderers.tsx

@@ -7,7 +7,7 @@
  */
 
 import React from 'react';
-import { createRoot } from 'react-dom/client';
+import { createRoot, type Root } from 'react-dom/client';
 import type { ICellRendererParams, ICellEditorParams } from 'ag-grid-community';
 import type { FieldMetadata } from '@object-ui/types';
 
@@ -72,7 +72,7 @@ function getFieldWidget(fieldType: string): React.ComponentType<any> | null {
  */
 export class FieldWidgetCellRenderer {
   public eGui!: HTMLDivElement;
-  public root: any;
+  public root: Root | null = null;
 
   init(params: ICellRendererParams & { field: FieldMetadata }) {
     const { value, field } = params;
@@ -132,8 +132,8 @@ export class FieldWidgetCellRenderer {
  */
 export class FieldWidgetCellEditor {
   public eGui!: HTMLDivElement;
-  public root: any;
-  public currentValue: any;
+  public root: Root | null = null;
+  public currentValue: unknown;
   public params!: ICellEditorParams & { field: FieldMetadata };
 
   init(params: ICellEditorParams & { field: FieldMetadata }) {