From 9b88991ffb75f587f641a4f1bfde8d84f064d24a Mon Sep 17 00:00:00 2001
From: labkey-nicka <nickk@labkey.com>
Date: Mon, 3 Aug 2026 12:46:38 -0700
Subject: [PATCH 1/7] Formsy: isShallowSame

---
 .../components/forms/formsy/Formsy.test.tsx   | 51 ++++++++++++++++++-
 .../components/forms/formsy/utils.test.ts     | 32 +++++++++++-
 .../internal/components/forms/formsy/utils.ts | 20 ++++++++
 .../components/forms/formsy/withFormsy.tsx    | 21 +++++++-
 4 files changed, 120 insertions(+), 4 deletions(-)

diff --git a/packages/components/src/internal/components/forms/formsy/Formsy.test.tsx b/packages/components/src/internal/components/forms/formsy/Formsy.test.tsx
index 3a1ce27a63..787d776c92 100644
--- a/packages/components/src/internal/components/forms/formsy/Formsy.test.tsx
+++ b/packages/components/src/internal/components/forms/formsy/Formsy.test.tsx
@@ -1,7 +1,7 @@
 // This file was originally derived from the "formsy-react" package, specifically, v2.3.2.
 // Credit: Christian Alfoni and the Formsy Authors
 // Repository: https://github.com/formsy/formsy-react/tree/0226fab133a25
-import React, { act, FC, memo, PropsWithChildren, useCallback, useRef, useState } from 'react';
+import React, { act, FC, memo, PropsWithChildren, useCallback, useMemo, useRef, useState } from 'react';
 import { createPortal } from 'react-dom';
 import { createEvent, fireEvent, render } from '@testing-library/react';
 import { userEvent } from '@testing-library/user-event';
@@ -448,6 +448,55 @@ describe('Formsy', () => {
         });
     });
 
+    describe('validation messages', () => {
+        // Mirrors an input whose error message is enriched by data that only starts loading once the validation
+        // failure is known, so the message changes while the rules and the value stay put.
+        const AsyncMessageForm: FC = () => {
+            const [isRegistered, setIsRegistered] = useState(false);
+            const [canAssociate, setCanAssociate] = useState(false);
+            const validations = useMemo(() => ({ isNotRegistered: isRegistered }), [isRegistered]);
+            const validationErrors = useMemo(
+                () => ({ isNotRegistered: canAssociate ? 'Already registered. Associate?' : 'Already registered.' }),
+                [canAssociate]
+            );
+            const onRegister = useCallback(() => setIsRegistered(true), []);
+            const onAssociate = useCallback(() => setCanAssociate(true), []);
+
+            return (
+                <>
+                    <Formsy>
+                        <TestInput
+                            name="one"
+                            testId="test-input"
+                            validationErrors={validationErrors}
+                            validations={validations}
+                            value="foo"
+                        />
+                    </Formsy>
+                    <button data-testid="register-btn" onClick={onRegister} type="button" />
+                    <button data-testid="associate-btn" onClick={onAssociate} type="button" />
+                </>
+            );
+        };
+
+        it('should re-resolve messages when the validationErrors prop changes', () => {
+            addFormsyRule<string>('isNotRegistered', (_values, _value, isRegistered: boolean) => !isRegistered);
+
+            const screen = render(<AsyncMessageForm />);
+            const input = screen.getByTestId('test-input');
+
+            expect(input.getAttribute('data-error-messages')).toEqual('');
+
+            // The rule starts failing, so the message resolved at that moment is displayed.
+            fireEvent.click(screen.getByTestId('register-btn'));
+            expect(input.getAttribute('data-error-messages')).toEqual('Already registered.');
+
+            // Only the message changes here. It should surface without waiting for another validation pass.
+            fireEvent.click(screen.getByTestId('associate-btn'));
+            expect(input.getAttribute('data-error-messages')).toEqual('Already registered. Associate?');
+        });
+    });
+
     describe('onChange', () => {
         it('should not trigger onChange when form is mounted', () => {
             const hasChanged = jest.fn();
diff --git a/packages/components/src/internal/components/forms/formsy/utils.test.ts b/packages/components/src/internal/components/forms/formsy/utils.test.ts
index c2dcd739a4..f855b4075c 100644
--- a/packages/components/src/internal/components/forms/formsy/utils.test.ts
+++ b/packages/components/src/internal/components/forms/formsy/utils.test.ts
@@ -19,7 +19,7 @@ const VALUES = [null, ...TYPES.isDate, ...TYPES.isFunction, ...TYPES.isObject, .
 describe('utils', () => {
     // For each function in types
     Object.keys(TYPES).forEach(isFn => {
-        // Create a test for that functiojn
+        // Create a test for that function
         it(isFn, () => {
             // For each value in values
             VALUES.forEach(value => {
@@ -101,4 +101,34 @@ describe('utils', () => {
             success: [],
         });
     });
+
+    describe('isShallowSame', () => {
+        it('compares message maps by key set and value identity', () => {
+            const message = 'This sequence has already been registered.';
+
+            expect(utils.isShallowSame({ isUnique: message }, { isUnique: message })).toBe(true);
+            expect(utils.isShallowSame({ isUnique: message }, { isUnique: 'Something else' })).toBe(false);
+            expect(utils.isShallowSame({ isUnique: message }, { isUnique: message, isRequired: message })).toBe(false);
+            expect(utils.isShallowSame({ isUnique: message }, { isValid: message })).toBe(false);
+        });
+
+        it('treats structurally equal but distinct values as changed, unlike isSame()', () => {
+            // Stands in for a ReactNode message. Elements are rebuilt on every render, so identity is the only
+            // signal that can be trusted -- isSame() would walk into React internals and report a false match.
+            const messages = { isUnique: { type: 'span', props: { children: 'Already registered.' } } };
+            const rebuilt = { isUnique: { type: 'span', props: { children: 'Already registered.' } } };
+
+            expect(utils.isSame(messages, rebuilt)).toBe(true);
+            expect(utils.isShallowSame(messages, rebuilt)).toBe(false);
+        });
+
+        it('compares resolved message arrays element-wise by identity', () => {
+            const message = { type: 'span', props: { children: 'Already registered.' } };
+
+            expect(utils.isShallowSame([message], [message])).toBe(true);
+            expect(utils.isShallowSame([message], [{ ...message }])).toBe(false);
+            expect(utils.isShallowSame(['Required'], ['Required'])).toBe(true);
+            expect(utils.isShallowSame([], ['Required'])).toBe(false);
+        });
+    });
 });
diff --git a/packages/components/src/internal/components/forms/formsy/utils.ts b/packages/components/src/internal/components/forms/formsy/utils.ts
index 99f735648b..4846ac705b 100644
--- a/packages/components/src/internal/components/forms/formsy/utils.ts
+++ b/packages/components/src/internal/components/forms/formsy/utils.ts
@@ -91,6 +91,26 @@ export function isSame(a: unknown, b: unknown): boolean {
     return a === b;
 }
 
+export function isShallowSame(a: unknown, b: unknown): boolean {
+    if (Object.is(a, b)) {
+        return true;
+    }
+
+    if (Array.isArray(a) && Array.isArray(b)) {
+        return a.length === b.length && a.every((item, index) => Object.is(item, b[index]));
+    }
+
+    if (isObject(a) && isObject(b)) {
+        const keys = Object.keys(a);
+        return (
+            keys.length === Object.keys(b).length &&
+            keys.every(key => b.hasOwnProperty(key) && Object.is(a[key], b[key]))
+        );
+    }
+
+    return false;
+}
+
 interface RulesResult {
     errors: ValidationError[];
     failed: string[];
diff --git a/packages/components/src/internal/components/forms/formsy/withFormsy.tsx b/packages/components/src/internal/components/forms/formsy/withFormsy.tsx
index c02440bbfc..d240050769 100644
--- a/packages/components/src/internal/components/forms/formsy/withFormsy.tsx
+++ b/packages/components/src/internal/components/forms/formsy/withFormsy.tsx
@@ -17,7 +17,7 @@ import {
     WrapperState,
 } from './types';
 
-import { isSame, isString } from './utils';
+import { isSame, isShallowSame, isString } from './utils';
 import { isDefaultRequiredValue } from './formsyRules';
 
 function convertValidationsToObject<V>(validations: false | Validations<V>): Validations<V> {
@@ -109,7 +109,8 @@ export function withFormsy<T, V>(
         };
 
         componentDidUpdate = (prevProps: WrappedProps): void => {
-            const { required, value, validations, validate } = this.props;
+            const { required, runValidation, value, validate, validationError, validationErrors, validations } =
+                this.props;
 
             // If the value passed has changed, set it. If a value is not passed, it will internally update, and this
             // will never run. Skip when the input already holds the value: a parent that owns the value and echoes it
@@ -123,6 +124,22 @@ export function withFormsy<T, V>(
             if (!isSame(validations, prevProps.validations) || !isSame(required, prevProps.required)) {
                 this.setValidations(validations, required);
                 validate(this);
+                return;
+            }
+
+            // Validation messages are resolved when validation runs and are then held in state, so a change to the
+            // messages alone -- a message whose content depends on data that loads asynchronously, for example --
+            // would not reach the user until some later interaction happened to trigger the next validation pass.
+            // Re-resolve the messages against the current props here. The rules and the value are unchanged, so this
+            // cannot change the validity of this input and the form does not need to revalidate.
+            if (
+                !isShallowSame(validationError, prevProps.validationError) ||
+                !isShallowSame(validationErrors, prevProps.validationErrors)
+            ) {
+                const validationState = runValidation(this);
+                if (!isShallowSame(validationState.validationError, this.state.validationError)) {
+                    this.setState(validationState);
+                }
             }
         };
 

From 94e641410f5972c377d5e2ff8255235aabddb7d1 Mon Sep 17 00:00:00 2001
From: labkey-nicka <nickk@labkey.com>
Date: Mon, 3 Aug 2026 12:47:03 -0700
Subject: [PATCH 2/7] useDataChangeCommentsRequired loading state

---
 .../components/container/FolderAPIWrapper.ts  | 17 +++-------
 .../input/useDataChangeCommentsRequired.ts    | 33 ++++++++++---------
 2 files changed, 22 insertions(+), 28 deletions(-)

diff --git a/packages/components/src/internal/components/container/FolderAPIWrapper.ts b/packages/components/src/internal/components/container/FolderAPIWrapper.ts
index 963f0921a8..f7001b3324 100644
--- a/packages/components/src/internal/components/container/FolderAPIWrapper.ts
+++ b/packages/components/src/internal/components/container/FolderAPIWrapper.ts
@@ -5,7 +5,7 @@
 import { ActionURL, Ajax, Utils } from '@labkey/api';
 
 import { Container } from '../base/models/Container';
-import { handleRequestFailure } from '../../request';
+import { handleRequestFailure, request } from '../../request';
 import { SAMPLE_MANAGER_APP_PROPERTIES } from '../../app/constants';
 import { FolderConfigurableDataType } from '../entities/models';
 import { getFolderDataTypeExclusions } from '../entities/actions';
@@ -126,22 +126,16 @@ export class ServerFolderAPIWrapper implements FolderAPIWrapper {
     };
 
     getAuditSettings = (containerPath?: string): Promise<AuditSettingsResponse> => {
-        return new Promise((resolve, reject) => {
-            Ajax.request({
-                url: ActionURL.buildURL('audit', 'getAuditSettings', containerPath),
-                method: 'POST',
-                success: Utils.getCallbackWrapper(response => {
-                    resolve(response);
-                }),
-                failure: handleRequestFailure(reject, 'Failed to retrieve audit settings.'),
-            });
+        return request<AuditSettingsResponse>({
+            url: ActionURL.buildURL('audit', 'getAuditSettings.api', containerPath),
+            errorLogMsg: 'Failed to retrieve audit settings.',
         });
     };
 
     setAuditCommentsRequired = (requireUserComments: boolean, containerPath?: string): Promise<void> => {
         return new Promise((resolve, reject) => {
             Ajax.request({
-                url: ActionURL.buildURL('audit', 'saveAuditSettings', containerPath),
+                url: ActionURL.buildURL('audit', 'saveAuditSettings.api', containerPath),
                 method: 'POST',
                 jsonData: { requireUserComments },
                 success: Utils.getCallbackWrapper(() => {
@@ -197,7 +191,6 @@ export class ServerFolderAPIWrapper implements FolderAPIWrapper {
         return new Promise((resolve, reject) => {
             Ajax.request({
                 url: ActionURL.buildURL(SAMPLE_MANAGER_APP_PROPERTIES.controllerName, 'getDataTypeExclusion.api'),
-                method: 'GET',
                 params: {
                     dataType,
                     dataTypeRowId,
diff --git a/packages/components/src/internal/components/forms/input/useDataChangeCommentsRequired.ts b/packages/components/src/internal/components/forms/input/useDataChangeCommentsRequired.ts
index d5470131ec..47d5fa5fbe 100644
--- a/packages/components/src/internal/components/forms/input/useDataChangeCommentsRequired.ts
+++ b/packages/components/src/internal/components/forms/input/useDataChangeCommentsRequired.ts
@@ -2,28 +2,29 @@
  * Copyright (c) 2024-2026 LabKey Corporation. All rights reserved. No portion of this work may be reproduced
  * in any form or by any electronic or mechanical means without written permission from LabKey Corporation.
  */
-import { useEffect, useState } from 'react';
+import { useCallback } from 'react';
 import { useServerContext } from '../../base/ServerContext';
 import { getAppHomeFolderPath } from '../../../app/utils';
 import { useAppContext } from '../../../AppContext';
+import { Loader, useLoadableState } from '../../../useLoadableState';
+import { LoadingState } from '../../../../public/LoadingState';
 
-export const useDataChangeCommentsRequired = (): { requiresUserComment: boolean } => {
+export type DataChangeCommentsRequired = {
+    loadingState: LoadingState;
+    requiresUserComment: boolean;
+};
+
+export const useDataChangeCommentsRequired = (): DataChangeCommentsRequired => {
     const { container, moduleContext } = useServerContext();
     const { api } = useAppContext();
-    const [requiresUserComment, setRequiresUserComment] = useState<boolean>(false);
 
-    useEffect(
-        () => {
-            (async () => {
-                const path = getAppHomeFolderPath(container, moduleContext);
-                try {
-                    const response = await api.folder.getAuditSettings(path);
-                    setRequiresUserComment(!!response?.requireUserComments);
-                } catch (error) {
-                    console.error('Unable to retrieve audit log settings for ' + path, error);
-                }
-            })();
-        }, [ /** on load only */ ]);
+    const loader = useCallback<Loader<boolean>>(async () => {
+        const path = getAppHomeFolderPath(container, moduleContext);
+        const response = await api.folder.getAuditSettings(path);
+        return !!response?.requireUserComments;
+    }, [api.folder, container, moduleContext]);
+
+    const { loadingState, value: requiresUserComment } = useLoadableState(loader);
 
-    return { requiresUserComment };
+    return { loadingState, requiresUserComment };
 };

From 99f22b5b9c1980780e09b7ac372f46657a4f1de6 Mon Sep 17 00:00:00 2001
From: labkey-nicka <nickk@labkey.com>
Date: Fri, 7 Aug 2026 14:29:44 -0700
Subject: [PATCH 3/7] AppLink: support targetBlank on <Link/>

---
 .../src/internal/url/AppLink.test.tsx         | 71 ++++++++++++++++++-
 .../components/src/internal/url/AppLink.tsx   | 20 +++---
 2 files changed, 79 insertions(+), 12 deletions(-)

diff --git a/packages/components/src/internal/url/AppLink.test.tsx b/packages/components/src/internal/url/AppLink.test.tsx
index eeec9f661c..4084c90b55 100644
--- a/packages/components/src/internal/url/AppLink.test.tsx
+++ b/packages/components/src/internal/url/AppLink.test.tsx
@@ -2,9 +2,11 @@
  * Copyright (c) 2025-2026 LabKey Corporation. All rights reserved. No portion of this work may be reproduced
  * in any form or by any electronic or mechanical means without written permission from LabKey Corporation.
  */
+import React from 'react';
+import { render, screen } from '@testing-library/react';
 import { __setAction, __setContainerPath, __setController } from '@labkey/api';
 
-import { parseAppPath } from './AppLink';
+import { AppLink, parseAppPath } from './AppLink';
 
 const TEST_PROJECT = '/My Project';
 const TEST_CHILD = '/My Child';
@@ -138,3 +140,70 @@ describe('parseAppPath', () => {
         );
     });
 });
+
+const EXTERNAL_URL = 'https://www.example.com/';
+const LINK_TEXT = 'Run 533';
+
+describe('AppLink', () => {
+    test('targetBlank', () => {
+        render(
+            <AppLink targetBlank to={EXTERNAL_URL}>
+                {LINK_TEXT}
+            </AppLink>
+        );
+
+        // Assert - targetBlank opens a new tab and defaults rel to protect it from the opener
+        const link = screen.getByRole('link', { name: LINK_TEXT });
+        expect(link).toHaveAttribute('href', EXTERNAL_URL);
+        expect(link).toHaveAttribute('target', '_blank');
+        expect(link).toHaveAttribute('rel', 'noopener noreferrer');
+    });
+
+    test('rel and target props', () => {
+        render(
+            <AppLink rel="nofollow" target="_parent" to={EXTERNAL_URL}>
+                {LINK_TEXT}
+            </AppLink>
+        );
+
+        // Assert - without targetBlank the rel/target props are used as-is
+        const link = screen.getByRole('link', { name: LINK_TEXT });
+        expect(link).toHaveAttribute('target', '_parent');
+        expect(link).toHaveAttribute('rel', 'nofollow');
+    });
+
+    test('targetBlank does not supersede rel and target props', () => {
+        render(
+            <AppLink rel="nofollow" target="_parent" targetBlank to={EXTERNAL_URL}>
+                {LINK_TEXT}
+            </AppLink>
+        );
+
+        // Assert - both props supersede the targetBlank defaults
+        const link = screen.getByRole('link', { name: LINK_TEXT });
+        expect(link).toHaveAttribute('target', '_parent');
+        expect(link).toHaveAttribute('rel', 'nofollow');
+    });
+
+    test('targetBlank with only a target prop', () => {
+        render(
+            <AppLink target="_parent" targetBlank to={EXTERNAL_URL}>
+                {LINK_TEXT}
+            </AppLink>
+        );
+
+        // Assert - the target prop supersedes its targetBlank default, but the rel default still applies
+        const link = screen.getByRole('link', { name: LINK_TEXT });
+        expect(link).toHaveAttribute('target', '_parent');
+        expect(link).toHaveAttribute('rel', 'noopener noreferrer');
+    });
+
+    test('no targetBlank, rel, or target', () => {
+        render(<AppLink to={EXTERNAL_URL}>{LINK_TEXT}</AppLink>);
+
+        // Assert - neither attribute is rendered when nothing asks for them
+        const link = screen.getByRole('link', { name: LINK_TEXT });
+        expect(link).not.toHaveAttribute('target');
+        expect(link).not.toHaveAttribute('rel');
+    });
+});
diff --git a/packages/components/src/internal/url/AppLink.tsx b/packages/components/src/internal/url/AppLink.tsx
index 6681b2e6b9..7ea35965b4 100644
--- a/packages/components/src/internal/url/AppLink.tsx
+++ b/packages/components/src/internal/url/AppLink.tsx
@@ -16,7 +16,7 @@ const TARGET_BLANK = '_blank';
 const URL_REL = 'noopener noreferrer';
 
 /**
- * If the given href points to our current container & product, return the react-router path, in all other cases
+ * If the given href points to our current container and product, return the react-router path, in all other cases
  * return undefined.
  * @param href
  */
@@ -40,13 +40,14 @@ export function parseAppPath(href: string): string | undefined {
 /** This is a subset of AnchorHTMLAttributes<HTMLAnchorElement> that are passed through to the anchor tag. */
 type InheritedHTMLAnchorProps = Omit<
     DetailedHTMLProps<AnchorHTMLAttributes<HTMLAnchorElement>, HTMLAnchorElement>,
-    'href' // overridden by AppLink which uses "to" instead
+    'href' // overridden by AppLink, which uses "to" instead
 >;
 
 /** These props are specific to <AppLink>. */
 interface Props extends InheritedHTMLAnchorProps {
+    /** Opens the link in a new tab, protected from the opener. Applies only to whichever of "target"/"rel" is not supplied. */
     targetBlank?: boolean;
-    to: string | AppURL | undefined;
+    to: AppURL | string | undefined;
 }
 
 /**
@@ -54,7 +55,9 @@ interface Props extends InheritedHTMLAnchorProps {
  * handles all the corner cases around moving within our apps, between our apps, to other parts of LKS, and externally.
  */
 export const AppLink: FC<Props> = memo(props => {
-    const { children, rel, target, targetBlank, to, ...anchorProps } = props;
+    const { children, rel: relProp, target: targetProp, targetBlank, to, ...anchorProps } = props;
+    const rel = relProp ?? (targetBlank ? URL_REL : undefined);
+    const target = targetProp ?? (targetBlank ? TARGET_BLANK : undefined);
 
     const appPath = useMemo<string | undefined>(() => {
         if (to instanceof AppURL && to.isAppPath()) {
@@ -70,19 +73,14 @@ export const AppLink: FC<Props> = memo(props => {
     // anchor tags in our tests.
     if (appPath && !isTestEnv()) {
         return (
-            <Link {...anchorProps} to={appPath}>
+            <Link {...anchorProps} rel={rel} target={target} to={appPath}>
                 {children}
             </Link>
         );
     }
 
     return (
-        <a
-            {...anchorProps}
-            href={to?.toString()}
-            rel={targetBlank ? URL_REL : rel}
-            target={targetBlank ? TARGET_BLANK : target}
-        >
+        <a {...anchorProps} href={to?.toString()} rel={rel} target={target}>
             {children}
         </a>
     );

From 6d32d03cb67db0c4d36fea0d655c917cc3fdf1a4 Mon Sep 17 00:00:00 2001
From: labkey-nicka <nickk@labkey.com>
Date: Wed, 12 Aug 2026 08:08:20 -0700
Subject: [PATCH 4/7] lint

---
 .../src/internal/components/container/FolderAPIWrapper.ts   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/components/src/internal/components/container/FolderAPIWrapper.ts b/packages/components/src/internal/components/container/FolderAPIWrapper.ts
index f7001b3324..63f93503e5 100644
--- a/packages/components/src/internal/components/container/FolderAPIWrapper.ts
+++ b/packages/components/src/internal/components/container/FolderAPIWrapper.ts
@@ -54,7 +54,7 @@ export interface FolderAPIWrapper {
         excludeArchived?: boolean
     ) => Promise<Container[]>;
     getDataTypeExcludedContainers: (dataType: FolderConfigurableDataType, dataTypeRowId: number) => Promise<string[]>;
-    getFolderDataTypeExclusions: (excludedContainer?: string, reload?: boolean) => Promise<{ [key: string]: number[] }>;
+    getFolderDataTypeExclusions: (excludedContainer?: string, reload?: boolean) => Promise<Record<string, number[]>>;
     getMultipleDataTypeExcludedContainers: (
         dataType: FolderConfigurableDataType,
         dataTypeRowIds: number[]
@@ -87,7 +87,7 @@ export class ServerFolderAPIWrapper implements FolderAPIWrapper {
         });
     };
 
-    archiveFolder = (archive: boolean = true, containerPath?: string): Promise<Container> => {
+    archiveFolder = (archive = true, containerPath?: string): Promise<Container> => {
         return new Promise((resolve, reject) => {
             Ajax.request({
                 url: ActionURL.buildURL(
@@ -223,7 +223,7 @@ export class ServerFolderAPIWrapper implements FolderAPIWrapper {
         }
 
         return new Promise((resolve, reject) => {
-            const promises: Array<Promise<Record<string, string[]>>> = [];
+            const promises: Promise<Record<string, string[]>>[] = [];
 
             dataTypeRowIds.forEach(id => {
                 promises.push(this.getDataTypeExcludedContainersAsRecord(dataType, id));

From 462443ff169ca6d5593ab0412e94d1a7d6231e01 Mon Sep 17 00:00:00 2001
From: labkey-nicka <nickk@labkey.com>
Date: Wed, 12 Aug 2026 09:52:57 -0700
Subject: [PATCH 5/7] Formsy: validate.notifyChange

---
 .../components/forms/formsy/Formsy.test.tsx   | 40 +++++++++++++++++--
 .../components/forms/formsy/Formsy.tsx        |  4 +-
 .../internal/components/forms/formsy/types.ts |  2 +-
 .../components/forms/formsy/withFormsy.tsx    |  3 +-
 4 files changed, 42 insertions(+), 7 deletions(-)

diff --git a/packages/components/src/internal/components/forms/formsy/Formsy.test.tsx b/packages/components/src/internal/components/forms/formsy/Formsy.test.tsx
index 787d776c92..d35cbd94e9 100644
--- a/packages/components/src/internal/components/forms/formsy/Formsy.test.tsx
+++ b/packages/components/src/internal/components/forms/formsy/Formsy.test.tsx
@@ -485,15 +485,15 @@ describe('Formsy', () => {
             const screen = render(<AsyncMessageForm />);
             const input = screen.getByTestId('test-input');
 
-            expect(input.getAttribute('data-error-messages')).toEqual('');
+            expect(input).toHaveAttribute('data-error-messages', '');
 
             // The rule starts failing, so the message resolved at that moment is displayed.
             fireEvent.click(screen.getByTestId('register-btn'));
-            expect(input.getAttribute('data-error-messages')).toEqual('Already registered.');
+            expect(input).toHaveAttribute('data-error-messages', 'Already registered.');
 
             // Only the message changes here. It should surface without waiting for another validation pass.
             fireEvent.click(screen.getByTestId('associate-btn'));
-            expect(input.getAttribute('data-error-messages')).toEqual('Already registered. Associate?');
+            expect(input).toHaveAttribute('data-error-messages', 'Already registered. Associate?');
         });
     });
 
@@ -556,6 +556,40 @@ describe('Formsy', () => {
 
             expect(hasChanged).toHaveBeenCalledTimes(1);
         });
+
+        it('should not trigger onChange when only the validations prop changes', () => {
+            addFormsyRule<string>('isNotConflicted', (_values, _value, isConflicted: boolean) => !isConflicted);
+            const hasChanged = jest.fn();
+
+            const TestForm: FC = () => {
+                const [isConflicted, setIsConflicted] = useState(true);
+                const validations = useMemo(() => ({ isNotConflicted: isConflicted }), [isConflicted]);
+                const resolveConflict = useCallback(() => setIsConflicted(false), []);
+
+                return (
+                    <>
+                        <Formsy onChange={hasChanged}>
+                            <TestInput name="one" testId="test-input" validations={validations} value="foo" />
+                        </Formsy>
+                        <button data-testid="resolve-btn" onClick={resolveConflict} type="button" />
+                    </>
+                );
+            };
+
+            const screen = render(<TestForm />);
+            const input = screen.getByTestId('test-input');
+
+            fireEvent.change(input, { target: { value: 'bar' } });
+            expect(hasChanged).toHaveBeenCalledTimes(1);
+            expect(input).toHaveAttribute('data-is-valid', 'false');
+
+            fireEvent.click(screen.getByTestId('resolve-btn'));
+            expect(hasChanged).toHaveBeenCalledTimes(1);
+            expect(input).toHaveAttribute('data-is-valid', 'true');
+
+            fireEvent.change(input, { target: { value: 'baz' } });
+            expect(hasChanged).toHaveBeenCalledTimes(2);
+        });
     });
 
     describe('Update a form', () => {
diff --git a/packages/components/src/internal/components/forms/formsy/Formsy.tsx b/packages/components/src/internal/components/forms/formsy/Formsy.tsx
index ec46c684ef..6e1d37bea0 100644
--- a/packages/components/src/internal/components/forms/formsy/Formsy.tsx
+++ b/packages/components/src/internal/components/forms/formsy/Formsy.tsx
@@ -371,9 +371,9 @@ export class Formsy extends Component<FormsyProps, FormsyState> {
     // Use the bound values and the actual input value to
     // validate the input and set its state. Then check the
     // state of the form itself
-    validate = (component: InputComponent<any>): void => {
+    validate = (component: InputComponent<any>, notifyChange = true): void => {
         if (!this._mounted) return;
-        this.triggerChange();
+        if (notifyChange) this.triggerChange();
 
         // Run through the validations, split them up and call
         // the validator IF there is a value or it is required
diff --git a/packages/components/src/internal/components/forms/formsy/types.ts b/packages/components/src/internal/components/forms/formsy/types.ts
index cce1f7b7d0..8991017a7e 100644
--- a/packages/components/src/internal/components/forms/formsy/types.ts
+++ b/packages/components/src/internal/components/forms/formsy/types.ts
@@ -96,7 +96,7 @@ export interface FormsyContextInterface {
     isFormDisabled: boolean;
     isValidValue: (component: InputComponent<any>, value: any) => boolean;
     runValidation: (component: InputComponent<any>, value?: any) => RunValidationResponse;
-    validate: (component: InputComponent<any>) => void;
+    validate: (component: InputComponent<any>, notifyChange?: boolean) => void;
 }
 
 export type OnSubmitCallback = (
diff --git a/packages/components/src/internal/components/forms/formsy/withFormsy.tsx b/packages/components/src/internal/components/forms/formsy/withFormsy.tsx
index d240050769..58cfd7fe9b 100644
--- a/packages/components/src/internal/components/forms/formsy/withFormsy.tsx
+++ b/packages/components/src/internal/components/forms/formsy/withFormsy.tsx
@@ -123,7 +123,8 @@ export function withFormsy<T, V>(
             // If validations or required is changed, run a new validation
             if (!isSame(validations, prevProps.validations) || !isSame(required, prevProps.required)) {
                 this.setValidations(validations, required);
-                validate(this);
+                // The rules changed, not the value: revalidate without emitting a change event
+                validate(this, false);
                 return;
             }
 

From f619ba0f3006a354b8b6338d3c49aba07068a79c Mon Sep 17 00:00:00 2001
From: labkey-nicka <nickk@labkey.com>
Date: Thu, 13 Aug 2026 12:20:48 -0700
Subject: [PATCH 6/7] 7.55.3-fb-associate.0

---
 packages/components/package-lock.json | 4 ++--
 packages/components/package.json      | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/components/package-lock.json b/packages/components/package-lock.json
index 2a10b0a9bf..4a139e5c66 100644
--- a/packages/components/package-lock.json
+++ b/packages/components/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "@labkey/components",
-  "version": "7.55.2",
+  "version": "7.55.3-fb-associate.0",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@labkey/components",
-      "version": "7.55.2",
+      "version": "7.55.3-fb-associate.0",
       "license": "SEE LICENSE IN LICENSE.txt",
       "dependencies": {
         "@hello-pangea/dnd": "18.0.1",
diff --git a/packages/components/package.json b/packages/components/package.json
index df3b7f07d5..d32d28ca32 100644
--- a/packages/components/package.json
+++ b/packages/components/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@labkey/components",
-  "version": "7.55.2",
+  "version": "7.55.3-fb-associate.0",
   "description": "Components, models, actions, and utility functions for LabKey applications and pages",
   "sideEffects": false,
   "files": [

From 8d514f403f8b017e4bf4077a27f7940edae95741 Mon Sep 17 00:00:00 2001
From: labkey-nicka <nickk@labkey.com>
Date: Fri, 14 Aug 2026 14:59:04 -0700
Subject: [PATCH 7/7] 7.55.3

---
 packages/components/package-lock.json | 4 ++--
 packages/components/package.json      | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/packages/components/package-lock.json b/packages/components/package-lock.json
index 4a139e5c66..a87e81efc6 100644
--- a/packages/components/package-lock.json
+++ b/packages/components/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "@labkey/components",
-  "version": "7.55.3-fb-associate.0",
+  "version": "7.55.3",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@labkey/components",
-      "version": "7.55.3-fb-associate.0",
+      "version": "7.55.3",
       "license": "SEE LICENSE IN LICENSE.txt",
       "dependencies": {
         "@hello-pangea/dnd": "18.0.1",
diff --git a/packages/components/package.json b/packages/components/package.json
index d32d28ca32..f86f597e7a 100644
--- a/packages/components/package.json
+++ b/packages/components/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@labkey/components",
-  "version": "7.55.3-fb-associate.0",
+  "version": "7.55.3",
   "description": "Components, models, actions, and utility functions for LabKey applications and pages",
   "sideEffects": false,
   "files": [