@mindfoldhq/trellis 0.4.0-beta.9 → 0.4.0-rc.0

package/dist/templates/claude/hooks/ralph-loop.py

@@ -265,17 +265,18 @@ def main():
         sys.exit(0)
 
     # Get subagent info
-    subagent_type = input_data.get("subagent_type", "")
-    agent_output = input_data.get("agent_output", "")
-    original_prompt = input_data.get("prompt", "")
+    # Field names per Claude Code SubagentStop event schema:
+    #   agent_type, last_assistant_message, agent_id, agent_transcript_path, cwd
+    # The event does NOT carry a `prompt` field, so finish-phase detection
+    # based on a `[finish]` marker in the user prompt is no longer possible
+    # here; finish-phase skip logic should be reintroduced via task.json
+    # state (e.g. current_phase) in a follow-up.
+    agent_type = input_data.get("agent_type", "")
+    last_assistant_message = input_data.get("last_assistant_message", "")
     cwd = input_data.get("cwd", os.getcwd())
 
     # Only control check agent
-    if subagent_type != TARGET_AGENT:
-        sys.exit(0)
-
-    # Skip Ralph Loop for finish phase (already verified in check phase)
-    if "[finish]" in original_prompt.lower():
+    if agent_type != TARGET_AGENT:
         sys.exit(0)
 
     # Find repo root
@@ -357,7 +358,7 @@ def main():
     else:
         # No verify commands, fall back to completion markers
         markers = get_completion_markers(repo_root, task_dir)
-        all_complete, missing = check_completion(agent_output, markers)
+        all_complete, missing = check_completion(last_assistant_message, markers)
 
         if all_complete:
             # All checks complete, allow stop

package/dist/templates/claude/hooks/session-start.py

@@ -288,13 +288,38 @@ def _resolve_spec_scope(
     return None  # Unknown scope type: full scan
 
 
+def _build_workflow_toc(workflow_path: Path) -> str:
+    """Build a compact section index for workflow.md (lazy-load the full file on demand).
+
+    Replaces full-file injection to keep additionalContext payload small.
+    The full file is accessible via: Read tool on .trellis/workflow.md
+    """
+    content = read_file(workflow_path)
+    if not content:
+        return "No workflow.md found"
+
+    toc_lines = [
+        "# Development Workflow — Section Index",
+        "Full guide: .trellis/workflow.md  (read on demand)",
+        "",
+    ]
+    for line in content.splitlines():
+        if line.startswith("## "):
+            toc_lines.append(line)
+
+    toc_lines += [
+        "",
+        "To read a section: use the Read tool on .trellis/workflow.md",
+    ]
+    return "\n".join(toc_lines)
+
+
 def main():
     if should_skip_injection():
         sys.exit(0)
 
     project_dir = Path(os.environ.get("CLAUDE_PROJECT_DIR", ".")).resolve()
     trellis_dir = project_dir / ".trellis"
-    claude_dir = project_dir / ".claude"
 
     # Load config for scope filtering and legacy detection
     is_mono, packages, scope_config, task_pkg, default_pkg = _load_trellis_config(trellis_dir)
@@ -320,8 +345,7 @@ Read and follow all instructions below carefully.
     output.write("\n</current-state>\n\n")
 
     output.write("<workflow>\n")
-    workflow_content = read_file(trellis_dir / "workflow.md", "No workflow.md found")
-    output.write(workflow_content)
+    output.write(_build_workflow_toc(trellis_dir / "workflow.md"))
     output.write("\n</workflow>\n\n")
 
     output.write("<guidelines>\n")
@@ -365,20 +389,13 @@ Read and follow all instructions below carefully.
 
     output.write("</guidelines>\n\n")
 
-    output.write("<instructions>\n")
-    start_md = read_file(
-        claude_dir / "commands" / "trellis" / "start.md", "No start.md found"
-    )
-    output.write(start_md)
-    output.write("\n</instructions>\n\n")
-
     # Check task status and inject structured tag
     task_status = _get_task_status(trellis_dir)
     output.write(f"<task-status>\n{task_status}\n</task-status>\n\n")
 
     output.write("""<ready>
-Context loaded. Steps 1-3 (workflow, context, guidelines) are already injected above — do NOT re-read them.
-Start from Step 4. Wait for user's first message, then follow <instructions> to handle their request.
+Context loaded. Workflow index, project state, and guidelines are already injected above — do NOT re-read them.
+Wait for the user's first message, then handle it following the workflow guide.
 If there is an active task, ask whether to continue it.
 </ready>""")
 

package/dist/templates/claude/hooks/statusline.py

@@ -11,12 +11,19 @@ Info line: model · ctx% · branch · duration · developer · tasks · rate lim
 """
 from __future__ import annotations
 
+import io
 import json
 import re
 import subprocess
 import sys
 from pathlib import Path
 
+# Fix: Windows Python defaults to GBK encoding, which corrupts UTF-8
+# characters like the middle dot (·). Wrap stdout/stderr with UTF-8.
+if sys.platform == "win32":
+    sys.stdout = io.TextIOWrapper(sys.stdout.detach(), encoding="utf-8")
+    sys.stderr = io.TextIOWrapper(sys.stderr.detach(), encoding="utf-8")
+
 
 def _read_text(path: Path) -> str:
     try:

package/dist/templates/codex/hooks/session-start.py

@@ -125,6 +125,32 @@ def _get_task_status(trellis_dir: Path) -> str:
     return f"Status: READY\nTask: {task_title}\nNext: Continue with implement or check"
 
 
+def _build_workflow_toc(workflow_path: Path) -> str:
+    """Build a compact section index for workflow.md (lazy-load the full file on demand).
+
+    Replaces full-file injection to keep additionalContext payload small.
+    The full file is accessible via: Read tool on .trellis/workflow.md
+    """
+    content = read_file(workflow_path)
+    if not content:
+        return "No workflow.md found"
+
+    toc_lines = [
+        "# Development Workflow — Section Index",
+        "Full guide: .trellis/workflow.md  (read on demand)",
+        "",
+    ]
+    for line in content.splitlines():
+        if line.startswith("## "):
+            toc_lines.append(line)
+
+    toc_lines += [
+        "",
+        "To read a section: use the Read tool on .trellis/workflow.md",
+    ]
+    return "\n".join(toc_lines)
+
+
 def main() -> None:
     if should_skip_injection():
         sys.exit(0)
@@ -137,7 +163,6 @@ def main() -> None:
         project_dir = Path(".").resolve()
 
     trellis_dir = project_dir / ".trellis"
-    codex_dir = project_dir / ".codex"
 
     output = StringIO()
 
@@ -154,8 +179,7 @@ Read and follow all instructions below carefully.
     output.write("\n</current-state>\n\n")
 
     output.write("<workflow>\n")
-    workflow_content = read_file(trellis_dir / "workflow.md", "No workflow.md found")
-    output.write(workflow_content)
+    output.write(_build_workflow_toc(trellis_dir / "workflow.md"))
     output.write("\n</workflow>\n\n")
 
     output.write("<guidelines>\n")
@@ -193,21 +217,12 @@ Read and follow all instructions below carefully.
 
     output.write("</guidelines>\n\n")
 
-    # Inject start skill as instructions (Codex uses skills, not slash commands)
-    start_skill = codex_dir / "skills" / "start" / "SKILL.md"
-    if not start_skill.is_file():
-        start_skill = project_dir / ".agents" / "skills" / "start" / "SKILL.md"
-    if start_skill.is_file():
-        output.write("<instructions>\n")
-        output.write(read_file(start_skill))
-        output.write("\n</instructions>\n\n")
-
     task_status = _get_task_status(trellis_dir)
     output.write(f"<task-status>\n{task_status}\n</task-status>\n\n")
 
     output.write("""<ready>
-Context loaded. Steps 1-3 (workflow, context, guidelines) are already injected above — do NOT re-read them.
-Start from Step 4. Wait for user's first message, then follow <instructions> to handle their request.
+Context loaded. Workflow index, project state, and guidelines are already injected above — do NOT re-read them.
+Wait for the user's first message, then handle it following the workflow guide.
 If there is an active task, ask whether to continue it.
 </ready>""")
 

package/dist/templates/copilot/hooks/session-start.py

@@ -125,6 +125,32 @@ def _get_task_status(trellis_dir: Path) -> str:
     return f"Status: READY\nTask: {task_title}\nNext: Continue with implement or check"
 
 
+def _build_workflow_toc(workflow_path: Path) -> str:
+    """Build a compact section index for workflow.md (lazy-load the full file on demand).
+
+    Replaces full-file injection to keep additionalContext payload small.
+    The full file is accessible via: Read tool on .trellis/workflow.md
+    """
+    content = read_file(workflow_path)
+    if not content:
+        return "No workflow.md found"
+
+    toc_lines = [
+        "# Development Workflow — Section Index",
+        "Full guide: .trellis/workflow.md  (read on demand)",
+        "",
+    ]
+    for line in content.splitlines():
+        if line.startswith("## "):
+            toc_lines.append(line)
+
+    toc_lines += [
+        "",
+        "To read a section: use the Read tool on .trellis/workflow.md",
+    ]
+    return "\n".join(toc_lines)
+
+
 def main() -> None:
     if should_skip_injection():
         sys.exit(0)
@@ -153,8 +179,7 @@ Read and follow all instructions below carefully.
     output.write("\n</current-state>\n\n")
 
     output.write("<workflow>\n")
-    workflow_content = read_file(trellis_dir / "workflow.md", "No workflow.md found")
-    output.write(workflow_content)
+    output.write(_build_workflow_toc(trellis_dir / "workflow.md"))
     output.write("\n</workflow>\n\n")
 
     output.write("<guidelines>\n")
@@ -196,8 +221,8 @@ Read and follow all instructions below carefully.
     output.write(f"<task-status>\n{task_status}\n</task-status>\n\n")
 
     output.write("""<ready>
-Context loaded. Steps 1-3 (workflow, context, guidelines) are already injected above — do NOT re-read them.
-Start from Step 4. Wait for user's first message, then follow the workflow to handle their request.
+Context loaded. Workflow index, project state, and guidelines are already injected above — do NOT re-read them.
+Wait for the user's first message, then handle it following the workflow guide.
 If there is an active task, ask whether to continue it.
 </ready>""")
 

package/dist/templates/iflow/hooks/session-start.py

@@ -278,6 +278,32 @@ def _resolve_spec_scope(
     return None
 
 
+def _build_workflow_toc(workflow_path: Path) -> str:
+    """Build a compact section index for workflow.md (lazy-load the full file on demand).
+
+    Replaces full-file injection to keep additionalContext payload small.
+    The full file is accessible via: Read tool on .trellis/workflow.md
+    """
+    content = read_file(workflow_path)
+    if not content:
+        return "No workflow.md found"
+
+    toc_lines = [
+        "# Development Workflow — Section Index",
+        "Full guide: .trellis/workflow.md  (read on demand)",
+        "",
+    ]
+    for line in content.splitlines():
+        if line.startswith("## "):
+            toc_lines.append(line)
+
+    toc_lines += [
+        "",
+        "To read a section: use the Read tool on .trellis/workflow.md",
+    ]
+    return "\n".join(toc_lines)
+
+
 def main():
     if should_skip_injection():
         sys.exit(0)
@@ -285,7 +311,6 @@ def main():
     # iFlow don't have an env for project, use `.` instead
     project_dir = Path(".").resolve()
     trellis_dir = project_dir / ".trellis"
-    iflow_dir = project_dir / ".iflow"
 
     # Load config for scope filtering and legacy detection
     is_mono, packages, scope_config, task_pkg, default_pkg = _load_trellis_config(trellis_dir)
@@ -311,8 +336,7 @@ Read and follow all instructions below carefully.
     output.write("\n</current-state>\n\n")
 
     output.write("<workflow>\n")
-    workflow_content = read_file(trellis_dir / "workflow.md", "No workflow.md found")
-    output.write(workflow_content)
+    output.write(_build_workflow_toc(trellis_dir / "workflow.md"))
     output.write("\n</workflow>\n\n")
 
     output.write("<guidelines>\n")
@@ -354,20 +378,13 @@ Read and follow all instructions below carefully.
 
     output.write("</guidelines>\n\n")
 
-    output.write("<instructions>\n")
-    start_md = read_file(
-        iflow_dir / "commands" / "trellis" / "start.md", "No start.md found"
-    )
-    output.write(start_md)
-    output.write("\n</instructions>\n\n")
-
     # R2: Check task status and inject structured tag
     task_status = _get_task_status(trellis_dir)
     output.write(f"<task-status>\n{task_status}\n</task-status>\n\n")
 
     output.write("""<ready>
-Context loaded. Steps 1-3 (workflow, context, guidelines) are already injected above — do NOT re-read them.
-Start from Step 4. Wait for user's first message, then follow <instructions> to handle their request.
+Context loaded. Workflow index, project state, and guidelines are already injected above — do NOT re-read them.
+Wait for the user's first message, then handle it following the workflow guide.
 If there is an active task, ask whether to continue it.
 </ready>""")
 

package/dist/templates/opencode/lib/trellis-context.js

@@ -1,23 +1,16 @@
 /**
  * Trellis Context Manager
  *
- * Unified context management for OpenCode plugins.
- * Handles detection of oh-my-opencode, .claude/hooks/, and other edge cases.
- *
- * Usage:
- *   import { TrellisContext } from "./trellis-context.js"
- *   const ctx = new TrellisContext(directory)
- *   if (ctx.shouldSkipHook("session-start")) return
+ * Utility class for OpenCode plugins providing file reading,
+ * JSONL parsing, and context building capabilities.
  */
 
 import { existsSync, readFileSync, appendFileSync, readdirSync } from "fs"
 import { isAbsolute, join } from "path"
-import { homedir, platform } from "os"
+import { platform } from "os"
 import { execSync } from "child_process"
 
-// Python command: Windows uses 'python', macOS/Linux use 'python3'
 const PYTHON_CMD = platform() === "win32" ? "python" : "python3"
-
 // Debug logging
 const DEBUG_LOG = "/tmp/trellis-plugin-debug.log"
 
@@ -33,151 +26,17 @@ function debugLog(prefix, ...args) {
 
 /**
  * Trellis Context Manager
- *
- * Centralized logic for:
- * - Detecting oh-my-opencode installation
- * - Checking .claude/hooks/ presence
- * - Determining which plugin should handle each hook
  */
 export class TrellisContext {
   constructor(directory) {
     this.directory = directory
-    this._omoInstalled = null
-    this._omoHooksEnabled = null
-    this._claudeHooksPresent = {}
-
     debugLog("context", "TrellisContext initialized", { directory })
   }
 
-  // ============================================================
-  // oh-my-opencode Detection
-  // ============================================================
-
-  /**
-   * Check if oh-my-opencode is installed
-   *
-   * Detection order:
-   * 1. Check if oh-my-opencode.json exists (most reliable)
-   * 2. Fallback: check opencode.json plugin list
-   */
-  isOmoInstalled() {
-    if (this._omoInstalled !== null) {
-      return this._omoInstalled
-    }
-
-    try {
-      const configDir = join(homedir(), ".config", "opencode")
-
-      // Method 1: Check oh-my-opencode.json existence (omo-specific config)
-      const omoConfigPath = join(configDir, "oh-my-opencode.json")
-      if (existsSync(omoConfigPath)) {
-        this._omoInstalled = true
-        debugLog("context", "omo installed: oh-my-opencode.json exists")
-        return true
-      }
-
-      // Method 2: Fallback to plugin list check
-      const configPath = join(configDir, "opencode.json")
-      if (!existsSync(configPath)) {
-        this._omoInstalled = false
-        debugLog("context", "omo not installed: no config files")
-        return false
-      }
-
-      const content = readFileSync(configPath, "utf-8")
-      const config = JSON.parse(content)
-      const plugins = config.plugin || []
-
-      this._omoInstalled = plugins.some(p =>
-        typeof p === "string" && p.toLowerCase().includes("oh-my-opencode")
-      )
-
-      debugLog("context", "omo installed (plugin list):", this._omoInstalled)
-      return this._omoInstalled
-    } catch (e) {
-      debugLog("context", "omo detection error:", e.message)
-      this._omoInstalled = false
-      return false
-    }
-  }
-
-  /**
-   * Check if omo's claude_code.hooks is enabled
-   * Reads oh-my-opencode.json or defaults to true
-   */
-  isOmoHooksEnabled() {
-    if (this._omoHooksEnabled !== null) {
-      return this._omoHooksEnabled
-    }
-
-    if (!this.isOmoInstalled()) {
-      this._omoHooksEnabled = false
-      return false
-    }
-
-    try {
-      // Check global config
-      const globalConfig = join(homedir(), ".config", "opencode", "oh-my-opencode.json")
-      if (existsSync(globalConfig)) {
-        const content = readFileSync(globalConfig, "utf-8")
-        const config = JSON.parse(content)
-        if (config.claude_code?.hooks === false) {
-          this._omoHooksEnabled = false
-          debugLog("context", "omo hooks disabled in global config")
-          return false
-        }
-      }
-
-      // Check project config
-      const projectConfig = join(this.directory, "oh-my-opencode.json")
-      if (existsSync(projectConfig)) {
-        const content = readFileSync(projectConfig, "utf-8")
-        const config = JSON.parse(content)
-        if (config.claude_code?.hooks === false) {
-          this._omoHooksEnabled = false
-          debugLog("context", "omo hooks disabled in project config")
-          return false
-        }
-      }
-
-      // Default: enabled
-      this._omoHooksEnabled = true
-      debugLog("context", "omo hooks enabled (default)")
-      return true
-    } catch (e) {
-      debugLog("context", "omo hooks detection error:", e.message)
-      this._omoHooksEnabled = true // Default to enabled
-      return true
-    }
-  }
-
-  // ============================================================
-  // .claude/hooks/ Detection
-  // ============================================================
-
-  /**
-   * Check if a specific .claude/hooks/ file exists
-   */
-  hasClaudeHook(hookName) {
-    if (hookName in this._claudeHooksPresent) {
-      return this._claudeHooksPresent[hookName]
-    }
-
-    const hookPath = join(this.directory, ".claude", "hooks", `${hookName}.py`)
-    const exists = existsSync(hookPath)
-
-    this._claudeHooksPresent[hookName] = exists
-    debugLog("context", `claude hook ${hookName}:`, exists)
-    return exists
-  }
-
   // ============================================================
   // Trellis Project Detection
   // ============================================================
 
-  /**
-   * Check if this is a Trellis-managed project
-   */
   isTrellisProject() {
     return existsSync(join(this.directory, ".trellis"))
   }
@@ -237,54 +96,10 @@ export class TrellisContext {
     return join(this.directory, ".trellis", "tasks", normalized)
   }
 
-  // ============================================================
-  // Hook Decision Logic
-  // ============================================================
-
-  /**
-   * Determine if our plugin should skip this hook
-   * (because omo will handle it via .claude/hooks/)
-   *
-   * @param {string} hookName - Hook name without extension (e.g., "session-start")
-   * @returns {boolean} - true if we should skip, false if we should handle
-   */
-  shouldSkipHook(hookName) {
-    // Not a Trellis project? Skip.
-    if (!this.isTrellisProject()) {
-      debugLog("context", `shouldSkipHook(${hookName}): skip - not Trellis project`)
-      return true
-    }
-
-    // omo not installed? We handle it.
-    if (!this.isOmoInstalled()) {
-      debugLog("context", `shouldSkipHook(${hookName}): handle - omo not installed`)
-      return false
-    }
-
-    // omo installed but hooks disabled? We handle it.
-    if (!this.isOmoHooksEnabled()) {
-      debugLog("context", `shouldSkipHook(${hookName}): handle - omo hooks disabled`)
-      return false
-    }
-
-    // omo installed + hooks enabled + .claude/hooks/ exists? Skip (omo handles).
-    if (this.hasClaudeHook(hookName)) {
-      debugLog("context", `shouldSkipHook(${hookName}): skip - omo will handle via .claude/hooks/`)
-      return true
-    }
-
-    // omo installed but no .claude/hooks/ file? We handle it.
-    debugLog("context", `shouldSkipHook(${hookName}): handle - no .claude/hooks/ file`)
-    return false
-  }
-
   // ============================================================
   // File Reading Utilities
   // ============================================================
 
-  /**
-   * Read a file, return null on error
-   */
   readFile(filePath) {
     try {
       if (existsSync(filePath)) {
@@ -296,16 +111,10 @@ export class TrellisContext {
     return null
   }
 
-  /**
-   * Read a file relative to project directory
-   */
   readProjectFile(relativePath) {
     return this.readFile(join(this.directory, relativePath))
   }
 
-  /**
-   * Run a Python script and return output
-   */
   runScript(scriptPath, cwd = null) {
     try {
       const result = execSync(`${PYTHON_CMD} "${scriptPath}"`, {
@@ -324,12 +133,6 @@ export class TrellisContext {
   // JSONL Reading
   // ============================================================
 
-  /**
-   * Read all .md files in a directory
-   * @param {string} dirPath - Directory path relative to project root
-   * @param {number} maxFiles - Max files to read (prevent huge directories)
-   * @returns {Array<{path: string, content: string}>}
-   */
   readDirectoryMdFiles(dirPath, maxFiles = 20) {
     const results = []
     const fullPath = join(this.directory, dirPath)
@@ -379,11 +182,9 @@ export class TrellisContext {
         if (!file) continue
 
         if (entryType === "directory") {
-          // Read all .md files in directory
           const dirEntries = this.readDirectoryMdFiles(file)
           results.push(...dirEntries)
         } else {
-          // Read single file
           const fullPath = join(this.directory, file)
           const fileContent = this.readFile(fullPath)
           if (fileContent) {
@@ -397,74 +198,29 @@ export class TrellisContext {
     return results
   }
 
-  /**
-   * Build context string from file entries
-   */
   buildContextFromEntries(entries) {
     return entries.map(e => `=== ${e.path} ===\n${e.content}`).join("\n\n")
   }
 }
 
 // ============================================================
-// Context Collector (for synthetic message injection)
+// Context Collector (for session deduplication)
 // ============================================================
 
-/**
- * Simple context collector for cross-hook communication
- * Similar to oh-my-opencode's contextCollector
- */
 class ContextCollector {
   constructor() {
-    this.pending = new Map()
     this.processed = new Set()
   }
 
-  /**
-   * Store context for a session
-   */
-  store(sessionID, content) {
-    this.pending.set(sessionID, {
-      content,
-      timestamp: Date.now()
-    })
-    debugLog("collector", "stored context for session:", sessionID, "length:", content.length)
-  }
-
-  /**
-   * Check if session has pending context
-   */
-  hasPending(sessionID) {
-    return this.pending.has(sessionID)
-  }
-
-  /**
-   * Get and consume pending context
-   */
-  consume(sessionID) {
-    const pending = this.pending.get(sessionID)
-    this.pending.delete(sessionID)
-    return pending
-  }
-
-  /**
-   * Mark session as processed (for first-message-only injection)
-   */
   markProcessed(sessionID) {
     this.processed.add(sessionID)
   }
 
-  /**
-   * Check if session was already processed
-   */
   isProcessed(sessionID) {
     return this.processed.has(sessionID)
   }
 
-  /**
-   * Clear session state
-   */
   clear(sessionID) {
-    this.pending.delete(sessionID)
     this.processed.delete(sessionID)
   }
 }