oh-my-claude-sisyphus 3.8.15 → 3.9.0

package/templates/hooks/persistent-mode.mjs

@@ -9,16 +9,15 @@
  */
 
 import { existsSync, readFileSync, writeFileSync, readdirSync, mkdirSync } from 'fs';
-import { join } from 'path';
+import { join, dirname } from 'path';
 import { homedir } from 'os';
+import { fileURLToPath } from 'url';
 
-async function readStdin() {
-  const chunks = [];
-  for await (const chunk of process.stdin) {
-    chunks.push(chunk);
-  }
-  return Buffer.concat(chunks).toString('utf-8');
-}
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+
+// Dynamic import for the shared stdin module
+const { readStdin } = await import(join(__dirname, 'lib', 'stdin.mjs'));
 
 function readJsonFile(path) {
   try {
@@ -43,6 +42,31 @@ function writeJsonFile(path, data) {
   }
 }
 
+/**
+ * Staleness threshold for mode states (2 hours in milliseconds).
+ * States older than this are treated as inactive to prevent stale state
+ * from causing the stop hook to malfunction in new sessions.
+ */
+const STALE_STATE_THRESHOLD_MS = 2 * 60 * 60 * 1000; // 2 hours
+
+/**
+ * Check if a state is stale based on its timestamps.
+ * A state is considered stale if it hasn't been updated recently.
+ * We check both `last_checked_at` and `started_at` - using whichever is more recent.
+ */
+function isStaleState(state) {
+  if (!state) return true;
+
+  const lastChecked = state.last_checked_at ? new Date(state.last_checked_at).getTime() : 0;
+  const startedAt = state.started_at ? new Date(state.started_at).getTime() : 0;
+  const mostRecent = Math.max(lastChecked, startedAt);
+
+  if (mostRecent === 0) return true; // No valid timestamps
+
+  const age = Date.now() - mostRecent;
+  return age > STALE_STATE_THRESHOLD_MS;
+}
+
 /**
  * Read state file from local or global location, tracking the source.
  */
@@ -206,34 +230,37 @@ async function main() {
     const totalIncomplete = taskCount + todoCount;
 
     // Priority 1: Ralph Loop (explicit persistence mode)
-    if (ralph.state?.active) {
+    // Skip if state is stale (older than 2 hours) - prevents blocking new sessions
+    if (ralph.state?.active && !isStaleState(ralph.state)) {
       const iteration = ralph.state.iteration || 1;
       const maxIter = ralph.state.max_iterations || 100;
 
       if (iteration < maxIter) {
         ralph.state.iteration = iteration + 1;
+        ralph.state.last_checked_at = new Date().toISOString();
         writeJsonFile(ralph.path, ralph.state);
 
         console.log(JSON.stringify({
           decision: 'block',
-          reason: `[RALPH LOOP - ITERATION ${iteration + 1}/${maxIter}] Work is NOT done. Continue. When complete, output: <promise>${ralph.state.completion_promise || 'DONE'}</promise>\n${ralph.state.prompt ? `Task: ${ralph.state.prompt}` : ''}`
+          reason: `[RALPH LOOP - ITERATION ${iteration + 1}/${maxIter}] Work is NOT done. Continue working.\nWhen FULLY complete (after Architect verification), run /oh-my-claudecode:cancel to cleanly exit ralph mode and clean up all state files. If cancel fails, retry with /oh-my-claudecode:cancel --force.\n${ralph.state.prompt ? `Task: ${ralph.state.prompt}` : ''}`
         }));
         return;
       }
     }
 
     // Priority 2: Autopilot (high-level orchestration)
-    if (autopilot.state?.active) {
+    if (autopilot.state?.active && !isStaleState(autopilot.state)) {
       const phase = autopilot.state.phase || 'unknown';
       if (phase !== 'complete') {
         const newCount = (autopilot.state.reinforcement_count || 0) + 1;
         if (newCount <= 20) {
           autopilot.state.reinforcement_count = newCount;
+          autopilot.state.last_checked_at = new Date().toISOString();
           writeJsonFile(autopilot.path, autopilot.state);
 
           console.log(JSON.stringify({
             decision: 'block',
-            reason: `[AUTOPILOT - Phase: ${phase}] Autopilot not complete. Continue working.`
+            reason: `[AUTOPILOT - Phase: ${phase}] Autopilot not complete. Continue working. When all phases are complete, run /oh-my-claudecode:cancel to cleanly exit and clean up state files. If cancel fails, retry with /oh-my-claudecode:cancel --force.`
           }));
           return;
         }
@@ -241,18 +268,19 @@ async function main() {
     }
 
     // Priority 3: Ultrapilot (parallel autopilot)
-    if (ultrapilot.state?.active) {
+    if (ultrapilot.state?.active && !isStaleState(ultrapilot.state)) {
       const workers = ultrapilot.state.workers || [];
       const incomplete = workers.filter(w => w.status !== 'complete' && w.status !== 'failed').length;
       if (incomplete > 0) {
         const newCount = (ultrapilot.state.reinforcement_count || 0) + 1;
         if (newCount <= 20) {
           ultrapilot.state.reinforcement_count = newCount;
+          ultrapilot.state.last_checked_at = new Date().toISOString();
           writeJsonFile(ultrapilot.path, ultrapilot.state);
 
           console.log(JSON.stringify({
             decision: 'block',
-            reason: `[ULTRAPILOT] ${incomplete} workers still running. Continue.`
+            reason: `[ULTRAPILOT] ${incomplete} workers still running. Continue working. When all workers complete, run /oh-my-claudecode:cancel to cleanly exit and clean up state files. If cancel fails, retry with /oh-my-claudecode:cancel --force.`
           }));
           return;
         }
@@ -260,17 +288,18 @@ async function main() {
     }
 
     // Priority 4: Swarm (coordinated agents with SQLite)
-    if (swarmMarker && swarmSummary?.active) {
+    if (swarmMarker && swarmSummary?.active && !isStaleState(swarmSummary)) {
       const pending = (swarmSummary.tasks_pending || 0) + (swarmSummary.tasks_claimed || 0);
       if (pending > 0) {
         const newCount = (swarmSummary.reinforcement_count || 0) + 1;
         if (newCount <= 15) {
           swarmSummary.reinforcement_count = newCount;
+          swarmSummary.last_checked_at = new Date().toISOString();
           writeJsonFile(join(stateDir, 'swarm-summary.json'), swarmSummary);
 
           console.log(JSON.stringify({
             decision: 'block',
-            reason: `[SWARM ACTIVE] ${pending} tasks remain. Continue working.`
+            reason: `[SWARM ACTIVE] ${pending} tasks remain. Continue working. When all tasks are done, run /oh-my-claudecode:cancel to cleanly exit and clean up state files. If cancel fails, retry with /oh-my-claudecode:cancel --force.`
           }));
           return;
         }
@@ -278,18 +307,19 @@ async function main() {
     }
 
     // Priority 5: Pipeline (sequential stages)
-    if (pipeline.state?.active) {
+    if (pipeline.state?.active && !isStaleState(pipeline.state)) {
       const currentStage = pipeline.state.current_stage || 0;
       const totalStages = pipeline.state.stages?.length || 0;
       if (currentStage < totalStages) {
         const newCount = (pipeline.state.reinforcement_count || 0) + 1;
         if (newCount <= 15) {
           pipeline.state.reinforcement_count = newCount;
+          pipeline.state.last_checked_at = new Date().toISOString();
           writeJsonFile(pipeline.path, pipeline.state);
 
           console.log(JSON.stringify({
             decision: 'block',
-            reason: `[PIPELINE - Stage ${currentStage + 1}/${totalStages}] Pipeline not complete. Continue.`
+            reason: `[PIPELINE - Stage ${currentStage + 1}/${totalStages}] Pipeline not complete. Continue working. When all stages complete, run /oh-my-claudecode:cancel to cleanly exit and clean up state files. If cancel fails, retry with /oh-my-claudecode:cancel --force.`
           }));
           return;
         }
@@ -297,16 +327,17 @@ async function main() {
     }
 
     // Priority 6: UltraQA (QA cycling)
-    if (ultraqa.state?.active) {
+    if (ultraqa.state?.active && !isStaleState(ultraqa.state)) {
       const cycle = ultraqa.state.cycle || 1;
       const maxCycles = ultraqa.state.max_cycles || 10;
       if (cycle < maxCycles && !ultraqa.state.all_passing) {
         ultraqa.state.cycle = cycle + 1;
+        ultraqa.state.last_checked_at = new Date().toISOString();
         writeJsonFile(ultraqa.path, ultraqa.state);
 
         console.log(JSON.stringify({
           decision: 'block',
-          reason: `[ULTRAQA - Cycle ${cycle + 1}/${maxCycles}] Tests not all passing. Continue fixing.`
+          reason: `[ULTRAQA - Cycle ${cycle + 1}/${maxCycles}] Tests not all passing. Continue fixing. When all tests pass, run /oh-my-claudecode:cancel to cleanly exit and clean up state files. If cancel fails, retry with /oh-my-claudecode:cancel --force.`
         }));
         return;
       }
@@ -314,7 +345,7 @@ async function main() {
 
     // Priority 7: Ultrawork - ALWAYS continue while active (not just when tasks exist)
     // This prevents false stops from bash errors, transient failures, etc.
-    if (ultrawork.state?.active) {
+    if (ultrawork.state?.active && !isStaleState(ultrawork.state)) {
       const newCount = (ultrawork.state.reinforcement_count || 0) + 1;
       const maxReinforcements = ultrawork.state.max_reinforcements || 50;
 
@@ -328,24 +359,29 @@ async function main() {
       ultrawork.state.last_checked_at = new Date().toISOString();
       writeJsonFile(ultrawork.path, ultrawork.state);
 
-      let reason = `[ULTRAWORK #${newCount}] Mode active - continue working.`;
+      let reason = `[ULTRAWORK #${newCount}/${maxReinforcements}] Mode active.`;
+
       if (totalIncomplete > 0) {
         const itemType = taskCount > 0 ? 'Tasks' : 'todos';
-        reason = `[ULTRAWORK #${newCount}] ${totalIncomplete} incomplete ${itemType}. Continue working.`;
+        reason += ` ${totalIncomplete} incomplete ${itemType} remain. Continue working.`;
+      } else if (newCount >= 3) {
+        // Only suggest cancel after minimum iterations (guard against no-tasks-created scenario)
+        reason += ` If all work is complete, run /oh-my-claudecode:cancel to cleanly exit ultrawork mode and clean up state files. If cancel fails, retry with /oh-my-claudecode:cancel --force. Otherwise, continue working.`;
+      } else {
+        // Early iterations with no tasks yet - just tell LLM to continue
+        reason += ` Continue working - create Tasks to track your progress.`;
       }
+
       if (ultrawork.state.original_prompt) {
         reason += `\nTask: ${ultrawork.state.original_prompt}`;
       }
 
-      console.log(JSON.stringify({
-        decision: 'block',
-        reason: reason
-      }));
+      console.log(JSON.stringify({ decision: 'block', reason }));
       return;
     }
 
     // Priority 8: Ecomode - ALWAYS continue while active
-    if (ecomode.state?.active) {
+    if (ecomode.state?.active && !isStaleState(ecomode.state)) {
       const newCount = (ecomode.state.reinforcement_count || 0) + 1;
       const maxReinforcements = ecomode.state.max_reinforcements || 50;
 
@@ -356,18 +392,23 @@ async function main() {
       }
 
       ecomode.state.reinforcement_count = newCount;
+      ecomode.state.last_checked_at = new Date().toISOString();
       writeJsonFile(ecomode.path, ecomode.state);
 
-      let reason = `[ECOMODE #${newCount}] Mode active - continue working.`;
+      let reason = `[ECOMODE #${newCount}/${maxReinforcements}] Mode active.`;
+
       if (totalIncomplete > 0) {
         const itemType = taskCount > 0 ? 'Tasks' : 'todos';
-        reason = `[ECOMODE #${newCount}] ${totalIncomplete} incomplete ${itemType}. Continue working.`;
+        reason += ` ${totalIncomplete} incomplete ${itemType} remain. Continue working.`;
+      } else if (newCount >= 3) {
+        // Only suggest cancel after minimum iterations (guard against no-tasks-created scenario)
+        reason += ` If all work is complete, run /oh-my-claudecode:cancel to cleanly exit ecomode and clean up state files. If cancel fails, retry with /oh-my-claudecode:cancel --force. Otherwise, continue working.`;
+      } else {
+        // Early iterations with no tasks yet - just tell LLM to continue
+        reason += ` Continue working - create Tasks to track your progress.`;
       }
 
-      console.log(JSON.stringify({
-        decision: 'block',
-        reason: reason
-      }));
+      console.log(JSON.stringify({ decision: 'block', reason }));
       return;
     }
 

package/templates/hooks/post-tool-use.mjs

@@ -4,7 +4,14 @@
 // Saves to .omc/notepad.md for compaction-resilient memory
 
 import { existsSync, readFileSync, writeFileSync, mkdirSync } from 'fs';
-import { join } from 'path';
+import { join, dirname } from 'path';
+import { fileURLToPath } from 'url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+
+// Dynamic import for the shared stdin module
+const { readStdin } = await import(join(__dirname, 'lib', 'stdin.mjs'));
 
 // Constants
 const NOTEPAD_TEMPLATE = '# Notepad\n' +
@@ -16,15 +23,6 @@ const NOTEPAD_TEMPLATE = '# Notepad\n' +
   '## MANUAL\n' +
   '<!-- User content. Never auto-pruned. -->\n';
 
-// Read all stdin
-async function readStdin() {
-  const chunks = [];
-  for await (const chunk of process.stdin) {
-    chunks.push(chunk);
-  }
-  return Buffer.concat(chunks).toString('utf-8');
-}
-
 // Initialize notepad.md if needed
 function initNotepad(directory) {
   const sisyphusDir = join(directory, '.omc');

package/templates/hooks/pre-tool-use.mjs

@@ -5,6 +5,14 @@
  */
 
 import * as path from 'path';
+import { dirname } from 'path';
+import { fileURLToPath } from 'url';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+
+// Dynamic import for the shared stdin module
+const { readStdin } = await import(path.join(__dirname, 'lib', 'stdin.mjs'));
 
 // Allowed path patterns (no warning)
 const ALLOWED_PATH_PATTERNS = [
@@ -70,12 +78,7 @@ This is a soft warning. Operation will proceed.`;
 }
 
 async function main() {
-  let input = '';
-
-  // Read stdin
-  for await (const chunk of process.stdin) {
-    input += chunk;
-  }
+  const input = await readStdin();
 
   let data;
   try {

package/templates/hooks/session-start.mjs

@@ -4,16 +4,15 @@
 // Cross-platform: Windows, macOS, Linux
 
 import { existsSync, readFileSync, readdirSync, writeFileSync, mkdirSync } from 'fs';
-import { join } from 'path';
+import { join, dirname } from 'path';
 import { homedir } from 'os';
+import { fileURLToPath } from 'url';
 
-async function readStdin() {
-  const chunks = [];
-  for await (const chunk of process.stdin) {
-    chunks.push(chunk);
-  }
-  return Buffer.concat(chunks).toString('utf-8');
-}
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+
+// Dynamic import for the shared stdin module
+const { readStdin } = await import(join(__dirname, 'lib', 'stdin.mjs'));
 
 function readJsonFile(path) {
   try {

package/agents/AGENTS.md

@@ -1,144 +0,0 @@
-<!-- Parent: ../AGENTS.md -->
-<!-- Generated: 2026-01-28 | Updated: 2026-01-28 -->
-
-# agents (Prompt Templates)
-
-Markdown prompt templates for all 32 agents in oh-my-claudecode.
-
-## Purpose
-
-This directory contains the prompt templates that define agent behavior. Each file is a markdown document with YAML frontmatter for metadata, loaded dynamically by `src/agents/definitions.ts`.
-
-## Key Files
-
-| File | Agent | Model | Purpose |
-|------|-------|-------|---------|
-| `architect.md` | architect | opus | Architecture, debugging, root cause analysis |
-| `architect-medium.md` | architect-medium | sonnet | Moderate analysis tasks |
-| `architect-low.md` | architect-low | haiku | Quick code questions |
-| `executor.md` | executor | sonnet | Focused task implementation |
-| `executor-high.md` | executor-high | opus | Complex multi-file changes |
-| `executor-low.md` | executor-low | haiku | Simple single-file tasks |
-| `explore.md` | explore | haiku | Fast codebase search |
-| `explore-medium.md` | explore-medium | sonnet | Thorough search with reasoning |
-| `explore-high.md` | explore-high | opus | Architectural discovery |
-| `designer.md` | designer | sonnet | UI/UX, component design |
-| `designer-high.md` | designer-high | opus | Complex UI architecture |
-| `designer-low.md` | designer-low | haiku | Simple styling tweaks |
-| `researcher.md` | researcher | sonnet | Documentation research |
-| `researcher-low.md` | researcher-low | haiku | Quick doc lookups |
-| `writer.md` | writer | haiku | Technical documentation |
-| `vision.md` | vision | sonnet | Image/screenshot analysis |
-| `critic.md` | critic | opus | Critical plan review |
-| `analyst.md` | analyst | opus | Pre-planning requirements |
-| `planner.md` | planner | opus | Strategic planning |
-| `qa-tester.md` | qa-tester | sonnet | Interactive CLI testing |
-| `qa-tester-high.md` | qa-tester-high | opus | Production-ready QA |
-| `scientist.md` | scientist | sonnet | Data analysis |
-| `scientist-high.md` | scientist-high | opus | Complex ML/research |
-| `scientist-low.md` | scientist-low | haiku | Quick data inspection |
-| `security-reviewer.md` | security-reviewer | opus | Security audits |
-| `security-reviewer-low.md` | security-reviewer-low | haiku | Quick security scans |
-| `build-fixer.md` | build-fixer | sonnet | Build error resolution |
-| `build-fixer-low.md` | build-fixer-low | haiku | Simple type errors |
-| `tdd-guide.md` | tdd-guide | sonnet | TDD workflow |
-| `tdd-guide-low.md` | tdd-guide-low | haiku | Test suggestions |
-| `code-reviewer.md` | code-reviewer | opus | Expert code review |
-| `code-reviewer-low.md` | code-reviewer-low | haiku | Quick code checks |
-
-## For AI Agents
-
-### Working In This Directory
-
-#### Prompt Template Format
-
-Each file follows this structure:
-```markdown
----
-name: agent-name
-description: Brief description of what this agent does
-model: opus | sonnet | haiku
-tools: [Read, Glob, Grep, ...]
----
-
-# Agent Name
-
-## Role
-What this agent is and its expertise.
-
-## Instructions
-Detailed instructions for how the agent should behave.
-
-## Constraints
-What the agent should NOT do.
-
-## Output Format
-How results should be formatted.
-```
-
-#### Creating a New Agent Prompt
-
-1. Create `new-agent.md` with YAML frontmatter
-2. Define clear role, instructions, and constraints
-3. Reference in `src/agents/definitions.ts`
-
-#### Tiered Variants
-
-For model routing, create variants with complexity-appropriate instructions:
-
-| Tier | File Suffix | Instructions Focus |
-|------|-------------|-------------------|
-| LOW (Haiku) | `-low.md` | Quick, simple tasks, minimal reasoning |
-| MEDIUM (Sonnet) | Base file or `-medium.md` | Standard complexity |
-| HIGH (Opus) | `-high.md` | Complex reasoning, deep analysis |
-
-### Common Patterns
-
-**Frontmatter parsing** (in `definitions.ts`):
-```typescript
-function loadAgentPrompt(agentName: string): string {
-  const content = readFileSync(`agents/${agentName}.md`, 'utf-8');
-  const match = content.match(/^---[\s\S]*?---\s*([\s\S]*)$/);
-  return match ? match[1].trim() : content.trim();
-}
-```
-
-**Tool assignment by agent type:**
-- Read-only: `[Read, Glob, Grep]`
-- Analysis: `[Read, Glob, Grep, WebSearch, WebFetch]`
-- Execution: `[Read, Glob, Grep, Edit, Write, Bash, TodoWrite]`
-- Data: `[Read, Glob, Grep, Bash, python_repl]`
-
-### Testing Requirements
-
-Agent prompts are tested via integration tests that spawn agents and verify behavior.
-
-## Dependencies
-
-### Internal
-- Loaded by `src/agents/definitions.ts`
-- Referenced by skill definitions in `skills/`
-
-### External
-None - pure markdown files.
-
-## Agent Categories
-
-| Category | Agents | Common Tools |
-|----------|--------|--------------|
-| Analysis | architect, architect-medium, architect-low | Read, Glob, Grep, lsp_diagnostics |
-| Execution | executor, executor-low, executor-high | Read, Glob, Grep, Edit, Write, Bash, lsp_diagnostics |
-| Search | explore, explore-medium, explore-high | Read, Glob, Grep, ast_grep_search, lsp_document_symbols, lsp_workspace_symbols |
-| Research | researcher, researcher-low | WebSearch, WebFetch |
-| Frontend | designer, designer-low, designer-high | Edit, Write, Bash |
-| Docs | writer | Edit, Write |
-| Visual | vision | Read, Glob, Grep |
-| Planning | planner, analyst, critic | Read, Glob, Grep |
-| Testing | qa-tester, qa-tester-high | Bash, Read, Grep, Glob, TodoWrite, lsp_diagnostics |
-| Security | security-reviewer, security-reviewer-low | Read, Grep, Bash |
-| Build | build-fixer, build-fixer-low | Read, Glob, Grep, Edit, Write, Bash, lsp_diagnostics, lsp_diagnostics_directory |
-| TDD | tdd-guide, tdd-guide-low | Read, Grep, Glob, Bash, lsp_diagnostics |
-| Review | code-reviewer, code-reviewer-low | Read, Grep, Glob, Bash, lsp_diagnostics |
-| Data | scientist, scientist-low, scientist-high | Read, Glob, Grep, Bash, python_repl |
-
-<!-- MANUAL: -->