mulmoclaude 0.1.2 → 0.4.0

package/src/composables/useSessionHistory.ts

@@ -1,10 +1,10 @@
-// Composable for the session-history dropdown in the header.
+// Composable for the session-history view at `/history`.
 //
-// Owns the `sessions` list (what the server knows about) and the
-// `showHistory` open/closed flag, plus the fetch + toggle helpers.
-// The dropdown lazy-loads the list only when opened, and callers
-// can invoke `fetchSessions()` directly after an end-of-run so the
-// sidebar title cache stays fresh.
+// Owns the `sessions` list (what the server knows about) plus the
+// fetch helper. The view's open/closed state is now URL-backed (see
+// plans/feat-history-url-route.md) — callers watch `route.name` and
+// invoke `fetchSessions()` on route enter rather than going through
+// an in-memory toggle flag.
 //
 // Since #205, `fetchSessions()` sends the server's last-issued
 // cursor back as `?since=<cursor>` so the server can reply with
@@ -26,15 +26,12 @@ interface SessionsResponse {
 
 export function useSessionHistory(): {
   sessions: Ref<SessionSummary[]>;
-  showHistory: Ref<boolean>;
   historyError: Ref<string | null>;
   fetchSessions: () => Promise<SessionSummary[]>;
-  toggleHistory: () => Promise<void>;
 } {
   const sessions = ref<SessionSummary[]>([]);
-  const showHistory = ref(false);
   // Surfaces the most recent fetch failure. Kept alongside the (stale)
-  // sessions list rather than wiping it — a dropdown that goes blank
+  // sessions list rather than wiping it — a panel that goes blank
   // the moment the network hiccups is worse UX than one that shows
   // "⚠ using cached list" with the last-known good entries.
   const historyError = ref<string | null>(null);
@@ -66,16 +63,9 @@ export function useSessionHistory(): {
     return sessions.value;
   }
 
-  async function toggleHistory(): Promise<void> {
-    showHistory.value = !showHistory.value;
-    if (showHistory.value) await fetchSessions();
-  }
-
   return {
     sessions,
-    showHistory,
     historyError,
     fetchSessions,
-    toggleHistory,
   };
 }

package/src/composables/useSessionSync.ts

@@ -29,20 +29,20 @@ export function useSessionSync(opts: {
       console.warn("[session-sync] failed to fetch sessions:", err);
       return;
     }
-    for (const s of summaries) {
-      const live = sessionMap.get(s.id);
+    for (const summary of summaries) {
+      const live = sessionMap.get(summary.id);
       if (!live) continue;
-      live.isRunning = s.isRunning ?? false;
-      live.statusMessage = s.statusMessage ?? "";
-      const unread = s.hasUnread ?? false;
-      if (!(unread && s.id === currentSessionId.value)) {
+      live.isRunning = summary.isRunning ?? false;
+      live.statusMessage = summary.statusMessage ?? "";
+      const unread = summary.hasUnread ?? false;
+      if (!(unread && summary.id === currentSessionId.value)) {
         live.hasUnread = unread;
       }
     }
   }
 
-  async function markSessionRead(id: string): Promise<void> {
-    const result = await apiPost<{ ok: boolean }>(API_ROUTES.sessions.markRead.replace(":id", encodeURIComponent(id)));
+  async function markSessionRead(sessionId: string): Promise<void> {
+    const result = await apiPost<{ ok: boolean }>(API_ROUTES.sessions.markRead.replace(":id", encodeURIComponent(sessionId)));
     if (!result.ok || result.data.ok === false) {
       await refreshSessionStates();
     }

package/src/composables/useViewLayout.ts

@@ -1,44 +1,31 @@
-// View-layout state: tracks whether the app is in a "stack" layout
-// (full-width canvas) or "single" (sidebar + canvas), remembers the
-// last chat-oriented view mode, and derives displayedCurrentSessionId
-// (blank in plugin views so no tab appears "current").
-
-import { computed, ref, watch, type ComputedRef, type Ref } from "vue";
-import { CANVAS_VIEW, type CanvasViewMode } from "../utils/canvas/viewMode";
-
-const CHAT_VIEWS = [CANVAS_VIEW.single, CANVAS_VIEW.stack] as const;
-type ChatViewMode = (typeof CHAT_VIEWS)[number];
-
-function isChatView(m: string): m is ChatViewMode {
-  return (CHAT_VIEWS as readonly string[]).includes(m);
-}
+// View-layout state. Derives two values that the template cares about:
+//
+//  - isStackLayout: true whenever the canvas column should be full-width
+//    (no sidebar). This is the case for /chat in stack mode AND for
+//    every non-chat page (/files, /todos, /wiki, etc.). Only /chat in
+//    single mode shows the left sidebar.
+//
+//  - displayedCurrentSessionId: blank on non-chat pages so no session
+//    tab appears "current" while the user is on Files, Todos, etc.
+//
+// Also flips activePane between "sidebar" and "main" so arrow-key
+// navigation follows whichever side of the layout is visible.
+
+import { computed, watch, type ComputedRef, type Ref } from "vue";
+import { LAYOUT_MODES, type LayoutMode } from "../utils/canvas/layoutMode";
 
 export function useViewLayout(opts: {
-  canvasViewMode: Ref<CanvasViewMode> | ComputedRef<CanvasViewMode>;
-  setCanvasViewMode: (mode: CanvasViewMode) => void;
+  layoutMode: Ref<LayoutMode> | ComputedRef<LayoutMode>;
+  isChatPage: Ref<boolean> | ComputedRef<boolean>;
   currentSessionId: Ref<string>;
   activePane: Ref<"sidebar" | "main">;
 }) {
-  const { canvasViewMode, setCanvasViewMode, currentSessionId, activePane } = opts;
-
-  const isStackLayout = computed(() => canvasViewMode.value !== CANVAS_VIEW.single);
-
-  const lastChatViewMode = ref<ChatViewMode>(isChatView(canvasViewMode.value) ? canvasViewMode.value : CANVAS_VIEW.stack);
-
-  watch(canvasViewMode, (mode) => {
-    if (isChatView(mode)) lastChatViewMode.value = mode;
-  });
+  const { layoutMode, isChatPage, currentSessionId, activePane } = opts;
 
-  function restoreChatViewForSession(): void {
-    if (!isChatView(canvasViewMode.value)) {
-      setCanvasViewMode(lastChatViewMode.value);
-    }
-  }
+  const isStackLayout = computed(() => !(isChatPage.value && layoutMode.value === LAYOUT_MODES.single));
 
-  const displayedCurrentSessionId = computed(() => (isChatView(canvasViewMode.value) ? currentSessionId.value : ""));
+  const displayedCurrentSessionId = computed(() => (isChatPage.value ? currentSessionId.value : ""));
 
-  // Keep arrow-key navigation tied to the canvas when the sidebar
-  // list doesn't exist (Stack layout has no ToolResultsPanel).
   watch(
     isStackLayout,
     (stack) => {
@@ -49,7 +36,6 @@ export function useViewLayout(opts: {
 
   return {
     isStackLayout,
-    restoreChatViewForSession,
     displayedCurrentSessionId,
   };
 }

package/src/config/roles.ts

@@ -330,6 +330,6 @@ export type BuiltInRoleId = (typeof BUILTIN_ROLE_IDS)[keyof typeof BUILTIN_ROLE_
 
 export const DEFAULT_ROLE_ID: BuiltInRoleId = BUILTIN_ROLE_IDS.general;
 
-export function getRole(id: string): Role {
-  return ROLES.find((r) => r.id === id) ?? ROLES[0];
+export function getRole(roleId: string): Role {
+  return ROLES.find((role) => role.id === roleId) ?? ROLES[0];
 }