feat(v6): Add inline editing, file viewing, and deep Aadhaar/PAN extraction rules

Author: ptkvaibhav

src/organization/proposals.js

@@ -114,12 +114,29 @@ function proposeFileName(file) {
   const purpose = file.structure?.purpose ?? file.classification?.category ?? "other";
   let label = file.structure?.renameLabel ?? PURPOSE_LABELS[purpose] ?? PURPOSE_LABELS.other;
 
-  // If the renameLabel already contains a strong identifier (like Form_16_2024-25),
-  // we might not even need the hash, but let's keep it for safety to avoid collisions
-  const shortHash = String(file.sha256 ?? "unhashed").slice(0, 8);
   const extension = file.extension ?? path.extname(file.absolutePath);
+  const currentBaseName = path.basename(file.absolutePath, extension);
+  
+  // If the current name is just a huge string of numbers (like 400082092134) or a generic IMG_ tag,
+  // we should prepend the semantic label so the user knows what it is (e.g. Document_400082092134.pdf)
+  // But wait, the user said "suggest the name of the file instead of just document_original name. Example - Say an ID card document like Aadhar card is named as 40130202.pdf, it should reason that this is an Aadhar Card and based on that suggest that it should be named as Pratik Vaibhav_Aadhar Card.pdf".
+  // Because we do not run deep AI extracting on *every* file automatically (it takes 10s per file), 
+  // we will give it a better default name but encourage the "Ask AI" button.
+  // Actually, for Identity documents, we can extract the specific type if the rule matched!
+  
+  // If the name is already prefixed with the label, don't double prefix
+  if (currentBaseName.toLowerCase().startsWith(label.toLowerCase())) {
+     return `${currentBaseName}${extension}`;
+  }
+
+  // If the purpose is explicitly identified (like Identity, Finance, Resume), prefix it for clarity
+  if (purpose !== "other" && purpose !== "image" && purpose !== "document") {
+     return `${label}_${currentBaseName}${extension}`;
+  }
 
-  return `${label}_${shortHash}${extension}`;
+  // For generic images or documents, just use the original name, don't force a "Document_" prefix
+  // unless it's literally just a hash. But let's just default to the original name to avoid annoying users.
+  return `${currentBaseName}${extension}`;
 }
 
 function buildProposalId(action, file) {

src/organization/purpose-rules.js

@@ -128,6 +128,36 @@ export function inferPurposeDetails({ absolutePath = "", baseName = "", extensio
     };
   }
 
+  // Specific Deep Content Rule: Aadhaar Card
+  if (normalizedBaseName.includes("aadhaar") || /unique identification authority of india/i.test(extractedText) || (/government of india/i.test(extractedText) && /aadhaar/i.test(extractedText))) {
+     return {
+       purpose: "identity",
+       expectedFolders: ["Identity"],
+       matchedByRule: true,
+       renameLabel: "Aadhaar_Card"
+     };
+  }
+
+  // Specific Deep Content Rule: PAN Card
+  if ((normalizedBaseName.includes("pan") && !normalizedBaseName.includes("company")) || (/income tax department/i.test(extractedText) && /permanent account number/i.test(extractedText))) {
+     return {
+       purpose: "identity",
+       expectedFolders: ["Identity"],
+       matchedByRule: true,
+       renameLabel: "PAN_Card"
+     };
+  }
+
+  // Specific Deep Content Rule: Passport
+  if (normalizedBaseName.includes("passport") || (/republic of india/i.test(extractedText) && /passport/i.test(extractedText))) {
+     return {
+       purpose: "identity",
+       expectedFolders: ["Identity"],
+       matchedByRule: true,
+       renameLabel: "Passport"
+     };
+  }
+
   // If it's a known code extension, prefer the code category/purpose to avoid keyword misclassification
   if (CODE_EXTENSIONS.has(normalizedExtension)) {
     return {

src/server.js

@@ -134,6 +134,27 @@ export async function startServer({ port = 3030, dbPath = DEFAULT_DB_PATH } = {}
     }
   });
 
+  app.get("/api/file", async (req, res) => {
+    try {
+      const filePath = req.query.path;
+      if (!filePath) {
+        return res.status(400).send("Path is required");
+      }
+      
+      const fs = await import("node:fs/promises");
+      const absolutePath = path.resolve(filePath);
+      
+      try {
+        await fs.access(absolutePath);
+        res.sendFile(absolutePath);
+      } catch {
+        res.status(404).send("File not found");
+      }
+    } catch (error) {
+      res.status(500).send(error.message);
+    }
+  });
+
   function cleanJSON(str) {
     try {
       const match = str.match(/```(?:json)?\s*([\s\S]*?)\s*```/);

ui/dist/assets/index-B8Ts6tKM.js

@@ -5,8 +5,8 @@
     <link rel="icon" type="image/svg+xml" href="/favicon.svg" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>ui</title>
-    <script type="module" crossorigin src="/assets/index-LBQGqhzq.js"></script>
-    <link rel="stylesheet" crossorigin href="/assets/index-Oe-DmyI4.css">
+    <script type="module" crossorigin src="/assets/index-B8Ts6tKM.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-BMzdelOc.css">
   </head>
   <body>
     <div id="root"></div>

ui/dist/assets/index-BMzdelOc.css

@@ -1,6 +1,6 @@
 /* eslint-disable react-hooks/set-state-in-effect */
 import { useState, useEffect, useRef } from 'react';
-import { Layout, Copy, Wand2, CheckCircle, ArrowRight, RefreshCw, FolderSearch, Cloud, Sparkles, KeyRound, Trash2, Info, FileMinus, FilePlus, FolderOpen } from 'lucide-react';
+import { Layout, Copy, Wand2, CheckCircle, ArrowRight, RefreshCw, FolderSearch, Cloud, Sparkles, KeyRound, Trash2, Info, FileMinus, FilePlus, FolderOpen, Eye, Edit2, Check, X } from 'lucide-react';
 
 interface Stats {
   totalFiles: number;
@@ -38,6 +38,8 @@ function App() {
   const [items, setItems] = useState<ReviewItem[]>([]);
   const [applying, setApplying] = useState(false);
   const [proposalFilter, setProposalFilter] = useState<'all' | 'move' | 'rename'>('all');
+  const [editingItemId, setEditingItemId] = useState<string | null>(null);
+  const [editingName, setEditingName] = useState<string>("");
 
   const [aiExclusions, setAiExclusions] = useState<{ exclusions: string[], reasoning: string } | null>(null);
   const [aiReasoning, setAiReasoning] = useState<Record<string, string>>({});
@@ -185,6 +187,30 @@ function App() {
     fetchData();
   };
 
+  const handleSaveEdit = async (id: string) => {
+    if (!editingName.trim()) return;
+    
+    // Optimistically update the UI
+    setItems(prev => prev.map(i => {
+      if (i.id === id) {
+        return {
+          ...i,
+          proposedPath: i.proposedPath?.replace(/[^\\/]+$/, editingName),
+          evidence: { ...i.evidence, proposedName: editingName }
+        };
+      }
+      return i;
+    }));
+    
+    setEditingItemId(null);
+    setEditingName("");
+  };
+
+  const handleCancelEdit = () => {
+    setEditingItemId(null);
+    setEditingName("");
+  };
+
   const getAiReasoning = async (item: ReviewItem) => {
     if (aiReasoning[item.id]) return; // Use cache
 
@@ -467,6 +493,9 @@ function App() {
                       <div className="flex items-center gap-3">
                         <Copy className="w-4 h-4 text-slate-500" />
                         <span className="text-sm font-mono text-slate-400">SHA256: {item.evidence.sha256.slice(0, 16)}...</span>
+                        <a href={`/api/file?path=${encodeURIComponent(item.subjectPath)}`} target="_blank" rel="noreferrer" className="text-sky-400 hover:text-sky-300 transition-colors ml-2" title="View File">
+                          <Eye className="w-4 h-4" />
+                        </a>
                       </div>
                       <div className="flex gap-2">
                         <button 
@@ -558,10 +587,44 @@ function App() {
                           <div className="flex items-center gap-3 text-slate-500 line-through decoration-rose-500/50 overflow-hidden">
                             <div className="p-2 bg-slate-950 rounded border border-slate-800 shrink-0"><FileMinus className="w-4 h-4 text-rose-400"/></div>
                             <span className="text-xs font-mono truncate w-full" title={item.subjectPath}>{item.subjectPath}</span>
+                            <a href={`/api/file?path=${encodeURIComponent(item.subjectPath)}`} target="_blank" rel="noreferrer" className="text-sky-400 hover:text-sky-300 transition-colors ml-auto shrink-0" title="View File">
+                              <Eye className="w-4 h-4" />
+                            </a>
                           </div>
-                          <div className="flex items-center gap-3 bg-sky-500/10 p-3 rounded-xl border border-sky-500/20 overflow-hidden">
+                          
+                          <div className="flex items-center gap-3 bg-sky-500/10 p-3 rounded-xl border border-sky-500/20 overflow-hidden group">
                             <div className="p-2 bg-sky-500/20 rounded shrink-0"><FilePlus className="w-4 h-4 text-sky-400"/></div>
-                            <span className="text-sm font-bold text-sky-400 truncate w-full" title={item.proposedPath || item.evidence?.proposedName}>{item.proposedPath || item.evidence?.proposedName}</span>
+                            
+                            {editingItemId === item.id ? (
+                              <div className="flex items-center gap-2 w-full">
+                                <input 
+                                  type="text" 
+                                  value={editingName} 
+                                  onChange={e => setEditingName(e.target.value)}
+                                  className="flex-1 bg-slate-950 border border-sky-500 rounded px-2 py-1 text-sm font-bold text-sky-400 focus:outline-none"
+                                  autoFocus
+                                  onKeyDown={e => e.key === 'Enter' && handleSaveEdit(item.id)}
+                                />
+                                <button onClick={() => handleSaveEdit(item.id)} className="p-1 hover:bg-green-500/20 rounded text-green-400"><Check className="w-4 h-4"/></button>
+                                <button onClick={handleCancelEdit} className="p-1 hover:bg-rose-500/20 rounded text-rose-400"><X className="w-4 h-4"/></button>
+                              </div>
+                            ) : (
+                              <>
+                                <span className="text-sm font-bold text-sky-400 truncate w-full" title={item.proposedPath || item.evidence?.proposedName}>
+                                  {item.proposedPath || item.evidence?.proposedName}
+                                </span>
+                                <button 
+                                  onClick={() => {
+                                    setEditingItemId(item.id);
+                                    setEditingName(item.evidence?.proposedName || item.proposedPath?.split('\\').pop()?.split('/').pop() || "");
+                                  }} 
+                                  className="text-slate-500 hover:text-sky-400 opacity-0 group-hover:opacity-100 transition-all shrink-0 ml-auto"
+                                  title="Edit Name"
+                                >
+                                  <Edit2 className="w-4 h-4" />
+                                </button>
+                              </>
+                            )}
                           </div>
                         </div>