@larkup/workspace-app 0.2.1 → 0.2.2

package/dist-types/components/data/corpus-panel.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"corpus-panel.d.ts","sourceRoot":"","sources":["../../../src/components/data/corpus-panel.tsx"],"names":[],"mappings":"AAqCA,OAAO,KAAK,EAAkB,cAAc,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AA+BpF,OAAO,EAGL,KAAK,iBAAiB,EACvB,MAAM,qCAAqC,CAAC;AAU7C,wBAAgB,WAAW,CAAC,EAC1B,SAAS,EACT,MAAW,EACX,SAAS,EACT,UAAU,EACV,uBAAuB,EACvB,cAAc,EACd,SAAS,GACV,EAAE;IACD,SAAS,EAAE,cAAc,EAAE,CAAC;IAC5B,MAAM,CAAC,EAAE,SAAS,EAAE,CAAC;IACrB,SAAS,EAAE,MAAM,IAAI,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IACzC,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,uBAAuB,EAAE,iBAAiB,CAAC;IAC3C,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,SAAS,CAAC,EAAE,OAAO,CAAC;CACrB,+BAwsBA"}
+{"version":3,"file":"corpus-panel.d.ts","sourceRoot":"","sources":["../../../src/components/data/corpus-panel.tsx"],"names":[],"mappings":"AAqCA,OAAO,KAAK,EAAkB,cAAc,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AA+BpF,OAAO,EAGL,KAAK,iBAAiB,EACvB,MAAM,qCAAqC,CAAC;AAU7C,wBAAgB,WAAW,CAAC,EAC1B,SAAS,EACT,MAAW,EACX,SAAS,EACT,UAAU,EACV,uBAAuB,EACvB,cAAc,EACd,SAAS,GACV,EAAE;IACD,SAAS,EAAE,cAAc,EAAE,CAAC;IAC5B,MAAM,CAAC,EAAE,SAAS,EAAE,CAAC;IACrB,SAAS,EAAE,MAAM,IAAI,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IACzC,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,uBAAuB,EAAE,iBAAiB,CAAC;IAC3C,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,SAAS,CAAC,EAAE,OAAO,CAAC;CACrB,+BAquBA"}

package/dist-types/components/data/upload-panel.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"upload-panel.d.ts","sourceRoot":"","sources":["../../../src/components/data/upload-panel.tsx"],"names":[],"mappings":"AA8BA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AA2E/D,wBAAgB,WAAW,CAAC,EAC1B,OAAO,EACP,cAAc,EACd,OAAO,GACR,EAAE;IACD,OAAO,EAAE,MAAM,IAAI,CAAC;IACpB,cAAc,CAAC,EAAE,CAAC,MAAM,EAAE,iBAAiB,GAAG,IAAI,KAAK,IAAI,CAAC;IAC5D,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB,+BA+qCA"}
+{"version":3,"file":"upload-panel.d.ts","sourceRoot":"","sources":["../../../src/components/data/upload-panel.tsx"],"names":[],"mappings":"AA8BA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAiF/D,wBAAgB,WAAW,CAAC,EAC1B,OAAO,EACP,cAAc,EACd,OAAO,GACR,EAAE;IACD,OAAO,EAAE,MAAM,IAAI,CAAC;IACpB,cAAc,CAAC,EAAE,CAAC,MAAM,EAAE,iBAAiB,GAAG,IAAI,KAAK,IAAI,CAAC;IAC5D,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB,+BAuuCA"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@larkup/workspace-app",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "type": "module",
   "description": "Canonical Larkup chat, data and settings pages with host adapters",
   "files": [

package/src/components/data/corpus-panel.tsx

@@ -102,6 +102,8 @@ export function CorpusPanel({
     { type: 'single'; doc: SourceDocument } | { type: 'all' } | { type: 'selected' } | null
   >(null);
   const [selectedIds, setSelectedIds] = useState<Set<string>>(new Set());
+  const [deletedIds, setDeletedIds] = useState<Set<string>>(new Set());
+  const [clearingAll, setClearingAll] = useState(false);
   const [movingGroupId, setMovingGroupId] = useState<string | null>(null);
   const [inspectorDoc, setInspectorDoc] = useState<SourceDocument | null>(null);
 
@@ -143,9 +145,18 @@ export function CorpusPanel({
         others.push(doc);
       }
     }
-    return [...Array.from(grouped.values()), ...others].sort(
-      (a, b) => new Date(b.createdAt).getTime() - new Date(a.createdAt).getTime(),
-    );
+    return [...Array.from(grouped.values()), ...others]
+      .filter((document) => !deletedIds.has(document.id))
+      .sort((a, b) => new Date(b.createdAt).getTime() - new Date(a.createdAt).getTime());
+  }, [deletedIds, documents]);
+
+  useEffect(() => {
+    setDeletedIds((current) => {
+      const documentIds = new Set(documents.map((document) => document.id));
+      const pending = new Set(Array.from(current).filter((id) => documentIds.has(id)));
+      return pending.size === current.size ? current : pending;
+    });
+    if (documents.length === 0) setClearingAll(false);
   }, [documents]);
 
   const filteredDocuments = groupedDocuments.filter(
@@ -182,6 +193,8 @@ export function CorpusPanel({
   }
 
   async function del(id: string, doc?: SourceDocument) {
+    setDeleteTask(null);
+    setDeletedIds((current) => new Set(current).add(id));
     try {
       if (doc?.metadata?.isGroup && doc?.metadata?.mediaAssetId) {
         await deleteRequest(workspaceUrl(`/api/media?id=${doc.metadata.mediaAssetId}`));
@@ -196,12 +209,20 @@ export function CorpusPanel({
       await onChanged();
       toast.success('Document deleted');
     } catch (error) {
+      setDeletedIds((current) => {
+        const next = new Set(current);
+        next.delete(id);
+        return next;
+      });
       toast.error(formatErrorMessage(error) || 'Could not delete this document.');
     }
   }
   async function delSelected() {
     if (selectedIds.size === 0) return;
     const toDelete = groupedDocuments.filter((d) => selectedIds.has(d.id));
+    const deletionIds = new Set(toDelete.map((document) => document.id));
+    setDeleteTask(null);
+    setDeletedIds((current) => new Set([...current, ...deletionIds]));
     try {
       for (const doc of toDelete) {
         if (doc.metadata?.isGroup && doc.metadata?.mediaAssetId) {
@@ -214,16 +235,24 @@ export function CorpusPanel({
       await onChanged();
       toast.success(`Deleted ${toDelete.length} item(s)`);
     } catch (error) {
+      setDeletedIds((current) => {
+        const next = new Set(current);
+        for (const id of deletionIds) next.delete(id);
+        return next;
+      });
       toast.error(formatErrorMessage(error) || 'Could not delete the selected documents.');
     }
   }
   async function clearAll() {
+    setDeleteTask(null);
+    setClearingAll(true);
     try {
       await deleteRequest(workspaceUrl('/api/documents'));
       setSelectedIds(new Set());
       await onChanged();
       toast.success('Corpus and vector index cleared');
     } catch (error) {
+      setClearingAll(false);
       toast.error(formatErrorMessage(error) || 'Could not clear the corpus.');
     }
   }
@@ -299,7 +328,7 @@ export function CorpusPanel({
     return <CorpusSkeleton />;
   }
 
-  if (documents.length === 0) {
+  if (clearingAll || groupedDocuments.length === 0) {
     return (
       <div className="flex flex-col  items-center justify-center rounded-lg   px-6 py-14 text-center">
         <div className="size-10 p-1.5 bg-white/60 border border-border  rounded-md flex items-center justify-center">
@@ -1144,7 +1173,11 @@ function DocumentDialog({
 
 function CorpusSkeleton() {
   return (
-    <div className="space-y-4 animate-in fade-in duration-150">
+    <div
+      role="status"
+      aria-label="Loading corpus"
+      className="space-y-4 animate-in fade-in duration-150"
+    >
       {/* Filter bar skeleton */}
       <div className="flex flex-col gap-3 sm:flex-row sm:items-center justify-between pb-3">
         <div className="flex flex-wrap items-center gap-2">

package/src/components/data/upload-panel.tsx

@@ -83,6 +83,12 @@ interface ImageIndexingCapability {
   message?: string;
 }
 
+interface PreparingFile {
+  id: string;
+  name: string;
+  size: number;
+}
+
 const stagedFilesByScope = new Map<string, StagedFile[]>();
 const UPLOAD_QUEUE_KEY = 'larkup.upload-queue.v1';
 
@@ -179,6 +185,7 @@ export function UploadPanel({
   const [remoteUrl, setRemoteUrl] = useState('');
   const [loadingRemoteFile, setLoadingRemoteFile] = useState(false);
   const [saving, setSaving] = useState(false);
+  const [preparingFiles, setPreparingFiles] = useState<PreparingFile[]>([]);
   const [progress, setProgress] = useState<{
     message?: string;
     current: number;
@@ -196,7 +203,17 @@ export function UploadPanel({
     const joinedActiveSave = savingRef.current;
     const next: StagedFile[] = [];
     const stage = (file: StagedFile) => next.push({ ...file, sourceUrl });
-    for (const file of Array.from(files)) {
+    const selectedFiles = Array.from(files).map((file) => ({
+      file,
+      pending: {
+        id: Math.random().toString(36).slice(2),
+        name: file.name,
+        size: file.size,
+      },
+    }));
+    setPreparingFiles((current) => [...current, ...selectedFiles.map(({ pending }) => pending)]);
+
+    for (const { file, pending } of selectedFiles) {
       const ext = file.name.split('.').pop()?.toLowerCase();
 
       // Prevent dragging media into the documents tab
@@ -205,10 +222,11 @@ export function UploadPanel({
         toast.error(
           `Unsupported file type: ${file.name}. Please upload media files in the Media tab.`,
         );
+        setPreparingFiles((current) => current.filter((item) => item.id !== pending.id));
         continue;
       }
 
-      const id = Math.random().toString(36).slice(2);
+      const id = pending.id;
       try {
         if (ext === 'csv') {
           const content = await file.text();
@@ -320,12 +338,18 @@ export function UploadPanel({
           });
 
           const data = await res.json().catch(() => ({}));
-          if (!res.ok) {
+          const legacyImageOnlyPdf =
+            ext === 'pdf' &&
+            res.status === 422 &&
+            /(?:no readable text|does not contain selectable text)/i.test(String(data.error ?? ''));
+          if (!res.ok && !legacyImageOnlyPdf) {
             throw new Error(data.error || `Could not extract text from ${file.name}`);
           }
 
           const { text } = data;
-          const hasReadableText = data.hasReadableText ?? Boolean(text?.trim());
+          const hasReadableText = legacyImageOnlyPdf
+            ? false
+            : (data.hasReadableText ?? Boolean(text?.trim()));
           const imageOnlyPdf = ext === 'pdf' && !hasReadableText;
           if (!hasReadableText && !imageOnlyPdf) {
             throw new Error(
@@ -363,6 +387,8 @@ export function UploadPanel({
         }
       } catch (err) {
         toast.error(`Could not read ${file.name}. ${formatErrorMessage(err)}`);
+      } finally {
+        setPreparingFiles((current) => current.filter((item) => item.id !== pending.id));
       }
     }
     setStaged((prev) => [...prev, ...next]);
@@ -687,7 +713,7 @@ export function UploadPanel({
   }
 
   useEffect(() => {
-    const actionBusy = saving || loadingRemoteFile;
+    const actionBusy = saving || loadingRemoteFile || preparingFiles.length > 0;
     onActionChange?.({
       label: 'Add file',
       onClick: isRemoteEntry ? () => void loadRemoteFile() : () => void ingest(),
@@ -695,7 +721,16 @@ export function UploadPanel({
       loading: actionBusy,
     });
     return () => onActionChange?.(null);
-  }, [entryTab, staged, saving, remoteUrl, loadingRemoteFile, groupId, onActionChange]);
+  }, [
+    entryTab,
+    staged,
+    saving,
+    remoteUrl,
+    loadingRemoteFile,
+    preparingFiles.length,
+    groupId,
+    onActionChange,
+  ]);
 
   function updateEditingFile(patch: Partial<StagedFile>) {
     setStaged((prev) => prev.map((f) => (f.id === editingFileId ? { ...f, ...patch } : f)));
@@ -775,6 +810,30 @@ export function UploadPanel({
         }}
       />
 
+      {preparingFiles.length > 0 && (
+        <div
+          role="status"
+          aria-live="polite"
+          className="space-y-2 rounded-md border border-border bg-card px-3 py-3"
+        >
+          <div className="flex items-center gap-2 text-sm font-medium">
+            <Loader2 className="size-4 animate-spin text-primary" />
+            Preparing {preparingFiles.length} file{preparingFiles.length === 1 ? '' : 's'}…
+          </div>
+          <ul className="space-y-1">
+            {preparingFiles.map((file) => (
+              <li
+                key={file.id}
+                className="flex items-center justify-between gap-3 text-xs text-muted-foreground"
+              >
+                <span className="truncate font-mono">{file.name}</span>
+                <span className="shrink-0 tabular-nums">{(file.size / 1024).toFixed(1)} KB</span>
+              </li>
+            ))}
+          </ul>
+        </div>
+      )}
+
       {staged.length > 0 && (
         <div className="space-y-2 pb-5">
           {staged.some((file) => file.name.toLowerCase().endsWith('.pdf')) &&
@@ -987,10 +1046,13 @@ export function UploadPanel({
         <Button
           onClick={isRemoteEntry ? () => void loadRemoteFile() : () => void ingest()}
           disabled={
-            loadingRemoteFile || saving || (isRemoteEntry ? !remoteUrl.trim() : staged.length === 0)
+            loadingRemoteFile ||
+            saving ||
+            preparingFiles.length > 0 ||
+            (isRemoteEntry ? !remoteUrl.trim() : staged.length === 0)
           }
         >
-          {loadingRemoteFile || saving ? (
+          {loadingRemoteFile || saving || preparingFiles.length > 0 ? (
             <Loader2 className="size-4 animate-spin" />
           ) : (
             <FileUp className="size-4" />