@nimblebrain/mpak 0.1.0 → 0.2.0

package/src/commands/packages/run.test.ts

@@ -1,261 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { homedir } from 'os';
-import { join } from 'path';
-import { parsePackageSpec, getCacheDir, resolveArgs, substituteUserConfig, substituteEnvVars, getLocalCacheDir, localBundleNeedsExtract } from './run.js';
-
-describe('parsePackageSpec', () => {
-  describe('scoped packages', () => {
-    it('parses @scope/name without version', () => {
-      expect(parsePackageSpec('@scope/name')).toEqual({
-        name: '@scope/name',
-      });
-    });
-
-    it('parses @scope/name@1.0.0', () => {
-      expect(parsePackageSpec('@scope/name@1.0.0')).toEqual({
-        name: '@scope/name',
-        version: '1.0.0',
-      });
-    });
-
-    it('parses prerelease versions @scope/name@1.0.0-beta.1', () => {
-      expect(parsePackageSpec('@scope/name@1.0.0-beta.1')).toEqual({
-        name: '@scope/name',
-        version: '1.0.0-beta.1',
-      });
-    });
-
-    it('parses version with build metadata @scope/name@1.0.0+build.123', () => {
-      expect(parsePackageSpec('@scope/name@1.0.0+build.123')).toEqual({
-        name: '@scope/name',
-        version: '1.0.0+build.123',
-      });
-    });
-  });
-
-  describe('edge cases', () => {
-    it('handles package name with multiple slashes @org/sub/name', () => {
-      // This is technically invalid per npm spec, but we should handle gracefully
-      const result = parsePackageSpec('@org/sub/name');
-      expect(result.name).toBe('@org/sub/name');
-    });
-
-    it('handles unscoped package name', () => {
-      expect(parsePackageSpec('simple-name')).toEqual({
-        name: 'simple-name',
-      });
-    });
-
-    it('treats unscoped@version as invalid (mpak requires scoped packages)', () => {
-      // mpak only supports scoped packages (@scope/name)
-      // An unscoped name with @ is treated as the full name, not name@version
-      expect(parsePackageSpec('unscoped@1.0.0')).toEqual({
-        name: 'unscoped@1.0.0',
-      });
-    });
-
-    it('handles empty string', () => {
-      expect(parsePackageSpec('')).toEqual({ name: '' });
-    });
-
-    it('handles @ only', () => {
-      expect(parsePackageSpec('@')).toEqual({ name: '@' });
-    });
-  });
-});
-
-describe('getCacheDir', () => {
-  const expectedBase = join(homedir(), '.mpak', 'cache');
-
-  it('converts @scope/name to scope-name', () => {
-    expect(getCacheDir('@nimblebraininc/echo')).toBe(
-      join(expectedBase, 'nimblebraininc-echo')
-    );
-  });
-
-  it('handles simple scoped names', () => {
-    expect(getCacheDir('@foo/bar')).toBe(join(expectedBase, 'foo-bar'));
-  });
-
-  it('handles unscoped names', () => {
-    expect(getCacheDir('simple')).toBe(join(expectedBase, 'simple'));
-  });
-});
-
-describe('resolveArgs', () => {
-  const cacheDir = '/Users/test/.mpak/cache/scope-name';
-
-  it('resolves ${__dirname} placeholder', () => {
-    expect(resolveArgs(['${__dirname}/dist/index.js'], cacheDir)).toEqual([
-      `${cacheDir}/dist/index.js`,
-    ]);
-  });
-
-  it('resolves multiple ${__dirname} in single arg', () => {
-    expect(
-      resolveArgs(['--config=${__dirname}/config.json'], cacheDir)
-    ).toEqual([`--config=${cacheDir}/config.json`]);
-  });
-
-  it('resolves ${__dirname} in multiple args', () => {
-    expect(
-      resolveArgs(
-        ['${__dirname}/index.js', '--config', '${__dirname}/config.json'],
-        cacheDir
-      )
-    ).toEqual([
-      `${cacheDir}/index.js`,
-      '--config',
-      `${cacheDir}/config.json`,
-    ]);
-  });
-
-  it('leaves args without placeholders unchanged', () => {
-    expect(resolveArgs(['-m', 'mcp_echo.server'], cacheDir)).toEqual([
-      '-m',
-      'mcp_echo.server',
-    ]);
-  });
-
-  it('handles empty args array', () => {
-    expect(resolveArgs([], cacheDir)).toEqual([]);
-  });
-
-  it('handles Windows-style paths in cacheDir', () => {
-    const winPath = 'C:\\Users\\test\\.mpak\\cache\\scope-name';
-    expect(resolveArgs(['${__dirname}\\dist\\index.js'], winPath)).toEqual([
-      `${winPath}\\dist\\index.js`,
-    ]);
-  });
-});
-
-describe('substituteUserConfig', () => {
-  it('substitutes single user_config variable', () => {
-    expect(
-      substituteUserConfig('${user_config.api_key}', { api_key: 'secret123' })
-    ).toBe('secret123');
-  });
-
-  it('substitutes multiple user_config variables', () => {
-    expect(
-      substituteUserConfig('key=${user_config.key}&secret=${user_config.secret}', {
-        key: 'mykey',
-        secret: 'mysecret',
-      })
-    ).toBe('key=mykey&secret=mysecret');
-  });
-
-  it('leaves unmatched variables unchanged', () => {
-    expect(
-      substituteUserConfig('${user_config.missing}', { other: 'value' })
-    ).toBe('${user_config.missing}');
-  });
-
-  it('handles mixed matched and unmatched variables', () => {
-    expect(
-      substituteUserConfig('${user_config.found}-${user_config.missing}', {
-        found: 'yes',
-      })
-    ).toBe('yes-${user_config.missing}');
-  });
-
-  it('handles empty config values', () => {
-    expect(
-      substituteUserConfig('${user_config.empty}', { empty: '' })
-    ).toBe('');
-  });
-
-  it('handles values with special characters', () => {
-    expect(
-      substituteUserConfig('${user_config.key}', { key: 'abc$def{ghi}' })
-    ).toBe('abc$def{ghi}');
-  });
-
-  it('leaves non-user_config placeholders unchanged', () => {
-    expect(
-      substituteUserConfig('${__dirname}/path', { dirname: '/cache' })
-    ).toBe('${__dirname}/path');
-  });
-});
-
-describe('substituteEnvVars', () => {
-  it('substitutes user_config in all env vars', () => {
-    const env = {
-      API_KEY: '${user_config.api_key}',
-      DEBUG: 'true',
-      TOKEN: '${user_config.token}',
-    };
-    const values = { api_key: 'key123', token: 'tok456' };
-
-    expect(substituteEnvVars(env, values)).toEqual({
-      API_KEY: 'key123',
-      DEBUG: 'true',
-      TOKEN: 'tok456',
-    });
-  });
-
-  it('handles undefined env', () => {
-    expect(substituteEnvVars(undefined, { key: 'value' })).toEqual({});
-  });
-
-  it('handles empty env', () => {
-    expect(substituteEnvVars({}, { key: 'value' })).toEqual({});
-  });
-
-  it('preserves env vars without placeholders', () => {
-    const env = { PATH: '/usr/bin', HOME: '/home/user' };
-    expect(substituteEnvVars(env, {})).toEqual(env);
-  });
-
-  it('leaves unsubstituted placeholders as-is', () => {
-    const env = {
-      API_KEY: '${user_config.api_key}',
-      DEBUG: 'true',
-    };
-    // api_key not provided, so placeholder remains
-    // (process.env will override this at merge time)
-    expect(substituteEnvVars(env, {})).toEqual({
-      API_KEY: '${user_config.api_key}',
-      DEBUG: 'true',
-    });
-  });
-});
-
-describe('getLocalCacheDir', () => {
-  const expectedBase = join(homedir(), '.mpak', 'cache', '_local');
-
-  it('returns consistent hash for same path', () => {
-    const dir1 = getLocalCacheDir('/path/to/bundle.mcpb');
-    const dir2 = getLocalCacheDir('/path/to/bundle.mcpb');
-    expect(dir1).toBe(dir2);
-  });
-
-  it('returns different hash for different paths', () => {
-    const dir1 = getLocalCacheDir('/path/to/bundle1.mcpb');
-    const dir2 = getLocalCacheDir('/path/to/bundle2.mcpb');
-    expect(dir1).not.toBe(dir2);
-  });
-
-  it('includes _local in path', () => {
-    const dir = getLocalCacheDir('/path/to/bundle.mcpb');
-    expect(dir).toContain('_local');
-    expect(dir.startsWith(expectedBase)).toBe(true);
-  });
-
-  it('produces a 12-character hash suffix', () => {
-    const dir = getLocalCacheDir('/path/to/bundle.mcpb');
-    const hashPart = dir.split('/').pop();
-    expect(hashPart).toHaveLength(12);
-  });
-});
-
-describe('localBundleNeedsExtract', () => {
-  it('returns true when cache directory does not exist', () => {
-    expect(localBundleNeedsExtract('/any/path.mcpb', '/nonexistent/cache')).toBe(true);
-  });
-
-  it('returns true when meta file does not exist in cache dir', () => {
-    // Using a directory that exists but has no .mpak-meta.json
-    expect(localBundleNeedsExtract('/any/path.mcpb', '/tmp')).toBe(true);
-  });
-});