@cnrai/pave 0.3.35 → 0.3.51

package/test/run-agent-init.js

@@ -1,297 +0,0 @@
-// Standalone test runner for 'pave agent init' (issue #109)
-// Tests args-parser changes and source code patterns.
-// Designed for Node 16 sandbox compatibility (no module imports that use # fields).
-
-const path = require('path');
-const fs = require('fs');
-
-let passed = 0;
-let failed = 0;
-
-function assert(cond, msg) {
-  if (cond) {
-    passed++;
-    console.log('\u2705 ' + msg);
-  } else {
-    failed++;
-    console.log('\u274C ' + msg);
-  }
-}
-
-function _assertEqual(actual, expected, msg) {
-  if (actual === expected) {
-    passed++;
-    console.log('\u2705 ' + msg);
-  } else {
-    failed++;
-    console.log('\u274C ' + msg + ' (expected ' + JSON.stringify(expected) + ', got ' + JSON.stringify(actual) + ')');
-  }
-}
-
-// =============================================
-// Read source files for pattern verification
-// =============================================
-
-const argsParserPath = path.join(__dirname, '..', 'lib', 'args-parser.js');
-const indexPath = path.join(__dirname, '..', 'index.js');
-const permissionPath = path.join(__dirname, '..', '..', 'opencode-lite', 'src', 'tools', 'permission.js');
-
-const argsParserSrc = fs.readFileSync(argsParserPath, 'utf8');
-const indexSrc = fs.readFileSync(indexPath, 'utf8');
-const permissionSrc = fs.readFileSync(permissionPath, 'utf8');
-
-console.log('Agent init tests (issue #109)');
-console.log('========================================');
-console.log('');
-
-// =============================================
-// Args-parser source verification
-// =============================================
-
-console.log('Args-parser changes:');
-
-assert(argsParserSrc.indexOf('agentSubcommand') !== -1,
-  'args-parser defines agentSubcommand field');
-
-assert(argsParserSrc.indexOf("agentSubcommand: null") !== -1,
-  'agentSubcommand defaults to null');
-
-assert(argsParserSrc.indexOf("a === 'init'") !== -1,
-  'args-parser checks for init token in positional args');
-
-assert(argsParserSrc.indexOf("agentSubcommand = 'init'") !== -1,
-  'args-parser sets agentSubcommand to init');
-
-assert(argsParserSrc.indexOf("a === 'init' && args.agentSubcommand === null && args.commandArgs.length === 0") !== -1,
-  'args-parser detects init as first positional token in main loop');
-
-assert(argsParserSrc.indexOf("agentSubcommand !== 'init'") !== -1,
-  'args-parser skips sleep/SOUL defaults for agent init');
-
-assert(argsParserSrc.indexOf("agentSubcommand !== 'init' && args.commandArgs.length > 1") !== -1,
-  'args-parser skips SOUL validation/warnings for agent init');
-
-assert(argsParserSrc.indexOf('agent init') !== -1,
-  'help text mentions agent init');
-
-console.log('');
-
-// =============================================
-// Permission.js source verification
-// =============================================
-
-console.log('Permission system changes:');
-
-assert(permissionSrc.indexOf('let PAVE_DIR') !== -1,
-  'PAVE_DIR is declared with let (mutable)');
-
-assert(permissionSrc.indexOf('let TOKENS_FILE') !== -1,
-  'TOKENS_FILE is declared with let (mutable)');
-
-assert(permissionSrc.indexOf('let PERMISSION_FILE') !== -1,
-  'PERMISSION_FILE is declared with let (mutable)');
-
-assert(permissionSrc.indexOf('function setConfigDir(dir)') !== -1,
-  'setConfigDir function exists');
-
-assert(permissionSrc.indexOf('function getConfigDir()') !== -1,
-  'getConfigDir function exists');
-
-assert(permissionSrc.indexOf('function createDefaultPermissions()') !== -1,
-  'createDefaultPermissions function exists');
-
-// Verify setConfigDir updates all three paths
-const setConfigDirStart = permissionSrc.indexOf('function setConfigDir(dir)');
-const setConfigDirEnd = permissionSrc.indexOf('\n/**', setConfigDirStart + 1);
-if (setConfigDirStart !== -1 && setConfigDirEnd !== -1) {
-  const body = permissionSrc.substring(setConfigDirStart, setConfigDirEnd);
-  assert(body.indexOf('PAVE_DIR =') !== -1, 'setConfigDir updates PAVE_DIR');
-  assert(body.indexOf('TOKENS_FILE =') !== -1, 'setConfigDir updates TOKENS_FILE');
-  assert(body.indexOf('PERMISSION_FILE =') !== -1, 'setConfigDir updates PERMISSION_FILE');
-  assert(body.indexOf('_permissionsLoaded = false') !== -1, 'setConfigDir resets _permissionsLoaded');
-  assert(body.indexOf('_privateTokenValues.clear()') !== -1, 'setConfigDir clears private token values');
-  assert(body.indexOf('_lastEnvFileMtime = 0') !== -1, 'setConfigDir resets token file mtime');
-  assert(body.indexOf('_lastEnvFilePath = null') !== -1, 'setConfigDir resets token file path');
-  assert(body.indexOf('permissions.network.clear()') !== -1, 'setConfigDir resets network permissions');
-  assert(body.indexOf('permissions.filesystem.read.clear()') !== -1, 'setConfigDir resets filesystem.read permissions');
-  assert(body.indexOf('permissions.filesystem.write.clear()') !== -1, 'setConfigDir resets filesystem.write permissions');
-  assert(body.indexOf('permissions.modules.clear()') !== -1, 'setConfigDir resets modules permissions');
-  assert(body.indexOf('permissions.system.clear()') !== -1, 'setConfigDir resets system permissions');
-  assert(body.indexOf('tokenConfigs.clear()') !== -1, 'setConfigDir resets token configs');
-  assert(body.indexOf('loadPermissions(true)') !== -1, 'setConfigDir reloads permissions from new directory');
-  assert(body.indexOf('_permissionsLoaded = true') !== -1, 'setConfigDir sets _permissionsLoaded to true after reload');
-} else {
-  failed++;
-  console.log('  FAIL: could not find setConfigDir body');
-}
-
-// Verify createDefaultPermissions has correct structure
-assert(permissionSrc.indexOf("version: 4") !== -1 ||
-       permissionSrc.indexOf("version: 4,") !== -1,
-'createDefaultPermissions sets version 4');
-
-assert(permissionSrc.indexOf("createDefaultPermissions") !== -1 &&
-       permissionSrc.substring(permissionSrc.indexOf('module.exports')).indexOf("createDefaultPermissions") !== -1,
-'createDefaultPermissions is exported');
-
-// Verify exports
-const exportsSection = permissionSrc.substring(permissionSrc.indexOf('module.exports'));
-assert(exportsSection.indexOf('setConfigDir') !== -1, 'setConfigDir is in exports');
-assert(exportsSection.indexOf('getConfigDir') !== -1, 'getConfigDir is in exports');
-assert(exportsSection.indexOf('createDefaultPermissions') !== -1, 'createDefaultPermissions is in exports');
-
-console.log('');
-
-// =============================================
-// Index.js source verification
-// =============================================
-
-console.log('Index.js changes:');
-
-assert(indexSrc.indexOf('Permission.setConfigDir(configPath)') !== -1,
-  'index.js calls Permission.setConfigDir in config setup');
-
-assert(indexSrc.indexOf('Permission.createDefaultPermissions()') !== -1,
-  'index.js calls Permission.createDefaultPermissions in config setup');
-
-assert(indexSrc.indexOf('handleAgentInit') !== -1,
-  'handleAgentInit function exists');
-
-assert(indexSrc.indexOf("agentSubcommand === 'init'") !== -1,
-  'agent command checks for init subcommand');
-
-// Verify handleAgentInit function exists and has correct structure
-const initMatch = indexSrc.match(/function handleAgentInit\(args\)\s*\{/);
-assert(initMatch !== null, 'handleAgentInit is defined as a function');
-
-// Verify handleAgentInit calls setConfigDir and createDefaultPermissions
-const initFuncStart = indexSrc.indexOf('function handleAgentInit(args)');
-const initFuncEnd = indexSrc.indexOf('\n/**', initFuncStart + 1);
-if (initFuncStart !== -1 && initFuncEnd !== -1) {
-  const initFuncBody = indexSrc.substring(initFuncStart, initFuncEnd);
-  assert(initFuncBody.indexOf('Permission.setConfigDir') !== -1,
-    'handleAgentInit calls Permission.setConfigDir');
-  assert(initFuncBody.indexOf('Permission.createDefaultPermissions') !== -1 ||
-         initFuncBody.indexOf('createDefaultPermissions') !== -1,
-  'handleAgentInit calls createDefaultPermissions');
-  assert(initFuncBody.indexOf('return 0') !== -1,
-    'handleAgentInit returns 0 on success');
-  assert(initFuncBody.indexOf('return 1') !== -1,
-    'handleAgentInit returns 1 on error');
-  assert(initFuncBody.indexOf('pave agent init') !== -1,
-    'handleAgentInit shows usage error for invalid args');
-  assert(initFuncBody.indexOf('hasCommandArgs') !== -1 || initFuncBody.indexOf('commandArgs') !== -1,
-    'handleAgentInit validates no extra command args');
-  assert(initFuncBody.indexOf('hasSleepFlag') !== -1 || initFuncBody.indexOf('args.sleep') !== -1,
-    'handleAgentInit validates no --sleep flag');
-  assert(initFuncBody.indexOf('hasSoulFlag') !== -1 || initFuncBody.indexOf('args.soul') !== -1,
-    'handleAgentInit validates no --soul flag');
-  assert(initFuncBody.indexOf('hasNameFlag') !== -1 || initFuncBody.indexOf('args.name') !== -1,
-    'handleAgentInit validates no --name flag');
-  assert(initFuncBody.indexOf('hasReinjectIntervalFlag') !== -1 || initFuncBody.indexOf('args.reinjectInterval') !== -1,
-    'handleAgentInit validates no --reinject-interval flag');
-} else {
-  failed++;
-  console.log('  FAIL: could not extract handleAgentInit function body');
-}
-
-// Verify help text includes init
-assert(indexSrc.indexOf('pave agent init') !== -1,
-  'agent help text mentions pave agent init');
-
-console.log('');
-
-// =============================================
-// Verify createDefaultPermissions content
-// =============================================
-
-console.log('Default permissions content:');
-
-// Extract the default permissions object from createDefaultPermissions
-const createFuncStart = permissionSrc.indexOf('function createDefaultPermissions()');
-const createFuncEnd = permissionSrc.indexOf('\n// Default safe', createFuncStart);
-if (createFuncStart !== -1 && createFuncEnd !== -1) {
-  const createFuncBody = permissionSrc.substring(createFuncStart, createFuncEnd);
-
-  assert(createFuncBody.indexOf("network: ['*']") !== -1,
-    'default permissions have network: [*]');
-  assert(createFuncBody.indexOf("read: ['*']") !== -1,
-    'default permissions have filesystem.read: [*]');
-  assert(createFuncBody.indexOf("write: ['*']") !== -1,
-    'default permissions have filesystem.write: [*]');
-  assert(createFuncBody.indexOf("modules: ['*']") !== -1,
-    'default permissions have modules: [*]');
-  assert(createFuncBody.indexOf("system: ['*']") !== -1,
-    'default permissions have system: [*]');
-  assert(createFuncBody.indexOf("allowed: ['*']") !== -1,
-    'default permissions have skills.allowed: [*]');
-  assert(createFuncBody.indexOf("denied: []") !== -1,
-    'default permissions have skills.denied: []');
-  assert(createFuncBody.indexOf("fs.existsSync(PERMISSION_FILE)") !== -1,
-    'createDefaultPermissions checks if file exists before writing');
-  assert(createFuncBody.indexOf("0o600") !== -1,
-    'permissions file is written with restricted mode 0o600');
-  assert(createFuncBody.indexOf("fs.openSync(PERMISSION_FILE, 'wx'") !== -1,
-    'createDefaultPermissions uses exclusive write (wx flag) to prevent TOCTOU race');
-  assert(createFuncBody.indexOf("EEXIST") !== -1,
-    'createDefaultPermissions handles EEXIST from concurrent creation');
-  assert(createFuncBody.indexOf("loadPermissions(true)") !== -1,
-    'createDefaultPermissions reloads permissions after successful file creation');
-} else {
-  failed++;
-  console.log('  FAIL: could not extract createDefaultPermissions function body');
-}
-
-console.log('');
-
-// =============================================
-// Verify auto-bootstrap in config setup
-// =============================================
-
-console.log('Auto-bootstrap in config setup:');
-
-// Find the config setup block in index.js
-const configSetupStart = indexSrc.indexOf("// Set custom config path if provided");
-const configSetupEnd = indexSrc.indexOf("// Handle skill management commands");
-if (configSetupStart !== -1 && configSetupEnd !== -1) {
-  const configSetup = indexSrc.substring(configSetupStart, configSetupEnd);
-
-  const setPaveHomeIdx = configSetup.indexOf('skillManager.setPaveHome(configPath)');
-  const setConfigDirIdx = configSetup.indexOf('Permission.setConfigDir(configPath)');
-  const createDefaultIdx = configSetup.indexOf('Permission.createDefaultPermissions()');
-
-  assert(setPaveHomeIdx !== -1, 'config setup calls skillManager.setPaveHome');
-  assert(setConfigDirIdx !== -1, 'config setup calls Permission.setConfigDir');
-  assert(createDefaultIdx !== -1, 'config setup calls Permission.createDefaultPermissions');
-
-  // Verify order: setConfigDir before createDefaultPermissions
-  if (setConfigDirIdx !== -1 && createDefaultIdx !== -1) {
-    assert(setConfigDirIdx < createDefaultIdx,
-      'setConfigDir is called before createDefaultPermissions');
-  }
-
-  // Verify agent init skip
-  assert(configSetup.indexOf("isAgentInit") !== -1,
-    'config setup skips auto-bootstrap for agent init');
-  assert(configSetup.indexOf("!isAgentInit") !== -1,
-    'auto-bootstrap is conditional on NOT being agent init');
-} else {
-  failed++;
-  console.log('  FAIL: could not find config setup block');
-}
-
-// =============================================
-// Summary
-// =============================================
-
-console.log('');
-console.log('========================================');
-console.log('Results: ' + passed + ' passed, ' + failed + ' failed, ' + (passed + failed) + ' total');
-console.log('');
-
-if (failed > 0) {
-  console.log('Some tests FAILED.');
-  process.exitCode = 1;
-} else {
-  console.log('All tests passed!');
-}

package/test/run-all.js

@@ -1,64 +0,0 @@
-#!/usr/bin/env node
-/**
- * Run all tests for the pave package
- * Usage: node test/run-all.js
- */
-
-const { execSync } = require('child_process');
-const fs = require('fs');
-const path = require('path');
-
-const testDir = __dirname;
-const testFiles = fs.readdirSync(testDir)
-  .filter((f) => f.endsWith('.test.js'))
-  .sort();
-
-console.log('🧪 Running PAVE Test Suite\n');
-
-let totalTests = 0;
-let passedTests = 0;
-let failedTests = 0;
-
-for (const testFile of testFiles) {
-  const testPath = path.join(testDir, testFile);
-  console.log(`📋 Running ${testFile}...`);
-
-  try {
-    const output = execSync(`node "${testPath}"`, {
-      encoding: 'utf8',
-      cwd: path.join(__dirname, '..'),
-    });
-
-    // Count test results from output
-    const lines = output.split('\n');
-    const passed = lines.filter((l) => l.startsWith('✅')).length;
-    const failed = lines.filter((l) => l.startsWith('❌')).length;
-
-    totalTests += passed + failed;
-    passedTests += passed;
-    failedTests += failed;
-
-    if (failed > 0) {
-      console.log(output);
-    } else {
-      console.log(`   ✅ ${passed} tests passed\n`);
-    }
-  } catch (error) {
-    console.log(`   ❌ Test suite failed to run: ${error.message}\n`);
-    failedTests++;
-  }
-}
-
-console.log('=' * 60);
-console.log('📊 Final Test Results:');
-console.log(`   Total:  ${totalTests} tests`);
-console.log(`   Passed: ${passedTests} tests`);
-console.log(`   Failed: ${failedTests} tests`);
-
-if (failedTests > 0) {
-  console.log(`\n❌ ${failedTests} test(s) failed. Please review and fix the issues above.`);
-  process.exit(1);
-} else {
-  console.log(`\n✅ All tests passed! The tool output functionality is working correctly. 🎉`);
-  process.exit(0);
-}

package/test/run-config-flag.js

@@ -1,159 +0,0 @@
-#!/usr/bin/env node
-/**
- * Standalone runner for config-flag behavioral tests (issue #126).
- * Run directly with: node src/packages/pave/test/run-config-flag.js
- *
- * Tests parseArgs() and setPaveHome() behavioral correctness.
- */
-
-let passed = 0;
-let failed = 0;
-
-function assert(cond, msg) {
-  if (cond) {
-    passed++;
-    console.log('\u2705 ' + msg);
-  } else {
-    failed++;
-    console.log('\u274c ' + msg);
-  }
-}
-
-const path = require('path');
-const os = require('os');
-
-const projDir = path.resolve(__dirname, '../../../..');
-const tmpPave = path.join(os.tmpdir(), '.pave');
-const tmpTestCfg = path.join(os.tmpdir(), 'test-pave-cfg');
-
-// ---- 1. parseArgs behavioral tests ----
-
-// pave list --config ./.pave
-const r1 = parseArgs(['list', '--config', './.pave']);
-assert(r1.command === 'list', 'parseArgs: list command detected');
-assert(r1.config === './.pave', 'parseArgs: --config value parsed');
-assert(r1.commandArgs.indexOf('./.pave') === -1, 'parseArgs: ./.pave NOT in commandArgs');
-assert(r1.commandArgs.length === 0, 'parseArgs: commandArgs empty for list --config');
-
-// pave install gmail --config <tmpdir>/.pave
-const r2 = parseArgs(['install', 'gmail', '--config', tmpPave]);
-assert(r2.config === tmpPave, 'parseArgs: --config value parsed for install');
-assert(r2.commandArgs.indexOf(tmpPave) === -1, 'parseArgs: config path NOT in install commandArgs');
-assert(r2.commandArgs.length === 1 && r2.commandArgs[0] === 'gmail',
-  'parseArgs: install commandArgs = [gmail]');
-
-// pave remove my-skill --config ./.pave
-const r3 = parseArgs(['remove', 'my-skill', '--config', './.pave']);
-assert(r3.commandArgs.indexOf('./.pave') === -1, 'parseArgs: ./.pave NOT in remove commandArgs');
-assert(r3.commandArgs[0] === 'my-skill', 'parseArgs: remove commandArgs = [my-skill]');
-
-// pave search query --category tools --config /custom
-const r4 = parseArgs(['search', 'query', '--category', 'tools', '--config', '/custom']);
-assert(r4.commandArgs.indexOf('/custom') === -1, 'parseArgs: /custom NOT in search commandArgs');
-assert(r4.commandArgs.indexOf('tools') === -1, 'parseArgs: --category value NOT in commandArgs');
-assert(r4.commandArgs[0] === 'query', 'parseArgs: search commandArgs = [query]');
-
-// pave update --config ./.pave
-const r5 = parseArgs(['update', '--config', './.pave']);
-assert(r5.commandArgs.length === 0, 'parseArgs: update commandArgs empty');
-
-// --bind value doesn't leak
-const r6 = parseArgs(['list', '--bind', '0.0.0.0', '--config', './.pave']);
-assert(r6.commandArgs.indexOf('0.0.0.0') === -1, 'parseArgs: --bind value NOT in commandArgs');
-
-// -b short form doesn't leak
-const r7 = parseArgs(['list', '-b', '0.0.0.0']);
-assert(r7.commandArgs.indexOf('0.0.0.0') === -1, 'parseArgs: -b value NOT in commandArgs');
-
-// --shell doesn't leak
-const r8 = parseArgs(['list', '--shell', '/bin/bash']);
-assert(r8.commandArgs.indexOf('/bin/bash') === -1, 'parseArgs: --shell value NOT in commandArgs');
-
-// --last doesn't leak
-const r9 = parseArgs(['list', '--last', '5']);
-assert(r9.commandArgs.indexOf('5') === -1, 'parseArgs: --last value NOT in commandArgs');
-
-// --export doesn't leak
-const r10 = parseArgs(['list', '--export', 'out.md']);
-assert(r10.commandArgs.indexOf('out.md') === -1, 'parseArgs: --export value NOT in commandArgs');
-
-// Multiple skills
-const r11 = parseArgs(['install', 'a', 'b', '--config', './.pave']);
-assert(r11.commandArgs.length === 2 && r11.commandArgs[0] === 'a' && r11.commandArgs[1] === 'b',
-  'parseArgs: install a b commandArgs = [a, b]');
-
-// Optional-value flags: --ish-mode and --memory-monitor
-const r13 = parseArgs(['list', '--ish-mode', 'true']);
-assert(r13.commandArgs.indexOf('true') === -1, 'parseArgs: --ish-mode value NOT in commandArgs');
-assert(r13.commandArgs.length === 0, 'parseArgs: list --ish-mode true -> empty commandArgs');
-
-const r14 = parseArgs(['list', '--memory-monitor', 'off']);
-assert(r14.commandArgs.indexOf('off') === -1, 'parseArgs: --memory-monitor value NOT in commandArgs');
-assert(r14.commandArgs.length === 0, 'parseArgs: list --memory-monitor off -> empty commandArgs');
-
-// --ish-mode without a recognized value should not consume next positional token
-const r15 = parseArgs(['install', '--ish-mode', 'gmail']);
-assert(r15.commandArgs.length === 1 && r15.commandArgs[0] === 'gmail',
-  'parseArgs: --ish-mode without value -> gmail stays in commandArgs');
-
-// --config at end (missing value) - must run in subprocess since parseArgs calls process.exit(1)
-const fs = require('fs');
-const child_process = require('child_process');
-const parseArgs = require('../lib/args-parser').parseArgs;
-
-const tmpMissing = path.join(projDir, '_test_tmp_missing_' + Date.now() + '.js');
-const configMissingScript = [
-  'var p = require("./src/packages/pave/lib/args-parser").parseArgs;',
-  'p(["list", "--config"]);',
-].join('\n');
-try {
-  fs.writeFileSync(tmpMissing, configMissingScript, 'utf8');
-  const child = child_process.spawnSync(process.execPath, [tmpMissing], {
-    cwd: projDir, encoding: 'utf8', timeout: 5000, stdio: 'pipe',
-  });
-  assert(child.status !== 0, 'parseArgs: --config at end -> non-zero exit in subprocess');
-  const output = (child.stderr || '') + (child.stdout || '');
-  if (output.length > 0) {
-    assert(output.indexOf('--config') !== -1, 'parseArgs: --config at end -> error mentions --config');
-  }
-} catch (e) {
-  assert(false, 'parseArgs: --config at end -> subprocess test failed: ' + e.message);
-} finally {
-  try { fs.unlinkSync(tmpMissing); } catch (e2) {}
-}
-
-// ---- 2. setPaveHome / getPaths behavioral tests ----
-const sm = require('../lib/skill-manager');
-const mp = require('../lib/marketplace');
-
-const originalHome = sm.getPaveHome();
-
-sm.setPaveHome(tmpTestCfg);
-mp.setPaveHome(tmpTestCfg);
-
-const p = sm.getPaths();
-const c = mp.getCachePaths();
-
-assert(p.skillsDir.indexOf(tmpTestCfg) !== -1, 'setPaveHome: skillsDir uses custom path');
-assert(p.lockFile.indexOf(tmpTestCfg) !== -1, 'setPaveHome: lockFile uses custom path');
-assert(p.permissionsFile.indexOf(tmpTestCfg) !== -1, 'setPaveHome: permissionsFile uses custom path');
-assert(c.cacheDir.indexOf(tmpTestCfg) !== -1, 'setPaveHome: cacheDir uses custom path');
-assert(c.cacheFile.indexOf(tmpTestCfg) !== -1, 'setPaveHome: cacheFile uses custom path');
-
-// Verify deprecated getters exist and delegate to dynamic functions
-assert(typeof sm.SKILLS_DIR === 'string', 'export: SKILLS_DIR is deprecated getter (string)');
-assert(typeof sm.LOCK_FILE === 'string', 'export: LOCK_FILE is deprecated getter (string)');
-assert(typeof mp.CACHE_DIR === 'string', 'export: CACHE_DIR is deprecated getter (string)');
-assert(typeof mp.CACHE_FILE === 'string', 'export: CACHE_FILE is deprecated getter (string)');
-assert(sm.SKILLS_DIR === sm.getPaths().skillsDir, 'export: SKILLS_DIR delegates to getPaths()');
-assert(sm.LOCK_FILE === sm.getPaths().lockFile, 'export: LOCK_FILE delegates to getPaths()');
-assert(mp.CACHE_DIR === mp.getCachePaths().cacheDir, 'export: CACHE_DIR delegates to getCachePaths()');
-assert(mp.CACHE_FILE === mp.getCachePaths().cacheFile, 'export: CACHE_FILE delegates to getCachePaths()');
-
-// Restore
-sm.setPaveHome(originalHome);
-mp.setPaveHome(originalHome);
-
-console.log('');
-console.log(passed + ' passed, ' + failed + ' failed');
-if (failed > 0) { process.exitCode = 1; throw new Error(failed + ' test(s) failed'); }

package/test/run-cursor-drift.js

@@ -1,82 +0,0 @@
-#!/usr/bin/env node
-// Standalone test runner for cursor drift fix (issue #112).
-// Source pattern verification via grep + cursor math tests.
-//
-// Blessed wraps when line.length > width (not >=), so at exact multiples
-// the cursor stays at col=wrapWidth on the same line.
-const child_process = require("child_process");
-const path = require("path");
-
-let passed = 0;
-let failed = 0;
-let skipped = 0;
-function assert(cond, msg) {
-  if (cond) { passed++; console.log("\u2705 " + msg); }
-  else { failed++; console.log("\u274c " + msg); }
-}
-function skip(msg) { skipped++; console.log("SKIP: " + msg); }
-const tuiPath = path.resolve(__dirname, "../../tui/index.js");
-function grepCount(pat, file) {
-  try {
-    const cmd = "grep -F -c " + JSON.stringify(pat) + " " + JSON.stringify(file);
-    return parseInt(child_process.execSync(cmd, { encoding: "utf8" }).trim(), 10);
-  } catch (e) {
-    if (e.stdout) {
-      const n = parseInt(String(e.stdout).trim(), 10);
-      if (!isNaN(n)) return n;
-    }
-    return -1;
-  }
-}
-// Source verification  the fix introduces wrapWidth = widgetWidth - 1
-const c = grepCount("wrapWidth = widgetWidth - 1", tuiPath);
-if (c < 0) { skip("source grep not available in sandbox"); }
-else {
-  assert(c >= 3, "source: 3+ wrapWidth = widgetWidth - 1 locations (found " + c + ")");
-  // Ensure the old pattern (input.iwidth - 1 baked into widgetWidth) is gone
-  const old = grepCount("input.iwidth - 1", tuiPath);
-  assert(old === 0, "source: no old iwidth - 1 in widgetWidth calc (found " + old + ")");
-  // Fallback value should be 80 (widgetWidth) since wrapWidth = 80-1 = 79
-  const fb = grepCount(": 80", tuiPath);
-  assert(fb >= 1, "source: fallback 80 in _typeScroll widgetWidth (found " + fb + ")");
-  // Check for textarea margin comments
-  const cm = grepCount("textarea", tuiPath);
-  assert(cm >= 1, "source: textarea wrap margin comments (found " + cm + ")");
-  // Check for (cursorCol - 1) pattern (blessed wraps on > not >=)
-  const cc = grepCount("cursorCol - 1", tuiPath);
-  assert(cc >= 2, "source: cursorCol - 1 pattern for blessed > boundary (found " + cc + ")");
-  // Check selection: per-char wrap uses >= wrapWidth, after-last-char uses > wrapWidth
-  const gte = grepCount(">= wrapWidth", tuiPath);
-  assert(gte >= 1, "source: >= wrapWidth for per-char wrap (found " + gte + ")");
-  const gt = grepCount("> wrapWidth", tuiPath);
-  assert(gt >= 1, "source: > wrapWidth for after-last-char cursor (found " + gt + ")");
-}
-// Cursor math simulation (model Blessed _wrapContent: no wrap on exact width)
-function sim(len, w) {
-  if (len <= 0) return { wraps: 0, col: 0 };
-  const wraps = Math.floor((len - 1) / w);
-  const col = ((len - 1) % w) + 1;
-  return { wraps, col };
-}
-const W = 39;
-assert(sim(0, W).col === 0, "math: empty -> col 0");
-assert(sim(20, W).col === 20 && sim(20, W).wraps === 0, "math: 20 -> col 20");
-assert(sim(38, W).col === 38 && sim(38, W).wraps === 0, "math: 38 -> col 38 (end of page 1)");
-assert(sim(39, W).col === W && sim(39, W).wraps === 0, "math: 39 -> col 39 (no wrap, matches blessed)");
-assert(sim(40, W).col === 1 && sim(40, W).wraps === 1, "math: 40 -> col 1");
-assert(sim(78, W).col === W && sim(78, W).wraps === 1, "math: 78 -> end of page 2");
-assert(sim(80, W).col === 2 && sim(80, W).wraps === 2, "math: 80 -> col 2");
-// Verify no drift at each page boundary (exact multiples stay at col=W)
-for (let p = 1; p <= 10; p++) { const ch = p * W; const r = sim(ch, W); assert(r.col === W && r.wraps === p - 1, "math: no drift page " + p); }
-// Verify old code drifts but new code does not
-const oldW = 40;
-for (let p = 1; p <= 5; p++) {
-  const ch = p * 39;
-  const oldPos = sim(ch, oldW);
-  const newPos = sim(ch, W);
-  assert(newPos.col === W, "no-drift: new code col=" + W + " at " + ch + " chars");
-  if (p > 1) assert(oldPos.col !== W, "drift: old code col!=" + W + " at " + ch + " chars (col=" + oldPos.col + ")");
-}
-console.log("");
-console.log(passed + " passed, " + failed + " failed" + (skipped ? ", " + skipped + " skipped" : ""));
-if (failed > 0) { process.exitCode = 1; throw new Error(failed + " test(s) failed"); }