ikanban-web 0.2.7 → 0.2.8

package/src/pages/layout.tsx

@@ -279,59 +279,6 @@ export default function Layout(props: ParentProps) {
     setLocale(next)
   }
 
-  const useUpdatePolling = () =>
-    onMount(() => {
-      if (!platform.checkUpdate || !platform.update || !platform.restart) return
-
-      let toastId: number | undefined
-      let interval: ReturnType<typeof setInterval> | undefined
-
-      const pollUpdate = () =>
-        platform.checkUpdate!().then(({ updateAvailable, version }) => {
-          if (!updateAvailable) return
-          if (toastId !== undefined) return
-          toastId = showToast({
-            persistent: true,
-            icon: "download",
-            title: language.t("toast.update.title"),
-            description: language.t("toast.update.description", { version: version ?? "" }),
-            actions: [
-              {
-                label: language.t("toast.update.action.installRestart"),
-                onClick: async () => {
-                  await platform.update!()
-                  await platform.restart!()
-                },
-              },
-              {
-                label: language.t("toast.update.action.notYet"),
-                onClick: "dismiss",
-              },
-            ],
-          })
-        })
-
-      createEffect(() => {
-        if (!settings.ready()) return
-
-        if (!settings.updates.startup()) {
-          if (interval === undefined) return
-          clearInterval(interval)
-          interval = undefined
-          return
-        }
-
-        if (interval !== undefined) return
-        void pollUpdate()
-        interval = setInterval(pollUpdate, 10 * 60 * 1000)
-      })
-
-      onCleanup(() => {
-        if (interval === undefined) return
-        clearInterval(interval)
-      })
-    })
-
   const useSDKNotificationToasts = () =>
     onMount(() => {
       const toastBySession = new Map<string, number>()
@@ -440,7 +387,6 @@ export default function Layout(props: ParentProps) {
       })
     })
 
-  useUpdatePolling()
   useSDKNotificationToasts()
 
   function scrollToSession(sessionId: string, sessionKey: string) {
@@ -871,6 +817,17 @@ export default function Layout(props: ParentProps) {
         keybind: "mod+o",
         onSelect: () => chooseProject(),
       },
+      {
+        id: "project.close",
+        title: language.t("command.project.close"),
+        category: language.t("command.category.project"),
+        disabled: !currentProject(),
+        onSelect: () => {
+          const project = currentProject()
+          if (!project) return
+          closeProject(project.worktree)
+        },
+      },
       {
         id: "provider.connect",
         title: language.t("command.provider.connect"),

package/src/pages/session/review-tab.tsx

@@ -21,6 +21,7 @@ export interface SessionReviewTabProps {
   diffStyle: DiffStyle
   onDiffStyleChange?: (style: DiffStyle) => void
   onViewFile?: (file: string) => void
+  onRevealFile?: (file: string) => void
   onLineComment?: (comment: { file: string; selection: SelectedLineRange; comment: string; preview?: string }) => void
   onLineCommentUpdate?: (comment: SessionReviewCommentUpdate) => void
   onLineCommentDelete?: (comment: SessionReviewCommentDelete) => void
@@ -56,7 +57,7 @@ export function SessionReviewTab(props: SessionReviewTabProps) {
 
   const readFile = async (path: string) => {
     return sdk.client.file
-      .read({ path })
+      .read({ path, directory: sdk.directory })
       .then((x) => x.data)
       .catch((error) => {
         console.debug("[session-review] failed to read file", { path, error })
@@ -164,6 +165,7 @@ export function SessionReviewTab(props: SessionReviewTabProps) {
       diffStyle={props.diffStyle}
       onDiffStyleChange={props.onDiffStyleChange}
       onViewFile={props.onViewFile}
+      onRevealFile={props.onRevealFile}
       focusedFile={props.focusedFile}
       readFile={readFile}
       onLineComment={props.onLineComment}

package/src/pages/session/session-side-panel.tsx

@@ -20,7 +20,6 @@ import { useCommand } from "@/context/command"
 import { useFile, type SelectedLineRange } from "@/context/file"
 import { useLanguage } from "@/context/language"
 import { useLayout } from "@/context/layout"
-import { useSync } from "@/context/sync"
 import { createFileTabListSync } from "@/pages/session/file-tab-scroll"
 import { FileTabContent } from "@/pages/session/file-tabs"
 import { createOpenSessionFileTab, getTabReorderIndex } from "@/pages/session/helpers"
@@ -31,10 +30,15 @@ export function SessionSidePanel(props: {
   reviewPanel: () => JSX.Element
   activeDiff?: string
   focusReviewDiff: (path: string) => void
+  showReview: () => boolean
+  reviewCount: () => number
+  hasReview: () => boolean
+  diffsReady: () => boolean
+  diffFiles: () => string[]
+  kinds: () => Map<string, "add" | "del" | "mix">
 }) {
   const params = useParams()
   const layout = useLayout()
-  const sync = useSync()
   const file = useFile()
   const language = useLanguage()
   const command = useCommand()
@@ -49,44 +53,6 @@ export function SessionSidePanel(props: {
   const open = createMemo(() => isDesktop() && (view().reviewPanel.opened() || layout.fileTree.opened()))
   const reviewTab = createMemo(() => isDesktop())
 
-  const info = createMemo(() => (params.id ? sync.session.get(params.id) : undefined))
-  const diffs = createMemo(() => (params.id ? (sync.data.session_diff[params.id] ?? []) : []))
-  const reviewCount = createMemo(() => Math.max(info()?.summary?.files ?? 0, diffs().length))
-  const hasReview = createMemo(() => reviewCount() > 0)
-  const diffsReady = createMemo(() => {
-    const id = params.id
-    if (!id) return true
-    if (!hasReview()) return true
-    return sync.data.session_diff[id] !== undefined
-  })
-
-  const diffFiles = createMemo(() => diffs().map((d) => d.file))
-  const kinds = createMemo(() => {
-    const merge = (a: "add" | "del" | "mix" | undefined, b: "add" | "del" | "mix") => {
-      if (!a) return b
-      if (a === b) return a
-      return "mix" as const
-    }
-
-    const normalize = (p: string) => p.replaceAll("\\\\", "/").replace(/\/+$/, "")
-
-    const out = new Map<string, "add" | "del" | "mix">()
-    for (const diff of diffs()) {
-      const file = normalize(diff.file)
-      const kind = diff.status === "added" ? "add" : diff.status === "deleted" ? "del" : "mix"
-
-      out.set(file, kind)
-
-      const parts = file.split("/")
-      for (const [idx] of parts.slice(0, -1).entries()) {
-        const dir = parts.slice(0, idx + 1).join("/")
-        if (!dir) continue
-        out.set(dir, merge(out.get(dir), kind))
-      }
-    }
-    return out
-  })
-
   const normalizeTab = (tab: string) => {
     if (!tab.startsWith("file://")) return tab
     return file.tab(tab)
@@ -121,7 +87,7 @@ export function SessionSidePanel(props: {
     const first = openedTabs()[0]
     if (first) return first
     if (contextOpen()) return "context"
-    if (reviewTab() && hasReview()) return "review"
+    if (reviewTab() && props.showReview()) return "review"
     return "empty"
   })
 
@@ -236,8 +202,8 @@ export function SessionSidePanel(props: {
                       <Tabs.Trigger value="review">
                         <div class="flex items-center gap-1.5">
                           <div>{language.t("session.tab.review")}</div>
-                          <Show when={hasReview()}>
-                            <div>{reviewCount()}</div>
+                          <Show when={props.hasReview()}>
+                            <div>{props.reviewCount()}</div>
                           </Show>
                         </div>
                       </Tabs.Trigger>
@@ -356,8 +322,8 @@ export function SessionSidePanel(props: {
               >
                 <Tabs.List data-scrolled={store.fileTreeScrolled ? "" : undefined}>
                   <Tabs.Trigger value="changes" class="flex-1" classes={{ button: "w-full" }}>
-                    {reviewCount()}{" "}
-                    {language.t(reviewCount() === 1 ? "session.review.change.one" : "session.review.change.other")}
+                    {props.reviewCount()}{" "}
+                    {language.t(props.reviewCount() === 1 ? "session.review.change.one" : "session.review.change.other")}
                   </Tabs.Trigger>
                   <Tabs.Trigger value="all" class="flex-1" classes={{ button: "w-full" }}>
                     {language.t("session.files.all")}
@@ -370,9 +336,9 @@ export function SessionSidePanel(props: {
                   class="bg-background-stronger px-3 py-0"
                 >
                   <Switch>
-                    <Match when={hasReview()}>
+                    <Match when={props.hasReview()}>
                       <Show
-                        when={diffsReady()}
+                        when={props.diffsReady()}
                         fallback={
                           <div class="px-2 py-2 text-12-regular text-text-weak">
                             {language.t("common.loading")}
@@ -382,8 +348,8 @@ export function SessionSidePanel(props: {
                       >
                         <FileTree
                           path=""
-                          allowed={diffFiles()}
-                          kinds={kinds()}
+                          allowed={props.diffFiles()}
+                          kinds={props.kinds()}
                           draggable={false}
                           active={props.activeDiff}
                           onFileClick={(node) => props.focusReviewDiff(node.path)}
@@ -405,8 +371,8 @@ export function SessionSidePanel(props: {
                 >
                   <FileTree
                     path=""
-                    modified={diffFiles()}
-                    kinds={kinds()}
+                    modified={props.diffFiles()}
+                    kinds={props.kinds()}
                     onFileClick={(node) => openTab(file.tab(node.path))}
                   />
                 </Tabs.Content>

package/src/pages/session/use-session-commands.tsx

@@ -1,4 +1,5 @@
 import { createMemo } from "solid-js";
+import { produce } from "solid-js/store";
 import { useNavigate, useParams } from "@solidjs/router";
 import { useCommand, type CommandOption } from "@/context/command";
 import { useDialog } from "ikanban-ui/context/dialog";
@@ -20,6 +21,7 @@ import { DialogSelectFile } from "@/components/dialog-select-file";
 import { DialogSelectModel } from "@/components/dialog-select-model";
 import { DialogSelectMcp } from "@/components/dialog-select-mcp";
 import { DialogFork } from "@/components/dialog-fork";
+import { DialogSessionTimeline } from "@/components/dialog-session-timeline";
 import { showToast } from "ikanban-ui/toast";
 import { findLast } from "ikanban-utils/array";
 import { extractPromptFromParts } from "@/utils/prompt";
@@ -79,6 +81,77 @@ export const useSessionCommands = (actions: SessionCommandContext) => {
     return userMessages().filter((m) => m.id < revert);
   });
 
+  const refreshReviewDiffs = async (sessionID: string) => {
+    const directory = sdk.directory;
+    sync.set(
+      produce((draft) => {
+        delete draft.session_diff[sessionID];
+        delete draft.project_diff[directory];
+      }),
+    );
+    await Promise.allSettled([
+      sync.session.diff(sessionID),
+      sync.projectDiff.diff(),
+    ]);
+  };
+
+  const revertToMessage = async (message: UserMessage) => {
+    const sessionID = params.id;
+    if (!sessionID) return;
+    if (status()?.type !== "idle") {
+      await sdk.client.session.abort({ sessionID }).catch(() => { });
+    }
+    await sdk.client.session.revert({ sessionID, messageID: message.id });
+    const parts = sync.data.part[message.id];
+    if (parts) {
+      const restored = extractPromptFromParts(parts, {
+        directory: sdk.directory,
+      });
+      prompt.set(restored);
+    }
+    await refreshReviewDiffs(sessionID);
+    const priorMessage = findLast(userMessages(), (x) => x.id < message.id);
+    setActiveMessage(priorMessage);
+  };
+
+  const redoToMessage = async (message: UserMessage | undefined) => {
+    const sessionID = params.id;
+    if (!sessionID) return;
+    const revertMessageID = info()?.revert?.messageID;
+    if (!revertMessageID) return;
+    if (!message) {
+      await sdk.client.session.unrevert({ sessionID });
+      prompt.reset();
+      await refreshReviewDiffs(sessionID);
+      const lastMsg = findLast(
+        userMessages(),
+        (x) => x.id >= revertMessageID,
+      );
+      setActiveMessage(lastMsg);
+      return;
+    }
+    await sdk.client.session.revert({
+      sessionID,
+      messageID: message.id,
+    });
+    await refreshReviewDiffs(sessionID);
+    const priorMsg = findLast(userMessages(), (x) => x.id < message.id);
+    setActiveMessage(priorMsg);
+  };
+
+  const jumpToMessage = async (message: UserMessage | undefined) => {
+    const revertMessageID = info()?.revert?.messageID;
+    if (!message) {
+      await redoToMessage(undefined);
+      return;
+    }
+    if (!revertMessageID || message.id < revertMessageID) {
+      await revertToMessage(message);
+      return;
+    }
+    await redoToMessage(message);
+  };
+
   const showAllFiles = () => {
     if (layout.fileTree.tab() !== "changes") return;
     layout.fileTree.setTab("all");
@@ -317,29 +390,31 @@ export const useSessionCommands = (actions: SessionCommandContext) => {
       slash: "undo",
       disabled: !params.id || visibleUserMessages().length === 0,
       onSelect: async () => {
-        const sessionID = params.id;
-        if (!sessionID) return;
-        if (status()?.type !== "idle") {
-          await sdk.client.session.abort({ sessionID }).catch(() => { });
-        }
         const revert = info()?.revert?.messageID;
         const message = findLast(
           userMessages(),
           (x) => !revert || x.id < revert,
         );
         if (!message) return;
-        await sdk.client.session.revert({ sessionID, messageID: message.id });
-        const parts = sync.data.part[message.id];
-        if (parts) {
-          const restored = extractPromptFromParts(parts, {
-            directory: sdk.directory,
-          });
-          prompt.set(restored);
-        }
-        const priorMessage = findLast(userMessages(), (x) => x.id < message.id);
-        setActiveMessage(priorMessage);
+        await revertToMessage(message);
       },
     }),
+    sessionCommand({
+      id: "session.timeline",
+      title: language.t("command.session.timeline"),
+      description: language.t("command.session.timeline.description"),
+      slash: "timeline",
+      disabled: !params.id || userMessages().length === 0,
+      onSelect: () =>
+        dialog.show(() => (
+          <DialogSessionTimeline
+            onSelect={(message) => {
+              dialog.close();
+              void jumpToMessage(message);
+            }}
+          />
+        )),
+    }),
     sessionCommand({
       id: "session.redo",
       title: language.t("command.session.redo"),
@@ -347,27 +422,10 @@ export const useSessionCommands = (actions: SessionCommandContext) => {
       slash: "redo",
       disabled: !params.id || !info()?.revert?.messageID,
       onSelect: async () => {
-        const sessionID = params.id;
-        if (!sessionID) return;
         const revertMessageID = info()?.revert?.messageID;
         if (!revertMessageID) return;
         const nextMessage = userMessages().find((x) => x.id > revertMessageID);
-        if (!nextMessage) {
-          await sdk.client.session.unrevert({ sessionID });
-          prompt.reset();
-          const lastMsg = findLast(
-            userMessages(),
-            (x) => x.id >= revertMessageID,
-          );
-          setActiveMessage(lastMsg);
-          return;
-        }
-        await sdk.client.session.revert({
-          sessionID,
-          messageID: nextMessage.id,
-        });
-        const priorMsg = findLast(userMessages(), (x) => x.id < nextMessage.id);
-        setActiveMessage(priorMsg);
+        await redoToMessage(nextMessage);
       },
     }),
     sessionCommand({

package/src/pages/session.tsx

@@ -360,9 +360,8 @@ export default function Page() {
   })
 
   const info = createMemo(() => (params.id ? sync.session.get(params.id) : undefined))
-  const diffs = createMemo(() => (params.id ? (sync.data.session_diff[params.id] ?? []) : []))
-  const reviewCount = createMemo(() => Math.max(info()?.summary?.files ?? 0, diffs().length))
-  const hasReview = createMemo(() => reviewCount() > 0)
+  const sessionDiffs = createMemo(() => (params.id ? (sync.data.session_diff[params.id] ?? []) : []))
+  const projectDiffs = createMemo(() => sync.data.project_diff[sdk.directory] ?? [])
   const revertMessageID = createMemo(() => info()?.revert?.messageID)
   const messages = createMemo(() => (params.id ? (sync.data.message[params.id] ?? []) : []))
   const messagesReady = createMemo(() => {
@@ -414,7 +413,7 @@ export default function Page() {
   const [store, setStore] = createStore({
     messageId: undefined as string | undefined,
     mobileTab: "session" as "session" | "changes",
-    changes: "session" as "session" | "turn",
+    changes: "project" as "session" | "project" | "turn",
     newSessionWorktree: "main",
     deferRender: false,
   })
@@ -431,7 +430,42 @@ export default function Page() {
   }, sessionKey())
 
   const turnDiffs = createMemo(() => lastUserMessage()?.summary?.diffs ?? [])
-  const reviewDiffs = createMemo(() => (store.changes === "session" ? diffs() : turnDiffs()))
+  const reviewDiffs = createMemo(() => {
+    if (store.changes === "project") return projectDiffs()
+    if (store.changes === "turn") return turnDiffs()
+    return sessionDiffs()
+  })
+  const reviewDiffFiles = createMemo(() => reviewDiffs().map((diff) => diff.file))
+  const reviewKinds = createMemo(() => {
+    const merge = (a: "add" | "del" | "mix" | undefined, b: "add" | "del" | "mix") => {
+      if (!a) return b
+      if (a === b) return a
+      return "mix" as const
+    }
+
+    const normalize = (p: string) => p.replaceAll("\\", "/").replace(/\/+$/, "")
+
+    const out = new Map<string, "add" | "del" | "mix">()
+    for (const diff of reviewDiffs()) {
+      const file = normalize(diff.file)
+      const kind = diff.status === "added" ? "add" : diff.status === "deleted" ? "del" : "mix"
+
+      out.set(file, kind)
+
+      const parts = file.split("/")
+      for (const [idx] of parts.slice(0, -1).entries()) {
+        const dir = parts.slice(0, idx + 1).join("/")
+        if (!dir) continue
+        out.set(dir, merge(out.get(dir), kind))
+      }
+    }
+    return out
+  })
+  const reviewCount = createMemo(() => reviewDiffs().length)
+  const hasReview = createMemo(() => reviewCount() > 0)
+  const hasAnyReview = createMemo(
+    () => sessionDiffs().length > 0 || projectDiffs().length > 0 || turnDiffs().length > 0,
+  )
 
   const newSessionWorktree = createMemo(() => {
     if (store.newSessionWorktree === "create") return "create"
@@ -471,10 +505,17 @@ export default function Page() {
   const diffsReady = createMemo(() => {
     const id = params.id
     if (!id) return true
+    if (store.changes === "turn") return true
     if (!hasReview()) return true
+    if (store.changes === "project") return sync.data.project_diff[sdk.directory] !== undefined
     return sync.data.session_diff[id] !== undefined
   })
   const reviewEmptyKey = createMemo(() => {
+    if (store.changes === "project") {
+      const project = sync.project
+      if (!project || project.vcs) return "session.review.emptyProject"
+      return "session.review.noVcsProject"
+    }
     const project = sync.project
     if (!project || project.vcs) return "session.review.empty"
     return "session.review.noVcs"
@@ -528,7 +569,7 @@ export default function Page() {
       sessionKey,
       () => {
         setStore("messageId", undefined)
-        setStore("changes", "session")
+        setStore("changes", "project")
       },
       { defer: true },
     ),
@@ -705,16 +746,18 @@ export default function Page() {
     loadFile: file.load,
   })
 
-  const changesOptions = ["session", "turn"] as const
+  const changesOptions = ["project", "session", "turn"] as const
   const changesOptionsList = [...changesOptions]
 
   const changesTitle = () => (
     <Select
       options={changesOptionsList}
       current={store.changes}
-      label={(option) =>
-        option === "session" ? language.t("ui.sessionReview.title") : language.t("ui.sessionReview.title.lastTurn")
-      }
+      label={(option) => {
+        if (option === "project") return language.t("ui.sessionReview.title.project")
+        if (option === "turn") return language.t("ui.sessionReview.title.lastTurn")
+        return language.t("ui.sessionReview.title")
+      }}
       onSelect={(option) => option && setStore("changes", option)}
       variant="ghost"
       size="small"
@@ -743,6 +786,10 @@ export default function Page() {
             title={changesTitle()}
             empty={emptyTurn()}
             diffs={reviewDiffs}
+            onRevealFile={(file) => {
+              if (store.changes !== "project") return
+              void sync.projectDiff.hydrate(file)
+            }}
             view={view}
             diffStyle={input.diffStyle}
             onDiffStyleChange={input.onDiffStyleChange}
@@ -767,6 +814,10 @@ export default function Page() {
             <SessionReviewTab
               title={changesTitle()}
               diffs={reviewDiffs}
+              onRevealFile={(file) => {
+                if (store.changes !== "project") return
+                void sync.projectDiff.hydrate(file)
+              }}
               view={view}
               diffStyle={input.diffStyle}
               onDiffStyleChange={input.onDiffStyleChange}
@@ -935,14 +986,14 @@ export default function Page() {
     const first = openedTabs()[0]
     if (first) return first
     if (contextOpen()) return "context"
-    if (reviewTab() && hasReview()) return "review"
+    if (reviewTab() && hasAnyReview()) return "review"
     return "empty"
   })
 
   createEffect(() => {
     if (!layout.ready()) return
     if (tabs().active()) return
-    if (openedTabs().length === 0 && !contextOpen() && !(reviewTab() && hasReview())) return
+    if (openedTabs().length === 0 && !contextOpen() && !(reviewTab() && hasAnyReview())) return
 
     const next = activeTab()
     if (next === "empty") return
@@ -958,7 +1009,7 @@ export default function Page() {
 
         if (opened) {
           const active = tabs().active()
-          const tab = active === "review" || (!active && hasReview()) ? "changes" : "all"
+          const tab = active === "review" || (!active && hasAnyReview()) ? "changes" : "all"
           layout.fileTree.setTab(tab)
         }
       },
@@ -974,9 +1025,19 @@ export default function Page() {
       ? desktopFileTreeOpen() || (desktopReviewOpen() && activeTab() === "review")
       : store.mobileTab === "changes"
     if (!wants) return
-    if (sync.data.session_diff[id] !== undefined) return
+    if (store.changes === "project") {
+      if (sync.data.project_diff[sdk.directory] !== undefined) return
+    } else if (store.changes === "session") {
+      if (sync.data.session_diff[id] !== undefined) return
+    } else {
+      return
+    }
     if (sync.status === "loading") return
 
+    if (store.changes === "project") {
+      void sync.projectDiff.diff()
+      return
+    }
     void sync.session.diff(id)
   })
 
@@ -1165,7 +1226,7 @@ export default function Page() {
         <SessionMobileTabs
           open={!isDesktop() && !!params.id}
           mobileTab={store.mobileTab}
-          hasReview={hasReview()}
+          hasReview={hasAnyReview()}
           reviewCount={reviewCount()}
           onSession={() => setStore("mobileTab", "session")}
           onChanges={() => setStore("mobileTab", "changes")}
@@ -1281,7 +1342,17 @@ export default function Page() {
           </Show>
         </div>
 
-        <SessionSidePanel reviewPanel={reviewPanel} activeDiff={tree.activeDiff} focusReviewDiff={focusReviewDiff} />
+        <SessionSidePanel
+          reviewPanel={reviewPanel}
+          activeDiff={tree.activeDiff}
+          focusReviewDiff={focusReviewDiff}
+          showReview={hasAnyReview}
+          reviewCount={reviewCount}
+          hasReview={hasReview}
+          diffsReady={diffsReady}
+          diffFiles={reviewDiffFiles}
+          kinds={reviewKinds}
+        />
       </div>
 
       <TerminalPanel />