From a3071a53cb1ae86ac39adce28a70943516ca4694 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Sun, 26 Apr 2026 22:46:47 -0400
Subject: [PATCH] feat(routing): land on Journal at root; move Knowledge to
 /knowledge
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Routing changes:
- / now redirects to /journal (Journal becomes the home view)
- /knowledge becomes a real route pointing at KnowledgeView (was a redirect to /)
- /notes redirect target updated from / to /knowledge (the Knowledge surface,
  which is where the notes-related dashboard lives)

To revert (Knowledge as home): change the redirect target on the / route
back to "/knowledge". Knowledge stays a real route either way, so the swap
is a one-line edit.

Nav: Knowledge link in AppHeader now points to /knowledge. The "active" check
simplified to route.path === "/knowledge" since / is no longer Knowledge's
home. The brand logo link stays at "/" — clicks still go "home", which is
now Journal.

Keyboard shortcuts in App.vue (Escape, g h, g t) still navigate to "/" and
correctly land on Journal via the redirect — no change needed there.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
---
 frontend/src/components/AppHeader.vue |  6 +++---
 frontend/src/router/index.ts          | 11 +++++++----
 2 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/frontend/src/components/AppHeader.vue b/frontend/src/components/AppHeader.vue
index 839574c..7bf8791 100644
--- a/frontend/src/components/AppHeader.vue
+++ b/frontend/src/components/AppHeader.vue
@@ -16,7 +16,7 @@ const router = useRouter();
 const route = useRoute();
 
 const isChatActive = computed(() => route.path.startsWith("/chat"))
-const isKnowledgeActive = computed(() => route.path === "/" || route.path === "/knowledge");
+const isKnowledgeActive = computed(() => route.path === "/knowledge");
 
 const mobileMenuOpen = ref(false);
 
@@ -74,7 +74,7 @@ router.afterEach(() => {
       <!-- Center: primary navigation (desktop) -->
       <div class="nav-center">
         <div class="nav-pill-bar">
-          <router-link to="/" class="nav-link" :class="{ 'router-link-active': isKnowledgeActive }">Knowledge</router-link>
+          <router-link to="/knowledge" class="nav-link" :class="{ 'router-link-active': isKnowledgeActive }">Knowledge</router-link>
           <router-link to="/chat" :class="['nav-link', { 'router-link-active': isChatActive }]">Chat</router-link>
           <router-link to="/journal" class="nav-link">Journal</router-link>
           <router-link to="/calendar" class="nav-link">Calendar</router-link>
@@ -122,7 +122,7 @@ router.afterEach(() => {
 
     <!-- Mobile dropdown -->
     <div v-if="mobileMenuOpen" class="mobile-menu">
-      <router-link to="/" class="nav-link" :class="{ 'router-link-active': isKnowledgeActive }">Knowledge</router-link>
+      <router-link to="/knowledge" class="nav-link" :class="{ 'router-link-active': isKnowledgeActive }">Knowledge</router-link>
       <router-link to="/chat" :class="['nav-link', { 'router-link-active': isChatActive }]">Chat</router-link>
       <router-link to="/journal" class="nav-link">Journal</router-link>
       <router-link to="/calendar" class="nav-link">Calendar</router-link>
diff --git a/frontend/src/router/index.ts b/frontend/src/router/index.ts
index f98ad5e..3999b04 100644
--- a/frontend/src/router/index.ts
+++ b/frontend/src/router/index.ts
@@ -5,13 +5,16 @@ const router = createRouter({
   history: createWebHistory(),
   routes: [
     {
+      // Root lands on the journal. To revert to Knowledge as home, change
+      // the redirect target below to "/knowledge" (Knowledge stays a real
+      // route at /knowledge, so the swap is a one-line edit).
       path: "/",
-      name: "knowledge",
-      component: () => import("@/views/KnowledgeView.vue"),
+      redirect: "/journal",
     },
     {
       path: "/knowledge",
-      redirect: "/",
+      name: "knowledge",
+      component: () => import("@/views/KnowledgeView.vue"),
     },
     {
       path: "/login",
@@ -45,7 +48,7 @@ const router = createRouter({
     },
     {
       path: "/notes",
-      redirect: "/",
+      redirect: "/knowledge",
     },
     {
       path: "/notes/new",