From 3cc0fd3b224812561aafe081ccb8af442b68c8f3 Mon Sep 17 00:00:00 2001
From: Andrew Phillips <andrew.phillips2@canada.ca>
Date: Wed, 27 Aug 2025 22:53:51 -0300
Subject: [PATCH] fix: handle null values to disable outputs

Co-authored-by: aider (openai/andrew/openrouter/deepseek/deepseek-chat-v3.1) <aider@aider.chat>
---
 src/meta_plugin/mod.rs | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/meta_plugin/mod.rs b/src/meta_plugin/mod.rs
index db7296c..946f620 100644
--- a/src/meta_plugin/mod.rs
+++ b/src/meta_plugin/mod.rs
@@ -141,6 +141,12 @@ pub enum MetaPluginType {
 pub fn process_metadata_outputs(internal_name: &str, value: serde_yaml::Value, outputs: &std::collections::HashMap<String, serde_yaml::Value>) -> Option<MetaData> {
     // Check if this output is disabled
     if let Some(mapping) = outputs.get(internal_name) {
+        // Check for null to disable the output
+        if mapping.is_null() {
+            debug!("META: Skipping disabled output (null): {}", internal_name);
+            return None;
+        }
+        // Check for boolean false to disable the output
         if let Some(false_val) = mapping.as_bool() {
             if !false_val {
                 debug!("META: Skipping disabled output: {}", internal_name);