go-scm/cmd/forge/cmd_prs.go

101 lines
2.3 KiB
Go
Raw Permalink Normal View History

// SPDX-License-Identifier: EUPL-1.2
package forge
import (
"dappco.re/go/core/cli/pkg/cli"
forgejo "codeberg.org/mvdkleijn/forgejo-sdk/forgejo/v2"
fg "dappco.re/go/core/scm/forge"
fix(review): address CodeRabbit PR #2 findings Critical/Major: - Remove dead functions syncRepoNameFromArg and repoNameFromArg (used url pkg without import, would cause compile error) - Migrate forge.lthn.ai/core/config → dappco.re/go/core/config in forge/config.go and gitea/config.go - Propagate ListIssueCommentsIter errors in forge/meta.go and gitea/meta.go (was silently returning truncated count) - Add RedactedToken() to gitea/client.go to avoid exposing raw API tokens - Add 30s timeout to http.DefaultClient usage in gitea/prs.go via package-level httpClient - Fix stringsx.Fields (bufio 64KiB limit), Repeat (wrong for negative/zero), Replace (ignored n param) to match stdlib - Fix fmtx.Println to use fmt.Sprintln so spaces appear between operands - Fix filepathx.Abs to use path/filepath for OS-aware path handling; wrap Getwd error - Fix stdio.Write to return io.ErrShortWrite on partial writes - Add mutex lock to jobrunner.Journal.Query to prevent data race with Append - Add sync.RWMutex to ScmProvider; protect p.index reads/writes in pkg/api/provider.go - Fix cmd/scm/cmd_index.go: append dir to repoPaths only after ReadDir confirms existence - Fix manifest/compile.go: copy manifest before applying version override to avoid mutating caller - Fix forge/labels.go: use ListOrgLabelsIter/ListRepoLabelsIter names in iterator error logs - Wrap single-segment validation error in syncutil.ParseRepoName with function context Minor: - Fix import ordering (stdlib → forge.lthn.ai → third-party) in cmd/forge, cmd/collect, repos, cmd/gitea files - Add t.Setenv("HOME", t.TempDir()) to gitea testhelpers and forge/labels_test.go - Add iterator yield guard in forge/orgs_test.go - Convert syncutil/repo_name_test.go to table-driven tests - Use json.Marshal in pkg/api/provider_test.go instead of string concatenation - Fix test naming (redundant/conflicting _Good/_Bad suffixes) across 10 test files Co-Authored-By: Virgil <virgil@lethean.io>
2026-04-07 09:25:42 +01:00
fmt "dappco.re/go/core/scm/internal/ax/fmtx"
strings "dappco.re/go/core/scm/internal/ax/stringsx"
)
// PRs command flags.
var (
prsState string
)
// addPRsCommand adds the 'prs' subcommand for listing pull requests.
func addPRsCommand(parent *cli.Command) {
cmd := &cli.Command{
Use: "prs <owner/repo>",
Short: "List pull requests",
Long: "List pull requests for a repository.",
Args: cli.ExactArgs(1),
RunE: func(cmd *cli.Command, args []string) error {
owner, repo, err := splitOwnerRepo(args[0])
if err != nil {
return err
}
return runListPRs(owner, repo)
},
}
cmd.Flags().StringVar(&prsState, "state", "open", "Filter by state (open, closed, all)")
parent.AddCommand(cmd)
}
func runListPRs(owner, repo string) error {
client, err := fg.NewFromConfig("", "")
if err != nil {
return err
}
prs, err := client.ListPullRequests(owner, repo, prsState)
if err != nil {
return err
}
if len(prs) == 0 {
cli.Text(fmt.Sprintf("No %s pull requests in %s/%s.", prsState, owner, repo))
return nil
}
cli.Blank()
cli.Print(" %s\n\n", fmt.Sprintf("%d %s pull requests in %s/%s", len(prs), prsState, owner, repo))
for _, pr := range prs {
printForgePR(pr)
}
return nil
}
func printForgePR(pr *forgejo.PullRequest) {
num := numberStyle.Render(fmt.Sprintf("#%d", pr.Index))
title := valueStyle.Render(cli.Truncate(pr.Title, 50))
var author string
if pr.Poster != nil {
author = infoStyle.Render("@" + pr.Poster.UserName)
}
// Branch info
branch := dimStyle.Render(pr.Head.Ref + " -> " + pr.Base.Ref)
// Merge status
var status string
if pr.HasMerged {
status = successStyle.Render("merged")
} else if pr.State == forgejo.StateClosed {
status = errorStyle.Render("closed")
} else {
status = warningStyle.Render("open")
}
// Labels
var labelStr string
if len(pr.Labels) > 0 {
var labels []string
for _, l := range pr.Labels {
labels = append(labels, l.Name)
}
labelStr = " " + warningStyle.Render("["+strings.Join(labels, ", ")+"]")
}
cli.Print(" %s %s %s %s %s%s\n", num, title, author, status, branch, labelStr)
}