feat(directory-access): enhance directory handling with explicit write permission checks for Android

Author: cs-util

index.html

@@ -125,81 +125,60 @@ <h1>Android File System Demo</h1>
             }
         });
 
-        // 3. OPEN DIRECTORY (Read/Write)
+        // 3. OPEN DIRECTORY (Fixed for Android)
         document.getElementById('btnDir').addEventListener('click', async () => {
             try {
                 log("Requesting directory access...");
-                // 1. Get the handle (Android often defaults this to Read-Only!)
                 const dirHandle = await window.showDirectoryPicker({
                     mode: "readwrite"
                 });
 
                 log(`📂 Directory selected: ${dirHandle.name}`);
 
-                // --- THE CRITICAL FIX ---
-                // We must explicitly verify/request WRITE permission now.
+                // 1. Verify Directory Permission
                 const options = { mode: 'readwrite' };
-                // Check if we already have permission
                 if ((await dirHandle.queryPermission(options)) !== 'granted') {
-                    log("⚠️ Handle is Read-Only. Requesting Write access...");
-                    // This triggers the browser prompt: "Allow site to edit files?"
                     if ((await dirHandle.requestPermission(options)) !== 'granted') {
-                        throw new Error("Write permission was denied by the user.");
+                        throw new Error("Directory write permission denied.");
                     }
                 }
-                log("✅ Write permission confirmed!");
-                // -------------------------
 
                 // List contents
                 const entries = [];
-                log("--- Listing Files ---");
                 for await (const entry of dirHandle.values()) {
                     entries.push(entry);
-                    log(` - ${entry.kind}: ${entry.name}`);
                 }
-                if (entries.length === 0) log("  (empty folder)");
+                log(`Files found: ${entries.length}`);
 
-                // Write/append to session log
-                let existingLog = "";
-                let visitCount = 1;
+                // 2. Create/Get the File Handle
+                const fileName = 'session-log.txt';
+                let fileHandle;
+                
                 try {
-                    const existingHandle = await dirHandle.getFileHandle('session-log.txt', { create: false });
-                    const file = await existingHandle.getFile();
-                    existingLog = await file.text();
-                    // Count previous visits
-                    const matches = existingLog.match(/Visit #\d+/g);
-                    visitCount = matches ? matches.length + 1 : 1;
-                    log(`📖 Found existing log (${visitCount - 1} previous visits).`);
-                } catch {
-                    log("📝 Creating fresh log...");
+                    fileHandle = await dirHandle.getFileHandle(fileName, { create: true });
+                } catch (e) {
+                    throw new Error(`Could not create file handle: ${e.message}`);
                 }
 
-                const timestamp = new Date().toISOString();
-                const newEntry = `\n--- Visit #${visitCount} ---\nTimestamp: ${timestamp}\nFiles found: ${entries.length}\n`;
-                
-                // Get handle for file
-                const logHandle = await dirHandle.getFileHandle('session-log.txt', { create: true });
-                
-                // --- SECOND FIX: Verify permission on the FILE handle too ---
-                // Sometimes the directory has permission, but the new file handle needs it too.
-                if ((await logHandle.queryPermission(options)) !== 'granted') {
-                     await logHandle.requestPermission(options);
+                // --- THE FIX: Request Permission on the FILE handle explicitly ---
+                // Android does not always inherit permission from the parent folder.
+                if ((await fileHandle.queryPermission(options)) !== 'granted') {
+                    log("⚠️ Requesting permission for the specific file...");
+                    if ((await fileHandle.requestPermission(options)) !== 'granted') {
+                        throw new Error("File write permission denied.");
+                    }
                 }
+                // ------------------------------------------------------------------
 
-                const logWritable = await logHandle.createWritable();
-                await logWritable.write(existingLog + newEntry);
-                await logWritable.close();
-                log(`✅ Updated session-log.txt (Visit #${visitCount})`);
-
-                // Create a unique timestamped file on each visit
-                const uniqueFileName = `test-${Date.now()}.txt`;
-                const uniqueHandle = await dirHandle.getFileHandle(uniqueFileName, { create: true });
-                const uniqueWritable = await uniqueHandle.createWritable();
-                await uniqueWritable.write(`Created during visit #${visitCount} at ${timestamp}`);
-                await uniqueWritable.close();
-                log(`✅ Created '${uniqueFileName}'`);
+                log("Attempting to write...");
+                const writable = await fileHandle.createWritable();
+                
+                // Read existing content if possible to append (optional)
+                const timestamp = new Date().toISOString();
+                await writable.write(`\nLog Entry: ${timestamp}`);
+                await writable.close();
 
-                log(`\n🎉 Demo complete! Reload page and select same folder to see persistence.`);
+                log(`✅ Success! Wrote to ${fileName}`);
 
             } catch (err) {
                 if (err.name === 'AbortError') log("User cancelled.");