oh-my-claude-sisyphus 2.4.1 → 2.6.0

package/scripts/test-max-attempts.ts

@@ -0,0 +1,94 @@
+#!/usr/bin/env tsx
+/**
+ * Test script for max-attempts counter in todo-continuation
+ *
+ * Tests the resetTodoContinuationAttempts functionality to verify
+ * that the counter tracking mechanism works correctly.
+ */
+
+import { resetTodoContinuationAttempts, checkPersistentModes } from '../src/hooks/persistent-mode/index.js';
+
+async function runTests() {
+  console.log('Testing max-attempts counter...\n');
+
+  let testsPassed = 0;
+  let testsFailed = 0;
+
+  // Test 1: Basic reset functionality
+  try {
+    console.log('Test 1: Basic reset (should not throw)');
+    resetTodoContinuationAttempts('test-session-1');
+    console.log('✓ PASS: resetTodoContinuationAttempts executed without error\n');
+    testsPassed++;
+  } catch (error) {
+    console.error('✗ FAIL: resetTodoContinuationAttempts threw error:', error);
+    testsFailed++;
+  }
+
+  // Test 2: Multiple resets on same session
+  try {
+    console.log('Test 2: Multiple resets on same session');
+    resetTodoContinuationAttempts('test-session-2');
+    resetTodoContinuationAttempts('test-session-2');
+    resetTodoContinuationAttempts('test-session-2');
+    console.log('✓ PASS: Multiple resets work correctly\n');
+    testsPassed++;
+  } catch (error) {
+    console.error('✗ FAIL: Multiple resets failed:', error);
+    testsFailed++;
+  }
+
+  // Test 3: Reset different sessions
+  try {
+    console.log('Test 3: Reset different sessions');
+    resetTodoContinuationAttempts('session-a');
+    resetTodoContinuationAttempts('session-b');
+    resetTodoContinuationAttempts('session-c');
+    console.log('✓ PASS: Can reset different sessions independently\n');
+    testsPassed++;
+  } catch (error) {
+    console.error('✗ FAIL: Different session resets failed:', error);
+    testsFailed++;
+  }
+
+  // Test 4: Indirect test via checkPersistentModes (no todos should not throw)
+  try {
+    console.log('Test 4: Indirect test via checkPersistentModes');
+    const result = await checkPersistentModes('test-session-indirect');
+    console.log(`✓ PASS: checkPersistentModes executed (shouldBlock=${result.shouldBlock}, mode=${result.mode})\n`);
+    testsPassed++;
+  } catch (error) {
+    console.error('✗ FAIL: checkPersistentModes threw error:', error);
+    testsFailed++;
+  }
+
+  // Test 5: Reset with empty string session ID
+  try {
+    console.log('Test 5: Reset with empty string session ID');
+    resetTodoContinuationAttempts('');
+    console.log('✓ PASS: Empty session ID handled correctly\n');
+    testsPassed++;
+  } catch (error) {
+    console.error('✗ FAIL: Empty session ID failed:', error);
+    testsFailed++;
+  }
+
+  // Summary
+  console.log('═══════════════════════════════════════');
+  console.log('SUMMARY');
+  console.log('═══════════════════════════════════════');
+  console.log(`Total tests: ${testsPassed + testsFailed}`);
+  console.log(`Passed: ${testsPassed}`);
+  console.log(`Failed: ${testsFailed}`);
+  console.log('═══════════════════════════════════════\n');
+
+  if (testsFailed === 0) {
+    console.log('✓ ALL TESTS PASSED');
+    process.exit(0);
+  } else {
+    console.log('✗ SOME TESTS FAILED');
+    process.exit(1);
+  }
+}
+
+runTests();

package/scripts/test-mutual-exclusion.ts

@@ -0,0 +1,152 @@
+#!/usr/bin/env tsx
+
+import { mkdtempSync, rmSync } from 'fs';
+import { join } from 'path';
+import { tmpdir } from 'os';
+import { mkdirSync } from 'fs';
+
+// Import the hooks
+import { startUltraQA, clearUltraQAState, isRalphLoopActive } from '../src/hooks/ultraqa-loop/index.js';
+import { createRalphLoopHook, clearRalphState, isUltraQAActive } from '../src/hooks/ralph-loop/index.js';
+
+// Test utilities
+function printTest(testName: string, passed: boolean) {
+  const status = passed ? '\x1b[32m✓ PASS\x1b[0m' : '\x1b[31m✗ FAIL\x1b[0m';
+  console.log(`${status} - ${testName}`);
+}
+
+async function runTests() {
+  console.log('\n=== Testing Mutual Exclusion Between UltraQA and Ralph Loop ===\n');
+
+  // Create temp directory with .sisyphus subfolder
+  const tempDir = mkdtempSync(join(tmpdir(), 'sisyphus-test-'));
+  const sisyphusDir = join(tempDir, '.sisyphus');
+  mkdirSync(sisyphusDir, { recursive: true });
+
+  console.log(`Using temp directory: ${tempDir}\n`);
+
+  let allTestsPassed = true;
+
+  try {
+    // Test 1: Start Ralph Loop, try to start UltraQA - should fail
+    console.log('Test 1: Ralph Loop blocks UltraQA');
+    console.log('  - Starting Ralph Loop...');
+
+    const ralphHook = createRalphLoopHook(tempDir);
+    const ralphStarted = ralphHook.startLoop(
+      'test-session-1',
+      'test task',
+      { maxIterations: 5, completionPromise: 'TASK_COMPLETE' }
+    );
+
+    if (!ralphStarted) {
+      console.log('    Failed to start Ralph Loop');
+      allTestsPassed = false;
+    }
+
+    console.log('  - Attempting to start UltraQA (should fail)...');
+    const ultraQAResult1 = startUltraQA(
+      tempDir,
+      'all-tests-pass',
+      'test-session-2'
+    );
+
+    if (ultraQAResult1.success) {
+      printTest('Test 1: UltraQA should be blocked by Ralph Loop', false);
+      allTestsPassed = false;
+    } else if (ultraQAResult1.error?.includes('Ralph Loop is active')) {
+      printTest('Test 1: UltraQA correctly blocked by Ralph Loop', true);
+    } else {
+      printTest('Test 1: UltraQA correctly blocked by Ralph Loop', false);
+      console.log(`    Unexpected error: ${ultraQAResult1.error}`);
+      allTestsPassed = false;
+    }
+
+    // Clear Ralph state
+    console.log('  - Clearing Ralph state...\n');
+    clearRalphState(tempDir);
+
+    // Test 2: Start UltraQA, try to start Ralph Loop - should fail
+    console.log('Test 2: UltraQA blocks Ralph Loop');
+    console.log('  - Starting UltraQA...');
+
+    const ultraQAResult2 = startUltraQA(
+      tempDir,
+      'all-tests-pass',
+      'test-session-3'
+    );
+
+    if (!ultraQAResult2.success) {
+      console.log(`    Failed to start UltraQA: ${ultraQAResult2.error}`);
+      allTestsPassed = false;
+    }
+
+    console.log('  - Attempting to start Ralph Loop (should fail)...');
+    const ralphHook2 = createRalphLoopHook(tempDir);
+    const ralphStarted2 = ralphHook2.startLoop(
+      'test-session-4',
+      'test task',
+      { maxIterations: 5, completionPromise: 'TASK_COMPLETE' }
+    );
+
+    if (ralphStarted2) {
+      printTest('Test 2: Ralph Loop should be blocked by UltraQA', false);
+      allTestsPassed = false;
+    } else {
+      // Check if it was blocked due to UltraQA
+      if (isUltraQAActive(tempDir)) {
+        printTest('Test 2: Ralph Loop correctly blocked by UltraQA', true);
+      } else {
+        printTest('Test 2: Ralph Loop correctly blocked by UltraQA', false);
+        console.log(`    Ralph Loop failed but UltraQA is not active`);
+        allTestsPassed = false;
+      }
+    }
+
+    // Clear UltraQA state
+    console.log('  - Clearing UltraQA state...\n');
+    clearUltraQAState(tempDir);
+
+    // Test 3: Start UltraQA without any blockers - should succeed
+    console.log('Test 3: UltraQA starts without blockers');
+    console.log('  - Attempting to start UltraQA (should succeed)...');
+    const ultraQAResult3 = startUltraQA(
+      tempDir,
+      'all-tests-pass',
+      'test-session-5'
+    );
+
+    if (ultraQAResult3.success) {
+      printTest('Test 3: UltraQA starts successfully without blockers', true);
+    } else {
+      printTest('Test 3: UltraQA should start without blockers', false);
+      console.log(`    Unexpected error: ${ultraQAResult3.error}`);
+      allTestsPassed = false;
+    }
+
+    // Final cleanup
+    console.log('  - Clearing UltraQA state...\n');
+    clearUltraQAState(tempDir);
+
+  } finally {
+    // Clean up temp directory
+    console.log(`Cleaning up temp directory: ${tempDir}`);
+    rmSync(tempDir, { recursive: true, force: true });
+  }
+
+  // Summary
+  console.log('\n=== Test Summary ===');
+  if (allTestsPassed) {
+    console.log('\x1b[32m✓ All tests passed!\x1b[0m\n');
+    process.exit(0);
+  } else {
+    console.log('\x1b[31m✗ Some tests failed\x1b[0m\n');
+    process.exit(1);
+  }
+}
+
+// Run tests
+runTests().catch(error => {
+  console.error('\x1b[31mTest execution failed:\x1b[0m', error);
+  process.exit(1);
+});