start-command 0.26.0 → 0.27.1

package/test/docker-autoremove.test.js

@@ -1,164 +0,0 @@
-#!/usr/bin/env bun
-/**
- * Tests for Docker auto-remove container feature
- */
-
-const { describe, it } = require('node:test');
-const assert = require('assert');
-const { isCommandAvailable } = require('../src/lib/isolation');
-const { runInDocker } = require('../src/lib/isolation');
-const { execSync } = require('child_process');
-
-// Helper to wait for a condition with timeout
-async function waitFor(conditionFn, timeout = 5000, interval = 100) {
-  const startTime = Date.now();
-  while (Date.now() - startTime < timeout) {
-    if (conditionFn()) {
-      return true;
-    }
-    await new Promise((resolve) => setTimeout(resolve, interval));
-  }
-  return false;
-}
-
-// Use the canRunLinuxDockerImages function from isolation module
-// to properly detect if Linux containers can run (handles Windows containers mode)
-const { canRunLinuxDockerImages } = require('../src/lib/isolation');
-
-describe('Docker Auto-Remove Container Feature', () => {
-  // These tests verify the --auto-remove-docker-container option
-  // which automatically removes the container after exit (disabled by default)
-
-  describe('auto-remove enabled', () => {
-    it('should automatically remove container when autoRemoveDockerContainer is true', async () => {
-      if (!canRunLinuxDockerImages()) {
-        console.log(
-          '  Skipping: docker not available, daemon not running, or Linux containers not supported'
-        );
-        return;
-      }
-
-      const containerName = `test-autoremove-${Date.now()}`;
-
-      // Run command with autoRemoveDockerContainer enabled
-      const result = await runInDocker('echo "test" && sleep 0.5', {
-        image: 'alpine:latest',
-        session: containerName,
-        detached: true,
-        keepAlive: false,
-        autoRemoveDockerContainer: true,
-      });
-
-      assert.strictEqual(result.success, true);
-      assert.ok(
-        result.message.includes('automatically removed'),
-        'Message should indicate auto-removal'
-      );
-
-      // Wait for container to finish and be removed
-      const containerRemoved = await waitFor(() => {
-        try {
-          execSync(`docker inspect -f '{{.State.Status}}' ${containerName}`, {
-            encoding: 'utf8',
-            stdio: ['pipe', 'pipe', 'pipe'],
-          });
-          return false; // Container still exists
-        } catch {
-          return true; // Container does not exist (removed)
-        }
-      }, 10000);
-
-      assert.ok(
-        containerRemoved,
-        'Container should be automatically removed after exit with --auto-remove-docker-container'
-      );
-
-      // Double-check with docker ps -a that container is completely removed
-      try {
-        const allContainers = execSync('docker ps -a', {
-          encoding: 'utf8',
-          stdio: ['pipe', 'pipe', 'pipe'],
-        });
-        assert.ok(
-          !allContainers.includes(containerName),
-          'Container should NOT appear in docker ps -a (completely removed)'
-        );
-        console.log(
-          '  ✓ Docker container auto-removed after exit (filesystem not preserved)'
-        );
-      } catch (err) {
-        assert.fail(`Failed to verify container removal: ${err.message}`);
-      }
-
-      // No cleanup needed - container should already be removed
-    });
-  });
-
-  describe('auto-remove disabled (default)', () => {
-    it('should preserve container filesystem by default (without autoRemoveDockerContainer)', async () => {
-      if (!canRunLinuxDockerImages()) {
-        console.log(
-          '  Skipping: docker not available, daemon not running, or Linux containers not supported'
-        );
-        return;
-      }
-
-      const containerName = `test-preserve-${Date.now()}`;
-
-      // Run command without autoRemoveDockerContainer
-      const result = await runInDocker('echo "test" && sleep 0.1', {
-        image: 'alpine:latest',
-        session: containerName,
-        detached: true,
-        keepAlive: false,
-        autoRemoveDockerContainer: false,
-      });
-
-      assert.strictEqual(result.success, true);
-      assert.ok(
-        result.message.includes('filesystem will be preserved'),
-        'Message should indicate filesystem preservation'
-      );
-
-      // Wait for container to exit
-      await waitFor(() => {
-        try {
-          const status = execSync(
-            `docker inspect -f '{{.State.Status}}' ${containerName}`,
-            {
-              encoding: 'utf8',
-              stdio: ['pipe', 'pipe', 'pipe'],
-            }
-          ).trim();
-          return status === 'exited';
-        } catch {
-          return false;
-        }
-      }, 10000);
-
-      // Container should still exist (in exited state)
-      try {
-        const allContainers = execSync('docker ps -a', {
-          encoding: 'utf8',
-          stdio: ['pipe', 'pipe', 'pipe'],
-        });
-        assert.ok(
-          allContainers.includes(containerName),
-          'Container should appear in docker ps -a (filesystem preserved)'
-        );
-        console.log(
-          '  ✓ Docker container filesystem preserved by default (can be re-entered)'
-        );
-      } catch (err) {
-        assert.fail(`Failed to verify container preservation: ${err.message}`);
-      }
-
-      // Clean up
-      try {
-        execSync(`docker rm -f ${containerName}`, { stdio: 'ignore' });
-      } catch {
-        // Ignore cleanup errors
-      }
-    });
-  });
-});

package/test/release-name.test.mjs

@@ -1,34 +0,0 @@
-import { describe, expect, it } from 'bun:test';
-import { releaseName, releaseTag } from '../../scripts/release-name.mjs';
-
-describe('releaseTag', () => {
-  it('uses plain "v${version}" when no prefix is given', () => {
-    expect(releaseTag('0.25.4')).toBe('v0.25.4');
-    expect(releaseTag('0.25.4', '')).toBe('v0.25.4');
-  });
-
-  it('prepends known language prefixes', () => {
-    expect(releaseTag('0.25.4', 'js-')).toBe('js-v0.25.4');
-    expect(releaseTag('0.14.0', 'rust-')).toBe('rust-v0.14.0');
-  });
-
-  it('passes arbitrary prefixes through', () => {
-    expect(releaseTag('1.0.0', 'api-')).toBe('api-v1.0.0');
-  });
-});
-
-describe('releaseName', () => {
-  it('returns bare version when prefix is empty (preserves pre-issue-108 behaviour)', () => {
-    expect(releaseName('0.25.4')).toBe('0.25.4');
-    expect(releaseName('0.25.4', '')).toBe('0.25.4');
-  });
-
-  it('decorates known language prefixes with human titles', () => {
-    expect(releaseName('0.25.4', 'js-')).toBe('[JavaScript] 0.25.4');
-    expect(releaseName('0.14.0', 'rust-')).toBe('[Rust] 0.14.0');
-  });
-
-  it('falls back to "${prefix}${version}" for unknown prefixes', () => {
-    expect(releaseName('1.0.0', 'api-')).toBe('api-1.0.0');
-  });
-});