@dpesch/mantisbt-mcp-server 1.8.1 → 1.8.2

package/CHANGELOG.md

@@ -11,6 +11,13 @@ This project adheres to [Semantic Versioning](https://semver.org/).
 
 ---
 
+## [1.8.2] – 2026-03-27
+
+### Fixed
+- HTTP transport (`TRANSPORT=http`): race condition when multiple sequential requests arrive quickly (e.g. `initialize` followed immediately by `ping`). The `res.on('close')` handler that calls `transport.close()` fires asynchronously, so the next request could arrive while the previous transport was still registered — causing `connect()` to throw "Already connected". Fixed by calling `await server.close()` before each `server.connect()` call (a no-op when not connected). Also added a `res.headersSent` guard in the error handler to prevent a secondary "Cannot set headers after they are sent" error from closing the socket without a response.
+
+---
+
 ## [1.8.1] – 2026-03-27
 
 ### Fixed

package/dist/index.js

@@ -107,13 +107,20 @@ async function runHttp() {
                         sessionIdGenerator: undefined,
                         enableJsonResponse: true,
                     });
-                    res.on('close', () => transport.close());
+                    res.on('close', () => { void transport.close(); });
+                    // Disconnect from any previous transport before connecting the new one.
+                    // The res.on('close') handler above closes the transport asynchronously,
+                    // but the next request may arrive before it fires — causing connect() to
+                    // throw "Already connected". Explicitly closing first avoids that race.
+                    await server.close();
                     await server.connect(transport);
                     await transport.handleRequest(req, res, body);
                 }
                 catch {
-                    res.writeHead(400, { 'Content-Type': 'application/json' });
-                    res.end(JSON.stringify({ error: 'Bad Request' }));
+                    if (!res.headersSent) {
+                        res.writeHead(400, { 'Content-Type': 'application/json' });
+                        res.end(JSON.stringify({ error: 'Bad Request' }));
+                    }
                 }
             });
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dpesch/mantisbt-mcp-server",
-  "version": "1.8.1",
+  "version": "1.8.2",
   "mcpName": "io.github.dpesch/mantisbt-mcp-server",
   "description": "MCP server for MantisBT REST API – read and manage bug tracker issues",
   "author": "Dominik Pesch",

package/scripts/hooks/pre-push.mjs

@@ -111,6 +111,16 @@ rl.on('close', () => {
   const branches = lines.filter(l => l.split(' ')[2]?.startsWith('refs/heads/'));
   const others   = lines.filter(l => !l.split(' ')[2]?.startsWith('refs/heads/'));
 
+  // Batch-fetch all tags from Codeberg (single ls-remote call, ref → sha).
+  const remoteTagsRaw = gitOptional(`ls-remote --tags "${remoteUrl}"`);
+  const remoteTagMap = new Map(
+    remoteTagsRaw
+      ? remoteTagsRaw.split('\n').filter(Boolean)
+          .map(l => { const [sha, ref] = l.split(/\s+/); return [ref, sha]; })
+          .filter(([ref]) => ref)
+      : []
+  );
+
   // localSha → filteredSha, accumulated across all refs in this push.
   const shaMap = {};
   let pushed = 0;
@@ -124,6 +134,17 @@ rl.on('close', () => {
 
     const label = remoteRef.replace('refs/heads/', '').replace('refs/tags/', '');
 
+    if (remoteRef.startsWith('refs/tags/')) {
+      if (remoteTagMap.has(remoteRef)) {
+        console.log(`  ↷ ${label} already on Codeberg — skipping`);
+        continue;
+      }
+      if (!gitOptional(`rev-parse --verify "${localSha}"`)) {
+        console.warn(`  ⚠ ${label} — commit ${localSha.slice(0, 8)} not found locally, skipping`);
+        continue;
+      }
+    }
+
     // If this commit was already filtered as part of a branch push, reuse it.
     if (localSha in shaMap) {
       try {
@@ -137,9 +158,11 @@ rl.on('close', () => {
       continue;
     }
 
-    // Get the actual current SHA on Codeberg for this ref.
-    const lsOut = gitOptional(`ls-remote "${remoteUrl}" "${remoteRef}"`);
-    const actualRemoteSha = lsOut ? lsOut.split(/\s+/)[0] : '';
+    // Tags: existing ones were skipped above, so only new tags reach here — no remote anchor.
+    // Branches: query individually to find the incremental anchor for filtered chain building.
+    const actualRemoteSha = remoteRef.startsWith('refs/tags/')
+      ? ''
+      : (gitOptional(`ls-remote "${remoteUrl}" "${remoteRef}"`)?.split(/\s+/)[0] ?? '');
 
     if (!actualRemoteSha) {
       // New ref on Codeberg — filter the tip with no parent.
@@ -167,15 +190,8 @@ rl.on('close', () => {
           shaMap[sha] = makeFilteredCommit(sha, filteredTree, mappedParents);
         }
       } else {
-        // Fallback: anchor not found.
-        // If the remote SHA doesn't exist locally (filtered commit from a prior push),
-        // tags can be safely skipped (immutable — already correct on Codeberg).
-        // Branches fall back to orphan filtering (best effort).
+        // Fallback: anchor not found — branch only (tags exit early above).
         const remoteExists = !!gitOptional(`rev-parse --verify "${actualRemoteSha}"`);
-        if (!remoteExists && remoteRef.startsWith('refs/tags/')) {
-          console.log(`  ↷ ${label} already on Codeberg — skipping`);
-          continue;
-        }
         console.warn(`  ⚠ Could not find local base for ${label}, filtering tip only`);
         shaMap[localSha] = makeFilteredCommit(localSha, filterTree(localSha), remoteExists ? [actualRemoteSha] : []);
       }

package/server.json

@@ -3,12 +3,12 @@
   "name": "io.github.dpesch/mantisbt-mcp-server",
   "title": "MantisBT MCP Server",
   "description": "MantisBT MCP server – manage issues, notes, files, tags, and relationships. With semantic search.",
-  "version": "1.8.1",
+  "version": "1.8.2",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "@dpesch/mantisbt-mcp-server",
-      "version": "1.8.1",
+      "version": "1.8.2",
       "runtimeHint": "npx",
       "transport": {
         "type": "stdio"