From 61d2b8440afbdfb1ca6a22db0d57761307fdbe48 Mon Sep 17 00:00:00 2001 From: Virgil Date: Thu, 2 Apr 2026 08:50:37 +0000 Subject: [PATCH] fix(forge): wrap helper parse errors Co-Authored-By: Virgil --- actions.go | 3 ++- admin.go | 3 ++- contents.go | 3 ++- notifications.go | 3 ++- pulls.go | 3 ++- repos.go | 7 ++++--- users.go | 3 ++- 7 files changed, 16 insertions(+), 9 deletions(-) diff --git a/actions.go b/actions.go index 23a69b4..bb26a99 100644 --- a/actions.go +++ b/actions.go @@ -6,6 +6,7 @@ import ( "net/url" "strconv" + core "dappco.re/go/core" "dappco.re/go/core/forge/types" ) @@ -214,7 +215,7 @@ func (s *ActionsService) ListRepoTasks(ctx context.Context, owner, repo string, if opts.Page > 0 || opts.Limit > 0 { u, err := url.Parse(path) if err != nil { - return nil, err + return nil, core.E("ActionsService.ListRepoTasks", "forge: parse path", err) } q := u.Query() if opts.Page > 0 { diff --git a/admin.go b/admin.go index c7f7f19..331c18f 100644 --- a/admin.go +++ b/admin.go @@ -7,6 +7,7 @@ import ( "net/url" "strconv" + core "dappco.re/go/core" "dappco.re/go/core/forge/types" ) @@ -439,7 +440,7 @@ func (s *AdminService) ListActionsRuns(ctx context.Context, filters AdminActions u, err := url.Parse("/api/v1/admin/actions/runs") if err != nil { - return nil, err + return nil, core.E("AdminService.ListActionsRuns", "forge: parse path", err) } q := u.Query() diff --git a/contents.go b/contents.go index ac08739..78d5c21 100644 --- a/contents.go +++ b/contents.go @@ -5,6 +5,7 @@ import ( "iter" "net/url" + core "dappco.re/go/core" "dappco.re/go/core/forge/types" ) @@ -30,7 +31,7 @@ func (s *ContentService) ListContents(ctx context.Context, owner, repo, ref stri if ref != "" { u, err := url.Parse(path) if err != nil { - return nil, err + return nil, core.E("ContentService.ListContents", "forge: parse path", err) } q := u.Query() q.Set("ref", ref) diff --git a/notifications.go b/notifications.go index 661de6d..cad311d 100644 --- a/notifications.go +++ b/notifications.go @@ -8,6 +8,7 @@ import ( "strconv" "time" + core "dappco.re/go/core" "dappco.re/go/core/forge/types" ) @@ -285,7 +286,7 @@ func (s *NotificationService) listPage(ctx context.Context, path string, opts Li u, err := url.Parse(path) if err != nil { - return nil, err + return nil, core.E("NotificationService.listPage", "forge: parse path", err) } values := u.Query() diff --git a/pulls.go b/pulls.go index 7916a34..56cd6eb 100644 --- a/pulls.go +++ b/pulls.go @@ -6,6 +6,7 @@ import ( "net/url" "strconv" + core "dappco.re/go/core" "dappco.re/go/core/forge/types" ) @@ -234,7 +235,7 @@ func (s *PullService) listPage(ctx context.Context, owner, repo string, opts Lis path := ResolvePath("/api/v1/repos/{owner}/{repo}/pulls", pathParams("owner", owner, "repo", repo)) u, err := url.Parse(path) if err != nil { - return nil, err + return nil, core.E("PullService.listPage", "forge: parse path", err) } values := u.Query() diff --git a/repos.go b/repos.go index 5de4411..8b826a8 100644 --- a/repos.go +++ b/repos.go @@ -8,6 +8,7 @@ import ( "strconv" "time" + core "dappco.re/go/core" "dappco.re/go/core/forge/types" ) @@ -466,7 +467,7 @@ func (s *RepoService) GetRawFileOrLFS(ctx context.Context, owner, repo, filepath if ref != "" { u, err := url.Parse(path) if err != nil { - return nil, err + return nil, core.E("RepoService.GetRawFileOrLFS", "forge: parse path", err) } q := u.Query() q.Set("ref", ref) @@ -482,7 +483,7 @@ func (s *RepoService) GetEditorConfig(ctx context.Context, owner, repo, filepath if ref != "" { u, err := url.Parse(path) if err != nil { - return err + return core.E("RepoService.GetEditorConfig", "forge: parse path", err) } q := u.Query() q.Set("ref", ref) @@ -651,7 +652,7 @@ func (s *RepoService) SearchRepositoriesPage(ctx context.Context, query string, u, err := url.Parse("/api/v1/repos/search") if err != nil { - return nil, err + return nil, core.E("RepoService.SearchRepositoriesPage", "forge: parse path", err) } q := u.Query() diff --git a/users.go b/users.go index 86d9125..d0254a8 100644 --- a/users.go +++ b/users.go @@ -7,6 +7,7 @@ import ( "net/url" "strconv" + core "dappco.re/go/core" "dappco.re/go/core/forge/types" ) @@ -132,7 +133,7 @@ func (s *UserService) SearchUsersPage(ctx context.Context, query string, pageOpt u, err := url.Parse("/api/v1/users/search") if err != nil { - return nil, err + return nil, core.E("UserService.SearchUsersPage", "forge: parse path", err) } q := u.Query()