start-command 0.13.0 → 0.16.0

package/experiments/test-screen-output.sh

@@ -1,27 +0,0 @@
-#!/bin/bash
-# Experiment to understand screen output behavior
-
-echo "=== Test 1: Direct screen with command ==="
-screen -S test1 /bin/sh -c 'echo "hello from test1"'
-
-echo ""
-echo "=== Test 2: Screen with -L logging ==="
-cd /tmp
-screen -L -Logfile screen-test.log -S test2 /bin/sh -c 'echo "hello from test2"'
-echo "Log contents:"
-cat /tmp/screen-test.log 2>/dev/null || echo "No log file created"
-
-echo ""
-echo "=== Test 3: Screen detached then capture ==="
-screen -dmS test3 /bin/sh -c 'echo "hello from test3" > /tmp/screen-test3-out.txt'
-sleep 0.5
-echo "Output from test3:"
-cat /tmp/screen-test3-out.txt 2>/dev/null || echo "No output file"
-
-echo ""
-echo "=== Test 4: Screen with wrap using script command ==="
-script -q /dev/null -c 'screen -S test4 /bin/sh -c "echo hello from test4"' 2>/dev/null || echo "Script method failed"
-
-echo ""
-echo "Cleanup"
-rm -f /tmp/screen-test.log /tmp/screen-test3-out.txt

package/experiments/test-screen-tee-debug.js

@@ -1,237 +0,0 @@
-#!/usr/bin/env bun
-/**
- * Debug experiment to find the root cause of issue #25
- * We're testing different ways of running screen commands with tee
- */
-
-const { execSync, spawn } = require('child_process');
-const fs = require('fs');
-const path = require('path');
-const os = require('os');
-
-async function sleep(ms) {
-  return new Promise((resolve) => setTimeout(resolve, ms));
-}
-
-async function testDebug() {
-  console.log('=== Debugging Screen Tee Fallback ===\n');
-
-  // Test 1: Simple command without quotes issues
-  console.log('Test 1: Simple command (no spaces in the command)');
-  const sessionName1 = `debug1-${Date.now()}`;
-  const logFile1 = path.join(os.tmpdir(), `debug1-${sessionName1}.log`);
-
-  try {
-    // Direct screen command with tee
-    const cmd = `screen -dmS "${sessionName1}" /bin/sh -c "(echo hello) 2>&1 | tee \\"${logFile1}\\""`;
-    console.log(`  Command: ${cmd}`);
-
-    execSync(cmd, { stdio: 'inherit' });
-    await sleep(500);
-
-    if (fs.existsSync(logFile1)) {
-      console.log(
-        `  Log content: "${fs.readFileSync(logFile1, 'utf8').trim()}"`
-      );
-      console.log(`  Result: SUCCESS ✓`);
-      fs.unlinkSync(logFile1);
-    } else {
-      console.log(`  Result: FAILED - No log file ✗`);
-    }
-  } catch (e) {
-    console.log(`  Error: ${e.message}`);
-  }
-  console.log('');
-
-  // Test 2: Command with nested quotes
-  console.log('Test 2: Command with "hello" (has quotes)');
-  const sessionName2 = `debug2-${Date.now()}`;
-  const logFile2 = path.join(os.tmpdir(), `debug2-${sessionName2}.log`);
-
-  try {
-    // Note: The original command has quotes: echo "hello"
-    // When we wrap it with tee, the quoting becomes complex
-    const cmd = `screen -dmS "${sessionName2}" /bin/sh -c "(echo \\"hello\\") 2>&1 | tee \\"${logFile2}\\""`;
-    console.log(`  Command: ${cmd}`);
-
-    execSync(cmd, { stdio: 'inherit' });
-    await sleep(500);
-
-    if (fs.existsSync(logFile2)) {
-      console.log(
-        `  Log content: "${fs.readFileSync(logFile2, 'utf8').trim()}"`
-      );
-      console.log(`  Result: SUCCESS ✓`);
-      fs.unlinkSync(logFile2);
-    } else {
-      console.log(`  Result: FAILED - No log file ✗`);
-    }
-  } catch (e) {
-    console.log(`  Error: ${e.message}`);
-  }
-  console.log('');
-
-  // Test 3: Using array-based command (like the current implementation)
-  console.log('Test 3: Using array-based args (current implementation style)');
-  const sessionName3 = `debug3-${Date.now()}`;
-  const logFile3 = path.join(os.tmpdir(), `debug3-${sessionName3}.log`);
-
-  try {
-    // This is what the current code does
-    const command = 'echo "hello"';
-    const effectiveCommand = `(${command}) 2>&1 | tee "${logFile3}"`;
-    const screenArgs = [
-      '-dmS',
-      sessionName3,
-      '/bin/sh',
-      '-c',
-      effectiveCommand,
-    ];
-
-    // Construct the command string as the code does
-    const cmdStr = `screen ${screenArgs.map((a) => `"${a}"`).join(' ')}`;
-    console.log(`  Constructed command: ${cmdStr}`);
-
-    execSync(cmdStr, { stdio: 'inherit' });
-    await sleep(500);
-
-    if (fs.existsSync(logFile3)) {
-      console.log(
-        `  Log content: "${fs.readFileSync(logFile3, 'utf8').trim()}"`
-      );
-      console.log(`  Result: SUCCESS ✓`);
-      fs.unlinkSync(logFile3);
-    } else {
-      console.log(`  Result: FAILED - No log file ✗`);
-    }
-  } catch (e) {
-    console.log(`  Error: ${e.message}`);
-  }
-  console.log('');
-
-  // Test 4: Check what happens with the nested quotes
-  console.log('Test 4: Checking quote escaping issue');
-  const sessionName4 = `debug4-${Date.now()}`;
-  const logFile4 = path.join(os.tmpdir(), `debug4-${sessionName4}.log`);
-
-  try {
-    const command = 'echo "hello from attached mode"';
-    const effectiveCommand = `(${command}) 2>&1 | tee "${logFile4}"`;
-    console.log(`  effectiveCommand: ${effectiveCommand}`);
-
-    // When we quote each arg with `"${a}"`, the command becomes double-quoted
-    // which can cause issues with nested quotes
-    const screenArgs = [
-      '-dmS',
-      sessionName4,
-      '/bin/sh',
-      '-c',
-      effectiveCommand,
-    ];
-    const cmdStr = `screen ${screenArgs.map((a) => `"${a}"`).join(' ')}`;
-    console.log(`  Full command: ${cmdStr}`);
-
-    // The problem: effectiveCommand has double quotes inside,
-    // and we're wrapping it with MORE double quotes
-    // This results in: screen "-dmS" "debug4-xxx" "/bin/sh" "-c" "(echo "hello from attached mode") 2>&1 | tee "...""
-    // The nested double quotes break the shell parsing!
-
-    execSync(cmdStr, { stdio: 'inherit' });
-    await sleep(500);
-
-    if (fs.existsSync(logFile4)) {
-      console.log(
-        `  Log content: "${fs.readFileSync(logFile4, 'utf8').trim()}"`
-      );
-      console.log(`  Result: SUCCESS ✓`);
-      fs.unlinkSync(logFile4);
-    } else {
-      console.log(`  Result: FAILED - No log file ✗`);
-    }
-  } catch (e) {
-    console.log(`  Error: ${e.message}`);
-  }
-  console.log('');
-
-  // Test 5: Proper escaping
-  console.log('Test 5: With proper quote escaping');
-  const sessionName5 = `debug5-${Date.now()}`;
-  const logFile5 = path.join(os.tmpdir(), `debug5-${sessionName5}.log`);
-
-  try {
-    const command = 'echo "hello from attached mode"';
-    // Escape the inner quotes
-    const escapedCommand = command.replace(/"/g, '\\"');
-    const effectiveCommand = `(${escapedCommand}) 2>&1 | tee "${logFile5}"`;
-    console.log(`  effectiveCommand: ${effectiveCommand}`);
-
-    const screenArgs = [
-      '-dmS',
-      sessionName5,
-      '/bin/sh',
-      '-c',
-      effectiveCommand,
-    ];
-    const cmdStr = `screen ${screenArgs.map((a) => `"${a}"`).join(' ')}`;
-    console.log(`  Full command: ${cmdStr}`);
-
-    execSync(cmdStr, { stdio: 'inherit' });
-    await sleep(500);
-
-    if (fs.existsSync(logFile5)) {
-      console.log(
-        `  Log content: "${fs.readFileSync(logFile5, 'utf8').trim()}"`
-      );
-      console.log(`  Result: SUCCESS ✓`);
-      fs.unlinkSync(logFile5);
-    } else {
-      console.log(`  Result: FAILED - No log file ✗`);
-    }
-  } catch (e) {
-    console.log(`  Error: ${e.message}`);
-  }
-  console.log('');
-
-  // Test 6: Use spawnSync instead of execSync with constructed string
-  console.log('Test 6: Using spawnSync with array (better approach)');
-  const sessionName6 = `debug6-${Date.now()}`;
-  const logFile6 = path.join(os.tmpdir(), `debug6-${sessionName6}.log`);
-
-  try {
-    const command = 'echo "hello from attached mode"';
-    const effectiveCommand = `(${command}) 2>&1 | tee "${logFile6}"`;
-    console.log(`  effectiveCommand: ${effectiveCommand}`);
-
-    const { spawnSync } = require('child_process');
-    const screenArgs = [
-      '-dmS',
-      sessionName6,
-      '/bin/sh',
-      '-c',
-      effectiveCommand,
-    ];
-    console.log(`  spawnSync args: screen ${screenArgs.join(' ')}`);
-
-    const result = spawnSync('screen', screenArgs, { stdio: 'inherit' });
-    console.log(`  spawnSync exit code: ${result.status}`);
-
-    await sleep(500);
-
-    if (fs.existsSync(logFile6)) {
-      console.log(
-        `  Log content: "${fs.readFileSync(logFile6, 'utf8').trim()}"`
-      );
-      console.log(`  Result: SUCCESS ✓`);
-      fs.unlinkSync(logFile6);
-    } else {
-      console.log(`  Result: FAILED - No log file ✗`);
-    }
-  } catch (e) {
-    console.log(`  Error: ${e.message}`);
-  }
-  console.log('');
-
-  console.log('=== Debug Tests Complete ===');
-}
-
-testDebug();

package/experiments/test-screen-tee-fallback.js

@@ -1,230 +0,0 @@
-#!/usr/bin/env bun
-/**
- * Experiment to test screen's tee fallback functionality
- * This simulates the behavior on macOS with older screen (< 4.5.1)
- * which doesn't support -Logfile option
- *
- * Issue #25: We don't get `Hello` output from `$ --isolated screen --verbose -- echo "hello"` command
- */
-
-const { execSync, spawnSync, spawn } = require('child_process');
-const fs = require('fs');
-const path = require('path');
-const os = require('os');
-
-async function sleep(ms) {
-  return new Promise((resolve) => setTimeout(resolve, ms));
-}
-
-async function testTeeFallback() {
-  console.log('=== Testing Screen Tee Fallback (macOS 4.0.3 simulation) ===\n');
-
-  // Test environment info
-  console.log('Environment:');
-  console.log(`  Platform: ${process.platform}`);
-  console.log(`  Node: ${process.version}`);
-  try {
-    const screenVersion = execSync('screen --version', {
-      encoding: 'utf8',
-    }).trim();
-    console.log(`  Screen: ${screenVersion}`);
-  } catch (e) {
-    console.log(`  Screen: Not available - ${e.message}`);
-    return;
-  }
-  console.log(
-    `  TTY: stdin=${process.stdin.isTTY}, stdout=${process.stdout.isTTY}`
-  );
-  console.log('');
-
-  // Test 1: The tee fallback approach (current implementation for macOS)
-  console.log('Test 1: Tee fallback approach (current implementation)');
-  const sessionName1 = `tee-test-${Date.now()}`;
-  const logFile1 = path.join(os.tmpdir(), `screen-tee-${sessionName1}.log`);
-  const command = 'echo "hello"';
-
-  try {
-    // This is the current implementation for older screen versions
-    const effectiveCommand = `(${command}) 2>&1 | tee "${logFile1}"`;
-    const shell = '/bin/sh';
-    const shellArg = '-c';
-    const screenArgs = [
-      '-dmS',
-      sessionName1,
-      shell,
-      shellArg,
-      effectiveCommand,
-    ];
-
-    console.log(`  Command: screen ${screenArgs.join(' ')}`);
-    console.log(`  Effective command inside screen: ${effectiveCommand}`);
-
-    execSync(`screen ${screenArgs.map((a) => `"${a}"`).join(' ')}`, {
-      stdio: 'inherit',
-    });
-
-    // Wait for completion and poll for session
-    let waited = 0;
-    const maxWait = 5000;
-    const interval = 100;
-
-    while (waited < maxWait) {
-      await sleep(interval);
-      waited += interval;
-
-      try {
-        const sessions = execSync('screen -ls', {
-          encoding: 'utf8',
-          stdio: ['pipe', 'pipe', 'pipe'],
-        });
-        if (!sessions.includes(sessionName1)) {
-          console.log(`  Session ended after ${waited}ms`);
-          break;
-        }
-      } catch {
-        // screen -ls returns non-zero if no sessions
-        console.log(`  Session ended after ${waited}ms (no sessions)`);
-        break;
-      }
-    }
-
-    // Check log file
-    if (fs.existsSync(logFile1)) {
-      const content = fs.readFileSync(logFile1, 'utf8');
-      console.log(`  Log file exists: YES`);
-      console.log(`  Log file size: ${content.length} bytes`);
-      console.log(`  Log content: "${content.trim()}"`);
-      console.log(
-        `  Contains expected output: ${content.includes('hello') ? 'YES ✓' : 'NO ✗'}`
-      );
-      fs.unlinkSync(logFile1);
-    } else {
-      console.log(`  Log file exists: NO ✗`);
-      console.log(`  Expected path: ${logFile1}`);
-    }
-
-    // Cleanup
-    try {
-      execSync(`screen -S ${sessionName1} -X quit 2>/dev/null`);
-    } catch {}
-  } catch (e) {
-    console.log(`  Error: ${e.message}`);
-  }
-  console.log('');
-
-  // Test 2: Test the attached mode WITHOUT TTY (hasTTY() returns false)
-  console.log(
-    'Test 2: Simulating attached mode without TTY (current code path)'
-  );
-  const sessionName2 = `tee-notty-${Date.now()}`;
-  const logFile2 = path.join(os.tmpdir(), `screen-tee-${sessionName2}.log`);
-
-  try {
-    // Simulate the runScreenWithLogCapture function behavior for older screen
-    const command2 = 'echo "hello from attached mode"';
-    const effectiveCommand2 = `(${command2}) 2>&1 | tee "${logFile2}"`;
-    const shell = '/bin/sh';
-    const shellArg = '-c';
-    const screenArgs = [
-      '-dmS',
-      sessionName2,
-      shell,
-      shellArg,
-      effectiveCommand2,
-    ];
-
-    console.log(`  Screen args: ${screenArgs.join(' ')}`);
-
-    execSync(`screen ${screenArgs.map((a) => `"${a}"`).join(' ')}`, {
-      stdio: 'inherit',
-    });
-
-    // Poll for session completion (as done in current implementation)
-    const checkInterval = 100;
-    const maxWait = 5000;
-    let waited = 0;
-
-    const checkCompletion = async () => {
-      while (waited < maxWait) {
-        await sleep(checkInterval);
-        waited += checkInterval;
-
-        try {
-          const sessions = execSync('screen -ls', {
-            encoding: 'utf8',
-            stdio: ['pipe', 'pipe', 'pipe'],
-          });
-          if (!sessions.includes(sessionName2)) {
-            break;
-          }
-        } catch {
-          break;
-        }
-      }
-    };
-
-    await checkCompletion();
-    console.log(`  Session ended after ${waited}ms`);
-
-    // Read output
-    if (fs.existsSync(logFile2)) {
-      const content = fs.readFileSync(logFile2, 'utf8');
-      console.log(`  Log file exists: YES`);
-      console.log(`  Log content: "${content.trim()}"`);
-      console.log(
-        `  Contains expected: ${content.includes('hello from attached mode') ? 'YES ✓' : 'NO ✗'}`
-      );
-      fs.unlinkSync(logFile2);
-    } else {
-      console.log(`  Log file exists: NO ✗`);
-    }
-
-    try {
-      execSync(`screen -S ${sessionName2} -X quit 2>/dev/null`);
-    } catch {}
-  } catch (e) {
-    console.log(`  Error: ${e.message}`);
-  }
-  console.log('');
-
-  // Test 3: What happens if we have a TTY?
-  console.log('Test 3: Test with attached mode WITH TTY (spawn with inherit)');
-  const sessionName3 = `tty-test-${Date.now()}`;
-
-  try {
-    const command3 = 'echo "hello TTY"';
-    const screenArgs = ['-S', sessionName3, '/bin/sh', '-c', command3];
-
-    console.log(`  Screen args for attached mode: ${screenArgs.join(' ')}`);
-    console.log(
-      `  hasTTY: stdin=${process.stdin.isTTY}, stdout=${process.stdout.isTTY}`
-    );
-
-    // This mimics what the current code does when hasTTY() returns true
-    return new Promise((resolve) => {
-      const child = spawn('screen', screenArgs, {
-        stdio: 'inherit',
-      });
-
-      child.on('exit', (code) => {
-        console.log(`  Exit code: ${code}`);
-        console.log(
-          `  Note: In attached mode with TTY, output goes directly to terminal`
-        );
-        resolve();
-      });
-
-      child.on('error', (err) => {
-        console.log(`  Error: ${err.message}`);
-        resolve();
-      });
-    });
-  } catch (e) {
-    console.log(`  Error: ${e.message}`);
-  }
-  console.log('');
-
-  console.log('=== Tests Complete ===');
-}
-
-testTeeFallback();

package/experiments/test-substitution.js

@@ -1,143 +0,0 @@
-#!/usr/bin/env bun
-/**
- * Test script for the substitution engine
- * Tests pattern matching with various inputs
- */
-
-const path = require('path');
-
-// Import the substitution module
-const {
-  parseLinoContent,
-  matchAndSubstitute,
-  loadDefaultSubstitutions,
-  processCommand,
-} = require('../src/lib/substitution');
-
-const testCases = [
-  // NPM install patterns
-  {
-    input: 'install gh-upload-log npm package',
-    expected: 'npm install gh-upload-log',
-    description: 'Basic npm install',
-  },
-  {
-    input: 'install 0.0.1 version of gh-upload-log npm package',
-    expected: 'npm install gh-upload-log@0.0.1',
-    description: 'npm install with version',
-  },
-  {
-    input: 'install lodash npm package globally',
-    expected: 'npm install -g lodash',
-    description: 'Global npm install',
-  },
-  {
-    input: 'install 4.17.21 version of lodash npm package globally',
-    expected: 'npm install -g lodash@4.17.21',
-    description: 'Global npm install with version',
-  },
-  {
-    input: 'uninstall lodash npm package',
-    expected: 'npm uninstall lodash',
-    description: 'npm uninstall',
-  },
-
-  // Git patterns
-  {
-    input: 'clone https://github.com/user/repo repository',
-    expected: 'git clone https://github.com/user/repo',
-    description: 'Git clone with URL',
-  },
-  {
-    input: 'clone git@github.com:user/repo.git repository',
-    expected: 'git clone git@github.com:user/repo.git',
-    description: 'Git clone with SSH URL',
-  },
-  {
-    input: 'checkout main branch',
-    expected: 'git checkout main',
-    description: 'Git checkout branch',
-  },
-  {
-    input: 'create feature-x branch',
-    expected: 'git checkout -b feature-x',
-    description: 'Git create branch',
-  },
-
-  // Common operations
-  {
-    input: 'list files',
-    expected: 'ls -la',
-    description: 'List files',
-  },
-  {
-    input: 'show current directory',
-    expected: 'pwd',
-    description: 'Show working directory',
-  },
-  {
-    input: 'create my-project directory',
-    expected: 'mkdir -p my-project',
-    description: 'Create directory',
-  },
-
-  // Python patterns
-  {
-    input: 'install requests python package',
-    expected: 'pip install requests',
-    description: 'pip install',
-  },
-
-  // Non-matching patterns (should return original)
-  {
-    input: 'echo hello world',
-    expected: 'echo hello world',
-    description: 'Non-matching command (pass through)',
-  },
-  {
-    input: 'npm test',
-    expected: 'npm test',
-    description: 'Regular npm command (pass through)',
-  },
-];
-
-console.log('=== Substitution Engine Tests ===\n');
-
-// Load default substitutions
-const rules = loadDefaultSubstitutions();
-console.log(`Loaded ${rules.length} substitution rules\n`);
-
-let passed = 0;
-let failed = 0;
-
-for (const test of testCases) {
-  const result = matchAndSubstitute(test.input, rules);
-
-  if (result.command === test.expected) {
-    console.log(`✓ PASS: ${test.description}`);
-    console.log(`  Input:    "${test.input}"`);
-    console.log(`  Output:   "${result.command}"`);
-    console.log(
-      `  Matched:  ${result.matched ? result.rule.pattern : 'none (pass through)'}`
-    );
-    passed++;
-  } else {
-    console.log(`✗ FAIL: ${test.description}`);
-    console.log(`  Input:    "${test.input}"`);
-    console.log(`  Expected: "${test.expected}"`);
-    console.log(`  Got:      "${result.command}"`);
-    console.log(`  Matched:  ${result.matched ? result.rule.pattern : 'none'}`);
-    failed++;
-  }
-  console.log('');
-}
-
-console.log('=== Summary ===');
-console.log(`Passed: ${passed}/${testCases.length}`);
-console.log(`Failed: ${failed}/${testCases.length}`);
-
-if (failed > 0) {
-  process.exit(1);
-}
-
-console.log('\n=== All tests passed! ===');