feat(ReportViewer): enhance chart rendering logic with type inference and data mapping

Author: hotlong

packages/plugin-report/src/ReportViewer.tsx

@@ -150,7 +150,47 @@ export const ReportViewer: React.FC<ReportViewerProps> = ({ schema }) => {
 
                 {section.type === 'chart' && section.chart && (
                   <div className="min-h-[300px]">
-                    <SchemaRenderer schema={{ ...section.chart, data: data || section.chart.data }} />
+                    {(() => {
+                      // 1. Determine Component Type
+                      // If explicit 'type' is missing, but 'chartType' exists (e.g. "line"), infer 'chart'
+                      let type = section.chart.type;
+                      const hasChartType = !!section.chart.chartType; 
+                      
+                      // If no strict type but has chartType, assume 'chart' generic renderer
+                      if (!type && hasChartType) {
+                        type = 'chart';
+                      }
+                      
+                      // Fallback validation: If resolved type is not registered, try 'chart'
+                      const isRegistered = type && !!ComponentRegistry.get(type);
+                      if (!isRegistered) {
+                         // Even if 'line' was somehow passed as type, fallback to 'chart'
+                         type = 'chart';
+                      }
+
+                      // 2. Data Adapter (Report Schema -> Chart Component Schema)
+                      // The generic 'chart' component needs mapped props (xAxisKey, series)
+                      // whereas Report schema uses (xAxisField, yAxisFields)
+                      const isGenericChart = type === 'chart';
+                      const adapterProps = isGenericChart ? {
+                        xAxisKey: section.chart.xAxisKey || section.chart.xAxisField || 'name',
+                        series: section.chart.series || (section.chart.yAxisFields ? section.chart.yAxisFields.map((f: any) => ({ dataKey: f })) : []),
+                        // Ensure chartType is passed if we are using the generic renderer
+                        chartType: section.chart.chartType || 'bar',
+                      } : {};
+
+                      // 3. Construct Safe Schema
+                      const safeSchema = {
+                        ...section.chart,
+                        type,
+                        ...adapterProps,
+                        data: data || section.chart.data,
+                        // Force explicit height to preventing Recharts "height(-1)" error
+                        className: section.chart.className || 'w-full h-[350px]'
+                      };
+
+                      return <SchemaRenderer schema={safeSchema} />;
+                    })()}
                   </div>
                 )}