Normalize issue dependency ref state to match other tools

The REST issue-dependency endpoints return lower-case issue states
(open/closed), whereas the previous GraphQL implementation and the
sibling get_parent tool populate MinimalIssueRef.State from the GraphQL
IssueState enum (OPEN/CLOSED). Upper-case the state in
issueToDependencyRef so the field stays consistent across every tool that
emits a MinimalIssueRef, and guard against a nil issue while here.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: tommaso-moro

pkg/github/issue_dependencies.go

@@ -164,12 +164,17 @@ func dependencyReadResult(issues []*github.Issue, resp *github.Response) *mcp.Ca
 
 // issueToDependencyRef converts a REST issue into the compact reference used by
 // the dependency tools, deriving the "owner/repo" name from the issue's
-// repository URL.
+// repository URL. The state is upper-cased so it matches the GraphQL-sourced
+// state (e.g. "OPEN"/"CLOSED") that MinimalIssueRef carries for the other issue
+// tools such as get_parent, keeping the field consistent across tools.
 func issueToDependencyRef(issue *github.Issue) MinimalIssueRef {
+	if issue == nil {
+		return MinimalIssueRef{}
+	}
 	ref := MinimalIssueRef{
 		Number: issue.GetNumber(),
 		Title:  issue.GetTitle(),
-		State:  issue.GetState(),
+		State:  strings.ToUpper(issue.GetState()),
 		URL:    issue.GetHTMLURL(),
 	}
 	if owner, repo, ok := parseRepositoryURL(issue.GetRepositoryURL()); ok {

pkg/github/issue_dependencies_test.go

@@ -88,6 +88,7 @@ func Test_IssueDependencyRead(t *testing.T) {
 		option        MockBackendOption
 		expectedCount int
 		expectedFirst int
+		expectedState string
 		expectedNext  bool
 	}{
 		{
@@ -96,6 +97,7 @@ func Test_IssueDependencyRead(t *testing.T) {
 			option:        WithRequestMatch(endpointBlockedBy, blockedByIssues),
 			expectedCount: 1,
 			expectedFirst: 7,
+			expectedState: "OPEN",
 			expectedNext:  false,
 		},
 		{
@@ -104,6 +106,7 @@ func Test_IssueDependencyRead(t *testing.T) {
 			option:        WithRequestMatchHandler(endpointBlocking, blockingHandler),
 			expectedCount: 2,
 			expectedFirst: 8,
+			expectedState: "OPEN",
 			expectedNext:  true,
 		},
 	}
@@ -136,6 +139,9 @@ func Test_IssueDependencyRead(t *testing.T) {
 			require.Len(t, payload.Issues, tc.expectedCount)
 			assert.Equal(t, tc.expectedFirst, payload.Issues[0].Number)
 			assert.Equal(t, "owner/repo", payload.Issues[0].Repository)
+			// State is normalized to upper case to match the GraphQL-sourced
+			// state used by other MinimalIssueRef producers (e.g. get_parent).
+			assert.Equal(t, tc.expectedState, payload.Issues[0].State)
 			assert.Equal(t, tc.expectedNext, payload.PageInfo.HasNextPage)
 		})
 	}