fix discrepancy view

backend/app/api/system.py

@@ -5,7 +5,7 @@ from datetime import datetime, timezone
 from typing import Any, Dict, List, Optional
 
 import pathspec
-from fastapi import APIRouter, BackgroundTasks, Depends, HTTPException
+from fastapi import APIRouter, BackgroundTasks, Depends, HTTPException, Query
 from fastapi.responses import FileResponse, StreamingResponse
 from loguru import logger
 from pydantic import BaseModel, ConfigDict
@@ -1405,10 +1405,16 @@ def delete_file_record(file_id: int, db_session: Session = Depends(get_db)):
 
 
 @router.get("/discrepancies/tree", response_model=List[TreeNodeSchema])
-def get_discrepancies_tree(db_session: Session = Depends(get_db)):
+def get_discrepancies_tree(
+    path: Optional[str] = Query(
+        default="ROOT", description="Root path to get tree for"
+    ),
+    db_session: Session = Depends(get_db),
+):
     """Returns tree of directories that contain discrepancy files, grouped by source root."""
-    from app.api.inventory import TreeNodeSchema, get_source_roots
+    from app.api.inventory import get_source_roots
 
+    # Get source roots
     roots = get_source_roots(db_session)
 
     # Query all discrepancy files
@@ -1433,8 +1439,8 @@ def get_discrepancies_tree(db_session: Session = Depends(get_db)):
         )
         if directory not in dir_nodes:
             dir_nodes[directory] = TreeNodeSchema(
-                name=directory.split("/")[-1] or directory,
-                path=directory,
+                name=directory.split("/")[-1] or directory or "ROOT",
+                path=directory or "ROOT",
                 has_children=True,
                 children=[],
             )
@@ -1446,8 +1452,12 @@ def get_discrepancies_tree(db_session: Session = Depends(get_db)):
             )
         )
 
-    # Build top-level nodes from source roots
+    # If path is "ROOT", return top-level nodes grouped by source root
+    if path == "ROOT":
         result = []
+        seen = set()
+
+        # First add source roots that have discrepancies
         for root in roots:
             root_dirs = [d for d in dir_nodes.keys() if d.startswith(root) or d == root]
             if root_dirs:
@@ -1457,16 +1467,66 @@ def get_discrepancies_tree(db_session: Session = Depends(get_db)):
                         name=root, path=root, has_children=True, children=children
                     )
                 )
+                seen.update(root_dirs)
+
+        # Add directories that don't match any source root as themselves
+        for d in sorted(dir_nodes.keys()):
+            if d not in seen:
+                result.append(dir_nodes[d])
+
+        return result
+
+    # Return immediate children of the given path
+    result = []
+    for dir_path, node in sorted(dir_nodes.items()):
+        if dir_path == path:
+            return node.children
+        elif dir_path.startswith(path + "/"):
+            rel_path = dir_path[len(path) :].strip("/")
+            if "/" not in rel_path:
+                result.append(node)
+
+    return result
+
+    # Return immediate children of the given path
+    result = []
+    for dir_path, node in sorted(dir_nodes.items()):
+        if dir_path == path:
+            # This is the exact node - return its children
+            return node.children
+        elif dir_path.startswith(path + "/"):
+            # This is a subdirectory - check if it's an immediate child
+            rel_path = dir_path[len(path) :].strip("/")
+            if "/" not in rel_path:
+                # Immediate child
+                result.append(node)
+
+    return result
+
+    # Return immediate children of the given path
+    # Path could be a directory like "/data" - return its children
+    result = []
+    for dir_path, node in sorted(dir_nodes.items()):
+        if dir_path == path:
+            # This is the exact node - return its children
+            return node.children
+        elif dir_path.startswith(path + "/"):
+            # This is a subdirectory - check if it's an immediate child
+            rel_path = dir_path[len(path) :].strip("/")
+            if "/" not in rel_path:
+                # Immediate child
+                result.append(node)
 
     return result
 
 
-@router.get("/discrepancies/browse", response_model=List[DiscrepancySchema])
+@router.get("/discrepancies/browse", response_model=dict)
 def browse_discrepancies(
-    path: Optional[str] = None, db_session: Session = Depends(get_db)
+    path: Optional[str] = Query(default="ROOT", description="Directory path to browse"),
+    db_session: Session = Depends(get_db),
 ):
-    """Returns discrepancy files under a given directory path."""
-    # Reuse the query logic from list_discrepancies
+    """Returns discrepancy files and directories under a given directory path."""
+    # Query all discrepancy files
     deleted_records = db_session.query(models.FilesystemState).filter(
         models.FilesystemState.is_deleted.is_(True),
         models.FilesystemState.is_ignored.is_(False),
@@ -1497,25 +1557,69 @@ def browse_discrepancies(
         )
         ids_with_valid_versions = {row[0] for row in valid_version_rows}
 
-    # Filter by path prefix if specified
+    # Build a dict of all file paths
+    all_paths = {r.file_path: r for r in all_records}
+
+    # Find immediate children under the given path
     results = []
-    seen_ids = set()
-    for record in all_records:
-        if record.id in seen_ids:
-            continue
-        seen_ids.add(record.id)
+    seen_paths = set()
 
-        # Filter by path prefix
-        if (
-            path
-            and not record.file_path.startswith(path + "/")
-            and record.file_path != path
-        ):
+    for file_path, record in all_paths.items():
+        if path == "ROOT":
+            # For ROOT, show top-level directories/files
+            if "/" in file_path:
+                # It's in a subdirectory - get top-level dir
+                parts = file_path.strip("/").split("/")
+                top_dir = parts[0]
+                child_path = "/" + top_dir
+                child_name = top_dir
+            else:
+                # File at root
+                child_path = file_path
+                child_name = file_path
+        else:
+            # Check if this file is under the requested path
+            if file_path != path and not file_path.startswith(path + "/"):
                 continue
 
+            # Get immediate child relative to path
+            rel_path = file_path[len(path) :].strip("/")
+            if "/" in rel_path:
+                # It's a subdirectory - get immediate child
+                child_name = rel_path.split("/")[0]
+                child_path = (
+                    path + "/" + child_name if path != "/" else "/" + child_name
+                )
+            else:
+                # It's a file
+                child_path = file_path
+                child_name = rel_path
+
+        # Skip duplicates
+        if child_path in seen_paths:
+            continue
+        seen_paths.add(child_path)
+
+        # Check if it's a directory or file
+        is_dir = any(
+            p != child_path and p.startswith(child_path + "/") for p in all_paths
+        )
+
+        if is_dir:
+            # Count discrepancy files in this directory
+            file_count = sum(1 for p in all_paths if p.startswith(child_path + "/"))
+            results.append(
+                {
+                    "name": child_name,
+                    "path": child_path,
+                    "type": "directory",
+                    "has_children": file_count > 0,
+                    "discrepancy_count": file_count,
+                }
+            )
+        else:
+            # It's a file
             has_valid_versions = record.id in ids_with_valid_versions
-
-        if record.is_deleted or not os.path.exists(record.file_path):
             results.append(
                 DiscrepancySchema(
                     id=record.id,
@@ -1529,4 +1633,4 @@ def browse_discrepancies(
                 )
             )
 
-    return results
+    return {"files": results}

backend/app/services/scanner.py

@@ -753,7 +753,7 @@ class ScannerService:
                 FETCH_LIMIT = HASH_BATCH_SIZE * 4
 
                 while self.is_hashing:
-                    # Find unindexed work (exclude deleted files)
+                    # Find unindexed work (exclude deleted files - they cannot be hashed)
                     hashing_targets = (
                         db_session.query(models.FilesystemState)
                         .filter(

backend/tests/test_service_scanner.py

@@ -165,18 +165,9 @@ def test_run_hashing_mocked(db_session, mocker):
     db_session.add(f)
     db_session.commit()
 
-    # Mock compute_sha256 to return a fixed hash
-    mocker.patch.object(ScannerService, "compute_sha256", return_value="mocked_hash")
-
     # run_hashing runs in a loop until work is done.
     # Since we aren't in 'is_running' state, it should process the 1 file and stop.
-    try:
     scanner.run_hashing()
-    except Exception as e:
-        print(f"DEBUG: run_hashing raised exception: {e}")
-        import traceback
-
-        traceback.print_exc()
 
     db_session.refresh(f)
     assert f.sha256_hash == "mocked_hash"

frontend/src/lib/components/file-browser/FileBrowser.svelte

@@ -10,13 +10,14 @@
                 CheckSquare,
                 Square
         } from "lucide-svelte";
+        import { onMount } from 'svelte';
         import { Button } from "$lib/components/ui/button";
         import { Checkbox } from "$lib/components/ui/checkbox";
         import { ScrollArea } from "$lib/components/ui/scroll-area";
         import { Input } from "$lib/components/ui/input";
         import FileBrowserTreeItem from "./FileBrowserTreeItem.svelte";
         import FileBrowserRowItem from "./FileBrowserRowItem.svelte";
-        import type { FileItem, Breadcrumb } from "$lib/types";
+        import type { FileItem, TreeNode, Breadcrumb } from "$lib/types";
         import { cn } from "$lib/utils";
         import {
             getSystemTreeSystemTreeGet,
@@ -185,14 +186,35 @@
                 hasChildren: true
         });
 
-        const discrepancyRoot = $derived({
+        const discrepancyRoot = $state<TreeNode>({
                 name: "Discrepancies",
                 path: "ROOT",
-                expanded: true,
+                expanded: false,
                 children: [],
                 hasChildren: true
         });
 
+        // Load discrepancy tree on mount
+        onMount(async () => {
+                if (mode === "discrepancies") {
+                        try {
+                                const response = await getDiscrepanciesTreeGet({ query: { path: "ROOT" } });
+                                if (response.data && Array.isArray(response.data)) {
+                                        discrepancyRoot.children = response.data.map((d: any) => ({
+                                                name: d.name,
+                                                path: d.path,
+                                                children: d.children || [],
+                                                expanded: false,
+                                                hasChildren: d.has_children
+                                        }));
+                                        discrepancyRoot.expanded = true;
+                                }
+                        } catch (error) {
+                                console.error("Failed to load discrepancy tree:", error);
+                        }
+                }
+        });
+
         const activeRoot = $derived(
                 mode === "host" ? sourceDataRoot :
                 mode === "index" ? virtualIndexRoot :

frontend/src/routes/discrepancies/+page.svelte

@@ -35,8 +35,33 @@
             const response = await listDiscrepanciesSystemDiscrepanciesGet();
             if (response.data) {
                 discrepancies = response.data;
-                // Convert discrepancies to FileItem format for FileBrowser
-                files = response.data.map((d: DiscrepancySchema) => ({
+            }
+            await loadFiles(currentPath);
+        } catch (error) {
+            console.error("Failed to load discrepancies:", error);
+            toast.error("Failed to load discrepancies");
+        } finally {
+            loading = false;
+        }
+    }
+
+    async function loadFiles(path: string) {
+        try {
+            const response = await browseDiscrepanciesGet({ query: { path } });
+            if (response.data?.files) {
+                files = response.data.files.map((d: any) => {
+                    // Check if it's a directory (has "type" property) or a file (has "id")
+                    if (d.type === 'directory') {
+                        // It's a directory
+                        return {
+                            name: d.name || d.path.split('/').pop() || d.path,
+                            path: d.path,
+                            type: 'directory',
+                            discrepancy_count: d.discrepancy_count || 0
+                        };
+                    } else {
+                        // It's a file (DiscrepancySchema)
+                        return {
                             name: d.path.split('/').pop() || d.path,
                             path: d.path,
                             type: 'file',
@@ -46,13 +71,12 @@
                             is_deleted: d.is_deleted,
                             has_versions: d.has_versions,
                             ignored: false
-                }));
+                        };
+                    }
+                });
             }
         } catch (error) {
-            console.error("Failed to load discrepancies:", error);
-            toast.error("Failed to load discrepancies");
-        } finally {
-            loading = false;
+            console.error("Failed to browse discrepancies:", error);
         }
     }
 
@@ -82,13 +106,22 @@
         }
     }
 
-    async function navigateTo(path: string) {
+    function navigateTo(path: string) {
         currentPath = path;
+        loadFiles(path);
     }
 
     const missingItems = $derived(discrepancies.filter(d => d.is_deleted));
     const pendingItems = $derived(discrepancies.filter(d => !d.is_deleted));
 
+    // Statistics
+    const missingWithNoBackup = $derived(
+        discrepancies.filter(d => d.is_deleted && !d.has_versions).length
+    );
+    const missingWithBackup = $derived(
+        discrepancies.filter(d => d.is_deleted && d.has_versions).length
+    );
+
     onMount(loadDiscrepancies);
 </script>
 
@@ -96,7 +129,7 @@
     <title>Discrepancies - TapeHoard</title>
 </svelte:head>
 
-<div class="flex flex-col gap-6 flex-1 min-h-0 animate-in fade-in duration-700">
+<div class="flex flex-col gap-6 h-full animate-in fade-in duration-700">
     <PageHeader
         title="Discrepancies"
         description="Files missing from disk or confirmed deleted"
@@ -130,12 +163,22 @@
     {:else}
         <!-- Summary Statistics -->
         <div class="grid grid-cols-2 gap-3 shrink-0">
-            <StatCard label="Missing from disk" value={missingItems.length} subLabel="Files the scanner did not find" variant="error" />
-            <StatCard label="Pending confirmation" value={pendingItems.length} subLabel="Tracked files not yet confirmed" variant="warning" />
+            <StatCard
+                label="Missing with no backup"
+                value={missingWithNoBackup}
+                subLabel="Files missing from disk with no copies on archive media"
+                variant="error"
+            />
+            <StatCard
+                label="Missing with backup"
+                value={missingWithBackup}
+                subLabel="Files missing from disk but have copies on archive media"
+                variant="warning"
+            />
         </div>
 
         <!-- FileBrowser Component in discrepancies mode -->
-        <div class="flex-1 min-h-0 overflow-hidden">
+        <div class="flex-1 min-h-[600px] bg-bg-secondary border border-border-color shadow-2xl rounded-lg flex flex-col relative overflow-hidden">
             <FileBrowser
                 bind:currentPath={currentPath}
                 files={files}