fixup! sqlite: fix permission VFS edge cases

Assisted-by: Claude Code
Signed-off-by: Guilherme Araújo <arauujogui@gmail.com>

Author: araujogui

src/node_sqlite_vfs.cc

@@ -316,8 +316,7 @@ bool FileControlRequiresWrite(int operation) {
 
 int PermissionFileControl(sqlite3_file* file, int operation, void* argument) {
   PermissionFile* permission_file = PermissionFileFromBase(file);
-  if (FileControlRequiresWrite(operation) &&
-      !permission_file->can_write) {
+  if (FileControlRequiresWrite(operation) && !permission_file->can_write) {
     return SQLITE_PERM;
   }
 

test/parallel/test-permission-sqlite.js

@@ -195,7 +195,8 @@ const readWrite = `const { DatabaseSync } = require('node:sqlite');
   // a sidecar does not inherit permission when opened as a main database.
   const code = `const { DatabaseSync } = require('node:sqlite');
     const db = new DatabaseSync(process.env.DB_PATH);
-    db.exec("ATTACH DATABASE '" + process.env.ATTACH_PATH + "' AS outside");`;
+    const attachPath = process.env.ATTACH_PATH.replaceAll("'", "''");
+    db.exec("ATTACH DATABASE '" + attachPath + "' AS outside");`;
   for (const attachPath of [deniedPath, suffixDbPath]) {
     const { status, stderr } = run(
       code,