fix(context-navigation): clear hash fragment on context changes

Hash fragments are intentionally cleared when context changes to prevent
inconsistent app state. When a context change resets the app to its root
view, preserving the hash would leave anchors pointing to sections that
may not exist in the new context.

This matches the existing behavior for sub-routes (intentionally dropped)
and maintains consistency across all three adapters (path, query, custom).

Adds test to verify hash is cleared on context change.

Addresses PR review feedback in #4751

Author: Noggling

packages/dev-portal/README.md

@@ -95,7 +95,7 @@ The portal is composed of these internal parts:
 
 - **`render`** — Entry point; creates a React root with theme, framework, and people-resolver providers.
 - **`configure`** — Configures all framework modules (telemetry, navigation, context navigation, bookmarks, feature flags, analytics, AG Grid, services).
-- **`configureDevPortalContext`** — Wires the context module's path generator and path extractor to the shared context-navigation URL utilities.
+- **Context configuration** — `enableContext` wires the context module's path generator and path extractor to the shared context-navigation URL utilities.
 - **`Router`** — Sets up routes with `react-router` via the navigation module; routes `/apps/:appKey/*` to the app loader.
 - **`AppLoader`** — Resolves, initializes, and mounts a Fusion app by key; handles loading states and errors.
 - **`Header`** — Top bar with the Fusion logo, context selector, bookmark toggle, and person settings.

packages/dev-portal/src/configure.ts

@@ -42,7 +42,7 @@ declare global {
  * Modules enabled:
  * - **Telemetry** — portal-scoped usage analytics with version metadata.
  * - **App** — application manifest loading and lifecycle.
- * - **Context** — context routing URL hooks via {@link configureDevPortalContext}.
+ * - **Context** — context routing URL hooks (path generator + extractor) wired to the shared context-navigation URL utilities.
  * - **Context Navigation plugin** — keeps the browser URL in sync with the
  *   active context, handles app-switch carry-over, and guards against
  *   accidental context loss. Telemetry is auto-resolved from the framework

packages/plugins/context-navigation/src/__tests__/active-app-navigation-events.test.ts

@@ -1,4 +1,4 @@
-import { BehaviorSubject, EMPTY, Subject } from 'rxjs';
+import { BehaviorSubject, Subject } from 'rxjs';
 import { describe, expect, it } from 'vitest';
 
 import type { AppModulesInstance } from '@equinor/fusion-framework-module-app';
@@ -15,6 +15,7 @@ describe('activeAppNavigationEvents$', () => {
     const currentApp$ = new BehaviorSubject({
       appKey: 'my-app',
       instance$: new BehaviorSubject(appModules),
+      manifest$: new BehaviorSubject(null),
     });
 
     const app = { current$: currentApp$ } as unknown as ContextNavigationPluginArgs['app'];
@@ -56,6 +57,7 @@ describe('activeAppNavigationEvents$', () => {
     const currentApp$ = new BehaviorSubject({
       appKey: 'my-app',
       instance$: new BehaviorSubject(null),
+      manifest$: new BehaviorSubject(null),
     });
 
     const app = { current$: currentApp$ } as unknown as ContextNavigationPluginArgs['app'];
@@ -79,6 +81,7 @@ describe('activeAppNavigationEvents$', () => {
     const currentApp$ = new BehaviorSubject({
       appKey: 'my-app',
       instance$: new BehaviorSubject(appModules),
+      manifest$: new BehaviorSubject(null),
     });
 
     const app = { current$: currentApp$ } as unknown as ContextNavigationPluginArgs['app'];
@@ -109,6 +112,7 @@ describe('activeAppNavigationEvents$', () => {
     const currentApp$ = new BehaviorSubject<unknown>({
       appKey: 'app-a',
       instance$: new BehaviorSubject(appModulesA),
+      manifest$: new BehaviorSubject(null),
     });
 
     const app = { current$: currentApp$ } as unknown as ContextNavigationPluginArgs['app'];
@@ -122,7 +126,7 @@ describe('activeAppNavigationEvents$', () => {
     navigationState$.next({});
     expect(emissions[0]?.appKey).toBe('app-a');
 
-    currentApp$.next({ appKey: 'app-b', instance$: new BehaviorSubject(appModulesB) });
+    currentApp$.next({ appKey: 'app-b', instance$: new BehaviorSubject(appModulesB), manifest$: new BehaviorSubject(null) });
     navigationState$.next({});
     expect(emissions[1]?.appKey).toBe('app-b');
 

packages/plugins/context-navigation/src/__tests__/plugin.test.ts

@@ -1,5 +1,5 @@
 import { BehaviorSubject, EMPTY, Subject } from 'rxjs';
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { afterEach, describe, expect, it, vi } from 'vitest';
 
 import type { AppModulesInstance } from '@equinor/fusion-framework-module-app';
 import type { ContextItem, ContextModule } from '@equinor/fusion-framework-module-context';
@@ -320,6 +320,35 @@ describe('createContextNavigationPlugin', () => {
         expect(targetURL.pathname).toBe('/apps/my-app/ctx-5');
       });
     });
+
+    it('clears hash fragment on context change to avoid inconsistent app state', async () => {
+      const adapter = makeAdapter({
+        encode: ({ context, currentURL }) => {
+          if (!context) return null;
+          const target = new URL(`/apps/my-app/${context.id}`, currentURL.origin);
+          target.search = currentURL.search;
+          // Hash intentionally not preserved — context change resets app to root view
+          return target;
+        },
+      });
+      harness = createHarness({ adapters: [adapter] });
+      Object.assign(harness.navigation, {
+        path: { pathname: '/apps/my-app', search: '', hash: '#section-2' },
+      });
+      const appModules = { context: { currentContext: null } } as unknown as AppModulesInstance<
+        [ContextModule]
+      >;
+
+      harness.source$.next({ appKey: 'my-app', appModules, contextState: makeContext('ctx-6') });
+
+      await vi.waitFor(() => {
+        const navigateCall = vi.mocked(harness.navigation.navigate).mock.calls[0];
+        const targetURL = navigateCall?.[0] as URL;
+        // Hash should be empty — dropped when context changes
+        expect(targetURL.hash).toBe('');
+        expect(targetURL.pathname).toBe('/apps/my-app/ctx-6');
+      });
+    });
   });
 
   // ─── Null Context URL ─────────────────────────────────────────────
@@ -373,6 +402,7 @@ describe('createContextNavigationPlugin', () => {
       const currentApp$ = new BehaviorSubject({
         appKey: 'my-app',
         instance$: new BehaviorSubject(appModules),
+        manifest$: new BehaviorSubject(null),
       });
 
       const app = { current$: currentApp$ } as unknown as ContextNavigationPluginArgs['app'];
@@ -423,6 +453,7 @@ describe('createContextNavigationPlugin', () => {
       const currentApp$ = new BehaviorSubject({
         appKey: 'my-app',
         instance$: new BehaviorSubject(appModules),
+        manifest$: new BehaviorSubject(null),
       });
 
       const app = { current$: currentApp$ } as unknown as ContextNavigationPluginArgs['app'];
@@ -480,6 +511,7 @@ describe('createContextNavigationPlugin', () => {
       const currentApp$ = new BehaviorSubject({
         appKey: 'my-app',
         instance$: new BehaviorSubject(appModules),
+        manifest$: new BehaviorSubject(null),
       });
 
       const app = { current$: currentApp$ } as unknown as ContextNavigationPluginArgs['app'];
@@ -530,6 +562,7 @@ describe('createContextNavigationPlugin', () => {
       const currentApp$ = new BehaviorSubject({
         appKey: 'my-app',
         instance$: new BehaviorSubject(appModules),
+        manifest$: new BehaviorSubject(null),
       });
 
       const app = { current$: currentApp$ } as unknown as ContextNavigationPluginArgs['app'];

packages/plugins/context-navigation/src/adapters/custom-adapter.ts

@@ -164,6 +164,7 @@ export function createCustomAdapter(): ContextNavigationAdapterFactory {
           url.search = currentURL.search;
           // Remove query-adapter param — custom apps encode context in the path
           stripContextQueryParam(url);
+          // Hash is intentionally not preserved — returning to root app view
           return url;
         }
 
@@ -191,6 +192,7 @@ export function createCustomAdapter(): ContextNavigationAdapterFactory {
         url.search = currentURL.search;
         // Remove query-adapter param — custom apps encode context in the path
         stripContextQueryParam(url);
+        // Hash is intentionally not preserved — context changes reset app to root view
         return url;
       },
 

packages/plugins/context-navigation/src/adapters/path-adapter.ts

@@ -78,6 +78,7 @@ export function createPathAdapter(): ContextNavigationAdapter {
       const url = new URL(targetPath, currentURL.origin);
       url.search = currentURL.search;
       stripContextQueryParam(url);
+      // Hash is intentionally not preserved — context changes reset app to root view
       return url;
     },
 

packages/plugins/context-navigation/src/adapters/query-adapter.ts

@@ -56,6 +56,9 @@ export function createQueryAdapter(paramName = CONTEXT_QUERY_PARAM_KEY): Context
      * After mutation, the `$` character is restored from `%24` encoding
      * because `URLSearchParams` percent-encodes `$` but we want it
      * human-readable in the address bar as a namespace marker.
+     *
+     * Hash fragment is intentionally cleared — context changes reset the app
+     * to its root view, so any hash pointing to sub-route sections would be invalid.
      */
     encode({ context, currentURL }: { context: ContextItem | null; currentURL: URL }): URL | null {
       const url = new URL(currentURL.href);
@@ -69,6 +72,9 @@ export function createQueryAdapter(paramName = CONTEXT_QUERY_PARAM_KEY): Context
       // Restore $ from %24 — $ is a reserved namespace prefix we want visible in URLs
       url.search = url.search.replace(/%24/gi, '$');
 
+      // Clear hash — context change resets app to root view
+      url.hash = '';
+
       return url;
     },