@brainwav/diagram 1.0.6 → 1.0.7

package/package.json

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

package/src/diagram.js

@@ -1469,7 +1469,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 +1520,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);
     }