@hanhnd/agent-kit 1.0.42 → 1.0.44

package/dist/mcp/jira.test.d.ts

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

package/dist/mcp/jira.test.js

@@ -0,0 +1,37 @@
+import * as assert from 'node:assert/strict';
+import { describe, test } from 'node:test';
+import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
+import { Client } from '@modelcontextprotocol/sdk/client/index.js';
+import { InMemoryTransport } from '@modelcontextprotocol/sdk/inMemory.js';
+import { buildAtlassianContext, registerIntegrationTools } from './integration.js';
+// Both products' tokens are present so J1 can prove they share an email but not a token.
+process.env.ATLASSIAN_CLOUD_ID = 'cloud-a';
+process.env.ATLASSIAN_USER_EMAIL = 'dev@acme.test';
+process.env.JIRA_API_TOKEN = 'token-123';
+process.env.CONFLUENCE_API_TOKEN = 'confluence-token';
+async function buildClient() {
+    const server = new McpServer({ name: 'test-kit', version: '0.0.0' });
+    registerIntegrationTools(server);
+    const [serverTransport, clientTransport] = InMemoryTransport.createLinkedPair();
+    const client = new Client({ name: 'test-client', version: '0.0.0' });
+    await Promise.all([server.connect(serverTransport), client.connect(clientTransport)]);
+    return client;
+}
+describe('kit_jira_get_ticket regression after the buildAtlassianContext refactor (BC21)', () => {
+    test('J1: jira and confluence share the cloud id and email but not the token', () => {
+        const jira = buildAtlassianContext('jira');
+        const confluence = buildAtlassianContext('confluence');
+        const decode = (auth) => Buffer.from(auth.replace('Basic ', ''), 'base64').toString('utf8');
+        assert.equal(jira.cloudId, 'cloud-a');
+        assert.equal(confluence.cloudId, 'cloud-a', 'both products must use the same ATLASSIAN_CLOUD_ID');
+        assert.equal(decode(jira.auth), 'dev@acme.test:token-123');
+        assert.equal(decode(confluence.auth), 'dev@acme.test:confluence-token');
+        assert.notEqual(jira.auth, confluence.auth, 'a per-app scoped token must never cross products');
+    });
+    test('J2: an invalid ticket ID still returns the pre-existing format message', async () => {
+        const client = await buildClient();
+        const result = await client.callTool({ name: 'kit_jira_get_ticket', arguments: { ticketId: 'not-a-ticket' } });
+        const content = result.content;
+        assert.equal(content[0].text, '❌ Invalid ticket ID format: not-a-ticket\n\nExpected format: PROJ-123');
+    });
+});

package/dist/mcp/registration.test.js

@@ -10,7 +10,11 @@ const EXPECTED_NON_MEMORY_TOOLS = [
     'kit_save_handoff',
     'kit_trigger_agent',
     'kit_get_bitbucket_pr',
+    'kit_create_bitbucket_pr',
+    'kit_find_bitbucket_reviewers',
+    'kit_get_bitbucket_pr_comments',
     'kit_jira_get_ticket',
+    'kit_confluence_get_page',
 ];
 async function buildClientWithTools() {
     const server = new McpServer({ name: 'test-kit', version: '0.0.0' });
@@ -23,7 +27,7 @@ async function buildClientWithTools() {
     return client;
 }
 describe('MCP tool registration smoke tests', () => {
-    test('R1: all 4 non-memory tool names are registered', async () => {
+    test('R1: all 8 non-memory tool names are registered', async () => {
         const client = await buildClientWithTools();
         const { tools } = await client.listTools();
         const names = tools.map((t) => t.name);
@@ -74,4 +78,45 @@ describe('MCP tool registration smoke tests', () => {
         assert.equal(tool.annotations.idempotentHint, true, 'idempotentHint must be true');
         assert.equal(tool.annotations.openWorldHint, true, 'openWorldHint must be true');
     });
+    test('R6: kit_confluence_get_page (readOnlyHint:true) has correct annotations (BC1)', async () => {
+        const client = await buildClientWithTools();
+        const { tools } = await client.listTools();
+        const tool = tools.find((t) => t.name === 'kit_confluence_get_page');
+        assert.ok(tool, 'kit_confluence_get_page must be registered');
+        assert.ok(tool.annotations, 'kit_confluence_get_page must have annotations');
+        assert.equal(tool.annotations.readOnlyHint, true, 'readOnlyHint must be true');
+        assert.equal(tool.annotations.idempotentHint, true, 'idempotentHint must be true');
+        assert.equal(tool.annotations.openWorldHint, true, 'openWorldHint must be true');
+    });
+    test('R7: kit_create_bitbucket_pr (readOnlyHint:false) has correct annotations', async () => {
+        const client = await buildClientWithTools();
+        const { tools } = await client.listTools();
+        const tool = tools.find((t) => t.name === 'kit_create_bitbucket_pr');
+        assert.ok(tool, 'kit_create_bitbucket_pr must be registered');
+        assert.ok(tool.annotations, 'kit_create_bitbucket_pr must have annotations');
+        assert.equal(tool.annotations.readOnlyHint, false, 'readOnlyHint must be false');
+        assert.equal(tool.annotations.destructiveHint, false, 'destructiveHint must be false');
+        assert.equal(tool.annotations.idempotentHint, false, 'idempotentHint must be false');
+        assert.equal(tool.annotations.openWorldHint, true, 'openWorldHint must be true');
+    });
+    test('R8: kit_get_bitbucket_pr_comments (readOnlyHint:true) has correct annotations', async () => {
+        const client = await buildClientWithTools();
+        const { tools } = await client.listTools();
+        const tool = tools.find((t) => t.name === 'kit_get_bitbucket_pr_comments');
+        assert.ok(tool, 'kit_get_bitbucket_pr_comments must be registered');
+        assert.ok(tool.annotations, 'kit_get_bitbucket_pr_comments must have annotations');
+        assert.equal(tool.annotations.readOnlyHint, true, 'readOnlyHint must be true');
+        assert.equal(tool.annotations.idempotentHint, true, 'idempotentHint must be true');
+        assert.equal(tool.annotations.openWorldHint, true, 'openWorldHint must be true');
+    });
+    test('R9: kit_find_bitbucket_reviewers (readOnlyHint:true) has correct annotations', async () => {
+        const client = await buildClientWithTools();
+        const { tools } = await client.listTools();
+        const tool = tools.find((t) => t.name === 'kit_find_bitbucket_reviewers');
+        assert.ok(tool, 'kit_find_bitbucket_reviewers must be registered');
+        assert.ok(tool.annotations, 'kit_find_bitbucket_reviewers must have annotations');
+        assert.equal(tool.annotations.readOnlyHint, true, 'readOnlyHint must be true');
+        assert.equal(tool.annotations.idempotentHint, true, 'idempotentHint must be true');
+        assert.equal(tool.annotations.openWorldHint, true, 'openWorldHint must be true');
+    });
 });