refactor(router): extract createSpaRouter factory

Seven SPA routers (CAHFS, CMS, CTS, ClinicalScheduler, Computing, Effort,
Students) duplicated the same createRouter + VITE_VIPER_HOME + scroll-top
+ useRouteFocus setup. Hoist into src/shared/createSpaRouter.ts so each
router only owns its bespoke beforeEach guard. No behavior change.

Author: rlorenzo

VueApp/src/CAHFS/router/index.ts

@@ -1,15 +1,9 @@
-import { createRouter, createWebHistory } from "vue-router"
+import { createSpaRouter } from "@/shared/createSpaRouter"
 import { routes } from "./routes"
 import { useRequireLogin } from "@/composables/RequireLogin"
 import { checkHasOnePermission } from "@/composables/CheckPagePermission"
-import { useRouteFocus } from "@/composables/use-route-focus"
 
-const baseUrl = import.meta.env.VITE_VIPER_HOME
-const router = createRouter({
-    scrollBehavior: () => ({ left: 0, top: 0 }),
-    history: createWebHistory(baseUrl),
-    routes,
-})
+const router = createSpaRouter(routes)
 
 router.beforeEach(async (to) => {
     const { requireLogin } = useRequireLogin(to)
@@ -25,6 +19,4 @@ router.beforeEach(async (to) => {
     }
 })
 
-useRouteFocus(router)
-
 export { router as CAHFSRouter }

VueApp/src/CMS/router/index.ts

@@ -1,15 +1,9 @@
-import { createRouter, createWebHistory } from "vue-router"
+import { createSpaRouter } from "@/shared/createSpaRouter"
 import { routes } from "./routes"
 import { useRequireLogin } from "@/composables/RequireLogin"
 import { checkHasOnePermission } from "@/composables/CheckPagePermission"
-import { useRouteFocus } from "@/composables/use-route-focus"
 
-const baseUrl = import.meta.env.VITE_VIPER_HOME
-const router = createRouter({
-    scrollBehavior: () => ({ left: 0, top: 0 }),
-    history: createWebHistory(baseUrl),
-    routes,
-})
+const router = createSpaRouter(routes)
 
 router.beforeEach(async (to) => {
     const { requireLogin } = useRequireLogin(to)
@@ -25,6 +19,4 @@ router.beforeEach(async (to) => {
     }
 })
 
-useRouteFocus(router)
-
 export { router as cmsRouter }

VueApp/src/CTS/router/index.ts

@@ -1,15 +1,9 @@
-import { createRouter, createWebHistory } from "vue-router"
+import { createSpaRouter } from "@/shared/createSpaRouter"
 import { routes } from "./routes"
 import { useRequireLogin } from "@/composables/RequireLogin"
 import { checkHasOnePermission } from "@/composables/CheckPagePermission"
-import { useRouteFocus } from "@/composables/use-route-focus"
 
-const baseUrl = import.meta.env.VITE_VIPER_HOME
-const router = createRouter({
-    scrollBehavior: () => ({ left: 0, top: 0 }),
-    history: createWebHistory(baseUrl),
-    routes,
-})
+const router = createSpaRouter(routes)
 
 router.beforeEach(async (to) => {
     const { requireLogin } = useRequireLogin(to)
@@ -25,6 +19,4 @@ router.beforeEach(async (to) => {
     }
 })
 
-useRouteFocus(router)
-
 export { router }

VueApp/src/ClinicalScheduler/router/index.ts

@@ -1,20 +1,12 @@
-import { createRouter, createWebHistory } from "vue-router"
+import { createSpaRouter } from "@/shared/createSpaRouter"
 import { clinicalSchedulerRoutes as routes } from "./routes"
 import { useRequireLogin } from "@/composables/RequireLogin"
-import { useRouteFocus } from "@/composables/use-route-focus"
 
-const baseUrl = import.meta.env.VITE_VIPER_HOME
-const router = createRouter({
-    scrollBehavior: () => ({ left: 0, top: 0 }),
-    history: createWebHistory(baseUrl),
-    routes,
-})
+const router = createSpaRouter(routes)
 
 router.beforeEach((to) => {
     const { requireLogin } = useRequireLogin(to)
     return requireLogin(true, "SVMSecure.ClnSched")
 })
 
-useRouteFocus(router)
-
 export { router as clinicalSchedulerRouter }

VueApp/src/Computing/router/index.ts

@@ -1,20 +1,12 @@
-import { createRouter, createWebHistory } from "vue-router"
+import { createSpaRouter } from "@/shared/createSpaRouter"
 import { routes } from "./routes"
 import { useRequireLogin } from "@/composables/RequireLogin"
-import { useRouteFocus } from "@/composables/use-route-focus"
 
-const baseUrl = import.meta.env.VITE_VIPER_HOME
-const router = createRouter({
-    scrollBehavior: () => ({ left: 0, top: 0 }),
-    history: createWebHistory(baseUrl),
-    routes,
-})
+const router = createSpaRouter(routes)
 
 router.beforeEach((to) => {
     const { requireLogin } = useRequireLogin(to)
     return requireLogin()
 })
 
-useRouteFocus(router)
-
 export { router }

VueApp/src/Effort/router/index.ts

@@ -1,17 +1,11 @@
-import { createRouter, createWebHistory } from "vue-router"
+import { createSpaRouter } from "@/shared/createSpaRouter"
 import { effortRoutes as routes } from "./routes"
 import { useRequireLogin } from "@/composables/RequireLogin"
 import { checkHasOnePermission } from "@/composables/CheckPagePermission"
 import { useFetch } from "@/composables/ViperFetch"
 import { useUserStore } from "@/store/UserStore"
-import { useRouteFocus } from "@/composables/use-route-focus"
 
-const baseUrl = import.meta.env.VITE_VIPER_HOME
-const router = createRouter({
-    scrollBehavior: () => ({ left: 0, top: 0 }),
-    history: createWebHistory(baseUrl),
-    routes,
-})
+const router = createSpaRouter(routes)
 
 // Dedup latch: reuse in-flight fetch so concurrent navigations don't fire multiple requests
 let evalPermissionsPromise: Promise<void> | null = null
@@ -63,6 +57,4 @@ router.beforeEach(async (to, from) => {
     }
 })
 
-useRouteFocus(router)
-
 export { router as effortRouter }

VueApp/src/Students/router/index.ts

@@ -1,17 +1,11 @@
-import { createRouter, createWebHistory } from "vue-router"
+import { createSpaRouter } from "@/shared/createSpaRouter"
 import { routes } from "./routes"
 import { useRequireLogin } from "@/composables/RequireLogin"
-import { useRouteFocus } from "@/composables/use-route-focus"
 import { checkHasOnePermission } from "@/composables/CheckPagePermission"
 import { useFetch } from "@/composables/ViperFetch"
 import { useUserStore } from "@/store/UserStore"
 
-const baseUrl = import.meta.env.VITE_VIPER_HOME
-const router = createRouter({
-    scrollBehavior: () => ({ left: 0, top: 0 }),
-    history: createWebHistory(baseUrl),
-    routes,
-})
+const router = createSpaRouter(routes)
 
 // In-flight latch: dedups concurrent navigations but resets after each attempt so later
 // sessions (e.g. re-auth into an SIS role) can re-fetch instead of reusing a stale resolution.
@@ -66,6 +60,4 @@ router.beforeEach(async (to, from) => {
     }
 })
 
-useRouteFocus(router)
-
 export { router }

VueApp/src/shared/createSpaRouter.ts

@@ -0,0 +1,20 @@
+import { createRouter, createWebHistory } from "vue-router"
+import type { RouteRecordRaw, Router } from "vue-router"
+import { useRouteFocus } from "@/composables/use-route-focus"
+
+/**
+ * Standard VIPER SPA router: web history rooted at VITE_VIPER_HOME,
+ * scroll-to-top on every navigation, and route-change focus management
+ * for accessibility. Callers wire their own `beforeEach` guard since
+ * auth/permission rules vary per SPA.
+ */
+export function createSpaRouter(routes: RouteRecordRaw[]): Router {
+    const baseUrl = import.meta.env.VITE_VIPER_HOME
+    const router = createRouter({
+        scrollBehavior: () => ({ left: 0, top: 0 }),
+        history: createWebHistory(baseUrl),
+        routes,
+    })
+    useRouteFocus(router)
+    return router
+}