@guiie/buda-mcp 1.1.1 → 1.1.2

package/.github/workflows/publish.yml

@@ -36,7 +36,15 @@ jobs:
           cache: "npm"
       - run: npm ci
       - run: npm run build
-      - run: npm publish --access public --provenance
+      - name: Publish (skip if version already exists)
+        run: |
+          PKG=$(node -e "console.log(require('./package.json').name)")
+          VER=$(node -e "console.log(require('./package.json').version)")
+          if npm view "${PKG}@${VER}" version &>/dev/null; then
+            echo "Version ${VER} already published — skipping npm publish."
+          else
+            npm publish --access public --provenance
+          fi
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
 
@@ -45,6 +53,8 @@ jobs:
     name: Publish to MCP Registry
     needs: npm
     runs-on: ubuntu-latest
+    env:
+      MCP_REGISTRY_TOKEN: ${{ secrets.MCP_REGISTRY_TOKEN }}
     steps:
       - uses: actions/checkout@v4
       - name: Install mcp-publisher
@@ -52,7 +62,7 @@ jobs:
           curl -L "https://github.com/modelcontextprotocol/registry/releases/latest/download/mcp-publisher_linux_amd64.tar.gz" \
             | tar xz mcp-publisher
           sudo mv mcp-publisher /usr/local/bin/
-      - name: Publish to MCP Registry
-        run: mcp-publisher publish
-        env:
-          MCP_REGISTRY_TOKEN: ${{ secrets.MCP_REGISTRY_TOKEN }}
+      - name: Authenticate and publish to MCP Registry
+        run: |
+          mcp-publisher login token "$MCP_REGISTRY_TOKEN"
+          mcp-publisher publish

package/CHANGELOG.md

@@ -7,6 +7,20 @@ This project uses [Semantic Versioning](https://semver.org/).
 
 ---
 
+## [1.1.2] – 2026-04-10
+
+### Fixed
+
+- **`get_price_history` OHLCV open/close reversed**: Buda returns trades newest-first; entries
+  are now sorted ascending by timestamp before candle aggregation so `open` is the
+  chronologically first price and `close` is the chronologically last price in each bucket.
+- **Cache double-fetch race condition**: `MemoryCache.getOrFetch` now deduplicates concurrent
+  requests for the same expired/missing key by storing the in-flight `Promise` and returning it
+  to all concurrent callers instead of spawning a second fetch.
+- **`User-Agent` version string**: corrected from `buda-mcp/1.1.0` to `buda-mcp/1.1.1` (and now `1.1.2`).
+
+---
+
 ## [1.1.1] – 2026-04-10
 
 ### Fixed

package/dist/cache.d.ts

@@ -1,5 +1,6 @@
 export declare class MemoryCache {
     private store;
+    private inflight;
     getOrFetch<T>(key: string, ttlMs: number, fetcher: () => Promise<T>): Promise<T>;
     invalidate(key: string): void;
     clear(): void;

package/dist/cache.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cache.d.ts","sourceRoot":"","sources":["../src/cache.ts"],"names":[],"mappings":"AAKA,qBAAa,WAAW;IACtB,OAAO,CAAC,KAAK,CAA0C;IAEjD,UAAU,CAAC,CAAC,EAAE,GAAG,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC;IAUtF,UAAU,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI;IAI7B,KAAK,IAAI,IAAI;CAGd;AAED,eAAO,MAAM,SAAS;;;;CAIZ,CAAC;AAEX,eAAO,MAAM,KAAK,aAAoB,CAAC"}
+{"version":3,"file":"cache.d.ts","sourceRoot":"","sources":["../src/cache.ts"],"names":[],"mappings":"AAKA,qBAAa,WAAW;IACtB,OAAO,CAAC,KAAK,CAA0C;IACvD,OAAO,CAAC,QAAQ,CAAuC;IAEjD,UAAU,CAAC,CAAC,EAAE,GAAG,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,CAAC,CAAC;IAyBtF,UAAU,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI;IAI7B,KAAK,IAAI,IAAI;CAGd;AAED,eAAO,MAAM,SAAS;;;;CAIZ,CAAC;AAEX,eAAO,MAAM,KAAK,aAAoB,CAAC"}

package/dist/cache.js

@@ -1,13 +1,27 @@
 export class MemoryCache {
     store = new Map();
+    inflight = new Map();
     async getOrFetch(key, ttlMs, fetcher) {
         const entry = this.store.get(key);
         if (entry && Date.now() < entry.expiry) {
             return entry.data;
         }
-        const data = await fetcher();
-        this.store.set(key, { data, expiry: Date.now() + ttlMs });
-        return data;
+        // Deduplicate concurrent requests for the same expired/missing key.
+        const pending = this.inflight.get(key);
+        if (pending)
+            return pending;
+        const promise = fetcher()
+            .then((data) => {
+            this.store.set(key, { data, expiry: Date.now() + ttlMs });
+            this.inflight.delete(key);
+            return data;
+        })
+            .catch((err) => {
+            this.inflight.delete(key);
+            throw err;
+        });
+        this.inflight.set(key, promise);
+        return promise;
     }
     invalidate(key) {
         this.store.delete(key);

package/dist/client.js

@@ -55,7 +55,7 @@ export class BudaClient {
         const urlPath = url.pathname + url.search;
         const headers = {
             Accept: "application/json",
-            "User-Agent": "buda-mcp/1.1.0",
+            "User-Agent": "buda-mcp/1.1.1",
             ...this.authHeaders("GET", urlPath),
         };
         const response = await fetch(url.toString(), { headers });
@@ -80,7 +80,7 @@ export class BudaClient {
         const headers = {
             Accept: "application/json",
             "Content-Type": "application/json",
-            "User-Agent": "buda-mcp/1.1.0",
+            "User-Agent": "buda-mcp/1.1.1",
             ...this.authHeaders("POST", urlPath, bodyStr),
         };
         const response = await fetch(url.toString(), {
@@ -109,7 +109,7 @@ export class BudaClient {
         const headers = {
             Accept: "application/json",
             "Content-Type": "application/json",
-            "User-Agent": "buda-mcp/1.1.0",
+            "User-Agent": "buda-mcp/1.1.1",
             ...this.authHeaders("PUT", urlPath, bodyStr),
         };
         const response = await fetch(url.toString(), {

package/dist/http.js

@@ -19,7 +19,7 @@ const PORT = parseInt(process.env.PORT ?? "3000", 10);
 const client = new BudaClient(undefined, process.env.BUDA_API_KEY, process.env.BUDA_API_SECRET);
 const authEnabled = client.hasAuth();
 function createServer() {
-    const server = new McpServer({ name: "buda-mcp", version: "1.1.1" });
+    const server = new McpServer({ name: "buda-mcp", version: "1.1.2" });
     // Per-request cache so caching works correctly for stateless HTTP
     const reqCache = new MemoryCache();
     markets.register(server, client, reqCache);
@@ -68,7 +68,7 @@ const app = express();
 app.use(express.json());
 // Health check for Railway / uptime monitors
 app.get("/health", (_req, res) => {
-    res.json({ status: "ok", server: "buda-mcp", version: "1.1.1", auth_mode: authEnabled ? "authenticated" : "public" });
+    res.json({ status: "ok", server: "buda-mcp", version: "1.1.2", auth_mode: authEnabled ? "authenticated" : "public" });
 });
 // Smithery static server card — lets Smithery scan tools without running the server
 app.get("/.well-known/mcp/server-card.json", (_req, res) => {
@@ -216,7 +216,7 @@ app.get("/.well-known/mcp/server-card.json", (_req, res) => {
         ]
         : [];
     res.json({
-        serverInfo: { name: "buda-mcp", version: "1.1.1" },
+        serverInfo: { name: "buda-mcp", version: "1.1.2" },
         authentication: { required: authEnabled },
         tools: [...publicTools, ...authTools],
         resources: [

package/dist/index.js

@@ -18,7 +18,7 @@ import * as cancelOrder from "./tools/cancel_order.js";
 const client = new BudaClient(undefined, process.env.BUDA_API_KEY, process.env.BUDA_API_SECRET);
 const server = new McpServer({
     name: "buda-mcp",
-    version: "1.1.1",
+    version: "1.1.2",
 });
 // Public tools
 markets.register(server, client, cache);

package/dist/tools/price_history.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"price_history.d.ts","sourceRoot":"","sources":["../../src/tools/price_history.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAEpE,OAAO,EAAE,UAAU,EAAgB,MAAM,cAAc,CAAC;AACxD,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAmB1C,wBAAgB,QAAQ,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,WAAW,GAAG,IAAI,CAqGzF"}
+{"version":3,"file":"price_history.d.ts","sourceRoot":"","sources":["../../src/tools/price_history.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AAEpE,OAAO,EAAE,UAAU,EAAgB,MAAM,cAAc,CAAC;AACxD,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAmB1C,wBAAgB,QAAQ,CAAC,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,WAAW,GAAG,IAAI,CA2GzF"}

package/dist/tools/price_history.js

@@ -40,9 +40,12 @@ export function register(server, client, _cache) {
                     ],
                 };
             }
+            // Buda returns trades newest-first; sort ascending so open = first chronological price
+            // and close = last chronological price within each candle bucket.
+            const sortedEntries = [...entries].sort(([a], [b]) => parseInt(a, 10) - parseInt(b, 10));
             const periodMs = PERIOD_MS[period];
             const buckets = new Map();
-            for (const [tsMs, amount, price, _direction] of entries) {
+            for (const [tsMs, amount, price, _direction] of sortedEntries) {
                 const ts = parseInt(tsMs, 10);
                 const bucketStart = Math.floor(ts / periodMs) * periodMs;
                 const p = parseFloat(price);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@guiie/buda-mcp",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "mcpName": "io.github.gtorreal/buda-mcp",
   "description": "MCP server for Buda.com's public cryptocurrency exchange API (Chile, Colombia, Peru)",
   "type": "module",

package/server.json

@@ -6,12 +6,12 @@
     "url": "https://github.com/gtorreal/buda-mcp",
     "source": "github"
   },
-  "version": "1.1.1",
+  "version": "1.1.2",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "@guiie/buda-mcp",
-      "version": "1.1.1",
+      "version": "1.1.2",
       "transport": {
         "type": "stdio"
       }

package/src/cache.ts

@@ -5,15 +5,31 @@ interface CacheEntry<T> {
 
 export class MemoryCache {
   private store = new Map<string, CacheEntry<unknown>>();
+  private inflight = new Map<string, Promise<unknown>>();
 
   async getOrFetch<T>(key: string, ttlMs: number, fetcher: () => Promise<T>): Promise<T> {
     const entry = this.store.get(key) as CacheEntry<T> | undefined;
     if (entry && Date.now() < entry.expiry) {
       return entry.data;
     }
-    const data = await fetcher();
-    this.store.set(key, { data, expiry: Date.now() + ttlMs });
-    return data;
+
+    // Deduplicate concurrent requests for the same expired/missing key.
+    const pending = this.inflight.get(key) as Promise<T> | undefined;
+    if (pending) return pending;
+
+    const promise = fetcher()
+      .then((data) => {
+        this.store.set(key, { data, expiry: Date.now() + ttlMs });
+        this.inflight.delete(key);
+        return data;
+      })
+      .catch((err: unknown) => {
+        this.inflight.delete(key);
+        throw err;
+      });
+
+    this.inflight.set(key, promise as Promise<unknown>);
+    return promise;
   }
 
   invalidate(key: string): void {

package/src/client.ts

@@ -68,7 +68,7 @@ export class BudaClient {
     const urlPath = url.pathname + url.search;
     const headers: Record<string, string> = {
       Accept: "application/json",
-      "User-Agent": "buda-mcp/1.1.0",
+      "User-Agent": "buda-mcp/1.1.1",
       ...this.authHeaders("GET", urlPath),
     };
 
@@ -95,7 +95,7 @@ export class BudaClient {
     const headers: Record<string, string> = {
       Accept: "application/json",
       "Content-Type": "application/json",
-      "User-Agent": "buda-mcp/1.1.0",
+      "User-Agent": "buda-mcp/1.1.1",
       ...this.authHeaders("POST", urlPath, bodyStr),
     };
 
@@ -126,7 +126,7 @@ export class BudaClient {
     const headers: Record<string, string> = {
       Accept: "application/json",
       "Content-Type": "application/json",
-      "User-Agent": "buda-mcp/1.1.0",
+      "User-Agent": "buda-mcp/1.1.1",
       ...this.authHeaders("PUT", urlPath, bodyStr),
     };
 

package/src/http.ts

@@ -28,7 +28,7 @@ const client = new BudaClient(
 const authEnabled = client.hasAuth();
 
 function createServer(): McpServer {
-  const server = new McpServer({ name: "buda-mcp", version: "1.1.1" });
+  const server = new McpServer({ name: "buda-mcp", version: "1.1.2" });
 
   // Per-request cache so caching works correctly for stateless HTTP
   const reqCache = new MemoryCache();
@@ -101,7 +101,7 @@ app.use(express.json());
 
 // Health check for Railway / uptime monitors
 app.get("/health", (_req, res) => {
-  res.json({ status: "ok", server: "buda-mcp", version: "1.1.1", auth_mode: authEnabled ? "authenticated" : "public" });
+  res.json({ status: "ok", server: "buda-mcp", version: "1.1.2", auth_mode: authEnabled ? "authenticated" : "public" });
 });
 
 // Smithery static server card — lets Smithery scan tools without running the server
@@ -252,7 +252,7 @@ app.get("/.well-known/mcp/server-card.json", (_req, res) => {
     : [];
 
   res.json({
-    serverInfo: { name: "buda-mcp", version: "1.1.1" },
+    serverInfo: { name: "buda-mcp", version: "1.1.2" },
     authentication: { required: authEnabled },
     tools: [...publicTools, ...authTools],
     resources: [

package/src/index.ts

@@ -25,7 +25,7 @@ const client = new BudaClient(
 
 const server = new McpServer({
   name: "buda-mcp",
-  version: "1.1.1",
+  version: "1.1.2",
 });
 
 // Public tools

package/src/tools/price_history.ts

@@ -65,10 +65,16 @@ export function register(server: McpServer, client: BudaClient, _cache: MemoryCa
           };
         }
 
+        // Buda returns trades newest-first; sort ascending so open = first chronological price
+        // and close = last chronological price within each candle bucket.
+        const sortedEntries = [...entries].sort(
+          ([a], [b]) => parseInt(a, 10) - parseInt(b, 10),
+        );
+
         const periodMs = PERIOD_MS[period];
         const buckets = new Map<number, OhlcvCandle>();
 
-        for (const [tsMs, amount, price, _direction] of entries) {
+        for (const [tsMs, amount, price, _direction] of sortedEntries) {
           const ts = parseInt(tsMs, 10);
           const bucketStart = Math.floor(ts / periodMs) * periodMs;
           const p = parseFloat(price);