@smartruns/mcp 1.1.1 → 1.1.2

package/dist/tools/comments.js

@@ -8,15 +8,18 @@ function handleError(err) {
 // Comments in SmartRuns are polymorphic but NOT via Rails-standard commentable_type /
 // commentable_id, and NOT nested under a parent route. The CommentsController
 // (app/controllers/comments_controller.rb) drives everything off two flat params,
-// `source_type` + `source_id`, validated against Comment::SUPPORTED_SCOPES
-// (app/models/comment.rb). The comment body lives on the `text` attribute. Keep this
-// enum in sync with Comment::SUPPORTED_SCOPES.
-const SOURCE_TYPES = ['test', 'test_plan', 'test_run', 'test_suite', 'spec'];
+// `source_type` + `source_id`, validated server-side against Comment::SUPPORTED_SCOPES
+// (app/models/comment.rb). The comment body lives on the `text` attribute.
+//
+// source_type is intentionally a permissive string, NOT a hardcoded enum: the server is
+// the single source of truth for valid scopes and rejects an unsupported one, so this
+// wrapper never goes stale when a scope is added (e.g. `defect`). Do not re-introduce a
+// client-side enum here — it only duplicates server validation and drifts.
 const sourceTypeSchema = z
-    .enum(SOURCE_TYPES)
-    .describe('Parent entity type the comment belongs to (Comment::SUPPORTED_SCOPES).');
+    .string()
+    .describe('Parent entity type the comment belongs to — a Comment::SUPPORTED_SCOPES value (currently test, test_plan, test_run, test_suite, spec, defect). Validated server-side; an unsupported scope is rejected by the API.');
 export function registerCommentTools(server, client) {
-    server.tool('list_comments', 'List comments on a parent entity (test, test_plan, test_run, test_suite, or spec). Comments are polymorphic: identify the parent with source_type + source_id. Scoped to the current account and project server-side.', {
+    server.tool('list_comments', 'List comments on a parent entity (test, test_plan, test_run, test_suite, spec, or defect). Comments are polymorphic: identify the parent with source_type + source_id. Scoped to the current account and project server-side.', {
         source_type: sourceTypeSchema,
         source_id: z.number().int().describe('ID of the parent entity the comments belong to'),
         project_id: z.string().optional().describe('Override the project for this call (WTProject header). Defaults to SMARTRUNS_PROJECT_ID.'),
@@ -29,7 +32,7 @@ export function registerCommentTools(server, client) {
             return handleError(err);
         }
     });
-    server.tool('create_comment', 'Create a comment on a parent entity (test, test_plan, test_run, test_suite, or spec). The comment body goes in the "text" field. The parent is identified by source_type + source_id. The author is set server-side from the authenticated token.', {
+    server.tool('create_comment', 'Create a comment on a parent entity (test, test_plan, test_run, test_suite, spec, or defect). The comment body goes in the "text" field. The parent is identified by source_type + source_id. The author is set server-side from the authenticated token.', {
         source_type: sourceTypeSchema,
         source_id: z.number().int().describe('ID of the parent entity to comment on'),
         text: z.string().describe('The comment body (required)'),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smartruns/mcp",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "MCP server for the SmartRuns REST API",
   "type": "module",
   "bin": {