From 378d008a91d90cee484c466f760aec135c8dc3ea Mon Sep 17 00:00:00 2001 From: schreifuchs Date: Thu, 12 Feb 2026 20:58:55 +0100 Subject: [PATCH] feat: gitea client --- GEMINI.md | 77 +++++++++++++++++++++++++++ cmd/pierre/main.go | 74 +++++++++++++++++++------ config.example.yaml | 17 ++++-- go.mod | 10 ++-- go.sum | 28 ++++++++-- internal/chatter/model.go | 8 +++ internal/gitadapters/adapter.go | 7 +++ internal/gitadapters/gitea/adapter.go | 30 +++++++++++ internal/pierre/judge.go | 6 +-- 9 files changed, 225 insertions(+), 32 deletions(-) create mode 100644 GEMINI.md create mode 100644 internal/gitadapters/adapter.go create mode 100644 internal/gitadapters/gitea/adapter.go diff --git a/GEMINI.md b/GEMINI.md new file mode 100644 index 0000000..a34759b --- /dev/null +++ b/GEMINI.md @@ -0,0 +1,77 @@ +# Pierre Bot + +Pierre Bot is an intelligent, AI-powered code review assistant designed for Bitbucket Server/Data Center. It automates the initial pass of code review by analyzing Pull Request diffs and identifying potential bugs, logic errors, and style issues using modern LLMs (Google Gemini 2.0 Flash or Ollama). + +## Project Overview + +* **Type:** Go CLI Application +* **Core Function:** Fetches PR diffs from Bitbucket -> Sends to LLM -> Prints structured review comments. +* **Key Technologies:** + * **Language:** Go (1.25+) + * **AI SDKs:** `google/generative-ai-go`, `ollama/ollama` + * **CLI Framework:** `alecthomas/kong` + +## Architecture + +The project follows a standard Go project layout: + +* **`cmd/pierre/`**: Contains the `main.go` entry point. It handles configuration parsing (flags, env vars, file), initializes adapters, and orchestrates the application flow. +* **`internal/pierre/`**: Contains the core business logic. + * `judge.go`: Defines the `JudgePR` function which prepares the system prompt and context for the LLM. +* **`internal/chatter/`**: Abstraction layer for LLM providers. + * `gemini.go`: Implements the `ChatAdapter` interface for Google Gemini. notably includes **dynamic JSON schema generation** via reflection (`schemaFromType`) to enforce structured output from the model. + * `ollama.go`: Implements the `ChatAdapter` for Ollama (local models). +* **`internal/gitadapters/`**: Abstraction for Version Control Systems. + * `bitbucket.go`: Client for fetching PR diffs from Bitbucket Server. + +## Building and Running + +### Prerequisites + +* Go 1.25 or later +* Access to a Bitbucket Server instance +* API Key for Google Gemini (or a running Ollama instance) + +### Build + +```bash +go build -o pierre ./cmd/pierre/main.go +``` + +### Configuration + +Configuration is handled via `kong` and supports a hierarchy: **Flags > Env Vars > Config File**. + +**1. Environment Variables:** + +* `BITBUCKET_URL`: Base URL of the Bitbucket instance. +* `BITBUCKET_TOKEN`: Personal Access Token (HTTP) for Bitbucket. +* `LLM_PROVIDER`: `gemini` or `ollama`. +* `LLM_API_KEY`: API Key for Gemini. +* `LLM_MODEL`: Model name (e.g., `gemini-2.0-flash`). + +**2. Configuration File (`config.yaml`):** + +See `config.example.yaml` for a template. Place it in the current directory or `~/.pierre.yaml`. + +### Usage + +Run the bot against a specific Pull Request: + +```bash +# Syntax: ./pierre [flags] +./pierre --llm-provider=gemini --llm-model=gemini-2.0-flash MYPROJ my-repo 123 +``` + +## Development Conventions + +* **Structured Output:** The bot relies on the LLM returning valid JSON matching the `Comment` struct. This is enforced in `internal/chatter/gemini.go` by converting the Go struct definition into a `genai.Schema`. +* **Dependency Injection:** Adapters (`gitadapters`, `chatter`) are initialized in `main` and passed to the core logic, making testing easier. +* **Error Handling:** strict error checks are preferred; the bot will exit if it cannot fetch the diff or initialize the AI. + +## Key Files + +* **`cmd/pierre/main.go`**: Application entry point and config wiring. +* **`internal/pierre/judge.go`**: The "brain" that constructs the prompt for the AI. +* **`internal/chatter/gemini.go`**: Gemini integration logic, including the reflection-based schema generator. +* **`config.example.yaml`**: Reference configuration file. diff --git a/cmd/pierre/main.go b/cmd/pierre/main.go index f210e56..ca8a814 100644 --- a/cmd/pierre/main.go +++ b/cmd/pierre/main.go @@ -9,18 +9,26 @@ import ( "bitbucket.bit.admin.ch/scm/~u80859501/pierre-bot/internal/chatter" "bitbucket.bit.admin.ch/scm/~u80859501/pierre-bot/internal/gitadapters" + "bitbucket.bit.admin.ch/scm/~u80859501/pierre-bot/internal/gitadapters/gitea" "bitbucket.bit.admin.ch/scm/~u80859501/pierre-bot/internal/pierre" "github.com/alecthomas/kong" kongyaml "github.com/alecthomas/kong-yaml" ) type BitbucketConfig struct { - BaseURL string `help:"Bitbucket Base URL (e.g. https://bitbucket.example.com)" required:"" env:"BITBUCKET_URL"` + BaseURL string `help:"Bitbucket Base URL (e.g. https://bitbucket.example.com)" env:"BITBUCKET_URL"` Token string `help:"Bearer Token" env:"BITBUCKET_TOKEN"` - // Positional arguments - Project string `arg:"" help:"Project Key (e.g. PROJ)" env:"BITBUCKET_PROJECT"` - Repo string `arg:"" help:"Repository Slug" env:"BITBUCKET_REPO"` - PRID int `arg:"" help:"Pull Request ID" name:"pr"` +} + +type GiteaConfig struct { + BaseURL string `help:"Gitea Base URL (e.g. https://gitea.com)" env:"GITEA_URL"` + Token string `help:"API Token" env:"GITEA_TOKEN"` +} + +type RepoArgs struct { + Owner string `arg:"" help:"Project Key or Owner" env:"PIERRE_OWNER"` + Repo string `arg:"" help:"Repository Slug" env:"PIERRE_REPO"` + PRID int `arg:"" help:"Pull Request ID" name:"pr"` } type LLMConfig struct { @@ -31,9 +39,12 @@ type LLMConfig struct { } type Config struct { - Bitbucket BitbucketConfig `embed:"" prefix:"bitbucket-"` - LLM LLMConfig `embed:"" prefix:"llm-"` - Config kong.ConfigFlag `help:"Path to a YAML config file"` + GitProvider string `help:"Git provider (bitbucket or gitea)" env:"GIT_PROVIDER"` + Bitbucket BitbucketConfig `embed:"" prefix:"bitbucket-"` + Gitea GiteaConfig `embed:"" prefix:"gitea-"` + Repo RepoArgs `embed:""` + LLM LLMConfig `embed:"" prefix:"llm-"` + Config kong.ConfigFlag `help:"Path to a YAML config file"` } func main() { @@ -46,25 +57,54 @@ func main() { defaultConfig := filepath.Join(home, ".config", "pierre", "config.yaml") // Parse flags, env vars, and config files - kong.Parse(cfg, + ctx := kong.Parse(cfg, kong.Name("pierre"), - kong.Description("AI-powered Pull Request reviewer for Bitbucket"), + kong.Description("AI-powered Pull Request reviewer"), kong.UsageOnError(), kong.Configuration(kongyaml.Loader, "config.yaml", defaultConfig), ) - // Initialize Bitbucket Adapter - bitbucket := gitadapters.NewBitbucket(cfg.Bitbucket.BaseURL, cfg.Bitbucket.Token) + // Auto-detect provider + provider := cfg.GitProvider + if provider == "" { + if cfg.Bitbucket.BaseURL != "" && cfg.Gitea.BaseURL == "" { + provider = "bitbucket" + } else if cfg.Gitea.BaseURL != "" && cfg.Bitbucket.BaseURL == "" { + provider = "gitea" + } else if cfg.Bitbucket.BaseURL != "" && cfg.Gitea.BaseURL != "" { + log.Fatal("Multiple git providers configured. Please specify one using --git-provider.") + } else { + log.Fatal("No git provider configured. Please provide Bitbucket or Gitea configuration.") + } + } - // Fetch Diff using positional args - diff, err := bitbucket.GetDiff(cfg.Bitbucket.Project, cfg.Bitbucket.Repo, cfg.Bitbucket.PRID) + var adapter gitadapters.Adapter + switch provider { + case "bitbucket": + if cfg.Bitbucket.BaseURL == "" { + log.Fatal("Bitbucket Base URL is required when using bitbucket provider.") + } + adapter = gitadapters.NewBitbucket(cfg.Bitbucket.BaseURL, cfg.Bitbucket.Token) + case "gitea": + if cfg.Gitea.BaseURL == "" { + log.Fatal("Gitea Base URL is required when using gitea provider.") + } + adapter, err = gitea.New(cfg.Gitea.BaseURL, cfg.Gitea.Token) + if err != nil { + log.Fatalf("Error initializing Gitea adapter: %v", err) + } + default: + log.Fatalf("Unknown git provider: %s", provider) + } + + // Fetch Diff using positional args from shared RepoArgs + diff, err := adapter.GetDiff(cfg.Repo.Owner, cfg.Repo.Repo, cfg.Repo.PRID) if err != nil { log.Fatalf("Error fetching diff: %v", err) } // Initialize AI Adapter - - var ai pierre.ChatAdapter + var ai chatter.ChatAdapter switch cfg.LLM.Provider { case "gemini": @@ -91,4 +131,6 @@ func main() { fmt.Printf("File: %s\nLine: %d\nMessage: %s\n%s\n", c.File, c.Line, c.Message, "---") } + + _ = ctx } diff --git a/config.example.yaml b/config.example.yaml index 14f5a30..af5053f 100644 --- a/config.example.yaml +++ b/config.example.yaml @@ -1,12 +1,19 @@ +git-provider: "bitbucket" # Optional if only one is configured (bitbucket or gitea) + bitbucket: base-url: "https://bitbucket.your-org.ch" token: "BMTY4OTU0NjU3OTo..." - # Positional defaults (optional) - project: "APP" - repo: "api-gateway" - prid: 45 + +gitea: + base-url: "https://gitea.com" + token: "your-gitea-token" + +# Shared positional defaults (optional) +owner: "APP" +repo: "api-gateway" +prid: 45 llm: provider: "gemini" api-key: "AIzaSy..." - model: "gemini-2.0-flash" + model: "gemini-2.0-flash" \ No newline at end of file diff --git a/go.mod b/go.mod index 122402f..288a542 100644 --- a/go.mod +++ b/go.mod @@ -3,6 +3,9 @@ module bitbucket.bit.admin.ch/scm/~u80859501/pierre-bot go 1.25.0 require ( + code.gitea.io/sdk/gitea v0.23.2 + github.com/alecthomas/kong v1.14.0 + github.com/alecthomas/kong-yaml v0.2.0 github.com/google/generative-ai-go v0.20.1 github.com/ollama/ollama v0.16.0 google.golang.org/api v0.186.0 @@ -15,11 +18,12 @@ require ( cloud.google.com/go/auth/oauth2adapt v0.2.2 // indirect cloud.google.com/go/compute/metadata v0.3.0 // indirect cloud.google.com/go/longrunning v0.5.7 // indirect - github.com/alecthomas/kong v1.14.0 // indirect - github.com/alecthomas/kong-yaml v0.2.0 // indirect + github.com/42wim/httpsig v1.2.3 // indirect github.com/bahlo/generic-list-go v0.2.0 // indirect github.com/buger/jsonparser v1.1.1 // indirect + github.com/davidmz/go-pageant v1.0.2 // indirect github.com/felixge/httpsnoop v1.0.4 // indirect + github.com/go-fed/httpsig v1.1.0 // indirect github.com/go-logr/logr v1.4.1 // indirect github.com/go-logr/stdr v1.2.2 // indirect github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da // indirect @@ -28,7 +32,7 @@ require ( github.com/google/uuid v1.6.0 // indirect github.com/googleapis/enterprise-certificate-proxy v0.3.2 // indirect github.com/googleapis/gax-go/v2 v2.12.5 // indirect - github.com/invopop/jsonschema v0.13.0 // indirect + github.com/hashicorp/go-version v1.7.0 // indirect github.com/mailru/easyjson v0.7.7 // indirect github.com/wk8/go-ordered-map/v2 v2.1.8 // indirect go.opencensus.io v0.24.0 // indirect diff --git a/go.sum b/go.sum index cf8de47..7e33b82 100644 --- a/go.sum +++ b/go.sum @@ -11,11 +11,19 @@ cloud.google.com/go/compute/metadata v0.3.0 h1:Tz+eQXMEqDIKRsmY3cHTL6FVaynIjX2Qx cloud.google.com/go/compute/metadata v0.3.0/go.mod h1:zFmK7XCadkQkj6TtorcaGlCW1hT1fIilQDwofLpJ20k= cloud.google.com/go/longrunning v0.5.7 h1:WLbHekDbjK1fVFD3ibpFFVoyizlLRl73I7YKuAKilhU= cloud.google.com/go/longrunning v0.5.7/go.mod h1:8GClkudohy1Fxm3owmBGid8W0pSgodEMwEAztp38Xng= +code.gitea.io/sdk/gitea v0.23.2 h1:iJB1FDmLegwfwjX8gotBDHdPSbk/ZR8V9VmEJaVsJYg= +code.gitea.io/sdk/gitea v0.23.2/go.mod h1:yyF5+GhljqvA30sRDreoyHILruNiy4ASufugzYg0VHM= +github.com/42wim/httpsig v1.2.3 h1:xb0YyWhkYj57SPtfSttIobJUPJZB9as1nsfo7KWVcEs= +github.com/42wim/httpsig v1.2.3/go.mod h1:nZq9OlYKDrUBhptd77IHx4/sZZD+IxTBADvAPI9G/EM= github.com/BurntSushi/toml v0.3.1/go.mod h1:xHWCNGjB5oqiDr8zfno3MHue2Ht5sIBksp03qcyfWMU= +github.com/alecthomas/assert/v2 v2.11.0 h1:2Q9r3ki8+JYXvGsDyBXwH3LcJ+WK5D0gc5E8vS6K3D0= +github.com/alecthomas/assert/v2 v2.11.0/go.mod h1:Bze95FyfUr7x34QZrjL+XP+0qgp/zg8yS+TtBj1WA3k= github.com/alecthomas/kong v1.14.0 h1:gFgEUZWu2ZmZ+UhyZ1bDhuutbKN1nTtJTwh19Wsn21s= github.com/alecthomas/kong v1.14.0/go.mod h1:wrlbXem1CWqUV5Vbmss5ISYhsVPkBb1Yo7YKJghju2I= github.com/alecthomas/kong-yaml v0.2.0 h1:iiVVqVttmOsHKawlaW/TljPsjaEv1O4ODx6dloSA58Y= github.com/alecthomas/kong-yaml v0.2.0/go.mod h1:vMvOIy+wpB49MCZ0TA3KMts38Mu9YfRP03Q1StN69/g= +github.com/alecthomas/repr v0.5.2 h1:SU73FTI9D1P5UNtvseffFSGmdNci/O6RsqzeXJtP0Qs= +github.com/alecthomas/repr v0.5.2/go.mod h1:Fr0507jx4eOXV7AlPV6AVZLYrLIuIeSOWtW57eE/O/4= github.com/bahlo/generic-list-go v0.2.0 h1:5sz/EEAK+ls5wF+NeqDpk5+iNdMDXrh3z3nPnH1Wvgk= github.com/bahlo/generic-list-go v0.2.0/go.mod h1:2KvAjgMlE5NNynlg/5iLrrCCZ2+5xWbdbCW3pNTGyYg= github.com/buger/jsonparser v1.1.1 h1:2PnMjfWD7wBILjqQbt530v576A/cAbQvEW9gGIpYMUs= @@ -26,12 +34,16 @@ github.com/cncf/udpa/go v0.0.0-20191209042840-269d4d468f6f/go.mod h1:M8M6+tZqaGX github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38= github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c= github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38= +github.com/davidmz/go-pageant v1.0.2 h1:bPblRCh5jGU+Uptpz6LgMZGD5hJoOt7otgT454WvHn0= +github.com/davidmz/go-pageant v1.0.2/go.mod h1:P2EDDnMqIwG5Rrp05dTRITj9z2zpGcD9efWSkTNKLIE= github.com/envoyproxy/go-control-plane v0.9.0/go.mod h1:YTl/9mNaCwkRvm6d1a2C3ymFceY/DCBVvsKhRF0iEA4= github.com/envoyproxy/go-control-plane v0.9.1-0.20191026205805-5f8ba28d4473/go.mod h1:YTl/9mNaCwkRvm6d1a2C3ymFceY/DCBVvsKhRF0iEA4= github.com/envoyproxy/go-control-plane v0.9.4/go.mod h1:6rpuAdCZL397s3pYoYcLgu1mIlRU8Am5FuJP05cCM98= github.com/envoyproxy/protoc-gen-validate v0.1.0/go.mod h1:iSmxcyjqTsJpI2R4NaDN7+kN2VEUnK/pcBlmesArF7c= github.com/felixge/httpsnoop v1.0.4 h1:NFTV2Zj1bL4mc9sqWACXbQFVBBg2W3GPvqp8/ESS2Wg= github.com/felixge/httpsnoop v1.0.4/go.mod h1:m8KPJKqk1gH5J9DgRY2ASl2lWCfGKXixSwevea8zH2U= +github.com/go-fed/httpsig v1.1.0 h1:9M+hb0jkEICD8/cAiNqEB66R87tTINszBRTjwjQzWcI= +github.com/go-fed/httpsig v1.1.0/go.mod h1:RCMrTZvN1bJYtofsG4rd5NaO5obxQ5xBkdiS7xsT7bM= github.com/go-logr/logr v1.2.2/go.mod h1:jdQByPbusPIv2/zmleS9BjJVeZ6kBagPoEUsqbVz/1A= github.com/go-logr/logr v1.4.1 h1:pKouT5E8xu9zeFC39JXRDukb6JFQPXM5p5I91188VAQ= github.com/go-logr/logr v1.4.1/go.mod h1:9T104GzyrTigFIr8wt5mBrctHMim0Nb2HLGrmQ40KvY= @@ -72,9 +84,15 @@ github.com/googleapis/enterprise-certificate-proxy v0.3.2 h1:Vie5ybvEvT75RniqhfF github.com/googleapis/enterprise-certificate-proxy v0.3.2/go.mod h1:VLSiSSBs/ksPL8kq3OBOQ6WRI2QnaFynd1DCjZ62+V0= github.com/googleapis/gax-go/v2 v2.12.5 h1:8gw9KZK8TiVKB6q3zHY3SBzLnrGp6HQjyfYBYGmXdxA= github.com/googleapis/gax-go/v2 v2.12.5/go.mod h1:BUDKcWo+RaKq5SC9vVYL0wLADa3VcfswbOMMRmB9H3E= -github.com/invopop/jsonschema v0.13.0 h1:KvpoAJWEjR3uD9Kbm2HWJmqsEaHt8lBUpd0qHcIi21E= -github.com/invopop/jsonschema v0.13.0/go.mod h1:ffZ5Km5SWWRAIN6wbDXItl95euhFz2uON45H2qjYt+0= +github.com/hashicorp/go-version v1.7.0 h1:5tqGy27NaOTB8yJKUZELlFAS/LTKJkrmONwQKeRZfjY= +github.com/hashicorp/go-version v1.7.0/go.mod h1:fltr4n8CU8Ke44wwGCBoEymUuxUHl09ZGVZPK5anwXA= +github.com/hexops/gotextdiff v1.0.3 h1:gitA9+qJrrTCsiCl7+kh75nPqQt1cx4ZkudSTLoUqJM= +github.com/hexops/gotextdiff v1.0.3/go.mod h1:pSWU5MAI3yDq+fZBTazCSJysOMbxWL1BSow5/V2vxeg= github.com/josharian/intern v1.0.0/go.mod h1:5DoeVV0s6jJacbCEi61lwdGj/aVlrQvzHFFd8Hwg//Y= +github.com/kr/pretty v0.1.0 h1:L/CwN0zerZDmRFUapSPitk6f+Q3+0za1rQkzVuMiMFI= +github.com/kr/pretty v0.1.0/go.mod h1:dAy3ld7l9f0ibDNOQOHHMYYIIbhfbHSm3C4ZsoJORNo= +github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY= +github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE= github.com/mailru/easyjson v0.7.7 h1:UGYAvKxe3sBsEDzO8ZeWOSlIQfWFlxbzLZe7hwFURr0= github.com/mailru/easyjson v0.7.7/go.mod h1:xzfreul335JAWq5oZzymOObrkdz5UnU4kGfJJLY9Nlc= github.com/ollama/ollama v0.16.0 h1:wDrjgZvx+ej1iYrD//q7crGRA4b4482WZodRYc7oQTI= @@ -106,6 +124,7 @@ go.opentelemetry.io/otel/trace v1.26.0 h1:1ieeAUb4y0TE26jUFrCIXKpTuVK7uJGN9/Z/2L go.opentelemetry.io/otel/trace v1.26.0/go.mod h1:4iDxvGDQuUkHve82hJJ8UqrwswHYsZuWCBllGV2U2y0= golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w= golang.org/x/crypto v0.0.0-20200622213623-75b288015ac9/go.mod h1:LzIPMQfyMNhhGPhUkYOs5KpL4U8rLKemX1yGLhDgUto= +golang.org/x/crypto v0.0.0-20210513164829-c07d793c2f9a/go.mod h1:P+XmwS30IXTQdn5tA2iutPOUgjI07+tq3H3K9MVA1s8= golang.org/x/crypto v0.43.0 h1:dduJYIi3A3KOfdGOHX8AVZ/jGiyPa3IbBozJ5kNuE04= golang.org/x/crypto v0.43.0/go.mod h1:BFbav4mRNlXJL4wNeejLpWxB7wMbc79PdRGhWKncxR0= golang.org/x/exp v0.0.0-20190121172915-509febef88a4/go.mod h1:CJ0aWSM057203Lf6IL+f9T1iT9GByDxfZKAQTCR3kQA= @@ -118,6 +137,7 @@ golang.org/x/net v0.0.0-20190213061140-3a22650c66bd/go.mod h1:mL1N/T3taQHkDXs73r golang.org/x/net v0.0.0-20190311183353-d8887717615a/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg= golang.org/x/net v0.0.0-20190404232315-eb5bcb51f2a3/go.mod h1:t9HGtf8HONx5eT2rtn7q6eTqICYqUVnKs3thJo3Qplg= golang.org/x/net v0.0.0-20201110031124-69a78807bb2b/go.mod h1:sp8m0HH+o8qH0wwXwYZr8TS3Oi6o0r6Gce1SSxlDquU= +golang.org/x/net v0.0.0-20210226172049-e18ecbb05110/go.mod h1:m0MpNAwzfU5UDzcl9v0D8zg8gWTRqZa9RBIspLL5mdg= golang.org/x/net v0.46.0 h1:giFlY12I07fugqwPuWJi68oOnpfqFnJIJzaIIm2JVV4= golang.org/x/net v0.46.0/go.mod h1:Q9BGdFy1y4nkUwiLvT5qtyhAnEHgnQ/zd8PfU6nc210= golang.org/x/oauth2 v0.0.0-20180821212333-d2e6202438be/go.mod h1:N/0e6XlmueqKjAGxoOufVs8QHGRruUQn6yWY3a++T0U= @@ -132,8 +152,10 @@ golang.org/x/sys v0.0.0-20180830151530-49385e6e1522/go.mod h1:STP8DvDyc/dI5b8T5h golang.org/x/sys v0.0.0-20190215142949-d0b11bdaac8a/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY= golang.org/x/sys v0.0.0-20190412213103-97732733099d/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= golang.org/x/sys v0.0.0-20200930185726-fdedc70b468f/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= +golang.org/x/sys v0.0.0-20201119102817-f84b799fce68/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= golang.org/x/sys v0.37.0 h1:fdNQudmxPjkdUTPnLn5mdQv7Zwvbvpaxqs831goi9kQ= golang.org/x/sys v0.37.0/go.mod h1:OgkHotnGiDImocRcuBABYBEXf8A9a87e/uXjp9XT3ks= +golang.org/x/term v0.0.0-20201126162022-7de9c90e9dd1/go.mod h1:bj7SfCRtBDWHUb9snDiAeCFNEtKQo2Wmx5Cou7ajbmo= golang.org/x/term v0.36.0 h1:zMPR+aF8gfksFprF/Nc/rd1wRS1EI6nDBGyWAvDzx2Q= golang.org/x/term v0.36.0/go.mod h1:Qu394IJq6V6dCBRgwqshf3mPF85AqzYEzofzRdZkWss= golang.org/x/text v0.3.0/go.mod h1:NqM8EUOU14njkJ3fqMW+pc6Ldnwhi/IjpwHt7yyuwOQ= @@ -177,9 +199,9 @@ google.golang.org/protobuf v1.23.1-0.20200526195155-81db48ad09cc/go.mod h1:EGpAD google.golang.org/protobuf v1.25.0/go.mod h1:9JNX74DMeImyA3h4bdi1ymwjUzf21/xIlbajtzgsN7c= google.golang.org/protobuf v1.34.2 h1:6xV6lTsCfpGD21XK49h7MhtcApnLqkfYgPcdHftf6hg= google.golang.org/protobuf v1.34.2/go.mod h1:qYOHts0dSfpeUzUFpOMr/WGzszTmLH+DiWniOlNbLDw= -gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM= gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0= gopkg.in/check.v1 v1.0.0-20190902080502-41f04d3bba15 h1:YR8cESwS4TdDjEe65xsg0ogRM/Nc3DYOhEAlW+xobZo= +gopkg.in/check.v1 v1.0.0-20190902080502-41f04d3bba15/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0= gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM= gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA= gopkg.in/yaml.v3 v3.0.1/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM= diff --git a/internal/chatter/model.go b/internal/chatter/model.go index 962682c..c103607 100644 --- a/internal/chatter/model.go +++ b/internal/chatter/model.go @@ -1,5 +1,9 @@ package chatter +import ( + "context" +) + // Role defines who sent the message type Role string @@ -15,6 +19,10 @@ type Message struct { Content string } +type ChatAdapter interface { + GenerateStructured(ctx context.Context, messages []Message, target interface{}) error +} + // PredictionConfig allows for per-request overrides type PredictionConfig struct { Temperature float64 diff --git a/internal/gitadapters/adapter.go b/internal/gitadapters/adapter.go new file mode 100644 index 0000000..75b5add --- /dev/null +++ b/internal/gitadapters/adapter.go @@ -0,0 +1,7 @@ +package gitadapters + +import "io" + +type Adapter interface { + GetDiff(owner, repo string, prID int) (io.Reader, error) +} diff --git a/internal/gitadapters/gitea/adapter.go b/internal/gitadapters/gitea/adapter.go new file mode 100644 index 0000000..6ce5286 --- /dev/null +++ b/internal/gitadapters/gitea/adapter.go @@ -0,0 +1,30 @@ +package gitea + +import ( + "bytes" + "io" + + "code.gitea.io/sdk/gitea" +) + +type Adapter struct { + client *gitea.Client +} + +func New(baseURL, token string) (*Adapter, error) { + client, err := gitea.NewClient(baseURL, gitea.SetToken(token)) + if err != nil { + return nil, err + } + return &Adapter{ + client: client, + }, nil +} + +func (g *Adapter) GetDiff(owner, repo string, prID int) (io.Reader, error) { + diff, _, err := g.client.GetPullRequestDiff(owner, repo, int64(prID), gitea.PullRequestDiffOptions{}) + if err != nil { + return nil, err + } + return bytes.NewReader(diff), nil +} diff --git a/internal/pierre/judge.go b/internal/pierre/judge.go index de91b3c..10a89a2 100644 --- a/internal/pierre/judge.go +++ b/internal/pierre/judge.go @@ -14,11 +14,7 @@ type Comment struct { Message string `json:"message"` } -type ChatAdapter interface { - GenerateStructured(ctx context.Context, messages []chatter.Message, target interface{}) error -} - -func JudgePR(ctx context.Context, chat ChatAdapter, diff io.Reader) (comments []Comment, err error) { +func JudgePR(ctx context.Context, chat chatter.ChatAdapter, diff io.Reader) (comments []Comment, err error) { diffBytes, err := io.ReadAll(diff) if err != nil { return nil, fmt.Errorf("failed to read diff: %w", err)