@asteby/metacore-runtime-react 39.2.4 → 39.2.6

package/dist/validator.js

@@ -259,3 +259,35 @@ function walk(fields, values, prefix, bag) {
 export function bagHasErrors(bag) {
     return Object.keys(bag).length > 0;
 }
+/** Issue codes produced by a field's declarative rule (regex / min / max /
+ *  custom builtins) — the ones the kernel grandfathers on update. */
+const RULE_CODES = new Set(['regex', 'min', 'max', 'email', 'uuid', 'url', 'numeric', 'integer', 'custom']);
+function sameValue(a, b) {
+    if (isEmpty(a) && isEmpty(b))
+        return true;
+    if (isNumeric(a) && isNumeric(b))
+        return numericValue(a) === numericValue(b);
+    return asString(a) === asString(b);
+}
+/**
+ * Edit mode: drop the rule issues (regex/min/max/custom) of a top-level field
+ * whose value is the one already persisted. The edit form posts every field
+ * back, so a rule added after the row was written (an RFC pattern over legacy
+ * data) would otherwise block saving a record over a field nobody touched.
+ * Mirrors the kernel (dynamic.validateWrite grandfathers unchanged values on
+ * update). required / invalid_option / invalid_type are kept.
+ */
+export function exemptUnchangedRuleIssues(bag, values, persisted) {
+    if (!persisted)
+        return bag;
+    const out = {};
+    for (const [path, issues] of Object.entries(bag)) {
+        const unchanged = !path.includes('.') &&
+            Object.prototype.hasOwnProperty.call(persisted, path) &&
+            sameValue(values?.[path], persisted[path]);
+        const kept = unchanged ? issues.filter(i => !RULE_CODES.has(String(i.code))) : issues;
+        if (kept.length)
+            out[path] = kept;
+    }
+    return out;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@asteby/metacore-runtime-react",
-  "version": "39.2.4",
+  "version": "39.2.6",
   "description": "React runtime for metacore hosts — renders addon contributions dynamically",
   "repository": {
     "type": "git",
@@ -27,7 +27,7 @@
   },
   "peerDependencies": {
     "@asteby/metacore-sdk": "^3.8.0",
-    "@asteby/metacore-ui": "^2.18.1",
+    "@asteby/metacore-ui": "^2.18.2",
     "@tanstack/react-query": ">=5",
     "@tanstack/react-router": ">=1.100",
     "@tanstack/react-table": ">=8",
@@ -51,7 +51,7 @@
   },
   "devDependencies": {
     "@asteby/metacore-sdk": "3.8.0",
-    "@asteby/metacore-ui": "2.18.1",
+    "@asteby/metacore-ui": "2.18.2",
     "@tanstack/react-query": "^5.95.0",
     "@tanstack/react-router": "^1.168.0",
     "@tanstack/react-table": "^8.20.0",

package/src/__tests__/use-options-resolver.test.ts

@@ -1,5 +1,5 @@
 import { afterEach, describe, it, expect, vi } from 'vitest'
-import { projectOption } from '../use-options-resolver'
+import { optionsRequestKey, projectOption } from '../use-options-resolver'
 import {
     resolveValidatorToken,
     setOrgConfigBridge,
@@ -11,6 +11,14 @@ import {
 // hook performs) so consumers can rely on the v0.9.0 envelope reading
 // without spinning up a renderer.
 
+describe('optionsRequestKey', () => {
+    it('keeps two orgs from sharing a column cache', () => {
+        const a = optionsRequestKey('org-a|branch', '/options/Product', 'id', '', 200, undefined)
+        const b = optionsRequestKey('org-b|branch', '/options/Product', 'id', '', 200, undefined)
+        expect(a).not.toBe(b)
+    })
+})
+
 describe('projectOption', () => {
     it('mirrors id into value and label into name when missing', () => {
         const out = projectOption({ id: 'abc', label: 'Hello' })

package/src/__tests__/validator.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest'
-import { parseRuleString, checkValue, validateValues, bagHasErrors } from '../validator'
+import { parseRuleString, checkValue, validateValues, bagHasErrors, exemptUnchangedRuleIssues } from '../validator'
 import type { ActionFieldDef } from '../types'
 
 describe('parseRuleString', () => {
@@ -68,3 +68,30 @@ describe('validateValues', () => {
         expect(bag.email?.[0]?.code).toBe('email')
     })
 })
+
+describe('exemptUnchangedRuleIssues (QA LIVE-19)', () => {
+    const fields: ActionFieldDef[] = [
+        { key: 'name', label: 'Nombre', type: 'text', required: true } as ActionFieldDef,
+        { key: 'tax_id', label: 'RFC', type: 'text', validation: { regex: '^[A-Z]{3,4}[0-9]{6}[A-Z0-9]{3}$' } } as ActionFieldDef,
+        { key: 'price', label: 'Precio', type: 'number', validation: { min: 0 } } as ActionFieldDef,
+    ]
+    it('keeps an untouched legacy value from blocking the edit', () => {
+        const values = { name: 'Renombrado', tax_id: 'abc-010101', price: '-5' }
+        const bag = validateValues(fields, values)
+        expect(bagHasErrors(bag)).toBe(true)
+        const out = exemptUnchangedRuleIssues(bag, values, { name: 'Viejo', tax_id: 'abc-010101', price: -5 })
+        expect(bagHasErrors(out)).toBe(false)
+    })
+    it('still reports a changed invalid value and required', () => {
+        const values = { name: '', tax_id: 'xyz', price: -6 }
+        const out = exemptUnchangedRuleIssues(validateValues(fields, values), values, { name: 'Viejo', tax_id: 'abc-010101', price: -5 })
+        expect(out.tax_id?.[0]?.code).toBe('regex')
+        expect(out.price?.[0]?.code).toBe('min')
+        expect(out.name?.[0]?.code).toBe('required')
+    })
+    it('is a no-op without a persisted record (create)', () => {
+        const values = { name: 'x', tax_id: 'abc' }
+        const bag = validateValues(fields, values)
+        expect(exemptUnchangedRuleIssues(bag, values, null)).toEqual(bag)
+    })
+})

package/src/activity-value-renderer.tsx

@@ -11,7 +11,7 @@
  */
 
 import * as React from 'react'
-import * as icons from 'lucide-react'
+import { Calendar, Check, ExternalLink, Mail, Minus } from 'lucide-react'
 import { es, enUS } from 'date-fns/locale'
 import {
     Badge,
@@ -231,7 +231,7 @@ export const ActivityValueRenderer: React.FC<ActivityValueRendererProps> = ({
                 className="inline-flex items-center gap-1 text-sm text-muted-foreground"
                 title={formatted.title}
             >
-                <icons.Calendar className="h-3 w-3 opacity-60" />
+                <Calendar className="h-3 w-3 opacity-60" />
                 {formatted.display}
             </span>
         )
@@ -245,9 +245,9 @@ export const ActivityValueRenderer: React.FC<ActivityValueRendererProps> = ({
         return (
             <span className="inline-flex items-center gap-1">
                 {value ? (
-                    <icons.Check className="h-3.5 w-3.5 text-green-500" />
+                    <Check className="h-3.5 w-3.5 text-green-500" />
                 ) : (
-                    <icons.Minus className="h-3.5 w-3.5 text-muted-foreground" />
+                    <Minus className="h-3.5 w-3.5 text-muted-foreground" />
                 )}
                 <span className="text-sm text-muted-foreground">{value ? 'Sí' : 'No'}</span>
             </span>
@@ -350,7 +350,7 @@ export const ActivityValueRenderer: React.FC<ActivityValueRendererProps> = ({
                 className="inline-flex items-center gap-1 text-sm text-primary hover:underline"
                 onClick={(e) => e.stopPropagation()}
             >
-                <icons.ExternalLink className="h-3 w-3 shrink-0" />
+                <ExternalLink className="h-3 w-3 shrink-0" />
                 <span className="truncate" style={{ maxWidth: 200 }}>{label}</span>
             </a>
         )
@@ -368,7 +368,7 @@ export const ActivityValueRenderer: React.FC<ActivityValueRendererProps> = ({
                 className="inline-flex items-center gap-1 text-sm text-primary hover:underline"
                 onClick={(e) => e.stopPropagation()}
             >
-                <icons.Mail className="h-3 w-3 shrink-0" />
+                <Mail className="h-3 w-3 shrink-0" />
                 <span className="truncate" style={{ maxWidth: 200 }}>{email}</span>
             </a>
         )

package/src/dialogs/dynamic-record.tsx

@@ -67,7 +67,7 @@ import { AssistInterview } from '../assist-interview'
 import { FieldCell } from '../field-grid'
 import { isNilUuid, normalizeNilUuid } from '../nil-uuid'
 import { normalizeRefFieldsForSubmit } from './normalize-submit'
-import { validateValues, bagHasErrors } from '../validator'
+import { validateValues, bagHasErrors, exemptUnchangedRuleIssues } from '../validator'
 import { DynamicIcon, isLucideIconName } from '../dynamic-icon'
 import { IconPickerField } from '../icon-picker-field'
 import { humanizeToken } from '../dynamic-columns-helpers'
@@ -868,7 +868,17 @@ export function DynamicRecordDialog({
             // Laravel-style: collect every issue from the shared validator
             // (required + rule strings / min/max / email…) on visible fields only.
             const visible = filterVisibleFields(modalMeta.fields, mode, formValues)
-            const bag = validateValues(visible as ActionFieldDef[], formValues)
+            let bag = validateValues(visible as ActionFieldDef[], formValues)
+            // Edit: a legacy value re-sent unchanged is not re-judged by a rule
+            // added after it was written (same grandfathering as the kernel).
+            if (mode === 'edit' && record) {
+                const persisted: Record<string, unknown> = {}
+                for (const f of visible) {
+                    const v = resolvePath(record, f.key)
+                    if (v !== undefined) persisted[f.key] = v
+                }
+                bag = exemptUnchangedRuleIssues(bag, formValues, persisted)
+            }
             if (bagHasErrors(bag)) {
                 const labels: Record<string, string> = {}
                 for (const f of visible) labels[f.key] = f.label
@@ -1077,6 +1087,13 @@ export function DynamicRecordDialog({
                                 blow the two columns past the dialog width. */}
                             <form
                                 id={formId}
+                                // Stable Aby / guidance contract: unique id={formId}
+                                // keeps nested create footers from submitting the
+                                // parent, while data-aby-* lets tours find the
+                                // open form without a hardcoded id.
+                                data-aby-record-form=""
+                                data-aby-model={model}
+                                data-aby-mode={mode}
                                 onSubmit={handleSubmit}
                                 className="grid gap-y-4"
                             >
@@ -1264,7 +1281,7 @@ function FieldRow({ field, record, value, mode, onChange, error, locked }: Field
     const isEditReadonly = (mode === 'edit' && !!field.readonly) || !!locked
 
     return (
-        <div className="flex flex-col gap-1.5">
+        <div className="flex flex-col gap-1.5" data-aby-field={field.key}>
             <Label className="text-xs font-medium text-muted-foreground uppercase tracking-wide">
                 {field.label}
                 {field.required && mode !== 'view' && !isEditReadonly && (

package/src/dynamic-columns.tsx

@@ -16,8 +16,7 @@ import * as React from 'react'
 import { ColumnDef } from '@tanstack/react-table'
 import { format, type Locale } from 'date-fns'
 import { es, enUS } from 'date-fns/locale'
-import * as icons from 'lucide-react'
-import { MoreHorizontal } from 'lucide-react'
+import { Calendar, Check, Copy, Mail, Minus, MoreHorizontal } from 'lucide-react'
 import {
     Avatar,
     AvatarFallback,
@@ -198,9 +197,9 @@ const CodeCell: React.FC<{ text: string; maxLength?: number }> = ({ text, maxLen
                 title="Copiar"
             >
                 {copied ? (
-                    <icons.Check className="h-3.5 w-3.5 text-green-500" />
+                    <Check className="h-3.5 w-3.5 text-green-500" />
                 ) : (
-                    <icons.Copy className="h-3.5 w-3.5" />
+                    <Copy className="h-3.5 w-3.5" />
                 )}
             </button>
         </div>
@@ -1082,7 +1081,7 @@ export function makeDefaultGetDynamicColumns(
                                     className="flex items-center gap-1.5 text-muted-foreground"
                                     title={formatted.title}
                                 >
-                                    <icons.Calendar className="h-3.5 w-3.5 opacity-70" />
+                                    <Calendar className="h-3.5 w-3.5 opacity-70" />
                                     <span className="text-sm font-medium">
                                         {formatted.display}
                                     </span>
@@ -1174,7 +1173,7 @@ export function makeDefaultGetDynamicColumns(
                                     className="inline-flex items-center gap-1.5 text-sm font-medium text-primary hover:underline"
                                     onClick={(e) => e.stopPropagation()}
                                 >
-                                    <icons.Mail className="h-3.5 w-3.5 shrink-0 opacity-70" />
+                                    <Mail className="h-3.5 w-3.5 shrink-0 opacity-70" />
                                     <span className="truncate max-w-[260px]">{email}</span>
                                 </a>
                             )
@@ -1337,9 +1336,9 @@ export function makeDefaultGetDynamicColumns(
                             return (
                                 <span className="inline-flex items-center gap-1.5">
                                     {value ? (
-                                        <icons.Check className="h-4 w-4 text-green-500" />
+                                        <Check className="h-4 w-4 text-green-500" />
                                     ) : (
-                                        <icons.Minus className="h-4 w-4 text-muted-foreground" />
+                                        <Minus className="h-4 w-4 text-muted-foreground" />
                                     )}
                                     {showText && (
                                         <span className="text-sm text-muted-foreground">

package/src/dynamic-icon.tsx

@@ -1,18 +1,48 @@
-// Minimal DynamicIcon — resolves a lucide-react icon by name. Used across
-// action modals and the default row-action menus. Hosts that need custom
-// icon sets can override by shadowing this component via their own prop.
-import * as icons from 'lucide-react'
+// DynamicIcon resolves a lucide glyph by name without pulling the full icon
+// set into the shared chunk. Each glyph is its own dynamic import.
+import { lazy, Suspense, useMemo, type ComponentType } from 'react'
+import dynamicIconImports from 'lucide-react/dynamicIconImports'
 
 export interface DynamicIconProps {
     name: string
     className?: string
 }
 
+type IconName = keyof typeof dynamicIconImports
+
+function pascalToKebab(pascal: string): string {
+    return pascal
+        .replace(/([a-z0-9])([A-Z])/g, '$1-$2')
+        .replace(/([A-Z])([A-Z][a-z])/g, '$1-$2')
+        .toLowerCase()
+}
+
+function kebabToPascal(kebab: string): string {
+    return kebab
+        .split('-')
+        .map((part) => (part ? part.charAt(0).toUpperCase() + part.slice(1) : ''))
+        .join('')
+}
+
+function loaderFor(pascal: string) {
+    const kebab = pascalToKebab(pascal) as IconName
+    return dynamicIconImports[kebab]
+}
+
 export function DynamicIcon({ name, className }: DynamicIconProps) {
-    const resolved = resolveLucideIconName(name) ?? name
-    const Icon = (icons as unknown as Record<string, React.ComponentType<{ className?: string }>>)[resolved]
+    const resolved = resolveLucideIconName(name)
+    const Icon = useMemo(() => {
+        if (!resolved) return null
+        const loader = loaderFor(resolved)
+        if (!loader) return null
+        return lazy(loader as () => Promise<{ default: ComponentType<{ className?: string }> }>)
+    }, [resolved])
     if (!Icon) return null
-    return <Icon className={className} />
+    return (
+        <Suspense fallback={null}>
+            <Icon className={className} />
+        </Suspense>
+    )
 }
 
 // resolveLucideIconName — canonical PascalCase lucide name for a value that is
@@ -24,13 +54,10 @@ export function resolveLucideIconName(value: unknown): string | null {
     if (/[/\\.:\s]/.test(value)) return null
     let name = value
     if (/^[a-z0-9]+(-[a-z0-9]+)*$/.test(value)) {
-        name = value
-            .split('-')
-            .map((p) => p.charAt(0).toUpperCase() + p.slice(1))
-            .join('')
+        name = kebabToPascal(value)
     }
     if (!/^[A-Z][A-Za-z0-9]*$/.test(name)) return null
-    return (icons as unknown as Record<string, unknown>)[name] ? name : null
+    return loaderFor(name) ? name : null
 }
 
 // isLucideIconName — true when a string is a lucide-react icon name
@@ -43,3 +70,8 @@ export function resolveLucideIconName(value: unknown): string | null {
 export function isLucideIconName(value: unknown): value is string {
     return resolveLucideIconName(value) !== null
 }
+
+/** PascalCase names of every lucide glyph, for the icon picker grid. */
+export function lucideIconNames(): string[] {
+    return Object.keys(dynamicIconImports).map(kebabToPascal)
+}

package/src/icon-picker-field.tsx

@@ -12,8 +12,8 @@
 // initial mode is inferred from the current value — path-like ("/" or ".")
 // means image, anything else means icon.
 import { useMemo, useState } from 'react'
-import { icons } from 'lucide-react'
 import { ChevronsUpDown } from 'lucide-react'
+import { lucideIconNames } from './dynamic-icon'
 import {
     Button,
     Input,
@@ -35,7 +35,7 @@ export interface IconPickerFieldProps {
  * the DOM light: the lucide catalog is ~1500 glyphs, so we never mount them all. */
 const PAGE = 60
 
-const ALL_ICON_NAMES = Object.keys(icons)
+const ALL_ICON_NAMES = lucideIconNames()
 
 /** True when a stored value looks like an image url/path rather than an icon name. */
 export function looksLikeImageValue(value: unknown): boolean {

package/src/index.ts

@@ -38,6 +38,7 @@ export {
     checkValue,
     validateValues,
     bagHasErrors,
+    exemptUnchangedRuleIssues,
     type ValidationSpec,
 } from './validator'
 export { VALIDATION_CATALOGS, validationCatalog, validationMessageKey } from './validation-catalog'

package/src/use-options-resolver.ts

@@ -100,11 +100,58 @@ export interface UseOptionsResolverResult {
  * `/api/options/:model?field=…` endpoint. Returns the v0.9.0 envelope
  * `{ data, meta: { type, count } }` projected into a stable shape.
  *
- * The hook is intentionally minimal: it does NOT debounce `query`
- * (callers should hold the controlled value and pass it post-debounce)
- * and does NOT cache across hook instances (apps that need shared state
- * compose this with TanStack Query in their own layer).
+ * The hook does NOT debounce `query` (callers should pass it post-debounce).
+ * Identical lookups share one in-flight request and a 30s cache, so a column
+ * of reference cells does not fan out one request per row. The key includes
+ * the org and branch from the host session.
  */
+const OPTIONS_TTL_MS = 30_000
+
+type OptionsPayload = { options: ResolvedOption[]; meta: OptionsMeta }
+
+const optionsCache = new Map<string, { payload: OptionsPayload; at: number }>()
+const optionsInflight = new Map<string, Promise<OptionsPayload>>()
+const optionsEpoch = new Map<string, number>()
+
+export function optionsRequestKey(
+    scope: string,
+    url: string,
+    field: string,
+    query: string,
+    limit: number | undefined,
+    filter: string | undefined,
+): string {
+    return [scope, url, field, query, String(limit ?? ''), filter ?? ''].join('\n')
+}
+
+function optionsScope(): string {
+    if (typeof localStorage === 'undefined') return ''
+    let org = ''
+    try {
+        const raw = localStorage.getItem('auth_user')
+        if (raw) org = String((JSON.parse(raw) as { organization_id?: string }).organization_id ?? '')
+    } catch {
+        org = ''
+    }
+    const branch = localStorage.getItem('active_branch_id') ?? ''
+    return `${org}|${branch}`
+}
+
+function readOptionsEnvelope(body: any): OptionsPayload {
+    const rawOptions: any[] = Array.isArray(body?.data) ? body.data : []
+    const metaPayload =
+        body?.meta && typeof body.meta === 'object'
+            ? body.meta
+            : { type: body?.type, count: rawOptions.length }
+    return {
+        options: rawOptions.map(projectOption),
+        meta: {
+            type: metaPayload?.type ?? 'dynamic',
+            count: typeof metaPayload?.count === 'number' ? metaPayload.count : rawOptions.length,
+        },
+    }
+}
+
 export function useOptionsResolver(args: UseOptionsResolverArgs): UseOptionsResolverResult {
     const {
         modelKey,
@@ -146,9 +193,7 @@ export function useOptionsResolver(args: UseOptionsResolverArgs): UseOptionsReso
         return ''
     }, [fieldKey, ref])
 
-    // Track the in-flight controller so a new fetch can abort the
-    // previous one — matters for typeahead callers passing changing `query`.
-    const abortRef = useRef<AbortController | null>(null)
+    const seenRefresh = useRef(0)
 
     useEffect(() => {
         if (!enabled || !url || !effectiveField) {
@@ -158,61 +203,75 @@ export function useOptionsResolver(args: UseOptionsResolverArgs): UseOptionsReso
             setError(null)
             return
         }
-        // Cancel any pending request before issuing a new one.
-        abortRef.current?.abort()
-        const controller = new AbortController()
-        abortRef.current = controller
+
+        const key = optionsRequestKey(
+            optionsScope(),
+            url,
+            effectiveField,
+            query ?? '',
+            limit,
+            filterValue,
+        )
+        if (refreshKey !== seenRefresh.current) {
+            seenRefresh.current = refreshKey
+            optionsEpoch.set(key, (optionsEpoch.get(key) ?? 0) + 1)
+            optionsCache.delete(key)
+            optionsInflight.delete(key)
+        }
+        const epoch = optionsEpoch.get(key) ?? 0
+        const cached = optionsCache.get(key)
+        if (cached && Date.now() - cached.at < OPTIONS_TTL_MS) {
+            setOptions(cached.payload.options)
+            setMeta(cached.payload.meta)
+            setLoading(false)
+            setError(null)
+            return
+        }
 
         setLoading(true)
         setError(null)
 
-        const params: Record<string, string | number> = { field: effectiveField }
-        if (query) params.q = query
-        if (typeof limit === 'number' && limit > 0) params.limit = limit
-        // Cascade scope: a dependent picker passes the value of the field it
-        // `dependsOn`. Skip empty strings so a cleared parent omits the param
-        // (no scope) rather than querying for the empty-string filter_value.
-        if (filterValue) params.filter_value = filterValue
-
-        api.get(url, { params, signal: controller.signal })
-            .then((res) => {
-                if (controller.signal.aborted) return
+        let pending = optionsInflight.get(key)
+        if (!pending) {
+            const params: Record<string, string | number> = { field: effectiveField }
+            if (query) params.q = query
+            if (typeof limit === 'number' && limit > 0) params.limit = limit
+            if (filterValue) params.filter_value = filterValue
+            pending = api.get(url, { params }).then((res) => {
                 const body = (res as { data: any }).data
                 if (!body || body.success !== true) {
                     throw new Error(body?.message || 'options resolver: unsuccessful response')
                 }
-                const rawOptions: any[] = Array.isArray(body.data) ? body.data : []
-                const projected = rawOptions.map(projectOption)
-                setOptions(projected)
-                // v0.9.0 envelope: meta.type / meta.count. We tolerate
-                // older deployments that still emit a root-level `type`
-                // by reading either spot — the projection prefers the
-                // canonical location so the SDK guides apps to the new
-                // shape without breaking grace-period upgrades.
-                const metaPayload =
-                    body.meta && typeof body.meta === 'object'
-                        ? body.meta
-                        : { type: body.type, count: rawOptions.length }
-                setMeta({
-                    type: metaPayload?.type ?? 'dynamic',
-                    count:
-                        typeof metaPayload?.count === 'number'
-                            ? metaPayload.count
-                            : rawOptions.length,
-                })
+                const payload = readOptionsEnvelope(body)
+                if ((optionsEpoch.get(key) ?? 0) === epoch) {
+                    optionsCache.set(key, { payload, at: Date.now() })
+                }
+                return payload
+            }).finally(() => {
+                if (optionsInflight.get(key) === pending) optionsInflight.delete(key)
+            })
+            optionsInflight.set(key, pending)
+        }
+
+        let cancelled = false
+        pending
+            .then((payload) => {
+                if (cancelled) return
+                setOptions(payload.options)
+                setMeta(payload.meta)
             })
             .catch((err: any) => {
-                if (controller.signal.aborted) return
+                if (cancelled) return
                 setError(err instanceof Error ? err : new Error(String(err)))
                 setOptions([])
                 setMeta(null)
             })
             .finally(() => {
-                if (!controller.signal.aborted) setLoading(false)
+                if (!cancelled) setLoading(false)
             })
 
         return () => {
-            controller.abort()
+            cancelled = true
         }
     }, [api, url, effectiveField, query, limit, enabled, filterValue, refreshKey])
 

package/src/validator.ts

@@ -273,3 +273,39 @@ function walk(
 export function bagHasErrors(bag: Record<string, FieldIssue[]>): boolean {
     return Object.keys(bag).length > 0
 }
+
+/** Issue codes produced by a field's declarative rule (regex / min / max /
+ *  custom builtins) — the ones the kernel grandfathers on update. */
+const RULE_CODES = new Set(['regex', 'min', 'max', 'email', 'uuid', 'url', 'numeric', 'integer', 'custom'])
+
+function sameValue(a: unknown, b: unknown): boolean {
+    if (isEmpty(a) && isEmpty(b)) return true
+    if (isNumeric(a) && isNumeric(b)) return numericValue(a) === numericValue(b)
+    return asString(a) === asString(b)
+}
+
+/**
+ * Edit mode: drop the rule issues (regex/min/max/custom) of a top-level field
+ * whose value is the one already persisted. The edit form posts every field
+ * back, so a rule added after the row was written (an RFC pattern over legacy
+ * data) would otherwise block saving a record over a field nobody touched.
+ * Mirrors the kernel (dynamic.validateWrite grandfathers unchanged values on
+ * update). required / invalid_option / invalid_type are kept.
+ */
+export function exemptUnchangedRuleIssues(
+    bag: Record<string, FieldIssue[]>,
+    values: Record<string, unknown>,
+    persisted: Record<string, unknown> | null | undefined,
+): Record<string, FieldIssue[]> {
+    if (!persisted) return bag
+    const out: Record<string, FieldIssue[]> = {}
+    for (const [path, issues] of Object.entries(bag)) {
+        const unchanged =
+            !path.includes('.') &&
+            Object.prototype.hasOwnProperty.call(persisted, path) &&
+            sameValue(values?.[path], persisted[path])
+        const kept = unchanged ? issues.filter(i => !RULE_CODES.has(String(i.code))) : issues
+        if (kept.length) out[path] = kept
+    }
+    return out
+}