@pay2s/mcp 0.4.0 → 0.5.0

package/README.md

@@ -1,3 +1,5 @@
+> **0.5.0 security migration:** OAuth is now owned by `https://api.pay2s.vn`. Upgrade backend, gateway, frontend and npm together; existing users must log in again. See `../pay2s-mcp/UPGRADE-0.5.0.md`. Older installation instructions below are historical and do not override that guide.
+
 # Pay2S MCP
 
 Kết nối Codex, Cursor và ứng dụng hỗ trợ MCP stdio với tài khoản Pay2S. Cần Node.js 22 trở lên trên máy chạy AI và máy chủ Pay2S MCP 0.4 đã đăng ký client `pay2s-cli`.
@@ -49,10 +51,11 @@ Tham khảo cấu hình stdio: https://learn.chatgpt.com/docs/extend/mcp?surface
 
 - Xem ngân hàng, giao dịch, thu chi, đơn và trạng thái thanh toán trong phạm vi được cấp.
 - Tạo link thanh toán khi được cấp quyền riêng.
-- Đề nghị thêm webhook; đề nghị bật/tắt ngân hàng hoặc webhook. AI trả đường dẫn để chủ tài khoản xem và xác nhận trên Pay2S. Gửi đề nghị chưa làm thay đổi trạng thái.
+- Bật/tắt ngân hàng và webhook trực tiếp bằng `set_bank_status`, `set_webhook_status` khi được cấp quyền tương ứng. AI trả trạng thái thực tế ngay, không tạo đề nghị chờ xác nhận.
+- Đề nghị tạo webhook mới trả đường dẫn để chủ tài khoản kiểm tra địa chỉ nhận trên Pay2S.
 - `--docs`: chỉ đọc tài liệu công khai, không đăng nhập.
 
-Kết nối cũ cần đăng nhập lại để chọn quyền mới. Ngân hàng đang tắt vẫn xuất hiện để đề nghị bật lại; truy vấn giao dịch vẫn cần ngân hàng hoạt động. Bật webhook không gửi lại giao dịch cũ.
+Kết nối đã có `banks:manage` và `webhooks:manage` dùng ngay công cụ bật/tắt sau khi máy chủ cập nhật 0.4.1 và AI tải lại danh sách công cụ. Chỉ đăng nhập lại nếu chưa cấp hai quyền này. Ngân hàng đang tắt vẫn xuất hiện để bật lại; truy vấn giao dịch vẫn cần ngân hàng hoạt động. Bật webhook không gửi lại giao dịch cũ.
 
 ## Phiên đăng nhập
 
@@ -74,3 +77,5 @@ pay2s-mcp login --profile codex
 ```
 
 Cấu hình ứng dụng lúc thử bản này dùng `command = "pay2s-mcp"`, `args = ["--profile", "codex"]`. Không chạy `npx @pay2s/mcp` trước khi package đã xuất bản.
+
+Máy chủ 0.4.2 bổ sung `create_webhook`: tạo ngay trong quyền `webhooks:request` đã cấp, không quay lại Pay2S duyệt. Gói npm hiện tại tương thích; khởi động lại kết nối để cập nhật danh sách công cụ.

package/bin/pay2s-mcp.mjs

@@ -30,6 +30,6 @@ try {
   else await close();
 } catch (error) {
   // SDK/HTTP exceptions may contain response bodies; never print them or credentials.
-  const safe = error.message?.startsWith('Profile ') ? error.message : error.code === 'EADDRINUSE' ? 'Port 14572 is busy. Finish the other Pay2S login and retry.' : 'Cannot connect to Pay2S. Check Node 22+, network, deployed MCP 0.4 and account access. Run login with the same --profile again.';
+  const safe = error.message?.startsWith('Profile ') ? error.message : error.code === 'EADDRINUSE' ? 'Port 14572 is busy. Finish the other Pay2S login and retry.' : 'Cannot connect to Pay2S. Check Node 22+, network, deployed MCP and API backend 0.5 and account access. Run login with the same --profile again.';
   process.stderr.write(`${safe}\n`); await close(); process.exitCode = 1;
 }

package/package.json

@@ -1,13 +1,13 @@
-{
-  "name": "@pay2s/mcp",
-  "version": "0.4.0",
-  "description": "Connect AI tools to Pay2S through OAuth and MCP",
-  "type": "module",
-  "license": "UNLICENSED",
-  "bin": { "pay2s-mcp": "bin/pay2s-mcp.mjs" },
-  "files": ["bin/", "src/", "README.md"],
-  "engines": { "node": ">=22" },
-  "publishConfig": { "access": "public", "registry": "https://registry.npmjs.org/" },
-  "scripts": { "test": "node --test test/*.test.mjs" },
-  "dependencies": { "@modelcontextprotocol/sdk": "1.31.0" }
-}
+{
+  "name": "@pay2s/mcp",
+  "version": "0.5.0",
+  "description": "Connect AI tools to Pay2S through OAuth and MCP",
+  "type": "module",
+  "license": "UNLICENSED",
+  "bin": { "pay2s-mcp": "bin/pay2s-mcp.mjs" },
+  "files": ["bin/", "src/", "README.md"],
+  "engines": { "node": ">=22" },
+  "publishConfig": { "access": "public", "registry": "https://registry.npmjs.org/" },
+  "scripts": { "test": "node --test test/*.test.mjs" },
+  "dependencies": { "@modelcontextprotocol/sdk": "1.31.0" }
+}

package/src/auth.mjs

@@ -2,15 +2,31 @@ import { createServer } from 'node:http';
 import { randomBytes, timingSafeEqual } from 'node:crypto';
 import { execFile } from 'node:child_process';
 import { auth } from '@modelcontextprotocol/sdk/client/auth.js';
-export const ISSUER = 'https://mcp.pay2s.vn';
-export const RESOURCE = `${ISSUER}/mcp`;
+export const ISSUER = 'https://api.pay2s.vn';
+export const MCP_ORIGIN = 'https://mcp.pay2s.vn';
+export const RESOURCE = `${MCP_ORIGIN}/mcp`;
+export const AUTHORIZE = `${ISSUER}/api/v1/mcp/oauth/authorize`;
+export const TOKEN = `${ISSUER}/api/v1/mcp/oauth/token`;
 export const REDIRECT = 'http://127.0.0.1:14572/callback';
 export const SCOPES = 'accounts:read transactions:read payments:read payments:create webhooks:read webhooks:request banks:manage webhooks:manage';
-export async function trustedFetch(input, init = {}) {
+export async function trustedFetch(input, init = {}, transport = fetch) {
   const url = new URL(typeof input === 'string' || input instanceof URL ? input : input.url);
-  if (url.origin !== ISSUER || url.username || url.password) throw Error('Unexpected Pay2S server');
+  const method=(init.method || input.method || 'GET').toUpperCase();
+  const headers=new Headers(init.headers || input.headers);
+  const publicMetadata=url.origin===MCP_ORIGIN && ['/.well-known/oauth-protected-resource','/.well-known/oauth-protected-resource/mcp'].includes(url.pathname);
+  const authMetadata=url.origin===ISSUER && url.pathname==='/.well-known/oauth-authorization-server';
+  const resource=url.origin===MCP_ORIGIN && ['/mcp','/docs'].includes(url.pathname);
+  const token=url.href===TOKEN;
+  if(url.username||url.password||url.search||url.hash||!((method==='GET'&&(publicMetadata||authMetadata))||(method==='POST'&&resource&&headers.get('content-type')?.startsWith('application/json'))||(method==='POST'&&token&&headers.get('content-type')?.startsWith('application/x-www-form-urlencoded'))))throw Error('Unexpected Pay2S server or endpoint');
+  if(headers.has('authorization') && !(resource && url.pathname==='/mcp'))throw Error('Unexpected credential destination');
   const signal = init.signal ? AbortSignal.any([init.signal, AbortSignal.timeout(45000)]) : AbortSignal.timeout(45000);
-  return fetch(input, { ...init, redirect: 'error', signal });
+  const response=await transport(input, { ...init, redirect: 'error', signal });
+  if(response.ok && (publicMetadata||authMetadata)){
+    const data=await response.clone().json();
+    if(publicMetadata && (data.resource!==RESOURCE||!Array.isArray(data.authorization_servers)||data.authorization_servers.length!==1||data.authorization_servers[0]!==ISSUER))throw Error('Unexpected authorization server');
+    if(authMetadata && (data.issuer!==ISSUER||data.authorization_endpoint!==AUTHORIZE||data.token_endpoint!==TOKEN||!data.code_challenge_methods_supported?.includes('S256')))throw Error('Unexpected authorization endpoints');
+  }
+  return response;
 }
 export function callbackHandler(state, accept) {
   let used = false;
@@ -36,7 +52,7 @@ export function callbackHandler(state, accept) {
 function openBrowser(url) {
   // Fixed HTTPS origin and URL serialization; no shell command interpolation.
   const target = new URL(url);
-  if (target.origin !== ISSUER || target.pathname !== '/oauth/authorize') throw Error('Unexpected authorization URL');
+  if (target.origin !== ISSUER || target.pathname !== '/api/v1/mcp/oauth/authorize') throw Error('Unexpected authorization URL');
   const command = process.platform === 'win32' ? 'rundll32.exe' : process.platform === 'darwin' ? 'open' : 'xdg-open';
   const args = process.platform === 'win32' ? ['url.dll,FileProtocolHandler', target.href] : [target.href];
   return new Promise(resolve => execFile(command, args, { windowsHide: true }, () => resolve()));
@@ -47,13 +63,14 @@ export class Provider {
   get clientMetadata() { return { client_name: 'Pay2S MCP CLI', redirect_uris: [REDIRECT], token_endpoint_auth_method: 'none', grant_types: ['authorization_code','refresh_token'], response_types: ['code'], scope: SCOPES }; }
   clientInformation() { return { client_id: 'pay2s-cli', issuer: ISSUER }; }
   tokens() { const t = this.store.read()?.tokens; return t?.issuer === ISSUER ? t : undefined; }
-  async saveTokens(tokens) { await this.store.save({ tokens, expires_at: Date.now() + Math.max(0, Number(tokens.expires_in || 0)) * 1000 }); }
+  async saveTokens(tokens) { if(tokens.issuer!==ISSUER)throw Error('Unexpected token issuer');await this.store.save({ tokens, expires_at: Date.now() + Math.max(0, Number(tokens.expires_in || 0)) * 1000 }); }
   async invalidateCredentials(scope) { if (['all','tokens'].includes(scope)) await this.store.save(undefined); if (['all','verifier'].includes(scope)) this.verifier = undefined; }
   saveCodeVerifier(value) { this.verifier = value; }
   codeVerifier() { if (!this.verifier) throw Error('Missing code verifier'); return this.verifier; }
   validateResourceURL(_url, resource) { if (resource !== RESOURCE) throw Error('Unexpected resource'); return Promise.resolve(new URL(RESOURCE)); }
   state() { this.nonce = randomBytes(32).toString('hex'); return this.nonce; }
   async redirectToAuthorization(url) {
+    if(url.origin!==ISSUER||url.pathname!=='/api/v1/mcp/oauth/authorize')throw Error('Unexpected authorization URL');
     if (!this.interactive) throw Error('Login required. Run npx -y @pay2s/mcp login with the same --profile.');
     this.callback = new Promise(resolve => { this.receive = resolve; });
     this.server = createServer(callbackHandler(this.nonce, this.receive));

package/src/proxy.mjs

@@ -1,22 +1,22 @@
-import { Client } from '@modelcontextprotocol/sdk/client/index.js';
-import { StreamableHTTPClientTransport } from '@modelcontextprotocol/sdk/client/streamableHttp.js';
-import { Server } from '@modelcontextprotocol/sdk/server/index.js';
-import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
-import { ListToolsRequestSchema, CallToolRequestSchema, CallToolResultSchema } from '@modelcontextprotocol/sdk/types.js';
-import { ISSUER, trustedFetch } from './auth.mjs';
-export async function proxy(provider, docs = false) {
-  const transport = new StreamableHTTPClientTransport(new URL(`${ISSUER}/${docs ? 'docs' : 'mcp'}`), { authProvider: provider, fetch: trustedFetch });
-  return connectProxy(transport, new StdioServerTransport());
-}
-export async function connectProxy(transport, localTransport) {
-  const remote = new Client({ name: '@pay2s/mcp', version: '0.4.0' });
-  await remote.connect(transport);
-  const local = new Server({ name: 'pay2s', version: '0.4.0' }, { capabilities: { tools: {} }, instructions: remote.getInstructions() });
-  // Serialize remote calls, including SDK token refresh. Never share a refresh token across processes.
-  let tail = Promise.resolve();
-  const enqueue = fn => { const next = tail.then(fn).catch(() => { throw Error('Pay2S request failed. Check your connection or sign in again.'); }); tail = next.catch(() => {}); return next; };
-  local.setRequestHandler(ListToolsRequestSchema, request => enqueue(() => remote.listTools(request.params)));
-  local.setRequestHandler(CallToolRequestSchema, request => enqueue(() => remote.callTool(request.params, CallToolResultSchema, { timeout: 60000 })));
-  await local.connect(localTransport);
-  return { async close() { await local.close(); await remote.close(); } };
-}
+import { Client } from '@modelcontextprotocol/sdk/client/index.js';
+import { StreamableHTTPClientTransport } from '@modelcontextprotocol/sdk/client/streamableHttp.js';
+import { Server } from '@modelcontextprotocol/sdk/server/index.js';
+import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
+import { ListToolsRequestSchema, CallToolRequestSchema, CallToolResultSchema } from '@modelcontextprotocol/sdk/types.js';
+import { MCP_ORIGIN, trustedFetch } from './auth.mjs';
+export async function proxy(provider, docs = false) {
+  const transport = new StreamableHTTPClientTransport(new URL(`${MCP_ORIGIN}/${docs ? 'docs' : 'mcp'}`), { authProvider: provider, fetch: trustedFetch });
+  return connectProxy(transport, new StdioServerTransport());
+}
+export async function connectProxy(transport, localTransport) {
+  const remote = new Client({ name: '@pay2s/mcp', version: '0.5.0' });
+  await remote.connect(transport);
+  const local = new Server({ name: 'pay2s', version: '0.5.0' }, { capabilities: { tools: {} }, instructions: remote.getInstructions() });
+  // Serialize remote calls, including SDK token refresh. Never share a refresh token across processes.
+  let tail = Promise.resolve();
+  const enqueue = fn => { const next = tail.then(fn).catch(() => { throw Error('Pay2S request failed. Check your connection or sign in again.'); }); tail = next.catch(() => {}); return next; };
+  local.setRequestHandler(ListToolsRequestSchema, request => enqueue(() => remote.listTools(request.params)));
+  local.setRequestHandler(CallToolRequestSchema, request => enqueue(() => remote.callTool(request.params, CallToolResultSchema, { timeout: 60000 })));
+  await local.connect(localTransport);
+  return { async close() { await local.close(); await remote.close(); } };
+}