Minor cleanup

Author: kpgalligan

src/features/issues/handlers/issue.handler.ts

@@ -6,13 +6,9 @@ import {
   IssueHandlerMethods,
   CreateIssueInput,
   CreateIssuesInput,
-  BulkUpdateIssuesInput,
   SearchIssuesInput,
   DeleteIssueInput,
-  DeleteIssuesInput,
   CreateIssueResponse,
-  CreateIssuesResponse,
-  UpdateIssuesResponse,
   SearchIssuesResponse,
   DeleteIssueResponse,
   Issue,
@@ -147,33 +143,4 @@ export class IssueHandler extends BaseHandler implements IssueHandlerMethods {
       this.handleError(error, 'delete issue');
     }
   }
-
-  /* // Remove bulk delete handler method
-  /**
-   * Deletes multiple issues in bulk.
-   */
-  /*
-  async handleDeleteIssues(args: DeleteIssuesInput): Promise<BaseToolResponse> {
-    try {
-      const client = this.verifyAuth();
-      this.validateRequiredParams(args, ['ids']);
-
-      if (!Array.isArray(args.ids)) {
-        throw new Error('Ids parameter must be an array');
-      }
-
-      const result = await client.deleteIssues(args.ids) as DeleteIssueResponse;
-
-      if (!result.issueDelete.success) {
-        throw new Error('Failed to delete issues');
-      }
-
-      return this.createResponse(
-        `Successfully deleted ${args.ids.length} issues: ${args.ids.join(', ')}`
-      );
-    } catch (error) {
-      this.handleError(error, 'delete issues');
-    }
-  }
-  */
 }

src/features/issues/types/issue.types.ts

@@ -26,11 +26,6 @@ export interface UpdateIssueInput {
   stateId?: string;
 }
 
-export interface BulkUpdateIssuesInput {
-  issueIds: string[];
-  update: UpdateIssueInput;
-}
-
 export interface SearchIssuesInput {
   query?: string;
   filter?: {
@@ -47,16 +42,13 @@ export interface SearchIssuesInput {
   first?: number;
   after?: string;
   orderBy?: string;
+  id: string;
 }
 
 export interface DeleteIssueInput {
   id: string;
 }
 
-export interface DeleteIssuesInput {
-  ids: string[];
-}
-
 /**
  * Response types for issue operations
  */

src/graphql/client.ts

@@ -2,15 +2,12 @@ import { LinearClient } from '@linear/sdk';
 import { DocumentNode } from 'graphql';
 import { 
   CreateIssueInput, 
-  CreateIssuesInput,
   CreateIssueResponse,
-  CreateIssuesResponse,
   UpdateIssueInput,
   UpdateIssuesResponse,
   SearchIssuesInput,
   SearchIssuesResponse,
   DeleteIssueResponse,
-  Issue,
   IssueBatchResponse
 } from '../features/issues/types/issue.types.js';
 import {
@@ -130,13 +127,6 @@ export class LinearGraphQLClient {
     });
   }
 
-  /* // Remove bulk update client method
-  // Bulk update issues
-  async updateIssues(ids: string[], input: UpdateIssueInput): Promise<UpdateIssuesResponse> {
-    return this.execute<UpdateIssuesResponse>(UPDATE_ISSUES_MUTATION, { ids, input });
-  }
-  */
-
   // Create multiple labels
   async createIssueLabels(labels: LabelInput[]): Promise<LabelResponse> {
     return this.execute<LabelResponse>(CREATE_ISSUE_LABELS, { labels });
@@ -181,40 +171,4 @@ export class LinearGraphQLClient {
   async deleteIssue(id: string): Promise<DeleteIssueResponse> {
     return this.execute<DeleteIssueResponse>(DELETE_ISSUE_MUTATION, { id: id });
   }
-
-  /* // Remove bulk delete client method
-  // Delete multiple issues by iterating single deletes
-  async deleteIssues(ids: string[]): Promise<DeleteIssueResponse> {
-    // const { DELETE_ISSUES_MUTATION } = await import('./mutations.js');
-    // return this.execute<DeleteIssueResponse>(DELETE_ISSUES_MUTATION, { ids });
-
-    let success = true;
-    const errors: string[] = [];
-
-    for (const id of ids) {
-      try {
-        // Call the verified single delete method for each ID
-        const result = await this.deleteIssue(id); 
-        if (!result.issueDelete.success) {
-          success = false;
-          errors.push(`Failed to delete ${id}`);
-          // Optionally log the specific error from result if available
-        }
-      } catch (error) {
-        success = false;
-        errors.push(`Error deleting ${id}: ${error instanceof Error ? error.message : error}`);
-      }
-    }
-
-    if (!success) {
-        // We might want a more structured error response here, 
-        // but for now, just indicate overall failure and list errors.
-        throw new Error(`Bulk delete failed for some issues: ${errors.join('; ')}`);
-    }
-    
-    // Return a success structure compatible with DeleteIssueResponse if needed
-    // Adjust based on actual expected return type for bulk operation success
-    return { issueDelete: { success: true } } as DeleteIssueResponse;
-  }
-  */
 }

src/index.ts

@@ -1,25 +1,13 @@
 #!/usr/bin/env node
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
-import {
-  // CallToolRequestSchema,
-  ErrorCode,
-  // ListToolsRequestSchema,
-  McpError,
-} from '@modelcontextprotocol/sdk/types.js';
-// import { LinearAuth } from './auth.js';
-// import { LinearGraphQLClient } from './graphql/client.js';
-// import { HandlerFactory } from './core/handlers/handler.factory.js';
-// import { toolSchemas } from './core/types/tool.types.js';
-
-// --- Start: Echo Server Example ---
-import { McpServer, ResourceTemplate } from "@modelcontextprotocol/sdk/server/mcp.js";
-import { z } from "zod"; // Make sure zod is installed: npm install zod
+import { ErrorCode, McpError } from '@modelcontextprotocol/sdk/types.js'; // Removed unused schemas
+import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
+import { z } from "zod";
 
 import { LinearAuth } from './auth.js';
 import { LinearGraphQLClient } from './graphql/client.js';
 import { HandlerFactory } from './core/handlers/handler.factory.js';
-// We don't need the old toolSchemas anymore
-// import { toolSchemas } from './core/types/tool.types.js'; 
+// Removed unused toolSchemas import
 
 async function runLinearServer() {
   console.error('Starting Linear MCP server using McpServer...');