@opencorestack/opengridx 1.2.0 → 1.2.2

package/lib/components/Cell/Cell.tsx

@@ -239,8 +239,10 @@ function CellImpl<R extends GridRowModel = GridRowModel>(props: CellProps<R>) {
                         />
                     )
                 ) : colDef.renderCell ? (
-                    <CellErrorBoundary field={colDef.field} resetKey={value}>
-                        {colDef.renderCell({
+                    <CellErrorBoundary
+                        field={colDef.field}
+                        resetKey={row}
+                        renderFn={() => colDef.renderCell!({
                             value,
                             row,
                             field: colDef.field,
@@ -249,7 +251,7 @@ function CellImpl<R extends GridRowModel = GridRowModel>(props: CellProps<R>) {
                             colIndex,
                             rowMeta,
                         })}
-                    </CellErrorBoundary>
+                    />
                 ) : (
                     formattedValue
                 )}

package/lib/components/Cell/CellErrorBoundary.tsx

@@ -1,7 +1,14 @@
 import React from 'react';
 
+// Must live at module scope (not inline) so React sees a stable component type.
+// Calling renderFn() here puts the throw one level BELOW CellErrorBoundary,
+// which is required — a component cannot catch errors thrown in its own render().
+function CellRenderTarget({ renderFn }: { renderFn: () => React.ReactNode }) {
+    return <>{renderFn()}</>;
+}
+
 interface CellErrorBoundaryProps {
-    children: React.ReactNode;
+    renderFn: () => React.ReactNode;
     field: string;
     resetKey?: unknown;
 }
@@ -35,6 +42,12 @@ export class CellErrorBoundary extends React.Component<CellErrorBoundaryProps, C
         return { hasError: true, error };
     }
 
+    override componentDidCatch(error: Error, info: React.ErrorInfo) {
+        if (process.env.NODE_ENV !== 'production') {
+            console.warn(`[CellErrorBoundary] field="${this.props.field}"`, error, info.componentStack);
+        }
+    }
+
     override render() {
         if (this.state.hasError) {
             return (
@@ -48,6 +61,6 @@ export class CellErrorBoundary extends React.Component<CellErrorBoundaryProps, C
                 </div>
             );
         }
-        return this.props.children;
+        return <CellRenderTarget renderFn={this.props.renderFn} />;
     }
 }

package/lib/components/DataGrid/DataGrid.tsx

@@ -467,6 +467,7 @@ export function DataGrid<R extends GridRowModel = GridRowModel>(props: DataGridP
 
     useEffect(() => {
         gridData.apiRef.current.getVisibleRows = () => pagination ? paginatedUnpinnedRows : sortedUnpinnedRows;
+        gridData.apiRef.current.getAllFilteredRows = () => sortedUnpinnedRows;
         gridData.apiRef.current.getVisibleColumns = () => effectiveColumns as unknown as GridColDef[];
     }, [sortedUnpinnedRows, paginatedUnpinnedRows, pagination, effectiveColumns, gridData.apiRef]);
 

package/lib/hooks/core/useDataGrid.ts

@@ -299,6 +299,7 @@ export function useDataGrid<R extends GridRowModel = GridRowModel>(params: UseDa
     getRow: (id: GridRowId) => stateRef.current.rows.idRowsLookup.get(id) || null,
     getAllRows: () => Array.from(stateRef.current.rows.idRowsLookup.values()),
     getVisibleRows: () => Array.from(stateRef.current.rows.idRowsLookup.values()),
+    getAllFilteredRows: () => Array.from(stateRef.current.rows.idRowsLookup.values()),
     getAggregationResult: () => null,
     getAggregationModel: () => null,
     getColumn: (field: string) => stateRef.current.columns.lookup.get(field) || null,

package/lib/types/index.ts

@@ -817,6 +817,8 @@ export interface GridApi {
   getAllRows: () => GridRowModel[];
   /** Returns all rows currently visible after filtering and sorting. */
   getVisibleRows: () => GridRowModel[];
+  /** Returns all filtered and sorted rows, ignoring pagination. Use this for full-dataset exports. */
+  getAllFilteredRows: () => GridRowModel[];
   /** Returns the current aggregation results. */
   getAggregationResult: () => Record<string, unknown> | null;
   /** Returns the active aggregation configuration. */

package/lib/utils/export/exportToPdf.test.ts

@@ -15,7 +15,6 @@ const mockAddImage = vi.fn();
 const mockSetPage = vi.fn();
 
 class MockJsPDF {
-    autoTable = mockAutoTable;
     save = mockSave;
     setFontSize = mockSetFontSize;
     setFont = mockSetFont;
@@ -35,7 +34,7 @@ class MockJsPDF {
 }
 
 vi.mock('jspdf', () => ({ default: MockJsPDF }));
-vi.mock('jspdf-autotable', () => ({}));
+vi.mock('jspdf-autotable', () => ({ default: mockAutoTable }));
 
 const sampleColumns: GridColDef[] = [
     { field: 'name', headerName: 'Name', width: 150 },
@@ -72,7 +71,7 @@ describe('exportToPdf', () => {
     it('excludes columns with exportable: false and system columns', async () => {
         const { exportToPdf } = await import('./exportToPdf');
         await exportToPdf(sampleRows, sampleColumns);
-        const call = mockAutoTable.mock.calls[0][0];
+        const call = mockAutoTable.mock.calls[0][1];
         // head should only have Name and Salary — not Hidden or __check__
         expect(call.head[0]).toEqual(['Name', 'Salary']);
     });
@@ -80,7 +79,7 @@ describe('exportToPdf', () => {
     it('applies valueFormatter to cell values', async () => {
         const { exportToPdf } = await import('./exportToPdf');
         await exportToPdf(sampleRows, sampleColumns);
-        const call = mockAutoTable.mock.calls[0][0];
+        const call = mockAutoTable.mock.calls[0][1];
         // salary column should be formatted
         expect(call.body[0][1]).toBe('$90000');
         expect(call.body[1][1]).toBe('$80000');
@@ -89,7 +88,7 @@ describe('exportToPdf', () => {
     it('filters to selectedRows when provided', async () => {
         const { exportToPdf } = await import('./exportToPdf');
         await exportToPdf(sampleRows, sampleColumns, { selectedRows: [1] });
-        const call = mockAutoTable.mock.calls[0][0];
+        const call = mockAutoTable.mock.calls[0][1];
         expect(call.body).toHaveLength(1);
         expect(call.body[0][0]).toBe('Alice');
     });
@@ -100,7 +99,7 @@ describe('exportToPdf', () => {
             aggregationResult: { salary: 170000 },
             aggregationModel: { salary: 'sum' },
         });
-        const call = mockAutoTable.mock.calls[0][0];
+        const call = mockAutoTable.mock.calls[0][1];
         expect(call.foot).toBeDefined();
         expect(call.foot[0][1]).toBe('$170000');
     });

package/lib/utils/export/exportToPdf.ts

@@ -6,9 +6,7 @@ import type {
 } from '../../types';
 import { formatAggregationValue } from '../../hooks/features/useAggregation';
 
-// jsPDF plugin augments the jsPDF prototype — we type only what we use
 interface JsPDFDoc {
-    autoTable: (opts: AutoTableOptions) => void;
     save: (filename: string) => void;
     setFontSize: (size: number) => void;
     setFont: (name: string, style: string) => void;
@@ -112,10 +110,14 @@ export async function exportToPdf<R extends GridRowModel>(
 ): Promise<void> {
     // Lazy-load peer deps — provides a clear error if not installed
     let JsPDF: new (opts: Record<string, unknown>) => JsPDFDoc;
+    let autoTable: (doc: JsPDFDoc, opts: AutoTableOptions) => void;
     try {
-        const mod = await import('jspdf');
-        JsPDF = mod.default as new (opts: Record<string, unknown>) => JsPDFDoc;
-        await import('jspdf-autotable');
+        const [jspdfMod, autoTableMod] = await Promise.all([
+            import('jspdf'),
+            import('jspdf-autotable'),
+        ]);
+        JsPDF = jspdfMod.default as unknown as new (opts: Record<string, unknown>) => JsPDFDoc;
+        autoTable = autoTableMod.default as unknown as (doc: JsPDFDoc, opts: AutoTableOptions) => void;
     } catch {
         throw new Error(
             "exportToPdf requires 'jspdf' and 'jspdf-autotable'. Run: npm install jspdf jspdf-autotable"
@@ -235,7 +237,7 @@ export async function exportToPdf<R extends GridRowModel>(
     const [hr, hg, hb] = hexToRgb(headerBackgroundColor);
     const [tr, tg, tb] = hexToRgb(headerTextColor);
 
-    doc.autoTable({
+    autoTable(doc, {
         head,
         body,
         ...(foot ? { foot } : {}),
@@ -250,7 +252,7 @@ export async function exportToPdf<R extends GridRowModel>(
             ? { alternateRowStyles: { fillColor: [248, 250, 252] } }
             : {}),
         ...(foot
-            ? { footStyles: { fillColor: [241, 245, 249], fontStyle: 'bold' } }
+            ? { footStyles: { fillColor: [241, 245, 249], fontStyle: 'bold', textColor: [0, 0, 0] } }
             : {}),
         columnStyles,
         showFoot: foot ? 'lastPage' : 'never',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opencorestack/opengridx",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "description": "OpenGridX: High-performance React data infrastructure. Unlock advanced Row Grouping, Excel Export, and Column Pinning without the usual \"Pro\" gatekeeping. Built for speed, scale, and complete architectural freedom. Fully open, virtualization-ready, and feature-complete.",
   "type": "module",
   "main": "./dist/opengridx.umd.js",
@@ -109,6 +109,8 @@
     "eslint-plugin-react-refresh": "^0.4.24",
     "globals": "^16.5.0",
     "jsdom": "^29.1.1",
+    "jspdf": "^4.2.1",
+    "jspdf-autotable": "^5.0.8",
     "react": "^19.2.0",
     "react-dom": "^19.2.0",
     "react-router-dom": "^7.13.1",