@object-ui/fields 17.5.0 → 17.6.0

package/src/widgets/AddressField.tsx

@@ -1,182 +0,0 @@
-import React, { useId } from 'react';
-import { Input, Label, EmptyValue } from '@object-ui/components';
-import { FieldWidgetComponentProps } from './types';
-import { toDomProps } from './toDomProps';
-import { toHostGroupProps } from './toHostGroupProps';
-// The value shape and the single-line formatting rule now live in a pure module
-// so the display (read) cell renderer formats a stored address exactly the way
-// this widget's readonly branch does — one rule, not two copies that drift
-// (objectui#4037). Behaviour here is unchanged; only the definition moved.
-import {
-  formatAddress,
-  readPostalCode,
-  type AddressValue,
-  type LegacyAddressValue,
-} from './address-format';
-
-// Re-exported through its declaring module rather than bare, so the spec-symbol
-// guard resolves the name to where it is actually defined: `address-format`
-// imports `AddressValue` from `@objectstack/spec/data` (objectui#4167), and a
-// bare `export type { AddressValue }` here would read to that guard as a second,
-// local declaration of a name the spec owns.
-export type { AddressValue } from './address-format';
-
-/**
- * Address field widget - provides a structured address input
- * Supports street, city, state, postal code, and country
- */
-export function AddressField({ value, onChange, field, readonly, error, ...props }: FieldWidgetComponentProps<AddressValue>) {
-  const address = value || {};
-  // DOM pass-through (objectui#3318): the whitelist spread goes onto the FIRST
-  // sub-input (street) — one carrier for the form renderer's aria-describedby;
-  // the composite's validation state goes onto EVERY focusable sub-input via
-  // `aria-invalid={!!error}` (a form-level failure means the whole address is
-  // missing/invalid, and whichever sub-input the user reaches must announce it).
-  //
-  // Two keys are held back from that spread and land on the group CONTAINER
-  // instead (objectui#3961), because they address the field as a WHOLE:
-  //
-  //  - `id` — the host's control id. It never reached the DOM at all: it was
-  //    spread here and then overwritten one line later by `id={subId('street')}`
-  //    (objectui#3343), which is why the form's "Shipping Address" label pointed
-  //    at an id no element carried. The sub-input ids stay exactly as they are;
-  //    only the host id moves out to the element it was always meant to name.
-  //  - `aria-labelledby` — the host's label, associated by IDREF. On the street
-  //    input it would be actively wrong: `aria-labelledby` OVERRIDES a control's
-  //    `<label for>`, so the first sub-input would be announced as "Shipping
-  //    Address" and its own "Street Address" label would vanish.
-  //
-  // Everything else (`aria-describedby`, `name`, focus handlers …) deliberately
-  // stays on the first sub-input: a description or error must be announced when
-  // focus lands on something focusable, and a container is not.
-  const {
-    id: _hostId,
-    'aria-labelledby': _hostLabelledBy,
-    ...domProps
-  } = toDomProps(props);
-  // The pair held back above, in the one spelling every group-labelled widget
-  // uses for it — and computed HERE, before the readonly early return below,
-  // because that return used to drop both keys on the floor: a published label
-  // id with no consumer in the document (objectui#3990). See `toHostGroupProps`.
-  const hostGroupProps = toHostGroupProps(props);
-  // Sub-input ids (objectui#3343): `useId()` prefix + sub-field name — the
-  // `groupId` paradigm of RadioField / CheckboxesField. Hardcoded literals
-  // ("street", "city", …) collide as soon as a form renders two address
-  // fields, and every label's htmlFor then resolves to the FIRST match.
-  const groupId = useId();
-  const subId = (name: keyof AddressValue) => `${groupId}-${name}`;
-
-  // Read through the legacy key, write only the canonical one (objectstack#5143).
-  const postalCode = readPostalCode(address);
-
-  const handleFieldChange = (fieldName: keyof AddressValue, fieldValue: string) => {
-    // Normalize while we are here: the object we write back never carries
-    // `zipCode`, and a legacy postal code is carried forward under
-    // `postalCode`. Without the carry, editing an unrelated part of a
-    // legacy-shaped address would write the postal code straight back out of
-    // the record — the very data loss this fixes, one build later.
-    const canonical: LegacyAddressValue = { ...address };
-    delete canonical.zipCode;
-    onChange({
-      ...canonical,
-      ...(postalCode ? { postalCode } : null),
-      [fieldName]: fieldValue,
-    });
-  };
-
-  if (readonly) {
-    // Readonly the composite collapses to ONE formatted line — no sub-inputs and
-    // no sub-labels — so that line is the surface the host label names, and
-    // `EmptyValue` is the same surface with nothing in it (objectui#3990). The
-    // placeholder's own `aria-label` ("No value") is outranked by
-    // `aria-labelledby` per accname, and on its `generic` role an author name was
-    // never exposed anyway.
-    const formatted = formatAddress(address);
-    return formatted ? (
-      <span {...hostGroupProps} className="text-sm">{formatted}</span>
-    ) : (
-      <EmptyValue {...hostGroupProps} />
-    );
-  }
-
-  return (
-    // `role="group"` only when a host actually named this container
-    // (objectui#3961): an unnamed group adds nothing for assistive tech, and
-    // standalone rendering — the inline grid editor, a bare SDUI node — must stay
-    // byte-identical to what it was before. That condition now lives in
-    // `toHostGroupProps` so this container and the readonly line above cannot
-    // answer differently (objectui#3990).
-    <div className="space-y-3" {...hostGroupProps}>
-      <div>
-        <Label htmlFor={subId('street')} className="text-xs">Street Address</Label>
-        <Input
-          {...domProps}
-          id={subId('street')}
-          type="text"
-          value={address.street || ''}
-          onChange={(e) => handleFieldChange('street', e.target.value)}
-          placeholder="123 Main St"
-          disabled={readonly || props.disabled}
-          className={props.className}
-          aria-invalid={!!error}
-        />
-      </div>
-      
-      <div className="grid grid-cols-2 gap-3">
-        <div>
-          <Label htmlFor={subId('city')} className="text-xs">City</Label>
-          <Input
-            id={subId('city')}
-            type="text"
-            value={address.city || ''}
-            onChange={(e) => handleFieldChange('city', e.target.value)}
-            placeholder="San Francisco"
-            disabled={readonly || props.disabled}
-            aria-invalid={!!error}
-          />
-        </div>
-        
-        <div>
-          <Label htmlFor={subId('state')} className="text-xs">State / Province</Label>
-          <Input
-            id={subId('state')}
-            type="text"
-            value={address.state || ''}
-            onChange={(e) => handleFieldChange('state', e.target.value)}
-            placeholder="CA"
-            disabled={readonly || props.disabled}
-            aria-invalid={!!error}
-          />
-        </div>
-      </div>
-      
-      <div className="grid grid-cols-2 gap-3">
-        <div>
-          <Label htmlFor={subId('postalCode')} className="text-xs">ZIP / Postal Code</Label>
-          <Input
-            id={subId('postalCode')}
-            type="text"
-            value={postalCode || ''}
-            onChange={(e) => handleFieldChange('postalCode', e.target.value)}
-            placeholder="94102"
-            disabled={readonly || props.disabled}
-            aria-invalid={!!error}
-          />
-        </div>
-        
-        <div>
-          <Label htmlFor={subId('country')} className="text-xs">Country</Label>
-          <Input
-            id={subId('country')}
-            type="text"
-            value={address.country || ''}
-            onChange={(e) => handleFieldChange('country', e.target.value)}
-            placeholder="United States"
-            disabled={readonly || props.disabled}
-            aria-invalid={!!error}
-          />
-        </div>
-      </div>
-    </div>
-  );
-}

package/src/widgets/AddressField.uniqueIds.test.tsx

@@ -1,93 +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.
- */
-
-/**
- * Unique sub-input ids for the composite `address` widget (objectui#3343).
- *
- * The widget used to hardcode literal ids ("street", "city", "state",
- * "postalCode" — spelled "zipCode" until objectstack#5143 —, "country") on its
- * sub-inputs. Two address fields in one form —
- * e.g. billing + shipping — then produced duplicate DOM ids, and every
- * `Label htmlFor` resolved to the FIRST match in the document: each sub-label
- * of the second field clicked/announced the first field's input.
- *
- * The fix follows the `groupId` paradigm of RadioField / CheckboxesField:
- * a `useId()` prefix + the sub-field name. These tests pin it by rendering
- * TWO instances in one form:
- *
- * 1. every `[id]` in the document is globally unique (red again if anyone
- *    reverts to hardcoded literals — both instances would emit id="street");
- * 2. each sub-label of the SECOND instance resolves — via the exact
- *    mechanism browsers use for label activation, `for` → getElementById —
- *    to an input inside its OWN instance, and focusing it lands there.
- */
-import { describe, it, expect, vi } from 'vitest';
-import React from 'react';
-import { render, within } from '@testing-library/react';
-import '@testing-library/jest-dom';
-import { AddressField } from './AddressField';
-
-const field = { name: 'address', type: 'address' } as any;
-
-const SUB_LABELS = [
-  'Street Address',
-  'City',
-  'State / Province',
-  'ZIP / Postal Code',
-  'Country',
-];
-
-function renderTwoInOneForm() {
-  return render(
-    <form>
-      <div data-testid="address-a">
-        <AddressField value={{}} onChange={vi.fn()} field={field} />
-      </div>
-      <div data-testid="address-b">
-        <AddressField value={{}} onChange={vi.fn()} field={field} />
-      </div>
-    </form>,
-  );
-}
-
-describe('AddressField — unique sub-input ids (objectui#3343)', () => {
-  it('two address fields in one form produce globally unique DOM ids', () => {
-    const { baseElement } = renderTwoInOneForm();
-    const ids = Array.from(baseElement.querySelectorAll('[id]')).map((el) => el.id);
-    // 5 sub-inputs per instance — both instances must actually render ids.
-    expect(ids.length).toBeGreaterThanOrEqual(10);
-    const duplicates = ids.filter((id, i) => ids.indexOf(id) !== i);
-    expect(duplicates).toEqual([]);
-  });
-
-  it("each sub-label of the SECOND field focuses that field's own input", () => {
-    const { getByTestId } = renderTwoInOneForm();
-    const first = getByTestId('address-a');
-    const second = getByTestId('address-b');
-
-    for (const text of SUB_LABELS) {
-      const label = within(second).getByText(text);
-      const forId = label.getAttribute('for');
-      expect(forId, `label "${text}" must carry htmlFor`).toBeTruthy();
-
-      // Exactly what the browser does on label click / SR announcement:
-      // resolve `for` against the document.
-      const control = document.getElementById(forId!);
-      expect(control, `htmlFor of "${text}" must resolve`).not.toBeNull();
-      expect(control!.tagName).toBe('INPUT');
-      expect(
-        second.contains(control),
-        `"${text}" must resolve into its OWN field, not the first one`,
-      ).toBe(true);
-      expect(first.contains(control)).toBe(false);
-
-      (control as HTMLInputElement).focus();
-      expect(control).toHaveFocus();
-    }
-  });
-});

package/src/widgets/AutoNumberField.tsx

@@ -1,18 +0,0 @@
-import React from 'react';
-import { EmptyValue } from '@object-ui/components';
-import { FieldWidgetComponentProps } from './types';
-
-/**
- * AutoNumberField - Read-only auto-generated field
- * Values are automatically generated by the system and cannot be edited
- */
-export function AutoNumberField({ value, field, ...props }: FieldWidgetComponentProps<string | number>) {
-  if (value == null) {
-    return <EmptyValue className={props.className} />;
-  }
-  return (
-    <span className={`text-sm font-mono text-gray-700 ${props.className || ''}`}>
-      {String(value)}
-    </span>
-  );
-}

package/src/widgets/AvatarField.tsx

@@ -1,117 +0,0 @@
-import React from 'react';
-import { Avatar, AvatarFallback, AvatarImage, Button } from '@object-ui/components';
-import { Upload, X } from 'lucide-react';
-import { FieldWidgetComponentProps } from './types';
-import { toDomProps } from './toDomProps';
-
-/**
- * Avatar field widget - provides an avatar/profile picture uploader
- * Supports image URLs or file uploads
- */
-export function AvatarField({ value, onChange, field, readonly, error, ...props }: FieldWidgetComponentProps<string>) {
-  const [isHovered, setIsHovered] = React.useState(false);
-  const fileInputRef = React.useRef<HTMLInputElement>(null);
-  
-  const avatarField = field as any;
-
-  const handleFileSelect = (event: React.ChangeEvent<HTMLInputElement>) => {
-    const file = event.target.files?.[0];
-    if (!file) return;
-
-    // Check file type
-    if (!file.type.startsWith('image/')) {
-      console.error('Please select an image file');
-      return;
-    }
-
-    // Check file size (max 5MB)
-    if (file.size > 5 * 1024 * 1024) {
-      console.error('File size must be less than 5MB');
-      return;
-    }
-
-    // Convert to base64 or upload to server
-    const reader = new FileReader();
-    reader.onloadend = () => {
-      onChange(reader.result as string);
-    };
-    reader.readAsDataURL(file);
-  };
-
-  const handleRemove = () => {
-    onChange('');
-  };
-
-  // Extract initials for fallback
-  const getInitials = (): string => {
-    const name = avatarField?.defaultName || avatarField?.label || 'User';
-    return name
-      .split(' ')
-      .map((word: string) => word[0])
-      .join('')
-      .toUpperCase()
-      .slice(0, 2);
-  };
-
-  if (readonly) {
-    return (
-      <Avatar className="w-16 h-16">
-        {value && <AvatarImage src={value} alt={avatarField?.label} />}
-        <AvatarFallback>{getInitials()}</AvatarFallback>
-      </Avatar>
-    );
-  }
-
-  return (
-    <div className="flex items-center gap-4">
-      <div
-        className="relative"
-        onMouseEnter={() => setIsHovered(true)}
-        onMouseLeave={() => setIsHovered(false)}
-      >
-        <Avatar className="w-16 h-16">
-          {value && <AvatarImage src={value} alt={avatarField?.label} />}
-          <AvatarFallback>{getInitials()}</AvatarFallback>
-        </Avatar>
-        {!readonly && isHovered && value && (
-          <button
-            type="button"
-            onClick={handleRemove}
-            className="absolute -top-1 -right-1 bg-destructive text-destructive-foreground rounded-full p-1 hover:bg-destructive/90"
-          >
-            <X className="w-3 h-3" />
-          </button>
-        )}
-      </div>
-      
-      <div className="flex flex-col gap-2">
-        <input
-          ref={fileInputRef}
-          type="file"
-          accept="image/*"
-          onChange={handleFileSelect}
-          className="hidden"
-        />
-        <Button
-          // DOM pass-through onto the widget's real focusable control — the
-          // upload button is the keyboard path to the hidden file input
-          // (objectui#3318).
-          {...toDomProps(props)}
-          type="button"
-          variant="outline"
-          size="sm"
-          onClick={() => fileInputRef.current?.click()}
-          disabled={readonly || props.disabled}
-          // AFTER the spread so this widget's own computation wins (#3222).
-          aria-invalid={!!error}
-        >
-          <Upload className="w-4 h-4 mr-2" />
-          {value ? 'Change' : 'Upload'} Avatar
-        </Button>
-        <p className="text-xs text-muted-foreground">
-          PNG, JPG up to 5MB
-        </p>
-      </div>
-    </div>
-  );
-}

package/src/widgets/BooleanField.tsx

@@ -1,86 +0,0 @@
-import React, { useId } from 'react';
-import { Switch, Checkbox, Label } from '@object-ui/components';
-import { FieldWidgetComponentProps } from './types';
-import { toDomProps } from './toDomProps';
-
-/**
- * BooleanField - Toggle input supporting switch and checkbox variants
- * Renders as Switch or Checkbox based on field widget configuration
- */
-export function BooleanField({ value, onChange, field, readonly, ...props }: FieldWidgetComponentProps<boolean>) {
-  const config = field as any;
-  // Use simple type assertion for arbitrary custom properties not in BaseFieldMetadata
-  const widget = config?.widget;
-  // Generate unique ID using React's useId hook - must be before early returns (rules of hooks)
-  const generatedId = useId();
-  /**
-   * The HOST owns the control's id (objectui#3952).
-   *
-   * `<FormControl>` is a Radix `Slot` that hands its child the id its
-   * `<FormLabel htmlFor>` already points at (`_r_N_-form-item`). This widget
-   * used to REPLACE it with the field name, so the form's visible label
-   * referenced an id no element carried: `label for="_r_3_-form-item"` against
-   * `all ids in form = ["notifications"]`. Clicking the label — the normal
-   * affordance for a switch/checkbox row — toggled nothing, on every generated
-   * form in every app.
-   *
-   * `id` is a DECLARED key of the widget contract (`FieldWidgetDomProps`) and
-   * `toDomProps` forwards it, so honouring it here is the contract's own
-   * direction: the fix belongs at the consumer that discarded a delivered
-   * value, not in the form renderer (AGENTS.md #0.1).
-   *
-   * The fallback chain stays for the STANDALONE hosts that hand down no id —
-   * `FieldEditWidget` (the grid's inline cell editor) and a bare SDUI node.
-   */
-  const hostId = props.id;
-  const id = hostId || config?.name || generatedId;
-  const label = config?.label || 'Checkbox';
-  /**
-   * A host that supplies the id is a host that renders its own label for it
-   * (that is what the id is FOR), so the widget's `sr-only` label would be the
-   * second copy of one name — the other half of #3952, and why a naive
-   * `getByText(label)` on a boolean field returned two nodes.
-   *
-   * Standalone it is the control's ONLY accessible name (`FieldEditWidget`
-   * renders no label of its own), so there it stays. Deliberately keyed on the
-   * host id rather than on `config.label`: inside a form the naming is the
-   * host's job for EVERY widget — a text field in a label-less form row is
-   * equally unnamed — and the generic `'Checkbox'` fallback emitted there was
-   * an actively WRONG accessible name, not a safety net.
-   */
-  const emitOwnLabel = !hostId;
-
-  if (readonly) {
-    return <span className="text-sm">{value ? 'Yes' : 'No'}</span>;
-  }
-
-  const domProps = toDomProps(props);
-
-  if (widget === 'checkbox') {
-     return (
-        <div className="flex items-center space-x-2">
-            <Checkbox
-                {...domProps}
-                id={id}
-                checked={!!value}
-                onCheckedChange={(checked) => onChange(!!checked)}
-                disabled={readonly || domProps.disabled}
-            />
-            {emitOwnLabel && <Label htmlFor={id} className="sr-only">{label}</Label>}
-        </div>
-     )
-  }
-
-  return (
-    <div className="flex items-center space-x-2">
-        <Switch
-            {...domProps}
-            id={id}
-            checked={!!value}
-            onCheckedChange={onChange}
-            disabled={readonly || domProps.disabled}
-        />
-        {emitOwnLabel && <Label htmlFor={id} className="sr-only">{label}</Label>}
-    </div>
-  );
-}

package/src/widgets/CapabilityMultiSelectField.test.tsx

@@ -1,168 +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.
- */
-
-/**
- * CapabilityMultiSelectField (ADR-0056 P2, epic #2398).
- *
- * Proves the two properties P2 depends on:
- *  1. the stored value parses tolerantly into a capability-name list, and
- *  2. every edit emits a JSON-**string** array of names — byte-equivalent to
- *     the `sys_permission_set.system_permissions` textarea storage, so the
- *     round-trip through the picker never changes the on-disk shape.
- */
-import { describe, it, expect, vi } from 'vitest';
-import React from 'react';
-import { render, screen, fireEvent } from '@testing-library/react';
-import '@testing-library/jest-dom';
-import { CapabilityMultiSelectField, parseCapabilityNames } from './CapabilityMultiSelectField';
-
-describe('parseCapabilityNames', () => {
-  it('parses the canonical JSON-string array', () => {
-    expect(parseCapabilityNames('["setup.access","studio.access"]')).toEqual([
-      'setup.access',
-      'studio.access',
-    ]);
-  });
-
-  it('treats empty / null / undefined as no selection', () => {
-    expect(parseCapabilityNames('')).toEqual([]);
-    expect(parseCapabilityNames('   ')).toEqual([]);
-    expect(parseCapabilityNames(null)).toEqual([]);
-    expect(parseCapabilityNames(undefined)).toEqual([]);
-    expect(parseCapabilityNames('[]')).toEqual([]);
-  });
-
-  it('tolerates an already-parsed array', () => {
-    expect(parseCapabilityNames(['a', 'b'])).toEqual(['a', 'b']);
-  });
-
-  it('falls back to comma-splitting a non-JSON legacy string', () => {
-    expect(parseCapabilityNames('setup.access, studio.access')).toEqual([
-      'setup.access',
-      'studio.access',
-    ]);
-  });
-});
-
-const CAPS = [
-  { name: 'manage_users', label: 'Manage Users', description: 'Manage users', scope: 'platform', active: true },
-  { name: 'studio.access', label: 'Studio Access', description: 'Enter Studio', scope: 'platform', active: true },
-  { name: 'manage_org_users', label: 'Manage Org Users', description: 'Org members', scope: 'org', active: true },
-];
-
-function mockDataSource(rows = CAPS) {
-  return { find: vi.fn().mockResolvedValue({ data: rows }) } as any;
-}
-
-describe('CapabilityMultiSelectField', () => {
-  it('renders capabilities grouped by scope with human labels', async () => {
-    render(
-      <CapabilityMultiSelectField
-        value={'["studio.access"]'}
-        onChange={vi.fn()}
-        field={{ name: 'system_permissions' } as any}
-        dataSource={mockDataSource()}
-      />,
-    );
-    // group headers
-    expect(await screen.findByText('Platform')).toBeInTheDocument();
-    expect(screen.getByText('Organization')).toBeInTheDocument();
-    // labels, not raw names
-    expect(screen.getByRole('button', { name: 'Studio Access' })).toHaveAttribute('aria-pressed', 'true');
-    expect(screen.getByRole('button', { name: 'Manage Users' })).toHaveAttribute('aria-pressed', 'false');
-  });
-
-  it('emits a JSON-string array when a capability is toggled on', async () => {
-    const onChange = vi.fn();
-    render(
-      <CapabilityMultiSelectField
-        value={'["studio.access"]'}
-        onChange={onChange}
-        field={{ name: 'system_permissions' } as any}
-        dataSource={mockDataSource()}
-      />,
-    );
-    fireEvent.click(await screen.findByRole('button', { name: 'Manage Users' }));
-    expect(onChange).toHaveBeenCalledTimes(1);
-    const emitted = onChange.mock.calls[0][0];
-    // The emitted value must be a JSON *string* (not a raw array), byte-equal
-    // to the textarea storage contract.
-    expect(typeof emitted).toBe('string');
-    expect(JSON.parse(emitted)).toEqual(['studio.access', 'manage_users']);
-  });
-
-  it('removes a capability (and keeps emitting a JSON string) when toggled off', async () => {
-    const onChange = vi.fn();
-    render(
-      <CapabilityMultiSelectField
-        value={'["studio.access","manage_users"]'}
-        onChange={onChange}
-        field={{ name: 'system_permissions' } as any}
-        dataSource={mockDataSource()}
-      />,
-    );
-    // Chip labels now resolve through the i18n bundle (objectui#2600 B5), so an
-    // async translation-settle can re-render the chips between findBy and the
-    // click — re-query at click time so we act on the live node, not a detached one.
-    await screen.findByRole('button', { name: 'Studio Access' });
-    fireEvent.click(screen.getByRole('button', { name: 'Studio Access' }));
-    const emitted = onChange.mock.calls[0][0];
-    expect(typeof emitted).toBe('string');
-    expect(JSON.parse(emitted)).toEqual(['manage_users']);
-  });
-
-  it('preserves an unknown/legacy selected name not in the registry', async () => {
-    render(
-      <CapabilityMultiSelectField
-        value={'["studio.access","legacy.custom"]'}
-        onChange={vi.fn()}
-        field={{ name: 'system_permissions' } as any}
-        dataSource={mockDataSource()}
-      />,
-    );
-    // The unknown name still renders as a (selected) chip so it is not dropped.
-    const legacy = await screen.findByRole('button', { name: 'legacy.custom' });
-    expect(legacy).toHaveAttribute('aria-pressed', 'true');
-  });
-
-  it('renders selected labels as read-only badges when readonly', async () => {
-    render(
-      <CapabilityMultiSelectField
-        value={'["studio.access"]'}
-        onChange={vi.fn()}
-        field={{ name: 'system_permissions' } as any}
-        dataSource={mockDataSource()}
-        readonly
-      />,
-    );
-    // No toggle buttons in readonly mode.
-    expect(await screen.findByText('Studio Access')).toBeInTheDocument();
-    expect(screen.queryByRole('button', { name: 'Manage Users' })).not.toBeInTheDocument();
-  });
-
-  // objectui#2600 B5 — curated platform caps get a localized label; package/
-  // admin-authored caps keep whatever label the registry served.
-  it('localizes curated capability labels but preserves registry labels for others', async () => {
-    render(
-      <CapabilityMultiSelectField
-        value={'[]'}
-        onChange={vi.fn()}
-        field={{ name: 'system_permissions' } as any}
-        dataSource={mockDataSource([
-          // Registry sends a shorter label; the curated client map wins.
-          { name: 'manage_org_users', label: 'Manage Org Users', description: 'Org members', scope: 'org', active: true },
-          // Not a curated platform capability — its registry label is kept.
-          { name: 'export_data', label: 'Export Data', description: 'Export', scope: 'org', active: true },
-        ])}
-      />,
-    );
-    expect(await screen.findByRole('button', { name: 'Manage Organization Users' })).toBeInTheDocument();
-    expect(screen.queryByRole('button', { name: 'Manage Org Users' })).not.toBeInTheDocument();
-    expect(screen.getByRole('button', { name: 'Export Data' })).toBeInTheDocument();
-  });
-});