Merge pull request 'fix: trim trailing slash from giteaURL when building review link' (#40 ) from fix/url-normalization into main

fix: trim trailing slash from giteaURL when building review link
Prevents double-slash in supersede URL if GITEA_URL ends with '/'. Aligns with how gitea.NewClient already normalizes the base URL.
2026-05-02 18:52:13 +00:00 · 2026-05-02 11:49:24 -07:00
1 changed files with 1 additions and 1 deletions
@@ -355,7 +355,7 @@ func main() {

 	// Supersede old review with link to the new one
 	if existingReview != nil && existingCommentID > 0 {
-		newReviewURL := fmt.Sprintf("%s/%s/%s/pulls/%d#pullrequestreview-%d", *giteaURL, owner, repoName, prNumber, posted.ID)
+		newReviewURL := fmt.Sprintf("%s/%s/%s/pulls/%d#pullrequestreview-%d", strings.TrimRight(*giteaURL, "/"), owner, repoName, prNumber, posted.ID)
 		supersededBody := buildSupersededBody(existingReview.Body, existingReview.CommitID, newReviewURL, sentinel)
 		if err := giteaClient.EditComment(ctx, owner, repoName, existingCommentID, supersededBody); err != nil {
 			slog.Warn("could not mark old review as superseded", "comment_id", existingCommentID, "error", err)
Author	SHA1	Message	Date
rodin	cd601bdcf4	Merge pull request 'fix: trim trailing slash from giteaURL when building review link' (#40 ) from fix/url-normalization into main CI / test (push) Successful in 14s Details CI / review (gpt-4.1, gpt, GPT_REVIEW_TOKEN) (push) Has been skipped Details CI / review (gpt-5, security, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (push) Has been skipped Details CI / review (gpt-5, sonnet, SONNET_REVIEW_TOKEN) (push) Has been skipped Details	2026-05-02 18:52:13 +00:00
Rodin	50091941e1	fix: trim trailing slash from giteaURL when building review link CI / test (pull_request) Successful in 14s Details CI / review (gpt-4.1, gpt, GPT_REVIEW_TOKEN) (pull_request) Successful in 20s Details CI / review (gpt-5, security, SECURITY_REVIEW.md, SECURITY_REVIEW_TOKEN) (pull_request) Successful in 28s Details CI / review (gpt-5, sonnet, SONNET_REVIEW_TOKEN) (pull_request) Successful in 41s Details Prevents double-slash in supersede URL if GITEA_URL ends with '/'. Aligns with how gitea.NewClient already normalizes the base URL.	2026-05-02 11:49:24 -07:00