fix(headless): give hosted reviews room to conclude (OpenCoven#119)

Signed-off-by: Timothy Wayne Gregg <Timothy.Gregg@complete.tech>

Author: CompleteDotTech

src-rust/crates/cli/src/headless.rs

@@ -1086,7 +1086,14 @@ impl ReviewTrace {
         let normalized = normalize_existing_or_lexical(&absolute);
         let root = normalize_existing_or_lexical(&self.workspace_root);
         let relative = normalized.strip_prefix(&root).ok()?;
-        normalize_relative_path(&relative.to_string_lossy())
+        if normalized.exists() && !normalized.is_file() {
+            return None;
+        }
+        let path = normalize_relative_path(&relative.to_string_lossy())?;
+        if path.split('/').any(|part| part == ".git") {
+            return None;
+        }
+        Some(path)
     }
 }
 
@@ -1746,6 +1753,8 @@ mod tests {
         std::fs::write(ws.join("src/support.rs"), "").unwrap();
         std::fs::write(ws.join("src/config.rs"), "").unwrap();
         std::fs::write(ws.join("README.md"), "").unwrap();
+        std::fs::create_dir_all(ws.join(".git/hooks")).unwrap();
+        std::fs::write(ws.join(".git/config"), "").unwrap();
 
         let mut trace = ReviewTrace::new(ws);
         trace.record_tool_start("Read", r#"{"file_path":"src/lib.rs"}"#);
@@ -1766,6 +1775,7 @@ mod tests {
             false,
         );
         trace.record_tool_end("Glob", "src/lib.rs\nsrc/support.rs\n", false);
+        trace.record_tool_end("Glob", ".git\n.git/config\nsrc\n", false);
         trace.record_tool_start("Read", r#"{"file_path":"../outside.rs"}"#);
 
         assert_eq!(

src-rust/crates/cli/src/main.rs

@@ -865,6 +865,7 @@ async fn main() -> anyhow::Result<()> {
     } else {
         tools
     };
+    apply_headless_review_query_defaults(&mut query_config, github_context.as_ref());
     let tools = filter_tools_for_hosted_review(tools, &config);
 
     // Spawn the background cron scheduler (fires cron tasks at scheduled times).
@@ -1564,6 +1565,18 @@ fn filter_search_only_tools() -> Arc<Vec<Box<dyn claurst_tools::Tool>>> {
     Arc::new(filtered)
 }
 
+fn apply_headless_review_query_defaults(
+    query_config: &mut claurst_query::QueryConfig,
+    github_context: Option<&SessionBrief>,
+) {
+    if github_context
+        .map(|brief| brief.review_mode() != headless::ReviewMode::None)
+        .unwrap_or(false)
+    {
+        query_config.max_turns = query_config.max_turns.max(20);
+    }
+}
+
 // ---------------------------------------------------------------------------
 // Headless mode: read prompt from arg/stdin, run, print response
 // ---------------------------------------------------------------------------
@@ -5253,6 +5266,49 @@ mod tests {
         }
     }
 
+    #[test]
+    fn hosted_review_headless_gets_extra_turn_budget() {
+        let raw = r#"{
+            "contract_version": "2",
+            "trigger": "issue_mention",
+            "repo": { "owner": "o", "name": "r", "clone_url": "https://github.com/o/r.git", "default_branch": "main" },
+            "task": { "kind": "respond_to_mention", "issue_number": 7, "comment_body": "review this" },
+            "familiar": { "id": "cody", "display_name": "Cody", "skills": [] },
+            "workspace": { "root": "/tmp/ws" },
+            "review_context": { "kind": "pull_request", "files": [{ "filename": "src/lib.rs" }] }
+        }"#;
+        let brief: SessionBrief = serde_json::from_str(raw).expect("brief parses");
+        let mut query_config = claurst_query::QueryConfig {
+            max_turns: 10,
+            ..Default::default()
+        };
+
+        apply_headless_review_query_defaults(&mut query_config, Some(&brief));
+
+        assert_eq!(query_config.max_turns, 20);
+    }
+
+    #[test]
+    fn non_review_headless_keeps_turn_budget() {
+        let raw = r#"{
+            "contract_version": "2",
+            "trigger": "issue_mention",
+            "repo": { "owner": "o", "name": "r", "clone_url": "https://github.com/o/r.git", "default_branch": "main" },
+            "task": { "kind": "respond_to_mention", "issue_number": 7, "comment_body": "answer this" },
+            "familiar": { "id": "cody", "display_name": "Cody", "skills": [] },
+            "workspace": { "root": "/tmp/ws" }
+        }"#;
+        let brief: SessionBrief = serde_json::from_str(raw).expect("brief parses");
+        let mut query_config = claurst_query::QueryConfig {
+            max_turns: 10,
+            ..Default::default()
+        };
+
+        apply_headless_review_query_defaults(&mut query_config, Some(&brief));
+
+        assert_eq!(query_config.max_turns, 10);
+    }
+
     #[test]
     fn hosted_review_filters_write_and_execute_tools_by_default() {
         let all = Arc::new(claurst_tools::all_tools());