@object-ui/data-objectstack 17.5.0 → 17.7.0

package/src/filter-entry-translation.test.ts

@@ -1,360 +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.
- */
-
-/**
- * Object-form filter entries (`[{ field, operator, value }, ...]`) → filter AST.
- *
- * `find()` has TWO routes to the server and both translate the filter: a plain
- * read goes through `convertQueryParams`, while a read with `$expand`/`$search`
- * goes through `rawFindWithPopulate`. They used to carry independent copies of
- * the "is this object form?" test, and the copies had already drifted — so the
- * tests below run every case down BOTH routes and assert they agree.
- *
- * The stakes are the same as objectstack#3948: a filter this layer fails to
- * translate is not a filter it may skip. Skipping one entry of an `and` returns
- * a superset of the rows asked for, and skipping the last one sends no filter
- * at all — every row in the table, reported as success.
- */
-
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { isFilterAST } from '@objectstack/spec/data';
-import { ObjectStackAdapter, clearSharedDiscoveryCache, isMalformedFilterError } from './index';
-
-function makeAdapter() {
-  const calls: string[] = [];
-  const fetchImpl = vi.fn(async (url: any) => {
-    const u = String(url);
-    calls.push(u);
-    if (u.includes('/api/v1/discovery')) {
-      return {
-        ok: true, status: 200, statusText: 'OK',
-        json: async () => ({ success: true, data: { version: 'v1', routes: {} } }),
-      } as any;
-    }
-    return {
-      ok: true, status: 200, statusText: 'OK',
-      json: async () => ({ success: true, data: { object: 'account', records: [], total: 0 } }),
-    } as any;
-  });
-  const adapter = new ObjectStackAdapter({
-    baseUrl: 'http://localhost:3000', token: 't', autoReconnect: false, fetch: fetchImpl as any,
-  });
-  return { adapter, calls };
-}
-
-/** The `filter=` this filter produced on the wire, or `undefined` if none was sent. */
-async function filterOnWire(
-  $filter: unknown,
-  route: 'plain' | 'expand',
-): Promise<unknown | undefined> {
-  const { adapter, calls } = makeAdapter();
-  await adapter.find('account', {
-    $filter,
-    ...(route === 'expand' ? { $expand: ['owner'] } : {}),
-  } as any);
-  const dataCall = calls.filter((u) => u.includes('/data/account')).pop();
-  const raw = dataCall ? new URL(dataCall).searchParams.get('filter') : null;
-  return raw === null ? undefined : JSON.parse(raw);
-}
-
-/** Run one input down both `find()` routes. */
-function bothRoutes(name: string, $filter: unknown, assert: (wire: unknown | undefined) => void) {
-  for (const route of ['plain', 'expand'] as const) {
-    it(`${name} (${route} route)`, async () => assert(await filterOnWire($filter, route)));
-  }
-}
-
-async function findRejects($filter: unknown, route: 'plain' | 'expand') {
-  const { adapter } = makeAdapter();
-  return adapter
-    .find('account', { $filter, ...(route === 'expand' ? { $expand: ['owner'] } : {}) } as any)
-    .then(() => null, (e) => e);
-}
-
-describe('object-form filter entries reach the wire as an AST', () => {
-  beforeEach(() => clearSharedDiscoveryCache());
-
-  bothRoutes(
-    'translates a single rule, mapping the operator alias',
-    [{ field: 'stage', operator: 'greater_than_or_equal', value: 3 }],
-    (wire) => expect(wire).toEqual(['stage', '>=', 3]),
-  );
-
-  bothRoutes(
-    'joins several rules with `and`',
-    [
-      { field: 'stage', operator: 'eq', value: 'won' },
-      { field: 'amount', operator: 'gt', value: 100 },
-    ],
-    (wire) => expect(wire).toEqual(['and', ['stage', '=', 'won'], ['amount', '>', 100]]),
-  );
-
-  bothRoutes(
-    'accepts the `op` shorthand key',
-    [{ field: 'owner', op: 'eq', value: 'me' }],
-    (wire) => expect(wire).toEqual(['owner', '=', 'me']),
-  );
-
-  bothRoutes(
-    'passes an AST tuple through untouched',
-    ['stage', '=', 'won'],
-    (wire) => expect(wire).toEqual(['stage', '=', 'won']),
-  );
-
-  bothRoutes(
-    'passes a logical AST node through untouched',
-    ['or', ['stage', '=', 'won'], ['stage', '=', 'lost']],
-    (wire) => expect(wire).toEqual(['or', ['stage', '=', 'won'], ['stage', '=', 'lost']]),
-  );
-
-  bothRoutes('sends no filter at all for an empty array', [], (wire) => expect(wire).toBeUndefined());
-});
-
-describe('a `name`-keyed entry is not an accepted shape', () => {
-  beforeEach(() => clearSharedDiscoveryCache());
-
-  /**
-   * `objectFilterEntryToAST` read `entry.field ?? entry.name` while the shape
-   * sniff keyed on `field` alone, so the `name` half could never be reached —
-   * dead from the commit that introduced it. The spec agrees it is not a real
-   * shape: `ViewFilterRuleSchema.field` is required, so such a rule cannot be
-   * saved as view metadata. It reaches the server unchanged and is refused
-   * there (objectstack#4121) rather than being quietly half-understood here.
-   */
-  bothRoutes(
-    'is passed through rather than translated',
-    [{ name: 'stage', operator: 'eq', value: 'won' }],
-    (wire) => expect(wire).toEqual([{ name: 'stage', operator: 'eq', value: 'won' }]),
-  );
-});
-
-describe('an untranslatable entry fails instead of narrowing the filter', () => {
-  beforeEach(() => clearSharedDiscoveryCache());
-
-  for (const route of ['plain', 'expand'] as const) {
-    it(`rejects a rule with a blank field rather than dropping it (${route})`, async () => {
-      // Survives `ViewFilterRuleSchema` (`field: z.string()` admits ''), so it
-      // can be genuine stored metadata. It used to be dropped, and dropping the
-      // only rule sent no filter — the whole table came back.
-      const err = await findRejects([{ field: '', operator: 'eq', value: 'x' }], route);
-      expect(err).toBeInstanceOf(Error);
-      expect(isMalformedFilterError(err)).toBe(true);
-    });
-
-    it(`rejects a bad entry rather than silently widening the AND (${route})`, async () => {
-      const err = await findRejects(
-        [
-          { field: 'stage', operator: 'eq', value: 'won' },
-          { field: 'amount', operator: 42 as any, value: 1 },
-        ],
-        route,
-      );
-      expect(isMalformedFilterError(err)).toBe(true);
-      // Names which entry, so the author can find it in the view config.
-      expect(String((err as Error).message)).toContain('entry 1');
-    });
-  }
-
-  it('carries the status and code that classify it as a rejected request', async () => {
-    const err: any = await findRejects([{ field: '', operator: 'eq' }], 'plain');
-    // plugin-list's `classifyLoadError` reads these to pick the error panel;
-    // without them a malformed filter reads as "check your connection" (#3066).
-    expect(err.code).toBe('INVALID_FILTER');
-    expect(err.httpStatus).toBe(400);
-  });
-});
-
-describe('object-form rules nested inside a logical node are translated too', () => {
-  beforeEach(() => clearSharedDiscoveryCache());
-
-  /**
-   * This is what a list builds the moment a view with a stored filter meets a
-   * user filter from the panel — `['and', <ViewFilterRule[]>, <AST tuples>]`.
-   * Translating only the top level shipped the rules raw, and the server has no
-   * good answer for that: `isFilterAST` rejects the shape (so a server with
-   * objectstack#4121 returns 400 and the list fails to load), while
-   * `parseFilterAST` on the same array returns `{ amount: { $gt: 1 } }` — the
-   * view's own `stage = won` dropped without a word.
-   */
-  bothRoutes(
-    'translates a rule group nested under `and`',
-    ['and', [{ field: 'stage', operator: 'eq', value: 'won' }], [['amount', '>', 1]]],
-    (wire) => expect(wire).toEqual(['and', ['stage', '=', 'won'], [['amount', '>', 1]]]),
-  );
-
-  bothRoutes(
-    'translates under `or`, and through more than one level',
-    ['or', ['and', [{ field: 'stage', operator: 'not_in', value: ['lost'] }]], ['x', '=', 1]],
-    (wire) => expect(wire).toEqual(['or', ['and', ['stage', 'nin', ['lost']]], ['x', '=', 1]]),
-  );
-
-  bothRoutes(
-    'translates a group inside a legacy flat array of children',
-    [[{ field: 'stage', operator: 'eq', value: 'won' }], ['amount', '>', 1]],
-    (wire) => expect(wire).toEqual([['stage', '=', 'won'], ['amount', '>', 1]]),
-  );
-
-  bothRoutes(
-    'leaves a field literally named `and` alone',
-    ['and', '=', true],
-    (wire) => expect(wire).toEqual(['and', '=', true]),
-  );
-
-  /**
-   * A mixed array used to lose the tuple: it did not translate as a rule, so it
-   * was filtered out and the query ran on the remaining condition alone —
-   * broader than asked for. Keep both rather than dropping one or failing the
-   * whole query.
-   */
-  bothRoutes(
-    'keeps both halves of a rule/tuple mixed array',
-    [{ field: 'stage', operator: 'eq', value: 'won' }, ['amount', '>', 1]],
-    (wire) => expect(wire).toEqual(['and', ['stage', '=', 'won'], ['amount', '>', 1]]),
-  );
-
-  it('rejects a malformed rule nested under `and` as well', async () => {
-    const err = await findRejects(['and', [{ field: '', operator: 'eq', value: 1 }]], 'plain');
-    expect(isMalformedFilterError(err)).toBe(true);
-  });
-});
-
-describe('what this adapter emits passes the server’s own gate', () => {
-  beforeEach(() => clearSharedDiscoveryCache());
-
-  /**
-   * The real contract is not a literal shape, it is "the server accepts this".
-   * `isFilterAST` is the function the protocol gates on, so assert against it
-   * directly rather than restating what it happens to accept today — a filter
-   * it rejects is a 400 (objectstack#4121), and was an unfiltered query before
-   * that (objectstack#3948).
-   */
-  const TRANSLATABLE: Array<[string, unknown]> = [
-    ['a single rule', [{ field: 'stage', operator: 'eq', value: 'won' }]],
-    ['several rules', [
-      { field: 'stage', operator: 'contains', value: 'w' },
-      { field: 'amount', operator: 'less_than_or_equal', value: 9 },
-    ]],
-    ['rules under `and` beside a tuple', [
-      'and', [{ field: 'stage', operator: 'before', value: '2024-01-01' }], ['amount', '>', 1],
-    ]],
-    ['rules inside a legacy flat array', [
-      [{ field: 'stage', operator: 'is_null', value: true }], ['amount', '>', 1],
-    ]],
-    ['a mixed rule/tuple array', [{ field: 'stage', operator: 'eq', value: 'won' }, ['a', '>', 1]]],
-  ];
-
-  for (const [name, input] of TRANSLATABLE) {
-    for (const route of ['plain', 'expand'] as const) {
-      it(`${name} → accepted by isFilterAST (${route})`, async () => {
-        expect(isFilterAST(await filterOnWire(input, route))).toBe(true);
-      });
-    }
-  }
-});
-
-describe('a bare rule object directly under a logical node', () => {
-  beforeEach(() => clearSharedDiscoveryCache());
-
-  /**
-   * Produced by any caller that SPREADS a `ViewFilterRule[]` into an `and`
-   * rather than wrapping it — `['and', ...rules, ...tuples]`. The rules land as
-   * bare objects where the AST expects nodes, `isFilterAST` rejects the whole
-   * node (a 400 since objectstack#4121), and `parseFilterAST` reads the rule as
-   * a Mongo condition on columns literally named `field`/`operator`/`value`.
-   * objectui's own producer was fixed to wrap; this is the chokepoint defence.
-   */
-  bothRoutes(
-    'is translated in place',
-    ['and', { field: 'stage', operator: 'eq', value: 'won' }, ['owner', '=', 'me']],
-    (wire) => expect(wire).toEqual(['and', ['stage', '=', 'won'], ['owner', '=', 'me']]),
-  );
-
-  bothRoutes(
-    'leaves a genuine MongoDB condition child alone',
-    // No `field` key, so it is a condition on a column named `status` — not a
-    // rule. Translating it would invent a filter the caller never wrote.
-    ['and', { status: 'active' }, ['owner', '=', 'me']],
-    (wire) => expect(wire).toEqual(['and', { status: 'active' }, ['owner', '=', 'me']]),
-  );
-});
-
-describe('an OBJECT filter reaches the same predicate on both routes', () => {
-  beforeEach(() => clearSharedDiscoveryCache());
-
-  /**
-   * The array branch was single-sourced in #3072; the object branch was not.
-   * `convertQueryParams` converted a MongoDB-style filter to AST while
-   * `translateFilterToAST` returned it verbatim — so the same `$filter` went out
-   * in two formats, decided by whether the query expanded a lookup.
-   *
-   * Measured across 21 operator shapes, four diverged. `$exists` vs `$null` was
-   * a cosmetic difference the server treats identically, but the other three
-   * were not: see below.
-   */
-  bothRoutes(
-    'converts a plain equality object',
-    { status: 'active' },
-    (wire) => expect(wire).toEqual(['status', '=', 'active']),
-  );
-
-  bothRoutes(
-    'converts an operator object',
-    { age: { $gte: 18 } },
-    (wire) => expect(wire).toEqual(['age', '>=', 18]),
-  );
-
-  bothRoutes(
-    'leaves a top-level Mongo logical node alone',
-    // `mergeFilters` (dashboard scope filters) produces this. It survives as a
-    // `['$and', '=', [...]]` comparison that `parseFilterAST` reads back as a
-    // real `$and` — verified, and the reason this is NOT rewritten here.
-    { $and: [{ a: 1 }, { b: 2 }] },
-    (wire) => expect(wire).toEqual(['$and', '=', [{ a: 1 }, { b: 2 }]]),
-  );
-
-  for (const route of ['plain', 'expand'] as const) {
-    it(`refuses an unknown operator on the ${route} route`, async () => {
-      // The guard exists "to avoid silent failure" — it used to run on one
-      // route only, so a typo shipped silently whenever a lookup was expanded.
-      const err = await findRejects({ s: { $bogus: 1 } }, route);
-      expect(err).toBeInstanceOf(Error);
-      expect(String((err as Error).message)).toContain('$bogus');
-      expect(isMalformedFilterError(err)).toBe(true);
-    });
-
-    it(`refuses $regex rather than answering a different question (${route})`, async () => {
-      // Was silently rewritten to `contains`: `$regex: 'a.c'` matches "abc",
-      // `contains 'a.c'` does not. The spec has no `$regex` to translate into.
-      const err = await findRejects({ s: { $regex: 'a.c' } }, route);
-      expect(isMalformedFilterError(err)).toBe(true);
-      expect(String((err as Error).message)).toMatch(/regex/i);
-    });
-  }
-});
-
-describe('the two routes agree on shapes that are neither form', () => {
-  beforeEach(() => clearSharedDiscoveryCache());
-
-  /**
-   * The inline copy of the sniff omitted the `!== null` guard that this one
-   * had, so `[null]` threw a TypeError on the plain route and was handled on
-   * the `$expand` route — the same stored filter, decided by whether the view
-   * happened to expand a lookup.
-   */
-  bothRoutes(
-    'does not crash on a null entry',
-    [null],
-    (wire) => expect(wire).toEqual([null]),
-  );
-
-  bothRoutes(
-    'leaves a nested legacy AST alone',
-    [['stage', '=', 'won'], ['amount', '>', 1]],
-    (wire) => expect(wire).toEqual([['stage', '=', 'won'], ['amount', '>', 1]]),
-  );
-});

package/src/filter-operator-ast-parity.test.ts

@@ -1,135 +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.
- */
-
-/**
- * Adapter operator table → filter-AST parity (#2901, objectstack#3948, #3641).
- *
- * `FILTER_OPERATOR_ALIASES` is the last translation a filter passes through
- * before it goes on the wire, and `normalizeFilterOperator` ends in `?? op` —
- * an unmapped operator is emitted verbatim. The server then rejects the shape
- * at `isFilterAST()`, passes the array through unconverted, and driver-sql
- * skips it entirely: **no WHERE clause, no error, every row returned.**
- *
- * So a missing row in this table is not a validation failure, it is an
- * unfiltered query. `before`/`after` — canonical members of the spec's
- * `VIEW_FILTER_OPERATORS` — were missing, which is exactly how a stored
- * "close_date before X" view came back unfiltered.
- *
- * That failure description is historical, and one part of it has moved (measured
- * for #3641, not assumed): on the published `@objectstack/driver-sql@17.0.0-rc.5`
- * the array form is lowered by `parseFilterAST()` at the engine and protocol
- * doors, and an array reaching the driver anyway is refused with a 400
- * (`unsupportedFilterError`, objectstack#5158) rather than skipped. What the
- * protocol door does with an array the AST gate REJECTS is not measurable from
- * this repo, so whether a missing row costs an unfiltered read or a hard 400
- * today is deliberately left open. Both are a broken query.
- *
- * ## What the coverage guarantee rests on (#3641)
- *
- * The coverage sweep below used to mirror production's `?? op` tail: resolve the
- * operator through the table, fall back to the raw view spelling, then ask
- * whether the result was a member of `VALID_AST_OPERATORS`. That reads as a
- * coverage assertion and is not one. `VALID_AST_OPERATORS` is derived upstream
- * from `AST_OPERATOR_MAP`, it grew until it spelled the canonical view operators
- * verbatim — `before` and `after` included — and from that moment the fallback
- * arm was always AST-valid. Emptying this entire table left the sweep green, so
- * the one thing it existed to catch, a missing row, had become invisible to it.
- *
- * The sweep therefore asserts the row EXISTS, with no fallback: the header above
- * says a missing row is an unfiltered query rather than a validation failure, so
- * a missing row is what is asserted, directly. What the rows resolve TO is a
- * separate question, kept as its own assertion below — and the pair that
- * actually regressed is pinned by spelling, not by membership.
- */
-import { describe, it, expect } from 'vitest';
-import { VALID_AST_OPERATORS } from '@objectstack/spec/data';
-import { VIEW_FILTER_OPERATORS } from '@objectstack/spec/ui';
-import { FILTER_OPERATOR_ALIASES } from './index';
-
-/**
- * View operators this adapter is not the bridge for — the value-shape ones the
- * view layer resolves to a null comparison before an operator is ever emitted.
- *
- * Every token here must still be a member of `VIEW_FILTER_OPERATORS` — the
- * ratchet below enforces it. Subtracting a name the spec has retired excuses
- * nothing and must be deleted rather than left as a dead subtraction (#3628).
- */
-const NOT_THIS_ADAPTERS_JOB = new Set(['is_empty', 'is_not_empty']);
-
-describe('FILTER_OPERATOR_ALIASES lands inside the spec AST vocabulary', () => {
-  it('reads both vocabularies from the spec', () => {
-    expect(VIEW_FILTER_OPERATORS.length).toBeGreaterThan(0);
-    expect(VALID_AST_OPERATORS.size).toBeGreaterThan(0);
-  });
-
-  // The exclusion ratchet (#3628). The coverage sweep further down subtracts a
-  // hand-written set from a spec-derived vocabulary, and that subtraction only
-  // excuses something while the spec still lists the subtracted tokens. Once
-  // upstream retires or renames one, the sweep stays green (it is still total
-  // over what remains) but the row becomes dead weight, and its comment goes on
-  // telling the next reader that the view layer resolves this one to a null
-  // comparison — about an operator no author can declare any more. That is the
-  // shape that rotted 37 of 82 deny-list entries in #3601 with nothing to report
-  // it: a hand-written list beside a spec-derived vocabulary and no assertion
-  // that its members still exist in that vocabulary.
-  //
-  // Collected rather than asserted per entry on purpose (same call as PR #3623):
-  // vocabulary retirements land as whole families, and failing on the first entry
-  // would hide the rest.
-  it('every NOT_THIS_ADAPTERS_JOB token is still in the spec view vocabulary', () => {
-    const vocabulary = new Set<string>(VIEW_FILTER_OPERATORS);
-    const retired = [...NOT_THIS_ADAPTERS_JOB].filter((op) => !vocabulary.has(op));
-    expect(
-      retired,
-      `VIEW_FILTER_OPERATORS no longer lists these NOT_THIS_ADAPTERS_JOB tokens: `
-        + `${retired.join(', ')}. The spec has retired them, so subtracting them from `
-        + 'the coverage sweep below excuses nothing — delete each from the set (with '
-        + 'the comment claiming the view layer resolves it) rather than leaving a dead '
-        + 'subtraction',
-    ).toEqual([]);
-  });
-
-  it('every alias target is an operator the AST gate accepts', () => {
-    const bad = Object.entries(FILTER_OPERATOR_ALIASES)
-      .filter(([, target]) => !VALID_AST_OPERATORS.has(String(target).toLowerCase()))
-      .map(([alias, target]) => `${alias} → ${target}`);
-    expect(
-      bad,
-      'these aliases translate to operators VALID_AST_OPERATORS rejects, so the '
-        + 'server drops the filter silently instead of erroring',
-    ).toEqual([]);
-  });
-
-  it('has a mapping row for every canonical view operator the spec defines', () => {
-    // Resolution mirrors `normalizeFilterOperator` — lowercased spelling first,
-    // then the operator as written — but stops short of its `?? op` tail. That
-    // tail is production behaviour and must stay there; reproducing it HERE is
-    // what cancelled this assertion (#3641), because the value it falls back to
-    // is the raw view spelling and the AST vocabulary now accepts all of those.
-    const hasRow = (op: string) =>
-      Object.prototype.hasOwnProperty.call(FILTER_OPERATOR_ALIASES, op.toLowerCase())
-      || Object.prototype.hasOwnProperty.call(FILTER_OPERATOR_ALIASES, op);
-
-    const missing = VIEW_FILTER_OPERATORS
-      .filter((op) => !NOT_THIS_ADAPTERS_JOB.has(op))
-      .filter((op) => !hasRow(op));
-    expect(
-      missing,
-      'FILTER_OPERATOR_ALIASES has no row for these. An author can declare them on a '
-        + 'ViewFilterRule and the spec validates them, so they reach the wire as the raw '
-        + 'view spelling. Do not settle for "the AST gate happens to accept that" — the '
-        + 'gate accepting a spelling is not the driver compiling it into a WHERE clause, '
-        + 'and an unmapped operator is how this adapter shipped an unfiltered query',
-    ).toEqual([]);
-  });
-
-  it('maps the date comparisons that regressed', () => {
-    expect(FILTER_OPERATOR_ALIASES.before).toBe('<');
-    expect(FILTER_OPERATOR_ALIASES.after).toBe('>');
-  });
-});

package/src/getObjectSchema.test.ts

@@ -1,145 +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.
- */
-
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { ObjectStackAdapter, clearSharedDiscoveryCache } from './index';
-
-/**
- * A fetch mock that answers discovery + records the `GET /meta/object/:name`
- * read so we can assert how it was issued.
- */
-function makeFetch(objectBody: unknown) {
-  const calls: Array<{ url: string; init?: any }> = [];
-  const fetchImpl = vi.fn(async (url: any, init?: any) => {
-    const u = String(url);
-    calls.push({ url: u, init });
-    if (u.includes('/api/v1/discovery')) {
-      return { ok: true, status: 200, statusText: 'OK', json: async () => ({ success: true, data: { version: 'v1', routes: {} } }) } as any;
-    }
-    if (u.includes('/api/v1/meta/object/')) {
-      return { ok: true, status: 200, statusText: 'OK', json: async () => objectBody } as any;
-    }
-    return { ok: true, status: 200, statusText: 'OK', json: async () => ({}) } as any;
-  });
-  return { fetchImpl, calls };
-}
-
-describe('ObjectStackAdapter.getObjectSchema', () => {
-  beforeEach(() => clearSharedDiscoveryCache());
-
-  it('reads with cache: "no-cache" so a freshly published field is not masked by the server max-age', async () => {
-    // The server marks single-object metadata `public, max-age=3600`; without
-    // revalidation the browser HTTP cache would keep serving the pre-publish
-    // schema and the create/edit form would miss fields published this session.
-    const { fetchImpl, calls } = makeFetch({
-      name: 'leave_request',
-      fields: { employee_name: { type: 'text' }, department: { type: 'text' } },
-    });
-    const adapter = new ObjectStackAdapter({
-      baseUrl: 'http://localhost:3000',
-      token: 'tok_123',
-      autoReconnect: false,
-      fetch: fetchImpl as any,
-    });
-
-    const schema: any = await adapter.getObjectSchema('leave_request');
-
-    // Returns the (unwrapped) object schema with its fields.
-    expect(Object.keys(schema.fields)).toEqual(['employee_name', 'department']);
-
-    const get = calls.find((c) => c.url.includes('/api/v1/meta/object/'))!;
-    expect(get.url).toBe('http://localhost:3000/api/v1/meta/object/leave_request');
-    expect(get.init.method).toBe('GET');
-    // The crux of the fix: revalidate the HTTP cache instead of serving the
-    // stale `max-age` body.
-    expect(get.init.cache).toBe('no-cache');
-    expect(get.init.headers.Authorization).toBe('Bearer tok_123');
-  });
-
-  it('unwraps the { item } envelope when the server wraps the object', async () => {
-    const { fetchImpl } = makeFetch({ item: { name: 'account', fields: { x: { type: 'text' } } } });
-    const adapter = new ObjectStackAdapter({
-      baseUrl: 'http://localhost:3000',
-      autoReconnect: false,
-      fetch: fetchImpl as any,
-    });
-
-    const schema: any = await adapter.getObjectSchema('account');
-
-    expect(schema.name).toBe('account');
-    expect(Object.keys(schema.fields)).toEqual(['x']);
-  });
-
-  it('canonicalizes the ObjectStack-convention `reference` key onto `reference_to` (and back)', async () => {
-    // The server names a relational field's target `reference`
-    // (showcase_project.account → { type: 'lookup', reference: 'showcase_account' }),
-    // while most consumers read `reference_to` (#2407 / PR #2587). getObjectSchema
-    // is the choke point every schema read goes through, so both keys must come
-    // back stamped — regardless of which convention the served schema used.
-    const { fetchImpl } = makeFetch({
-      name: 'showcase_project',
-      fields: {
-        name: { type: 'text' },
-        account: { type: 'lookup', reference: 'showcase_account' },
-        team_members: { type: 'user', reference: 'sys_user', multiple: true },
-        legacy: { type: 'master_detail', reference_to: 'showcase_order' },
-      },
-    });
-    const adapter = new ObjectStackAdapter({
-      baseUrl: 'http://localhost:3000',
-      autoReconnect: false,
-      fetch: fetchImpl as any,
-    });
-
-    const schema: any = await adapter.getObjectSchema('showcase_project');
-
-    expect(schema.fields.account.reference_to).toBe('showcase_account');
-    expect(schema.fields.team_members.reference_to).toBe('sys_user');
-    // Mirror direction: `.reference`-only readers (e.g. attachInlineSubforms)
-    // must also see the target on a reference_to-authored schema.
-    expect(schema.fields.legacy.reference).toBe('showcase_order');
-    // Non-relational fields stay untouched.
-    expect('reference_to' in schema.fields.name).toBe(false);
-  });
-
-  it('preserves `validations` (incl. state_machine transitions) — the seam the inline editor depends on', async () => {
-    // Inert-metadata regression guard. The inline select editor filters its
-    // options by the object's `state_machine` validation (objectui#2110). That
-    // feature only works because `getObjectSchema` passes the served
-    // `validations` through untouched — the server enforces the same rule. If a
-    // future change here started stripping or reshaping top-level metadata, the
-    // filter would silently no-op (valid-but-inert) with no error. Pin the
-    // pass-through so the data path can't go dark unnoticed.
-    const { fetchImpl } = makeFetch({
-      name: 'showcase_task',
-      fields: { status: { type: 'select', options: [{ value: 'done', label: 'Done' }] } },
-      validations: [
-        {
-          type: 'state_machine',
-          field: 'status',
-          transitions: { in_review: ['done', 'in_progress'], done: ['in_progress'] },
-        },
-      ],
-    });
-    const adapter = new ObjectStackAdapter({
-      baseUrl: 'http://localhost:3000',
-      autoReconnect: false,
-      fetch: fetchImpl as any,
-    });
-
-    const schema: any = await adapter.getObjectSchema('showcase_task');
-
-    const sm = (schema.validations ?? []).find((v: any) => v?.type === 'state_machine');
-    expect(sm).toBeTruthy();
-    expect(sm.field).toBe('status');
-    // The exact map the consumer filters on — `done` only transitions to
-    // `in_progress` (so the editor must not offer `in_review`).
-    expect(sm.transitions.done).toEqual(['in_progress']);
-    expect(sm.transitions.in_review).toEqual(['done', 'in_progress']);
-  });
-});