@sk-labs/copilot-kit 3.0.2 → 3.0.4

package/lib/commands.js

@@ -77,7 +77,8 @@ async function init(options) {
     const vscodeSettingsEntry = entries.find(e => e.entryName === `${rootFolder}/.vscode/settings.json`);
     
     if (vscodeSettingsEntry) {
-      const newSettings = JSON.parse(vscodeSettingsEntry.getData().toString('utf8'));
+      const newSettingsContent = vscodeSettingsEntry.getData().toString('utf8');
+      const newSettings = parseJsonWithComments(newSettingsContent);
       
       if (fs.existsSync(vscodeSettingsPath)) {
         // .vscode/settings.json already exists
@@ -99,10 +100,19 @@ async function init(options) {
           console.log(chalk.green('✓ Replaced .vscode/settings.json'));
         } else if (choice === 'm') {
           // Merge
-          const existingSettings = JSON.parse(fs.readFileSync(vscodeSettingsPath, 'utf8'));
-          const mergedSettings = mergeSettings(existingSettings, newSettings);
-          fs.writeFileSync(vscodeSettingsPath, JSON.stringify(mergedSettings, null, 2));
-          console.log(chalk.green('✓ Merged Copilot Kit settings into existing settings.json'));
+          try {
+            const existingContent = fs.readFileSync(vscodeSettingsPath, 'utf8');
+            const existingSettings = parseJsonWithComments(existingContent);
+            const mergedSettings = mergeSettings(existingSettings, newSettings);
+            fs.writeFileSync(vscodeSettingsPath, JSON.stringify(mergedSettings, null, 2));
+            console.log(chalk.green('✓ Merged Copilot Kit settings into existing settings.json'));
+          } catch (error) {
+            console.log(chalk.red('✗ Failed to merge settings:'), error.message);
+            console.log(chalk.yellow('  Creating backup and replacing...'));
+            fs.copyFileSync(vscodeSettingsPath, `${vscodeSettingsPath}.backup`);
+            fs.writeFileSync(vscodeSettingsPath, JSON.stringify(newSettings, null, 2));
+            console.log(chalk.green('✓ Replaced settings (backup saved)'));
+          }
         } else {
           // Skip
           console.log(chalk.yellow('⊘ Skipped .vscode/settings.json'));
@@ -128,6 +138,9 @@ async function init(options) {
       console.log(chalk.dim('  2. Test auto-detection: "Create a responsive card component"'));
       console.log(chalk.dim('  3. Docs: https://github.com/sk-labs/copilot-kit\n'));
     }
+    
+    // Ensure process exits cleanly
+    process.exit(0);
   } catch (error) {
     if (spinner) spinner.fail(chalk.red('Installation failed'));
     console.error(chalk.red('\n❌ Error:'), error.message);
@@ -182,6 +195,9 @@ async function update(options) {
     if (!options.quiet) {
       console.log(chalk.dim(`\n📦 Updated ${fileCount} files\n`));
     }
+    
+    // Ensure process exits cleanly
+    process.exit(0);
   } catch (error) {
     if (spinner) spinner.fail(chalk.red('Update failed'));
     console.error(chalk.red('\n❌ Error:'), error.message);
@@ -200,7 +216,7 @@ async function status(options) {
   if (!fs.existsSync(githubPath)) {
     console.log(chalk.red('❌ Not installed'));
     console.log(chalk.dim('   Run "copilot-kit init" to install\n'));
-    return;
+    process.exit(0);
   }
 
   console.log(chalk.green('✓ Installed'), chalk.dim(`at ${githubPath}`));
@@ -238,7 +254,8 @@ async function status(options) {
   
   if (fs.existsSync(vscodeSettingsPath)) {
     try {
-      const settings = JSON.parse(fs.readFileSync(vscodeSettingsPath, 'utf8'));
+      const settingsContent = fs.readFileSync(vscodeSettingsPath, 'utf8');
+      const settings = parseJsonWithComments(settingsContent);
       
       // Check critical setting
       const criticalSetting = settings['chat.customAgentInSubagent.enabled'];
@@ -265,6 +282,7 @@ async function status(options) {
       
     } catch (error) {
       console.log(chalk.yellow('⚠️  settings.json exists but could not be parsed'));
+      console.log(chalk.dim(`     Error: ${error.message}`));
     }
   } else {
     console.log(chalk.yellow('⚠️  .vscode/settings.json not found'));
@@ -272,6 +290,9 @@ async function status(options) {
   }
   
   console.log();
+  
+  // Ensure process exits cleanly
+  process.exit(0);
 }
 
 function downloadRepo(branch = 'main') {
@@ -327,6 +348,19 @@ function askQuestion(question) {
   });
 }
 
+function parseJsonWithComments(jsonString) {
+  // Remove single-line comments (// ...)
+  let cleaned = jsonString.replace(/\/\/.*$/gm, '');
+  
+  // Remove multi-line comments (/* ... */)
+  cleaned = cleaned.replace(/\/\*[\s\S]*?\*\//g, '');
+  
+  // Remove trailing commas before } or ]
+  cleaned = cleaned.replace(/,(\s*[}\]])/g, '$1');
+  
+  return JSON.parse(cleaned);
+}
+
 function mergeSettings(existing, newSettings) {
   // Merge settings, with Copilot Kit settings taking priority
   const merged = { ...existing };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sk-labs/copilot-kit",
-  "version": "3.0.2",
+  "version": "3.0.4",
   "description": "CLI tool to install Custom Agents, Skills & Prompt Workflows for GitHub Copilot",
   "main": "index.js",
   "bin": {