@cnrai/pave 0.3.35 → 0.3.51

package/test/build-config.test.js

@@ -1,120 +0,0 @@
-#!/usr/bin/env node
-/**
- * Tests for build configuration
- * Verifies that Bun compile targets are correctly configured
- *
- * Issue #73: CPU AVX Support Warning on macOS Startup
- * Solution: Use baseline builds for x64 platforms to avoid AVX requirement
- *
- * This test validates the TARGETS configuration from build-binary.js
- */
-
-const path = require('path');
-
-// Import the actual TARGETS from build-binary.js to validate the real source of truth
-// This ensures tests will catch regressions if build-binary.js is changed
-const buildBinaryPath = path.join(__dirname, '..', 'build-binary.js');
-const { TARGETS } = require(buildBinaryPath);
-
-// Test state tracking
-let testsPassed = 0;
-let testsFailed = 0;
-
-// Test helpers
-function runTest(name, testFn) {
-  try {
-    testFn();
-    console.log(`✅ ${name}`);
-    testsPassed++;
-  } catch (error) {
-    console.log(`❌ ${name}: ${error.message}`);
-    testsFailed++;
-    process.exitCode = 1;
-  }
-}
-
-function assertEqual(actual, expected, message) {
-  if (actual !== expected) {
-    throw new Error(`${message || 'Assertion'}: expected ${expected}, got ${actual}`);
-  }
-}
-
-function assertTrue(condition, message) {
-  if (!condition) {
-    throw new Error(message || 'Expected condition to be true');
-  }
-}
-
-function assertContains(str, substring, message) {
-  if (!str.includes(substring)) {
-    throw new Error(`${message || 'Assertion'}: expected "${str}" to contain "${substring}"`);
-  }
-}
-
-function assertNotContains(str, substring, message) {
-  if (str.includes(substring)) {
-    throw new Error(`${message || 'Assertion'}: expected "${str}" to NOT contain "${substring}"`);
-  }
-}
-
-console.log('🔧 Build Configuration Tests (Issue #73: AVX Compatibility)\n');
-
-// Test: darwin-x64 should use baseline build
-runTest('darwin-x64 should use baseline build to avoid AVX requirement', () => {
-  assertEqual(TARGETS['darwin-x64'], 'bun-darwin-x64-baseline', 'darwin-x64 target');
-  assertContains(TARGETS['darwin-x64'], 'baseline', 'darwin-x64 should contain baseline');
-});
-
-// Test: linux-x64 should use baseline build
-runTest('linux-x64 should use baseline build to avoid AVX requirement', () => {
-  assertEqual(TARGETS['linux-x64'], 'bun-linux-x64-baseline', 'linux-x64 target');
-  assertContains(TARGETS['linux-x64'], 'baseline', 'linux-x64 should contain baseline');
-});
-
-// Test: darwin-arm64 should use standard build (ARM does not have AVX)
-runTest('darwin-arm64 should use standard build (ARM does not have AVX)', () => {
-  assertEqual(TARGETS['darwin-arm64'], 'bun-darwin-arm64', 'darwin-arm64 target');
-  assertNotContains(TARGETS['darwin-arm64'], 'baseline', 'darwin-arm64 should NOT contain baseline');
-});
-
-// Test: linux-arm64 should use standard build (ARM does not have AVX)
-runTest('linux-arm64 should use standard build (ARM does not have AVX)', () => {
-  assertEqual(TARGETS['linux-arm64'], 'bun-linux-arm64', 'linux-arm64 target');
-  assertNotContains(TARGETS['linux-arm64'], 'baseline', 'linux-arm64 should NOT contain baseline');
-});
-
-// Test: Required platform targets should be defined
-// Using Object.prototype.hasOwnProperty.call() to avoid issues if hasOwnProperty is shadowed
-runTest('required platform targets should be defined', () => {
-  assertTrue(Object.prototype.hasOwnProperty.call(TARGETS, 'darwin-arm64'), 'darwin-arm64 should be defined');
-  assertTrue(Object.prototype.hasOwnProperty.call(TARGETS, 'darwin-x64'), 'darwin-x64 should be defined');
-  assertTrue(Object.prototype.hasOwnProperty.call(TARGETS, 'linux-arm64'), 'linux-arm64 should be defined');
-  assertTrue(Object.prototype.hasOwnProperty.call(TARGETS, 'linux-x64'), 'linux-x64 should be defined');
-});
-
-// Test: x64 targets should use baseline builds, ARM targets should not
-runTest('x64 targets should use baseline builds, ARM targets should not', () => {
-  const x64Targets = Object.entries(TARGETS).filter(([key]) => key.includes('x64'));
-  const arm64Targets = Object.entries(TARGETS).filter(([key]) => key.includes('arm64'));
-
-  // All x64 targets should use baseline
-  for (const [key, value] of x64Targets) {
-    assertContains(value, 'baseline', `${key} should use baseline build`);
-  }
-
-  // No ARM64 targets should use baseline
-  for (const [key, value] of arm64Targets) {
-    assertNotContains(value, 'baseline', `${key} should NOT use baseline build`);
-  }
-});
-
-// Print summary based on test results
-// Note: Don't start with ✅/❌ as run-all.js counts those as test results
-if (testsFailed === 0) {
-  console.log(`\nSummary: All ${testsPassed} build configuration tests passed!`);
-} else {
-  console.log(`\nSummary: ${testsFailed} of ${testsPassed + testsFailed} tests failed`);
-  // Throw an error so the test runner marks this file as failed when any test fails
-  // The test runner uses require() which only fails if the module throws
-  throw new Error(`build-config.test.js: ${testsFailed} test(s) failed`);
-}

package/test/build-npm.test.js

@@ -1,388 +0,0 @@
-/**
- * Tests for build-npm.js (Issue #157)
- *
- * Verifies that the npm build script:
- * - Exists and is properly structured
- * - Generates correct package.json for @cnrai/pave
- * - Obfuscates sandbox files
- * - Integrates with GitHub Actions workflow
- * - Produces valid npm package structure
- */
-
-const fs = require('fs');
-const path = require('path');
-
-const paveDir = path.join(__dirname, '..');
-const buildNpmSource = fs.readFileSync(path.join(paveDir, 'build-npm.js'), 'utf-8');
-const packageJson = JSON.parse(fs.readFileSync(path.join(paveDir, 'package.json'), 'utf-8'));
-const workflowPath = path.join(paveDir, '..', '..', '..', '.github', 'workflows', 'publish-obfuscated-dist.yml');
-const workflowSource = fs.readFileSync(workflowPath, 'utf-8');
-
-let passed = 0;
-let failed = 0;
-let total = 0;
-
-function test(name, fn) {
-  total++;
-  try {
-    fn();
-    console.log('\u2705 ' + name);
-    passed++;
-  } catch (err) {
-    console.log('\u274c ' + name);
-    console.log('  ' + err.message);
-    failed++;
-  }
-}
-
-function assert(cond, msg) {
-  if (!cond) throw new Error(msg || 'Assertion failed');
-}
-
-function assertIncludes(haystack, needle, msg) {
-  if (haystack.indexOf(needle) === -1) {
-    throw new Error(msg || 'Expected to find "' + needle + '"');
-  }
-}
-
-function assertEqual(actual, expected, msg) {
-  if (actual !== expected) {
-    throw new Error((msg || '') + ' Expected: ' + JSON.stringify(expected) + ', got: ' + JSON.stringify(actual));
-  }
-}
-
-console.log('\nTesting npm publish build (Issue #157)\n');
-console.log('='.repeat(60));
-
-// --- build-npm.js structure ---
-console.log('\n--- build-npm.js structure ---\n');
-
-test('build-npm.js exists', () => {
-  assert(buildNpmSource.length > 0, 'build-npm.js should not be empty');
-});
-
-test('build-npm.js has shebang', () => {
-  assert(buildNpmSource.startsWith('#!/usr/bin/env node'), 'Should start with node shebang');
-});
-
-test('build-npm.js uses esbuild for bundling', () => {
-  assertIncludes(buildNpmSource, "require('esbuild')", 'Should require esbuild');
-  assertIncludes(buildNpmSource, 'esbuild.build(', 'Should call esbuild.build');
-});
-
-test('build-npm.js targets node16', () => {
-  assertIncludes(buildNpmSource, "'node16'", 'Should target node16');
-});
-
-test('build-npm.js uses javascript-obfuscator', () => {
-  assertIncludes(buildNpmSource, 'javascript-obfuscator', 'Should use javascript-obfuscator');
-  assertIncludes(buildNpmSource, 'obfuscateFile', 'Should have obfuscateFile function');
-});
-
-test('build-npm.js outputs to dist/npm/', () => {
-  assertIncludes(buildNpmSource, "'dist', 'npm'", 'Should output to dist/npm/');
-});
-
-test('build-npm.js reads version from package.json', () => {
-  assertIncludes(buildNpmSource, 'packageJson.version', 'Should read version from package.json');
-});
-
-// --- Package metadata ---
-console.log('\n--- Package metadata ---\n');
-
-test('build-npm.js sets package name to @cnrai/pave', () => {
-  assertIncludes(buildNpmSource, "'@cnrai/pave'", 'Should use @cnrai/pave as package name');
-});
-
-test('build-npm.js sets bin.pave to ./pave.js', () => {
-  assertIncludes(buildNpmSource, "'./pave.js'", 'Should set bin to ./pave.js');
-  // Verify it's in the bin field
-  const idx = buildNpmSource.indexOf('bin:');
-  assert(idx > -1, 'Should have bin field');
-  const block = buildNpmSource.substring(idx, idx + 100);
-  assertIncludes(block, 'pave', 'bin should include pave');
-});
-
-test('build-npm.js sets engines >= 16', () => {
-  assertIncludes(buildNpmSource, "'>=16.0.0'", 'Should require Node >= 16.0.0');
-});
-
-test('build-npm.js includes blessed as dependency', () => {
-  assertIncludes(buildNpmSource, "blessed", 'Should include blessed as dependency');
-});
-
-test('build-npm.js includes js-yaml as dependency', () => {
-  assertIncludes(buildNpmSource, "js-yaml", 'Should include js-yaml as dependency');
-});
-
-test('build-npm.js specifies files field', () => {
-  assertIncludes(buildNpmSource, "'pave.js'", 'Should include pave.js in files');
-  assertIncludes(buildNpmSource, "'sandbox/'", 'Should include sandbox/ in files');
-  assertIncludes(buildNpmSource, "'README.md'", 'Should include README.md in files');
-  assertIncludes(buildNpmSource, "'LICENSE'", 'Should include LICENSE in files');
-});
-
-test('build-npm.js generates README.md', () => {
-  assertIncludes(buildNpmSource, 'README.md', 'Should generate README');
-  assertIncludes(buildNpmSource, 'npm install -g @cnrai/pave', 'README should have install command');
-});
-
-test('build-npm.js generates LICENSE', () => {
-  assertIncludes(buildNpmSource, 'MIT License', 'Should generate MIT LICENSE');
-});
-
-// --- Obfuscation ---
-console.log('\n--- Obfuscation ---\n');
-
-test('build-npm.js uses same obfuscation settings as build-binary.js', () => {
-  assertIncludes(buildNpmSource, "'low-obfuscation'", 'Should use low-obfuscation preset');
-  assertIncludes(buildNpmSource, "'base64'", 'Should use base64 string array encoding');
-  assertIncludes(buildNpmSource, "'mangled'", 'Should use mangled identifier names');
-});
-
-test('build-npm.js uses deterministic obfuscation for CI', () => {
-  assertIncludes(buildNpmSource, "'1.0'", 'Should use threshold 1.0 to move all strings');
-  assertIncludes(buildNpmSource, "'--seed'", 'Should use fixed seed for determinism');
-});
-
-test('build-npm.js surfaces obfuscator stderr on failure', () => {
-  assertIncludes(buildNpmSource, 'err.stderr', 'Should include stderr in error message');
-  assertIncludes(buildNpmSource, 'err.stdout', 'Should include stdout in error message');
-  assertIncludes(buildNpmSource, 'javascript-obfuscator failed for', 'Should identify which file failed');
-});
-
-test('build-npm.js obfuscates sandbox files', () => {
-  assertIncludes(buildNpmSource, 'obfuscateFile(tempPath', 'Should obfuscate sandbox files');
-});
-
-test('build-npm.js adds shebang to obfuscated bundle', () => {
-  assertIncludes(buildNpmSource, '#!/usr/bin/env node', 'Should add shebang');
-});
-
-test('build-npm.js fails build if sensitive strings visible', () => {
-  assertIncludes(buildNpmSource, 'process.exit(1)', 'Should exit with error on obfuscation failure');
-  assertIncludes(buildNpmSource, "Error: Some sensitive strings", 'Should log error message');
-});
-
-test('build-npm.js removes intermediate bundle file', () => {
-  assertIncludes(buildNpmSource, 'unlinkSync(bundlePath)', 'Should remove intermediate bundle');
-});
-
-test('build-npm.js uses execFileSync for safe path handling', () => {
-  assertIncludes(buildNpmSource, 'execFileSync', 'Should use execFileSync');
-  assertIncludes(buildNpmSource, "npxPath, args", 'Should pass arguments as array');
-  assertIncludes(buildNpmSource, 'cwd: ROOT_DIR', 'Should set cwd for deterministic resolution');
-});
-
-test('build-npm.js uses octal chmod mode', () => {
-  assertIncludes(buildNpmSource, '0o755', 'Should use octal 0o755 for chmod');
-  assert(buildNpmSource.indexOf("'755'") === -1, 'Should NOT use string chmod mode');
-});
-
-test('build-npm.js fails on missing sandbox files', () => {
-  assertIncludes(buildNpmSource, 'Required sandbox file is missing', 'Should error on missing sandbox files');
-});
-
-test('build-npm.js fails fast if javascript-obfuscator not installed', () => {
-  assertIncludes(buildNpmSource, '--no-install', 'Should use npx --no-install');
-  assertIncludes(buildNpmSource, 'execFileSync(npxCheck', 'Availability check should use execFileSync');
-  assertIncludes(buildNpmSource, "cwd: ROOT_DIR", 'Availability check should set cwd: ROOT_DIR');
-});
-
-test('obfuscateFile uses --no-install to prevent network installs', () => {
-  // The obfuscateFile function args array should include --no-install
-  const fnMatch = buildNpmSource.match(/function obfuscateFile\b[\s\S]*?(?:const|let|var) args\s*=\s*\[([\s\S]*?)\];/);
-  assert(fnMatch, 'Should find obfuscateFile args array');
-  assertIncludes(fnMatch[1], "'--no-install'", 'obfuscateFile args should include --no-install');
-});
-
-test('build-npm.js strips developer-specific paths from permission.js', () => {
-  // permission.js has a hard-coded /Users/raymond/... path that must be stripped
-  const idx = buildNpmSource.indexOf("file.dest === 'permission.js'");
-  assert(idx > -1, 'Should find permission.js patching block');
-  const block = buildNpmSource.substring(idx, idx + 500);
-  // The regex pattern uses escaped forward slashes: \/Users\/
-  assert(block.indexOf('Users') > -1, 'Should have regex to strip /Users/ paths');
-});
-
-test('build-npm.js checks sandbox outputs for developer paths', () => {
-  assertIncludes(buildNpmSource, 'devPathPatterns', 'Should define developer path patterns');
-  assertIncludes(buildNpmSource, 'Developer-specific path found', 'Should report developer path errors');
-});
-
-test('build-npm.js treats sandbox obfuscation failure as hard error', () => {
-  assertIncludes(buildNpmSource, 'Failed to obfuscate sandbox/', 'Should error on sandbox obfuscation failure');
-  // yaml.js is the exception
-  assertIncludes(buildNpmSource, "utils/yaml.js", 'Should allow yaml.js exception');
-});
-
-test('build-npm.js verifies sandbox output files for sensitive strings', () => {
-  assertIncludes(buildNpmSource, 'sandboxSensitive', 'Should define sandbox-specific sensitive strings');
-  assertIncludes(buildNpmSource, 'whitelistedSandbox', 'Should whitelist yaml.js from sandbox check');
-  assertIncludes(buildNpmSource, 'Sensitive strings in sandbox/', 'Should report which sandbox file has issues');
-  assertIncludes(buildNpmSource, 'Sandbox files verified', 'Should log success after verification');
-  // Ensure SandboxRunner is checked in sandbox files too (matches the workflow grep)
-  const idx = buildNpmSource.indexOf('sandboxSensitive');
-  assert(idx > -1, 'Should find sandboxSensitive definition');
-  const line = buildNpmSource.substring(idx, idx + 100);
-  assertIncludes(line, 'SandboxRunner', 'sandboxSensitive should include SandboxRunner');
-});
-
-// --- Sandbox files ---
-console.log('\n--- Sandbox files ---\n');
-
-test('build-npm.js copies all 4 sandbox files', () => {
-  assertIncludes(buildNpmSource, "'pave-run.js'", 'Should copy pave-run.js');
-  assertIncludes(buildNpmSource, "'SandboxRunner.js'", 'Should copy SandboxRunner.js');
-  assertIncludes(buildNpmSource, "'permission.js'", 'Should copy permission.js');
-  assertIncludes(buildNpmSource, "'utils/yaml.js'", 'Should copy utils/yaml.js');
-});
-
-test('build-npm.js fixes require paths in sandbox files', () => {
-  // Same path fixing as build.js
-  assertIncludes(buildNpmSource, "require('./permission')", 'Should fix permission require path');
-  assertIncludes(buildNpmSource, "require('./utils/yaml')", 'Should fix yaml require path');
-});
-
-test('build-npm.js creates sandbox utils subdirectory', () => {
-  assertIncludes(buildNpmSource, "'utils'", 'Should create utils subdirectory');
-});
-
-// --- Publish support ---
-console.log('\n--- Publish support ---\n');
-
-test('build-npm.js supports --publish flag', () => {
-  assertIncludes(buildNpmSource, "'--publish'", 'Should support --publish flag');
-  assertIncludes(buildNpmSource, 'npm publish --access public', 'Should use npm publish --access public');
-});
-
-test('build-npm.js supports --dry-run flag', () => {
-  assertIncludes(buildNpmSource, "'--dry-run'", 'Should support --dry-run flag');
-});
-
-test('--dry-run treats npm pack failures as errors', () => {
-  // Should only ignore ENOENT / exit code 127 (command not found)
-  assertIncludes(buildNpmSource, 'ENOENT', 'Should check for ENOENT (command not found)');
-  assertIncludes(buildNpmSource, '127', 'Should check for exit code 127');
-  assertIncludes(buildNpmSource, 'npm pack --dry-run failed', 'Should report pack failures');
-});
-
-test('build-npm.js rejects --publish and --dry-run together', () => {
-  assertIncludes(buildNpmSource, 'hasPublishFlag && hasDryRunFlag', 'Should check for mutual exclusivity');
-  assertIncludes(buildNpmSource, 'cannot be used together', 'Should have clear error message');
-});
-
-// --- package.json integration ---
-console.log('\n--- package.json integration ---\n');
-
-test('package.json has build:npm script', () => {
-  assert(packageJson.scripts['build:npm'], 'Should have build:npm script');
-  assertEqual(packageJson.scripts['build:npm'], 'node build-npm.js');
-});
-
-test('package.json has build:npm:publish script', () => {
-  assert(packageJson.scripts['build:npm:publish'], 'Should have build:npm:publish script');
-  assertEqual(packageJson.scripts['build:npm:publish'], 'node build-npm.js --publish');
-});
-
-// --- GitHub Actions workflow ---
-console.log('\n--- GitHub Actions workflow ---\n');
-
-test('workflow has publish-npm job', () => {
-  assertIncludes(workflowSource, 'publish-npm:', 'Should have publish-npm job');
-});
-
-test('publish-npm job depends on smoke-test', () => {
-  // Find the publish-npm job block (needs may not be on the immediately next line due to if: guard)
-  const idx = workflowSource.indexOf('publish-npm:');
-  assert(idx > -1, 'Should find publish-npm job');
-  const block = workflowSource.substring(idx, idx + 500);
-  assertIncludes(block, 'needs: smoke-test', 'Should depend on smoke-test');
-});
-
-test('publish-npm job uses setup-node with registry-url', () => {
-  const idx = workflowSource.indexOf('publish-npm:');
-  const block = workflowSource.substring(idx);
-  assertIncludes(block, 'registry-url', 'Should set registry-url for npm auth');
-  assertIncludes(block, 'registry.npmjs.org', 'Should use npmjs.org registry');
-});
-
-test('publish-npm job runs build-npm.js', () => {
-  const idx = workflowSource.indexOf('publish-npm:');
-  const block = workflowSource.substring(idx);
-  assertIncludes(block, 'node build-npm.js', 'Should run build-npm.js');
-});
-
-test('publish-npm job uses NPM_TOKEN secret', () => {
-  const idx = workflowSource.indexOf('publish-npm:');
-  const block = workflowSource.substring(idx);
-  assertIncludes(block, 'NPM_TOKEN', 'Should use NPM_TOKEN secret');
-  assertIncludes(block, 'NODE_AUTH_TOKEN', 'Should set NODE_AUTH_TOKEN env var');
-});
-
-test('publish-npm job publishes with --access public', () => {
-  const idx = workflowSource.indexOf('publish-npm:');
-  const block = workflowSource.substring(idx);
-  assertIncludes(block, 'npm publish --access public', 'Should publish with --access public');
-});
-
-test('publish-npm job verifies obfuscation in bundle and sandbox files', () => {
-  const idx = workflowSource.indexOf('publish-npm:');
-  const block = workflowSource.substring(idx);
-  assertIncludes(block, 'Verify obfuscation', 'Should have obfuscation verification step');
-  assertIncludes(block, 'SandboxRunner', 'Should check for SandboxRunner string');
-  assertIncludes(block, 'exit 1', 'Should exit 1 if SandboxRunner found');
-  assertIncludes(block, "find sandbox -name '*.js'", 'Should recursively check sandbox JS files');
-});
-
-test('publish-npm job guards against duplicate publish', () => {
-  const idx = workflowSource.indexOf('publish-npm:');
-  const block = workflowSource.substring(idx, idx + 300);
-  assertIncludes(block, "github.event_name != 'push'", 'Should skip on push events to prevent duplicates');
-});
-
-test('publish-npm job syncs version before build', () => {
-  const idx = workflowSource.indexOf('publish-npm:');
-  const block = workflowSource.substring(idx);
-  assertIncludes(block, 'Sync version', 'Should sync version to package.json');
-});
-
-test('publish-npm job installs all dependencies', () => {
-  const idx = workflowSource.indexOf('publish-npm:');
-  const block = workflowSource.substring(idx);
-  assertIncludes(block, 'opencode-lite', 'Should install opencode-lite deps');
-  assertIncludes(block, 'npm install', 'Should run npm install');
-});
-
-test('publish-npm job checks out correct ref for workflow_dispatch', () => {
-  const idx = workflowSource.indexOf('publish-npm:');
-  const block = workflowSource.substring(idx, idx + 800);
-  assertIncludes(block, 'workflow_dispatch', 'Should reference workflow_dispatch in checkout');
-  assertIncludes(block, 'ref:', 'Should specify a ref for checkout');
-  assertIncludes(block, 'github.event.inputs.version', 'Should use input version for ref');
-});
-
-// --- Consistency with build.js ---
-console.log('\n--- Consistency with build.js ---\n');
-
-test('build-npm.js uses same externals as build.js', () => {
-  assertIncludes(buildNpmSource, "'blessed'", 'Should externalize blessed');
-  assertIncludes(buildNpmSource, "'abort-controller'", 'Should externalize abort-controller');
-});
-
-test('build-npm.js bundles same entry point as build.js', () => {
-  assertIncludes(buildNpmSource, "'index.js'", 'Should bundle index.js');
-});
-
-test('build-npm.js uses PAVE_VERSION define like build.js', () => {
-  assertIncludes(buildNpmSource, 'PAVE_VERSION', 'Should define PAVE_VERSION');
-});
-
-// Results
-console.log('\n' + '='.repeat(60));
-console.log('Results: ' + passed + ' passed, ' + failed + ' failed, ' + total + ' total');
-console.log('='.repeat(60));
-
-if (failed > 0) {
-  throw new Error(failed + ' test(s) failed in build-npm');
-}

package/test/chat-command.test.js

@@ -1,137 +0,0 @@
-#!/usr/bin/env node
-/**
- * Test cases for chat command functionality
- * Tests the parameter passing and integration
- * Run with: node test/chat-command.test.js
- */
-
-const { parseArgs } = require('../lib/args-parser');
-
-function runTest(name, testFn) {
-  try {
-    testFn();
-    console.log(` ${name}`);
-  } catch (error) {
-    console.log(`L ${name}: ${error.message}`);
-    process.exitCode = 1;
-  }
-}
-
-function assertEqual(actual, expected, message) {
-  if (actual !== expected) {
-    throw new Error(`${message}: expected ${expected}, got ${actual}`);
-  }
-}
-
-// Mock the handleChatCommand function to test parameter extraction
-function mockHandleChatCommand(args) {
-  // This simulates the parameter extraction in the actual function
-  const { commandArgs, verbose, json, session, noStream, color, model, showTools } = args;
-
-  return {
-    commandArgs,
-    verbose,
-    json,
-    session,
-    noStream,
-    color,
-    model,
-    showTools,
-  };
-}
-
-// Test parameter extraction for default case (showTools defaults to true)
-runTest('handleChatCommand should extract showTools parameter (default true)', () => {
-  const args = parseArgs(['chat', 'hello world']);
-  const extracted = mockHandleChatCommand(args);
-
-  assertEqual(extracted.showTools, true, 'showTools should default to true');
-  assertEqual(extracted.commandArgs.join(' '), 'hello world', 'commandArgs should be correct');
-});
-
-// Test parameter extraction with --no-tools
-runTest('handleChatCommand should extract showTools parameter (explicit false)', () => {
-  const args = parseArgs(['chat', '--no-tools', 'hello world']);
-  const extracted = mockHandleChatCommand(args);
-
-  assertEqual(extracted.showTools, false, 'showTools should be false with --no-tools');
-  assertEqual(extracted.commandArgs.join(' '), 'hello world', 'commandArgs should exclude flags');
-});
-
-// Test with multiple parameters
-runTest('handleChatCommand should extract all parameters correctly', () => {
-  const args = parseArgs(['chat', '--no-tools', '--verbose', 'test message']);
-  const extracted = mockHandleChatCommand(args);
-
-  assertEqual(extracted.showTools, false, 'showTools should be false');
-  assertEqual(extracted.verbose, true, 'verbose should be true');
-});
-
-// Mock the sendMessageAndStream function to test options passing
-function mockSendMessageAndStream(serverUrl, sessionId, message, options = {}) {
-  // This simulates the options parameter reception
-  return {
-    receivedOptions: options,
-    success: true,
-  };
-}
-
-// Test options passing to sendMessageAndStream (default true)
-runTest('sendMessageAndStream should receive showTools option (default true)', () => {
-  const args = parseArgs(['chat', 'test message']);
-  const extracted = mockHandleChatCommand(args);
-
-  const options = {
-    verbose: extracted.verbose,
-    json: extracted.json,
-    noStream: extracted.noStream,
-    model: extracted.model,
-    showTools: extracted.showTools,
-  };
-
-  const result = mockSendMessageAndStream('http://localhost:4096', 'session123', 'test message', options);
-
-  assertEqual(result.receivedOptions.showTools, true, 'showTools should be passed as true by default');
-});
-
-// Test options passing with --no-tools
-runTest('sendMessageAndStream should receive showTools option (false with --no-tools)', () => {
-  const args = parseArgs(['chat', '--no-tools', 'test message']);
-  const extracted = mockHandleChatCommand(args);
-
-  const options = {
-    verbose: extracted.verbose,
-    json: extracted.json,
-    noStream: extracted.noStream,
-    model: extracted.model,
-    showTools: extracted.showTools,
-  };
-
-  const result = mockSendMessageAndStream('http://localhost:4096', 'session123', 'test message', options);
-
-  assertEqual(result.receivedOptions.showTools, false, 'showTools should be passed as false with --no-tools');
-});
-
-// Test real-world command scenarios
-runTest('Real-world scenario: default file listing (shows tool output)', () => {
-  const args = parseArgs(['chat', 'list', 'all', 'files', 'in', 'current', 'directory']);
-  const extracted = mockHandleChatCommand(args);
-
-  assertEqual(extracted.showTools, true, 'Should show tool output by default');
-  assertEqual(extracted.commandArgs.join(' '), 'list all files in current directory', 'Should preserve full message');
-});
-
-runTest('Real-world scenario: clean summary only (no tool output)', () => {
-  const args = parseArgs(['chat', '--no-tools', 'summarize', 'project', 'structure']);
-  const extracted = mockHandleChatCommand(args);
-
-  assertEqual(extracted.showTools, false, 'Should hide tool output for clean summary');
-  assertEqual(extracted.commandArgs.join(' '), 'summarize project structure', 'Should preserve message without flags');
-});
-
-console.log('\n=� Test Summary:');
-if (process.exitCode === 1) {
-  console.log('Some tests failed. Please fix the issues above.');
-} else {
-  console.log('All tests passed! <�');
-}