fix(plugin-grid, plugin-detail): a returnType currency formula reads as money in list cells and on the record page - #27689

domain:ui#2 · os-bill · session_01Utp6CY2n1JmZZKrHgtp9jC

Requested by Bill · project thread

Fixes #27494

Clause-②: no
Metadata-protocol: none

维护者速读

• 改了什么：列表网格的三条列构建路径和记录页的字段补全，现在把公式字段声明的 returnType 一并传给单元格；returnType: 'currency' 的公式（如演示应用的「Budget Remaining」）按货币显示 $180,000.00，与旁边的货币列一致。
• 为什么改：公式单元格本来就会把声明为 currency 的结果交给货币单元格渲染，但列表和记录页在重建字段时只挑了 currencyConfig、scale 等键，漏了 returnType，单元格以为是未声明类型的数字，显示成裸数字 180,000.00。
• 风险与回滚：只多传一个已有的 spec 键，共 4 行；不新增格式化函数、不改 spec、不改服务端；声明 number 或未声明的公式照旧显示数字。回滚即 revert 本 PR。
• 先看：ui/packages/plugin-grid/src/objectGridGenerateColumns.tsx（三处 fieldMeta.returnType）→ ui/packages/plugin-detail/src/fieldEnrichment.ts（ENRICHED_FIELD_METADATA_KEYS 加 returnType）→ 新测试 gridFormulaCurrency-27494.test.tsx。
• 两分钟验证：pnpm --dir ui exec vitest run packages/plugin-grid/src/__tests__/gridFormulaCurrency-27494.test.tsx packages/plugin-detail/src/__tests__/formulaReturnTypeRelay-27494.test.tsx，预期 13/13 通过。
• 需要你做的：无，席位落地。

What changes

FormulaCellRenderer (the table face) and FormulaField (the form face) already draw a formula declaring returnType: 'currency' through the currency field's own faces (CurrencyCellRenderer / CurrencyField, formatCurrency), reading the formula's own currencyConfig. The renderers were never reached with that key: the hosts that rebuild a field bag from a picked key list carried currencyConfig and scale but not returnType. The cell then saw an undeclared formula holding a JS number and drew the number face at scale: 2, which is the 90,000.00 the card reports beside $150,000.00.

• plugin-grid, objectGridGenerateColumns.tsx. The three configured-column paths (ListColumn objects, a string column list, inline rows with a fields projection) now copy returnType verbatim beside currencyConfig into the cell's fieldMeta. The auto-generated (object-schema) path already hands the whole def and is unchanged.
• plugin-detail, fieldEnrichment.ts. ENRICHED_FIELD_METADATA_KEYS, shared by DetailSection, HeaderHighlight and the summary chips, gains returnType, so a section entry naming the field ({ name }) reaches the formula cell with its declared type. A view entry that declares its own returnType still wins (the list's existing fill-the-gaps rule). The list is not exported from the package entry, so no published surface moves.
• No second money formatter, no spec change, no new key: returnType and currencyConfig are the spec's existing formula keys (spec: a formula field cannot declare a currency result — returnType is number | text | boolean | date, so a formula over money renders as a bare number #22727).

Tests

• New ui/packages/plugin-grid/src/__tests__/gridFormulaCurrency-27494.test.tsx, under a EUR tenant so the $ can only come from the formula's own fixed currencyConfig: on each of the three configured paths, the currency formula reads $90,000.00 beside the currency field's $150,000.00; controls, a returnType: 'number' formula at scale: 2 reads 1,234.50 and an undeclared formula reads 4,321; every formula cell reads what the whole-def FormulaCellRenderer reads; the auto-generated columns are the reference face.
• New ui/packages/plugin-detail/src/__tests__/formulaReturnTypeRelay-27494.test.tsx: the fixture is a spec-valid formula def (FieldSchema.safeParse); enrichDetailField copies returnType and a view-entry one wins; DetailSection draws $90,000.00 beside $150,000.00, and a number formula stays 1,234.50.

Acceptance notes

• Measured premise (both halves of the order's step 1). (a) The served metadata carries the keys: on a fresh examples/app-showcase boot, GET /api/v1/meta/object/showcase_project returned budget_remaining as { type: 'formula', returnType: 'currency', currencyConfig: { currencyMode: 'fixed', defaultCurrency: 'USD' }, scale: 2 }. (b) The renderers: the list cell is FormulaCellRenderer reached from objectGridGenerateColumns' configured-column paths; the related-list cell is FormulaCellRenderer reached from RelatedList's makeCell bag; the record page's field is FormulaCellRenderer reached through enrichDetailField. None goes through FormulaField, and none needed a renderer change.
• console analytics: a formula declared returnType currency renders as a bare number, a KPI tile cannot show its description, percent fields always draw a progress bar, and report charts cannot split a series by a second dimension #27430 row 1, the related-list cell, is deferred (Refs console analytics: a formula declared returnType currency renders as a bare number, a KPI tile cannot show its description, percent fields always draw a progress bar, and report charts cannot split a series by a second dimension #27430, row 1 only). RelatedList.tsx's makeCell builds the same picked-key bag and drops returnType the same way; the one-line fix is a returnType spread beside its currencyConfig: def.currencyConfig spread. That file is being rewritten by an open PR (perf(plugin-detail,app-shell): the record page's related column loads counts first, then rows only for the cards in view, and never blocks the details #27636), so the seat holds it until that lands; this PR does not pin the related list. Rows 2 to 4 of console analytics: a formula declared returnType currency renders as a bare number, a KPI tile cannot show its description, percent fields always draw a progress bar, and report charts cannot split a series by a second dimension #27430 are not touched.
• Same family, not in this diff. RecordDetailPanel's buildRecordDetailFields (the quick-look overlay) also builds its field from a picked list (currencyConfig, scale, …) without returnType. Reported to the seat with the related-list site; not changed here.
• On the live showcase the record page's layout does not place Budget Remaining, so the record-page half is pinned by the DetailSection test only.

Verification

Generated by Claude Code