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

Author: cs-util

index.html

@@ -128,61 +128,68 @@ <h1>Android File System Demo</h1>
         // 3. OPEN DIRECTORY (Read/Write)
         document.getElementById('btnDir').addEventListener('click', async () => {
             try {
-                log("Requesting directory access (ReadWrite)...");
+                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}`);
 
-                // List contents and count files
+                // --- THE CRITICAL FIX ---
+                // We must explicitly verify/request WRITE permission now.
+                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.");
+                    }
+                }
+                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)");
-                }
+                if (entries.length === 0) log("  (empty folder)");
 
-                // Check if session-log.txt exists and read its content
+                // Write/append to session log
                 let existingLog = "";
-                let visitCount = 1;
                 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 session-log.txt (${visitCount - 1} previous visits)`);
+                    log("📖 Found existing log.");
                 } catch {
-                    log("📝 No previous session-log.txt found, creating fresh log...");
+                    log("📝 Creating fresh log...");
                 }
 
-                // Write/append to session log
                 const timestamp = new Date().toISOString();
-                const newEntry = `\n--- Visit #${visitCount} ---\nTimestamp: ${timestamp}\nFiles found: ${entries.length}\n`;
+                const newEntry = `\n--- Visit ---\nTimestamp: ${timestamp}\nFiles: ${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);
+                }
+
                 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(`\n🎉 Demo complete! Reload page and select same folder to see persistence.`);
+                
+                log(`✅ Success! Wrote to session-log.txt`);
 
             } catch (err) {
-                if (err.name === 'AbortError') log("User cancelled folder picker.");
+                if (err.name === 'AbortError') log("User cancelled.");
                 else log(`Error: ${err.message}`, 'error');
             }
         });