@brainwav/diagram 1.0.6 → 1.0.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brainwav/diagram",
-  "version": "1.0.6",
+  "version": "1.0.8",
   "description": "Generate architecture diagrams from codebases",
   "main": "src/diagram.js",
   "bin": {

package/src/diagram.js

@@ -11,6 +11,9 @@ const crypto = require('crypto');
 const zlib = require('zlib');
 const { getOpenCommand, getNpxCommandCandidates } = require('./utils/commands');
 
+// Read version from package.json
+const packageJson = require('../package.json');
+
 // Video generation (lazy loaded)
 let videoModule;
 function getVideoModule() {
@@ -1294,7 +1297,7 @@ function validateExistingPathInRoot(targetPath, rootPath, label = 'path') {
 program
   .name('diagram')
   .description('Generate architecture diagrams from code')
-  .version('1.0.0');
+  .version(packageJson.version);
 
 program
   .command('analyze [path]')
@@ -1469,7 +1472,10 @@ program
   .option('-d, --manifest-dir <dir>', 'Directory containing manifest.json', '.diagram')
   .option('-o, --output <file>', 'Write summary JSON to a file')
   .option('--require-types <list>', 'Require all listed diagram types, comma-separated')
-  .option('--fail-on-placeholder', 'Fail if any diagram was a placeholder')
+  .option(
+    '--fail-on-placeholder',
+    'Fail if any required diagram was a placeholder (or any placeholder if no required types are set)'
+  )
   .action(async (targetPath, options) => {
     const root = resolveRootPathOrExit(targetPath);
     const manifestDir = path.join(root, options.manifestDir || '.diagram');
@@ -1517,8 +1523,16 @@ program
       process.exit(2);
     }
 
-    if (options.failOnPlaceholder && summary.placeholders > 0) {
-      console.error(chalk.yellow(`⚠️  Manifest includes ${summary.placeholders} placeholder diagram(s)`));
+    const placeholderTypesToCheck = required.length > 0
+      ? summary.placeholderTypes.filter((type) => required.includes(type))
+      : summary.placeholderTypes;
+
+    if (options.failOnPlaceholder && placeholderTypesToCheck.length > 0) {
+      console.error(
+        chalk.yellow(
+          `⚠️  Manifest includes ${placeholderTypesToCheck.length} required placeholder diagram(s): ${placeholderTypesToCheck.join(', ')}`
+        )
+      );
       process.exit(2);
     }