@carlonicora/nextjs-jsonapi 6.32.2 → 6.33.0

package/src/features/billing/stripe-coupon/hooks/useStripeCouponCodeTableStructure.tsx

@@ -70,7 +70,7 @@ export const useStripeCouponCodeTableStructure: UseTableStructureHook<
         const code: StripeCouponCodeInterface = row.original.jsonApiData;
         return <span className="text-xs">{code.code}</span>;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripeCouponCodeFields.places]: () => ({
@@ -91,7 +91,8 @@ export const useStripeCouponCodeTableStructure: UseTableStructureHook<
           </span>
         );
       },
-      enableSorting: false,
+      meta: { sortFields: ["timesRedeemed"] },
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripeCouponCodeFields.expiresAt]: () => ({
@@ -106,7 +107,7 @@ export const useStripeCouponCodeTableStructure: UseTableStructureHook<
           </span>
         );
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripeCouponCodeFields.automatic]: () => ({
@@ -121,7 +122,7 @@ export const useStripeCouponCodeTableStructure: UseTableStructureHook<
           <span className="text-muted-foreground text-xs">—</span>
         );
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripeCouponCodeFields.status]: () => ({

package/src/features/billing/stripe-coupon/hooks/useStripeCouponTableStructure.tsx

@@ -37,7 +37,7 @@ export const useStripeCouponTableStructure: UseTableStructureHook<StripeCouponIn
         const coupon: StripeCouponInterface = row.original.jsonApiData;
         return <Link href={generateUrl({ page: STRIPE_COUPON_DETAIL_PAGE_URL, id: coupon.id })}>{coupon.name}</Link>;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripeCouponFields.discount]: () => ({
@@ -48,7 +48,8 @@ export const useStripeCouponTableStructure: UseTableStructureHook<StripeCouponIn
         const coupon: StripeCouponInterface = row.original.jsonApiData;
         return <span className="text-xs">{formatCouponDiscount(coupon, t, fmt)}</span>;
       },
-      enableSorting: false,
+      meta: { sortFields: ["discountType", "percentOff", "amountOff"] },
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripeCouponFields.places]: () => ({
@@ -70,7 +71,8 @@ export const useStripeCouponTableStructure: UseTableStructureHook<StripeCouponIn
           </span>
         );
       },
-      enableSorting: false,
+      meta: { sortFields: ["timesRedeemed", "maxRedemptions"] },
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripeCouponFields.codes]: () => ({
@@ -96,7 +98,8 @@ export const useStripeCouponTableStructure: UseTableStructureHook<StripeCouponIn
           <Badge variant="softGray">{t("billing.admin.coupons.status.ended")}</Badge>
         );
       },
-      enableSorting: false,
+      meta: { sortFields: ["valid"] },
+      enableSorting: true,
       enableHiding: false,
     }),
   };

package/src/features/billing/stripe-price/components/lists/PricesList.tsx

@@ -50,6 +50,8 @@ export function PricesList({ productId, fullWidth }: PricesListProps) {
     retriever: (params) => StripePriceService.listPrices({ ...params, productId }),
     retrieverParams: {},
     module: Modules.StripePrice,
+    // Display only (call C8): the server's own order when no sort is chosen.
+    defaultSort: "-createdAt",
   });
 
   useEffect(() => {

package/src/features/billing/stripe-price/data/stripe-price.service.ts

@@ -13,6 +13,8 @@ export class StripePriceService extends AbstractService {
     active?: boolean;
     next?: NextRef;
     prev?: PreviousRef;
+    /** JSON:API `sort` (`name`, `-name`), forwarded to the API. */
+    sort?: string;
   }): Promise<StripePriceInterface[]> {
     const endpoint = new EndpointCreator({
       endpoint: Modules.StripePrice,
@@ -25,6 +27,8 @@ export class StripePriceService extends AbstractService {
       endpoint.addAdditionalParam("active", params.active.toString());
     }
 
+    if (params?.sort) endpoint.addAdditionalParam("sort", params?.sort);
+
     return this.callApi({
       type: Modules.StripePrice,
       method: HttpMethod.GET,

package/src/features/billing/stripe-price/hooks/__tests__/useStripePriceTableStructure.sort.spec.tsx

@@ -0,0 +1,29 @@
+import { renderHook } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import { StripePriceFields } from "../../data/stripe-price.fields";
+import { useStripePriceTableStructure } from "../useStripePriceTableStructure";
+
+vi.mock("next-intl", () => ({ useTranslations: () => (key: string) => key, useLocale: () => "en" }));
+
+/** Spec row A22: nickname, amount, interval, token, status. Only an explicit `enableSorting: true` opts a column in (call C3). */
+describe("useStripePriceTableStructure sorting", () => {
+  it("opts in exactly the row's sortable columns", () => {
+    const { result } = renderHook(() =>
+      useStripePriceTableStructure({ data: [], fields: Object.values(StripePriceFields) as any }),
+    );
+    const columns = result.current.columns as any[];
+    const sortable = columns.filter((column) => column.enableSorting === true).map((column) => column.id);
+    expect(sortable.sort()).toEqual(["amount", "interval", "nickname", "status", "token"]);
+  });
+
+  it("maps the columns whose key differs from the stored property (call C4)", () => {
+    const { result } = renderHook(() =>
+      useStripePriceTableStructure({ data: [], fields: Object.values(StripePriceFields) as any }),
+    );
+    const columns = result.current.columns as any[];
+    const sortFields = Object.fromEntries(
+      columns.filter((column) => column.meta?.sortFields).map((column) => [column.id, column.meta.sortFields]),
+    );
+    expect(sortFields).toEqual({ amount: ["unitAmount"], interval: ["recurringInterval"], status: ["active"] });
+  });
+});

package/src/features/billing/stripe-price/hooks/useStripePriceTableStructure.tsx

@@ -61,7 +61,7 @@ export const useStripePriceTableStructure: UseTableStructureHook<StripePriceInte
           </Link>
         );
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripePriceFields.amount]: () => ({
@@ -77,7 +77,8 @@ export const useStripePriceTableStructure: UseTableStructureHook<StripePriceInte
           </span>
         );
       },
-      enableSorting: false,
+      meta: { sortFields: ["unitAmount"] },
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripePriceFields.interval]: () => ({
@@ -89,7 +90,8 @@ export const useStripePriceTableStructure: UseTableStructureHook<StripePriceInte
         // Typography role 8 (UI chrome): single-line, plain text-xs.
         return <span className="text-xs">{formatInterval(price, fmt)}</span>;
       },
-      enableSorting: false,
+      meta: { sortFields: ["recurringInterval"] },
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripePriceFields.token]: () => ({
@@ -100,7 +102,7 @@ export const useStripePriceTableStructure: UseTableStructureHook<StripePriceInte
         const price: StripePriceInterface = row.original.jsonApiData;
         return <span className="block text-xs tabular-nums text-end">{price.token ?? ""}</span>;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripePriceFields.status]: () => ({
@@ -123,7 +125,8 @@ export const useStripePriceTableStructure: UseTableStructureHook<StripePriceInte
           </span>
         );
       },
-      enableSorting: false,
+      meta: { sortFields: ["active"] },
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripePriceFields.actions]: () => ({

package/src/features/billing/stripe-product/components/lists/ProductsList.tsx

@@ -31,6 +31,8 @@ export function ProductsList({ fullWidth }: ProductsListProps = {}) {
     retriever: (params) => StripeProductService.listProducts(params),
     retrieverParams: {},
     module: Modules.StripeProduct,
+    // Display only (call C8): the server's own order when no sort is chosen.
+    defaultSort: "name",
   });
 
   // Archiving and restoring happen on the detail page, so the list has to learn

package/src/features/billing/stripe-product/data/stripe-product.service.ts

@@ -9,6 +9,8 @@ export class StripeProductService extends AbstractService {
     active?: boolean;
     next?: NextRef;
     prev?: PreviousRef;
+    /** JSON:API `sort` (`name`, `-name`), forwarded to the API. */
+    sort?: string;
   }): Promise<StripeProductInterface[]> {
     const endpoint = new EndpointCreator({
       endpoint: Modules.StripeProduct,
@@ -18,6 +20,8 @@ export class StripeProductService extends AbstractService {
       endpoint.addAdditionalParam("active", params.active.toString());
     }
 
+    if (params?.sort) endpoint.addAdditionalParam("sort", params?.sort);
+
     return this.callApi({
       type: Modules.StripeProduct,
       method: HttpMethod.GET,

package/src/features/billing/stripe-product/hooks/__tests__/useStripeProductTableStructure.sort.spec.tsx

@@ -0,0 +1,29 @@
+import { renderHook } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import { StripeProductFields } from "../../data/stripe-product.fields";
+import { useStripeProductTableStructure } from "../useStripeProductTableStructure";
+
+vi.mock("next-intl", () => ({ useTranslations: () => (key: string) => key, useLocale: () => "en" }));
+
+/** Spec row A23: name, status (active); prices (a count) is not sortable. Only an explicit `enableSorting: true` opts a column in (call C3). */
+describe("useStripeProductTableStructure sorting", () => {
+  it("opts in exactly the row's sortable columns", () => {
+    const { result } = renderHook(() =>
+      useStripeProductTableStructure({ data: [], fields: Object.values(StripeProductFields) as any }),
+    );
+    const columns = result.current.columns as any[];
+    const sortable = columns.filter((column) => column.enableSorting === true).map((column) => column.id);
+    expect(sortable.sort()).toEqual(["name", "status"]);
+  });
+
+  it("maps the columns whose key differs from the stored property (call C4)", () => {
+    const { result } = renderHook(() =>
+      useStripeProductTableStructure({ data: [], fields: Object.values(StripeProductFields) as any }),
+    );
+    const columns = result.current.columns as any[];
+    const sortFields = Object.fromEntries(
+      columns.filter((column) => column.meta?.sortFields).map((column) => [column.id, column.meta.sortFields]),
+    );
+    expect(sortFields).toEqual({ status: ["active"] });
+  });
+});

package/src/features/billing/stripe-product/hooks/useStripeProductTableStructure.tsx

@@ -35,7 +35,7 @@ export const useStripeProductTableStructure: UseTableStructureHook<StripeProduct
         const product: StripeProductInterface = row.original.jsonApiData;
         return <Link href={generateUrl({ page: Modules.StripeProduct, id: product.id })}>{product.name}</Link>;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripeProductFields.status]: () => ({
@@ -50,7 +50,8 @@ export const useStripeProductTableStructure: UseTableStructureHook<StripeProduct
           <Badge variant="softGray">{t("billing.admin.products.status.archived")}</Badge>
         );
       },
-      enableSorting: false,
+      meta: { sortFields: ["active"] },
+      enableSorting: true,
       enableHiding: false,
     }),
     [StripeProductFields.prices]: () => ({

package/src/features/blog/components/admin/BlogPostsList.tsx

@@ -24,6 +24,8 @@ export function BlogPostsList({ fullWidth }: BlogPostsListProps = {}) {
     retriever: (params) => BlogPostService.findMany(params),
     retrieverParams: {},
     module: Modules.BlogPost,
+    // Display only (call C8): the server's own order when no sort is chosen.
+    defaultSort: "-updatedAt",
   });
 
   // Creating navigates to the new post's page (EditorSheet's default without onSuccess).

package/src/features/blog/data/BlogPostService.ts

@@ -3,11 +3,13 @@ import { BLOG_ACTION } from "../types/blog.constants";
 import { BlogPostInput, BlogPostInterface } from "./BlogPostInterface";
 
 export class BlogPostService extends AbstractService {
-  static async findMany(params: { search?: string; next?: NextRef } = {}): Promise<BlogPostInterface[]> {
+  static async findMany(params: { search?: string; next?: NextRef; sort?: string } = {}): Promise<BlogPostInterface[]> {
     const endpoint = new EndpointCreator({ endpoint: Modules.BlogPost });
 
     if (params.search) endpoint.addAdditionalParam("search", params.search);
 
+    if (params.sort) endpoint.addAdditionalParam("sort", params.sort);
+
     return this.callApi<BlogPostInterface[]>({
       type: Modules.BlogPost,
       method: HttpMethod.GET,

package/src/features/blog/hooks/__tests__/useBlogPostTableStructure.sort.spec.tsx

@@ -0,0 +1,19 @@
+import { renderHook } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import { BlogPostFields } from "../../data";
+import { useBlogPostTableStructure } from "../useBlogPostTableStructure";
+
+vi.mock("next-intl", () => ({ useTranslations: () => (key: string) => key, useLocale: () => "en" }));
+vi.mock("../useBlogCategoryNames", () => ({ useBlogCategoryNames: () => () => "" }));
+
+/** Spec row A14: status, publishedAt. Only an explicit `enableSorting: true` opts a column in (call C3). */
+describe("useBlogPostTableStructure sorting", () => {
+  it("opts in exactly the row's sortable columns", () => {
+    const { result } = renderHook(() =>
+      useBlogPostTableStructure({ data: [], fields: Object.values(BlogPostFields) as any }),
+    );
+    const columns = result.current.columns as any[];
+    const sortable = columns.filter((column) => column.enableSorting === true).map((column) => column.id);
+    expect(sortable.sort()).toEqual(["publishedAt", "status"]);
+  });
+});

package/src/features/blog/hooks/useBlogPostTableStructure.tsx

@@ -62,7 +62,7 @@ export const useBlogPostTableStructure: UseTableStructureHook<BlogPostInterface,
         }[post.status];
         return <Badge variant={BLOG_STATUS_BADGE[post.status] ?? "softGray"}>{label ?? post.status}</Badge>;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [BlogPostFields.categories]: () => ({
@@ -87,7 +87,7 @@ export const useBlogPostTableStructure: UseTableStructureHook<BlogPostInterface,
         if (!post.publishedAt) return <span className="text-muted-foreground text-xs">—</span>;
         return <span className="text-xs tabular-nums">{dates.format(post.publishedAt, "dateMedium")}</span>;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [BlogPostFields.languages]: () => ({

package/src/features/company/components/lists/CompaniesList.tsx

@@ -26,6 +26,8 @@ export function CompaniesList({ fullWidth }: CompaniesListProps = {}) {
     retriever: (params) => CompanyService.findMany(params),
     retrieverParams: {},
     module: Modules.Company,
+    // Display only (call C8): the server's own order when no sort is chosen.
+    defaultSort: "name",
   });
 
   const functions: ReactNode[] = [<CompanyEditor key="create-account" />];

package/src/features/company/data/company.service.ts

@@ -11,11 +11,13 @@ export class CompanyService extends AbstractService {
     });
   }
 
-  static async findMany(params: { search?: string; next?: NextRef }): Promise<CompanyInterface[]> {
+  static async findMany(params: { search?: string; next?: NextRef; sort?: string }): Promise<CompanyInterface[]> {
     const endpoint = new EndpointCreator({ endpoint: Modules.Company });
 
     if (params.search) endpoint.addAdditionalParam("search", params.search);
 
+    if (params.sort) endpoint.addAdditionalParam("sort", params.sort);
+
     return this.callApi<CompanyInterface[]>({
       type: Modules.Company,
       method: HttpMethod.GET,

package/src/features/company/hooks/__tests__/useCompanyTableStructure.sort.spec.tsx

@@ -0,0 +1,19 @@
+import { renderHook } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import { CompanyFields } from "../../data";
+import { useCompanyTableStructure } from "../useCompanyTableStructure";
+
+vi.mock("next-intl", () => ({ useTranslations: () => (key: string) => key, useLocale: () => "en" }));
+vi.mock("../../../user/contexts", () => ({ useCurrentUserContext: () => ({ currentUser: undefined }) }));
+
+/** Spec row A24: name. Only an explicit `enableSorting: true` opts a column in (call C3). */
+describe("useCompanyTableStructure sorting", () => {
+  it("opts in exactly the row's sortable columns", () => {
+    const { result } = renderHook(() =>
+      useCompanyTableStructure({ data: [], fields: Object.values(CompanyFields) as any }),
+    );
+    const columns = result.current.columns as any[];
+    const sortable = columns.filter((column) => column.enableSorting === true).map((column) => column.id);
+    expect(sortable.sort()).toEqual(["name"]);
+  });
+});

package/src/features/company/hooks/useCompanyTableStructure.tsx

@@ -60,7 +60,7 @@ export const useCompanyTableStructure: UseTableStructureHook<CompanyInterface, C
           </Link>
         );
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [CompanyFields.createdAt]: () =>

package/src/features/content/data/content.service.ts

@@ -8,6 +8,8 @@ export class ContentService extends AbstractService {
     fetchAll?: boolean;
     next?: NextRef;
     prev?: PreviousRef;
+    /** JSON:API `sort` (`name`, `-name`), forwarded to the API. */
+    sort?: string;
   }): Promise<ContentInterface[]> {
     const endpoint = new EndpointCreator({ endpoint: Modules.Content });
 
@@ -21,6 +23,8 @@ export class ContentService extends AbstractService {
     if (Modules.Content.inclusions?.lists?.fields) endpoint.limitToFields(Modules.Content.inclusions.lists.fields);
     if (Modules.Content.inclusions?.lists?.types) endpoint.limitToType(Modules.Content.inclusions.lists.types);
 
+    if (params.sort) endpoint.addAdditionalParam("sort", params.sort);
+
     return this.callApi({
       type: Modules.Content,
       method: HttpMethod.GET,
@@ -29,7 +33,12 @@ export class ContentService extends AbstractService {
     });
   }
 
-  static async findRelevant(params: { id: string; next?: NextRef; prev?: PreviousRef }): Promise<ContentInterface[]> {
+  static async findRelevant(params: {
+    id: string;
+    next?: NextRef;
+    prev?: PreviousRef;
+    sort?: string;
+  }): Promise<ContentInterface[]> {
     const endpoint: EndpointCreator = new EndpointCreator({
       endpoint: Modules.Content,
       id: params.id,
@@ -39,6 +48,8 @@ export class ContentService extends AbstractService {
     if (Modules.Content.inclusions?.lists?.fields) endpoint.limitToFields(Modules.Content.inclusions.lists.fields);
     if (Modules.Content.inclusions?.lists?.types) endpoint.limitToType(Modules.Content.inclusions.lists.types);
 
+    if (params.sort) endpoint.addAdditionalParam("sort", params.sort);
+
     return this.callApi<ContentInterface[]>({
       type: Modules.Content,
       method: HttpMethod.GET,

package/src/features/content/hooks/__tests__/useContentTableStructure.sort.spec.tsx

@@ -0,0 +1,21 @@
+import { renderHook } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+// Loads the core barrel first, as the app does: the hook reaches it through the components barrel,
+// which on its own resolves a model module before its base class (circular import).
+import "../../../../core";
+import { ContentFields } from "../../data";
+import { useContentTableStructure } from "../useContentTableStructure";
+
+vi.mock("next-intl", () => ({ useTranslations: () => (key: string) => key, useLocale: () => "en" }));
+
+/** Spec rows A7, A10: name. Only an explicit `enableSorting: true` opts a column in (call C3). */
+describe("useContentTableStructure sorting", () => {
+  it("opts in exactly the row's sortable columns", () => {
+    const { result } = renderHook(() =>
+      useContentTableStructure({ data: [], fields: Object.values(ContentFields) as any }),
+    );
+    const columns = result.current.columns as any[];
+    const sortable = columns.filter((column) => column.enableSorting === true).map((column) => column.id);
+    expect(sortable.sort()).toEqual(["name"]);
+  });
+});

package/src/features/content/hooks/useContentTableStructure.tsx

@@ -58,7 +58,7 @@ export const useContentTableStructure = <U extends string = ContentFields>(
           </Tooltip>
         );
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [ContentFields.relevance]: () => ({

package/src/features/email-automation/components/lists/EmailAutomationRecipientsList.tsx

@@ -28,6 +28,8 @@ export function EmailAutomationRecipientsList() {
     retriever: (params) => EmailAutomationService.findEnrollments(params),
     retrieverParams: { automationId: automation?.id },
     module: Modules.EmailAutomationEnrollment,
+    // Display only (call C8): the server's own order when no sort is chosen.
+    defaultSort: "-enrolledAt",
   });
 
   const handleStatusChange = (value: string | null) => {

package/src/features/email-automation/components/lists/EmailAutomationsList.tsx

@@ -31,6 +31,8 @@ export function EmailAutomationsList({ fullWidth }: EmailAutomationsListProps =
     retriever: (params) => EmailAutomationService.findMany(params),
     retrieverParams: {},
     module: Modules.EmailAutomation,
+    // Display only (call C8): the server's own order when no sort is chosen.
+    defaultSort: "-updatedAt",
   });
 
   // The trigger column shows the label the app declares, not the stored key.

package/src/features/email-automation/data/EmailAutomationService.ts

@@ -8,7 +8,9 @@ import { EmailAutomationInput, EmailAutomationInterface } from "./EmailAutomatio
 import { EmailAutomationTriggerInterface } from "./EmailAutomationTriggerInterface";
 
 export class EmailAutomationService extends AbstractService {
-  static async findMany(params: { next?: NextRef; previous?: PreviousRef } = {}): Promise<EmailAutomationInterface[]> {
+  static async findMany(
+    params: { next?: NextRef; previous?: PreviousRef; sort?: string } = {},
+  ): Promise<EmailAutomationInterface[]> {
     const endpoint = new EndpointCreator({ endpoint: Modules.EmailAutomation });
 
     if (Modules.EmailAutomation.inclusions?.lists?.fields)
@@ -16,6 +18,8 @@ export class EmailAutomationService extends AbstractService {
     if (Modules.EmailAutomation.inclusions?.lists?.types)
       endpoint.limitToType(Modules.EmailAutomation.inclusions.lists.types);
 
+    if (params.sort) endpoint.addAdditionalParam("sort", params.sort);
+
     return this.callApi<EmailAutomationInterface[]>({
       type: Modules.EmailAutomation,
       method: HttpMethod.GET,
@@ -90,6 +94,8 @@ export class EmailAutomationService extends AbstractService {
     status?: string;
     next?: NextRef;
     previous?: PreviousRef;
+    /** JSON:API `sort` (`name`, `-name`), forwarded to the API. */
+    sort?: string;
   }): Promise<EmailAutomationEnrollmentInterface[]> {
     const endpoint = new EndpointCreator({
       endpoint: Modules.EmailAutomation,
@@ -99,6 +105,8 @@ export class EmailAutomationService extends AbstractService {
 
     if (params.status) endpoint.addAdditionalParam("status", params.status);
 
+    if (params.sort) endpoint.addAdditionalParam("sort", params.sort);
+
     return this.callApi<EmailAutomationEnrollmentInterface[]>({
       type: Modules.EmailAutomationEnrollment,
       method: HttpMethod.GET,

package/src/features/email-automation/hooks/__tests__/useEmailAutomationEnrollmentTableStructure.sort.spec.tsx

@@ -0,0 +1,21 @@
+import { renderHook } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import { EmailAutomationEnrollmentFields } from "../../data/email-automation-enrollment.fields";
+import { useEmailAutomationEnrollmentTableStructure } from "../useEmailAutomationEnrollmentTableStructure";
+
+vi.mock("next-intl", () => ({ useTranslations: () => (key: string) => key, useLocale: () => "en" }));
+
+/** Spec row A4: email, name, status, nextSendAt; the computed step and sent count are not sortable. Only an explicit `enableSorting: true` opts a column in (call C3). */
+describe("useEmailAutomationEnrollmentTableStructure sorting", () => {
+  it("opts in exactly the row's sortable columns", () => {
+    const { result } = renderHook(() =>
+      useEmailAutomationEnrollmentTableStructure({
+        data: [],
+        fields: Object.values(EmailAutomationEnrollmentFields) as any,
+      }),
+    );
+    const columns = result.current.columns as any[];
+    const sortable = columns.filter((column) => column.enableSorting === true).map((column) => column.id);
+    expect(sortable.sort()).toEqual(["email", "name", "nextSendAt", "status"]);
+  });
+});

package/src/features/email-automation/hooks/__tests__/useEmailAutomationTableStructure.sort.spec.tsx

@@ -0,0 +1,29 @@
+import { renderHook } from "@testing-library/react";
+import { describe, expect, it, vi } from "vitest";
+import { EmailAutomationFields } from "../../data/email-automation.fields";
+import { useEmailAutomationTableStructure } from "../useEmailAutomationTableStructure";
+
+vi.mock("next-intl", () => ({ useTranslations: () => (key: string) => key, useLocale: () => "en" }));
+
+/** Spec row A5: name, purpose, trigger, status (isActive); the counts stay unsortable. Only an explicit `enableSorting: true` opts a column in (call C3). */
+describe("useEmailAutomationTableStructure sorting", () => {
+  it("opts in exactly the row's sortable columns", () => {
+    const { result } = renderHook(() =>
+      useEmailAutomationTableStructure({ data: [], fields: Object.values(EmailAutomationFields) as any }),
+    );
+    const columns = result.current.columns as any[];
+    const sortable = columns.filter((column) => column.enableSorting === true).map((column) => column.id);
+    expect(sortable.sort()).toEqual(["name", "purpose", "status", "trigger"]);
+  });
+
+  it("maps the columns whose key differs from the stored property (call C4)", () => {
+    const { result } = renderHook(() =>
+      useEmailAutomationTableStructure({ data: [], fields: Object.values(EmailAutomationFields) as any }),
+    );
+    const columns = result.current.columns as any[];
+    const sortFields = Object.fromEntries(
+      columns.filter((column) => column.meta?.sortFields).map((column) => [column.id, column.meta.sortFields]),
+    );
+    expect(sortFields).toEqual({ status: ["isActive"] });
+  });
+});

package/src/features/email-automation/hooks/useEmailAutomationEnrollmentTableStructure.tsx

@@ -47,7 +47,7 @@ export const useEmailAutomationEnrollmentTableStructure: UseTableStructureHook<
         const enrollment: EmailAutomationEnrollmentInterface = row.original.jsonApiData;
         return <span className="text-sm">{enrollment.email}</span>;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [EmailAutomationEnrollmentFields.name]: () => ({
@@ -58,7 +58,7 @@ export const useEmailAutomationEnrollmentTableStructure: UseTableStructureHook<
         const enrollment: EmailAutomationEnrollmentInterface = row.original.jsonApiData;
         return enrollment.name ? <span className="text-sm">{enrollment.name}</span> : empty;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [EmailAutomationEnrollmentFields.status]: () => ({
@@ -76,7 +76,7 @@ export const useEmailAutomationEnrollmentTableStructure: UseTableStructureHook<
           </Badge>
         );
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [EmailAutomationEnrollmentFields.currentStep]: () => ({
@@ -103,7 +103,7 @@ export const useEmailAutomationEnrollmentTableStructure: UseTableStructureHook<
           </span>
         );
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [EmailAutomationEnrollmentFields.sentCount]: () => ({

package/src/features/email-automation/hooks/useEmailAutomationTableStructure.tsx

@@ -59,7 +59,7 @@ export const useEmailAutomationTableStructure: UseTableStructureHook<
         const automation: EmailAutomationInterface = row.original.jsonApiData;
         return <Link href={generateUrl({ page: Modules.EmailAutomation, id: automation.id })}>{automation.name}</Link>;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [EmailAutomationFields.purpose]: () => ({
@@ -78,7 +78,7 @@ export const useEmailAutomationTableStructure: UseTableStructureHook<
           </Badge>
         );
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [EmailAutomationFields.trigger]: () => ({
@@ -90,7 +90,7 @@ export const useEmailAutomationTableStructure: UseTableStructureHook<
         if (!automation.trigger) return <span className="text-muted-foreground text-xs">—</span>;
         return <span className="text-sm">{triggerLabels[automation.trigger] ?? automation.trigger}</span>;
       },
-      enableSorting: false,
+      enableSorting: true,
       enableHiding: false,
     }),
     [EmailAutomationFields.status]: () => ({
@@ -105,7 +105,8 @@ export const useEmailAutomationTableStructure: UseTableStructureHook<
           <Badge variant="softGray">{t("emailAutomations.list.status.off")}</Badge>
         );
       },
-      enableSorting: false,
+      meta: { sortFields: ["isActive"] },
+      enableSorting: true,
       enableHiding: false,
     }),
     [EmailAutomationFields.steps]: () =>