fix(app-shell): a number column in an objectList config field, so a screen field's Min / Max save as numbers (objectui#11664)

[objectstack-fleet]

Fixes #11664

Clause-②: no

What changes

The flow designer edits a screen node's fields as an object list, one row per screen field. The engine's screen descriptor publishes the item properties min and max as type: 'number', but the list had no number column, so both were text cells. Authoring Min 1 / Max 10 saved min: '1', max: '10', and every run of the flow then failed at the screen node against ScreenFieldConfigSchema (z.number()). Triage's direction (comment 6007030030) is applied as ruled:

• A number kind. FlowConfigColumn['kind'] gains number. columnsFor maps a number / integer item property to it by reusing the top-level scalarField mapping, so there is no second number detector. An enum still wins and stays a select, as at the top level.
• A number cell that commits a number. The cell is the same Input type="number" control the top-level number field renders, flushed on blur like the text cell beside it. A typed 0 commits 0. An emptied cell commits no key at all: not '', null or NaN. An entry the browser cannot read as a number commits nothing, as in the top-level number field.
• Stored numbers stay numbers. toRows and rowsToList keep a stored number a number. A string already stored in a number column is kept verbatim until the author types over it. Stored data is not coerced, and the screen contract still refuses that string. Other columns' strings are unchanged.

A second reason pin 2 failed, found in the live console

Driving the real designer against an objectstack main backend showed that triage's pin 2 (re-saving a code-authored screen with min: 0 keeps 0) failed for a second reason, which neither the card nor a synchronous unit test could see. The inspector renders the hand-written fallback columns until GET /api/v1/automation/actions answers, then the engine's columns. FlowObjectListField built its rows against the first set, which has no min / max column, and rebuilt them only when the value changed. So editing any cell of the first selected screen field saved it without min and max, and without options, defaultValue, placeholder, inlineHelpText and reference. This happens on main too: there the bounds were dropped, not turned into strings.

The fix is in the same file, inside the claimed surface. Each row keeps the stored item it was read from. When the column set changes, a column the rows do not hold yet, or one whose kind changed, reads its cell from that item. Every other cell is kept, including unflushed typing. Nothing is written when the schema arrives; the author's next edit writes. The effect is keyed on a string of the columns' keys and kinds, not on the array's identity (AGENTS.md #10). The screen pins now run on the real descriptor hook with a late answer, in the order the console sees.

Tests

Final head d8002fe unless a row says otherwise. Each exit code was written to a file as it ran.

Run	Result
pnpm exec vitest run on the new FlowNodeInspector.screenNumberBounds-11664.test.tsx, the four FlowObjectListField.* suites and json-schema-to-fields.test.ts	6 files, 56 tests passed
pnpm exec vitest related --run on FlowObjectListField.tsx	113 files, 1021 tests passed
pnpm exec vitest run packages/app-shell/ at 14f9b48 (before the merge)	1012 files passed, 1 skipped; 10043 tests passed
the same at 91d45ca (after merging main f9f4a62)	1013 passed, 1 failed, 1 skipped. The failure is outside this diff; see Acceptance notes.
turbo run build --filter='@object-ui/app-shell^...', then pnpm --filter @object-ui/app-shell type-check / lint / build	exit 0 each. type-check runs tsconfig.test.json, which includes src/**/*.test.tsx. lint: 0 errors. The one warning in FlowObjectListField.tsx (react-hooks/refs at the rows map) is also on main.
check:control-bytes, check:new-line-citations, check:changeset-claims, check:pending-changeset-literals, check:vi-mock-specifiers, check:vi-mock-inherit, check:vi-mock-override-shape, check:test-path-roots, check:spec-symbols, check:designer-field-key-parity, check:i18n-keys, check:i18n-designer-parity, check:installed-pin-claims, check:unreferenced-sources, check:action-forward-parity, check:handler-key-reads, check:shell-escape-residue, check-changeset-presence.mjs, check-changeset-no-major.mjs	exit 0 each

The final suite was narrowed, and the narrowing is declared here. After 91d45ca the commits touch only FlowObjectListField.tsx, its test file and the changeset. vitest related selects every test file whose import graph reaches that file. CI runs the full suite.

Ablations. Each behaviour this PR adds was ablated at d8002fe with objectstack's scripts/ablation-replace.mjs in WRAP mode. Every leg landed on disk with its anchor going from 1 to 0, and every restore was proven: the blob equals HEAD and git diff HEAD is empty. Every leg turned the pinned file red:

Leg: what was removed	Red
columnsFor number mapping	9 of 10
a number cell reads a stored number as a number	7 of 10
a stored string coerced with Number() instead of kept	1 of 10
rowsToList commits a number	8 of 10
rowsToList keeps a stored string	1 of 10
an empty cell commits nothing	4 of 10
the cell parses its input (raw string committed instead)	4 of 10
a non-finite parse never becomes a string	1 of 10
the cell is type="number"	1 of 10
the late-column re-read	7 of 10
the kind-change half of that re-read	1 of 10

Live check. The console ran from this worktree (vite, port 5216) against objectstack main faf8dce482 (the showcase in its own worktree with --fresh, port 4116). The browser was Playwright Chromium at /opt/pw-browsers/chromium, signed in as the seeded admin with the session cookie. A flow start → screen → end was seeded through PUT /api/v1/meta/flow/NAME. The screen node was edited in the designer and published, the flow was read back through the API and run with POST /api/v1/automation/NAME/trigger. For the before leg, the three source files were swapped to their fc3c2cc bytes and restored afterwards, with the restore proven.

	Authored Min 1 / Max 10	Code-authored min: 0, max: 5, Label edited
before	cells are text inputs; saved "min":"1","max":"10"; trigger answers 400 FLOW_FAILED: "screen 'ask': config does not satisfy the screen contract — config.fields[0].min: Invalid input: expected number, received string; config.fields[0].max: …"	cells empty; saved without min / max; trigger pauses at the screen with no bounds
after	cells are number inputs; saved "min":1,"max":10; trigger pauses at screen ask with min: 1, max: 10	cells show 0 and 5; saved "min":0,"max":5; trigger pauses with min: 0, max: 5

The configSchema this backend served for screen equals the test's transcribed SCREEN_CONFIG_SCHEMA, compared as canonical JSON.

Clause-② fence. On the final build, a walk of relative imports from packages/app-shell/dist/index.d.ts reaches 164 declaration files, 22 of them under views/metadata-admin. None is flow-node-config.d.ts, FlowObjectListField.d.ts, json-schema-to-fields.d.ts or the inspectors' index.d.ts. The built flow-node-config.d.ts carries the new 'number' member. No export, entry-reachable prop or type member, or language-pack key is added.

Acceptance notes

• Zone 2 readings.
  • 1. Confirmed as the PM read it, and the existing mapping is reused.
  • 2. The top-level control, InspectorNumberField, is a label above an Input type="number" that commits on every keystroke. The cell reuses that input and its empty-commits-nothing rule, but not the wrapper: the wrapper's label would duplicate the row label, and its per-keystroke commits differ from the row's flush on blur. No i18n key is added; the placeholder is the column's schema description, as for every other cell.
  • 3. An empty cell is an absent key, and 0 is 0. A browser number input reports '' for an entry it cannot read, so that entry commits nothing; typed over a stored number, it removes that number on blur, exactly as the top-level field does.
  • 4. A stored string in a number column is handed to the input as-is. A numeric string such as '1' shows as 1; a non-numeric one shows a blank box. The row keeps the string either way until typed over (pinned).
  • 5. The engine source is pinned with the transcribed descriptor. The offline table lists fields columns (Name, Label, Type, Required, Visible when) but no min / max, so it has no number cell.
  • 6. Measured; see the live check above.
• Observations, not filed (no public door measured):
  • Offline table. When no configSchema is served, a re-save of a screen's fields drops every key the hand-written columns do not list: min, max, options, defaultValue, placeholder, inlineHelpText, reference. Measured with a scratch test at 14f9b48. A backend that publishes the screen descriptor leaves that table once its answer lands. Carrier: none.
  • Untyped properties are text columns. The descriptor publishes options[].value and defaultValue with no type, so they are text columns, and a stored number there still saves as a string through String(v). Both are z.unknown() in the spec, so nothing refuses the string. This is outside the ruling, which covers number columns. Carrier: none.
  • A full-suite failure after the merge. At 91d45ca, AppContent.deniedVsUnpublished.test.tsx ("that way back follows the DECLARED landing") failed once on a network-escape guard: reads of /api/v1/meta, /meta/package and /meta/diagnostics under full parallel load. Alone it passes 15/15. This diff does not touch that file. main's f9f4a62 changed HomePage.tsx. Carrier: none.
• Not in this PR. No change in objectstack. objectstack#21898's save-time judge is that card's own business. Triage's enumeration pin for a third face of the family belongs to a later card.

---

Generated by Claude Code

[github-actions]

✅ Console Performance Budget

Metric	Value	Budget
Eager closure (gzip, 331 chunks)	3319.4 KB	3330.4 KB
Main entry chunk (gzip)	152.4 KB	350 KB
Entry file	index--2JUSFG8.js	—
Status	PASS	—

The eager closure is every chunk the entry reaches through static imports — what the browser fetches and parses before the app renders. The entry chunk on its own is a small fraction of it.

---

📦 Bundle Size Report

Package	Size	Gzipped
app-shell (consoleActionDispatch.js)	0.20KB	0.19KB
app-shell (index.js)	17.22KB	6.37KB
app-shell (runtime-config.js)	22.52KB	7.86KB
app-shell (types.js)	0.01KB	0.04KB
app-shell (urlParams.js)	10.11KB	3.87KB
auth (ActiveOrganizationStorage.js)	27.95KB	10.04KB
auth (AuthContext.js)	0.31KB	0.24KB
auth (AuthGuard.js)	2.07KB	1.00KB
auth (AuthProvider.js)	40.22KB	10.61KB
auth (AuthShell.js)	3.49KB	1.40KB
auth (ForgotPasswordForm.js)	12.21KB	3.45KB
auth (LoginForm.js)	18.15KB	5.40KB
auth (PreviewBanner.js)	0.90KB	0.50KB
auth (RegisterForm.js)	6.72KB	2.24KB
auth (SocialSignInButtons.js)	9.70KB	3.93KB
auth (UserMenu.js)	3.39KB	1.21KB
auth (auth-gate-events.js)	1.29KB	0.66KB
auth (authStyles.js)	5.04KB	1.72KB
auth (createAuthClient.js)	40.70KB	10.94KB
auth (createAuthenticatedFetch.js)	8.54KB	3.46KB
auth (index.js)	3.63KB	1.64KB
auth (invitation-status.js)	1.22KB	0.70KB
auth (org-roles.js)	6.66KB	2.78KB
auth (phone-identifier.js)	1.11KB	0.66KB
auth (types.js)	0.59KB	0.35KB
auth (useAuth.js)	5.30KB	1.02KB
auth (useWorkspaceAdminStatus.js)	11.08KB	4.58KB
collaboration (CommentThread.js)	27.11KB	7.97KB
collaboration (LiveCursors.js)	3.17KB	1.27KB
collaboration (PresenceAvatars.js)	6.28KB	2.60KB
collaboration (PresenceProvider.js)	2.79KB	1.13KB
collaboration (index.js)	1.68KB	0.73KB
collaboration (useCollaborationTranslation.js)	6.50KB	2.52KB
collaboration (useCommentSearch.js)	1.98KB	0.88KB
collaboration (useConflictResolution.js)	7.75KB	1.86KB
collaboration (useMentionNotifications.js)	1.81KB	0.68KB
collaboration (usePresence.js)	6.33KB	1.84KB
collaboration (useRealtimeSubscription.js)	7.91KB	2.01KB
components (index.js)	574.72KB	137.94KB
core (index.js)	10.00KB	3.96KB
create-plugin (index.js)	27.94KB	9.51KB
data-objectstack (index.js)	232.57KB	64.51KB
fields (index.js)	262.75KB	66.62KB
i18n (LocalizationContext.js)	1.76KB	0.96KB
i18n (builtinAggregateLabels.js)	0.86KB	0.49KB
i18n (currency.js)	2.59KB	1.22KB
i18n (fallbackInterpolation.js)	6.25KB	2.77KB
i18n (i18n.js)	8.87KB	3.64KB
i18n (index.js)	5.24KB	2.27KB
i18n (pickLocalized.js)	9.86KB	3.95KB
i18n (provider.js)	39.35KB	12.88KB
i18n (translateFn.js)	0.20KB	0.18KB
i18n (useDisplayLocale.js)	3.52KB	1.76KB
i18n (useObjectLabel.js)	35.66KB	9.49KB
i18n (useSafeTranslation.js)	7.14KB	2.92KB
layout (index.js)	41.18KB	11.71KB
mobile (MobileProvider.js)	0.92KB	0.49KB
mobile (ResponsiveContainer.js)	0.94KB	0.38KB
mobile (breakpoints.js)	1.51KB	0.70KB
mobile (createOfflineDataSource.js)	5.61KB	1.75KB
mobile (index.js)	1.99KB	0.87KB
mobile (offlineQueue.js)	3.91KB	1.35KB
mobile (pwa.js)	0.97KB	0.49KB
mobile (serviceWorker.js)	1.48KB	0.62KB
mobile (serviceWorkerSource.js)	3.41KB	1.48KB
mobile (useBreakpoint.js)	1.54KB	0.65KB
mobile (useGesture.js)	6.96KB	1.98KB
mobile (useOfflineSync.js)	1.99KB	0.72KB
mobile (usePullToRefresh.js)	6.62KB	2.45KB
mobile (useResponsive.js)	0.72KB	0.42KB
mobile (useSpecGesture.js)	5.52KB	2.10KB
mobile (useTouchTarget.js)	1.01KB	0.54KB
permissions (MePermissionsProvider.js)	13.86KB	5.00KB
permissions (PermissionContext.js)	0.31KB	0.25KB
permissions (PermissionGuard.js)	0.89KB	0.45KB
permissions (PermissionProvider.js)	6.52KB	2.26KB
permissions (discardProofCache.js)	1.04KB	0.55KB
permissions (evaluator.js)	8.33KB	3.07KB
permissions (index.js)	0.93KB	0.41KB
permissions (store.js)	0.91KB	0.42KB
permissions (useFieldPermissions.js)	1.28KB	0.53KB
permissions (usePermissions.js)	4.83KB	2.27KB
plugin-ai (index.js)	16.04KB	3.92KB
plugin-calendar (index.js)	53.17KB	15.46KB
plugin-charts (index.js)	84.26KB	23.05KB
plugin-chatbot (index.js)	198.39KB	47.02KB
plugin-dashboard (index.js)	143.75KB	38.87KB
plugin-designer (index.js)	231.41KB	48.84KB
plugin-detail (index.js)	247.23KB	65.04KB
plugin-editor (index.js)	2.23KB	1.05KB
plugin-form (index.js)	177.09KB	45.89KB
plugin-gantt (index.js)	179.16KB	45.06KB
plugin-grid (index.js)	238.48KB	65.51KB
plugin-kanban (index.js)	52.17KB	16.37KB
plugin-list (index.js)	116.72KB	29.10KB
plugin-map (index.js)	25.60KB	8.62KB
plugin-markdown (index.js)	13.88KB	4.80KB
plugin-report (index.js)	44.12KB	12.29KB
plugin-timeline (index.js)	38.80KB	11.71KB
plugin-tree (index.js)	14.51KB	5.15KB
plugin-view (index.js)	90.23KB	22.73KB
providers (DataSourceProvider.js)	0.75KB	0.39KB
providers (MetadataProvider.js)	1.37KB	0.59KB
providers (ThemeProvider.js)	1.90KB	0.85KB
providers (UploadProvider.js)	11.81KB	3.58KB
providers (index.js)	0.45KB	0.23KB
providers (types.js)	0.01KB	0.04KB
react-runtime (index.js)	5.62KB	2.34KB
react (LazyPluginLoader.js)	4.47KB	1.63KB
react (SchemaRenderer.js)	120.63KB	39.56KB
react (data-invalidation.js)	5.05KB	2.08KB
react (index.js)	4.50KB	2.06KB
react (schema-input.js)	4.31KB	2.07KB
react (spec-input.js)	0.20KB	0.18KB
sdui-parser (body-dialect.js)	4.50KB	1.99KB
sdui-parser (codegen.js)	9.45KB	3.76KB
sdui-parser (dashboard-widget-options.js)	3.08KB	1.30KB
sdui-parser (index.js)	7.30KB	3.12KB
sdui-parser (input-type.js)	2.84KB	1.40KB
sdui-parser (parse.js)	25.28KB	7.80KB
sdui-parser (provenance.js)	3.84KB	1.90KB
sdui-parser (types.js)	0.28KB	0.23KB
sdui-parser (validate.js)	23.87KB	7.83KB
types (ai.js)	4.39KB	2.17KB
types (api-types.js)	0.20KB	0.18KB
types (app.js)	4.12KB	1.61KB
types (authoring-nodes.js)	0.20KB	0.19KB
types (base.js)	0.20KB	0.18KB
types (blocks.js)	0.20KB	0.18KB
types (cloud.js)	0.20KB	0.18KB
types (complex.js)	4.44KB	2.07KB
types (crud.js)	0.20KB	0.18KB
types (dashboard-filter-alias.js)	6.23KB	2.74KB
types (dashboard-widget-layout.js)	2.06KB	0.96KB
types (data-display.js)	3.75KB	1.85KB
types (data-protocol.js)	0.20KB	0.19KB
types (data.js)	0.20KB	0.18KB
types (designer.js)	1.85KB	0.85KB
types (disclosure.js)	0.20KB	0.18KB
types (error-code.js)	1.54KB	0.88KB
types (expression.js)	0.20KB	0.18KB
types (feedback.js)	0.20KB	0.18KB
types (field-types.js)	1.13KB	0.65KB
types (form.js)	0.20KB	0.18KB
types (http-inflight.js)	8.87KB	3.73KB
types (http-retry.js)	4.32KB	2.02KB
types (icon-key-migration.js)	4.26KB	1.63KB
types (index.js)	5.78KB	2.70KB
types (layout.js)	0.20KB	0.18KB
types (managed-by.js)	0.19KB	0.18KB
types (mobile.js)	5.00KB	2.39KB
types (navigation.js)	0.20KB	0.18KB
types (node-slots.js)	7.18KB	2.34KB
types (objectql.js)	0.20KB	0.18KB
types (overlay.js)	0.20KB	0.18KB
types (permissions.js)	2.52KB	1.31KB
types (plugin-scope.js)	0.20KB	0.18KB
types (record-components.js)	0.20KB	0.19KB
types (record-semantics.js)	1.28KB	0.67KB
types (registry.js)	0.20KB	0.18KB
types (reports.js)	0.20KB	0.18KB
types (select-option.js)	0.20KB	0.19KB
types (spec-report.js)	4.99KB	1.96KB
types (spec-ui-namespace.js)	0.20KB	0.19KB
types (strict-authoring-face.js)	19.93KB	7.25KB
types (system-fields.js)	3.33KB	1.54KB
types (theme.js)	6.28KB	2.87KB
types (ui-action.js)	8.11KB	3.32KB
types (views.js)	0.20KB	0.18KB
types (widget.js)	0.20KB	0.18KB

Size Limits

• ✅ Core packages should be < 50KB gzipped
• ✅ Component packages should be < 100KB gzipped
• ⚠️ Plugin packages should be < 150KB gzipped

[objectstack-fleet]

ACCEPT: PR objectui#11670, head d8002fe. It lands when every check on this head is green

domain:ui execution seat 1 @ objectui · session_015W8GBu6sBiqus2L2xjMsAL (os-steve) · 2026-10-06T04:10Z. Reviewed against GitHub and origin/main, not against the report's prose (report on objectui#11664).

• Shape.
  • The PR is a draft against main, and its assignee is os-steve. Its first line is Fixes #11664, the only line with a closing keyword next to an issue number. Clause-②: no is on its own line.
  • The diff is 5 files, +545/−14, inside the claim's surface: json-schema-to-fields.ts, FlowObjectListField.tsx, the FlowConfigColumn['kind'] union in flow-node-config.ts, one pin file and the changeset.
  • main has moved since its one merge (91d45ca); the merge queue tests it against the current base.
• The ruling, as written (triage 6007030030).
  • columnsFor maps a number/integer item property to the new number kind through the existing scalarField check, not a second detector.
  • The cell is the same Input type="number" the top-level field uses, flushed on blur. A typed 0 commits 0, an empty cell commits no key, and a non-numeric entry commits nothing.
  • toRows and rowsToList keep a stored number a number.
  • A string stored in a number column is kept verbatim until the author types over it, and other columns' strings are untouched. ⛔ No coercion of stored data, as the dispatch asked.
• One fix beyond Zone 3, inside the claimed file, accepted (19c1e69).
  • The inspector renders the hand-written fallback columns until the engine's configSchema answers. Rows built against them held no cell for min/max (or options, defaultValue, placeholder, inlineHelpText, reference), so the first edit of a code-authored screen field dropped those keys. The dev saw this live.
  • Each row now keeps its stored item, and a column that arrives later, or changes kind, reads its cell from it. Nothing is written when the schema arrives.
  • Triage's second pin ("re-saving a code-authored screen with min: 0 keeps 0") does not hold in the product without it. It is pinned, ablated (7 red, and 1 for the kind-change half), and stated in the changeset.
• Ablation. 11 legs, each red and each restore proven. These include the number mapping (9 red), the cell keeping a stored number (7), rowsToList committing a number (8), and empty-commits-nothing (4).
• Live (objectstack main faf8dce482, showcase, engine descriptors).
  • Before: Min 1 / Max 10 saved as strings, and the trigger answered 400 FLOW_FAILED (config.fields[0].min: … expected number, received string). A code-authored min 0/max 5 was dropped on a Label edit.
  • After: numbers are saved, the code-authored bounds survive the edit, and both runs pause at the screen with the bounds.
• Clause-②: no, measured. The walk of dist/index.d.ts reaches 164 declaration files, 22 of them under views/metadata-admin as the positive control. It reaches no inspector module, while the built flow-node-config.d.ts carries the new number member.
• Gates. app-shell type-check, lint and build exit 0 at d8002fe, as do the touched check:* set and both changeset checks.
  • The final suite at d8002fe was the declared vitest related narrowing: 113 files and 1021 tests green, since the commits after the merge touch only FlowObjectListField.tsx, its test and the changeset.
  • The full app-shell suite was green at 14f9b48. At 91d45ca it had one red, AppContent.deniedVsUnpublished.test.tsx ("that way back follows the DECLARED landing", the network-escape guard under full load), which passed 15/15 alone. This diff does not touch it.
  • That is the shape of the closed objectui#10162, and this seat saw it on 2026-10-05 too. CI's full farm on this head is the binding reading.
• Changeset, sentence by sentence. It is @object-ui/app-shell: patch. The number column, the commit rules, the late-column re-read, the no-coercion line, the "every published number column" scope and the Clause-②: no line match the diff.

Left as noted, not filed (from the report):

• With no served configSchema, the built-in screen form has no min/max column. A re-save there drops keys the hand-written columns do not list. That path is offline-only.
• The screen descriptor publishes options[].value and defaultValue untyped, so a stored number there still saves as a string through the text cell. The spec types both as z.unknown(), so nothing refuses it, and it is outside this ruling.
• The AppContent.deniedVsUnpublished red under full load is a possible recurrence of the closed objectui#10162. The seat carries it to its round report.

Landing: on all-green checks on this head, ready, then auto-merge into the merge queue.

---

Generated by Claude Code