sync

Author: superbasicstudio

JOURNAL.md

@@ -1,5 +1,23 @@
 # Engineering Journal
 
+## 2025-01-18 (v1.1.1)
+
+### Fixed Security Scan Prompt Logic When User Declines
+- **What**: Fixed issue where security scan instruction appeared in recommended next step even when user chose "N"
+- **Why**: User reported that declining security scan still showed security scan instructions in the prompt
+- **How**: Added conditional logic to check if user chose security checkup and only include security scan text when appropriate
+- **Issues**: None - clean implementation
+- **Result**: Recommended prompt now correctly excludes security scan instruction when user declines
+
+### Added Contributors Link to Thank You Message
+- **What**: Added link to THANKS.md at the end of initialization script
+- **Why**: Acknowledge contributors and make it easy for users to see who helped build the project
+- **How**: Updated animateThankYou() function to include contributor message with link to THANKS.md
+- **Issues**: None
+- **Result**: Users now see "Made with ❤️ by our amazing contributors" with link to recognition page
+
+---
+
 ## 2025-01-14 19:15
 
 ### Added Legal Disclaimer to README

bin/init.js

@@ -330,6 +330,7 @@ async function initializeFramework(targetDir, options) {
   console.log('');
 
   // Ask if they want to run a health check
+  let userChoseSecurityCheckup = false;
   if (!options.yes) {
     console.log('');
     console.log(chalk.cyan('┌─────────────────────────────────────────────────────────────────┐'));
@@ -341,6 +342,7 @@ async function initializeFramework(targetDir, options) {
     console.log('');
 
     const runCheckup = await promptConfirmation(chalk.yellow('Run security checkup? (y/N): '));
+    userChoseSecurityCheckup = runCheckup;
 
     if (runCheckup) {
       console.log('');
@@ -356,13 +358,21 @@ async function initializeFramework(targetDir, options) {
   console.log(chalk.yellow('│') + ' ask Claude Code:                                                ' + chalk.yellow('│'));
   console.log(chalk.yellow('│') + '                                                                 ' + chalk.yellow('│'));
   console.log(chalk.yellow('│') + chalk.cyan('  "Please review this codebase and update the CLAUDE.md') + '         ' + chalk.yellow('│'));
-  console.log(chalk.yellow('│') + chalk.cyan('   and CONDUCTOR.md files with the actual project details.') + '      ' + chalk.yellow('│'));
-  console.log(chalk.yellow('│') + chalk.cyan('   Also perform a security health check and list any') + '            ' + chalk.yellow('│'));
-  console.log(chalk.yellow('│') + chalk.cyan('   potential vulnerabilities or concerns (like exposed') + '          ' + chalk.yellow('│'));
-  console.log(chalk.yellow('│') + chalk.cyan('   .env files, API keys in code, missing .gitignore') + '            ' + chalk.yellow('│'));
-  console.log(chalk.yellow('│') + chalk.cyan('   entries, outdated dependencies with known vulnerabilities,') + '   ' + chalk.yellow('│'));
-  console.log(chalk.yellow('│') + chalk.cyan('   or insecure configurations) - just list them as warnings,') + '   ' + chalk.yellow('│'));
-  console.log(chalk.yellow('│') + chalk.cyan('   don\'t fix anything."') + '                                          ' + chalk.yellow('│'));
+  
+  if (userChoseSecurityCheckup && !options.yes) {
+    // User already ran security checkup, so don't include it in the prompt
+    console.log(chalk.yellow('│') + chalk.cyan('   and CONDUCTOR.md files with the actual project details."') + '     ' + chalk.yellow('│'));
+  } else {
+    // User didn't run security checkup OR used --yes flag, so include it in the prompt
+    console.log(chalk.yellow('│') + chalk.cyan('   and CONDUCTOR.md files with the actual project details.') + '      ' + chalk.yellow('│'));
+    console.log(chalk.yellow('│') + chalk.cyan('   Also perform a security health check and list any') + '            ' + chalk.yellow('│'));
+    console.log(chalk.yellow('│') + chalk.cyan('   potential vulnerabilities or concerns (like exposed') + '          ' + chalk.yellow('│'));
+    console.log(chalk.yellow('│') + chalk.cyan('   .env files, API keys in code, missing .gitignore') + '            ' + chalk.yellow('│'));
+    console.log(chalk.yellow('│') + chalk.cyan('   entries, outdated dependencies with known vulnerabilities,') + '   ' + chalk.yellow('│'));
+    console.log(chalk.yellow('│') + chalk.cyan('   or insecure configurations) - just list them as warnings,') + '   ' + chalk.yellow('│'));
+    console.log(chalk.yellow('│') + chalk.cyan('   don\'t fix anything."') + '                                          ' + chalk.yellow('│'));
+  }
+  
   console.log(chalk.yellow('│') + '                                                                 ' + chalk.yellow('│'));
   console.log(chalk.yellow('│') + ' This one-time setup ensures your docs match your project! 🚀   ' + chalk.yellow('│'));
   console.log(chalk.yellow('└─────────────────────────────────────────────────────────────────┘'));
@@ -408,6 +418,9 @@ async function animateThankYou() {
   console.log('');
   console.log(chalk.gray('    Happy documenting with Claude!'));
   console.log('');
+  console.log(chalk.gray('    Improved with contributions from our amazing contributors'));
+  console.log(chalk.gray('    See: ') + chalk.blue.underline('https://github.com/superbasicstudio/claude-conductor/blob/main/THANKS.md'));
+  console.log('');
 }
 
 async function copyTemplates(templatesDir, targetPath, force, full = false, newlyCreatedFiles) {