@izumisy-tailor/tailor-data-viewer 0.2.21 → 0.2.22

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@izumisy-tailor/tailor-data-viewer",
   "private": false,
-  "version": "0.2.21",
+  "version": "0.2.22",
   "type": "module",
   "description": "Flexible data viewer component for Tailor Platform",
   "files": [

package/src/component/data-table/data-table-context.tsx

@@ -1,5 +1,11 @@
 import { createContext, useContext } from "react";
-import type { Column, PageInfo, RowAction, RowOperations, SortState } from "../types";
+import type {
+  Column,
+  PageInfo,
+  RowAction,
+  RowOperations,
+  SortState,
+} from "../types";
 
 /**
  * Context value provided by `DataTable.Provider`.

package/src/component/data-table/data-table.tsx

@@ -218,30 +218,21 @@ function DataTableBody({
     <Table.Body className={className}>
       {loading && (!rows || rows.length === 0) && (
         <Table.Row>
-          <Table.Cell
-            colSpan={totalColSpan}
-            className="h-24 text-center"
-          >
+          <Table.Cell colSpan={totalColSpan} className="h-24 text-center">
             <span className="text-muted-foreground">Loading...</span>
           </Table.Cell>
         </Table.Row>
       )}
       {error && (
         <Table.Row>
-          <Table.Cell
-            colSpan={totalColSpan}
-            className="h-24 text-center"
-          >
+          <Table.Cell colSpan={totalColSpan} className="h-24 text-center">
             <span className="text-destructive">Error: {error.message}</span>
           </Table.Cell>
         </Table.Row>
       )}
       {!loading && !error && (!rows || rows.length === 0) && (
         <Table.Row>
-          <Table.Cell
-            colSpan={totalColSpan}
-            className="h-24 text-center"
-          >
+          <Table.Cell colSpan={totalColSpan} className="h-24 text-center">
             <span className="text-muted-foreground">No data</span>
           </Table.Cell>
         </Table.Row>
@@ -374,15 +365,16 @@ function RowActionsMenu<TRow extends Record<string, unknown>>({
   }, [open]);
 
   // Close on Escape
-  const handleKeyDown = useCallback(
-    (e: React.KeyboardEvent) => {
-      if (e.key === "Escape") setOpen(false);
-    },
-    [],
-  );
+  const handleKeyDown = useCallback((e: React.KeyboardEvent) => {
+    if (e.key === "Escape") setOpen(false);
+  }, []);
 
   return (
-    <div className="relative inline-block" ref={menuRef} onKeyDown={handleKeyDown}>
+    <div
+      className="relative inline-block"
+      ref={menuRef}
+      onKeyDown={handleKeyDown}
+    >
       <button
         type="button"
         className="inline-flex h-8 w-8 items-center justify-center rounded-md text-sm hover:bg-accent"

package/src/component/field-helpers.test.ts

@@ -73,6 +73,43 @@ describe("createColumnHelper()", () => {
     expect(col.width).toBe(100);
     expect(typeof col.render).toBe("function");
   });
+
+  it("inferColumns returns column/columns helpers with TRow bound", () => {
+    type TaskRow = { id: string; title: string; status: string };
+    const metadata = {
+      name: "task",
+      pluralForm: "tasks",
+      readAllowedRoles: [],
+      fields: [
+        { name: "id", type: "uuid", required: true },
+        { name: "title", type: "string", required: true },
+        {
+          name: "status",
+          type: "enum",
+          required: true,
+          enumValues: ["todo", "done"],
+        },
+      ],
+    } as const;
+
+    const { inferColumns, display } = createColumnHelper<TaskRow>();
+    const { column, columns } = inferColumns(metadata);
+
+    // column() works with auto-detection
+    const titleCol = column("title");
+    expect(titleCol.dataKey).toBe("title");
+    expect(titleCol.sort).toEqual({ type: "string" });
+
+    // columns() works
+    const cols = columns(["title", "status"]);
+    expect(cols).toHaveLength(2);
+
+    // display still works alongside inferColumns
+    const actionsCol = display("actions", {
+      render: (row) => `${row.title}`,
+    });
+    expect(actionsCol.kind).toBe("display");
+  });
 });
 
 describe("fieldTypeToSortConfig", () => {

package/src/component/field-helpers.ts

@@ -87,21 +87,27 @@ export function display<TRow extends Record<string, unknown>>(
 // =============================================================================
 
 /**
- * Create `field` and `display` helpers with TRow bound once.
+ * Create `field`, `display`, and `inferColumns` helpers with TRow bound once.
  *
- * This avoids repeating the row type parameter on every `field()` / `display()` call.
+ * This avoids repeating the row type parameter on every helper call.
+ * Use `inferColumns(tableMetadata)` to create metadata-inferred columns
+ * without specifying TRow again.
  *
  * @example
  * ```tsx
  * type Order = { id: string; title: string; amount: number };
  *
- * const { field, display } = createColumnHelper<Order>();
+ * const { field, display, inferColumns } = createColumnHelper<Order>();
  *
- * const columns = [
+ * // Manual columns
+ * const manualColumns = [
  *   field("title", { label: "Title", sort: { type: "string" } }),
- *   field("amount", { label: "Amount", sort: { type: "number" } }),
  *   display("actions", { render: (row) => <ActionMenu row={row} /> }),
  * ];
+ *
+ * // Metadata-inferred columns (no need to pass TRow again)
+ * const { column } = inferColumns(tableMetadata.order);
+ * const inferredColumns = [column("title"), column("amount")];
  * ```
  */
 export function createColumnHelper<TRow extends Record<string, unknown>>(): {
@@ -113,10 +119,23 @@ export function createColumnHelper<TRow extends Record<string, unknown>>(): {
     id: string,
     options: DisplayColumnOptions<TRow>,
   ) => DisplayColumn<TRow>;
+  inferColumns: <const TTable extends TableMetadata>(
+    tableMetadata: TTable,
+  ) => {
+    column: (
+      dataKey: TableFieldName<TTable>,
+      options?: MetadataFieldOptions<TRow>,
+    ) => FieldColumn<TRow>;
+    columns: (
+      dataKeys: TableFieldName<TTable>[],
+      options?: MetadataFieldsOptions,
+    ) => FieldColumn<TRow>[];
+  };
 } {
   return {
     field: (dataKey, options) => field<TRow>(dataKey, options),
     display: (id, options) => display<TRow>(id, options),
+    inferColumns: (tableMetadata) => inferColumnHelper<TRow>(tableMetadata),
   };
 }
 

package/src/component/types.ts

@@ -405,9 +405,11 @@ export interface UseCollectionOptions<
  * Methods that accept a field name are typed with `TFieldName` so that
  * auto-completion works when a concrete union is supplied.
  *
- * **Note:** Methods use *method syntax* (not property syntax) intentionally
+ * **Note:** Methods that accept `TFieldName` use *method syntax* intentionally
  * so that `UseCollectionReturn<"a" | "b">` remains assignable to
  * `UseCollectionReturn<string>` (bivariant method check).
+ * Methods that don't depend on `TFieldName` use property syntax so they
+ * can be safely destructured without triggering `unbound-method` lint rules.
  *
  * @typeParam TFieldName - Union of allowed field name strings (default: `string`).
  * @typeParam TQueryArgs - Type returned by `toQueryArgs()`. Contains both
@@ -428,7 +430,7 @@ export interface UseCollectionReturn<
    * const [result] = useQuery({ ...collection.toQueryArgs() });
    * ```
    */
-  toQueryArgs(): TQueryArgs;
+  toQueryArgs: () => TQueryArgs;
 
   // Filter operations
   /** Current active filters */
@@ -440,11 +442,11 @@ export interface UseCollectionReturn<
     value: unknown,
   ): void;
   /** Replace all filters at once */
-  setFilters(filters: Filter[]): void;
+  setFilters: (filters: Filter[]) => void;
   /** Remove filter for a specific field */
   removeFilter(field: TFieldName): void;
   /** Clear all filters */
-  clearFilters(): void;
+  clearFilters: () => void;
 
   // Sort operations
   /** Current sort states (supports multi-sort) */
@@ -456,7 +458,7 @@ export interface UseCollectionReturn<
     append?: boolean,
   ): void;
   /** Clear all sort states */
-  clearSort(): void;
+  clearSort: () => void;
 
   // Pagination operations
   /** Number of items per page */
@@ -466,17 +468,17 @@ export interface UseCollectionReturn<
   /** Current pagination direction */
   paginationDirection: "forward" | "backward";
   /** Navigate to next page using endCursor from pageInfo */
-  nextPage(endCursor: string): void;
+  nextPage: (endCursor: string) => void;
   /** Navigate to previous page using startCursor from pageInfo */
-  prevPage(startCursor: string): void;
+  prevPage: (startCursor: string) => void;
   /** Reset to first page */
-  resetPage(): void;
+  resetPage: () => void;
   /** Whether there is a previous page (from GraphQL pageInfo) */
   hasPrevPage: boolean;
   /** Whether there is a next page (from GraphQL pageInfo) */
   hasNextPage: boolean;
   /** Set pageInfo from graphql result to track hasPrevPage/hasNextPage */
-  setPageInfo(pageInfo: PageInfo): void;
+  setPageInfo: (pageInfo: PageInfo) => void;
 }
 
 // =============================================================================