consolidate response structs and remove unneeded message field

Author: mattdholloway

pkg/github/gists.go

@@ -259,7 +259,6 @@ func UpdateGist(getClient GetClientFn, t translations.TranslationHelperFunc) (to
 			minimalResponse := MinimalUpdateResponse{
 				URL:     updatedGist.GetHTMLURL(),
 				Updated: true,
-				Message: "Gist updated successfully",
 			}
 
 			r, err := json.Marshal(minimalResponse)

pkg/github/issues.go

@@ -873,7 +873,7 @@ func CreateIssue(getClient GetClientFn, t translations.TranslationHelperFunc) (t
 			}
 
 			// Return minimal response with just essential information
-			minimalResponse := MinimalIssueResponse{
+			minimalResponse := MinimalResourceResponse{
 				URL:    issue.GetHTMLURL(),
 				Number: issue.GetNumber(),
 				State:  issue.GetState(),
@@ -1254,7 +1254,6 @@ func UpdateIssue(getClient GetClientFn, t translations.TranslationHelperFunc) (t
 			minimalResponse := MinimalUpdateResponse{
 				URL:     updatedIssue.GetHTMLURL(),
 				Updated: true,
-				Message: "Issue updated successfully",
 			}
 
 			r, err := json.Marshal(minimalResponse)

pkg/github/issues_test.go

@@ -713,7 +713,7 @@ func Test_CreateIssue(t *testing.T) {
 			textContent := getTextResult(t, result)
 
 			// Unmarshal and verify the minimal result
-			var returnedIssue MinimalIssueResponse
+			var returnedIssue MinimalResourceResponse
 			err = json.Unmarshal([]byte(textContent.Text), &returnedIssue)
 			require.NoError(t, err)
 

pkg/github/minimal_types.go

@@ -107,17 +107,8 @@ type MinimalBranch struct {
 	Protected bool   `json:"protected"`
 }
 
-// MinimalCreateResponse represents a minimal response for resource creation operations.
-type MinimalCreateResponse struct {
-	URL    string `json:"url"`
-	ID     int64  `json:"id,omitempty"`
-	Number int    `json:"number,omitempty"`
-	Name   string `json:"name,omitempty"`
-	State  string `json:"state,omitempty"`
-}
-
-// MinimalPullRequestResponse represents a minimal response for pull request operations.
-type MinimalPullRequestResponse struct {
+// MinimalResourceResponse represents a minimal response for numbered resource operations (issues, pull requests).
+type MinimalResourceResponse struct {
 	URL    string `json:"url"`
 	Number int    `json:"number"`
 	State  string `json:"state"`
@@ -132,19 +123,10 @@ type MinimalRepositoryResponse struct {
 	FullName string `json:"full_name"`
 }
 
-// MinimalIssueResponse represents a minimal response for issue operations.
-type MinimalIssueResponse struct {
-	URL    string `json:"url"`
-	Number int    `json:"number"`
-	State  string `json:"state"`
-	Title  string `json:"title,omitempty"`
-}
-
 // MinimalUpdateResponse represents a minimal response for update operations.
 type MinimalUpdateResponse struct {
 	URL     string `json:"url"`
 	Updated bool   `json:"updated"`
-	Message string `json:"message,omitempty"`
 }
 
 // MinimalGistResponse represents a minimal response for gist operations.

pkg/github/pullrequests.go

@@ -194,7 +194,7 @@ func CreatePullRequest(getClient GetClientFn, t translations.TranslationHelperFu
 			}
 
 			// Return minimal response with just essential information
-			minimalResponse := MinimalPullRequestResponse{
+			minimalResponse := MinimalResourceResponse{
 				URL:    pr.GetHTMLURL(),
 				Number: pr.GetNumber(),
 				State:  pr.GetState(),
@@ -476,7 +476,6 @@ func UpdatePullRequest(getClient GetClientFn, getGQLClient GetGQLClientFn, t tra
 			minimalResponse := MinimalUpdateResponse{
 				URL:     finalPR.GetHTMLURL(),
 				Updated: true,
-				Message: "Pull request updated successfully",
 			}
 
 			r, err := json.Marshal(minimalResponse)

pkg/github/pullrequests_test.go

@@ -1955,7 +1955,7 @@ func Test_CreatePullRequest(t *testing.T) {
 			textContent := getTextResult(t, result)
 
 			// Unmarshal and verify the minimal result
-			var returnedPR MinimalPullRequestResponse
+			var returnedPR MinimalResourceResponse
 			err = json.Unmarshal([]byte(textContent.Text), &returnedPR)
 			require.NoError(t, err)
 			assert.Equal(t, tc.expectedPR.GetNumber(), returnedPR.Number)