@chrisromp/copilot-bridge 0.6.0 → 0.6.1-dev.10

package/scripts/install-service.ts

@@ -21,6 +21,7 @@ import {
 import { execSync } from 'node:child_process';
 
 function main() {
+  const isCli = process.env.COPILOT_BRIDGE_CLI === '1';
   const osPlatform = detectPlatform();
   const bridgePath = process.cwd();
   const homePath = os.homedir();
@@ -35,7 +36,9 @@ function main() {
 
     const distPath = path.join(bridgePath, 'dist', 'index.js');
     if (!fs.existsSync(distPath)) {
-      fail('dist/index.js not found. Run "npm run build" first.');
+      fail(isCli
+        ? 'dist/index.js not found. Package may be corrupted — try reinstalling.'
+        : 'dist/index.js not found. Run "npm run build" first.');
       process.exit(1);
     }
 
@@ -67,10 +70,11 @@ function main() {
     info('Linux detected — installing systemd service (system-scoped).');
     dim('The service starts at boot and restarts on crash.\n');
 
-    // Check if dist/index.js exists
     const distPath = path.join(bridgePath, 'dist', 'index.js');
     if (!fs.existsSync(distPath)) {
-      fail('dist/index.js not found. Run "npm run build" first.');
+      fail(isCli
+        ? 'dist/index.js not found. Package may be corrupted — try reinstalling.'
+        : 'dist/index.js not found. Run "npm run build" first.');
       process.exit(1);
     }
 
@@ -115,7 +119,9 @@ function main() {
 
   } else {
     fail('Unsupported platform for automatic service install.');
-    info('Run the bridge manually: npm run dev (development) or npm start (production)');
+    info(isCli
+      ? 'Run the bridge manually: copilot-bridge start'
+      : 'Run the bridge manually: npm run dev (development) or npm start (production)');
     process.exit(1);
   }
 }

package/scripts/lib/config-gen.ts

@@ -33,7 +33,6 @@ export interface GeneratedConfig {
   platforms: {
     mattermost: {
       url: string;
-      botToken?: string;
       bots?: Record<string, { token: string; admin?: boolean; agent?: string }>;
     };
   };
@@ -62,10 +61,8 @@ export function buildConfig(opts: {
     channels: [],
   };
 
-  // Single bot → botToken; multiple → bots object
-  if (opts.bots.length === 1) {
-    config.platforms.mattermost.botToken = opts.bots[0].token;
-  } else if (opts.bots.length > 1) {
+  // Always use named bots object (clearer schema, supports admin flag and multi-bot)
+  if (opts.bots.length > 0) {
     config.platforms.mattermost.bots = {};
     for (const bot of opts.bots) {
       config.platforms.mattermost.bots[bot.name] = {
@@ -81,7 +78,7 @@ export function buildConfig(opts: {
       id: ch.id,
       ...(ch.name ? { name: ch.name } : {}),
       platform: ch.platform,
-      ...(opts.bots.length > 1 ? { bot: ch.bot } : {}),
+      bot: ch.bot,
       workingDirectory: ch.workingDirectory,
     });
   }