fix(fn-110.8): consolidate hooks, add phase skipping, remove schema_version from output

- Reduce ralph-guard hook registrations from 4 to 2 (PreToolUse Bash, Stop)
- Remove ralph-guard from PostToolUse, SubagentStop, and PreToolUse Edit|Write
- Add 60-second filesystem check cache for commit-gate hook
- Add is_memory_enabled() config reader; phase 11 (Memory Auto-Save) now
  skipped when memory.enabled is false/unset (default: false)
- Phase 9 (Outputs) and Phase 4 (TDD) already skip via config/flags
- Add skip_serializing to schema_version on Epic and Task structs so it
  no longer appears in serialized JSON output or disk files

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: z23cc

flowctl/crates/flowctl-cli/src/commands/workflow/phase.rs

@@ -104,25 +104,65 @@ fn is_outputs_enabled() -> bool {
     }
 }
 
+/// Read `memory.enabled` from .flow/config.json. Default: false.
+fn is_memory_enabled() -> bool {
+    use flowctl_core::types::{CONFIG_FILE, FLOW_DIR};
+    let cfg_path = std::env::current_dir()
+        .unwrap_or_else(|_| std::path::PathBuf::from("."))
+        .join(FLOW_DIR)
+        .join(CONFIG_FILE);
+    if !cfg_path.exists() {
+        return false;
+    }
+    match std::fs::read_to_string(&cfg_path) {
+        Ok(content) => {
+            let cfg: serde_json::Value =
+                serde_json::from_str(&content).unwrap_or(serde_json::json!({}));
+            cfg.get("memory")
+                .and_then(|m| m.get("enabled"))
+                .and_then(serde_json::Value::as_bool)
+                .unwrap_or(false)
+        }
+        Err(_) => false,
+    }
+}
+
 /// Build the phase sequence based on mode flags.
+/// Skips phases based on config:
+///   - Phase 4 (TDD): only included when --tdd flag is set
+///   - Phase 9 (Outputs): only included when outputs.enabled is true (default)
+///   - Phase 11 (Memory): only included when memory.enabled is true (default: false)
 fn build_phase_sequence(tdd: bool, review: bool) -> Vec<&'static str> {
     let mut phases = HashSet::new();
+    // Start with default sequence (excludes phase 11 — added conditionally below)
     for p in PHASE_SEQ_DEFAULT {
+        if *p == "11" {
+            continue; // handled by is_memory_enabled() below
+        }
         phases.insert(*p);
     }
     if tdd {
         for p in PHASE_SEQ_TDD {
+            if *p == "11" {
+                continue;
+            }
             phases.insert(*p);
         }
     }
     if review {
         for p in PHASE_SEQ_REVIEW {
+            if *p == "11" {
+                continue;
+            }
             phases.insert(*p);
         }
     }
     if is_outputs_enabled() {
         phases.insert("9");
     }
+    if is_memory_enabled() {
+        phases.insert("11");
+    }
     CANONICAL_ORDER.iter().copied().filter(|p| phases.contains(p)).collect()
 }
 

flowctl/crates/flowctl-core/src/types.rs

@@ -119,7 +119,7 @@ impl std::fmt::Display for ReviewStatus {
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct Epic {
     /// Schema version for forward compatibility.
-    #[serde(default = "default_schema_version")]
+    #[serde(default = "default_schema_version", skip_serializing)]
     pub schema_version: u32,
 
     /// Unique ID, e.g. `fn-1-add-auth`.
@@ -191,7 +191,7 @@ pub struct Epic {
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct Task {
     /// Schema version for forward compatibility.
-    #[serde(default = "default_schema_version")]
+    #[serde(default = "default_schema_version", skip_serializing)]
     pub schema_version: u32,
 
     /// Unique ID, e.g. `fn-1-add-auth.3`.

hooks/hooks.json

@@ -39,34 +39,14 @@
         "hooks": [
           {
             "type": "command",
-            "command": "F=\"${DROID_PLUGIN_ROOT:-${CLAUDE_PLUGIN_ROOT:-}}/bin/flowctl\"; { [ ! -d .flow ] || [ ! -x \"$F\" ]; } || \"$F\" hook commit-gate",
+            "command": "F=\"${DROID_PLUGIN_ROOT:-${CLAUDE_PLUGIN_ROOT:-}}/bin/flowctl\"; C=\"${TMPDIR:-/tmp}/flowctl-commit-gate-ts\"; { [ ! -d .flow ] || [ ! -x \"$F\" ]; } && exit 0; [ -f \"$C\" ] && [ $(( $(date +%s) - $(cat \"$C\" 2>/dev/null || echo 0) )) -lt 60 ] && exit 0; \"$F\" hook commit-gate; echo $(date +%s) > \"$C\"",
             "timeout": 10
           }
         ],
-        "description": "Block git commit when flow task is in_progress but guard has not passed"
-      },
-      {
-        "matcher": "Edit|Write",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "F=\"${DROID_PLUGIN_ROOT:-${CLAUDE_PLUGIN_ROOT:-}}/bin/flowctl\"; [ -x \"$F\" ] && \"$F\" hook ralph-guard || true",
-            "timeout": 5
-          }
-        ]
+        "description": "Block git commit when flow task is in_progress but guard has not passed (cached 60s)"
       }
     ],
     "PostToolUse": [
-      {
-        "matcher": "Bash|Execute",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "F=\"${DROID_PLUGIN_ROOT:-${CLAUDE_PLUGIN_ROOT:-}}/bin/flowctl\"; [ -x \"$F\" ] && \"$F\" hook ralph-guard || true",
-            "timeout": 5
-          }
-        ]
-      },
       {
         "matcher": "Bash",
         "hooks": [
@@ -100,17 +80,7 @@
         "description": "Auto-capture session memories to .flow/memory/ (requires memory.enabled or memory.auto in config)"
       }
     ],
-    "SubagentStop": [
-      {
-        "hooks": [
-          {
-            "type": "command",
-            "command": "F=\"${DROID_PLUGIN_ROOT:-${CLAUDE_PLUGIN_ROOT:-}}/bin/flowctl\"; [ -x \"$F\" ] && \"$F\" hook ralph-guard || true",
-            "timeout": 5
-          }
-        ]
-      }
-    ],
+    "SubagentStop": [],
     "SubagentStart": [
       {
         "matcher": "",