diff --git a/internal/gitprovider/github/client.go b/internal/gitprovider/github/client.go index 06a9be67..4fe0419c 100644 --- a/internal/gitprovider/github/client.go +++ b/internal/gitprovider/github/client.go @@ -24,10 +24,10 @@ const ( // default client. All responses here are JSON payloads, so two minutes is // generous — the bound exists because an unbounded client parks the whole // run indefinitely on a connection the server's edge silently abandoned. - defaultHTTPTimeout = 2 * time.Minute - gitHubAppRefreshSkew = 5 * time.Minute - gitHubAppJWTBackdate = 60 * time.Second - gitHubAppJWTLifetime = 9 * time.Minute + defaultHTTPTimeout = 2 * time.Minute + gitHubAppRefreshSkew = 5 * time.Minute + gitHubAppJWTBackdate = 60 * time.Second + gitHubAppJWTLifetime = 9 * time.Minute ) var ( diff --git a/internal/gitprovider/github/rest_writes_test.go b/internal/gitprovider/github/rest_writes_test.go index 6a15a3b7..3c2e9b03 100644 --- a/internal/gitprovider/github/rest_writes_test.go +++ b/internal/gitprovider/github/rest_writes_test.go @@ -65,10 +65,10 @@ func TestRESTWriteMethodsMapRequests(t *testing.T) { t.Fatalf("review comment[0] = %#v, want object", comments[0]) } requireJSONExact(t, comment, map[string]any{ - "body": "line body", - "path": "dir/file.go", - "side": "RIGHT", - "line": float64(9), + "body": "line body", + "path": "dir/file.go", + "side": "RIGHT", + "line": float64(9), }) writeJSON(t, w, map[string]any{"id": 301}) default: diff --git a/internal/view/agents_test.go b/internal/view/agents_test.go index cad64a27..003ed077 100644 --- a/internal/view/agents_test.go +++ b/internal/view/agents_test.go @@ -135,17 +135,17 @@ func TestRenderAgentsShowJSON(t *testing.T) { func testAgentsCatalog() agents.Catalog { return agents.Catalog{ Agents: []agents.Agent{{ - ID: "harness:architecture", - Name: "architecture", - Category: agents.Category{Name: "harness", Description: "Engineering reviewers.", Owner: "rianjs"}, - Description: "Reviews architecture.", - ModelTier: "medium", - Effort: "medium", - FileGlobs: []string{"**/*.go"}, - AppliesWhen: []string{"Go files changed"}, + ID: "harness:architecture", + Name: "architecture", + Category: agents.Category{Name: "harness", Description: "Engineering reviewers.", Owner: "rianjs"}, + Description: "Reviews architecture.", + ModelTier: "medium", + Effort: "medium", + FileGlobs: []string{"**/*.go"}, + AppliesWhen: []string{"Go files changed"}, RequiredOnMatch: true, - Prompt: "Read the diff carefully.\n", - Provenance: agents.Provenance{Kind: agents.SourceRepo, Ref: "refs/heads/main", SHA: "abc123456789"}, + Prompt: "Read the diff carefully.\n", + Provenance: agents.Provenance{Kind: agents.SourceRepo, Ref: "refs/heads/main", SHA: "abc123456789"}, }}, Repo: &agents.RepoInfo{Ref: "refs/heads/main", SHA: "abc123456789", Provenance: "repo@refs/heads/main:abc123456789"}, }