@freelancercom/phabricator-mcp 1.0.0 → 1.0.2

package/README.md

@@ -11,7 +11,7 @@ An [MCP (Model Context Protocol)](https://modelcontextprotocol.io/) server that
 ### Claude Code (CLI)
 
 ```bash
-claude mcp add --scope user phabricator -- npx github:freelancer/phabricator-mcp
+claude mcp add --scope user phabricator -- npx @freelancercom/phabricator-mcp@latest
 ```
 
 Or with environment variables (if not using `~/.arcrc`):
@@ -20,7 +20,7 @@ Or with environment variables (if not using `~/.arcrc`):
 claude mcp add --scope user phabricator \
   -e PHABRICATOR_URL=https://phabricator.example.com \
   -e PHABRICATOR_API_TOKEN=api-xxxxx \
-  -- npx github:freelancer/phabricator-mcp
+  -- npx @freelancercom/phabricator-mcp@latest
 ```
 
 The `--scope user` flag installs the server globally, making it available in all projects.
@@ -34,7 +34,7 @@ Add to your Codex config (`~/.codex/config.json`):
   "mcpServers": {
     "phabricator": {
       "command": "npx",
-      "args": ["github:freelancer/phabricator-mcp"],
+      "args": ["@freelancercom/phabricator-mcp@latest"],
       "env": {
         "PHABRICATOR_URL": "https://phabricator.example.com",
         "PHABRICATOR_API_TOKEN": "api-xxxxxxxxxxxxx"
@@ -54,7 +54,7 @@ Add to your opencode config (`~/.config/opencode/config.json`):
     "servers": {
       "phabricator": {
         "command": "npx",
-        "args": ["github:freelancer/phabricator-mcp"],
+        "args": ["@freelancercom/phabricator-mcp@latest"],
         "env": {
           "PHABRICATOR_URL": "https://phabricator.example.com",
           "PHABRICATOR_API_TOKEN": "api-xxxxxxxxxxxxx"
@@ -74,7 +74,7 @@ Add to your VS Code `settings.json`:
   "claude.mcpServers": {
     "phabricator": {
       "command": "npx",
-      "args": ["github:freelancer/phabricator-mcp"],
+      "args": ["@freelancercom/phabricator-mcp@latest"],
       "env": {
         "PHABRICATOR_URL": "https://phabricator.example.com",
         "PHABRICATOR_API_TOKEN": "api-xxxxxxxxxxxxx"
@@ -93,7 +93,7 @@ Add to your Cursor MCP config (`~/.cursor/mcp.json`):
   "mcpServers": {
     "phabricator": {
       "command": "npx",
-      "args": ["github:freelancer/phabricator-mcp"],
+      "args": ["@freelancercom/phabricator-mcp@latest"],
       "env": {
         "PHABRICATOR_URL": "https://phabricator.example.com",
         "PHABRICATOR_API_TOKEN": "api-xxxxxxxxxxxxx"
@@ -112,7 +112,7 @@ Add to your VS Code `settings.json`:
   "github.copilot.chat.mcp.servers": {
     "phabricator": {
       "command": "npx",
-      "args": ["github:freelancer/phabricator-mcp"],
+      "args": ["@freelancercom/phabricator-mcp@latest"],
       "env": {
         "PHABRICATOR_URL": "https://phabricator.example.com",
         "PHABRICATOR_API_TOKEN": "api-xxxxxxxxxxxxx"
@@ -122,6 +122,32 @@ Add to your VS Code `settings.json`:
 }
 ```
 
+## Upgrading
+
+The default install uses `@freelancercom/phabricator-mcp@latest`, which tells npx to check for updates on each run. No action needed.
+
+If you pinned a specific version (e.g. `@freelancercom/phabricator-mcp@1.0.0`) or omitted the version suffix, npx caches the package and won't pick up new versions. To upgrade:
+
+```bash
+npx clear-npx-cache
+```
+
+Then restart your MCP client.
+
+### Migrating from `github:freelancer/phabricator-mcp`
+
+If you previously installed using the GitHub URL, update your config to use the npm package instead:
+
+```bash
+# Remove old server
+claude mcp remove phabricator -s user
+
+# Add new one
+claude mcp add --scope user phabricator -- npx @freelancercom/phabricator-mcp@latest
+```
+
+For JSON configs, replace `["github:freelancer/phabricator-mcp"]` with `["@freelancercom/phabricator-mcp@latest"]` in your args.
+
 ## Configuration
 
 The server automatically reads configuration from `~/.arcrc` (created by [Arcanist](https://secure.phabricator.com/book/phabricator/article/arcanist/)). No additional configuration is needed if you've already set up `arc`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@freelancercom/phabricator-mcp",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "MCP server for Phabricator Conduit API - manage tasks, code reviews, repositories, and more",
   "type": "module",
   "main": "dist/index.js",

package/dist/client/conduit.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/client/conduit.test.js

@@ -1,97 +0,0 @@
-import { describe, it, mock, beforeEach, afterEach } from 'node:test';
-import assert from 'node:assert';
-import { ConduitClient, ConduitError } from './conduit.js';
-describe('ConduitClient', () => {
-    const mockConfig = {
-        phabricatorUrl: 'https://phabricator.example.com',
-        apiToken: 'api-test-token',
-    };
-    let originalFetch;
-    beforeEach(() => {
-        originalFetch = global.fetch;
-    });
-    afterEach(() => {
-        global.fetch = originalFetch;
-    });
-    it('should construct correct API URL', async () => {
-        let capturedUrl;
-        global.fetch = mock.fn(async (url) => {
-            capturedUrl = url;
-            return new Response(JSON.stringify({ result: {}, error_code: null, error_info: null }));
-        });
-        const client = new ConduitClient(mockConfig);
-        await client.call('user.whoami');
-        assert.strictEqual(capturedUrl, 'https://phabricator.example.com/api/user.whoami');
-    });
-    it('should include API token in request body', async () => {
-        let capturedBody;
-        global.fetch = mock.fn(async (_url, init) => {
-            capturedBody = init?.body;
-            return new Response(JSON.stringify({ result: {}, error_code: null, error_info: null }));
-        });
-        const client = new ConduitClient(mockConfig);
-        await client.call('user.whoami');
-        assert.ok(capturedBody);
-        const params = new URLSearchParams(capturedBody);
-        const paramsJson = JSON.parse(params.get('params'));
-        assert.strictEqual(paramsJson.__conduit__.token, 'api-test-token');
-    });
-    it('should pass parameters to the API', async () => {
-        let capturedBody;
-        global.fetch = mock.fn(async (_url, init) => {
-            capturedBody = init?.body;
-            return new Response(JSON.stringify({ result: {}, error_code: null, error_info: null }));
-        });
-        const client = new ConduitClient(mockConfig);
-        await client.call('maniphest.search', { queryKey: 'assigned', limit: 10 });
-        const params = new URLSearchParams(capturedBody);
-        const paramsJson = JSON.parse(params.get('params'));
-        assert.strictEqual(paramsJson.queryKey, 'assigned');
-        assert.strictEqual(paramsJson.limit, 10);
-    });
-    it('should return result on success', async () => {
-        const expectedResult = { userName: 'testuser', realName: 'Test User' };
-        global.fetch = mock.fn(async () => {
-            return new Response(JSON.stringify({ result: expectedResult, error_code: null, error_info: null }));
-        });
-        const client = new ConduitClient(mockConfig);
-        const result = await client.call('user.whoami');
-        assert.deepStrictEqual(result, expectedResult);
-    });
-    it('should throw ConduitError on API error', async () => {
-        global.fetch = mock.fn(async () => {
-            return new Response(JSON.stringify({
-                result: null,
-                error_code: 'ERR-CONDUIT-CORE',
-                error_info: 'Invalid token',
-            }));
-        });
-        const client = new ConduitClient(mockConfig);
-        await assert.rejects(() => client.call('user.whoami'), (err) => {
-            assert.ok(err instanceof ConduitError);
-            assert.strictEqual(err.code, 'ERR-CONDUIT-CORE');
-            assert.strictEqual(err.message, 'Invalid token');
-            return true;
-        });
-    });
-    it('should throw ConduitError on HTTP error', async () => {
-        global.fetch = mock.fn(async () => {
-            return new Response('Not Found', { status: 404, statusText: 'Not Found' });
-        });
-        const client = new ConduitClient(mockConfig);
-        await assert.rejects(() => client.call('user.whoami'), (err) => {
-            assert.ok(err instanceof ConduitError);
-            assert.strictEqual(err.code, 'HTTP_ERROR');
-            assert.ok(err.message.includes('404'));
-            return true;
-        });
-    });
-});
-describe('ConduitError', () => {
-    it('should have correct name and properties', () => {
-        const error = new ConduitError('TEST_CODE', 'Test message');
-        assert.strictEqual(error.name, 'ConduitError');
-        assert.strictEqual(error.code, 'TEST_CODE');
-        assert.strictEqual(error.message, 'Test message');
-    });
-});

package/dist/config.test.d.ts

@@ -1 +0,0 @@
-export {};

package/dist/config.test.js

@@ -1,54 +0,0 @@
-import { describe, it, beforeEach, afterEach } from 'node:test';
-import assert from 'node:assert';
-describe('loadConfig', () => {
-    const originalEnv = { ...process.env };
-    beforeEach(() => {
-        // Clear relevant env vars
-        delete process.env.PHABRICATOR_URL;
-        delete process.env.PHABRICATOR_API_TOKEN;
-    });
-    afterEach(() => {
-        process.env = { ...originalEnv };
-    });
-    it('should load config from environment variables', async () => {
-        process.env.PHABRICATOR_URL = 'https://phabricator.example.com';
-        process.env.PHABRICATOR_API_TOKEN = 'api-test-token';
-        // Re-import to get fresh module
-        const { loadConfig } = await import('./config.js');
-        const config = loadConfig();
-        assert.strictEqual(config.phabricatorUrl, 'https://phabricator.example.com');
-        assert.strictEqual(config.apiToken, 'api-test-token');
-    });
-    it('should strip trailing slash from URL', async () => {
-        process.env.PHABRICATOR_URL = 'https://phabricator.example.com/';
-        process.env.PHABRICATOR_API_TOKEN = 'api-test-token';
-        const { loadConfig } = await import('./config.js');
-        const config = loadConfig();
-        assert.strictEqual(config.phabricatorUrl, 'https://phabricator.example.com');
-    });
-    it('should throw error for invalid URL', async () => {
-        process.env.PHABRICATOR_URL = 'not-a-valid-url';
-        process.env.PHABRICATOR_API_TOKEN = 'api-test-token';
-        const { loadConfig } = await import('./config.js?v=1');
-        assert.throws(() => loadConfig(), /Invalid url/);
-    });
-    it('should throw error for empty token', async () => {
-        process.env.PHABRICATOR_URL = 'https://phabricator.example.com';
-        process.env.PHABRICATOR_API_TOKEN = '';
-        // When token is empty string, it should either throw or fall back to arcrc
-        // Since arcrc exists on this machine, it will use that - so we skip this test
-        // if arcrc is present. The important thing is it doesn't accept empty string.
-        const { loadConfig } = await import('./config.js?v=2');
-        // This test verifies the schema validation - empty string should fail zod validation
-        // but it may fall back to arcrc first, so we just verify it doesn't crash
-        try {
-            const config = loadConfig();
-            // If it succeeds, it used arcrc fallback which is fine
-            assert.ok(config.apiToken.length > 0);
-        }
-        catch (e) {
-            // If it throws, that's also fine - means it correctly rejected empty token
-            assert.ok(e.message.includes('too_small') || e.message.includes('API_TOKEN'));
-        }
-    });
-});