@object-ui/fields 17.5.0 → 17.6.0

package/src/widgets/RatingField.tsx

@@ -1,94 +0,0 @@
-import React from 'react';
-import { Star } from 'lucide-react';
-import { cn } from '@object-ui/components';
-import { FieldWidgetComponentProps } from './types';
-import { toDomProps } from './toDomProps';
-import { toHostGroupProps } from './toHostGroupProps';
-
-/**
- * Rating field widget - provides a star rating input
- * Supports numeric values from 0 to max (default 5)
- */
-export function RatingField({ value, onChange, field, readonly, className, error, ...props }: FieldWidgetComponentProps<number>) {
-  // Get rating-specific configuration from field metadata
-  const ratingField = field as any;
-  const max = ratingField?.max ?? 5;
-  const currentValue = value ?? 0;
-
-  const [hoverValue, setHoverValue] = React.useState<number | null>(null);
-
-  const displayValue = hoverValue !== null ? hoverValue : currentValue;
-
-  if (readonly) {
-    // The readonly star row is the field's whole rendered surface, so it — not
-    // only the editable row below — consumes the host's group label
-    // (objectui#3990). See `toHostGroupProps`.
-    return (
-      <div {...toHostGroupProps(props)} className={cn("flex items-center gap-1", className)}>
-        {Array.from({ length: max }, (_, i) => (
-          <Star
-            key={i}
-            className={`w-5 h-5 ${
-              i < currentValue
-                ? 'fill-yellow-400 text-yellow-400'
-                : 'text-muted-foreground'
-            }`}
-          />
-        ))}
-        <span className="ml-2 text-sm text-muted-foreground">
-          {currentValue} / {max}
-        </span>
-      </div>
-    );
-  }
-
-  // DOM pass-through (objectui#3318): the container carries the form
-  // renderer's id / aria-describedby, but NOT its `aria-invalid` — a plain
-  // wrapper div is not where assistive tech reads the invalid state. That
-  // state goes onto every focusable star button below, computed from the
-  // published `error` slot (#3222). `name` is withheld too: it is only
-  // DOM-legal on form controls, and on this div it is exactly the leak
-  // #3291 sweeps for.
-  const {
-    'aria-invalid': _hostAriaInvalid,
-    name: _domName,
-    ...groupDomProps
-  } = toDomProps(props);
-  // When the host named this container by IDREF (`aria-labelledby`,
-  // objectui#3961) it IS the labelled group of stars, so it answers with the
-  // matching role; a `label for` pointing at this `div` was inert. Standalone
-  // (no host label) the markup is unchanged.
-  const isLabelledGroup = groupDomProps['aria-labelledby'] != null;
-
-  return (
-    <div
-      {...groupDomProps}
-      role={isLabelledGroup ? 'group' : undefined}
-      className={cn("flex items-center gap-1", className)}
-    >
-      {Array.from({ length: max }, (_, i) => (
-        <button
-          key={i}
-          type="button"
-          onClick={() => onChange(i + 1)}
-          onMouseEnter={() => setHoverValue(i + 1)}
-          onMouseLeave={() => setHoverValue(null)}
-          className="focus:outline-none focus:ring-2 focus:ring-ring focus:ring-offset-2 rounded"
-          disabled={readonly || props.disabled}
-          aria-invalid={!!error}
-        >
-          <Star
-            className={`w-5 h-5 transition-colors ${
-              i < displayValue
-                ? 'fill-yellow-400 text-yellow-400 hover:fill-yellow-500 hover:text-yellow-500'
-                : 'text-muted-foreground hover:text-yellow-400'
-            }`}
-          />
-        </button>
-      ))}
-      <span className="ml-2 text-sm text-muted-foreground">
-        {currentValue} / {max}
-      </span>
-    </div>
-  );
-}

package/src/widgets/RecipientPickerField.test.tsx

@@ -1,157 +0,0 @@
-/**
- * ObjectUI
- * Copyright (c) 2024-present ObjectStack Inc.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-/**
- * RecipientPickerField — the sharing-rule recipient picker (objectstack#3821).
- *
- * Two regressions are pinned here:
- *  1. the candidate query must use the STRUCTURED `$orderby` form. It used to
- *     pass the clause string `'name asc'`, which ApiDataSource walked
- *     character by character into `0 n,1 a,2 m,…`; the server then sorted by
- *     columns that don't exist and every recipient list came back empty, so
- *     the control read "No matches" forever and no rule could be saved.
- *  2. the placeholder must come from a per-type i18n key, not from
- *     interpolating the enum value into an English sentence.
- */
-import { describe, it, expect, vi } from 'vitest';
-import React from 'react';
-import { render, screen, waitFor, fireEvent } from '@testing-library/react';
-import '@testing-library/jest-dom';
-import { RecipientPickerField } from './RecipientPickerField';
-
-const USERS = [
-  { id: 'usr_1', name: 'Ada Auditor' },
-  { id: 'usr_2', name: 'Bob Builder' },
-];
-
-function mockDataSource(rows: any[] = USERS) {
-  return { find: vi.fn().mockResolvedValue({ data: rows }) } as any;
-}
-
-function renderPicker(recipientType: string, dataSource: any, value = '') {
-  return render(
-    <RecipientPickerField
-      value={value}
-      onChange={vi.fn()}
-      field={{ name: 'recipient_id' } as any}
-      dataSource={dataSource}
-      dependentValues={{ recipient_type: recipientType }}
-    />,
-  );
-}
-
-describe('RecipientPickerField — candidate query', () => {
-  it('orders candidates with the structured $orderby form, never a clause string', async () => {
-    const ds = mockDataSource();
-    renderPicker('user', ds);
-
-    await waitFor(() => expect(ds.find).toHaveBeenCalled());
-    const [object, params] = ds.find.mock.calls[0];
-    expect(object).toBe('sys_user');
-    expect(params.$orderby).toEqual({ name: 'asc' });
-    // A bare string is what regressed; assert the shape explicitly.
-    expect(typeof params.$orderby).not.toBe('string');
-  });
-
-  it('queries the object mapped to each recipient type', async () => {
-    for (const [type, object] of [
-      ['user', 'sys_user'],
-      ['team', 'sys_team'],
-      ['business_unit', 'sys_business_unit'],
-      ['unit_and_subordinates', 'sys_business_unit'],
-      ['position', 'sys_position'],
-    ] as const) {
-      const ds = mockDataSource([]);
-      const { unmount } = renderPicker(type, ds);
-      await waitFor(() => expect(ds.find).toHaveBeenCalled());
-      expect(ds.find.mock.calls[0][0]).toBe(object);
-      unmount();
-    }
-  });
-
-  it('shows the loaded records as options once the list is opened', async () => {
-    const ds = mockDataSource();
-    renderPicker('user', ds);
-    await waitFor(() => expect(ds.find).toHaveBeenCalled());
-
-    fireEvent.click(screen.getByRole('combobox'));
-    expect(await screen.findByText('Ada Auditor')).toBeInTheDocument();
-    expect(screen.getByText('Bob Builder')).toBeInTheDocument();
-  });
-});
-
-describe('RecipientPickerField — keeping the stored recipient', () => {
-  function renderControlled(initialType: string, value: string, onChange: any) {
-    const ds = mockDataSource();
-    const view = render(
-      <RecipientPickerField
-        value={value}
-        onChange={onChange}
-        field={{ name: 'recipient_id' } as any}
-        dataSource={ds}
-        dependentValues={{ recipient_type: initialType }}
-      />,
-    );
-    const rerenderWith = (type: string) =>
-      view.rerender(
-        <RecipientPickerField
-          value={value}
-          onChange={onChange}
-          field={{ name: 'recipient_id' } as any}
-          dataSource={ds}
-          dependentValues={{ recipient_type: type }}
-        />,
-      );
-    return { ...view, rerenderWith };
-  }
-
-  it('keeps the saved recipient when the type arrives late (edit form hydrating)', async () => {
-    // The edit dialog mounts the widget before the record lands, so
-    // recipient_type is '' on the first render and 'user' on the next. That is
-    // hydration, not the admin changing the type — clearing there wiped the
-    // saved recipient and the blank got persisted on save.
-    const onChange = vi.fn();
-    const { rerenderWith } = renderControlled('', 'usr_1', onChange);
-    rerenderWith('user');
-    await waitFor(() => expect(onChange).not.toHaveBeenCalled());
-  });
-
-  it('clears the recipient when the admin actually switches type', async () => {
-    const onChange = vi.fn();
-    const { rerenderWith } = renderControlled('user', 'usr_1', onChange);
-    rerenderWith('team');
-    await waitFor(() => expect(onChange).toHaveBeenCalledWith(''));
-  });
-
-  it('does not clear when the type is unset back to empty', async () => {
-    const onChange = vi.fn();
-    const { rerenderWith } = renderControlled('user', 'usr_1', onChange);
-    rerenderWith('');
-    await waitFor(() => expect(onChange).not.toHaveBeenCalled());
-  });
-});
-
-describe('RecipientPickerField — localizable copy', () => {
-  it('takes the placeholder from a per-type key rather than the raw enum value', async () => {
-    // With no I18nProvider the widget falls back to the English defaults, so
-    // the assertion is on the *wording*: 'Select a business unit' can only come
-    // from the keyed string — interpolating the enum produced
-    // 'Select a business unit' only by accident and 'Select a unit and
-    // subordinates' for the next type over.
-    const ds = mockDataSource([]);
-    renderPicker('unit_and_subordinates', ds);
-    await waitFor(() => expect(ds.find).toHaveBeenCalled());
-    expect(await screen.findByText('Select a business unit')).toBeInTheDocument();
-    expect(screen.queryByText(/unit and subordinates/i)).not.toBeInTheDocument();
-  });
-
-  it('prompts for a recipient type before one is chosen', () => {
-    renderPicker('', mockDataSource([]));
-    expect(screen.getByText('Select a recipient type first.')).toBeInTheDocument();
-  });
-});

package/src/widgets/RecipientPickerField.tsx

@@ -1,188 +0,0 @@
-import React from 'react';
-import { Combobox, EmptyValue, cn } from '@object-ui/components';
-import { SchemaRendererContext } from '@object-ui/react';
-import type { FieldWidgetComponentProps } from './types';
-import { toDomProps } from './toDomProps';
-import { useFieldTranslation } from './useFieldTranslation';
-
-/**
- * RecipientPickerField — dependent record picker for a polymorphic recipient
- * reference (e.g. `sys_sharing_rule.recipient_id`), whose target object is
- * decided by a sibling `recipient_type` select.
- *
- * Reached via the field `widget: 'recipient-picker'` hint (resolves as
- * `field:recipient-picker`). Reads the live `recipient_type` from
- * `dependentValues` (the form record), loads candidate records of the mapped
- * object via `dataSource.find(...)`, and stores the value the sharing evaluator
- * expects for that type:
- *
- *   user                  → sys_user, store `id`
- *   team                  → sys_team, store `id`
- *   business_unit         → sys_business_unit, store `id`
- *   unit_and_subordinates → sys_business_unit, store `id`
- *   position              → sys_position, store `name` (matched against
- *                           sys_user_position.position at evaluation time)
- *
- * When `recipient_type` changes after mount the stored id is reset (an id valid
- * for one type is meaningless for another). Unknown types degrade to a plain
- * text input so nothing breaks.
- */
-interface RecipientMapping {
-  object: string;
-  /** Which record field to persist into recipient_id. */
-  storeField: 'id' | 'name';
-  /** Candidate display-label fields, in preference order. */
-  labelFields: string[];
-  /**
-   * i18n key for the "choose one" placeholder. Keyed per type rather than
-   * interpolating the enum value into an English sentence — "Select a
-   * business unit" and "选择业务单元" share no structure (objectstack#3821).
-   */
-  placeholderKey: string;
-}
-
-const TYPE_TO_OBJECT: Record<string, RecipientMapping> = {
-  user: { object: 'sys_user', storeField: 'id', labelFields: ['name', 'full_name', 'email'], placeholderKey: 'fields.recipient.selectUser' },
-  team: { object: 'sys_team', storeField: 'id', labelFields: ['name', 'label'], placeholderKey: 'fields.recipient.selectTeam' },
-  business_unit: { object: 'sys_business_unit', storeField: 'id', labelFields: ['name', 'label'], placeholderKey: 'fields.recipient.selectBusinessUnit' },
-  unit_and_subordinates: { object: 'sys_business_unit', storeField: 'id', labelFields: ['name', 'label'], placeholderKey: 'fields.recipient.selectUnitAndSubordinates' },
-  position: { object: 'sys_position', storeField: 'name', labelFields: ['label', 'name'], placeholderKey: 'fields.recipient.selectPosition' },
-};
-
-export function RecipientPickerField({
-  value,
-  onChange,
-  readonly,
-  className,
-  error,
-  ...props
-}: FieldWidgetComponentProps<string>) {
-  const ctx = React.useContext(SchemaRendererContext);
-  const { t } = useFieldTranslation();
-  const dataSource: any = props.dataSource ?? (ctx as any)?.dataSource ?? null;
-  const disabled = props.disabled;
-  const dependentValues: Record<string, any> = (props as any).dependentValues ?? {};
-  const recipientType = String(dependentValues.recipient_type ?? '');
-  const mapping = TYPE_TO_OBJECT[recipientType];
-
-  const [records, setRecords] = React.useState<any[] | null>(null);
-
-  // Reset the stored recipient when the admin PICKS a different type (an id for
-  // a user is not a valid team/business-unit id).
-  //
-  // "Different type" means one non-empty type replacing another. The empty
-  // string is not a type — it is the edit form before the record has hydrated,
-  // and treating `'' → 'user'` as a change wiped the saved recipient the moment
-  // an existing rule was opened for editing (objectstack#3821): the picker went
-  // blank, and saving persisted the blank. Only a non-empty predecessor can
-  // invalidate the stored id.
-  const prevType = React.useRef<string>('');
-  React.useEffect(() => {
-    if (prevType.current && recipientType && prevType.current !== recipientType && value) {
-      onChange('' as any);
-    }
-    prevType.current = recipientType;
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [recipientType]);
-
-  React.useEffect(() => {
-    setRecords(null);
-    if (!dataSource || !mapping || typeof dataSource.find !== 'function') return;
-    let cancelled = false;
-    (async () => {
-      try {
-        // Object form, not the 'name asc' clause string: the clause string is
-        // supported again (objectstack#3821 fixed ApiDataSource walking it
-        // character by character), but the structured form can't regress that
-        // way for any data source.
-        const res = await dataSource.find(mapping.object, { $top: 500, $orderby: { name: 'asc' } });
-        const list: any[] = res?.data ?? res?.records ?? (Array.isArray(res) ? res : []);
-        if (!cancelled) setRecords(Array.isArray(list) ? list : []);
-      } catch {
-        if (!cancelled) setRecords([]);
-      }
-    })();
-    return () => {
-      cancelled = true;
-    };
-  }, [dataSource, mapping?.object]);
-
-  const labelOf = (r: any): string => {
-    for (const f of mapping?.labelFields ?? ['name']) if (r?.[f]) return String(r[f]);
-    return String(r?.id ?? '');
-  };
-  const valueOf = (r: any): string => String(r?.[mapping?.storeField ?? 'id'] ?? '');
-
-  const options = React.useMemo(() => {
-    const opts = (records ?? [])
-      .map((r) => ({ value: valueOf(r), label: labelOf(r) }))
-      .filter((o) => o.value);
-    if (value && !opts.some((o) => o.value === value)) opts.unshift({ value, label: value });
-    return opts;
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [records, value, mapping]);
-
-  if (!recipientType) {
-    return (
-      <p className={cn('text-sm text-muted-foreground', className)}>
-        {t('fields.recipient.selectTypeFirst')}
-      </p>
-    );
-  }
-
-  if (!mapping) {
-    // Unknown / unsupported recipient type — keep a plain text input so the
-    // field is never un-editable.
-    return (
-      <input
-        // DOM pass-through onto the real focusable control (objectui#3318).
-        {...toDomProps(props)}
-        className={cn(
-          'w-full rounded border bg-background px-2 py-1 text-sm',
-          className,
-        )}
-        value={value ?? ''}
-        disabled={disabled || readonly}
-        onChange={(e) => onChange(e.target.value as any)}
-        aria-invalid={!!error}
-      />
-    );
-  }
-
-  if (readonly) {
-    if (!value) return <EmptyValue />;
-    return <span className={className}>{options.find((o) => o.value === value)?.label ?? value}</span>;
-  }
-
-  // DOM pass-through onto the combobox trigger — the widget's real focusable
-  // control (objectui#3318). `name` is withheld: the trigger is a button, not
-  // a submission control (same reasoning as #3306's SelectTrigger).
-  //
-  // NOTE this widget stays on the #3318 ledger regardless: its dependency-
-  // gated state (no `recipient_type` chosen yet — the state a fresh form and
-  // the registry sweep render) is a plain hint paragraph with no focusable
-  // control, so there is nothing there to carry the attribute.
-  const { name: _domName, ...triggerDomProps } = toDomProps(props);
-
-  return (
-    <Combobox
-      {...triggerDomProps}
-      options={options}
-      value={value ?? ''}
-      onValueChange={(v) => onChange(v as any)}
-      placeholder={
-        records === null
-          ? t('fields.recipient.loading')
-          : t(mapping.placeholderKey ?? 'fields.recipient.select')
-      }
-      searchPlaceholder={t('fields.recipient.search')}
-      emptyText={records === null ? t('fields.recipient.loading') : t('fields.recipient.empty')}
-      disabled={disabled}
-      className={cn('w-full', className)}
-      // AFTER the spread so this widget's own computation wins (#3222).
-      aria-invalid={!!error}
-    />
-  );
-}
-
-export default RecipientPickerField;

package/src/widgets/RecordPickerDialog.baseFilterShapes.test.tsx

@@ -1,191 +0,0 @@
-/**
- * ObjectUI
- * Copyright (c) 2024-present ObjectStack Inc.
- *
- * This source code is licensed under the MIT license found in the
- * LICENSE file in the root directory of this source tree.
- */
-
-/**
- * `RecordPickerDialog.baseFilter` accepts TWO shapes — objectui#3831.
- *
- * The slot was declared `Record< string, any >` and merged by object spread,
- * which serves the dependent-lookup chain (#2215) exactly right and cannot
- * carry a spec `ViewFilterRule[]` at all: TypeScript accepts an array there
- * (an array satisfies a string index of `any`), the spread flattens it to
- * `{"0": rule, "1": rule}`, and the query then filters on columns literally
- * named `0`/`1` — green types, wrong query, no diagnostic. That is the slot
- * `record:related_list.add.picker.filter` has to reach.
- *
- * So the merge now discriminates on shape, and both directions are pinned here:
- *
- *  - **record form** keeps KEY-OVERWRITE precedence, byte for byte. A cascaded
- *    parent value must REPLACE a stale same-field `lookupFilters` entry, not
- *    intersect with it — `account = 'stale' AND account = 'a1'` returns nothing.
- *    (`LookupField.dependsOn.test.tsx` owns the #2215 behaviour end-to-end and
- *    is deliberately left untouched; this file pins the same precedence at the
- *    dialog's own boundary, so a future merge change cannot pass by only
- *    satisfying the array path.)
- *  - **rule array** lowers through `mergeFilterNodes`, the repo's single filter
- *    sink, so all 19 `VIEW_FILTER_OPERATORS` survive — including the four
- *    (`before`, `after`, `is_empty`, `is_not_empty`) the MongoDB-style record
- *    form has no `$op` for and which a hand-rolled second lowering would have
- *    had to drop or throw on.
- */
-
-import { render, waitFor } from '@testing-library/react';
-import { describe, it, expect, vi } from 'vitest';
-import { RecordPickerDialog } from './RecordPickerDialog';
-
-const records = [{ id: 'r1', name: 'One' }];
-
-function makeDataSource() {
-  const find = vi.fn(async () => ({ data: records, total: records.length }));
-  return { find } as any;
-}
-
-/** The `$filter` of the most recent query. */
-function lastFilter(ds: any) {
-  const calls = ds.find.mock.calls;
-  return calls[calls.length - 1][1]?.$filter;
-}
-
-describe('RecordPickerDialog baseFilter — record form (#2215 precedence)', () => {
-  it('spreads a record baseFilter last, overwriting lookupFilters on the same field', async () => {
-    const ds = makeDataSource();
-    render(
-      <RecordPickerDialog
-        open
-        onOpenChange={vi.fn()}
-        dataSource={ds}
-        objectName="contacts"
-        onSelect={vi.fn()}
-        lookupFilters={[{ field: 'account', operator: 'eq', value: 'stale' }]}
-        baseFilter={{ account: 'a1' }}
-      />,
-    );
-
-    await waitFor(() => expect(ds.find).toHaveBeenCalled());
-    // Still the plain record form — NOT lowered to an AST, and emphatically not
-    // an `and` of the stale value with the cascaded one.
-    expect(lastFilter(ds)).toEqual({ account: 'a1' });
-    expect(Array.isArray(lastFilter(ds))).toBe(false);
-  });
-
-  it('sends no $filter at all when nothing is constrained', async () => {
-    const ds = makeDataSource();
-    render(
-      <RecordPickerDialog
-        open
-        onOpenChange={vi.fn()}
-        dataSource={ds}
-        objectName="contacts"
-        onSelect={vi.fn()}
-      />,
-    );
-
-    await waitFor(() => expect(ds.find).toHaveBeenCalled());
-    expect(lastFilter(ds)).toBeUndefined();
-  });
-});
-
-describe('RecordPickerDialog baseFilter — spec ViewFilterRule[] (#3831)', () => {
-  it('lowers a rule array to AST nodes instead of spreading it into index keys', async () => {
-    const ds = makeDataSource();
-    render(
-      <RecordPickerDialog
-        open
-        onOpenChange={vi.fn()}
-        dataSource={ds}
-        objectName="sys_position"
-        onSelect={vi.fn()}
-        baseFilter={[{ field: 'is_active', operator: 'equals', value: true }]}
-      />,
-    );
-
-    await waitFor(() => expect(ds.find).toHaveBeenCalled());
-    expect(lastFilter(ds)).toEqual([['is_active', 'equals', true]]);
-    // The old object-spread failure mode, pinned by structure rather than by
-    // key: the spread produced a PLAIN OBJECT whose values were still rule
-    // objects (`{"0": {field, operator, value}}`), so the query asked for a
-    // column named `0`. Asserting on `Object.keys` cannot tell the two apart —
-    // an array's keys are its indices too — so what is pinned is that each
-    // element is an AST TRIPLE and no longer carries a `field` member.
-    const filter = lastFilter(ds) as unknown[];
-    expect(Array.isArray(filter)).toBe(true);
-    expect(Array.isArray(filter[0])).toBe(true);
-    expect(filter[0]).not.toHaveProperty('field');
-  });
-
-  it('preserves the four operators the record form cannot spell', async () => {
-    const ds = makeDataSource();
-    render(
-      <RecordPickerDialog
-        open
-        onOpenChange={vi.fn()}
-        dataSource={ds}
-        objectName="sys_license"
-        onSelect={vi.fn()}
-        baseFilter={[
-          { field: 'starts_at', operator: 'before', value: '2026-01-01' },
-          { field: 'expires_at', operator: 'after', value: '2026-01-01' },
-          { field: 'revoked_at', operator: 'is_empty' },
-          { field: 'assigned_to', operator: 'is_not_empty' },
-        ]}
-      />,
-    );
-
-    await waitFor(() => expect(ds.find).toHaveBeenCalled());
-    expect(lastFilter(ds)).toEqual([
-      ['starts_at', 'before', '2026-01-01'],
-      ['expires_at', 'after', '2026-01-01'],
-      // A rule with no `value` stays two-element: inventing `null` here would be
-      // a real `{revoked_at: null}` predicate, i.e. a different question.
-      ['revoked_at', 'is_empty'],
-      ['assigned_to', 'is_not_empty'],
-    ]);
-  });
-
-  it('keeps a record baseFilter and a rule array as separate `and` children', async () => {
-    const ds = makeDataSource();
-    render(
-      <RecordPickerDialog
-        open
-        onOpenChange={vi.fn()}
-        dataSource={ds}
-        objectName="sys_position"
-        onSelect={vi.fn()}
-        // A rule array is the author's restriction; the record-form base still
-        // arrives via `lookupFilters` here, and the two must BOTH apply.
-        lookupFilters={[{ field: 'company', operator: 'eq', value: 'c1' }]}
-        baseFilter={[{ field: 'is_active', operator: 'equals', value: true }]}
-      />,
-    );
-
-    await waitFor(() => expect(ds.find).toHaveBeenCalled());
-    expect(lastFilter(ds)).toEqual([
-      'and',
-      ['company', '=', 'c1'],
-      [['is_active', 'equals', true]],
-    ]);
-  });
-
-  it('sends no $filter for an empty rule array', async () => {
-    const ds = makeDataSource();
-    render(
-      <RecordPickerDialog
-        open
-        onOpenChange={vi.fn()}
-        dataSource={ds}
-        objectName="sys_position"
-        onSelect={vi.fn()}
-        baseFilter={[]}
-      />,
-    );
-
-    await waitFor(() => expect(ds.find).toHaveBeenCalled());
-    // An empty restriction is no restriction — not an empty `$filter` the wire
-    // has to interpret.
-    expect(lastFilter(ds)).toBeUndefined();
-  });
-});