Greg Gauthier
0aa806be70
- Implemented `grokkit docs` command for generating language-specific documentation comments (godoc, PEP 257, Doxygen, etc.) with previews, backups, and auto-apply option - Extracted message builder functions for commit, history, pr-describe, and review commands - Added comprehensive unit tests for all command message builders (commit_test.go, docs_test.go, history_test.go, lint_test.go, prdescribe_test.go, review_test.go) - Enforced 70% test coverage threshold in CI workflow - Added .golangci.yml configuration with linters like govet, errcheck, staticcheck - Updated Makefile to include -race in tests and add help target - Updated README.md with new docs command details, workflows, and quality features - Added .claude/ to .gitignore - Configured default model for docs command in config.go
44 lines
1.2 KiB
Go
44 lines
1.2 KiB
Go
package cmd
|
|
|
|
import (
|
|
"fmt"
|
|
|
|
"github.com/fatih/color"
|
|
"github.com/spf13/cobra"
|
|
"gmgauthier.com/grokkit/config"
|
|
"gmgauthier.com/grokkit/internal/git"
|
|
"gmgauthier.com/grokkit/internal/grok"
|
|
)
|
|
|
|
var reviewCmd = &cobra.Command{
|
|
Use: "review [path]",
|
|
Short: "Review the current repository or directory",
|
|
Run: func(cmd *cobra.Command, args []string) {
|
|
modelFlag, _ := cmd.Flags().GetString("model")
|
|
model := config.GetModel("review", modelFlag)
|
|
|
|
client := grok.NewClient()
|
|
diff, err := git.Run([]string{"diff", "--no-color"})
|
|
if err != nil {
|
|
color.Red("Failed to get git diff: %v", err)
|
|
return
|
|
}
|
|
status, err := git.Run([]string{"status", "--short"})
|
|
if err != nil {
|
|
color.Red("Failed to get git status: %v", err)
|
|
return
|
|
}
|
|
|
|
messages := buildReviewMessages(status, diff)
|
|
color.Yellow("Grok is reviewing the repo...")
|
|
client.Stream(messages, model)
|
|
},
|
|
}
|
|
|
|
func buildReviewMessages(status, diff string) []map[string]string {
|
|
return []map[string]string{
|
|
{"role": "system", "content": "You are an expert code reviewer. Give a concise summary + 3-5 actionable improvements."},
|
|
{"role": "user", "content": fmt.Sprintf("Git status:\n%s\n\nGit diff:\n%s", status, diff)},
|
|
}
|
|
}
|