@enrichlayer/el-linear 1.7.0 → 1.10.0

package/dist/utils/graphql-issues-service.js

@@ -1,10 +1,11 @@
 import { resolveUserDisplayName } from "../config/resolver.js";
-import { BATCH_RESOLVE_FOR_CREATE_QUERY, BATCH_RESOLVE_FOR_SEARCH_QUERY, BATCH_RESOLVE_FOR_UPDATE_QUERY, buildResolveLabelsByNameQuery, CREATE_ISSUE_MUTATION, FILTERED_SEARCH_ISSUES_QUERY, GET_ISSUE_BY_ID_QUERY, GET_ISSUE_BY_IDENTIFIER_QUERY, GET_ISSUE_TEAM_QUERY, GET_ISSUES_QUERY, SEARCH_ISSUES_QUERY, UPDATE_ISSUE_MUTATION, } from "../queries/issues.js";
+import { ARCHIVE_ISSUE_MUTATION, BATCH_RESOLVE_FOR_CREATE_QUERY, BATCH_RESOLVE_FOR_SEARCH_QUERY, BATCH_RESOLVE_FOR_UPDATE_QUERY, buildResolveLabelsByNameQuery, CREATE_ISSUE_MUTATION, DELETE_ISSUE_MUTATION, FILTERED_SEARCH_ISSUES_QUERY, GET_ISSUE_BY_ID_QUERY, GET_ISSUE_BY_IDENTIFIER_QUERY, GET_ISSUE_TEAM_QUERY, GET_ISSUES_QUERY, SEARCH_ISSUES_QUERY, UPDATE_ISSUE_MUTATION, } from "../queries/issues.js";
 import { CREATE_LABEL_MUTATION } from "../queries/labels.js";
 import { toISOStringOrNow } from "./date-format.js";
 import { extractEmbeds } from "./embed-parser.js";
 import { notFoundError } from "./error-messages.js";
 import { parseIssueIdentifier, tryParseIssueIdentifier, } from "./identifier-parser.js";
+import { logger } from "./logger.js";
 import { isUuid } from "./uuid.js";
 const TEAM_KEY_REGEX = /^[A-Z0-9]+$/i;
 function extractSummaryText(summary) {
@@ -13,7 +14,7 @@ function extractSummaryText(summary) {
     }
     const parts = [];
     const walk = (node) => {
-        if (node.text) {
+        if (typeof node.text === "string") {
             parts.push(node.text);
         }
         if (Array.isArray(node.content)) {
@@ -33,15 +34,12 @@ export class GraphQLIssuesService {
         this.linearService = linearService;
     }
     async getIssues(limit = 25) {
-        const result = await this.graphQLService.rawRequest(GET_ISSUES_QUERY, {
-            first: limit,
-            orderBy: "updatedAt",
-        });
-        const issues = result.issues;
-        if (!issues?.nodes) {
+        const result = await this.graphQLService.rawRequest(GET_ISSUES_QUERY, { first: limit, orderBy: "updatedAt" });
+        const nodes = result.issues?.nodes;
+        if (!nodes?.length) {
             return [];
         }
-        return issues.nodes.map((issue) => this.transformIssueData(issue));
+        return nodes.map((issue) => this.transformIssueData(issue));
     }
     async getIssueById(issueId) {
         let issueData;
@@ -54,12 +52,8 @@ export class GraphQLIssuesService {
         }
         else {
             const { teamKey, issueNumber } = parseIssueIdentifier(issueId);
-            const result = await this.graphQLService.rawRequest(GET_ISSUE_BY_IDENTIFIER_QUERY, {
-                teamKey,
-                number: issueNumber,
-            });
-            const issues = result.issues;
-            const nodes = issues?.nodes;
+            const result = await this.graphQLService.rawRequest(GET_ISSUE_BY_IDENTIFIER_QUERY, { teamKey, number: issueNumber });
+            const nodes = result.issues?.nodes;
             if (!nodes?.length) {
                 throw notFoundError("Issue", issueId);
             }
@@ -76,15 +70,18 @@ export class GraphQLIssuesService {
         const finalLabelIds = await this.resolveLabelsWithMode(args.labelIds, resolveResult, labelMode, currentIssueLabels, teamIdForLabels, args.teamId);
         const finalProjectId = args.projectId
             ? this.resolveProjectId(args.projectId, resolveResult)
-            : args.projectId;
+            : undefined;
         const { projectMilestoneNodes, issueProjectMilestoneNodes } = this.extractMilestoneNodes(args, resolveResult);
         const finalMilestoneId = this.resolveMilestoneId(args.milestoneId, resolveResult, projectMilestoneNodes, issueProjectMilestoneNodes);
         const finalCycleId = await this.resolveCycleIdForUpdate(args, resolveResult, resolvedIssueId);
         const resolvedStatusId = await this.resolveStatusIdForUpdate(args, resolvedIssueId);
-        if (args.assigneeId && !isUuid(args.assigneeId)) {
-            args.assigneeId = await this.linearService.resolveUserId(args.assigneeId);
+        // Resolve assignee in place — `buildUpdateInput` reads it
+        // directly. Match the createIssue pattern.
+        let assigneeId = args.assigneeId;
+        if (assigneeId && !isUuid(assigneeId)) {
+            assigneeId = await this.linearService.resolveUserId(assigneeId);
         }
-        const updateInput = this.buildUpdateInput(args, {
+        const updateInput = this.buildUpdateInput({ ...args, assigneeId }, {
             statusId: resolvedStatusId,
             projectId: finalProjectId,
             labelIds: finalLabelIds,
@@ -93,15 +90,40 @@ export class GraphQLIssuesService {
         });
         return this.executeUpdateMutation(resolvedIssueId, args.id, updateInput);
     }
+    async archiveIssue(issueId) {
+        const resolvedIssueId = await this.linearService.resolveIssueId(issueId);
+        const result = await this.graphQLService.rawRequest(ARCHIVE_ISSUE_MUTATION, { id: resolvedIssueId });
+        const payload = result.issueArchive;
+        if (!payload.success) {
+            throw new Error(`Failed to archive issue ${issueId}`);
+        }
+        return {
+            id: resolvedIssueId,
+            entity: payload.entity ?? undefined,
+            lastSyncId: payload.lastSyncId,
+        };
+    }
+    async deleteIssue(issueId, options = {}) {
+        const resolvedIssueId = await this.linearService.resolveIssueId(issueId);
+        const result = await this.graphQLService.rawRequest(DELETE_ISSUE_MUTATION, {
+            id: resolvedIssueId,
+            permanentlyDelete: Boolean(options.permanentlyDelete),
+        });
+        const payload = result.issueDelete;
+        if (!payload.success) {
+            throw new Error(`Failed to delete issue ${issueId}`);
+        }
+        return {
+            id: resolvedIssueId,
+            entity: payload.entity ?? undefined,
+            lastSyncId: payload.lastSyncId,
+        };
+    }
     extractMilestoneNodes(args, resolveResult) {
-        const projectNodes = resolveResult.projects?.nodes;
         const projectMilestoneNodes = args.projectId
-            ? projectNodes?.[0]?.projectMilestones?.nodes
+            ? resolveResult.projects?.nodes?.[0]?.projectMilestones?.nodes
             : undefined;
-        const issueNodes = resolveResult.issues
-            ?.nodes;
-        const issueProjectMilestoneNodes = issueNodes?.[0]?.project
-            ?.projectMilestones?.nodes;
+        const issueProjectMilestoneNodes = resolveResult.issues?.nodes?.[0]?.project?.projectMilestones.nodes;
         return { projectMilestoneNodes, issueProjectMilestoneNodes };
     }
     async executeUpdateMutation(resolvedIssueId, originalId, updateInput) {
@@ -136,49 +158,51 @@ export class GraphQLIssuesService {
         let resolveResult = {};
         if (Object.keys(resolveVariables).length > 0) {
             const { __labelNames, ...queryVars } = resolveVariables;
-            const requests = [
-                this.graphQLService.rawRequest(BATCH_RESOLVE_FOR_CREATE_QUERY, queryVars),
-            ];
+            const batchPromise = this.graphQLService.rawRequest(BATCH_RESOLVE_FOR_CREATE_QUERY, queryVars);
             if (__labelNames) {
                 const labelQuery = buildResolveLabelsByNameQuery(__labelNames);
-                requests.push(this.graphQLService.rawRequest(labelQuery.query, labelQuery.variables));
+                const [batch, labels] = await Promise.all([
+                    batchPromise,
+                    this.graphQLService.rawRequest(labelQuery.query, labelQuery.variables),
+                ]);
+                resolveResult = { ...batch, labels: labels.labels };
             }
-            const results = await Promise.all(requests);
-            resolveResult = results[0];
-            if (results[1]) {
-                resolveResult.labels = results[1].labels;
+            else {
+                resolveResult = await batchPromise;
             }
         }
         const resolved = await this.resolveCreateFields(args, resolveResult);
-        if (args.assigneeId && !isUuid(args.assigneeId)) {
-            args.assigneeId = await this.linearService.resolveUserId(args.assigneeId);
-        }
-        const createInput = this.buildCreateInput(args, resolved);
+        // Mutate-the-input feels gross but matches the existing pattern
+        // — `resolved` doesn't carry assigneeId, so the buildCreateInput
+        // step reads `args.assigneeId` directly. Resolve in place.
+        let assigneeId = args.assigneeId;
+        if (assigneeId && !isUuid(assigneeId)) {
+            assigneeId = await this.linearService.resolveUserId(assigneeId);
+        }
+        const createInput = this.buildCreateInput({ ...args, assigneeId }, resolved);
         return this.executeCreateMutation(createInput);
     }
     async resolveCreateFields(args, resolveResult) {
         let teamId = args.teamId
             ? await this.resolveTeamId(args.teamId, resolveResult)
-            : args.teamId;
+            : undefined;
         const projectId = args.projectId
             ? this.resolveProjectId(args.projectId, resolveResult)
-            : args.projectId;
+            : undefined;
         // Validate team-project compatibility and auto-correct when possible
-        if (projectId && teamId) {
+        if (projectId && teamId && args.projectId) {
             teamId = this.validateProjectTeam(teamId, args.projectId, resolveResult);
         }
         const labelIds = await this.resolveLabels(args.labelIds, resolveResult, teamId, args.teamInput);
         const parentId = this.resolveParentId(args.parentId, resolveResult);
         const milestoneId = this.resolveMilestoneIdForCreate(args.milestoneId, resolveResult, projectId);
         let cycleId = args.cycleId;
-        if (args.cycleId &&
-            typeof args.cycleId === "string" &&
-            !isUuid(args.cycleId)) {
-            cycleId = await this.linearService.resolveCycleId(args.cycleId, teamId);
+        if (cycleId && !isUuid(cycleId)) {
+            cycleId = await this.linearService.resolveCycleId(cycleId, teamId);
         }
         let statusId = args.statusId;
-        if (args.statusId && !isUuid(args.statusId)) {
-            statusId = await this.linearService.resolveStatusId(args.statusId, teamId);
+        if (statusId && !isUuid(statusId)) {
+            statusId = await this.linearService.resolveStatusId(statusId, teamId);
         }
         return {
             teamId,
@@ -225,25 +249,27 @@ export class GraphQLIssuesService {
         }
         let resolveResult = {};
         if (Object.keys(resolveVariables).length > 0) {
-            resolveResult = await this.graphQLService.rawRequest(BATCH_RESOLVE_FOR_SEARCH_QUERY, resolveVariables);
+            resolveResult =
+                await this.graphQLService.rawRequest(BATCH_RESOLVE_FOR_SEARCH_QUERY, resolveVariables);
         }
         const finalTeamId = args.teamId
             ? await this.resolveTeamId(args.teamId, resolveResult)
-            : args.teamId;
+            : undefined;
         const finalProjectId = args.projectId
             ? this.resolveProjectId(args.projectId, resolveResult)
-            : args.projectId;
+            : undefined;
         const finalAssigneeId = this.resolveAssigneeId(args.assigneeId, resolveResult);
+        const limit = args.limit ?? 10;
         if (args.query) {
             const searchResult = await this.graphQLService.rawRequest(SEARCH_ISSUES_QUERY, {
                 term: args.query,
-                first: args.limit || 10,
+                first: limit,
             });
-            const searchIssues = searchResult.searchIssues;
-            if (!searchIssues?.nodes) {
+            const nodes = searchResult.searchIssues?.nodes;
+            if (!nodes?.length) {
                 return [];
             }
-            const results = searchIssues.nodes.map((issue) => this.transformIssueData(issue));
+            const results = nodes.map((issue) => this.transformIssueData(issue));
             return this.applySearchFilters(results, {
                 teamId: finalTeamId,
                 assigneeId: finalAssigneeId,
@@ -263,9 +289,9 @@ export class GraphQLIssuesService {
             priority: args.priority,
         });
         const searchResult = await this.graphQLService.rawRequest(FILTERED_SEARCH_ISSUES_QUERY, {
-            first: args.limit || 10,
+            first: limit,
             filter: Object.keys(filter).length > 0 ? filter : undefined,
-            orderBy: args.orderBy || "updatedAt",
+            orderBy: args.orderBy ?? "updatedAt",
         });
         const filteredIssues = searchResult.issues;
         if (!filteredIssues?.nodes) {
@@ -278,8 +304,7 @@ export class GraphQLIssuesService {
         if (isUuid(teamId)) {
             return teamId;
         }
-        const teams = resolveResult.teams;
-        const teamNodes = teams?.nodes;
+        const teamNodes = resolveResult.teams?.nodes;
         const resolvedTeam = teamNodes?.[0];
         if (resolvedTeam &&
             (resolvedTeam.key.toUpperCase() === teamId.toUpperCase() ||
@@ -293,8 +318,7 @@ export class GraphQLIssuesService {
         if (isUuid(projectId)) {
             return projectId;
         }
-        const projects = resolveResult.projects;
-        const projectNodes = projects?.nodes;
+        const projectNodes = resolveResult.projects?.nodes;
         if (!projectNodes?.length) {
             throw notFoundError("Project", projectId);
         }
@@ -305,13 +329,11 @@ export class GraphQLIssuesService {
      * If not, auto-switch when the project belongs to exactly one team.
      */
     validateProjectTeam(teamId, projectInput, resolveResult) {
-        const projects = resolveResult.projects;
-        const projectNode = projects?.nodes?.[0];
+        const projectNode = resolveResult.projects?.nodes?.[0];
         if (!projectNode) {
             return teamId;
         }
-        const teamsConn = projectNode.teams;
-        const teamNodes = teamsConn?.nodes;
+        const teamNodes = projectNode.teams?.nodes;
         if (!teamNodes?.length) {
             return teamId;
         }
@@ -324,7 +346,7 @@ export class GraphQLIssuesService {
         if (teamNodes.length === 1) {
             const correctTeamId = teamNodes[0].id;
             const correctKey = teamKeys[0];
-            process.stderr.write(`Auto-switched team to ${correctKey} (the only team associated with project "${projectName}").\n`);
+            logger.error(`Auto-switched team to ${correctKey} (the only team associated with project "${projectName}").`);
             return correctTeamId;
         }
         throw new Error(`Project "${projectName}" is not associated with the specified team.\n` +
@@ -339,8 +361,7 @@ export class GraphQLIssuesService {
             return labelIds;
         }
         const resolvedLabels = [];
-        const labels = resolveResult.labels;
-        const labelNodes = labels?.nodes;
+        const labelNodes = resolveResult.labels?.nodes;
         for (const labelIdOrName of labelIds) {
             if (isUuid(labelIdOrName)) {
                 resolvedLabels.push(labelIdOrName);
@@ -350,10 +371,7 @@ export class GraphQLIssuesService {
                 const candidates = labelNodes?.filter((l) => l.name.toLowerCase() === lowerName);
                 let label;
                 if (candidates && candidates.length > 1 && teamId) {
-                    label = candidates.find((l) => {
-                        const team = l.team;
-                        return team?.id === teamId;
-                    });
+                    label = candidates.find((l) => l.team?.id === teamId);
                 }
                 if (!label) {
                     label = candidates?.[0];
@@ -395,12 +413,10 @@ export class GraphQLIssuesService {
             }
             const result = await this.graphQLService.rawRequest(CREATE_LABEL_MUTATION, { input });
             const created = result.issueLabelCreate;
-            if (created?.success && created.issueLabel) {
-                const labelData = created.issueLabel;
-                const teamInfo = labelData.team;
-                const teamKey = teamInfo?.key ?? "";
-                process.stderr.write(`Auto-created label "${name}" on team ${teamKey}\n`);
-                return labelData.id;
+            if (created.success && created.issueLabel) {
+                const teamKey = created.issueLabel.team?.key ?? "";
+                logger.error(`Auto-created label "${name}" on team ${teamKey}`);
+                return created.issueLabel.id;
             }
             return null;
         }
@@ -428,37 +444,27 @@ export class GraphQLIssuesService {
         // Only fall back to first team if there's exactly one (unambiguous).
         // With multiple teams, returning an arbitrary one produces misleading error hints.
         const topLevelNodes = resolveResult.teams?.nodes;
-        return topLevelNodes?.length === 1
-            ? topLevelNodes[0].key
-            : undefined;
+        return topLevelNodes?.length === 1 ? topLevelNodes[0].key : undefined;
     }
     collectTeamNodes(resolveResult) {
         const nodes = [];
         const seen = new Set();
-        const topTeams = resolveResult.teams;
-        const topNodes = topTeams?.nodes;
+        const topNodes = resolveResult.teams?.nodes;
         if (topNodes) {
             for (const t of topNodes) {
-                const id = t.id;
-                if (!seen.has(id)) {
-                    seen.add(id);
+                if (!seen.has(t.id)) {
+                    seen.add(t.id);
                     nodes.push(t);
                 }
             }
         }
         // Also check project teams (which are always fetched when --project is used)
-        const projects = resolveResult.projects;
-        const projectNode = projects?.nodes?.[0];
-        if (projectNode) {
-            const projectTeams = projectNode.teams;
-            const projectTeamNodes = projectTeams?.nodes;
-            if (projectTeamNodes) {
-                for (const t of projectTeamNodes) {
-                    const id = t.id;
-                    if (!seen.has(id)) {
-                        seen.add(id);
-                        nodes.push(t);
-                    }
+        const projectTeamNodes = resolveResult.projects?.nodes?.[0]?.teams?.nodes;
+        if (projectTeamNodes) {
+            for (const t of projectTeamNodes) {
+                if (!seen.has(t.id)) {
+                    seen.add(t.id);
+                    nodes.push(t);
                 }
             }
         }
@@ -481,8 +487,7 @@ export class GraphQLIssuesService {
                 resolved = issueMilestone.id;
             }
         }
-        const milestones = resolveResult.milestones;
-        const milestoneNodes = milestones?.nodes;
+        const milestoneNodes = resolveResult.milestones?.nodes;
         if (!isUuid(resolved) && milestoneNodes?.length) {
             resolved = milestoneNodes[0].id;
         }
@@ -496,19 +501,14 @@ export class GraphQLIssuesService {
             return milestoneId;
         }
         let resolved;
-        const createProjects = resolveResult.projects;
-        const createProjectNodes = createProjects?.nodes;
-        const createProjectMilestones = createProjectNodes?.[0]
-            ?.projectMilestones;
-        const createProjectMilestoneNodes = createProjectMilestones?.nodes;
+        const createProjectMilestoneNodes = resolveResult.projects?.nodes?.[0]?.projectMilestones?.nodes;
         if (createProjectMilestoneNodes) {
             const projectMilestone = createProjectMilestoneNodes.find((m) => m.name === milestoneId);
             if (projectMilestone) {
                 resolved = projectMilestone.id;
             }
         }
-        const createMilestones = resolveResult.milestones;
-        const createMilestoneNodes = createMilestones?.nodes;
+        const createMilestoneNodes = resolveResult.milestones?.nodes;
         if (!resolved && createMilestoneNodes?.length) {
             resolved = createMilestoneNodes[0].id;
         }
@@ -521,15 +521,18 @@ export class GraphQLIssuesService {
         return resolved;
     }
     async fetchIssueTeamId(issueId) {
-        const result = await this.graphQLService.rawRequest(GET_ISSUE_TEAM_QUERY, {
-            issueId,
-        });
-        const fetchedIssue = result.issue;
-        const fetchedTeam = fetchedIssue?.team;
-        return fetchedTeam?.id;
+        const result = await this.graphQLService.rawRequest(GET_ISSUE_TEAM_QUERY, { issueId });
+        return result.issue?.team?.id;
     }
     buildCreateInput(args, resolved) {
-        const input = { title: args.title };
+        const input = {};
+        // Title is required by Linear's API unless --from-template is set,
+        // in which case Linear copies the template's title. We only include
+        // the field when explicitly set so a missing title with templateId
+        // works as the API documents.
+        if (args.title) {
+            input.title = args.title;
+        }
         if (resolved.teamId) {
             input.teamId = resolved.teamId;
         }
@@ -548,9 +551,7 @@ export class GraphQLIssuesService {
         if (resolved.statusId) {
             input.stateId = resolved.statusId;
         }
-        if (resolved.labelIds &&
-            Array.isArray(resolved.labelIds) &&
-            resolved.labelIds.length > 0) {
+        if (resolved.labelIds && resolved.labelIds.length > 0) {
             input.labelIds = resolved.labelIds;
         }
         if (args.estimate !== undefined) {
@@ -565,14 +566,18 @@ export class GraphQLIssuesService {
         if (resolved.cycleId) {
             input.cycleId = resolved.cycleId;
         }
-        if (args.subscriberIds &&
-            Array.isArray(args.subscriberIds) &&
-            args.subscriberIds.length > 0) {
+        if (args.subscriberIds && args.subscriberIds.length > 0) {
             input.subscriberIds = args.subscriberIds;
         }
         if (args.dueDate !== undefined) {
             input.dueDate = args.dueDate;
         }
+        if (args.templateId !== undefined && args.templateId !== null) {
+            // Server-side template instantiation: Linear copies the
+            // template's title/description/labels/priority onto the
+            // new issue. Any explicit field above wins by override.
+            input.templateId = args.templateId;
+        }
         return input;
     }
     buildUpdateInput(args, resolved) {
@@ -688,8 +693,7 @@ export class GraphQLIssuesService {
         if (!assigneeId || isUuid(assigneeId) || !assigneeId.includes("@")) {
             return assigneeId;
         }
-        const users = resolveResult.users;
-        const userNodes = users?.nodes;
+        const userNodes = resolveResult.users?.nodes;
         if (!userNodes?.length) {
             throw notFoundError("User", assigneeId);
         }
@@ -705,7 +709,7 @@ export class GraphQLIssuesService {
             resolveVariables.teamKey = teamKey;
             resolveVariables.issueNumber = issueNumber;
         }
-        if (args.labelIds && Array.isArray(args.labelIds)) {
+        if (args.labelIds && args.labelIds.length > 0) {
             const nonUuidLabels = args.labelIds.filter((id) => !isUuid(id));
             if (nonUuidLabels.length > 0) {
                 resolveVariables.__labelNames = nonUuidLabels;
@@ -714,35 +718,31 @@ export class GraphQLIssuesService {
         if (args.projectId && !isUuid(args.projectId)) {
             resolveVariables.projectName = args.projectId;
         }
-        if (args.milestoneId &&
-            typeof args.milestoneId === "string" &&
-            !isUuid(args.milestoneId)) {
+        if (args.milestoneId && !isUuid(args.milestoneId)) {
             resolveVariables.milestoneName = args.milestoneId;
         }
         const { __labelNames, ...updateQueryVars } = resolveVariables;
-        const requests = [
-            this.graphQLService.rawRequest(BATCH_RESOLVE_FOR_UPDATE_QUERY, updateQueryVars),
-        ];
+        const batchPromise = this.graphQLService.rawRequest(BATCH_RESOLVE_FOR_UPDATE_QUERY, updateQueryVars);
+        let resolveResult;
         if (__labelNames) {
             const labelQuery = buildResolveLabelsByNameQuery(__labelNames);
-            requests.push(this.graphQLService.rawRequest(labelQuery.query, labelQuery.variables));
+            const [batch, labels] = await Promise.all([
+                batchPromise,
+                this.graphQLService.rawRequest(labelQuery.query, labelQuery.variables),
+            ]);
+            resolveResult = { ...batch, labels: labels.labels };
         }
-        const results = await Promise.all(requests);
-        const resolveResult = results[0];
-        if (results[1]) {
-            resolveResult.labels = results[1].labels;
+        else {
+            resolveResult = await batchPromise;
         }
         if (!isUuid(args.id)) {
-            const resolvedIssues = resolveResult.issues;
-            const resolvedIssueNodes = resolvedIssues.nodes;
+            const resolvedIssueNodes = resolveResult.issues?.nodes ?? [];
             if (!resolvedIssueNodes.length) {
                 throw notFoundError("Issue", args.id);
             }
             resolvedIssueId = resolvedIssueNodes[0].id;
-            const team = resolvedIssueNodes[0].team;
-            issueTeamId = team?.id;
-            const issueLabels = resolvedIssueNodes[0].labels;
-            currentIssueLabels = issueLabels.nodes.map((l) => l.id);
+            issueTeamId = resolvedIssueNodes[0].team?.id;
+            currentIssueLabels = resolvedIssueNodes[0].labels.nodes.map((l) => l.id);
         }
         return { resolvedIssueId, issueTeamId, currentIssueLabels, resolveResult };
     }
@@ -750,7 +750,7 @@ export class GraphQLIssuesService {
         if (args.cycleId === undefined || args.cycleId === null) {
             return args.cycleId;
         }
-        if (typeof args.cycleId !== "string" || isUuid(args.cycleId)) {
+        if (isUuid(args.cycleId)) {
             return args.cycleId;
         }
         let teamIdForCycle = resolveResult.issues?.nodes?.[0]?.team?.id;
@@ -780,7 +780,7 @@ export class GraphQLIssuesService {
         if (args.milestoneId && !isUuid(args.milestoneId)) {
             resolveVariables.milestoneName = args.milestoneId;
         }
-        if (args.labelIds && Array.isArray(args.labelIds)) {
+        if (args.labelIds && args.labelIds.length > 0) {
             const nonUuidLabels = args.labelIds.filter((id) => !isUuid(id));
             if (nonUuidLabels.length > 0) {
                 resolveVariables.__labelNames = nonUuidLabels;
@@ -799,30 +799,32 @@ export class GraphQLIssuesService {
         if (!parentId || isUuid(parentId)) {
             return parentId;
         }
-        const parentIssues = resolveResult.parentIssues;
-        const parentNodes = parentIssues?.nodes;
+        const parentNodes = resolveResult.parentIssues?.nodes;
         if (!parentNodes?.length) {
             throw notFoundError("Parent issue", parentId);
         }
         return parentNodes[0].id;
     }
     transformIssueData(issue) {
-        const labels = issue.labels;
+        // `labels`/`children`/`comments` are connection fields that the
+        // fragment always selects, so production responses populate them.
+        // Test mocks pass partial objects, so read defensively to keep
+        // those test fixtures small.
         return {
             ...this.transformIssueCoreFields(issue),
             ...this.transformIssueRelations(issue),
-            summary: issue.summary
-                ? extractSummaryText(issue.summary)
-                : undefined,
+            summary: issue.summary ? extractSummaryText(issue.summary) : undefined,
             priority: issue.priority,
-            estimate: issue.estimate || undefined,
-            dueDate: issue.dueDate || undefined,
-            labels: labels.nodes.map((label) => ({
+            estimate: issue.estimate ?? undefined,
+            dueDate: issue.dueDate ?? undefined,
+            labels: (issue.labels?.nodes ?? []).map((label) => ({
                 id: label.id,
                 name: label.name,
             })),
             ...this.transformIssueHierarchy(issue),
-            comments: this.transformIssueComments(issue),
+            comments: "comments" in issue && issue.comments
+                ? this.transformIssueComments(issue)
+                : [],
             createdAt: toISOStringOrNow(issue.createdAt),
             updatedAt: toISOStringOrNow(issue.updatedAt),
         };
@@ -833,93 +835,84 @@ export class GraphQLIssuesService {
             identifier: issue.identifier,
             url: issue.url,
             title: issue.title,
-            description: issue.description || undefined,
+            description: issue.description ?? undefined,
             branchName: issue.branchName || undefined,
-            embeds: issue.description
-                ? extractEmbeds(issue.description)
-                : undefined,
+            embeds: issue.description ? extractEmbeds(issue.description) : undefined,
         };
     }
     transformIssueRelations(issue) {
-        const state = issue.state;
-        const assignee = issue.assignee;
-        const team = issue.team;
-        const project = issue.project;
-        const cycle = issue.cycle;
-        const milestone = issue.projectMilestone;
         return {
-            state: state
-                ? { id: state.id, name: state.name }
+            state: issue.state
+                ? { id: issue.state.id, name: issue.state.name }
                 : undefined,
-            assignee: assignee
+            assignee: issue.assignee
                 ? {
-                    id: assignee.id,
-                    name: resolveUserDisplayName(assignee.id, assignee.name),
-                    url: assignee.url || undefined,
+                    id: issue.assignee.id,
+                    name: resolveUserDisplayName(issue.assignee.id, issue.assignee.name),
+                    url: issue.assignee.url ?? undefined,
                 }
                 : undefined,
-            team: team
+            team: issue.team
                 ? {
-                    id: team.id,
-                    key: team.key,
-                    name: team.name,
+                    id: issue.team.id,
+                    key: issue.team.key,
+                    name: issue.team.name,
                 }
                 : undefined,
-            project: project
-                ? { id: project.id, name: project.name }
+            project: issue.project
+                ? { id: issue.project.id, name: issue.project.name }
                 : undefined,
-            cycle: cycle
+            cycle: issue.cycle
                 ? {
-                    id: cycle.id,
-                    name: cycle.name,
-                    number: cycle.number,
+                    id: issue.cycle.id,
+                    name: issue.cycle.name ?? "",
+                    number: issue.cycle.number,
                 }
                 : undefined,
-            projectMilestone: milestone
+            projectMilestone: issue.projectMilestone
                 ? {
-                    id: milestone.id,
-                    name: milestone.name,
-                    targetDate: milestone.targetDate || undefined,
+                    id: issue.projectMilestone.id,
+                    name: issue.projectMilestone.name,
+                    targetDate: issue.projectMilestone.targetDate ?? undefined,
                 }
                 : undefined,
         };
     }
     transformIssueHierarchy(issue) {
-        const parent = issue.parent;
-        const children = issue.children;
+        // children is a connection field — always present in production
+        // responses. Defensive read keeps test mocks small.
+        const childNodes = issue.children?.nodes ?? [];
         return {
-            parentIssue: parent
+            parentIssue: issue.parent
                 ? {
-                    id: parent.id,
-                    identifier: parent.identifier,
-                    title: parent.title,
+                    id: issue.parent.id,
+                    identifier: issue.parent.identifier,
+                    title: issue.parent.title,
                 }
                 : undefined,
-            subIssues: children?.nodes?.map((child) => ({
-                id: child.id,
-                identifier: child.identifier,
-                title: child.title,
-            })) || undefined,
+            subIssues: childNodes.length > 0
+                ? childNodes.map((child) => ({
+                    id: child.id,
+                    identifier: child.identifier,
+                    title: child.title,
+                }))
+                : undefined,
         };
     }
     transformIssueComments(issue) {
-        const comments = issue.comments;
-        return (comments?.nodes?.map((comment) => {
-            const commentUser = comment.user;
-            return {
-                id: comment.id,
-                body: comment.body,
-                embeds: extractEmbeds(comment.body),
-                user: commentUser
-                    ? {
-                        id: commentUser.id,
-                        name: resolveUserDisplayName(commentUser.id, commentUser.name),
-                        url: commentUser.url || undefined,
-                    }
-                    : undefined,
-                createdAt: toISOStringOrNow(comment.createdAt),
-                updatedAt: toISOStringOrNow(comment.updatedAt),
-            };
-        }) || []);
+        return issue.comments.nodes.map((comment) => ({
+            id: comment.id,
+            body: comment.body,
+            embeds: extractEmbeds(comment.body),
+            user: comment.user
+                ? {
+                    id: comment.user.id,
+                    name: resolveUserDisplayName(comment.user.id, comment.user.name),
+                    url: comment.user.url ?? undefined,
+                }
+                : undefined,
+            createdAt: toISOStringOrNow(comment.createdAt),
+            updatedAt: toISOStringOrNow(comment.updatedAt),
+        }));
     }
 }