Fix list_issues tool schema validation error #7841

Author: tobiu

.github/.sync-metadata.json

@@ -1,6 +1,6 @@
 {
-  "lastSync": "2025-11-21T12:58:36.740Z",
-  "releasesLastFetched": "2025-11-21T12:58:36.750Z",
+  "lastSync": "2025-11-21T13:02:52.974Z",
+  "releasesLastFetched": "2025-11-21T13:02:52.978Z",
   "pushFailures": [],
   "issues": {
     "3789": {
@@ -6787,11 +6787,18 @@
       "contentHash": "dfe129dcdea080a6a80636853f906a7a30ca83e1ee17a420b473b5ad8d938d8e"
     },
     "7840": {
-      "state": "OPEN",
+      "state": "CLOSED",
       "path": "/Users/Shared/github/neomjs/neo/.github/ISSUE/issue-7840.md",
+      "closedAt": "2025-11-21T13:00:34Z",
+      "updatedAt": "2025-11-21T13:00:34Z",
+      "contentHash": "4213e43b1e848f61fdba176407fc6cac0f81846e3981413579953386e2fed0bb"
+    },
+    "7841": {
+      "state": "OPEN",
+      "path": "/Users/Shared/github/neomjs/neo/.github/ISSUE/issue-7841.md",
       "closedAt": null,
-      "updatedAt": "2025-11-21T12:58:27Z",
-      "contentHash": "afe24bb425a5a9cec47908d348775cc102550f606f077a1938c10bb7ae2cc107"
+      "updatedAt": "2025-11-21T13:02:43Z",
+      "contentHash": "2e9a5e1a2effecf96658c92798e77a6e779aba3bc4f12c004ef33ee7da54c6dd"
     }
   },
   "releases": {

.github/ISSUE/issue-7840.md

@@ -1,13 +1,14 @@
 ---
 id: 7840
 title: Optimize SessionService.summarizeSessions with Promise.all
-state: OPEN
+state: CLOSED
 labels:
   - enhancement
   - ai
-assignees: []
+assignees:
+  - tobiu
 createdAt: '2025-11-21T12:58:27Z'
-updatedAt: '2025-11-21T12:58:27Z'
+updatedAt: '2025-11-21T13:00:34Z'
 githubUrl: 'https://github.com/neomjs/neo/issues/7840'
 author: tobiu
 commentsCount: 0
@@ -17,6 +18,7 @@ subIssuesCompleted: 0
 subIssuesTotal: 0
 blockedBy: []
 blocking: []
+closedAt: '2025-11-21T13:00:34Z'
 ---
 # Optimize SessionService.summarizeSessions with Promise.all
 
@@ -26,4 +28,7 @@ Refactor `SessionService.summarizeSessions` to process unsummarized sessions in
 
 - 2025-11-21 @tobiu added the `enhancement` label
 - 2025-11-21 @tobiu added the `ai` label
+- 2025-11-21 @tobiu assigned to @tobiu
+- 2025-11-21 @tobiu referenced in commit `636b6ba` - "Optimize SessionService.summarizeSessions with Promise.all #7840"
+- 2025-11-21 @tobiu closed this issue
 

.github/ISSUE/issue-7841.md

@@ -0,0 +1,36 @@
+---
+id: 7841
+title: Fix list_issues tool schema validation error
+state: OPEN
+labels:
+  - bug
+  - ai
+assignees: []
+createdAt: '2025-11-21T13:02:43Z'
+updatedAt: '2025-11-21T13:02:43Z'
+githubUrl: 'https://github.com/neomjs/neo/issues/7841'
+author: tobiu
+commentsCount: 0
+parentIssue: null
+subIssues: []
+subIssuesCompleted: 0
+subIssuesTotal: 0
+blockedBy: []
+blocking: []
+---
+# Fix list_issues tool schema validation error
+
+The `list_issues` tool fails with a schema validation error because the `IssueService.listIssues` method returns raw GraphQL response structures for `labels` and `assignees` (nested under `nodes`), whereas the OpenAPI schema expects flat arrays.
+
+**Error:**
+`data/issues/0/labels must be array`
+`data/issues/0/assignees must be array`
+
+**Fix:**
+Transform the GraphQL response in `IssueService.listIssues` to map `labels.nodes` to `labels` and `assignees.nodes` to `assignees` before returning the data.
+
+## Activity Log
+
+- 2025-11-21 @tobiu added the `bug` label
+- 2025-11-21 @tobiu added the `ai` label
+

ai/mcp/server/github-workflow/services/IssueService.mjs

@@ -6,7 +6,7 @@ import logger            from '../logger.mjs';
 import {exec}            from 'child_process';
 import {promisify}       from 'util';
 import {spawn}           from 'child_process';
-import {GET_ISSUE_AND_LABEL_IDS, GET_ISSUE_PARENT, GET_BLOCKED_BY, FETCH_ISSUES_FOR_SYNC} from './queries/issueQueries.mjs';
+import {GET_ISSUE_AND_LABEL_IDS, GET_ISSUE_PARENT, GET_BLOCKED_BY, FETCH_ISSUES_FOR_SYNC, FETCH_ISSUES_LIST} from './queries/issueQueries.mjs';
 import {GET_PULL_REQUEST_ID} from './queries/pullRequestQueries.mjs';
 import {ADD_LABELS, REMOVE_LABELS, ADD_SUB_ISSUE, REMOVE_SUB_ISSUE, ADD_BLOCKED_BY, REMOVE_BLOCKED_BY, GET_ISSUE_ID, ADD_COMMENT} from './queries/mutations.mjs';
 
@@ -498,16 +498,12 @@ class IssueService extends Base {
             limit,
             cursor,
             states,
-            since           : null,
             maxLabels       : aiConfig.issueSync.maxLabelsPerIssue,
-            maxAssignees    : aiConfig.issueSync.maxAssigneesPerIssue,
-            maxComments     : aiConfig.issueSync.maxCommentsPerIssue,
-            maxSubIssues    : aiConfig.issueSync.maxSubIssuesPerIssue,
-            maxTimelineItems: aiConfig.issueSync.maxTimelineItemsPerIssue
+            maxAssignees    : aiConfig.issueSync.maxAssigneesPerIssue
         };
 
         try {
-            const data = await GraphqlService.query(FETCH_ISSUES_FOR_SYNC, variables);
+            const data = await GraphqlService.query(FETCH_ISSUES_LIST, variables);
             let issues = data.repository.issues.nodes || [];
 
             // client-side label filtering if requested
@@ -527,6 +523,12 @@ class IssueService extends Base {
                 });
             }
 
+            // Transform in-place to match OpenAPI schema
+            for (const issue of issues) {
+                issue.labels    = issue.labels?.nodes    || [];
+                issue.assignees = issue.assignees?.nodes || [];
+            }
+
             return {
                 count: issues.length,
                 issues

ai/mcp/server/github-workflow/services/queries/issueQueries.mjs

@@ -219,6 +219,68 @@ export const FETCH_ISSUES_FOR_SYNC = `
   }
 `;
 
+/**
+ * Optimized query for listing issues with minimal fields required by the Issue schema.
+ *
+ * Variables required:
+ * - $owner: String!
+ * - $repo: String!
+ * - $limit: Int!
+ * - $cursor: String
+ * - $states: [IssueState!]
+ * - $maxLabels: Int!
+ * - $maxAssignees: Int!
+ */
+export const FETCH_ISSUES_LIST = `
+  query FetchIssuesList(
+    $owner: String!
+    $repo: String!
+    $limit: Int!
+    $cursor: String
+    $states: [IssueState!]
+    $maxLabels: Int!
+    $maxAssignees: Int!
+  ) {
+    repository(owner: $owner, name: $repo) {
+      issues(
+        first: $limit
+        after: $cursor
+        states: $states
+        orderBy: {field: UPDATED_AT, direction: DESC}
+      ) {
+        pageInfo {
+          hasNextPage
+          endCursor
+        }
+        nodes {
+          number
+          title
+          body
+          state
+          createdAt
+          url
+          
+          author {
+            login
+          }
+          
+          labels(first: $maxLabels) {
+            nodes {
+              name
+            }
+          }
+          
+          assignees(first: $maxAssignees) {
+            nodes {
+              login
+            }
+          }
+        }
+      }
+    }
+  }
+`;
+
 /**
  * Simplified query for fetching a single issue's details.
  * Used for individual updates or debugging.