fix: flatten dist structure to fix GitHub Actions checksum generation

- Move report-template.html directly to dist/ instead of dist/templates/
- Update template loading paths to check new location
- Simplify checksums generation to only process files in dist root
- This avoids 'sha256sum: dist/templates: Is a directory' error

Author: mickmicksh

.github/workflows/npm-publish.yml

@@ -60,10 +60,14 @@ jobs:
           fi
           echo "Build output verified ✓"
           
-      - name: Generate build checksums
+      - name: Verify build checksums
         run: |
-          echo "Generating SHA256 checksums..."
-          sha256sum dist/* > dist/checksums.sha256
+          echo "Verifying checksums were generated during build..."
+          if [ ! -f dist/checksums.sha256 ]; then
+            echo "Error: dist/checksums.sha256 not found"
+            exit 1
+          fi
+          echo "Checksums file contents:"
           cat dist/checksums.sha256
           
       - name: Upload build artifacts

CHANGELOG.md

@@ -5,17 +5,7 @@ All notable changes to the vibe-log-cli project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [0.5.4] - 2025-09-05
-
-### Added
-- **install-auto-sync command**: New shortcut command for quick access to auto-sync configuration
-- **Documentation link**: Added link to auto-sync documentation in the menu
-
-### Improved
-- **Auto-sync menu text**: Simplified educational text for better clarity and readability
-- **Help text**: Updated to show the new install-auto-sync command
-
-## [0.6.0] - 2025-09-04
+## [0.6.0] - 2025-09-05
 
 ### Added
 - **Template-Based Report Generation**: Complete replacement of direct HTML generation with JSON→Template system
@@ -39,6 +29,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Report Generation Flow**: Claude provides data, template provides structure (separation of concerns)
 - **Error Messages**: More descriptive feedback during report generation process
 
+### Improved
+- **Report Generation Output**: Tool failures are now hidden during report generation
+  - "Tool failed" messages no longer shown for expected errors (file not found, etc.)
+  - Cleaner, less intimidating output during analysis
+  - Error messages still visible in debug mode (VIBELOG_DEBUG=1)
+  - Reduces user anxiety during the 4-5 minute generation process
+- **Auto-sync menu**: Simplified educational text for better clarity
+  - Added shortcut command `install-auto-sync` for quick access
+  - Added documentation link in the menu
+  - Updated help text to show the new command
+
 ## [0.5.3] - 2025-08-31
 
 ### Fixed

scripts/copy-templates.js

@@ -3,19 +3,13 @@
 const fs = require('fs');
 const path = require('path');
 
-// Ensure dist/templates directory exists
-const distTemplatesDir = path.join(__dirname, '..', 'dist', 'templates');
-if (!fs.existsSync(distTemplatesDir)) {
-  fs.mkdirSync(distTemplatesDir, { recursive: true });
-}
-
-// Copy template file
+// Copy template file directly to dist/ to avoid directory issues with sha256sum
 const srcTemplate = path.join(__dirname, '..', 'src', 'templates', 'report-template.html');
-const destTemplate = path.join(__dirname, '..', 'dist', 'templates', 'report-template.html');
+const destTemplate = path.join(__dirname, '..', 'dist', 'report-template.html');
 
 if (fs.existsSync(srcTemplate)) {
   fs.copyFileSync(srcTemplate, destTemplate);
-  console.log('✅ Template copied to dist/templates/');
+  console.log('✅ Template copied to dist/');
 } else {
   console.error('❌ Template file not found:', srcTemplate);
   process.exit(1);

scripts/generate-checksums.js

@@ -18,42 +18,23 @@ function generateChecksum(filePath) {
   });
 }
 
-/**
- * Get all files in a directory recursively
- */
-function getAllFiles(dirPath, arrayOfFiles = []) {
-  const files = fs.readdirSync(dirPath);
-  
-  files.forEach(file => {
-    const filePath = path.join(dirPath, file);
-    if (fs.statSync(filePath).isDirectory()) {
-      arrayOfFiles = getAllFiles(filePath, arrayOfFiles);
-    } else {
-      arrayOfFiles.push(filePath);
-    }
-  });
-  
-  return arrayOfFiles;
-}
-
 async function main() {
   try {
     const distPath = path.join(process.cwd(), 'dist');
     const checksums = [];
 
-    // Get all files in dist directory (including subdirectories)
-    const allFiles = getAllFiles(distPath);
-    
-    // Filter out checksums.sha256 if it exists and sort for consistent output
-    const filesToChecksum = allFiles
-      .filter(file => !file.endsWith('checksums.sha256'))
+    // Get all files in dist directory (only files, not directories)
+    const files = fs.readdirSync(distPath)
+      .filter(file => {
+        const filePath = path.join(distPath, file);
+        return fs.statSync(filePath).isFile() && file !== 'checksums.sha256';
+      })
       .sort();
 
-    for (const filePath of filesToChecksum) {
+    for (const file of files) {
+      const filePath = path.join(distPath, file);
       const checksum = await generateChecksum(filePath);
-      // Get relative path from dist directory for the checksum file
-      const relativePath = path.relative(distPath, filePath).replace(/\\/g, '/');
-      checksums.push(`${checksum}  ${relativePath}`);
+      checksums.push(`${checksum}  ${file}`);
     }
 
     // Write checksums file

src/lib/report-executor.ts

@@ -82,7 +82,11 @@ export async function executeClaudePrompt(
               const isError = content.is_error || false;
 
               if (isError) {
-                console.log(colors.dim(formatTimestamp(now)) + ' ❌ ' + colors.error('Tool failed'));
+                // Don't show tool failures - they're usually expected (file not found, etc.)
+                // Only show in debug mode
+                if (process.env.VIBELOG_DEBUG) {
+                  console.log(colors.dim(formatTimestamp(now)) + ' ❌ ' + colors.error('Tool failed'));
+                }
               } else {
                 console.log(colors.dim(formatTimestamp(now)) + ' ✓ ' + colors.success('Tool completed'));
 

src/lib/report-template-engine.ts

@@ -31,14 +31,16 @@ export class ReportTemplateEngine {
   async loadTemplate(): Promise<void> {
     // Look for template in multiple locations to support both development and NPX usage
     const possiblePaths = [
-      // NPX package - template is in dist/templates
+      // NPX package - template is now directly in dist/
+      join(__dirname, '..', 'report-template.html'),
+      // NPX package - old location for compatibility
       join(__dirname, '..', 'templates', 'report-template.html'),
       // Development - template is in src/templates
       join(process.cwd(), 'src', 'templates', 'report-template.html'),
       join(process.cwd(), 'vibe-log-cli', 'src', 'templates', 'report-template.html'),
       // Fallback paths
       join(__dirname, '..', '..', 'src', 'templates', 'report-template.html'),
-      join(__dirname, '..', '..', 'dist', 'templates', 'report-template.html'),
+      join(__dirname, '..', '..', 'dist', 'report-template.html'),
     ];
 
     let templatePath: string | null = null;