@uarlouski/testrail-mcp-server 2.0.0 → 2.1.0

package/dist/client/testrail.js

@@ -25,7 +25,9 @@ export class TestRailClient {
         return this.get(`${API_BASE_V2}/get_case/${caseId}`);
     }
     async getCasesRecursively(projectId, sectionId, filter, excludedSectionNames) {
-        const sections = await this.getSections(projectId);
+        // Extract suite_id from filter if present, for getSections call
+        const suiteId = filter?.suite_id ? parseInt(filter.suite_id) : undefined;
+        const sections = await this.getSections(projectId, suiteId);
         const sectionIds = this.getSubSectionIds(sections, sectionId, excludedSectionNames);
         sectionIds.push(sectionId);
         const promises = sectionIds.map(id => this.getCases(projectId, id, filter));
@@ -105,8 +107,11 @@ export class TestRailClient {
     async updateRun(runId, fields) {
         return this.post(`${API_BASE_V2}/update_run/${runId}`, fields);
     }
-    async getSections(projectId) {
-        const url = `${API_BASE_V2}/get_sections/${projectId}`;
+    async getSections(projectId, suiteId) {
+        let url = `${API_BASE_V2}/get_sections/${projectId}`;
+        if (suiteId) {
+            url += `&suite_id=${suiteId}`;
+        }
         return this.paginateAll(url, 'sections');
     }
     async addSection(projectId, data) {
@@ -152,8 +157,7 @@ export class TestRailClient {
     }
     async getProjects() {
         if (!this.projectsPromise) {
-            this.projectsPromise = this.get(`${API_BASE_V2}/get_projects`)
-                .then(response => response.projects);
+            this.projectsPromise = this.paginateAll(`${API_BASE_V2}/get_projects`, 'projects');
         }
         return this.projectsPromise;
     }
@@ -221,6 +225,12 @@ export class TestRailClient {
         let nextUrl = url;
         while (nextUrl) {
             const response = await this.get(nextUrl);
+            // Handle older TestRail versions (pre-7.x) that return flat arrays
+            // instead of paginated objects like { "suites": [...], "_links": {...} }
+            if (Array.isArray(response)) {
+                allItems.push(...response);
+                break;
+            }
             if (response[dataKey] && Array.isArray(response[dataKey])) {
                 allItems.push(...response[dataKey]);
             }

package/dist/index.js

@@ -23,7 +23,7 @@ if (!parseResult.success) {
 const { TESTRAIL_INSTANCE_URL, TESTRAIL_USERNAME, TESTRAIL_API_KEY, TESTRAIL_ENABLE_SHARED_STEPS, TESTRAIL_ALLOW_WRITE_OPERATIONS, TESTRAIL_ALLOW_READ_OPERATIONS, TESTRAIL_ALLOW_DELETE_OPERATIONS } = parseResult.data;
 const server = new McpServer({
     name: "TestRail MCP Server",
-    version: "2.0.0",
+    version: "2.1.0",
 });
 const client = new TestRailClient(TESTRAIL_INSTANCE_URL, TESTRAIL_USERNAME, TESTRAIL_API_KEY);
 const tools = getToolsToRegister({

package/dist/tools/cases/get_cases.js

@@ -1,8 +1,9 @@
 import { z } from "zod";
 import fs from "fs";
-import { validateCaseFields } from "../../utils/validator.js";
+import { validateCaseFields, validateSuiteId } from "../../utils/validator.js";
 const parameters = {
     project_id: z.number().describe("The ID of the project. Use get_projects to find available projects"),
+    suite_id: z.number().optional().describe("The ID of the test suite (required for multi-suite projects, i.e. suite_mode=3). Use get_suites to find available suites"),
     section: z.object({
         id: z.number(),
         recursive: z.boolean().optional().default(false).describe("If true, fetches cases from the section and all its child sections"),
@@ -18,7 +19,8 @@ export const getCasesTool = {
     mode: "read",
     description: "Get all test cases for a project. Filter by section, API params (priority, type), or any field including custom fields via 'where'. Returns case IDs, titles, and any additional requested fields.",
     parameters,
-    handler: async ({ project_id, section, filter, where, fields, output_file }, client) => {
+    handler: async ({ project_id, suite_id, section, filter, where, fields, output_file }, client) => {
+        await validateSuiteId(client, project_id, suite_id);
         const caseFields = await client.getCaseFields();
         if (fields) {
             validateCaseFields(fields, caseFields);
@@ -26,6 +28,10 @@ export const getCasesTool = {
         if (where) {
             validateCaseFields(Object.keys(where), caseFields);
         }
+        // Inject suite_id into filter for multi-suite projects
+        if (suite_id) {
+            filter = { ...filter, suite_id: suite_id.toString() };
+        }
         let cases;
         if (section?.recursive) {
             cases = await client.getCasesRecursively(project_id, section.id, filter, section.excludes);

package/dist/tools/sections/get_sections.js

@@ -1,8 +1,10 @@
 import { z } from "zod";
 import fs from "fs";
 import { SectionSchema } from "./types.js";
+import { validateSuiteId } from "../../utils/validator.js";
 const parameters = {
     project_id: z.number().describe("The ID of the project. Use get_projects to find available projects"),
+    suite_id: z.number().optional().describe("The ID of the test suite (required for multi-suite projects, i.e. suite_mode=3). Use get_suites to find available suites"),
     output_file: z.string().optional().describe("Absolute file path to save the JSON response to. Use this for large datasets to avoid blowing up context limits."),
 };
 export const getSectionsTool = {
@@ -10,8 +12,9 @@ export const getSectionsTool = {
     mode: "read",
     description: "Get all sections for a project. Returns section IDs and names that can be used with add_case",
     parameters,
-    handler: async ({ project_id, output_file }, client) => {
-        const sections = await client.getSections(project_id);
+    handler: async ({ project_id, suite_id, output_file }, client) => {
+        await validateSuiteId(client, project_id, suite_id);
+        const sections = await client.getSections(project_id, suite_id);
         const response = {
             sections: sections.map(s => SectionSchema.parse(s)),
         };

package/dist/utils/validator.js

@@ -1,5 +1,19 @@
 import { SYSTEM_FIELDS, mapToFieldSchema } from "../tools/cases/get_case_fields.js";
 import { isActive } from "./sanitizer.js";
+const SUITE_MODE_MULTI = 3;
+/**
+ * Validates that suite_id is provided when the project uses multiple test suites (suite_mode=3).
+ * Throws a descriptive Error if suite_id is required but missing.
+ */
+export async function validateSuiteId(client, projectId, suiteId) {
+    if (suiteId)
+        return;
+    const project = await client.getProject(projectId);
+    if (project.suite_mode === SUITE_MODE_MULTI) {
+        throw new Error(`Project "${project.name}" (id: ${project.id}) uses multiple test suites (suite_mode=3). ` +
+            `The suite_id parameter is required. Use get_suites to find available suites for this project.`);
+    }
+}
 /**
  * Validates that the provided fields exist in the TestRail case schema.
  * Throws an Error if any field is invalid.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uarlouski/testrail-mcp-server",
-  "version": "2.0.0",
+  "version": "2.1.0",
   "mcpName": "io.github.uarlouski/testrail-mcp-server",
   "description": "Model Context Protocol (MCP) server for TestRail integration, enabling LLMs to fetch test cases and metadata.",
   "main": "dist/index.js",