add pr-agent and golangci, fix lint issue

[kazeburo]

User description

• add pr-agent.yml
• add golangci.yml and make lint
• fix some lint issues

---

PR Type

Enhancement, Bug fix

---

Description

• Add PR Agent GitHub Action for automated reviews

• Configure golangci-lint with custom linter rules

• Update Makefile with lint and verbose tests

• Fix error wrapping and resource leaks in tests

---

Diagram Walkthrough

flowchart LR
  pr_agent["PR Agent Workflow"] -- "triggers" --> golangci["golangci-lint Config"]
  golangci -- "configures" --> makefile["Makefile Targets"]
  makefile -- "runs" --> errors["Error Wrapping"]
  errors -- "validates" --> tests["Test Improvements"]

Loading

File Walkthrough

	Relevant files
Bug fix	selfsigned.go Improve error wrapping in TLS config generation                    selfsigned.go Wrapped errors using %w instead of %v for better error chaining. +5/-5
Tests	selfsigned_test.go Fix resource leak and lint warnings in HTTPS test                selfsigned_test.go Fixed HTTP response body leak by adding defer close and discard. Added missing return value check for w.Write to satisfy linters. +6/-1
Enhancement	pr-agent.yml Add PR Agent GitHub Action workflow                                            .github/workflows/pr-agent.yml Added GitHub Action workflow for PR Agent with auto-review and auto-describe features. +28/-0    Makefile Update Makefile with lint and verbose test targets              Makefile Updated test target to run verbosely and added new lint target for golangci-lint. +4/-1
Configuration changes	.golangci.yml Add golangci-lint configuration file for static analysis  .golangci.yml Configured golangci-lint with multiple linters, complexity thresholds, and error check exclusions. +35/-0

---

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ No major issues detected

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Guard response body access to prevent panic on error The defer block accesses resp.Body unconditionally. If client.Get returns an error, resp will be nil, causing a panic when the deferred function executes. Add a nil check before registering the defer to ensure safe cleanup. selfsigned_test.go [71-76] resp, err := client.Get(baseURL + "/healthz") assert.NoError(t, err, "unexpected error") -defer func() { - _, _ = io.Copy(io.Discard, resp.Body) - resp.Body.Close() -}() +if resp != nil { + defer func() { + _, _ = io.Copy(io.Discard, resp.Body) + resp.Body.Close() + }() +} Suggestion importance[1-10]: 7 __ Why: The suggestion correctly identifies a potential nil pointer dereference in the deferred function if client.Get fails. Adding a nil check prevents a panic during test execution, improving robustness and safety.	Medium