@cnrai/pave 0.3.35 → 0.3.51

package/test/agent-init.test.js

@@ -1,264 +0,0 @@
-#!/usr/bin/env node
-/**
- * Standalone test runner for 'pave agent init'
- * Designed to work in Node 16 sandbox (no private class fields)
- *
- * Always runs args-parser tests for 'agent init'. When the Permission
- * module can be imported, also runs Permission-related tests; full
- * Permission system coverage is exercised in CI where Node supports # fields.
- *
- * Uses checkmark/X output format (same as other .test.js files) so run-all.js
- * correctly counts individual test results. Also throws on failure so
- * run-tests.js (require-based runner) reliably catches regressions.
- */
-
-let passCount = 0;
-let failCount = 0;
-let skipCount = 0;
-
-function runTest(name, testFn) {
-  try {
-    testFn();
-    console.log('\u2705 ' + name);
-    passCount++;
-  } catch (error) {
-    if (error.message === 'SKIP') {
-      console.log('SKIP: ' + name);
-      skipCount++;
-    } else {
-      console.log('\u274C ' + name + ': ' + error.message);
-      failCount++;
-      process.exitCode = 1;
-    }
-  }
-}
-
-function assertEqual(actual, expected, message) {
-  if (actual !== expected) {
-    throw new Error(message + ': expected ' + JSON.stringify(expected) + ', got ' + JSON.stringify(actual));
-  }
-}
-
-// Import parseArgs via absolute path to bypass any test runner mocks
-const parseArgs = require(require('path').resolve(__dirname, '..', 'lib', 'args-parser.js')).parseArgs;
-
-// =============================================
-// Args-parser tests for 'agent init'
-// =============================================
-
-runTest('agent init sets agentSubcommand to init', () => {
-  const args = parseArgs(['agent', 'init']);
-  assertEqual(args.command, 'agent', 'command');
-  assertEqual(args.agentSubcommand, 'init', 'agentSubcommand');
-});
-
-runTest('agent init does not set SOUL defaults', () => {
-  const args = parseArgs(['agent', 'init']);
-  assertEqual(args.sleep, null, 'sleep should be null');
-  assertEqual(args.commandArgs.length, 0, 'commandArgs should be empty');
-});
-
-runTest('agent init still applies config default', () => {
-  const args = parseArgs(['agent', 'init']);
-  assertEqual(args.config, './.pave', 'config should default to ./.pave');
-});
-
-runTest('agent init with --config uses custom path', () => {
-  const args = parseArgs(['agent', 'init', '--config', '/tmp/my-pave']);
-  assertEqual(args.command, 'agent', 'command');
-  assertEqual(args.agentSubcommand, 'init', 'agentSubcommand');
-  assertEqual(args.config, '/tmp/my-pave', 'config');
-});
-
-runTest('agent init does not treat init as a SOUL path', () => {
-  const args = parseArgs(['agent', 'init']);
-  assertEqual(args.commandArgs.length, 0, 'init should not be in commandArgs');
-});
-
-runTest('agent without init does not set agentSubcommand', () => {
-  const args = parseArgs(['agent']);
-  assertEqual(args.agentSubcommand, null, 'agentSubcommand should be null');
-  assertEqual(args.command, 'agent', 'command');
-  // Normal agent should still get defaults
-  assertEqual(args.sleep, '1m', 'sleep should default to 1m');
-  assertEqual(args.commandArgs[0], 'AGENTS.md', 'SOUL should default to AGENTS.md');
-});
-
-runTest('agent with SOUL file does not set agentSubcommand', () => {
-  const args = parseArgs(['agent', 'SOUL.md']);
-  assertEqual(args.agentSubcommand, null, 'agentSubcommand should be null');
-  assertEqual(args.commandArgs[0], 'SOUL.md', 'SOUL.md should be in commandArgs');
-});
-
-runTest('agent init --help sets help flag', () => {
-  const args = parseArgs(['agent', 'init', '--help']);
-  assertEqual(args.command, 'agent', 'command');
-  assertEqual(args.help, true, 'help should be true');
-  assertEqual(args.agentSubcommand, 'init', 'agentSubcommand should still be init when using --help');
-  // When --help is set, SOUL-related defaults are not applied, but agentSubcommand
-  // detection has already run during the initial positional-argument scan.
-});
-
-runTest('agent init with --config before init works', () => {
-  // '--config path init' - config flag with value then init; parser skips the
-  // config value so 'init' is still detected as the agentSubcommand.
-  const args = parseArgs(['agent', '--config', '/tmp/test', 'init']);
-  assertEqual(args.command, 'agent', 'command');
-  assertEqual(args.config, '/tmp/test', 'config');
-  assertEqual(args.agentSubcommand, 'init', 'agentSubcommand should be init');
-  assertEqual(args.commandArgs.length, 0, 'init should not appear in commandArgs');
-});
-
-runTest('agent init with --verbose passes through', () => {
-  const args = parseArgs(['agent', 'init', '--verbose']);
-  assertEqual(args.command, 'agent', 'command');
-  assertEqual(args.agentSubcommand, 'init', 'agentSubcommand');
-  assertEqual(args.verbose, true, 'verbose');
-});
-
-runTest('agent init with --debug passes through', () => {
-  const args = parseArgs(['agent', 'init', '--debug']);
-  assertEqual(args.command, 'agent', 'command');
-  assertEqual(args.agentSubcommand, 'init', 'agentSubcommand');
-  assertEqual(args.debug, true, 'debug');
-});
-
-runTest('normal agent command with --config still gets defaults', () => {
-  const args = parseArgs(['agent', '--config', '/tmp/test']);
-  assertEqual(args.command, 'agent', 'command');
-  assertEqual(args.agentSubcommand, null, 'agentSubcommand should be null');
-  assertEqual(args.config, '/tmp/test', 'config');
-  assertEqual(args.sleep, '1m', 'sleep should default to 1m');
-  assertEqual(args.commandArgs[0], 'AGENTS.md', 'SOUL should default to AGENTS.md');
-});
-
-runTest('agent --help does not set agentSubcommand', () => {
-  const args = parseArgs(['agent', '--help']);
-  assertEqual(args.command, 'agent', 'command');
-  assertEqual(args.help, true, 'help');
-  // Help skips defaults, so agentSubcommand stays null
-  assertEqual(args.agentSubcommand, null, 'agentSubcommand should be null');
-});
-
-// =============================================
-// Permission system tests (skip in sandbox)
-// =============================================
-
-runTest('Permission.setConfigDir updates paths', () => {
-  let Permission;
-  try {
-    Permission = require('../../opencode-lite/src/tools/permission');
-  } catch (e) {
-    if (e instanceof SyntaxError || (e && typeof e.message === 'string' && /private field|class fields|Unexpected token #|not followed by identifier/i.test(e.message))) {
-      throw new Error('SKIP');
-    }
-    throw e;
-  }
-  const os = require('os');
-  const path = require('path');
-  const testDir = path.join(os.tmpdir(), 'pave-test-config-' + Date.now());
-
-  try {
-    Permission.setConfigDir(testDir);
-    const paths = Permission.getConfigPaths();
-    assertEqual(paths.paveDir, testDir, 'paveDir');
-    assertEqual(paths.permissionsFile, path.join(testDir, 'permissions.yaml'), 'permissionsFile');
-    assertEqual(Permission.getConfigDir(), testDir, 'getConfigDir');
-  } finally {
-    Permission.setConfigDir(path.join(os.homedir(), '.pave'));
-  }
-});
-
-runTest('Permission.createDefaultPermissions creates file', () => {
-  let Permission;
-  try {
-    Permission = require('../../opencode-lite/src/tools/permission');
-  } catch (e) {
-    if (e instanceof SyntaxError || (e && typeof e.message === 'string' && /private field|class fields|Unexpected token #|not followed by identifier/i.test(e.message))) {
-      throw new Error('SKIP');
-    }
-    throw e;
-  }
-  const fs = require('fs');
-  const os = require('os');
-  const path = require('path');
-  const testDir = path.join(os.tmpdir(), 'pave-test-create-' + Date.now());
-
-  try {
-    Permission.setConfigDir(testDir);
-    const result = Permission.createDefaultPermissions();
-    assertEqual(result, true, 'should return true');
-
-    const permFile = path.join(testDir, 'permissions.yaml');
-    assertEqual(fs.existsSync(permFile), true, 'file should exist');
-
-    const content = fs.readFileSync(permFile, 'utf8');
-    assertEqual(content.indexOf('version: 4') !== -1, true, 'should contain version 4');
-    assertEqual(content.indexOf("- '*'") !== -1, true, 'should contain wildcard');
-
-    // Verify in-memory permissions reflect the newly created defaults
-    // createDefaultPermissions() now calls loadPermissions(true) internally,
-    // so permission checks should pass immediately without a separate reload
-    assertEqual(
-      Permission.checkPermission('network', 'example.com'),
-      true,
-      'network permission should be granted by default permissions',
-    );
-  } finally {
-    // Cleanup
-    try { fs.unlinkSync(path.join(testDir, 'permissions.yaml')); } catch (e) {}
-    try { fs.rmdirSync(testDir); } catch (e) {}
-    Permission.setConfigDir(path.join(os.homedir(), '.pave'));
-  }
-});
-
-runTest('Permission.createDefaultPermissions does not overwrite', () => {
-  let Permission;
-  try {
-    Permission = require('../../opencode-lite/src/tools/permission');
-  } catch (e) {
-    if (e instanceof SyntaxError || (e && typeof e.message === 'string' && /private field|class fields|Unexpected token #|not followed by identifier/i.test(e.message))) {
-      throw new Error('SKIP');
-    }
-    throw e;
-  }
-  const fs = require('fs');
-  const os = require('os');
-  const path = require('path');
-  const testDir = path.join(os.tmpdir(), 'pave-test-nooverwrite-' + Date.now());
-  fs.mkdirSync(testDir, { recursive: true });
-
-  const permFile = path.join(testDir, 'permissions.yaml');
-  try {
-    fs.writeFileSync(permFile, 'custom: content\n');
-
-    Permission.setConfigDir(testDir);
-    const result = Permission.createDefaultPermissions();
-    assertEqual(result, false, 'should return false');
-
-    const content = fs.readFileSync(permFile, 'utf8');
-    assertEqual(content, 'custom: content\n', 'file should not be overwritten');
-  } finally {
-    // Cleanup
-    try { fs.unlinkSync(permFile); } catch (e) {}
-    try { fs.rmdirSync(testDir); } catch (e) {}
-    Permission.setConfigDir(path.join(os.homedir(), '.pave'));
-  }
-});
-
-// =============================================
-// Summary
-// =============================================
-
-console.log('\n--- Agent Init Test Summary ---');
-console.log('Passed: ' + passCount);
-console.log('Failed: ' + failCount);
-console.log('Skipped: ' + skipCount);
-console.log('Total:  ' + (passCount + failCount + skipCount));
-
-if (failCount > 0) {
-  console.log('\nSome tests FAILED.');
-  throw new Error('Some tests FAILED.');
-} else {
-  console.log('\nAll runnable tests passed!');
-}

package/test/agent-interrupt.test.js

@@ -1,314 +0,0 @@
-#!/usr/bin/env node
-/**
- * Tests for agent interrupt protocol (Issue #201)
- * Phase 4: Urgent interrupt via --urgent flag + sleep wake-up
- */
-
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-
-// Determine registry path
-const registryPath = path.join(__dirname, '..', 'lib', 'agent-registry.js');
-const parserPath = path.join(__dirname, '..', 'lib', 'args-parser.js');
-const registry = require(registryPath);
-const parseArgs = require(parserPath).parseArgs;
-
-// Use a unique test directory to avoid interfering with real agents
-const testDir = path.join(os.tmpdir(), 'pave-test-interrupt-' + process.pid);
-
-let passed = 0;
-let failed = 0;
-
-function assert(condition, msg) {
-  if (!condition) throw new Error('Assertion failed: ' + msg);
-}
-
-function runTest(name, fn) {
-  try {
-    fn();
-    console.log('[PASS] ' + name);
-    passed++;
-  } catch (e) {
-    console.log('[FAIL] ' + name + ': ' + e.message);
-    failed++;
-  }
-}
-
-function resetTestDir() {
-  try { fs.rmSync(testDir, { recursive: true }); } catch (e) {}
-  registry.setAgentsDir(testDir);
-}
-
-function cleanup() {
-  try { fs.rmSync(testDir, { recursive: true }); } catch (e) {}
-  registry.resetAgentsDir();
-}
-
-// ============================================================
-// Registry: writeInterrupt / readInterrupt / hasInterrupt / clearInterrupt
-// ============================================================
-
-resetTestDir();
-
-runTest('writeInterrupt: returns success for valid message', () => {
-  resetTestDir();
-  const result = registry.writeInterrupt('test-agent', 'stop everything');
-  assert(result.success === true, 'should succeed');
-  assert(result.error === null, 'should have no error');
-});
-
-runTest('writeInterrupt: creates interrupt.json file', () => {
-  resetTestDir();
-  registry.writeInterrupt('test-agent', 'urgent message');
-  const interruptPath = registry.getInterruptPath('test-agent');
-  assert(fs.existsSync(interruptPath), 'interrupt.json should exist');
-  const content = JSON.parse(fs.readFileSync(interruptPath, 'utf8'));
-  assert(content.message === 'urgent message', 'message should match');
-  assert(content.action === 'abort', 'default action should be abort');
-  assert(content.from === 'tui', 'default from should be tui');
-  assert(typeof content.timestamp === 'number', 'timestamp should be a number');
-});
-
-runTest('writeInterrupt: rejects empty message', () => {
-  resetTestDir();
-  const result = registry.writeInterrupt('test-agent', '');
-  assert(result.success === false, 'should fail');
-  assert(result.error.indexOf('non-empty') !== -1, 'error should mention non-empty');
-});
-
-runTest('writeInterrupt: rejects null message', () => {
-  resetTestDir();
-  const result = registry.writeInterrupt('test-agent', null);
-  assert(result.success === false, 'should fail');
-});
-
-runTest('writeInterrupt: accepts custom from and action', () => {
-  resetTestDir();
-  registry.writeInterrupt('test-agent', 'msg', { from: 'api', action: 'abort' });
-  const content = JSON.parse(fs.readFileSync(registry.getInterruptPath('test-agent'), 'utf8'));
-  assert(content.from === 'api', 'from should be api');
-  assert(content.action === 'abort', 'action should be abort');
-});
-
-runTest('writeInterrupt: latest write wins (overwrite)', () => {
-  resetTestDir();
-  registry.writeInterrupt('test-agent', 'first message');
-  registry.writeInterrupt('test-agent', 'second message');
-  const content = JSON.parse(fs.readFileSync(registry.getInterruptPath('test-agent'), 'utf8'));
-  assert(content.message === 'second message', 'should be overwritten with latest');
-});
-
-runTest('hasInterrupt: returns false when no interrupt', () => {
-  resetTestDir();
-  assert(registry.hasInterrupt('nonexistent') === false, 'should return false');
-});
-
-runTest('hasInterrupt: returns true when interrupt exists', () => {
-  resetTestDir();
-  registry.writeInterrupt('test-agent', 'msg');
-  assert(registry.hasInterrupt('test-agent') === true, 'should return true');
-});
-
-runTest('readInterrupt: returns envelope when interrupt exists', () => {
-  resetTestDir();
-  registry.writeInterrupt('test-agent', 'check production');
-  const result = registry.readInterrupt('test-agent');
-  assert(result !== null, 'should not be null');
-  assert(result.message === 'check production', 'message should match');
-  assert(result.action === 'abort', 'action should be abort');
-  assert(result.from === 'tui', 'from should be tui');
-});
-
-runTest('readInterrupt: returns null when no interrupt', () => {
-  resetTestDir();
-  const result = registry.readInterrupt('nonexistent');
-  assert(result === null, 'should return null');
-});
-
-runTest('readInterrupt: returns null for malformed JSON', () => {
-  resetTestDir();
-  const dir = registry.getAgentDir('test-agent');
-  fs.mkdirSync(dir, { recursive: true });
-  fs.writeFileSync(path.join(dir, 'interrupt.json'), 'not json');
-  const result = registry.readInterrupt('test-agent');
-  assert(result === null, 'should return null for malformed JSON');
-});
-
-runTest('readInterrupt: returns null for missing message field', () => {
-  resetTestDir();
-  const dir = registry.getAgentDir('test-agent');
-  fs.mkdirSync(dir, { recursive: true });
-  fs.writeFileSync(path.join(dir, 'interrupt.json'), JSON.stringify({ action: 'abort' }));
-  const result = registry.readInterrupt('test-agent');
-  assert(result === null, 'should return null when message field missing');
-});
-
-runTest('clearInterrupt: removes the interrupt file', () => {
-  resetTestDir();
-  registry.writeInterrupt('test-agent', 'msg');
-  assert(registry.hasInterrupt('test-agent') === true, 'should exist before clear');
-  registry.clearInterrupt('test-agent');
-  assert(registry.hasInterrupt('test-agent') === false, 'should not exist after clear');
-});
-
-runTest('clearInterrupt: no error when no interrupt exists', () => {
-  resetTestDir();
-  // Should not throw
-  registry.clearInterrupt('nonexistent');
-});
-
-runTest('getInterruptPath: returns expected path', () => {
-  resetTestDir();
-  const p = registry.getInterruptPath('my-agent');
-  assert(p.indexOf('my-agent') !== -1, 'should contain agent name');
-  assert(p.endsWith('interrupt.json'), 'should end with interrupt.json');
-});
-
-// ============================================================
-// Args parser: --urgent flag
-// ============================================================
-
-runTest('args-parser: --urgent sets urgent=true', () => {
-  const result = parseArgs(['chat', '-a', 'my-agent', '--urgent', 'stop everything']);
-  assert(result.urgent === true, 'urgent should be true');
-  assert(result.agent === 'my-agent', 'agent should be set');
-});
-
-runTest('args-parser: urgent defaults to false', () => {
-  const result = parseArgs(['chat', '-a', 'my-agent', 'hello']);
-  assert(result.urgent === false, 'urgent should default to false');
-});
-
-runTest('args-parser: --urgent works with --agent long form', () => {
-  const result = parseArgs(['chat', '--agent', 'test', '--urgent', 'message']);
-  assert(result.urgent === true, 'urgent should be true');
-  assert(result.agent === 'test', 'agent should be test');
-});
-
-runTest('args-parser: --urgent without -a still parses', () => {
-  const result = parseArgs(['chat', '--urgent', 'message']);
-  assert(result.urgent === true, 'urgent should be true');
-  assert(result.agent === null, 'agent should be null');
-});
-
-// ============================================================
-// Integration: interrupt + inbox interaction
-// ============================================================
-
-runTest('interrupt does not affect inbox', () => {
-  resetTestDir();
-  // Write both interrupt and inbox message
-  registry.writeInterrupt('test-agent', 'urgent msg');
-  registry.writeInboxMessage('test-agent', 'normal msg');
-
-  // Both should exist
-  assert(registry.hasInterrupt('test-agent') === true, 'interrupt should exist');
-  assert(registry.inboxHasMessages('test-agent') === true, 'inbox should have messages');
-
-  // Clear interrupt shouldn't affect inbox
-  registry.clearInterrupt('test-agent');
-  assert(registry.hasInterrupt('test-agent') === false, 'interrupt should be cleared');
-  assert(registry.inboxHasMessages('test-agent') === true, 'inbox should still have messages');
-});
-
-runTest('multiple interrupts: only last survives', () => {
-  resetTestDir();
-  registry.writeInterrupt('test-agent', 'first');
-  registry.writeInterrupt('test-agent', 'second');
-  registry.writeInterrupt('test-agent', 'third');
-
-  const result = registry.readInterrupt('test-agent');
-  assert(result.message === 'third', 'should be the last write');
-});
-
-// ============================================================
-// signalAwareSleep: verify polling behavior via source inspection
-// ============================================================
-
-runTest('signalAwareSleep uses setInterval not setTimeout', () => {
-  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
-  // Find signalAwareSleep function
-  const funcStart = source.indexOf('function signalAwareSleep');
-  assert(funcStart !== -1, 'should find signalAwareSleep');
-  const funcEnd = source.indexOf('\n}', funcStart + 100);
-  const funcBody = source.substring(funcStart, funcEnd + 2);
-
-  assert(funcBody.indexOf('setInterval') !== -1, 'should use setInterval for polling');
-  assert(funcBody.indexOf('POLL_INTERVAL') !== -1, 'should define POLL_INTERVAL');
-  // Should NOT use setTimeout as main timer
-  // (clearInterval is acceptable)
-  assert(funcBody.indexOf('setTimeout') === -1, 'should not use setTimeout');
-});
-
-runTest('sleep calls include interrupt check in condition', () => {
-  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
-  // Count lines that have both signalAwareSleep and hasInterrupt
-  const lines = source.split('\n');
-  let count = 0;
-  for (let i = 0; i < lines.length; i++) {
-    if (lines[i].indexOf('signalAwareSleep') !== -1 && lines[i].indexOf('hasInterrupt') !== -1) {
-      count++;
-    }
-  }
-  assert(count >= 2, 'at least 2 sleep calls should check hasInterrupt, found ' + count);
-});
-
-// ============================================================
-// Agent loop: interrupt check placement
-// ============================================================
-
-runTest('agent loop checks interrupt before inbox/SOUL', () => {
-  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
-  const interruptCheck = source.indexOf('hasInterrupt(agentName)');
-  const inboxCheck = source.indexOf('inboxHasMessages(agentName)');
-  assert(interruptCheck !== -1, 'should check hasInterrupt in agent loop');
-  assert(inboxCheck !== -1, 'should check inboxHasMessages in agent loop');
-  assert(interruptCheck < inboxCheck, 'interrupt check should come before inbox check');
-});
-
-runTest('agent loop formats interrupt with URGENT prefix', () => {
-  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
-  assert(source.indexOf('URGENT INTERRUPT') !== -1, 'should format interrupt with URGENT prefix');
-});
-
-runTest('agent loop clears interrupt after reading', () => {
-  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
-  assert(source.indexOf('clearInterrupt(agentName)') !== -1, 'should call clearInterrupt');
-});
-
-// ============================================================
-// handleChatToAgent: --urgent routing
-// ============================================================
-
-runTest('handleChatToAgent routes --urgent to writeInterrupt (source check)', () => {
-  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
-  const funcStart = source.indexOf('async function handleChatToAgent');
-  assert(funcStart !== -1, 'should find handleChatToAgent');
-  let funcEnd = source.indexOf('\nasync function', funcStart + 50);
-  if (funcEnd === -1) funcEnd = source.length;
-  const funcBody = source.substring(funcStart, funcEnd);
-
-  assert(funcBody.indexOf('writeInterrupt') !== -1, 'should call writeInterrupt for urgent');
-  assert(funcBody.indexOf('writeInboxMessage') !== -1, 'should still call writeInboxMessage for non-urgent');
-});
-
-runTest('handleChatToAgent errors on --urgent without --agent (source check)', () => {
-  const source = fs.readFileSync(path.join(__dirname, '..', 'index.js'), 'utf8');
-  const funcStart = source.indexOf('async function handleChatToAgent');
-  let funcEnd = source.indexOf('\nasync function', funcStart + 50);
-  if (funcEnd === -1) funcEnd = source.length;
-  const funcBody = source.substring(funcStart, funcEnd);
-
-  assert(funcBody.indexOf('Cannot send urgent interrupt without specifying an agent') !== -1,
-    'should have error message for --urgent without --agent');
-});
-
-// ============================================================
-// Cleanup & Summary
-// ============================================================
-
-cleanup();
-
-console.log('\nTotal: ' + (passed + failed) + ', Passed: ' + passed + ', Failed: ' + failed);
-if (failed > 0) process.exitCode = 1;