Surface server violations on untouched single relations

RelationInput gated its field message and status on the field being touched
(focused then blurred). For a single (non-multiple) relation, a server-sourced
violation — such as a required error from the save-time 422 — stayed hidden
until the user focused the field, and was also excluded from the dialog's
anchorless banner because its property name matches a rendered field. An empty
required relation could pass review invisibly.

Server-sourced violations now surface regardless of touch; only the live client
check still waits for blur. The creator is unaffected: its dry-run already
strips required and label-required, so an untouched required relation produces
no server violation to show.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: alistair3149

resources/ext.neowiki/src/components/Value/RelationInput.vue

@@ -80,18 +80,22 @@ function relevantServerViolations(): readonly SubjectViolation[] {
 	return ( props.serverViolations ?? [] ).filter( ( v ) => v.propertyName === name );
 }
 
-const fieldMessages = computed<ValidationMessages>( () => {
-	if ( Object.keys( liveFieldMessages.value ).length > 0 ) {
-		return liveFieldMessages.value;
-	}
-	// Fall back to server violations (field-level, since NeoMultiLookupInput has no per-index slot).
+// Field-level server violation (NeoMultiLookupInput has no per-index slot).
+function serverFieldMessages(): ValidationMessages {
 	const hit = relevantServerViolations()[ 0 ];
 	if ( hit ) {
 		return {
 			error: mw.message( `neowiki-field-${ hit.code }`, ...( hit.args as string[] ) ).text()
 		};
 	}
 	return {};
+}
+
+const fieldMessages = computed<ValidationMessages>( () => {
+	if ( Object.keys( liveFieldMessages.value ).length > 0 ) {
+		return liveFieldMessages.value;
+	}
+	return serverFieldMessages();
 } );
 
 const displayedFieldMessages = computed( (): ValidationMessages => {
@@ -101,17 +105,16 @@ const displayedFieldMessages = computed( (): ValidationMessages => {
 	if ( props.property.multiple || touched.value ) {
 		return fieldMessages.value;
 	}
-	return {};
+	// Server-sourced violations (dry-run / save-time 422) surface before the field
+	// is touched; only the live client check waits for blur.
+	return serverFieldMessages();
 } );
 
 const fieldStatus = computed( (): 'error' | 'default' => {
 	if ( props.property.multiple || singleHasUnmatchedText.value ) {
 		return 'default';
 	}
-	if ( touched.value && fieldMessages.value.error ) {
-		return 'error';
-	}
-	return 'default';
+	return displayedFieldMessages.value.error !== undefined ? 'error' : 'default';
 } );
 
 function emitClearIfServerViolationPresent(): void {

resources/ext.neowiki/tests/components/Value/RelationInput.spec.ts

@@ -166,6 +166,37 @@ describe( 'RelationInput', () => {
 			expect( field.props( 'messages' ) ).toEqual( {} );
 			expect( field.props( 'status' ) ).toBe( 'default' );
 		} );
+
+		it( 'surfaces a server violation on an untouched single relation', () => {
+			const wrapper = newWrapper( {
+				// Optional field so the live client check produces nothing; the message
+				// can only originate from the server-sourced violation.
+				property: newRelationProperty( { name: 'Owner', targetSchema: 'Company' } ),
+				serverViolations: [
+					{ propertyName: 'Owner', code: 'type-mismatch', args: [], valuePartIndex: null },
+				],
+			} );
+
+			const field = wrapper.findComponent( CdxField );
+			expect( field.props( 'messages' ) ).toEqual( { error: 'neowiki-field-type-mismatch' } );
+			expect( field.props( 'status' ) ).toBe( 'error' );
+		} );
+
+		it( 'keeps a server violation suppressed while the lookup reports unmatched text', async () => {
+			const wrapper = newWrapper( {
+				property: newRelationProperty( { name: 'Owner', targetSchema: 'Company' } ),
+				serverViolations: [
+					{ propertyName: 'Owner', code: 'type-mismatch', args: [], valuePartIndex: null },
+				],
+			} );
+
+			wrapper.findComponent( SubjectLookup ).vm.$emit( 'blur', true );
+			await wrapper.vm.$nextTick();
+
+			const field = wrapper.findComponent( CdxField );
+			expect( field.props( 'messages' ) ).toEqual( {} );
+			expect( field.props( 'status' ) ).toBe( 'default' );
+		} );
 	} );
 
 	describe( 'multiple mode', () => {