refactor: improve readability by adding braces to conditional statements and using array for path construction

Author: Noggling

packages/plugins/context-navigation/src/guard-handlers/consume-own-nav-token.ts

@@ -21,6 +21,7 @@ import type { OwnNavigationTokens } from '../apply-navigation';
  */
 export function consumeOwnNavToken(currentURL: URL, ownNavTokens: OwnNavigationTokens): boolean {
   const normalized = normalizePath(currentURL);
+  
   // Token present — this navigation was plugin-initiated; consume it and signal the caller to bail.
   if (ownNavTokens.has(normalized)) {
     ownNavTokens.delete(normalized);

packages/plugins/context-navigation/src/guard-handlers/handle-push-mode-guard.ts

@@ -35,14 +35,21 @@ export function handlePushModeGuard(
 
   // Resolve the active context and adapter — both are required to fall back.
   const activeContext = appModules.context.currentContext;
+
   // No active context to fall back to — nothing to do.
-  if (!activeContext) return;
+  if (!activeContext) {
+    return;
+  }
+  
   const adapter = resolveAdapter(
     { appKey, appContext: appModules.context, routingStrategy, currentURL },
     deps.config.adapters,
   );
+
   // No adapter matched — plugin is not responsible for this app's URL shape.
-  if (!adapter) return;
+  if (!adapter) {
+    return;
+  }
 
   log(`URL guard: URL has context [${urlContextId}] for [${appKey}] — setting context from URL`);
 

packages/plugins/context-navigation/src/guard-handlers/handle-replace-mode-guard.ts

@@ -29,14 +29,21 @@ export function handleReplaceModeGuard(
 
   // Resolve the active context and adapter for re-encoding.
   const activeContext = appModules.context.currentContext;
+  
   // No active context to re-encode — nothing to correct.
-  if (!activeContext) return;
+  if (!activeContext) {
+    return;
+  }
+
   const adapter = resolveAdapter(
     { appKey, appContext: appModules.context, routingStrategy, currentURL },
     deps.config.adapters,
   );
+
   // No adapter matched — plugin is not responsible for this app's URL shape.
-  if (!adapter) return;
+  if (!adapter) {
+    return;
+  }
 
   log(`URL guard: context missing from URL, re-applying for [${appKey}]`);
   // Always replace when correcting URL drift. The URL has no context segment

packages/plugins/context-navigation/src/helpers/resolve-adapter.ts

@@ -23,7 +23,9 @@ export function resolveAdapter(
   for (const entry of adapters) {
     const adapter = typeof entry === 'function' ? entry(ctx) : entry.canHandle(ctx) ? entry : null;
     // Return the first adapter that can handle this context
-    if (adapter) return adapter;
+    if (adapter) {
+      return adapter;
+    }
   }
   return null;
 }

packages/plugins/context-navigation/src/utils/url/build-app-route.ts

@@ -8,13 +8,13 @@
  * @returns The constructed app route path string.
  */
 export const buildAppRoute = (appKey: string, contextId?: string, rest?: string): string => {
-  let path = `/apps/${appKey}`;
+  let path = [`/apps/${appKey}`];
 
   // Context segment is optional — omitted when no context is actively selected
-  if (contextId) path += `/${contextId}`;
+  if (contextId) path.push(`${contextId}`);
 
   // Preserve any sub-route depth that exists beyond the context segment
-  if (rest) path += `/${rest}`;
+  if (rest) path.push(`${rest}`);
 
-  return path;
+  return path.join('/');
 };

packages/plugins/context-navigation/src/utils/url/build-context-url-for-strategy.ts

@@ -39,7 +39,11 @@ export const buildContextUrlForStrategy = (
 
   // Path routing embeds the context id directly in the URL structure
   const match = parseAppRoute(path);
+
   // Path is not a recognised app route — return it unchanged
-  if (!match) return path;
+  if (!match) {
+    return path;
+  } 
+  
   return buildAppRoute(match.appKey, contextId ?? undefined);
 };

packages/plugins/context-navigation/src/utils/url/parse-app-route.ts

@@ -39,11 +39,18 @@ export const parseAppRoute = (pathname: string): AppRouteMatch | undefined => {
 
   const normalized = pathname.replace(/\/+$/, '') || '/';
   const result = APP_ROUTE_PATTERN.exec({ pathname: normalized });
+  
   // Pathname does not match the /apps/:appKey pattern
-  if (!result) return undefined;
+  if (!result) {
+    return undefined;
+  }
+
   const { appKey, contextId, rest } = result.pathname.groups;
+
   // Guard against an empty appKey capture group (malformed pattern match)
-  if (!appKey) return undefined;
+  if (!appKey) {
+    return undefined;
+  }
   return {
     appKey,
     contextId: contextId || undefined,