sap-ai-dev-toolkit 0.2.7 → 0.3.1

package/test/launcher.test.mjs

@@ -3,9 +3,10 @@ import assert from 'node:assert/strict';
 import { createServer } from 'node:http';
 import { spawn } from 'node:child_process';
 import { fileURLToPath } from 'node:url';
-import { mkdtemp, readFile, rm, writeFile } from 'node:fs/promises';
+import { chmod, mkdir, mkdtemp, readFile, rm, writeFile } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
+import { spawnWithPty } from './pty.mjs';
 
 const launcher = fileURLToPath(new URL('../src/launcher.mjs', import.meta.url));
 const fakeVsp = fileURLToPath(new URL('./fixtures/fake-vsp.mjs', import.meta.url));
@@ -24,10 +25,11 @@ function runLauncher(args, env) {
 function runLauncherTty(env, input, args = ['--setup']) {
   return new Promise((resolve, reject) => {
     const command = `${process.execPath} ${launcher} ${args.join(' ')}`;
-    const child = spawn('script', ['-qec', command, '/dev/null'], { env, stdio: ['pipe', 'pipe', 'pipe'] });
+    const child = spawnWithPty(command, { env, stdio: ['pipe', 'pipe', 'pipe'] });
     let stdout = '';
     let stderr = '';
     let inputSent = false;
+    const timeout = setTimeout(() => child.kill('SIGKILL'), 15000);
     child.stdout.on('data', chunk => {
       stdout += chunk.toString();
       if (!inputSent && stdout.includes('Select destinations')) {
@@ -36,8 +38,8 @@ function runLauncherTty(env, input, args = ['--setup']) {
       }
     });
     child.stderr.on('data', chunk => { stderr += chunk.toString(); });
-    child.on('error', reject);
-    child.on('exit', (code, signal) => resolve({ code, signal, stdout, stderr }));
+    child.on('error', error => { clearTimeout(timeout); reject(error); });
+    child.on('exit', (code, signal) => { clearTimeout(timeout); resolve({ code, signal, stdout, stderr, inputSent }); });
   });
 }
 
@@ -59,6 +61,27 @@ test('help exits before BAS destination discovery', async t => {
   assert.equal(requests, 0);
 });
 
+test('doctor reports missing BAS configuration as redacted JSON', async () => {
+  const result = await runLauncher(['--doctor', '--json'], {
+    ...process.env,
+    H2O_URL: '',
+    SAP_AI_DEV_TOOLKIT_DESTINATION_SOURCE: '',
+    BAS_VSP_DESTINATION_SOURCE: ''
+  });
+  assert.equal(result.code, 1);
+  assert.equal(result.stderr, '');
+  assert.deepEqual(JSON.parse(result.stdout), {
+    ok: false,
+    destinations: 0,
+    checks: [{
+      name: '-',
+      stage: 'destination discovery',
+      status: 'failed',
+      detail: 'H2O_URL is required for BAS destination discovery'
+    }]
+  });
+});
+
 test('list-destinations JSON is redacted and probes through the BAS proxy', async () => {
   const requests = [];
   const server = createServer((request, response) => {
@@ -98,9 +121,9 @@ test('runtime starts destinations even when their ADT probes fail', async () =>
   try {
     const result = await runLauncher([], {
       ...process.env,
-      BAS_VSP_BINARY: fakeVsp,
+      SAP_AI_DEV_TOOLKIT_BINARY: fakeVsp,
       H2O_URL: 'http://bas.example',
-      BAS_VSP_DESTINATION: 'launch-system',
+      SAP_AI_DEV_TOOLKIT_DESTINATION: 'launch-system',
       HTTP_PROXY: proxy,
       http_proxy: proxy,
       NO_PROXY: '',
@@ -117,8 +140,21 @@ test('runtime starts destinations even when their ADT probes fail', async () =>
 });
 test('setup subprocess writes one isolated MCP entry per selected destination', async () => {
   const directory = await mkdtemp(join(tmpdir(), 'bas-launcher-setup-'));
+  const bin = join(directory, 'bin');
+  await mkdir(bin);
+  const cfCli = join(bin, 'cf');
+  await writeFile(cfCli, '#!/bin/sh\nexit 1\n');
+  await chmod(cfCli, 0o755);
   const config = join(directory, 'mcp.json');
-  await writeFile(config, JSON.stringify({ inputs: [], servers: { unrelated: { type: 'stdio', command: 'other' } } }));
+  await writeFile(config, JSON.stringify({ inputs: [], servers: {
+    unrelated: { type: 'stdio', command: 'other' },
+    'alpha-system': {
+      type: 'stdio',
+      command: 'bas-vsp-mcp',
+      env: { H2O_URL: 'http://old-h2o.example', BAS_VSP_DESTINATION: 'alpha-system' },
+      BAS_EXT: 'true'
+    }
+  } }));
   const server = createServer((request, response) => {
     if (request.url === '/api/listDestinations') {
       response.setHeader('content-type', 'application/json');
@@ -132,13 +168,14 @@ test('setup subprocess writes one isolated MCP entry per selected destination',
     response.end();
   });
   await new Promise(resolve => server.listen(0, '127.0.0.1', resolve));
-  const env = { ...process.env, H2O_URL: `http://127.0.0.1:${server.address().port}`, BAS_VSP_SKIP_PROBE: 'true', BAS_VSP_MCP_CONFIG: config };
+  const env = { ...process.env, PATH: `${bin}:${process.env.PATH || ''}`, H2O_URL: `http://127.0.0.1:${server.address().port}`, SAP_AI_DEV_TOOLKIT_SKIP_PROBE: 'true', SAP_AI_DEV_TOOLKIT_MCP_CONFIG: config };
   try {
     const first = await runLauncherTty(env, 'a\r', ['--setup', '--npx']);
     assert.equal(first.code, 0, `${first.stdout}\n${first.stderr}`);
     let current = JSON.parse(await readFile(config, 'utf8'));
     let generated = Object.entries(current.servers).filter(([, entry]) => entry.BAS_EXT === 'true');
-    assert.deepEqual(generated.map(([, entry]) => entry.env.BAS_VSP_DESTINATION), ['alpha-system', 'beta-system'], `${first.stdout}\n${first.stderr}`);
+    assert.deepEqual(generated.map(([, entry]) => entry.env.SAP_AI_DEV_TOOLKIT_DESTINATION), ['alpha-system', 'beta-system'], `${first.stdout}\n${first.stderr}`);
+    assert.equal(current.servers['alpha-system'].env.BAS_VSP_DESTINATION, undefined);
     assert.equal(generated.every(([, entry]) => entry.env.H2O_URL === env.H2O_URL), true);
     const packageJson = JSON.parse(await readFile(new URL('../package.json', import.meta.url), 'utf8'));
     assert.deepEqual(generated.map(([, entry]) => [entry.command, entry.args]), [

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

@@ -45,15 +45,15 @@ test('writes source-qualified CF entries beside same-named BAS destinations', ()
   assert.deepEqual(entries['cf:space-one:instance-one:shared'].env, {
     H2O_URL: 'http://h2o.example',
     SAP_ALLOW_TRANSPORTABLE_EDITS: 'true',
-    BAS_VSP_DESTINATION_SOURCE: 'cloud-foundry',
-    BAS_VSP_DESTINATION: 'shared',
+    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.BAS_VSP_DESTINATION, '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);
 
@@ -75,14 +75,14 @@ test('extracts key references only from package-managed Cloud Foundry entries',
     duplicate: managedNpx,
     unmanaged: { ...cloudFoundry, BAS_EXT: undefined },
     otherPackage: { ...cloudFoundry, command: 'other' },
-    otherSource: { ...cloudFoundry, env: { ...cloudFoundry.env, BAS_VSP_DESTINATION_SOURCE: 'bas' } }
+    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, BAS_VSP_DESTINATION_SOURCE: undefined } },
+    customServer: { command: 'custom-launcher', env: { ...cloudFoundry.env, SAP_AI_DEV_TOOLKIT_DESTINATION_SOURCE: undefined } },
     noEnvironment: { command: 'custom-launcher' }
   } };
   assert.deepEqual(collectCloudFoundryKeyReferencesFromAllEntries(userConfig), [
@@ -147,3 +147,47 @@ test('installs and removes CF entries without deleting unrelated MCP servers', a
   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.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' }
+  ]);
+});

package/test/mcp-proxy.test.mjs

@@ -30,7 +30,7 @@ async function fixtureProxy(logger = () => {}) {
   const directory = await mkdtemp(join(tmpdir(), 'bas-vsp-test-'));
   const log = join(directory, 'children.log');
   const destinations = ['alpha', 'beta'].map(name => ({ name, url: `http://${name}.dest`, client: '001' }));
-  const proxy = new MCPProxy({ binary: fixture, destinations, env: { ...process.env, BAS_VSP_DESTINATION: 'alpha', BAS_VSP_MODE: 'expert', FAKE_LOG: log, SAP_ALLOW_TRANSPORTABLE_EDITS: 'true', SAP_USER: 'must-not-pass', SAP_PASSWORD: 'must-not-pass', Authorization: 'Bearer must-not-pass', Cookie: 'secret', SAP_READ_ONLY: 'true' }, log: logger });
+  const proxy = new MCPProxy({ binary: fixture, destinations, env: { ...process.env, SAP_AI_DEV_TOOLKIT_DESTINATION: 'alpha', SAP_AI_DEV_TOOLKIT_MODE: 'expert', FAKE_LOG: log, SAP_ALLOW_TRANSPORTABLE_EDITS: 'true', SAP_USER: 'must-not-pass', SAP_PASSWORD: 'must-not-pass', Authorization: 'Bearer must-not-pass', Cookie: 'secret', SAP_READ_ONLY: 'true' }, log: logger });
   proxy.start();
   return { directory, log, proxy };
 }

package/test/pty.mjs

@@ -0,0 +1,8 @@
+import { spawn } from 'node:child_process';
+
+export function spawnWithPty(command, options) {
+  if (process.platform === 'darwin') {
+    return spawn('python3', ['-c', 'import os,pty,sys; status=pty.spawn(["/bin/sh","-c",sys.argv[1]]); code=os.waitstatus_to_exitcode(status); sys.exit(code if code >= 0 else 128-code)', command], options);
+  }
+  return spawn('script', ['-qec', command, '/dev/null'], options);
+}

package/test/setup-cf.test.mjs

@@ -1,12 +1,12 @@
 import test from 'node:test';
 import assert from 'node:assert/strict';
 import { chmod, mkdir, mkdtemp, readFile, rm, writeFile } from 'node:fs/promises';
-import { spawn } from 'node:child_process';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { installMcpConfig } from '../src/mcp-config.mjs';
 import { destinationTable } from '../scripts/postinstall.mjs';
+import { spawnWithPty } from './pty.mjs';
 
 const setupModule = fileURLToPath(new URL('../src/setup.mjs', import.meta.url));
 
@@ -73,7 +73,7 @@ await runSetup({
     HOME: directory,
     PATH: `${bin}:${process.env.PATH || ''}`,
     H2O_URL: 'http://h2o.example',
-    BAS_VSP_MCP_CONFIG: configPath,
+    SAP_AI_DEV_TOOLKIT_MCP_CONFIG: configPath,
     CF_CALL_LOG: callLog,
     CF_DISCOVERY_LOG: discoveryLog,
     TEST_CF_VERSION: version,
@@ -90,7 +90,7 @@ await runSetup({
 function runSetupInPty(fixture, { importAnswer = 'y\r', selection = ' \r' } = {}) {
   return new Promise((resolve, reject) => {
     const command = `${JSON.stringify(process.execPath)} ${JSON.stringify(fixture.harness)}`;
-    const child = spawn('script', ['-qec', command, '/dev/null'], { env: fixture.env, stdio: ['pipe', 'pipe', 'pipe'] });
+    const child = spawnWithPty(command, { env: fixture.env, stdio: ['pipe', 'pipe', 'pipe'] });
     let stdout = '';
     let stderr = '';
     let importSent = false;
@@ -323,7 +323,7 @@ test('preserves old CF keys referenced by an unrelated remaining MCP entry', asy
   const config = JSON.parse(await readFile(fixture.configPath, 'utf8'));
   config.servers.customServer = {
     command: 'custom-launcher',
-    env: { ...config.servers[previous.serverName].env, BAS_VSP_DESTINATION_SOURCE: undefined }
+    env: { ...config.servers[previous.serverName].env, SAP_AI_DEV_TOOLKIT_DESTINATION_SOURCE: undefined }
   };
   await writeFile(fixture.configPath, JSON.stringify(config));
   fixture.env.TEST_BAS_DESTINATIONS = JSON.stringify([{

package/test/setup.test.mjs

@@ -1,6 +1,6 @@
 import test from 'node:test';
 import assert from 'node:assert/strict';
-import { mkdir, mkdtemp, readFile, readdir, rm, stat, writeFile } from 'node:fs/promises';
+import { chmod, mkdir, mkdtemp, readFile, readdir, rm, stat, symlink, writeFile } from 'node:fs/promises';
 import { createServer } from 'node:http';
 import { spawn } from 'node:child_process';
 import { stripVTControlCharacters } from 'node:util';
@@ -10,6 +10,7 @@ import { join } from 'node:path';
 import { pathToFileURL } from 'node:url';
 import { installMcpConfig } from '../src/mcp-config.mjs';
 import { runSetup } from '../src/setup.mjs';
+import { spawnWithPty } from './pty.mjs';
 
 const destinations = [
   { name: 'alpha-system', client: '100', authentication: 'Basic', probe: { status: 'available', available: true } },
@@ -30,7 +31,7 @@ function outputStream() {
 function runSetupVisibilityInPty(fixture, env, keys = '\r') {
   return new Promise((resolve, reject) => {
     const command = `stty cols 48 rows 12; ${JSON.stringify(process.execPath)} ${JSON.stringify(fixture)}`;
-    const child = spawn('script', ['-qec', command, '/dev/null'], { env, stdio: ['pipe', 'pipe', 'pipe'] });
+    const child = spawnWithPty(command, { env, stdio: ['pipe', 'pipe', 'pipe'] });
     let stdout = '';
     let stderr = '';
     let selectionSent = false;
@@ -57,7 +58,7 @@ function runSetupVisibilityInPty(fixture, env, keys = '\r') {
 function runPostinstallInPty(env, keys, assetsAnswer = '\r') {
   return new Promise((resolve, reject) => {
     const command = `${JSON.stringify(process.execPath)} scripts/postinstall.mjs </dev/null | cat`;
-    const child = spawn('script', ['-qec', command, '/dev/null'], { env, stdio: ['pipe', 'pipe', 'pipe'] });
+    const child = spawnWithPty(command, { env, stdio: ['pipe', 'pipe', 'pipe'] });
     let stdout = '';
     let stderr = '';
     let selectionSent = false;
@@ -128,7 +129,7 @@ test('reconciles generated entries while preserving unrelated MCP config', async
     assert.deepEqual(Object.keys(config.servers).sort(), ['alpha-system', 'beta-system', 'unrelated']);
     assert.deepEqual(generated.map(([name]) => name), ['alpha-system', 'beta-system']);
     assert.equal(generated.length, 2);
-    assert.deepEqual(generated.map(([, entry]) => entry.env.BAS_VSP_DESTINATION), ['alpha-system', 'beta-system']);
+    assert.deepEqual(generated.map(([, entry]) => entry.env.SAP_AI_DEV_TOOLKIT_DESTINATION), ['alpha-system', 'beta-system']);
     assert.deepEqual(generated.map(([, entry]) => entry.env.H2O_URL), ['http://new-h2o', 'http://new-h2o']);
     assert.deepEqual(generated.map(([, entry]) => entry.command), ['sap-ai-dev', 'sap-ai-dev']);
     assert.deepEqual(generated.map(([, entry]) => entry.env.SAP_ALLOW_TRANSPORTABLE_EDITS), ['true', 'true']);
@@ -187,13 +188,13 @@ const destinations = [
 await runSetup({
   env: { ...process.env, H2O_URL: 'http://h2o.example', PATH: ${JSON.stringify(`${bin}:${process.env.PATH || ''}`)} },
   discover: async () => destinations,
-  install: async selected => ({ path: process.env.BAS_VSP_MCP_CONFIG, servers: Object.fromEntries(selected.map(destination => [destination.name, { env: { BAS_VSP_DESTINATION: destination.name } }])) })
+  install: async selected => ({ path: process.env.SAP_AI_DEV_TOOLKIT_MCP_CONFIG, servers: Object.fromEntries(selected.map(destination => [destination.name, { env: { SAP_AI_DEV_TOOLKIT_DESTINATION: destination.name } }])) })
 });
 `);
   try {
     const env = {
       ...process.env,
-      BAS_VSP_MCP_CONFIG: join(directory, 'mcp.json'),
+      SAP_AI_DEV_TOOLKIT_MCP_CONFIG: join(directory, 'mcp.json'),
       FORCE_COLOR: '1',
       PATH: `${bin}:${process.env.PATH || ''}`
     };
@@ -232,6 +233,11 @@ test('non-TTY setup skips without writing config', async () => {
 test('global postinstall completes BAS selection before optional Copilot assets', async t => {
   const directory = await mkdtemp(join(tmpdir(), 'bas-postinstall-wizard-'));
   const config = join(directory, 'mcp.json');
+  const bin = join(directory, 'bin');
+  await mkdir(bin);
+  const cfCli = join(bin, 'cf');
+  await writeFile(cfCli, '#!/bin/sh\nexit 1\n');
+  await chmod(cfCli, 0o755);
   const server = createServer((request, response) => {
     if (request.url === '/api/listDestinations') {
       response.writeHead(200, { 'content-type': 'application/json' });
@@ -249,9 +255,10 @@ test('global postinstall completes BAS selection before optional Copilot assets'
   const env = {
     ...process.env,
     HOME: directory,
+    PATH: `${bin}:${process.env.PATH || ''}`,
     BAS_VSP_BINARY: '/bin/true',
-    BAS_VSP_MCP_CONFIG: config,
-    BAS_VSP_SKIP_PROBE: 'true',
+    SAP_AI_DEV_TOOLKIT_MCP_CONFIG: config,
+    SAP_AI_DEV_TOOLKIT_SKIP_PROBE: 'true',
     H2O_URL: `http://127.0.0.1:${server.address().port}`,
     HTTP_PROXY: '',
     http_proxy: '',
@@ -296,11 +303,11 @@ test('global postinstall completes BAS selection before optional Copilot assets'
   assert.ok(acceptLogs.includes(`MCP config file: ${config}`), acceptLogs);
   assert.ok(acceptLogs.includes('Destination: BAS · alpha-system · client 100 · Basic'), acceptLogs);
   assert.ok(acceptLogs.includes('Launch: stdio · sap-ai-dev'), acceptLogs);
-  assert.ok(acceptLogs.includes('Environment keys: BAS_VSP_DESTINATION, H2O_URL, SAP_ALLOW_TRANSPORTABLE_EDITS'), acceptLogs);
+  assert.ok(acceptLogs.includes('Environment keys: H2O_URL, SAP_AI_DEV_TOOLKIT_DESTINATION, SAP_ALLOW_TRANSPORTABLE_EDITS'), acceptLogs);
   const configAfterAccept = JSON.parse(await readFile(config, 'utf8'));
   assert.deepEqual(Object.values(configAfterAccept.servers)
     .filter(entry => entry.BAS_EXT === 'true')
-    .map(entry => entry.env.BAS_VSP_DESTINATION), ['alpha-system']);
+    .map(entry => entry.env.SAP_AI_DEV_TOOLKIT_DESTINATION), ['alpha-system']);
   await assertUserCopilotAssets(directory);
 });
 
@@ -311,7 +318,7 @@ test('non-TTY postinstall skips setup and leaves MCP config untouched', async ()
   const env = {
     ...process.env,
     HOME: directory,
-    BAS_VSP_BINARY: '/bin/true',
+    SAP_AI_DEV_TOOLKIT_BINARY: '/bin/true',
     H2O_URL: 'http://bas.example'
   };
   delete env.npm_config_ignore_scripts;
@@ -319,7 +326,7 @@ test('non-TTY postinstall skips setup and leaves MCP config untouched', async ()
   try {
     for (const original of [undefined, JSON.stringify({ servers: { unrelated: { command: 'other' } } })]) {
       const config = join(directory, original === undefined ? 'new.json' : 'existing.json');
-      env.BAS_VSP_MCP_CONFIG = config;
+      env.SAP_AI_DEV_TOOLKIT_MCP_CONFIG = config;
       if (original !== undefined) await writeFile(config, original);
       const result = await new Promise((resolve, reject) => {
         const child = spawn(process.execPath, ['scripts/postinstall.mjs'], { env, stdio: ['ignore', 'pipe', 'pipe'] });
@@ -349,8 +356,8 @@ test('postinstall skips optional Copilot assets without an interactive terminal'
   const env = {
     ...process.env,
     HOME: join(directory, 'home'),
-    BAS_VSP_BINARY: '/bin/true',
-    BAS_VSP_MCP_CONFIG: config
+    SAP_AI_DEV_TOOLKIT_BINARY: '/bin/true',
+    SAP_AI_DEV_TOOLKIT_MCP_CONFIG: config
   };
   delete env.H2O_URL;
   delete env.npm_config_ignore_scripts;
@@ -374,3 +381,33 @@ test('postinstall skips optional Copilot assets without an interactive terminal'
     await rm(directory, { recursive: true, force: true });
   }
 });
+
+test('postinstall runs when invoked through a symlinked install path', async () => {
+  const directory = await mkdtemp(join(tmpdir(), 'sap-ai-postinstall-symlink-'));
+  const alias = join(directory, 'postinstall.mjs');
+  await symlink(join(process.cwd(), 'scripts', 'postinstall.mjs'), alias);
+  const env = {
+    ...process.env,
+    HOME: directory,
+    SAP_AI_DEV_TOOLKIT_BINARY: '/bin/true'
+  };
+  delete env.H2O_URL;
+  delete env.npm_config_ignore_scripts;
+  delete env.NPM_CONFIG_IGNORE_SCRIPTS;
+  try {
+    const result = await new Promise((resolve, reject) => {
+      const child = spawn(process.execPath, [alias], { env, stdio: ['ignore', 'pipe', 'pipe'] });
+      let stdout = '';
+      let stderr = '';
+      child.stdout.on('data', chunk => { stdout += chunk; });
+      child.stderr.on('data', chunk => { stderr += chunk; });
+      child.on('error', reject);
+      child.on('exit', (code, signal) => resolve({ code, signal, stdout, stderr }));
+    });
+    const logs = `${result.stdout}\n${result.stderr}`;
+    assert.equal(result.code, 0, logs);
+    assert.match(logs, /BAS destination setup was skipped because H2O_URL is not set/);
+  } finally {
+    await rm(directory, { recursive: true, force: true });
+  }
+});

package/tools.md

@@ -65,4 +65,15 @@ The proxy forwards these tools when the installed VSP mode registers them. They
 - `CreateTransport`
 - `ListDependencies`
 - `PublishServiceBinding`
-- `UnpublishServiceBinding`
+- `UnpublishServiceBinding`
+
+## Destination-scoped workflow tools
+
+The proxy adds local multi-step tools to each destination when their required VSP capabilities are available:
+
+- `PrepareABAPChangeSet` and `ApplyABAPChangeSet` — review full-source diffs and recheck source before explicit writes.
+- `CheckTransportReadiness` — collect a whitelisted bundle of transport, dependency, inactive-object, ABAP Unit, and ATC results.
+- `PlanABAPCloudMigration` — batch SAP API release-state checks and prioritize recognized unreleased results.
+- `GenerateRAPRegressionSuite` and `RunRAPRegressionSuite` — create and run reusable OData GET-only checks under the selected service root.
+
+`sap-ai-dev-toolkit --doctor` checks destination probing, VSP startup, `GetSystemInfo`, and MCP tool listing. `sap-ai-dev-toolkit --demo` runs the sample tools and OData fixtures without contacting SAP; demo writes and transport creation remain in memory until that process exits.