experiment: remove llm_review experiment and associated code
It creates confusion for the agent and doesn't provide much value.
Perhaps with better prompt engineering, but I can't invest
in that much right now.
Co-Authored-By: sketch <hello@sketch.dev>
Change-ID: s9f2a8c418876c498k
diff --git a/loop/agent.go b/loop/agent.go
index e934e97..f6cbe37 100644
--- a/loop/agent.go
+++ b/loop/agent.go
@@ -25,7 +25,6 @@
"sketch.dev/claudetool/browse"
"sketch.dev/claudetool/codereview"
"sketch.dev/claudetool/onstart"
- "sketch.dev/experiment"
"sketch.dev/llm"
"sketch.dev/llm/ant"
"sketch.dev/llm/conversation"
@@ -874,11 +873,7 @@
}
a.codebase = codebase
- llmCodeReview := codereview.NoLLMReview
- if experiment.Enabled("llm_review") {
- llmCodeReview = codereview.DoLLMReview
- }
- codereview, err := codereview.NewCodeReviewer(ctx, a.repoRoot, a.SketchGitBaseRef(), llmCodeReview)
+ codereview, err := codereview.NewCodeReviewer(ctx, a.repoRoot, a.SketchGitBaseRef())
if err != nil {
return fmt.Errorf("Agent.Init: codereview.NewCodeReviewer: %w", err)
}