@comfanion/workflow 4.36.63 → 4.36.65

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@comfanion/workflow",
-  "version": "4.36.63",
+  "version": "4.36.65",
   "description": "Initialize OpenCode Workflow system for AI-assisted development with semantic code search",
   "type": "module",
   "bin": {

package/src/build-info.json

@@ -1,6 +1,6 @@
 {
-  "version": "4.36.63",
-  "buildDate": "2026-01-26T02:37:19.900Z",
+  "version": "4.36.65",
+  "buildDate": "2026-01-26T03:14:43.281Z",
   "files": [
     "config.yaml",
     "FLOW.yaml",

package/src/opencode/agents/reviewer.md

@@ -88,13 +88,7 @@ permission:
     <action>search() in docs for architecture requirements</action>
   </phase>
 
-  <phase name="2. Run Tests & Lint">
-    <action>Run test suite: go test / npm test / pytest / cargo test</action>
-    <action>Run linter: golangci-lint / eslint / ruff / cargo clippy</action>
-    <action>If failures → include in review report as HIGH priority</action>
-  </phase>
-
-  <phase name="3. Security First">
+  <phase name="2. Security Analysis">
     <action>Check for hardcoded secrets</action>
     <action>Verify input validation on all user inputs</action>
     <action>Check SQL injection, XSS vulnerabilities</action>
@@ -102,20 +96,26 @@ permission:
     <action>Check if sensitive data is logged</action>
   </phase>
 
-  <phase name="4. Correctness">
+  <phase name="3. Correctness Analysis">
     <action>Verify all acceptance criteria are met</action>
     <action>Check edge cases and error handling</action>
     <action>Look for logic errors and race conditions</action>
     <action>Verify tests cover critical paths</action>
   </phase>
 
-  <phase name="5. Code Quality">
+  <phase name="4. Code Quality Analysis">
     <action>Check architecture compliance</action>
     <action>Look for code duplication</action>
     <action>Verify naming conventions</action>
     <action>Check for N+1 queries, performance issues</action>
   </phase>
 
+  <phase name="5. Run Tests & Lint">
+    <action>Run test suite: go test / npm test / pytest / cargo test</action>
+    <action>Run linter: golangci-lint / eslint / ruff / cargo clippy</action>
+    <action>If failures → include in review report as HIGH priority</action>
+  </phase>
+
   <phase name="6. Report">
     <action>Categorize issues: High/Medium/Low</action>
     <action>Provide specific fixes for each issue</action>

package/src/opencode/opencode.json

@@ -4,6 +4,7 @@
   "instructions": [
     "AGENTS.md",
     ".opencode/FLOW.yaml",
+    ".opencode/config.yaml",
     "docs/prd.md",
     "docs/architecture.md"
   ],

package/src/opencode/plugins/custom-compaction.ts

@@ -1,7 +1,26 @@
 import type { Plugin } from "@opencode-ai/plugin"
-import { readFile, access, readdir } from "fs/promises"
+import { readFile, access, readdir, appendFile } from "fs/promises"
 import { join } from "path"
 
+// Debug logging to file
+async function log(directory: string, message: string): Promise<void> {
+  const logPath = join(directory, ".opencode", "compaction.log")
+  const timestamp = new Date().toISOString()
+  try {
+    await appendFile(logPath, `[${timestamp}] ${message}\n`)
+  } catch {
+    // ignore logging errors
+  }
+}
+
+// Service agents that should be ignored
+const SERVICE_AGENTS = ["title", "compaction", "summary", "system"]
+
+function isRealAgent(agent: string | null): boolean {
+  if (!agent) return false
+  return !SERVICE_AGENTS.includes(agent.toLowerCase())
+}
+
 interface TaskStatus {
   id: string
   content: string
@@ -486,30 +505,55 @@ Previous task was completed successfully.
   return {
     // Track active agent from chat messages
     "chat.message": async (input, output) => {
+      await log(directory, `chat.message: agent=${input.agent}, sessionID=${input.sessionID}`)
       if (input.agent) {
         // Handle both string and object agent (e.g., { name: "dev" })
-        lastActiveAgent = typeof input.agent === 'string' 
+        const agent = typeof input.agent === 'string' 
           ? input.agent 
           : (input.agent as any)?.name || null
-        lastSessionId = input.sessionID
+        
+        // Only track real agents, not service agents
+        if (isRealAgent(agent)) {
+          lastActiveAgent = agent
+          lastSessionId = input.sessionID
+          await log(directory, `  -> tracked agent: ${lastActiveAgent}`)
+        } else {
+          await log(directory, `  -> ignored service agent: ${agent}`)
+        }
       }
     },
 
     // Also track from chat params (backup)
     "chat.params": async (input, output) => {
+      await log(directory, `chat.params: agent=${input.agent}`)
       if (input.agent) {
-        lastActiveAgent = typeof input.agent === 'string' 
+        const agent = typeof input.agent === 'string' 
           ? input.agent 
           : (input.agent as any)?.name || null
+        
+        // Only track real agents, not service agents
+        if (isRealAgent(agent)) {
+          lastActiveAgent = agent
+          await log(directory, `  -> tracked agent: ${lastActiveAgent}`)
+        } else {
+          await log(directory, `  -> ignored service agent: ${agent}`)
+        }
       }
     },
 
     "experimental.session.compacting": async (input, output) => {
+      await log(directory, `=== COMPACTION STARTED ===`)
+      await log(directory, `  lastActiveAgent: ${lastActiveAgent}`)
+      
       // Use tracked agent or try to detect from session
       const agent = lastActiveAgent
       const ctx = await buildContext(agent)
       ctx.activeAgent = agent
       
+      await log(directory, `  story: ${ctx.story?.path || 'none'}`)
+      await log(directory, `  todos: ${ctx.todos.length}`)
+      await log(directory, `  relevantFiles: ${ctx.relevantFiles.length}`)
+      
       const context = formatContext(ctx)
       const instructions = formatInstructions(ctx)
       const readCommands = generateReadCommands(agent, ctx.story)
@@ -532,12 +576,17 @@ ${context}
 ---
 
 ${instructions}`)
+      
+      await log(directory, `  -> output.context pushed (${output.context.length} items)`)
+      await log(directory, `=== COMPACTION DONE ===`)
     },
 
     event: async ({ event }) => {
+      await log(directory, `event: ${event.type}`)
       if (event.type === "session.idle") {
         const story = await getActiveStory()
         if (story && story.pendingTasks.length === 0) {
+          await log(directory, `  -> Story complete: ${story.title}`)
           console.log(`[compaction] Story complete: ${story.title}`)
         }
       }

package/src/opencode/skills/dev-story/SKILL.md

@@ -15,30 +15,44 @@ How to transform story tasks into executable instructions for @coder.
 
 ## Task Transformation
 
-Story task is a specification. @coder needs executable instruction with full context.
+Story task is specification with Approach steps. Transform it into executable instruction with full context.
 
-### Add Context
+### Step 1: Read Required Reading
 
-@coder doesn't see the full story. Include:
-- **Existing files** - actual paths to domain entities, interfaces, services that exist
-- **Patterns to follow** - link to existing similar code ("follow pattern from X")
-- **What was done** - results of previous tasks that this task depends on
+Story has "Required Reading" and task has "Read First":
+1. Open each linked document
+2. Find the specific sections mentioned
+3. Extract patterns, signatures, constraints
+
+### Step 2: Find Existing Code
+
+From "Read First" paths:
+1. Read existing similar code (e.g., "existing service example")
+2. Note the structure, imports, error handling
+3. This becomes "Pattern Reference" for @coder
+
+### Step 3: Build Context
+
+@coder doesn't see story. Provide:
+- **Existing files** - actual paths with what they contain
+- **Patterns to follow** - link to existing similar code
+- **What was done** - results of previous tasks this depends on
 - **Imports** - what packages to use
 
-### Add Implementation Direction
+### Step 4: Add Direction
 
-Provide guidance, not code:
-- Interface signatures (what methods to implement)
+Story has "Approach" with high-level steps. Expand with:
+- Interface signatures (method names, params, return types)
 - Error handling approach (what errors to return)
-- Logging requirements (what to log)
 - Validation rules (what to validate)
+- Constraints from documentation
 
-### Add Verification
+### Step 5: Make Verification Concrete
 
-Story has "Done when". Make it concrete:
+Story has "Done when". Add:
 - Specific test commands to run
-- Expected output format
 - Files that must compile
+- Test coverage expectations
 
 ## Formulating Task for @coder
 

package/src/opencode/skills/story-writing/SKILL.md

@@ -95,6 +95,24 @@ Summary table + detailed tasks.
 - [ ] Tests pass
 - [ ] Code reviewed
 
+## No Code in Stories
+
+**Stories are specifications, not implementations.**
+
+✅ **Task provides:**
+- Goal (what to achieve)
+- Read First (where to find patterns)
+- Output Files (what to create)
+- Approach (high-level steps)
+- Done When (how to verify)
+
+❌ **Task does NOT provide:**
+- Code Sketch with implementation
+- Ready-to-copy code blocks
+- Full method bodies
+
+**@coder writes the code.** Story defines WHAT, not HOW.
+
 ## Task Structure (MANDATORY)
 
 Each task MUST be self-contained:
@@ -126,8 +144,6 @@ Each task MUST be self-contained:
 - [ ] Tests pass
 ```
 
-
-
 ## Reference Format
 
 Always use `→` prefix: