iAPI: Fix using getServerContext in derived state getters (#73518)

* Add failing test for context

* Fix test

* Add changelog

* Don't trigger invalidations twice in the navigation phase

* Debounce navigation signal invalidation to prevent redundant updates within a microtask

* Revert "Debounce navigation signal invalidation to prevent redundant updates within a microtask"

This reverts commit c2bf701.

* Use a different navigation signal for context

Co-authored-by: luisherranz <[email protected]>
Co-authored-by: DAreRodz <[email protected]>

Author: luisherranz

packages/e2e-tests/plugins/interactive-blocks/get-server-context/render.php

@@ -55,6 +55,7 @@
 		<div data-testid="nonChanging" data-wp-text="context.nonChanging"></div>
 		<div data-testid="onlyInMain" data-wp-text="context.onlyInMain"></div>
 		<div data-testid="onlyInModified" data-wp-text="context.onlyInModified"></div>
+		<div data-testid="serverProp" data-wp-text="state.serverProp"></div>
 
 		<button
 			data-testid="tryToModifyServerContext"

packages/e2e-tests/plugins/interactive-blocks/get-server-context/view.js

@@ -9,6 +9,11 @@ import {
 } from '@wordpress/interactivity';
 
 store( 'test/get-server-context', {
+	state: {
+		get serverProp() {
+			return getServerContext().prop;
+		},
+	},
 	actions: {
 		navigate: withSyncEvent( function* ( e ) {
 			e.preventDefault();

packages/interactivity/CHANGELOG.md

@@ -5,6 +5,7 @@
 ### Bug Fixes
 
 -   Return a deep-clone object from `getServerState` and `getServerContext` functions. ([#73437](https://github.com/WordPress/gutenberg/pull/73437))
+-   Fix using `getServerContext` in derived state getters. ([#73518](https://github.com/WordPress/gutenberg/pull/73518))
 -   Fix derived state closures processing on client-side navigation. ([#72725](https://github.com/WordPress/gutenberg/pull/72725))
 
 ## 6.33.0 (2025-10-17)

packages/interactivity/src/directives.tsx

@@ -10,7 +10,7 @@ import {
 	type VNode,
 	type RefObject,
 } from 'preact';
-import { useContext, useMemo, useRef } from 'preact/hooks';
+import { useContext, useLayoutEffect, useMemo, useRef } from 'preact/hooks';
 import { signal, type Signal } from '@preact/signals';
 
 /**
@@ -33,7 +33,7 @@ import {
 	type DirectiveCallback,
 	type DirectiveEntry,
 } from './hooks';
-import { getScope } from './scopes';
+import { getScope, navigationContextSignal } from './scopes';
 import { proxifyState, proxifyContext, deepMerge } from './proxies';
 import { PENDING_GETTER } from './proxies/state';
 
@@ -1025,6 +1025,15 @@ export default () => {
 			// Get the content of this router region.
 			const vdom = routerRegions.get( regionId )!.value;
 
+			// Triggers an invalidation after the directive data-wp-context has
+			// been evaluated and the value of the server context has changed.
+			useLayoutEffect( () => {
+				if ( vdom && typeof vdom.type !== 'string' ) {
+					navigationContextSignal.value =
+						navigationContextSignal.peek() + 1;
+				}
+			}, [ vdom ] );
+
 			if ( vdom && typeof vdom.type !== 'string' ) {
 				// The scope needs to be injected.
 				const previousScope = getScope();

packages/interactivity/src/scopes.ts

@@ -2,12 +2,13 @@
  * External dependencies
  */
 import type { h as createElement, RefObject } from 'preact';
+import { signal } from '@preact/signals';
 
 /**
  * Internal dependencies
  */
 import { getNamespace } from './namespaces';
-import { deepReadOnly, navigationSignal, deepClone } from './utils';
+import { deepReadOnly, deepClone } from './utils';
 import type { Evaluate } from './hooks';
 
 export interface Scope {
@@ -82,6 +83,8 @@ export const getElement = () => {
 	} );
 };
 
+export const navigationContextSignal = signal( 0 );
+
 /**
  * Gets the context defined and updated from the server.
  *
@@ -125,7 +128,7 @@ export function getServerContext< T extends object >( namespace?: string ): T {
 
 	// Accesses the signal to make this reactive. It assigns it to `subscribe`
 	// to prevent the JavaScript minifier from removing this line.
-	getServerContext.subscribe = navigationSignal.value;
+	getServerContext.subscribe = navigationContextSignal.value;
 
 	return deepClone( scope.serverContext[ namespace || getNamespace() ] );
 }

packages/interactivity/src/store.ts

@@ -328,7 +328,6 @@ export const populateServerData = ( data?: {
 			}
 		);
 	}
-	navigationSignal.value += 1; // Triggers invalidations.
 };
 
 // Parse and populate the initial state and config.

test/e2e/specs/interactivity/get-sever-context.spec.ts

@@ -259,4 +259,18 @@ test.describe( 'getServerContext()', () => {
 		await expect( prop ).toBeEmpty();
 		await expect( nestedProp ).toBeEmpty();
 	} );
+
+	test( 'should get server context using derived state getters', async ( {
+		page,
+	} ) => {
+		const serverProp = page.getByTestId( 'serverProp' );
+
+		await expect( page ).toHaveTitle( /main/ );
+		await expect( serverProp ).toHaveText( 'child' );
+
+		await page.getByTestId( 'modified' ).click();
+		await expect( page ).toHaveTitle( /modified/ );
+
+		await expect( serverProp ).toHaveText( 'childModified' );
+	} );
 } );