@skyramp/mcp 0.1.3 → 0.1.5

package/build/tools/submitReportTool.js

@@ -120,8 +120,9 @@ export function registerSubmitReportTool(server) {
                 .string()
                 .optional()
                 .default(DEFAULT_COMMIT_MESSAGE)
-                .describe("Succinct commit message (under 72 chars) summarizing what the testbot did, " +
-                "e.g. 'add contract tests for /products endpoint' or 'update smoke tests for order API changes'"),
+                .describe("Succinct commit message (if possible, under 72 chars) summarizing what Testbot did, " +
+                "e.g. 'add contract tests for /products endpoint' or 'update smoke tests for order API changes'. " +
+                "Used as both the git commit subject and the side PR title — the consumer applies truncation as needed."),
         },
         _meta: {
             keywords: ["report", "summary", "testbot", "submit"],
@@ -161,7 +162,7 @@ export function registerSubmitReportTool(server) {
             testResults: params.testResults,
             issuesFound: params.issuesFound,
             nextSteps: params.nextSteps ?? [],
-            commitMessage: (params.commitMessage ?? "").replace(/[\r\n]+/g, " ").trim().slice(0, 72) || DEFAULT_COMMIT_MESSAGE,
+            commitMessage: (params.commitMessage ?? "").replace(/[\r\n]+/g, " ").trim() || DEFAULT_COMMIT_MESSAGE,
         }, null, 2);
         logger.info("Submitting testbot report", {
             outputFile: params.summaryOutputFile,

package/build/tools/submitReportTool.test.js

@@ -128,7 +128,7 @@ describe("registerSubmitReportTool", () => {
         const written = JSON.parse(await fs.readFile(outputFile, "utf-8"));
         expect(written.commitMessage).toBe("Added recommendations by Skyramp Testbot.");
     });
-    it("should sanitize commitMessage (newlines, length)", async () => {
+    it("should sanitize commitMessage (collapse newlines, trim whitespace)", async () => {
         const tmpDir = await fs.mkdtemp(path.join(os.tmpdir(), "submit-report-test-"));
         tmpDirs.push(tmpDir);
         const outputFile = path.join(tmpDir, "report.json");
@@ -139,7 +139,21 @@ describe("registerSubmitReportTool", () => {
         expect(result.isError).toBeUndefined();
         const written = JSON.parse(await fs.readFile(outputFile, "utf-8"));
         expect(written.commitMessage).toBe("line one line two line three");
-        expect(written.commitMessage.length).toBeLessThanOrEqual(72);
+        // No length cap — truncation is the consumer's responsibility (SKYR-3757).
+    });
+    it("should preserve commitMessage longer than 72 chars without truncation", async () => {
+        const tmpDir = await fs.mkdtemp(path.join(os.tmpdir(), "submit-report-test-"));
+        tmpDirs.push(tmpDir);
+        const outputFile = path.join(tmpDir, "report.json");
+        const longMessage = "add contract and integration tests for the new PATCH /orders/{order_id} endpoint including discount recalculation and line-item edits";
+        const result = await handler({
+            ...sampleReportParams(outputFile),
+            commitMessage: longMessage,
+        });
+        expect(result.isError).toBeUndefined();
+        const written = JSON.parse(await fs.readFile(outputFile, "utf-8"));
+        expect(written.commitMessage).toBe(longMessage);
+        expect(written.commitMessage.length).toBeGreaterThan(72);
     });
     it("should use default commitMessage when provided as empty string", async () => {
         const tmpDir = await fs.mkdtemp(path.join(os.tmpdir(), "submit-report-test-"));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyramp/mcp",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "main": "build/index.js",
   "exports": {
     ".": "./build/index.js",