sandboxbox 3.0.24 → 3.0.26

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sandboxbox",
-  "version": "3.0.24",
+  "version": "3.0.26",
   "description": "Lightweight process containment sandbox for CLI tools - Playwright, Claude Code, and more. Pure Node.js, no dependencies.",
   "type": "module",
   "main": "cli.js",

package/utils/sandbox.js

@@ -1,4 +1,4 @@
-import { mkdtempSync, rmSync, cpSync, existsSync, mkdirSync, writeFileSync, symlinkSync } from 'fs';
+import { mkdtempSync, rmSync, cpSync, existsSync, mkdirSync, writeFileSync, symlinkSync, realpathSync } from 'fs';
 import { tmpdir, homedir, platform } from 'os';
 import { join, resolve } from 'path';
 import { spawn, execSync } from 'child_process';
@@ -144,7 +144,7 @@ export function createSandbox(projectDir) {
         const settingsPath = join(hostClaudeDir, 'settings.json');
         if (existsSync(settingsPath)) {
           const settings = JSON.parse(readFileSync(settingsPath, 'utf8'));
-          if (settings.hooks) {
+          if (false && settings.hooks) {
             console.log('✅ Linked to host Claude settings directory');
             console.log('📋 Hooks configured:');
             Object.keys(settings.hooks).forEach(hookType => {
@@ -182,14 +182,42 @@ export function createSandbox(projectDir) {
         const sandboxFile = join(sandboxClaudeDir, file);
 
         if (existsSync(hostFile) && hostFile !== sandboxFile) {
+          // Additional check using real paths to detect symlink/bind mount issues
+          let shouldCopy = true;
+          try {
+            const hostRealPath = realpathSync(hostFile);
+            const sandboxRealPath = realpathSync(sandboxFile);
+            if (hostRealPath === sandboxRealPath) {
+              shouldCopy = false;
+            }
+          } catch (e) {
+            // If we can't resolve real paths, proceed with copy attempt
+          }
+
           if (VERBOSE_OUTPUT) {
             console.log(`   Copying ${file}: ${hostFile} -> ${sandboxFile}`);
+            console.log(`   Path comparison: hostFile === sandboxFile = ${hostFile === sandboxFile}`);
+            console.log(`   Host file exists: ${existsSync(hostFile)}`);
+            console.log(`   Sandbox file exists: ${existsSync(sandboxFile)}`);
+            try {
+              const hostRealPath = realpathSync(hostFile);
+              const sandboxRealPath = realpathSync(sandboxFile);
+              console.log(`   Real paths: host=${hostRealPath}, sandbox=${sandboxRealPath}`);
+              console.log(`   Real path comparison: hostRealPath === sandboxRealPath = ${hostRealPath === sandboxRealPath}`);
+            } catch (e) {
+              console.log(`   Real path check failed: ${e.message}`);
+            }
+          }
+
+          if (shouldCopy) {
+            cpSync(hostFile, sandboxFile);
+          } else if (VERBOSE_OUTPUT) {
+            console.log(`   Skipping copy - real paths are the same`);
           }
-          cpSync(hostFile, sandboxFile);
           if (VERBOSE_OUTPUT && file === 'settings.json') {
             // Show hook information for copied settings
-            const settings = JSON.parse(readFileSync(hostFile, 'utf8'));
-            if (settings.hooks) {
+            // const settings = JSON.parse(readFileSync(hostFile, 'utf8'));
+            if (false && settings.hooks) {
               console.log('📋 Hooks configured in copied settings:');
               Object.keys(settings.hooks).forEach(hookType => {
                 const hookCount = settings.hooks[hookType].length;
@@ -207,8 +235,8 @@ export function createSandbox(projectDir) {
           }
           if (VERBOSE_OUTPUT && file === 'settings.json') {
             // Show hook information for existing settings
-            const settings = JSON.parse(readFileSync(hostFile, 'utf8'));
-            if (settings.hooks) {
+            // const settings = JSON.parse(readFileSync(hostFile, 'utf8'));
+            if (false && settings.hooks) {
               console.log('📋 Hooks configured in existing settings:');
               Object.keys(settings.hooks).forEach(hookType => {
                 const hookCount = settings.hooks[hookType].length;