@bobfrankston/npmglobalize 1.0.85 → 1.0.86

package/.claude/settings.local.json

@@ -14,7 +14,16 @@
       "Bash(dir:*)",
       "Bash(findstr:*)",
       "Bash(npm access:*)",
-      "Bash(gh repo view:*)"
+      "Bash(gh repo view:*)",
+      "Bash(npmglobalize)",
+      "Bash(kill:*)",
+      "Bash(test:*)",
+      "Bash(cmd.exe /c \"dir y:\\\\dev\\\\homecontrol\\\\utils\\\\addone /a\")",
+      "Bash(cmd.exe /c \"dir y:\\\\dev\\\\homecontrol\\\\utils\\\\addone\\\\*.* /a\")",
+      "Bash(wsl ls:*)",
+      "Bash(cmd.exe /c \"dir y:\\\\dev\\\\homecontrol\\\\utils\\\\addone /a /b\")",
+      "Bash(cmd.exe /c \"dir /a /b y:\\\\dev\\\\homecontrol\\\\utils\\\\addone\")",
+      "Bash(icacls:*)"
     ]
   }
 }

package/lib.js

@@ -816,6 +816,37 @@ export function runCommandOrThrow(cmd, args, options = {}) {
     const stderr = result.stderr || '';
     const output = stdout + stderr;
     if (result.status !== 0) {
+        // Detect git safe.directory ownership issue and auto-fix
+        if (cmd === 'git' && stderr.includes('dubious ownership')) {
+            console.log(colors.yellow('Git "dubious ownership" error — directory owner SID differs from current user.'));
+            console.log(colors.yellow('Adding safe.directory \'*\' to global git config...'));
+            const fix = spawnSync('git', ['config', '--global', '--add', 'safe.directory', '*'], {
+                encoding: 'utf-8',
+                stdio: 'pipe',
+                shell: true
+            });
+            if (fix.status === 0) {
+                console.log(colors.green('✓ Fixed. Retrying...'));
+                // Retry the original command
+                const retry = spawnSync(cmd, args, {
+                    encoding: 'utf-8',
+                    stdio: 'pipe',
+                    cwd: options.cwd,
+                    env: process.env,
+                    shell: needsShell
+                });
+                if (retry.status === 0) {
+                    return (retry.stdout || '') + (retry.stderr || '');
+                }
+                // Retry also failed
+                const retryStderr = retry.stderr || '';
+                if (retryStderr.trim()) {
+                    console.error(colors.red(retryStderr.trim()));
+                }
+                throw new Error(`Command failed after safe.directory fix: ${cmd} ${args.join(' ')}`);
+            }
+            throw new Error('Failed to add safe.directory to git config.');
+        }
         // Show the error output
         if (stderr.trim()) {
             console.error(colors.red(stderr.trim()));
@@ -1277,6 +1308,27 @@ export async function initGit(cwd, visibility, dryRun) {
         console.log(`  [dry-run] Would run: gh repo create ${repoName} --${visibility} --source=. --push`);
         return true;
     }
+    // Check for dubious ownership before proceeding
+    const ownerCheck = spawnSync('git', ['rev-parse', '--git-dir'], {
+        encoding: 'utf-8',
+        stdio: 'pipe',
+        cwd
+    });
+    if (ownerCheck.stderr && ownerCheck.stderr.includes('dubious ownership')) {
+        console.log(colors.yellow('Git "dubious ownership" error — directory owner SID differs from current user.'));
+        console.log(colors.yellow('Adding safe.directory \'*\' to global git config...'));
+        const fix = spawnSync('git', ['config', '--global', '--add', 'safe.directory', '*'], {
+            encoding: 'utf-8',
+            stdio: 'pipe',
+            shell: true
+        });
+        if (fix.status === 0) {
+            console.log(colors.green('  ✓ Fixed git safe.directory'));
+        }
+        else {
+            throw new Error('Failed to fix git safe.directory. Run manually: git config --global --add safe.directory \'*\'');
+        }
+    }
     // Ensure .gitignore exists and includes node_modules
     ensureGitignore(cwd);
     // Ensure .npmignore exists with recommended patterns
@@ -1693,8 +1745,8 @@ export async function globalize(cwd, options = {}, configOptions = {}) {
     }
     // Read package.json
     const pkg = readPackageJson(cwd);
-    // Run build step if package.json has a build script
-    if (pkg.scripts?.build) {
+    // Run build step if package.json has a build script (skip if CLI already built)
+    if (pkg.scripts?.build && !options._fromCli) {
         console.log('Running build...');
         if (!dryRun) {
             const buildResult = runCommand('npm', ['run', 'build'], { cwd, silent: !verbose });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bobfrankston/npmglobalize",
-  "version": "1.0.85",
+  "version": "1.0.86",
   "description": "Transform file: dependencies to npm versions for publishing",
   "main": "index.js",
   "type": "module",