sap-ai-dev-toolkit 0.5.0 → 0.5.6

package/test/live-s4h.test.mjs

@@ -1,202 +0,0 @@
-import test from 'node:test';
-import assert from 'node:assert/strict';
-import { spawn } from 'node:child_process';
-import { once } from 'node:events';
-import { readFile } from 'node:fs/promises';
-import { homedir } from 'node:os';
-import { join } from 'node:path';
-import { fileURLToPath } from 'node:url';
-
-const launcher = fileURLToPath(new URL('../src/launcher.mjs', import.meta.url));
-const LIVE = /^(1|true|yes)$/iu.test(process.env.SAP_AI_DEV_LIVE_S4H || '');
-const DESTINATION = process.env.SAP_AI_DEV_LIVE_DESTINATION || 'S4H';
-const SLUG = process.env.SAP_AI_DEV_LIVE_DESTINATION_SLUG || DESTINATION.toLowerCase().replace(/[^a-z0-9_-]+/gu, '-');
-const REQUEST_TIMEOUT_MS = Number(process.env.SAP_AI_DEV_LIVE_TIMEOUT_MS || 45_000);
-
-async function readConfiguredServer() {
-  if (/^(0|false|no)$/iu.test(process.env.SAP_AI_DEV_LIVE_USE_MCP_CONFIG || '')) return null;
-  const candidates = [
-    process.env.SAP_AI_DEV_MCP_CONFIG,
-    process.env.BAS_VSP_MCP_CONFIG,
-    join(homedir(), '.vscode', 'data', 'User', 'mcp.json'),
-    join(homedir(), '.vscode-server', 'data', 'User', 'mcp.json'),
-    join(homedir(), '.code-server', 'data', 'User', 'mcp.json')
-  ].filter(Boolean);
-  for (const path of candidates) {
-    let parsed;
-    try {
-      parsed = JSON.parse(await readFile(path, 'utf8'));
-    } catch {
-      continue;
-    }
-    const server = parsed?.servers?.[DESTINATION];
-    if (server?.command) return { path, server };
-  }
-  return null;
-}
-
-class StdioMcpClient {
-  constructor(command, args, env) {
-    this.child = spawn(command, args, { env, stdio: ['pipe', 'pipe', 'pipe'] });
-    this.nextId = 1;
-    this.pending = new Map();
-    this.stdout = '';
-    this.stderr = '';
-    this.notifications = [];
-    this.child.stdout.setEncoding('utf8');
-    this.child.stderr.setEncoding('utf8');
-    this.child.stdout.on('data', chunk => this.#onStdout(chunk));
-    this.child.stderr.on('data', chunk => { this.stderr += chunk; });
-    this.child.on('exit', (code, signal) => {
-      const error = new Error(`MCP process exited (${code ?? signal})\n${this.stderr.slice(-4000)}`);
-      for (const pending of this.pending.values()) pending.reject(error);
-      this.pending.clear();
-    });
-  }
-
-  #onStdout(chunk) {
-    this.stdout += chunk;
-    let newline;
-    while ((newline = this.stdout.indexOf('\n')) >= 0) {
-      const line = this.stdout.slice(0, newline).trim();
-      this.stdout = this.stdout.slice(newline + 1);
-      if (!line) continue;
-      let message;
-      try {
-        message = JSON.parse(line);
-      } catch (error) {
-        for (const pending of this.pending.values()) pending.reject(new Error(`Invalid JSON-RPC line: ${line}`));
-        this.pending.clear();
-        return;
-      }
-      if (message.id === undefined || message.id === null) {
-        this.notifications.push(message);
-        continue;
-      }
-      const pending = this.pending.get(message.id);
-      if (!pending) continue;
-      this.pending.delete(message.id);
-      clearTimeout(pending.timeout);
-      if (message.error) pending.reject(Object.assign(new Error(message.error.message), { response: message }));
-      else pending.resolve(message.result);
-    }
-  }
-
-  request(method, params) {
-    const id = this.nextId++;
-    const payload = { jsonrpc: '2.0', id, method, ...(params === undefined ? {} : { params }) };
-    return new Promise((resolve, reject) => {
-      const timeout = setTimeout(() => {
-        this.pending.delete(id);
-        reject(new Error(`Timed out waiting for ${method}\n${this.stderr.slice(-4000)}`));
-      }, REQUEST_TIMEOUT_MS);
-      this.pending.set(id, { resolve, reject, timeout });
-      this.child.stdin.write(`${JSON.stringify(payload)}\n`);
-    });
-  }
-
-  notify(method, params) {
-    this.child.stdin.write(`${JSON.stringify({ jsonrpc: '2.0', method, ...(params === undefined ? {} : { params }) })}\n`);
-  }
-
-  async close() {
-    this.child.stdin.end();
-    this.child.kill('SIGTERM');
-    await Promise.race([
-      once(this.child, 'exit').catch(() => {}),
-      new Promise(resolve => setTimeout(resolve, 1500))
-    ]);
-    if (!this.child.killed) this.child.kill('SIGKILL');
-  }
-}
-
-function textPayload(result) {
-  return (result?.content || []).filter(item => item.type === 'text').map(item => item.text).join('\n');
-}
-
-async function withLiveClient(t) {
-  const configured = await readConfiguredServer();
-  const server = configured?.server;
-  const command = process.env.SAP_AI_DEV_LIVE_COMMAND || server?.command || process.execPath;
-  const args = process.env.SAP_AI_DEV_LIVE_COMMAND
-    ? (process.env.SAP_AI_DEV_LIVE_ARGS ? JSON.parse(process.env.SAP_AI_DEV_LIVE_ARGS) : [])
-    : (server?.command ? (server.args || []) : [launcher]);
-  const client = new StdioMcpClient(command, args, {
-    ...process.env,
-    ...(server?.env || {}),
-    SAP_AI_DEV_TOOLKIT_DESTINATION: server?.env?.SAP_AI_DEV_TOOLKIT_DESTINATION || DESTINATION,
-    SAP_AI_DEV_TOOLKIT_MODE: process.env.SAP_AI_DEV_TOOLKIT_MODE || server?.env?.SAP_AI_DEV_TOOLKIT_MODE || 'expert',
-    SAP_ALLOW_TRANSPORTABLE_EDITS: process.env.SAP_ALLOW_TRANSPORTABLE_EDITS || server?.env?.SAP_ALLOW_TRANSPORTABLE_EDITS || 'true'
-  });
-  t.after(() => client.close());
-  const initialized = await client.request('initialize', { protocolVersion: '2024-11-05', capabilities: {}, clientInfo: { name: 'sap-ai-dev-live-test', version: '1.0.0' } });
-  client.notify('notifications/initialized');
-  assert.equal(initialized.serverInfo.name, DESTINATION);
-  return client;
-}
-
-test('live S4H MCP exposes a curated ABAP lifecycle surface under 60 tools', { skip: LIVE ? false : 'set SAP_AI_DEV_LIVE_S4H=1 to run against a live BAS/S4H destination' }, async t => {
-  const client = await withLiveClient(t);
-  const listed = await client.request('tools/list', {});
-  const tools = listed.tools || [];
-  const names = new Set(tools.map(tool => tool.name));
-  const prefix = `${SLUG}__`;
-
-  assert.ok(tools.length > 25, `expected a useful ABAP lifecycle surface, got ${tools.length}`);
-  assert.ok(tools.length < 60, `curated destination tool count must stay below 60, got ${tools.length}`);
-  assert.ok(tools.every(tool => tool.name.startsWith(prefix)), 'every tool is destination-prefixed for the selected S4H server');
-
-  for (const required of [
-    'LintABAP', 'GetSystemInfo', 'GetConnectionInfo', 'GetFeatures',
-    'SearchObject', 'GrepObjects', 'GrepPackages', 'GetSource', 'WriteSource', 'EditSource', 'CompareSource',
-    'SyntaxCheck', 'PrettyPrint', 'Activate', 'RunUnitTests', 'RunATCCheck', 'GetInactiveObjects',
-    'GetClass', 'GetProgram', 'GetPackage', 'GetTable', 'GetTableContents', 'RunQuery',
-    'GetTransport', 'GetTransportInfo', 'ListTransports', 'CreateTransport',
-    'GetApplicationLog', 'PrepareABAPChangeSet', 'ApplyABAPChangeSet', 'CheckTransportReadiness', 'PlanABAPCloudMigration'
-  ]) {
-    assert.ok(names.has(`${prefix}${required}`), `expected curated lifecycle tool ${required}`);
-  }
-
-  for (const hidden of [
-    'SAP', 'ReleaseTransport', 'DeleteTransport', 'DeleteObject', 'ExecuteABAP', 'CallRFC',
-    'ListSQLTraces', 'AnalyzeABAPCode', 'CreateObject', 'CreateClassWithTests', 'WriteClass', 'UpdateSource'
-  ]) {
-    assert.equal(names.has(`${prefix}${hidden}`), false, `broad/destructive tool ${hidden} must remain hidden`);
-  }
-});
-
-test('live S4H MCP handles safe local and read-only SAP calls', { skip: LIVE ? false : 'set SAP_AI_DEV_LIVE_S4H=1 to run against a live BAS/S4H destination' }, async t => {
-  const client = await withLiveClient(t);
-  const prefix = `${SLUG}__`;
-  await client.request('tools/list', {});
-
-  const lint = await client.request('tools/call', {
-    name: `${prefix}LintABAP`,
-    arguments: { files: [{ filename: 'zlive_probe.prog.abap', source: "REPORT zlive_probe.\nWRITE / 'ok'.\n" }] }
-  });
-  const lintJson = JSON.parse(textPayload(lint));
-  assert.equal(lintJson.filesChecked, 1);
-
-  for (const [tool, args] of [
-    ['GetSystemInfo', {}],
-    ['GetConnectionInfo', {}],
-    ['GetFeatures', {}],
-    ['GetInstalledComponents', {}]
-  ]) {
-    const result = await client.request('tools/call', { name: `${prefix}${tool}`, arguments: args });
-    assert.notEqual(result.isError, true, `${tool} returned an MCP tool error: ${textPayload(result)}`);
-  }
-
-  const query = await client.request('tools/call', {
-    name: `${prefix}RunQuery`,
-    arguments: { sql_query: 'SELECT * FROM T000', max_rows: 1 }
-  });
-  assert.notEqual(query.isError, true, `RunQuery returned an MCP tool error: ${textPayload(query)}`);
-  assert.match(textPayload(query), /T000|MANDT|client|rows|\[/iu);
-
-  const appLog = await client.request('tools/call', {
-    name: `${prefix}GetApplicationLog`,
-    arguments: { max_results: 1 }
-  });
-  assert.notEqual(appLog.isError, true, `GetApplicationLog returned an MCP tool error: ${textPayload(appLog)}`);
-});

package/test/mcp-config-cf.test.mjs

@@ -1,407 +0,0 @@
-import test from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtemp, readFile, rm, writeFile } from 'node:fs/promises';
-import { join } from 'node:path';
-import { tmpdir } from 'node:os';
-import { buildMcpEntries, buildSapDevelopmentMcpEntries, collectCloudFoundryKeyReferencesFromAllEntries, collectManagedCloudFoundryKeyReferences, installMcpConfig, readMcpConfig, repairManagedMcpConfig, resolveMcpConfigPath } from '../src/mcp-config.mjs';
-
-const bas = {
-  source: 'bas', name: 'shared', serverName: 'shared', url: 'http://shared.dest',
-  client: '001', authentication: 'NoAuthentication', probe: { status: 'available', available: true }
-};
-
-function cfDestination(spaceGuid, destinationInstanceGuid, keyName, proxyType = 'Internet') {
-  return {
-    source: 'cloud-foundry',
-    name: 'shared',
-    serverName: `cf:${spaceGuid}:${destinationInstanceGuid}:shared`,
-    client: '100',
-    authentication: 'BasicAuthentication',
-    proxyType,
-    probe: { status: 'auth-required', available: true },
-    cf: {
-      spaceGuid,
-      destinationInstanceGuid,
-      destinationInstanceName: `destination-${destinationInstanceGuid}`,
-      destinationKeyName: keyName,
-      ...(proxyType === 'OnPremise' ? {
-        connectivityInstanceGuid: 'connectivity-guid',
-        connectivityInstanceName: 'connectivity-service',
-        connectivityKeyName: 'connectivity-key'
-      } : {})
-    }
-  };
-}
-
-test('writes source-qualified CF entries beside same-named BAS destinations', () => {
-  const first = cfDestination('space-one', 'instance-one', 'key-one');
-  const second = cfDestination('space-one', 'instance-two', 'key-two');
-  const entries = buildMcpEntries([bas, first, second], { H2O_URL: 'http://h2o.example' });
-  assert.deepEqual(Object.keys(entries).sort(), [
-    'cf:space-one:instance-one:shared',
-    'cf:space-one:instance-two:shared',
-    'shared'
-  ]);
-  assert.deepEqual(entries['cf:space-one:instance-one:shared'].env, {
-    H2O_URL: 'http://h2o.example',
-    SAP_ALLOW_TRANSPORTABLE_EDITS: 'true',
-    SAP_AI_DEV_TOOLKIT_DESTINATION_SOURCE: 'cloud-foundry',
-    SAP_AI_DEV_TOOLKIT_DESTINATION: 'shared',
-    BAS_CF_SPACE_GUID: 'space-one',
-    BAS_CF_DESTINATION_INSTANCE_GUID: 'instance-one',
-    BAS_CF_DESTINATION_INSTANCE: 'destination-instance-one',
-    BAS_CF_DESTINATION_KEY: 'key-one',
-    BAS_CF_DESTINATION_NAME: 'shared'
-  });
-  assert.equal(entries.shared.env.SAP_AI_DEV_TOOLKIT_DESTINATION, 'shared');
-  assert.equal(JSON.stringify(entries).includes('shared.dest'), false);
-  assert.equal(JSON.stringify(entries).includes('Password'), false);
-
-  const onPremise = buildMcpEntries([cfDestination('space-one', 'onprem-instance', 'destination-key', 'OnPremise')], { H2O_URL: 'http://h2o.example' });
-  assert.equal(onPremise['cf:space-one:onprem-instance:shared'].env.BAS_CF_CONNECTIVITY_KEY, 'connectivity-key');
-});
-
-test('stores and reconciles the MCP launcher executable location', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'bas-mcp-launcher-location-'));
-  const path = join(directory, 'mcp.json');
-  const command = join(directory, 'sap-ai-dev');
-  await writeFile(command, '#!/bin/sh\nexit 0\n', { mode: 0o755 });
-  await writeFile(path, JSON.stringify({ servers: { userServer: { command: 'custom-server' } } }));
-  t.after(() => rm(directory, { recursive: true, force: true }));
-
-  const env = { H2O_URL: 'http://h2o.example', PATH: directory };
-  const installed = await installMcpConfig([bas], { env, path });
-  assert.equal(installed.servers.shared.command, command);
-  assert.equal((await readMcpConfig(path)).servers.shared.command, command);
-
-  await installMcpConfig([], { env, path });
-  assert.deepEqual(Object.keys((await readMcpConfig(path)).servers), ['userServer']);
-});
-
-test('resolves documented, toolkit, and legacy MCP config paths in order', async () => {
-  assert.equal(await resolveMcpConfigPath({
-    SAP_AI_DEV_MCP_CONFIG: '/tmp/documented-mcp.json',
-    SAP_AI_DEV_TOOLKIT_MCP_CONFIG: '/tmp/toolkit-mcp.json',
-    BAS_VSP_MCP_CONFIG: '/tmp/legacy-mcp.json'
-  }), '/tmp/documented-mcp.json');
-  assert.equal(await resolveMcpConfigPath({
-    SAP_AI_DEV_TOOLKIT_MCP_CONFIG: '/tmp/toolkit-mcp.json',
-    BAS_VSP_MCP_CONFIG: '/tmp/legacy-mcp.json'
-  }), '/tmp/toolkit-mcp.json');
-  assert.equal(await resolveMcpConfigPath({ BAS_VSP_MCP_CONFIG: '/tmp/legacy-mcp.json' }), '/tmp/legacy-mcp.json');
-});
-
-test('doctor repair updates only discovered toolkit BAS entries and is idempotent', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'sap-ai-toolkit-doctor-repair-'));
-  const path = join(directory, 'mcp.json');
-  t.after(() => rm(directory, { recursive: true, force: true }));
-  const cf = cfDestination('space-one', 'instance-one', 'cf-key');
-  const strictDestination = { ...bas, name: 'strict', serverName: 'strict', url: 'http://strict.dest' };
-  const managed = buildMcpEntries([bas], { H2O_URL: 'http://old-h2o.example' }).shared;
-  const strictManaged = buildMcpEntries([strictDestination], { H2O_URL: 'http://old-h2o.example' }).strict;
-  const cfEntry = buildMcpEntries([cf], { H2O_URL: 'http://cf-h2o.example' })[cf.serverName];
-  const companion = buildSapDevelopmentMcpEntries(['sap-fiori-tools'])['sap-fiori-tools'];
-  const unrelated = { type: 'stdio', command: 'external-server' };
-  const unverifiedManaged = {
-    ...managed,
-    env: { ...managed.env, SAP_AI_DEV_TOOLKIT_DESTINATION: 'temporarily-unavailable' }
-  };
-  const config = {
-    userFlag: true,
-    servers: {
-      shared: { ...managed, env: { ...managed.env, H2O_URL: 'http://old-h2o.example' } },
-      strict: { ...strictManaged, env: { ...strictManaged.env, SAP_ALLOW_TRANSPORTABLE_EDITS: 'false' } },
-      'temporarily-unavailable': unverifiedManaged,
-      [cf.serverName]: cfEntry,
-      'sap-fiori-tools': companion,
-      ActionS4D_100: unrelated
-    }
-  };
-  await writeFile(path, JSON.stringify(config, null, 2));
-
-  const first = await repairManagedMcpConfig([bas, strictDestination], { env: { H2O_URL: 'http://current-h2o.example' }, path, discoveryComplete: true });
-  assert.equal(first.changed, true);
-  assert.equal(first.repaired, 2);
-  const repaired = await readMcpConfig(path);
-  assert.equal(repaired.userFlag, true);
-  assert.equal(repaired.servers.shared.env.H2O_URL, 'http://current-h2o.example');
-  assert.equal(repaired.servers.shared.env.SAP_ALLOW_TRANSPORTABLE_EDITS, 'true');
-  assert.equal(repaired.servers.strict.env.SAP_ALLOW_TRANSPORTABLE_EDITS, 'false', "doctor must preserve a user's restricted write setting");
-  assert.deepEqual(repaired.servers['temporarily-unavailable'], unverifiedManaged);
-  assert.deepEqual(repaired.servers[cf.serverName], cfEntry);
-  assert.deepEqual(repaired.servers['sap-fiori-tools'], companion);
-  assert.deepEqual(repaired.servers.ActionS4D_100, unrelated);
-  assert.equal(repaired.servers.newDestination, undefined, 'repair must not add unselected destinations');
-
-  const beforeSecondRepair = await readFile(path, 'utf8');
-  const second = await repairManagedMcpConfig([bas, strictDestination], { env: { H2O_URL: 'http://current-h2o.example' }, path, discoveryComplete: true });
-  assert.equal(second.changed, false);
-  assert.equal(second.repaired, 0);
-  assert.equal(await readFile(path, 'utf8'), beforeSecondRepair, 'an already-healed config must not be rewritten');
-});
-
-test('doctor repair leaves malformed config untouched', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'sap-ai-toolkit-doctor-repair-invalid-'));
-  const path = join(directory, 'mcp.json');
-  const malformed = '{ "servers": ';
-  await writeFile(path, malformed);
-  t.after(() => rm(directory, { recursive: true, force: true }));
-  await assert.rejects(
-    () => repairManagedMcpConfig([bas], { env: { H2O_URL: 'http://h2o.example' }, path, discoveryComplete: true }),
-    /invalid JSON/
-  );
-  assert.equal(await readFile(path, 'utf8'), malformed);
-});
-
-test('doctor repair skips config changes if destination discovery is incomplete', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'sap-ai-toolkit-doctor-repair-incomplete-'));
-  const path = join(directory, 'mcp.json');
-  const entry = buildMcpEntries([bas], { H2O_URL: 'http://old-h2o.example' }).shared;
-  const initial = JSON.stringify({ servers: { shared: entry } });
-  await writeFile(path, initial);
-  t.after(() => rm(directory, { recursive: true, force: true }));
-  const result = await repairManagedMcpConfig([bas], { env: { H2O_URL: 'http://new-h2o.example' }, path });
-  assert.equal(result.changed, false);
-  assert.match(result.skipped, /incomplete/);
-  assert.equal(await readFile(path, 'utf8'), initial);
-});
-
-test('doctor repair refreshes stale toolkit launchers without changing npx behavior', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'sap-ai-toolkit-doctor-launcher-repair-'));
-  const path = join(directory, 'mcp.json');
-  const bin = join(directory, 'bin');
-  await import('node:fs/promises').then(({ mkdir }) => mkdir(bin));
-  const command = join(bin, 'sap-ai-dev');
-  await writeFile(command, '#!/bin/sh\nexit 0\n', { mode: 0o755 });
-  t.after(() => rm(directory, { recursive: true, force: true }));
-  const npx = {
-    ...buildMcpEntries([bas], { H2O_URL: 'http://old-h2o.example' }).shared,
-    command: 'npx',
-    args: ['--yes', '--package=sap-ai-dev-toolkit@0.4.2', 'sap-ai-dev'],
-    env: {
-      ...buildMcpEntries([bas], { H2O_URL: 'http://old-h2o.example' }).shared.env,
-      SAP_AI_DEV_TOOLKIT_DESTINATION: 'shared',
-      OPERATOR_SETTING: 'preserve'
-    }
-  };
-  const direct = {
-    ...buildMcpEntries([bas], { H2O_URL: 'http://old-h2o.example' }).shared,
-    command: 'stale-sap-ai-dev'
-  };
-  await writeFile(path, JSON.stringify({ servers: { shared: npx, other: { ...direct, env: { ...direct.env, SAP_AI_DEV_TOOLKIT_DESTINATION: 'other' } }, userServer: { command: 'other' } } }));
-
-  const result = await repairManagedMcpConfig([bas], {
-    env: { H2O_URL: 'http://current-h2o.example', PATH: bin },
-    path,
-    discoveryComplete: true,
-    packageVersion: '0.4.5'
-  });
-  assert.equal(result.repaired, 1, 'only the entry that maps to the discovered destination is repaired');
-  const repaired = await readMcpConfig(path);
-  assert.equal(repaired.servers.shared.command, 'npx');
-  assert.deepEqual(repaired.servers.shared.args, ['--yes', '--package=sap-ai-dev-toolkit@0.4.5', 'sap-ai-dev']);
-  assert.equal(repaired.servers.shared.env.OPERATOR_SETTING, 'preserve');
-  assert.equal(repaired.servers.shared.env.H2O_URL, 'http://current-h2o.example');
-  assert.equal(repaired.servers.other.command, 'stale-sap-ai-dev', 'unmatched toolkit entry must remain untouched');
-  assert.deepEqual(repaired.servers.userServer, { command: 'other' });
-});
-
-test('extracts key references only from package-managed Cloud Foundry entries', () => {
-  const cloudFoundry = buildMcpEntries([
-    cfDestination('space-one', 'instance-one', 'destination-key', 'OnPremise')
-  ], { H2O_URL: 'http://h2o.example' })['cf:space-one:instance-one:shared'];
-  const managedNpx = {
-    ...cloudFoundry,
-    command: 'npx',
-    args: ['--yes', '--package=sap-ai-dev-toolkit@0.1.0', 'sap-ai-dev']
-  };
-  const config = { servers: {
-    first: cloudFoundry,
-    duplicate: managedNpx,
-    unmanaged: { ...cloudFoundry, BAS_EXT: undefined },
-    otherPackage: { ...cloudFoundry, command: 'other' },
-    otherSource: { ...cloudFoundry, env: { ...cloudFoundry.env, SAP_AI_DEV_TOOLKIT_DESTINATION_SOURCE: 'bas' } }
-  } };
-  assert.deepEqual(collectManagedCloudFoundryKeyReferences(config), [
-    { kind: 'destination', spaceGuid: 'space-one', instanceGuid: 'instance-one', instanceName: 'destination-instance-one', keyName: 'destination-key' },
-    { kind: 'connectivity', spaceGuid: 'space-one', instanceGuid: 'connectivity-guid', instanceName: 'connectivity-service', keyName: 'connectivity-key' }
-  ]);
-  const userConfig = { servers: {
-    customServer: { command: 'custom-launcher', env: { ...cloudFoundry.env, SAP_AI_DEV_TOOLKIT_DESTINATION_SOURCE: undefined } },
-    noEnvironment: { command: 'custom-launcher' }
-  } };
-  assert.deepEqual(collectCloudFoundryKeyReferencesFromAllEntries(userConfig), [
-    { kind: 'destination', spaceGuid: 'space-one', instanceGuid: 'instance-one', instanceName: 'destination-instance-one', keyName: 'destination-key' },
-    { kind: 'connectivity', spaceGuid: 'space-one', instanceGuid: 'connectivity-guid', instanceName: 'connectivity-service', keyName: 'connectivity-key' }
-  ]);
-});
-
-test('builds SAP development companion MCP entries', () => {
-  const entries = buildSapDevelopmentMcpEntries(['sap-fiori-tools', 'ui5-tools']);
-  assert.deepEqual(Object.keys(entries), ['sap-fiori-tools', 'ui5-tools']);
-  assert.deepEqual(entries['sap-fiori-tools'].args, ['--yes', '--package=@sap-ux/fiori-mcp-server', 'fiori-mcp']);
-  assert.equal(entries['ui5-tools'].command, 'npx');
-  assert.equal(entries['ui5-tools'].BAS_EXT_KIND, 'sap-development-companion');
-  assert.throws(() => buildSapDevelopmentMcpEntries(['missing-tool']), /Unknown SAP development MCP server id/);
-});
-
-test('configures the HANA inspector without persisting credentials', () => {
-  const entry = buildSapDevelopmentMcpEntries(['hana-cloud-inspector'])['hana-cloud-inspector'];
-  assert.equal(entry.command, 'npx');
-  assert.deepEqual(entry.args, ['--yes', '--ignore-scripts', '--package=sap-ai-dev-toolkit', 'sap-ai-hana']);
-  assert.equal(entry.BAS_EXT_KIND, 'sap-development-companion');
-  assert.equal(entry.displayName, 'HANA Cloud inspector');
-  assert.equal(entry.env, undefined);
-  assert.doesNotMatch(JSON.stringify(entry), /"(?:HANA_RO_PASSWORD|VCAP_SERVICES)"\s*:/);
-  assert.doesNotMatch(JSON.stringify(entry), /"(?:password|pwd)"\s*:/i);
-});
-
-test('installs and removes SAP development companion entries', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'bas-mcp-tools-config-'));
-  const path = join(directory, 'mcp.json');
-  await writeFile(path, JSON.stringify({ servers: { userServer: { type: 'stdio', command: 'custom-server' } } }));
-  t.after(() => rm(directory, { recursive: true, force: true }));
-
-  const result = await installMcpConfig([], {
-    env: { H2O_URL: 'http://h2o.example' },
-    path,
-    sapDevelopmentServers: ['sap-fiori-tools', 'cap-tools']
-  });
-  assert.deepEqual(Object.keys(result.servers).sort(), ['cap-tools', 'sap-fiori-tools']);
-  const written = await readMcpConfig(path);
-  assert.deepEqual(Object.keys(written.servers).sort(), ['cap-tools', 'sap-fiori-tools', 'userServer']);
-
-  await installMcpConfig([], { env: { H2O_URL: 'http://h2o.example' }, path });
-  const cleared = await readMcpConfig(path);
-  assert.deepEqual(Object.keys(cleared.servers), ['userServer']);
-});
-
-test('installs and removes the HANA inspector without changing unrelated MCP servers', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'bas-mcp-hana-config-'));
-  const path = join(directory, 'mcp.json');
-  const initial = {
-    custom: true,
-    servers: { userServer: { type: 'stdio', command: 'custom-server' } }
-  };
-  await writeFile(path, JSON.stringify(initial));
-  t.after(() => rm(directory, { recursive: true, force: true }));
-
-  const installed = await installMcpConfig([], {
-    env: { H2O_URL: 'http://h2o.example' },
-    path,
-    sapDevelopmentServers: ['hana-cloud-inspector']
-  });
-  assert.deepEqual(Object.keys(installed.servers), ['hana-cloud-inspector']);
-  const configured = await readMcpConfig(path);
-  assert.deepEqual(Object.keys(configured.servers).sort(), ['hana-cloud-inspector', 'userServer']);
-  assert.equal(configured.custom, true);
-  assert.equal(configured.servers['hana-cloud-inspector'].env, undefined);
-  assert.doesNotMatch(JSON.stringify(configured), /"(?:HANA_RO_PASSWORD|VCAP_SERVICES)"\s*:/);
-  assert.doesNotMatch(JSON.stringify(configured), /"(?:password|pwd)"\s*:/i);
-
-  await installMcpConfig([], { env: { H2O_URL: 'http://h2o.example' }, path });
-  const removed = await readMcpConfig(path);
-  assert.deepEqual(removed.servers, initial.servers);
-  assert.equal(removed.custom, true);
-});
-
-test('installs and removes CF entries without deleting unrelated MCP servers', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'bas-mcp-cf-config-'));
-  const path = join(directory, 'mcp.json');
-  await writeFile(path, JSON.stringify({
-    unrelated: true,
-    servers: { userServer: { type: 'stdio', command: 'custom-server' } }
-  }));
-  t.after(() => rm(directory, { recursive: true, force: true }));
-
-  const result = await installMcpConfig([bas, cfDestination('space-one', 'instance-one', 'managed-key')], {
-    env: { H2O_URL: 'http://h2o.example' },
-    path,
-    command: 'npx',
-    args: ['--yes', '--package=sap-ai-dev-toolkit@0.1.0', 'sap-ai-dev']
-  });
-  assert.deepEqual(Object.keys(result.servers).sort(), ['cf:space-one:instance-one:shared', 'shared']);
-  const written = await readMcpConfig(path);
-  assert.equal(written.unrelated, true);
-  assert.deepEqual(Object.keys(written.servers).sort(), ['cf:space-one:instance-one:shared', 'shared', 'userServer']);
-  assert.deepEqual(collectManagedCloudFoundryKeyReferences(written), [
-    { kind: 'destination', spaceGuid: 'space-one', instanceGuid: 'instance-one', instanceName: 'destination-instance-one', keyName: 'managed-key' }
-  ]);
-
-  await installMcpConfig([], { env: { H2O_URL: 'http://h2o.example' }, path });
-  const cleared = JSON.parse(await readFile(path, 'utf8'));
-  assert.deepEqual(Object.keys(cleared.servers), ['userServer']);
-});
-
-test('wizard replaces legacy launcher entries with the branded command and environment', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'sap-ai-toolkit-migration-'));
-  const path = join(directory, 'mcp.json');
-  t.after(() => rm(directory, { recursive: true, force: true }));
-  await writeFile(path, JSON.stringify({
-    servers: {
-      shared: {
-        type: 'stdio',
-        command: 'bas-vsp-mcp',
-        env: { H2O_URL: 'http://h2o.example', BAS_VSP_DESTINATION: 'shared' },
-        BAS_EXT: 'true'
-      },
-      userServer: { type: 'stdio', command: 'custom-server' }
-    }
-  }));
-
-  await installMcpConfig([bas], { env: { H2O_URL: 'http://h2o.example', BAS_VSP_MCP_CONFIG: path } });
-  const migrated = await readMcpConfig(path);
-  assert.deepEqual(Object.keys(migrated.servers).sort(), ['shared', 'userServer']);
-  assert.equal(migrated.servers.shared.command, 'sap-ai-dev');
-  assert.equal(migrated.servers.shared.env.SAP_AI_DEV_TOOLKIT_DESTINATION, 'shared');
-  assert.equal(migrated.servers.shared.env.BAS_VSP_DESTINATION, undefined);
-  assert.equal(migrated.servers.userServer.command, 'custom-server');
-});
-
-test('recognizes old npx Cloud Foundry entries so setup can reuse their service keys', () => {
-  const current = buildMcpEntries([cfDestination('space-one', 'instance-one', 'old-key')], { H2O_URL: 'http://h2o.example' })['cf:space-one:instance-one:shared'];
-  const legacy = {
-    ...current,
-    command: 'npx',
-    args: ['--yes', '--package=bas-mcp-addon@0.1.0', 'bas-vsp-mcp'],
-    env: {
-      ...current.env,
-      BAS_VSP_DESTINATION_SOURCE: current.env.SAP_AI_DEV_TOOLKIT_DESTINATION_SOURCE,
-      BAS_VSP_DESTINATION: current.env.SAP_AI_DEV_TOOLKIT_DESTINATION
-    }
-  };
-  delete legacy.BAS_EXT;
-  delete legacy.env.SAP_AI_DEV_TOOLKIT_DESTINATION_SOURCE;
-  delete legacy.env.SAP_AI_DEV_TOOLKIT_DESTINATION;
-  assert.deepEqual(collectManagedCloudFoundryKeyReferences({ servers: { legacy } }), [
-    { kind: 'destination', spaceGuid: 'space-one', instanceGuid: 'instance-one', instanceName: 'destination-instance-one', keyName: 'old-key' }
-  ]);
-});
-
-test('setup removes untagged legacy bas-mcp-addon launchers that expose the full VSP surface', async t => {
-  const directory = await mkdtemp(join(tmpdir(), 'sap-ai-toolkit-untagged-legacy-'));
-  const path = join(directory, 'mcp.json');
-  t.after(() => rm(directory, { recursive: true, force: true }));
-  await writeFile(path, JSON.stringify({
-    servers: {
-      shared: {
-        type: 'stdio',
-        command: 'npx',
-        args: ['--yes', '--package=bas-mcp-addon@0.2.0', 'bas-vsp-mcp'],
-        env: { H2O_URL: 'http://old-h2o.example', BAS_VSP_DESTINATION: 'shared' }
-      },
-      legacyCommand: {
-        type: 'stdio',
-        command: 'bas-vsp-mcp',
-        env: { H2O_URL: 'http://old-h2o.example', BAS_VSP_DESTINATION: 'other' }
-      },
-      userServer: { type: 'stdio', command: 'custom-server' }
-    }
-  }));
-
-  await installMcpConfig([bas], { env: { H2O_URL: 'http://h2o.example' }, path });
-  const migrated = await readMcpConfig(path);
-  assert.deepEqual(Object.keys(migrated.servers).sort(), ['shared', 'userServer']);
-  assert.equal(migrated.servers.shared.command, 'sap-ai-dev');
-  assert.equal(migrated.servers.shared.env.SAP_AI_DEV_TOOLKIT_DESTINATION, 'shared');
-});