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/claudetool/codereview/codereview.go b/claudetool/codereview/codereview.go
index 2745957..5d10d79 100644
--- a/claudetool/codereview/codereview.go
+++ b/claudetool/codereview/codereview.go
@@ -21,19 +21,12 @@
initialStatus []fileStatus // git status of files at initial commit, absolute paths
reviewed []string // history of all commits which have been reviewed
initialWorktree string // git worktree at initial commit, absolute path
- llmReview bool // enables a subagent LLM review
}
-const (
- NoLLMReview = false
- DoLLMReview = true
-)
-
-func NewCodeReviewer(ctx context.Context, repoRoot, sketchBaseRef string, llmReview bool) (*CodeReviewer, error) {
+func NewCodeReviewer(ctx context.Context, repoRoot, sketchBaseRef string) (*CodeReviewer, error) {
r := &CodeReviewer{
repoRoot: repoRoot,
sketchBaseRef: sketchBaseRef,
- llmReview: llmReview,
}
if r.repoRoot == "" {
return nil, fmt.Errorf("NewCodeReviewer: repoRoot must be non-empty")