fix: Add ESM banner for __dirname/__filename compatibility

esbuild was tree-shaking the ESM polyfill for __dirname and __filename,
causing "ReferenceError: __dirname is not defined" in compiled CLI files.

- Add ESM banner to esbuild.config.js that injects the polyfill
- Remove redundant manual polyfills from source files
- Add claude-sm-danger wrapper for --dangerously-skip-permissions

Author: jonathanpeterwu

bin/claude-smd

@@ -0,0 +1,6 @@
+#!/usr/bin/env node
+/**
+ * Claude-SM-Danger CLI Launcher (ESM)
+ * Delegates to built CLI in dist without requiring tsx.
+ */
+import('../dist/cli/claude-sm-danger.js');

esbuild.config.js

@@ -4,9 +4,15 @@ import { glob } from 'glob';
 
 // Get all TypeScript files except tests
 const entryPoints = glob.sync('src/**/*.ts', {
-  ignore: ['**/*.test.ts', '**/*.spec.ts', '**/__tests__/**']
+  ignore: ['**/*.test.ts', '**/*.spec.ts', '**/__tests__/**'],
 });
 
+// ESM polyfill for __dirname and __filename
+const esmBanner = `import { fileURLToPath as __fileURLToPath } from 'url';
+import { dirname as __pathDirname } from 'path';
+const __filename = __fileURLToPath(import.meta.url);
+const __dirname = __pathDirname(__filename);`;
+
 // Build configuration
 const buildConfig = {
   entryPoints,
@@ -19,6 +25,9 @@ const buildConfig = {
   logLevel: 'info',
   preserveSymlinks: false,
   splitting: false,
+  banner: {
+    js: esmBanner,
+  },
 };
 
 // Build function
@@ -37,4 +46,4 @@ if (import.meta.url === `file://${process.argv[1]}`) {
   build();
 }
 
-export { buildConfig, build };
+export { buildConfig, build };

src/cli/claude-sm-danger.ts

@@ -0,0 +1,32 @@
+#!/usr/bin/env node
+
+/**
+ * claude-sm-danger: Claude-SM wrapper with --dangerously-skip-permissions
+ * Shorthand for: claude-sm --dangerously-skip-permissions [args]
+ */
+
+import { spawn } from 'child_process';
+import * as path from 'path';
+
+// __filename and __dirname are provided by esbuild banner for ESM compatibility
+
+// Get the claude-sm script path
+const claudeSmPath = path.join(__dirname, 'claude-sm.js');
+
+// Prepend the danger flag to all args
+const args = ['--dangerously-skip-permissions', ...process.argv.slice(2)];
+
+// Spawn claude-sm with the danger flag
+const child = spawn('node', [claudeSmPath, ...args], {
+  stdio: 'inherit',
+  env: process.env,
+});
+
+child.on('exit', (code) => {
+  process.exit(code || 0);
+});
+
+child.on('error', (err) => {
+  console.error('Failed to launch claude-sm:', err.message);
+  process.exit(1);
+});

src/cli/claude-sm.ts

@@ -14,6 +14,8 @@ import { v4 as uuidv4 } from 'uuid';
 import chalk from 'chalk';
 import { initializeTracing, trace } from '../core/trace/index.js';
 
+// __filename and __dirname are provided by esbuild banner for ESM compatibility
+
 interface ClaudeConfig {
   instanceId: string;
   worktreePath?: string;

src/cli/commands/shell.ts

@@ -15,11 +15,9 @@ import {
   chmodSync,
   appendFileSync,
 } from 'fs';
-import { join, dirname } from 'path';
-import { fileURLToPath } from 'url';
+import { join } from 'path';
 
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
+// __filename and __dirname are provided by esbuild banner for ESM compatibility
 
 function getShellType(): 'zsh' | 'bash' | 'unknown' {
   const shell = process.env.SHELL || '';

src/cli/commands/sweep.ts

@@ -19,12 +19,10 @@ import {
   copyFileSync,
   chmodSync,
 } from 'fs';
-import { join, dirname } from 'path';
-import { fileURLToPath } from 'url';
+import { join } from 'path';
 import { spawn, execSync } from 'child_process';
 
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = dirname(__filename);
+// __filename and __dirname are provided by esbuild banner for ESM compatibility
 
 interface SweepStatus {
   installed: boolean;