f98395f000
* fix(agentctl): start per-repo workspace trackers in passthrough mode CLI passthrough mode starts the agent CLI via the PTY-based InteractiveRunner instead of agentctl's startAgent / startOneShot, so the per-repo workspace trackers created for multi-repo task roots were never started. The bare root tracker exits immediately because it has no git index, leaving polling completely idle — file edits made by the agent never produced workspace-stream notifications and the Files panel stayed stale. Move the tracker startup to instance.Manager.CreateInstance via a new StartAllWorkspaceTrackers helper so polling fires for every repo regardless of whether the agent runs via ACP or passthrough. Fixes #982 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix(lifecycle): serialize workspace poll-mode pushes to prevent slow-stuck mode When a session subscribes and then immediately focuses (the normal page-load sequence), the workspace poll aggregator used to spawn two independent goroutines — one pushing slow, one pushing fast — whose HTTP arrival order at agentctl was undefined. With the slow request landing after the fast request the tracker ended up in slow mode (~30s polling) while the aggregator's lastPushed claimed fast, and the suppress-duplicate guard then blocked any future focus event from reconciling the divergence. Multi-repo passthrough sessions hit this every page load: the Files panel saw ~30s-stale state even with the per-repo trackers running. Replace the fire-and-forget goroutine per push with a single in-flight pusher per workspace plus a last-write-wins pending slot — concurrent callers all converge on the same pusher, which drains the queue in order so the final queued mode is what reaches the wire last. Regression test in manager_subscription_test.go simulates the race with an httptest server that adds artificial latency on slow pushes; the test fails on the old parallel-goroutine implementation and passes on the serialized one. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix(agentctl): address PR review feedback for passthrough tracker fix - Goroutine leak: extract stopWorkspaceTrackers() and call it unconditionally at the top of Manager.Stop(). Previously the early return when status==Stopped skipped tracker teardown in passthrough mode (where Start() never runs), leaking 2 goroutines per per-repo tracker per stopped instance. stopShellAndProcesses delegates to the same helper. - Trim multi-line doc comments to one-liners per CLAUDE.md (the StartAllWorkspaceTrackers docstring and the CreateInstance call-site comment). - Replace the fixed time.Sleep in the race regression test with a poll on len(calls), exiting as soon as the pusher has drained, plus a brief settle window after to keep the test sensitive to the late racing slow push the fix guards against. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix(web): refresh expanded subtrees on polling-detected file changes The agentctl workspace tracker emits a generic `refresh` event with empty path when its poll cycle detects file changes (it knows something changed but not which file). The Files panel previously only refreshed the root folder on these events, so files added under an expanded subdir never became visible — the merge preserved the stale subtree, and the user had to manually collapse + re-expand to see new files. This is the user-visible symptom of #982 (Files panel does not refresh after agent makes changes in CLI passthrough mode): the backend fixes in this PR get the events flowing, but the frontend needs to act on them correctly. On a `refresh` event we now refresh every currently-expanded folder in addition to the root; for multi-repo task roots we scope to the affected repo's branch using `repository_name`. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix: address iteration-2 review feedback - Trim doc + inline comment blocks per CLAUDE.md (manager.go, file-browser-hooks.ts, workspace-files.ts). - Replace the hand-rolled fmtSscanf/numErr port parser with strconv.Atoi in the race regression test. - Assert on slowCalls in the race test to lock in the queue-collapse guarantee (at most one slow push lands). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix: address iteration-3 review feedback - Remove redundant stopWorkspaceTrackers call from stopShellAndProcesses (it's already invoked at the top of Manager.Stop), and drop the now- misleading "stopping workspace tracker" debug log from the regular shutdown path. - Trim three multi-line comment blocks in manager_subscription.go to one-liners per CLAUDE.md (workspacePushTarget, pushAsync, pushLoop). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix: trim multi-line comment block on pendingPush field * fix: trim test docstring on race regression test * fix(web): preserve sibling-repo subtrees on scoped refresh The setTree reducer ran patchNode(updated) at the root, which re-matched folderUpdates.has("") and overwrote the already-merged root children with the raw depth-1 server response — wiping the loaded children of every top-level repo that wasn't part of the refresh scope. Recurse over updated.children instead of updated itself: the root has already been merged in the block above, and each top-level repo either gets its targeted update (if folderUpdates.has(path)) or is left alone (preserving its loaded subtree). Includes a regression test that fails on the old code: a refresh scoped to "thm" used to discard "kandev"'s loaded README.md child. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * fix(web): trim three more comment blocks to one-liners * fix: trim five more comment blocks to one-liners --------- Co-authored-by: Kandev Agent <agent@kandev.dev> Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
600 lines
21 KiB
TypeScript
600 lines
21 KiB
TypeScript
"use client";
|
|
|
|
import { useEffect, useState, useCallback, useRef, useMemo } from "react";
|
|
import { getWebSocketClient } from "@/lib/ws/connection";
|
|
import {
|
|
requestFileTree,
|
|
requestFileContent,
|
|
searchWorkspaceFiles,
|
|
} from "@/lib/ws/workspace-files";
|
|
import type { FileTreeNode, FileContentResponse, OpenFileTab } from "@/lib/types/backend";
|
|
import { useSessionAgentctl } from "@/hooks/domains/session/use-session-agentctl";
|
|
import {
|
|
getFilesPanelExpandedPaths,
|
|
setFilesPanelExpandedPaths,
|
|
getFilesPanelScrollPosition,
|
|
setFilesPanelScrollPosition,
|
|
} from "@/lib/local-storage";
|
|
import type { useToast } from "@/components/toast-provider";
|
|
import { useTree, type VisibleRow } from "@/hooks/use-tree";
|
|
import { mergeTreeNodes } from "./file-browser-parts";
|
|
import { compareTreeNodes, sortRootChildren } from "./file-tree-utils";
|
|
|
|
const FB_GET_PATH = (n: FileTreeNode) => n.path;
|
|
// Children are sorted (dirs first, then files, alphabetically) on every
|
|
// access so the flat visibleRows list comes out in the same order the
|
|
// recursive render produced. The backend doesn't guarantee order and tree
|
|
// mutations (rename / create) can insert arbitrarily.
|
|
const FB_GET_CHILDREN = (n: FileTreeNode) =>
|
|
n.children ? [...n.children].sort(compareTreeNodes) : undefined;
|
|
const FB_IS_DIR = (n: FileTreeNode) => n.is_dir;
|
|
|
|
export type FileBrowserRow = VisibleRow<FileTreeNode>;
|
|
|
|
const MAX_RETRY_ATTEMPTS = 4;
|
|
const RETRY_DELAYS_MS = [1000, 2000, 5000, 10000];
|
|
|
|
export type LoadState = "loading" | "waiting" | "loaded" | "manual" | "error";
|
|
|
|
/** Hook encapsulating file search state and handlers. */
|
|
export function useFileBrowserSearch(sessionId: string) {
|
|
const [isSearchActive, setIsSearchActive] = useState(false);
|
|
const [localSearchQuery, setLocalSearchQuery] = useState("");
|
|
const [searchResults, setSearchResults] = useState<string[] | null>(null);
|
|
const [isSearching, setIsSearching] = useState(false);
|
|
const searchTimeoutRef = useRef<NodeJS.Timeout | null>(null);
|
|
const searchInputRef = useRef<HTMLInputElement>(null);
|
|
|
|
// Focus search input when search opens
|
|
useEffect(() => {
|
|
if (isSearchActive && searchInputRef.current) {
|
|
searchInputRef.current.focus();
|
|
}
|
|
}, [isSearchActive]);
|
|
|
|
// Clear search when closing
|
|
useEffect(() => {
|
|
if (!isSearchActive) {
|
|
setLocalSearchQuery("");
|
|
setSearchResults(null);
|
|
setIsSearching(false);
|
|
if (searchTimeoutRef.current) clearTimeout(searchTimeoutRef.current);
|
|
}
|
|
}, [isSearchActive]);
|
|
|
|
// Cleanup on unmount
|
|
useEffect(() => {
|
|
return () => {
|
|
if (searchTimeoutRef.current) clearTimeout(searchTimeoutRef.current);
|
|
};
|
|
}, []);
|
|
|
|
const handleSearchChange = useCallback(
|
|
(e: React.ChangeEvent<HTMLInputElement>) => {
|
|
const value = e.target.value;
|
|
setLocalSearchQuery(value);
|
|
if (searchTimeoutRef.current) clearTimeout(searchTimeoutRef.current);
|
|
if (!value.trim()) {
|
|
setSearchResults(null);
|
|
setIsSearching(false);
|
|
return;
|
|
}
|
|
setIsSearching(true);
|
|
searchTimeoutRef.current = setTimeout(async () => {
|
|
try {
|
|
const client = getWebSocketClient();
|
|
if (!client) return;
|
|
const response = await searchWorkspaceFiles(client, sessionId, value, 50);
|
|
setSearchResults(response.files || []);
|
|
} catch (error) {
|
|
console.error("Failed to search files:", error);
|
|
setSearchResults([]);
|
|
} finally {
|
|
setIsSearching(false);
|
|
}
|
|
}, 300);
|
|
},
|
|
[sessionId],
|
|
);
|
|
|
|
const handleCloseSearch = useCallback(() => {
|
|
setIsSearchActive(false);
|
|
setLocalSearchQuery("");
|
|
setSearchResults(null);
|
|
setIsSearching(false);
|
|
if (searchTimeoutRef.current) clearTimeout(searchTimeoutRef.current);
|
|
}, []);
|
|
|
|
return {
|
|
isSearchActive,
|
|
setIsSearchActive,
|
|
localSearchQuery,
|
|
searchResults,
|
|
isSearching,
|
|
searchInputRef,
|
|
handleSearchChange,
|
|
handleCloseSearch,
|
|
};
|
|
}
|
|
|
|
/** Apply incoming file changes to the tree by refreshing affected folders. */
|
|
export function applyFileChanges(ctx: {
|
|
client: ReturnType<typeof getWebSocketClient>;
|
|
sessionId: string;
|
|
expandedPaths: ReadonlySet<string>;
|
|
changes: Array<{ path: string; operation?: string; repository_name?: string }>;
|
|
setTree: React.Dispatch<React.SetStateAction<FileTreeNode | null>>;
|
|
setLoadState: React.Dispatch<React.SetStateAction<LoadState>>;
|
|
}) {
|
|
const { client, sessionId, expandedPaths, changes, setTree, setLoadState } = ctx;
|
|
const foldersToRefresh = new Set<string>();
|
|
for (const change of changes) {
|
|
// `refresh` events have empty path; refresh root + every expanded folder under the affected repo so new files show up.
|
|
if (change.operation === "refresh") {
|
|
foldersToRefresh.add("");
|
|
const repo = change.repository_name;
|
|
for (const exp of expandedPaths) {
|
|
if (!repo || exp === repo || exp.startsWith(repo + "/")) {
|
|
foldersToRefresh.add(exp);
|
|
}
|
|
}
|
|
continue;
|
|
}
|
|
const p = change.path;
|
|
const lastSlash = p.lastIndexOf("/");
|
|
const parent = lastSlash === -1 ? "" : p.substring(0, lastSlash);
|
|
if (parent === "" || expandedPaths.has(parent)) foldersToRefresh.add(parent);
|
|
if (p === "" || expandedPaths.has(p)) foldersToRefresh.add(p);
|
|
}
|
|
if (foldersToRefresh.size === 0) return;
|
|
|
|
void (async () => {
|
|
try {
|
|
const folderUpdates = new Map<string, FileTreeNode[] | undefined>();
|
|
await Promise.all(
|
|
Array.from(foldersToRefresh).map(async (folder) => {
|
|
try {
|
|
const res = await requestFileTree(client!, sessionId, folder || "", 1);
|
|
folderUpdates.set(folder, res.root?.children);
|
|
} catch {
|
|
/* Folder may have been removed */
|
|
}
|
|
}),
|
|
);
|
|
setTree((prev) => {
|
|
if (!prev) return prev;
|
|
let updated = prev;
|
|
if (folderUpdates.has("")) {
|
|
const freshRootChildren = folderUpdates.get("");
|
|
const existingByPath = new Map((updated.children ?? []).map((c) => [c.path, c]));
|
|
const mergedRootChildren = freshRootChildren?.map((incoming) => {
|
|
const existing = existingByPath.get(incoming.path);
|
|
return existing && existing.is_dir && incoming.is_dir
|
|
? mergeTreeNodes(existing, incoming)
|
|
: incoming;
|
|
});
|
|
updated = { ...updated, children: mergedRootChildren };
|
|
}
|
|
const subFolders = Array.from(folderUpdates.keys()).filter((k) => k !== "");
|
|
if (subFolders.length === 0) return updated;
|
|
const patchNode = (node: FileTreeNode): FileTreeNode => {
|
|
if (node.is_dir && folderUpdates.has(node.path)) {
|
|
return { ...node, children: folderUpdates.get(node.path)?.map(patchNode) };
|
|
}
|
|
return node.children ? { ...node, children: node.children.map(patchNode) } : node;
|
|
};
|
|
// Skip root: already merged above; re-matching folderUpdates.has("") here would overwrite preserved subtrees.
|
|
return { ...updated, children: updated.children?.map(patchNode) };
|
|
});
|
|
setLoadState("loaded");
|
|
} catch (error) {
|
|
console.error("[FileBrowser] Failed to refresh file tree:", error);
|
|
}
|
|
})();
|
|
}
|
|
|
|
function useLoadingTimers() {
|
|
const loadingTimersRef = useRef<Map<string, NodeJS.Timeout>>(new Map());
|
|
const activeLoadsRef = useRef<Set<string>>(new Set());
|
|
const [visibleLoadingPaths, setVisibleLoadingPaths] = useState<Set<string>>(new Set());
|
|
|
|
const showLoading = useCallback((path: string) => {
|
|
activeLoadsRef.current.add(path);
|
|
const timer = setTimeout(() => {
|
|
setVisibleLoadingPaths((prev) => new Set(prev).add(path));
|
|
loadingTimersRef.current.delete(path);
|
|
}, 150);
|
|
loadingTimersRef.current.set(path, timer);
|
|
}, []);
|
|
|
|
const hideLoading = useCallback((path: string) => {
|
|
activeLoadsRef.current.delete(path);
|
|
const timer = loadingTimersRef.current.get(path);
|
|
if (timer) {
|
|
clearTimeout(timer);
|
|
loadingTimersRef.current.delete(path);
|
|
}
|
|
setVisibleLoadingPaths((prev) => {
|
|
const next = new Set(prev);
|
|
next.delete(path);
|
|
return next;
|
|
});
|
|
}, []);
|
|
|
|
const isLoading = useCallback((path: string) => activeLoadsRef.current.has(path), []);
|
|
|
|
return { visibleLoadingPaths, showLoading, hideLoading, isLoading };
|
|
}
|
|
|
|
type TreeLoaderContext = {
|
|
sessionId: string;
|
|
clearRetryTimer: () => void;
|
|
retryAttemptRef: React.MutableRefObject<number>;
|
|
retryTimerRef: React.MutableRefObject<NodeJS.Timeout | null>;
|
|
setTree: React.Dispatch<React.SetStateAction<FileTreeNode | null>>;
|
|
setIsLoadingTree: React.Dispatch<React.SetStateAction<boolean>>;
|
|
setLoadState: React.Dispatch<React.SetStateAction<LoadState>>;
|
|
setLoadError: React.Dispatch<React.SetStateAction<string | null>>;
|
|
};
|
|
|
|
function useTreeLoader(ctx: TreeLoaderContext) {
|
|
const {
|
|
clearRetryTimer,
|
|
retryAttemptRef,
|
|
retryTimerRef,
|
|
setTree,
|
|
setIsLoadingTree,
|
|
setLoadState,
|
|
setLoadError,
|
|
} = ctx;
|
|
// Use ref for sessionId so loadTree stays stable across session switches.
|
|
// This prevents the tree-clearing effect from re-firing when only sessionId changes.
|
|
const sessionIdRef = useRef(ctx.sessionId);
|
|
sessionIdRef.current = ctx.sessionId;
|
|
const loadInFlightRef = useRef(false);
|
|
const loadTree = useCallback(
|
|
async (options?: { resetRetry?: boolean }) => {
|
|
if (loadInFlightRef.current) return;
|
|
loadInFlightRef.current = true;
|
|
setIsLoadingTree(true);
|
|
setLoadState("loading");
|
|
setLoadError(null);
|
|
if (options?.resetRetry) {
|
|
retryAttemptRef.current = 0;
|
|
clearRetryTimer();
|
|
}
|
|
try {
|
|
const client = getWebSocketClient();
|
|
if (!client) throw new Error("WebSocket client not available");
|
|
const response = await requestFileTree(client, sessionIdRef.current, "", 1);
|
|
setTree(response.root ?? null);
|
|
setLoadState("loaded");
|
|
retryAttemptRef.current = 0;
|
|
clearRetryTimer();
|
|
} catch (error) {
|
|
const message = error instanceof Error ? error.message : "Failed to load file tree";
|
|
setLoadError(message);
|
|
if (retryAttemptRef.current < MAX_RETRY_ATTEMPTS) {
|
|
const delay =
|
|
RETRY_DELAYS_MS[Math.min(retryAttemptRef.current, RETRY_DELAYS_MS.length - 1)];
|
|
retryAttemptRef.current += 1;
|
|
setLoadState("waiting");
|
|
clearRetryTimer();
|
|
retryTimerRef.current = setTimeout(() => {
|
|
void loadTree();
|
|
}, delay);
|
|
} else {
|
|
setLoadState("manual");
|
|
}
|
|
} finally {
|
|
setIsLoadingTree(false);
|
|
loadInFlightRef.current = false;
|
|
}
|
|
},
|
|
[
|
|
clearRetryTimer,
|
|
retryAttemptRef,
|
|
retryTimerRef,
|
|
setIsLoadingTree,
|
|
setLoadError,
|
|
setLoadState,
|
|
setTree,
|
|
],
|
|
);
|
|
return loadTree;
|
|
}
|
|
|
|
function useFileChangeSubscription({
|
|
sessionIdRef,
|
|
expandedPathsRef,
|
|
setTree,
|
|
setLoadState,
|
|
}: {
|
|
sessionIdRef: React.MutableRefObject<string>;
|
|
expandedPathsRef: React.MutableRefObject<ReadonlySet<string>>;
|
|
setTree: React.Dispatch<React.SetStateAction<FileTreeNode | null>>;
|
|
setLoadState: React.Dispatch<React.SetStateAction<LoadState>>;
|
|
}) {
|
|
// expandedPaths is read via ref so that toggling a directory (which mints a
|
|
// new Set) does not tear down and re-attach this WS listener — any event
|
|
// arriving during the swap would otherwise be silently dropped.
|
|
useEffect(() => {
|
|
const client = getWebSocketClient();
|
|
if (!client) return;
|
|
return client.on("session.workspace.file.changes", (msg) => {
|
|
const changes = msg.payload?.changes;
|
|
if (!changes || changes.length === 0) return;
|
|
applyFileChanges({
|
|
client,
|
|
sessionId: sessionIdRef.current,
|
|
expandedPaths: expandedPathsRef.current,
|
|
changes,
|
|
setTree,
|
|
setLoadState,
|
|
});
|
|
});
|
|
}, [sessionIdRef, expandedPathsRef, setTree, setLoadState]);
|
|
}
|
|
|
|
/**
|
|
* Hook for tree loading with retry logic, file-change subscription, and expanded state.
|
|
* @param sessionId - Session ID for API calls (agentctl routing).
|
|
* @param resetKey - Optional stable key (e.g. environmentId) that controls when the tree
|
|
* does a full reset. When sessions share the same environment, this prevents the tree
|
|
* from flashing on tab switch.
|
|
*/
|
|
export function useFileBrowserTree(sessionId: string, resetKey?: string) {
|
|
const effectiveResetKey = resetKey ?? sessionId;
|
|
const [tree, setTree] = useState<FileTreeNode | null>(null);
|
|
// Expansion + flat visibleRows come from the shared useTree hook. tree is
|
|
// a single root node with children; useTree wants an array, so feed it the
|
|
// root's children (pre-sorted — see sortRootChildren).
|
|
const treeNodes = useMemo(() => sortRootChildren(tree), [tree]);
|
|
const treeApi = useTree<FileTreeNode>({
|
|
nodes: treeNodes,
|
|
getPath: FB_GET_PATH,
|
|
getChildren: FB_GET_CHILDREN,
|
|
isDir: FB_IS_DIR,
|
|
});
|
|
const expandedPaths = treeApi.expanded;
|
|
const setExpandedPaths = treeApi.setExpanded;
|
|
const visibleRows = treeApi.visibleRows;
|
|
// Stable ref over `expandedPaths` so the WS file-change subscription does
|
|
// not re-attach on every toggle (each toggle mints a new Set reference).
|
|
const expandedPathsRef = useRef<ReadonlySet<string>>(expandedPaths);
|
|
expandedPathsRef.current = expandedPaths;
|
|
const [isLoadingTree, setIsLoadingTree] = useState(true);
|
|
const [loadState, setLoadState] = useState<LoadState>("loading");
|
|
const [loadError, setLoadError] = useState<string | null>(null);
|
|
const hasInitializedExpandedRef = useRef<string | null>(null);
|
|
const retryAttemptRef = useRef(0);
|
|
const retryTimerRef = useRef<NodeJS.Timeout | null>(null);
|
|
const sessionIdRef = useRef(sessionId);
|
|
sessionIdRef.current = sessionId;
|
|
const agentctlStatus = useSessionAgentctl(sessionId);
|
|
const { visibleLoadingPaths, showLoading, hideLoading, isLoading } = useLoadingTimers();
|
|
|
|
const clearRetryTimer = useCallback(() => {
|
|
if (retryTimerRef.current) {
|
|
clearTimeout(retryTimerRef.current);
|
|
retryTimerRef.current = null;
|
|
}
|
|
}, []);
|
|
|
|
const loadTree = useTreeLoader({
|
|
sessionId,
|
|
clearRetryTimer,
|
|
retryAttemptRef,
|
|
retryTimerRef,
|
|
setTree,
|
|
setIsLoadingTree,
|
|
setLoadState,
|
|
setLoadError,
|
|
});
|
|
|
|
// Refs for effects below to read without adding deps — avoids infinite
|
|
// retry loops and wipe-on-reconnect when `isReady`/`loadState`/`tree` tick.
|
|
const agentctlIsReadyRef = useRef(agentctlStatus.isReady);
|
|
const loadStateRef = useRef(loadState);
|
|
const treeRef = useRef(tree);
|
|
agentctlIsReadyRef.current = agentctlStatus.isReady;
|
|
loadStateRef.current = loadState;
|
|
treeRef.current = tree;
|
|
useEffect(() => {
|
|
setTree(null);
|
|
setIsLoadingTree(true);
|
|
setLoadState(agentctlIsReadyRef.current ? "loading" : "waiting");
|
|
setLoadError(null);
|
|
retryAttemptRef.current = 0;
|
|
clearRetryTimer();
|
|
hasInitializedExpandedRef.current = null;
|
|
const savedPaths = getFilesPanelExpandedPaths(effectiveResetKey);
|
|
setExpandedPaths(savedPaths.length > 0 ? new Set(savedPaths) : new Set());
|
|
if (savedPaths.length > 0) hasInitializedExpandedRef.current = effectiveResetKey;
|
|
// Only load now if agentctl is ready; otherwise the sibling effect fires
|
|
// on the ready flip. Prevents burning the retry budget during slow prepare.
|
|
if (agentctlIsReadyRef.current) void loadTree({ resetRetry: true });
|
|
else setIsLoadingTree(false);
|
|
return () => {
|
|
clearRetryTimer();
|
|
};
|
|
}, [clearRetryTimer, loadTree, effectiveResetKey, sessionId, setExpandedPaths]);
|
|
|
|
// Fire the initial load on the waiting → ready transition. `loadState` is
|
|
// read via a ref so a failed load (which sets state back to "waiting") does
|
|
// not re-fire this effect and cancel the retry timer via `resetRetry: true`.
|
|
// "loaded" only counts when a tree is actually present — `applyFileChanges`
|
|
// can flip state to "loaded" while the tree is still null.
|
|
useEffect(() => {
|
|
if (!agentctlStatus.isReady) return;
|
|
if (loadStateRef.current === "loading") return;
|
|
if (loadStateRef.current === "loaded" && treeRef.current) return;
|
|
void loadTree({ resetRetry: true });
|
|
}, [agentctlStatus.isReady, loadTree, sessionId]);
|
|
|
|
useEffect(() => {
|
|
if (!tree || isLoadingTree || hasInitializedExpandedRef.current === effectiveResetKey) return;
|
|
hasInitializedExpandedRef.current = effectiveResetKey;
|
|
setExpandedPaths(new Set());
|
|
// eslint-disable-next-line react-hooks/exhaustive-deps
|
|
}, [tree?.children?.length, isLoadingTree, effectiveResetKey]);
|
|
|
|
useEffect(() => {
|
|
if (expandedPaths.size > 0 || hasInitializedExpandedRef.current === effectiveResetKey) {
|
|
setFilesPanelExpandedPaths(effectiveResetKey, Array.from(expandedPaths));
|
|
}
|
|
}, [expandedPaths, effectiveResetKey]);
|
|
|
|
useFileChangeSubscription({ sessionIdRef, expandedPathsRef, setTree, setLoadState });
|
|
|
|
const collapseAll = treeApi.collapseAll;
|
|
|
|
return {
|
|
tree,
|
|
setTree,
|
|
expandedPaths,
|
|
setExpandedPaths,
|
|
visibleRows,
|
|
visibleLoadingPaths,
|
|
isLoadingTree,
|
|
loadState,
|
|
loadError,
|
|
loadTree,
|
|
showLoading,
|
|
hideLoading,
|
|
isLoading,
|
|
collapseAll,
|
|
};
|
|
}
|
|
|
|
/** Hook for scroll position persistence in the file browser. */
|
|
export function useScrollPersistence(
|
|
sessionId: string,
|
|
isTreeLoaded: boolean,
|
|
scrollAreaRef: React.RefObject<HTMLDivElement | null>,
|
|
tree: FileTreeNode | null,
|
|
) {
|
|
const scrollSaveTimeoutRef = useRef<NodeJS.Timeout | null>(null);
|
|
const hasRestoredScrollRef = useRef<string | null>(null);
|
|
|
|
// Restore scroll position after tree loads
|
|
useEffect(() => {
|
|
if (!isTreeLoaded || hasRestoredScrollRef.current === sessionId) return;
|
|
const savedScroll = getFilesPanelScrollPosition(sessionId);
|
|
if (savedScroll > 0 && scrollAreaRef.current) {
|
|
const viewport = scrollAreaRef.current.querySelector("[data-radix-scroll-area-viewport]");
|
|
if (viewport) {
|
|
viewport.scrollTop = savedScroll;
|
|
hasRestoredScrollRef.current = sessionId;
|
|
}
|
|
} else {
|
|
hasRestoredScrollRef.current = sessionId;
|
|
}
|
|
}, [isTreeLoaded, sessionId, scrollAreaRef]);
|
|
|
|
// Attach scroll listener to ScrollArea viewport
|
|
useEffect(() => {
|
|
const el = scrollAreaRef.current;
|
|
if (!el) return;
|
|
const viewport = el.querySelector("[data-radix-scroll-area-viewport]");
|
|
if (!viewport) return;
|
|
const onScroll = (event: Event) => {
|
|
const target = event.target as HTMLElement;
|
|
if (scrollSaveTimeoutRef.current) clearTimeout(scrollSaveTimeoutRef.current);
|
|
scrollSaveTimeoutRef.current = setTimeout(() => {
|
|
setFilesPanelScrollPosition(sessionId, target.scrollTop);
|
|
}, 150);
|
|
};
|
|
viewport.addEventListener("scroll", onScroll);
|
|
return () => {
|
|
viewport.removeEventListener("scroll", onScroll);
|
|
if (scrollSaveTimeoutRef.current) clearTimeout(scrollSaveTimeoutRef.current);
|
|
};
|
|
}, [sessionId, tree, scrollAreaRef]);
|
|
}
|
|
|
|
/** Fetch children for a folder node if not already loaded. */
|
|
export async function loadNodeChildren(
|
|
node: FileTreeNode,
|
|
sessionId: string,
|
|
treeState: ReturnType<typeof useFileBrowserTree>,
|
|
) {
|
|
if (node.children && node.children.length > 0) return;
|
|
// Dedupe in-flight fetches so rapid double-clicks don't issue two WS round-trips.
|
|
if (treeState.isLoading(node.path)) return;
|
|
treeState.showLoading(node.path);
|
|
try {
|
|
const client = getWebSocketClient();
|
|
if (!client) return;
|
|
const response = await requestFileTree(client, sessionId, node.path, 1);
|
|
const updateNode = (n: FileTreeNode): FileTreeNode => {
|
|
if (n.path === node.path) return { ...n, children: response.root.children };
|
|
return n.children ? { ...n, children: n.children.map(updateNode) } : n;
|
|
};
|
|
if (treeState.tree) treeState.setTree(updateNode(treeState.tree));
|
|
} catch (error) {
|
|
console.error("Failed to load children:", error);
|
|
} finally {
|
|
treeState.hideLoading(node.path);
|
|
}
|
|
}
|
|
|
|
export type ToggleFolderExpandDeps = {
|
|
node: FileTreeNode;
|
|
sessionId: string;
|
|
treeState: ReturnType<typeof useFileBrowserTree>;
|
|
setActiveFolderPath: (path: string) => void;
|
|
loadChildren?: typeof loadNodeChildren;
|
|
};
|
|
|
|
// Flip expanded synchronously *then* fetch children so the chevron rotates on the first click.
|
|
export async function toggleFolderExpand({
|
|
node,
|
|
sessionId,
|
|
treeState,
|
|
setActiveFolderPath,
|
|
loadChildren = loadNodeChildren,
|
|
}: ToggleFolderExpandDeps): Promise<void> {
|
|
if (!node.is_dir) return;
|
|
setActiveFolderPath(node.path);
|
|
const wasExpanded = treeState.expandedPaths.has(node.path);
|
|
treeState.setExpandedPaths((prev) => {
|
|
const next = new Set(prev);
|
|
if (next.has(node.path)) next.delete(node.path);
|
|
else next.add(node.path);
|
|
return next;
|
|
});
|
|
if (!wasExpanded) {
|
|
await loadChildren(node, sessionId, treeState);
|
|
}
|
|
}
|
|
|
|
/** Fetch and open a file by path. */
|
|
export async function fetchAndOpenFile(
|
|
sessionId: string,
|
|
path: string,
|
|
onOpenFile: (file: OpenFileTab) => void,
|
|
toast: ReturnType<typeof useToast>["toast"],
|
|
) {
|
|
try {
|
|
const client = getWebSocketClient();
|
|
if (!client) return;
|
|
const response: FileContentResponse = await requestFileContent(client, sessionId, path);
|
|
const { calculateHash } = await import("@/lib/utils/file-diff");
|
|
const hash = await calculateHash(response.content);
|
|
const name = path.split("/").pop() || path;
|
|
onOpenFile({
|
|
path,
|
|
name,
|
|
content: response.content,
|
|
originalContent: response.content,
|
|
originalHash: hash,
|
|
isDirty: false,
|
|
isBinary: response.is_binary,
|
|
});
|
|
} catch (error) {
|
|
const reason = error instanceof Error ? error.message : "Unknown error";
|
|
toast({ title: "Failed to open file", description: reason, variant: "error" });
|
|
}
|
|
}
|