feat: add remote links to issue view (ankitpokhrel#873)

Cherry-picked from upstream PR ankitpokhrel#873 by vkareh.
Shows external/remote links in the issue view below linked issues.

Author: Ismar Iljazovic

api/client.go

@@ -108,6 +108,7 @@ func ProxyGetIssueRaw(c *jira.Client, key string) (string, error) {
 // ProxyGetIssue uses either a v2 or v3 version of the Jira GET /issue/{key}
 // endpoint to fetch the issue details based on configured installation type.
 // Defaults to v3 if installation type is not defined in the config.
+// Also fetches remote links for the issue.
 func ProxyGetIssue(c *jira.Client, key string, opts ...filter.Filter) (*jira.Issue, error) {
 	var (
 		iss *jira.Issue
@@ -122,6 +123,19 @@ func ProxyGetIssue(c *jira.Client, key string, opts ...filter.Filter) (*jira.Iss
 		iss, err = c.GetIssue(key, opts...)
 	}
 
+	if err != nil {
+		return iss, err
+	}
+
+	// Fetch remote links for the issue
+	remoteLinks, err := c.GetIssueRemoteLinks(key)
+	if err != nil {
+		// Don't fail the entire request if remote links can't be fetched
+		// Just log and continue without remote links
+		remoteLinks = []jira.RemoteLink{}
+	}
+	iss.Fields.RemoteLinks = remoteLinks
+
 	return iss, err
 }
 

internal/cmd/issue/view/view.go

@@ -2,6 +2,7 @@
 package view
 
 import (
+	"encoding/json"
 	"fmt"
 
 	"github.com/spf13/cobra"
@@ -83,7 +84,20 @@ func viewRaw(cmd *cobra.Command, args []string) {
 		defer s.Stop()
 
 		client := api.DefaultClient(debug)
-		return api.ProxyGetIssueRaw(client, key)
+
+		// Fetch the issue with remote links included (same as structured view)
+		issue, err := api.ProxyGetIssue(client, key)
+		if err != nil {
+			return "", err
+		}
+
+		// Convert back to JSON for raw output
+		rawJSON, err := json.MarshalIndent(issue, "", "  ")
+		if err != nil {
+			return "", err
+		}
+
+		return string(rawJSON), nil
 	}()
 	cmdutil.ExitIfError(err)
 

internal/view/issue.go

@@ -115,6 +115,9 @@ func (i Issue) String() string {
 	if len(i.Data.Fields.IssueLinks) > 0 {
 		s.WriteString(fmt.Sprintf("\n\n%s\n\n%s\n", i.separator("Linked Issues"), i.linkedIssues()))
 	}
+	if len(i.Data.Fields.RemoteLinks) > 0 {
+		s.WriteString(fmt.Sprintf("\n\n%s\n\n%s\n", i.separator("External Links"), i.remoteLinks()))
+	}
 	total := i.Data.Fields.Comment.Total
 	if total > 0 && i.Options.NumComments > 0 {
 		sep := fmt.Sprintf("%d Comments", total)
@@ -172,6 +175,17 @@ func (i Issue) fragments() []fragment {
 		)
 	}
 
+	if len(i.Data.Fields.RemoteLinks) > 0 {
+		scraps = append(
+			scraps,
+			newBlankFragment(singleLineSpacing),
+			fragment{Body: i.separator("External Links")},
+			newBlankFragment(sectionSpacing),
+			fragment{Body: i.remoteLinks()},
+			newBlankFragment(singleLineSpacing),
+		)
+	}
+
 	if i.Data.Fields.Comment.Total > 0 && i.Options.NumComments > 0 {
 		scraps = append(
 			scraps,
@@ -392,6 +406,40 @@ func (i Issue) linkedIssues() string {
 	return linked.String()
 }
 
+func (i Issue) remoteLinks() string {
+	if len(i.Data.Fields.RemoteLinks) == 0 {
+		return ""
+	}
+
+	var (
+		remote      strings.Builder
+		maxTitleLen int
+		summaryLen  = defaultSummaryLength
+	)
+
+	// Calculate max lengths for formatting
+	for _, link := range i.Data.Fields.RemoteLinks {
+		maxTitleLen = max(len(link.Object.Title), maxTitleLen)
+	}
+
+	if maxTitleLen < summaryLen {
+		summaryLen = maxTitleLen
+	}
+
+	remote.WriteString("\n")
+	for _, link := range i.Data.Fields.RemoteLinks {
+		remote.WriteString(
+			fmt.Sprintf(
+				"  %s\n  %s\n\n",
+				coloredOut(shortenAndPad(link.Object.Title, summaryLen), color.FgCyan, color.Bold),
+				coloredOut(link.Object.URL, color.FgBlue, color.Underline),
+			),
+		)
+	}
+
+	return remote.String()
+}
+
 func (i Issue) comments() []issueComment {
 	total := i.Data.Fields.Comment.Total
 	comments := make([]issueComment, 0, total)

pkg/jira/issue.go

@@ -476,6 +476,37 @@ func (c *Client) RemoteLinkIssue(issueID, title, url string) error {
 	return nil
 }
 
+// GetIssueRemoteLinks fetches remote links for an issue using GET /issue/{issueId}/remotelink endpoint.
+func (c *Client) GetIssueRemoteLinks(issueID string) ([]RemoteLink, error) {
+	path := fmt.Sprintf("/issue/%s/remotelink", issueID)
+
+	res, err := c.GetV2(context.Background(), path, nil)
+	if err != nil {
+		return nil, err
+	}
+	if res == nil {
+		return nil, ErrEmptyResponse
+	}
+	defer func() { _ = res.Body.Close() }()
+
+	if res.StatusCode != http.StatusOK {
+		return nil, formatUnexpectedResponse(res)
+	}
+
+	body, err := io.ReadAll(res.Body)
+	if err != nil {
+		return nil, err
+	}
+
+	var remoteLinks []RemoteLink
+	err = json.Unmarshal(body, &remoteLinks)
+	if err != nil {
+		return nil, err
+	}
+
+	return remoteLinks, nil
+}
+
 // WatchIssue adds user as a watcher using v2 version of the POST /issue/{key}/watchers endpoint.
 func (c *Client) WatchIssue(key, watcher string) error {
 	return c.watchIssue(key, watcher, apiVersion3)

pkg/jira/types.go

@@ -124,8 +124,9 @@ type IssueFields struct {
 		InwardIssue  *Issue `json:"inwardIssue,omitempty"`
 		OutwardIssue *Issue `json:"outwardIssue,omitempty"`
 	} `json:"issueLinks"`
-	Created string `json:"created"`
-	Updated string `json:"updated"`
+	RemoteLinks []RemoteLink `json:"remoteLinks,omitempty"`
+	Created     string       `json:"created"`
+	Updated     string       `json:"updated"`
 }
 
 // Field holds field info.
@@ -167,6 +168,16 @@ type IssueLinkType struct {
 	Outward string `json:"outward"`
 }
 
+// RemoteLink holds remote link info.
+type RemoteLink struct {
+	ID     int    `json:"id"`
+	Self   string `json:"self"`
+	Object struct {
+		URL   string `json:"url"`
+		Title string `json:"title"`
+	} `json:"object"`
+}
+
 // Sprint holds sprint info.
 type Sprint struct {
 	ID           int    `json:"id"`