Fix empty descriptions in PR and commit issue queries (#283)

* Fix empty descriptions in PR and commit issue queries

The PR and commit queries read the `explanation` field from the
AnalysisIssue type, which the server returns as empty. Use
`issue { shortDescription }` instead, matching the pattern used
by the default-branch query.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Change PR issues Issue field from pointer to value struct

Prevents nil pointer dereference when accessing Issue.ShortDescription,
consistent with the same pattern in run_issues.go.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Signed-off-by: Sourya Vatsyayan <sourya@deepsource.io>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: Sourya Vatsyayan <sourya@deepsource.io>

Author: 3 people

command/issues/tests/golden_files/commit_scope_response.json

@@ -18,9 +18,11 @@
                     "endLine": 42,
                     "title": "Unchecked error return value of os.ReadFile",
                     "shortcode": "GO-W1007",
-                    "explanation": "Return value of io.ReadAll is not checked for errors",
                     "category": "BUG_RISK",
-                    "severity": "MAJOR"
+                    "severity": "MAJOR",
+                    "issue": {
+                      "shortDescription": "Return value of io.ReadAll is not checked for errors"
+                    }
                   }
                 },
                 {
@@ -31,9 +33,11 @@
                     "endLine": 91,
                     "title": "HTTP request built with user-controlled URL",
                     "shortcode": "GO-S1010",
-                    "explanation": "Constructing HTTP request with user-controlled URL allows SSRF",
                     "category": "SECURITY",
-                    "severity": "MAJOR"
+                    "severity": "MAJOR",
+                    "issue": {
+                      "shortDescription": "Constructing HTTP request with user-controlled URL allows SSRF"
+                    }
                   }
                 }
               ]

command/issues/tests/golden_files/pr_scope_response.json

@@ -13,7 +13,9 @@
               "shortcode": "GO-W1007",
               "category": "BUG_RISK",
               "severity": "MAJOR",
-              "explanation": "Return value of io.ReadAll is not checked for errors"
+              "issue": {
+                "shortDescription": "Return value of io.ReadAll is not checked for errors"
+              }
             }
           },
           {
@@ -26,7 +28,9 @@
               "shortcode": "GO-S1010",
               "category": "SECURITY",
               "severity": "MAJOR",
-              "explanation": "Constructing HTTP request with user-controlled URL allows SSRF"
+              "issue": {
+                "shortDescription": "Constructing HTTP request with user-controlled URL allows SSRF"
+              }
             }
           }
         ],

deepsource/issues/queries/pr_issues.go

@@ -36,6 +36,7 @@ const fetchPRIssuesQuery = `query GetPRIssues(
             severity
             explanation
             issue {
+              shortDescription
               analyzer {
                 name
                 shortcode
@@ -83,7 +84,8 @@ type PRIssuesListResponse struct {
 						Category   string `json:"category"`
 						Severity   string `json:"severity"`
 						Explanation string `json:"explanation"`
-						Issue       *struct {
+						Issue       struct {
+							ShortDescription string `json:"shortDescription"`
 							Analyzer struct {
 								Name      string `json:"name"`
 								Shortcode string `json:"shortcode"`
@@ -142,7 +144,7 @@ func (r *PRIssuesListRequest) Do(ctx context.Context) ([]issues.Issue, error) {
 				IssueCategory: node.Category,
 				IssueSeverity: node.Severity,
 				IssueSource:   node.Source,
-				Description:   node.Explanation,
+				Description:   node.Issue.ShortDescription,
 				Location: issues.Location{
 					Path: node.Path,
 					Position: issues.Position{
@@ -151,11 +153,9 @@ func (r *PRIssuesListRequest) Do(ctx context.Context) ([]issues.Issue, error) {
 					},
 				},
 			}
-			if node.Issue != nil {
-				issue.Analyzer = issues.AnalyzerMeta{
-					Name:      node.Issue.Analyzer.Name,
-					Shortcode: node.Issue.Analyzer.Shortcode,
-				}
+			issue.Analyzer = issues.AnalyzerMeta{
+				Name:      node.Issue.Analyzer.Name,
+				Shortcode: node.Issue.Analyzer.Shortcode,
 			}
 			allIssues = append(allIssues, issue)
 		}

deepsource/issues/queries/run_issues.go

@@ -27,9 +27,11 @@ const fetchRunIssuesFlatQuery = `query GetRunIssues($commitOid: String!, $limit:
                 endLine
                 title
                 shortcode
-                explanation
                 category
                 severity
+                issue {
+                  shortDescription
+                }
               }
             }
           }
@@ -70,9 +72,11 @@ type RunIssuesFlatResponse struct {
 								EndLine   int    `json:"endLine"`
 								Title       string `json:"title"`
 								Shortcode   string `json:"shortcode"`
-								Explanation string `json:"explanation"`
 								Category    string `json:"category"`
 								Severity    string `json:"severity"`
+								Issue       struct {
+									ShortDescription string `json:"shortDescription"`
+								} `json:"issue"`
 							} `json:"node"`
 						} `json:"edges"`
 					} `json:"issues"`
@@ -119,7 +123,7 @@ func (r *RunIssuesFlatRequest) Do(ctx context.Context) ([]issues.Issue, error) {
 				IssueCategory: node.Category,
 				IssueSeverity: node.Severity,
 				IssueSource:   node.Source,
-				Description:   node.Explanation,
+				Description:   node.Issue.ShortDescription,
 				Location: issues.Location{
 					Path: node.Path,
 					Position: issues.Position{