@startsimpli/ui 0.4.158 → 0.4.159

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@startsimpli/ui",
-  "version": "0.4.158",
+  "version": "0.4.159",
   "description": "Shared UI components package for StartSimpli applications",
   "main": "./src/index.ts",
   "types": "./src/index.ts",
@@ -55,8 +55,8 @@
     "@codemirror/language": "^6.0.0",
     "@codemirror/state": "^6.0.0",
     "@codemirror/view": "^6.0.0",
-    "@startsimpli/i18n": "^0.1.3",
     "@startsimpli/api": "^0.5.68",
+    "@startsimpli/i18n": "^0.1.3",
     "@startsimpli/auth": "^0.4.42"
   },
   "peerDependenciesMeta": {

package/src/components/collection/InlineReviewActions.tsx

@@ -64,9 +64,23 @@ export interface InlineReviewActionsProps {
   config?: ReviewConfig;
   /** Fired after a successful save so the parent can refetch/invalidate. */
   onSaved?: () => void;
+  /**
+   * Whether this reader may change THIS record (bd startsim-m7fdm.32). False
+   * draws the record's state as a plain pill and no decisions, the same pill
+   * ReviewDrawer shows in place of its State select, so a view-only reader is
+   * not offered buttons that can only end in "Could not save.". Defaults to true.
+   */
+  canEdit?: boolean;
 }
 
-export function InlineReviewActions({ client, type, record, config, onSaved }: InlineReviewActionsProps) {
+export function InlineReviewActions({
+  client,
+  type,
+  record,
+  config,
+  onSaved,
+  canEdit = true,
+}: InlineReviewActionsProps) {
   const cfg = useMemo(() => resolveReviewConfig(type, config), [type, config]);
   /** The type's declared attribute names — the spellings a write must send back. */
   const declaredNames = useMemo(() => (type.attributes ?? []).map((a) => a.name), [type]);
@@ -152,6 +166,14 @@ export function InlineReviewActions({ client, type, record, config, onSaved }: I
   }
 
   if (decisions.length === 0) return null;
+  if (!canEdit) {
+    const status = fieldStr(record.data, cfg.statusName);
+    return status ? (
+      <span className="whitespace-nowrap rounded-full bg-muted px-2 py-0.5 text-xs capitalize text-muted-foreground">
+        {status.replace(/_/g, ' ')}
+      </span>
+    ) : null;
+  }
   return (
     <div className="flex items-center justify-end gap-1" onClick={(e) => e.stopPropagation()}>
       {decisions.map((d) =>

package/src/components/collection/__tests__/inline-review-reader.test.tsx

@@ -0,0 +1,77 @@
+/**
+ * The table-row and board-card decision cluster follows the record's
+ * permissions, like the drawer (bd startsim-m7fdm.32; drawer half startsim-j9agp).
+ *
+ * Before this, InlineReviewActions drew Reject / Approve / Needs work for every
+ * reader, and a view-only reader learned from "Could not save." that they could
+ * not decide. With `canEdit={false}` it draws the record's state instead, the
+ * same pill the drawer shows in place of its State select.
+ */
+import * as React from 'react';
+import { render, screen } from '@testing-library/react';
+
+import { InlineReviewActions } from '../InlineReviewActions';
+import type { CollectionClient, EntityRecord, EntityTypeDef, Paginated } from '../types';
+
+const TOPIC_TYPE: EntityTypeDef = {
+  id: 1,
+  key: 'topic',
+  label: 'Topic',
+  attributes: [
+    {
+      id: 'status',
+      name: 'status',
+      dataType: 'enum',
+      required: false,
+      config: { choices: ['suggested', 'ready', 'written', 'rejected'] },
+    },
+    { id: 'title', name: 'title', dataType: 'text', required: false, config: {} },
+    { id: 'team_verdict', name: 'team_verdict', dataType: 'text', required: false, config: {} },
+  ],
+};
+
+function topic(status: string): EntityRecord {
+  return {
+    id: 1,
+    entityType: 'topic',
+    externalId: null,
+    name: 'Oman customs',
+    data: { title: 'Oman customs', status },
+    createdAt: '2026-09-01T00:00:00Z',
+  };
+}
+
+function page<T>(results: T[]): Paginated<T> {
+  return { count: results.length, next: null, previous: null, results };
+}
+
+function stubClient(): CollectionClient {
+  return {
+    listTypes: jest.fn(async () => page([TOPIC_TYPE])),
+    listAllEntities: jest.fn(async () => []),
+    updateEntity: jest.fn(async () => topic('ready')),
+  } as unknown as CollectionClient;
+}
+
+describe('InlineReviewActions for a view-only reader', () => {
+  it('draws the state and no decision buttons', () => {
+    render(
+      <InlineReviewActions client={stubClient()} type={TOPIC_TYPE} record={topic('ready_for_review')} canEdit={false} />,
+    );
+    expect(screen.queryByRole('button')).toBeNull();
+    expect(screen.getByText('ready for review')).toBeInTheDocument();
+  });
+
+  it('draws nothing when the record has no state', () => {
+    const record = { ...topic('x'), data: { title: 'Oman customs' } };
+    const { container } = render(
+      <InlineReviewActions client={stubClient()} type={TOPIC_TYPE} record={record} canEdit={false} />,
+    );
+    expect(container).toBeEmptyDOMElement();
+  });
+
+  it('keeps all three decisions for an editor (the default)', () => {
+    render(<InlineReviewActions client={stubClient()} type={TOPIC_TYPE} record={topic('suggested')} />);
+    expect(screen.getAllByRole('button')).toHaveLength(3);
+  });
+});

package/src/components/unified-table/__tests__/components/TableFilters.option-labels.test.tsx

@@ -0,0 +1,52 @@
+/**
+ * A chip filter can show a label while it filters on a key (review bead
+ * startsim-y78xg). The marketing-agents Kind chips read "weekly_brief" and
+ * "lead_magnet" beside a sidebar saying "Weekly Briefs" and "Evergreen"; the
+ * key has to stay the value because it is what travels in the URL.
+ */
+import * as React from 'react'
+import { fireEvent, render, screen } from '@testing-library/react'
+
+import { TableFilters } from '../../components/TableFilters'
+import type { FilterConfig, UseFiltersReturn } from '../../types'
+
+const config: FilterConfig = {
+  sections: [
+    {
+      id: 'kind',
+      type: 'chips',
+      filters: [
+        {
+          id: 'content_type',
+          label: 'Kind',
+          options: ['weekly_brief', 'lead_magnet', 'general'],
+          optionLabels: { weekly_brief: 'Weekly Briefs', lead_magnet: 'Evergreen' },
+        },
+      ],
+    },
+  ],
+}
+
+function filtersStub(value: Record<string, string> = {}): UseFiltersReturn {
+  return {
+    filters: value,
+    setFilter: jest.fn(),
+    clearFilter: jest.fn(),
+    clearAllFilters: jest.fn(),
+    hasActiveFilters: () => Object.keys(value).length > 0,
+    getActiveFilterCount: () => Object.keys(value).length,
+  }
+}
+
+describe('TableFilters chip labels', () => {
+  it('shows the label and filters on the key', () => {
+    const filters = filtersStub()
+    render(<TableFilters config={config} filters={filters} defaultExpanded />)
+
+    expect(screen.queryByRole('button', { name: 'weekly_brief' })).toBeNull()
+    fireEvent.click(screen.getByRole('button', { name: 'Evergreen' }))
+    expect(filters.setFilter).toHaveBeenCalledWith('content_type', 'lead_magnet')
+    // An option with no label still reads as itself.
+    expect(screen.getByRole('button', { name: 'general' })).toBeInTheDocument()
+  })
+})

package/src/components/unified-table/components/TableFilters/index.tsx

@@ -64,7 +64,7 @@ export function TableFilters({ config, filters, className, defaultExpanded = fal
                       }}
                       className="h-7 px-2 text-xs"
                     >
-                      {option}
+                      {filter.optionLabels?.[option] ?? option}
                     </Button>
                   )
                 })}
@@ -122,7 +122,7 @@ export function TableFilters({ config, filters, className, defaultExpanded = fal
             <SelectItem value="all">All</SelectItem>
             {filteredOptions.map((option: string) => (
               <SelectItem key={option} value={option}>
-                {option === 'null' ? 'Not Set' : option}
+                {option === 'null' ? 'Not Set' : (filter.optionLabels?.[option] ?? option)}
               </SelectItem>
             ))}
           </SelectContent>

package/src/components/unified-table/types.ts

@@ -164,6 +164,9 @@ export interface FilterItem {
   label: string
   type?: 'chips' | 'dropdown' | 'search' | 'checkbox'
   options?: string[]
+  /** What each option reads as on screen, keyed by option. The option itself
+   *  stays the value the filter holds, so a URL or API key is never a label. */
+  optionLabels?: Record<string, string>
   multiple?: boolean
 }